| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-22 | |||
| 07:36:18 | bauzas | gibi : ^ | |
| 07:41:09 | gibi | thanks, +A | |
| 07:50:27 | bauzas | gibi: I'm still confused by the urgency | |
| 07:50:31 | bauzas | gibi: https://github.com/openstack/nova/blame/master/requirements.txt#L50 | |
| 07:50:45 | bauzas | gibi: we use os-brick 5.2 for 4 months now | |
| 07:52:23 | bauzas | gibi: and that's only then we upgraded from 4.3.1 https://github.com/openstack/nova/commit/b5e2128f3847d444a808a2b0f89e6f1e4ffb77fc | |
| 08:06:10 | gibi | bauzas: there is a common confusion why it is not happening all the time for us upstream | |
| 08:06:21 | gibi | but there was reports back from january that it happens locally | |
| 08:06:40 | gibi | and I think now it happens downstream 100% | |
| 08:06:47 | gibi | hence the urgency :) | |
| 08:07:16 | gibi | honestly when it was first reported I was not able to reproduce it locally on my previous laptop | |
| 08:07:23 | gibi | but with my new laptop I saw it once | |
| 08:07:35 | gibi | so I guess it depends on some version caches | |
| 08:13:28 | bauzas | gibi: ack thanks | |
| 08:14:34 | kashyap | gibi: Getting out of the venv and recreating the 'tox' env resolved it. (I thought I tried this last night) | |
| 08:14:58 | gibi | kashyap: great, then mistery resolved :) | |
| 08:15:03 | kashyap | Yep | |
| 08:57:55 | opendevreview | Merged openstack/nova master: Update python testing as per zed cycle teting runtime https://review.opendev.org/c/openstack/nova/+/838943 | |
| 08:58:00 | opendevreview | Merged openstack/placement master: Make perfload jobs fail if write allocation fails https://review.opendev.org/c/openstack/placement/+/828438 | |
| 08:58:07 | opendevreview | Merged openstack/nova master: enable locking test fixture https://review.opendev.org/c/openstack/nova/+/838942 | |
| 09:21:31 | stephenfin | bauzas: Can you look at https://review.opendev.org/c/openstack/nova/+/837161/ at some point. Last patch left in that series that still needs a +W | |
| 09:22:03 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Add workaround to skip compareCPU() check on the destination https://review.opendev.org/c/openstack/nova/+/838926 | |
| 09:22:34 | bauzas | stephenfin: on a chat with rajesh sorry | |
| 09:22:43 | opendevreview | Stephen Finucane proposed openstack/nova master: Add Python 3.10 functional jobs https://review.opendev.org/c/openstack/nova/+/839029 | |
| 09:25:58 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Close connection on early return https://review.opendev.org/c/openstack/nova/+/839030 | |
| 09:26:04 | stephenfin | gibi: ^ per your feedback | |
| 09:26:42 | stephenfin | bauzas: all good, absolutely any time today would be great | |
| 09:50:49 | gibi | stephenfin: thanks. looking | |
| 09:57:38 | opendevreview | Jorhson Deng proposed openstack/nova master: Reset the instance's task_state to None when the status is error https://review.opendev.org/c/openstack/nova/+/838991 | |
| 10:49:37 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838033 | |
| 13:04:10 | sean-k-mooney | gibi: bauzas im +2 on https://review.opendev.org/c/openstack/nova-specs/+/819510/7 by the way | |
| 13:08:19 | bauzas | sean-k-mooney: ack, I'll try to look at it | |
| 13:18:11 | opendevreview | Elod Illes proposed openstack/nova stable/train: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838037 | |
| 13:26:00 | sean-k-mooney | bauzas: refresh my memory am i allowd to singel core approve a reporpsoed previously apporved spec | |
| 13:26:31 | bauzas | sean-k-mooney: for fast approvals, it depends whether it added or changed some stuff | |
| 13:26:34 | sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications does not say explcitly othen it follwo the normal process which woudl imply no | |
| 13:26:55 | sean-k-mooney | bauzas: im just reviewing melwitt reposoed version fo the encyuption specs | |
| 13:27:06 | sean-k-mooney | i think tehy are idential which is wht im checking now | |
| 13:27:24 | sean-k-mooney | but wanted to knwo if i should +2w them or just +2 once i complete reviewing it | |
| 13:31:43 | sean-k-mooney | bauzas: never mind the primary assine need to be updated | |
| 13:33:03 | opendevreview | Merged openstack/nova-specs master: Make 'tox -e fast-specs' fail on sphinx warnings https://review.opendev.org/c/openstack/nova-specs/+/835930 | |
| 13:55:20 | gibi | sean-k-mooney: I've started reading the fresh OWNER_NOVA spec but got distracted with bug triage, I will get back to it before end of today | |
| 14:00:39 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | |
| 14:01:27 | sean-k-mooney | gibi: no rush. i would have prefered the upgade impact workign in my comment as i incled that plamcent must be upgraded first but honestly i think we can jsut adress that in the code review of the release note adn docs | |
| 14:09:09 | opendevreview | Merged openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| 14:15:10 | bauzas | gibi: I gave my approval | |
| 14:18:39 | gibi | bauzas: thanks, one less thing for me to do before the weekend :) | |
| 15:10:33 | gibi | bauzas: have nice one | |
| 15:10:44 | bauzas | thanks | |
| 15:11:58 | gibi | I don't need the details, please :D | |
| 16:08:41 | opendevreview | Elod Illes proposed openstack/nova stable/stein: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838038 | |
| 16:29:56 | opendevreview | Merged openstack/nova master: VMware: Early fail spawn if memory is not multiple of 4. https://review.opendev.org/c/openstack/nova/+/835739 | |
| 16:32:04 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | |
| 16:36:08 | opendevreview | Elod Illes proposed openstack/nova stable/rocky: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838041 | |
| 17:02:31 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | |
| 17:23:17 | opendevreview | Merged openstack/nova master: db: Remove inplicit coercion of SELECTs https://review.opendev.org/c/openstack/nova/+/837159 | |
| 17:29:06 | opendevreview | Merged openstack/nova master: db: Replace use of Column.copy() method https://review.opendev.org/c/openstack/nova/+/837160 | |
| 17:29:14 | opendevreview | Merged openstack/nova master: db: Don't rely on autocommit behavior https://review.opendev.org/c/openstack/nova/+/837161 | |
| 18:28:25 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | |
| 18:32:57 | sean-k-mooney | artom_: does ^ actully repoduce it | |
| 18:41:11 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Run against performance.json patch https://review.opendev.org/c/openstack/nova/+/838934 | |
| 20:25:10 | artom_ | sean-k-mooney, yeah, and because I frogot the FIXME assertRaises, it quite literally blows up in the same way as "real life" | |
| 21:09:09 | opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | |
| 21:44:30 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | |
| 22:20:04 | opendevreview | Jakub Libosvar proposed openstack/os-vif master: Check for hybrid plugging in OVS https://review.opendev.org/c/openstack/os-vif/+/839102 | |
| 22:24:01 | opendevreview | melanie witt proposed openstack/nova master: scheduler: Add an ephemeral encryption pre filter https://review.opendev.org/c/openstack/nova/+/760456 | |
| 22:24:01 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add DriverImageBlockDevice to block_device_info https://review.opendev.org/c/openstack/nova/+/826527 | |
| 22:24:02 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to image and ephemeral disks https://review.opendev.org/c/openstack/nova/+/826528 | |
| #openstack-nova - 2022-04-23 | |||
| 00:55:38 | opendevreview | melanie witt proposed openstack/nova master: virt: Add block_device_info helper to find encrypted disks https://review.opendev.org/c/openstack/nova/+/826529 | |
| 00:55:39 | opendevreview | melanie witt proposed openstack/nova master: blockinfo: Add encryption details to the disk_info mappings when provided https://review.opendev.org/c/openstack/nova/+/772272 | |
| 02:50:15 | opendevreview | Merged openstack/nova master: db: Close connection on early return https://review.opendev.org/c/openstack/nova/+/839030 | |
| 13:00:19 | opendevreview | kiran pawar proposed openstack/nova master: VMware: Split out VMwareAPISession https://review.opendev.org/c/openstack/nova/+/832156 | |
| 13:00:20 | opendevreview | kiran pawar proposed openstack/nova master: VMware: StableMoRefProxy for moref recovery https://review.opendev.org/c/openstack/nova/+/832164 | |
| 13:21:29 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/839091 | |
| 16:15:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [api] use_forwarded_for https://review.opendev.org/c/openstack/nova/+/836253 | |
| #openstack-nova - 2022-04-24 | |||
| 08:11:22 | opendevreview | Wenping Song proposed openstack/os-traits master: Add nova and cyborg owner traits for resources https://review.opendev.org/c/openstack/os-traits/+/839128 | |
| 11:31:11 | opendevreview | Merged openstack/python-novaclient master: Update python classifier as per testing runtime https://review.opendev.org/c/openstack/python-novaclient/+/838944 | |
| #openstack-nova - 2022-04-25 | |||
| 06:30:46 | opendevreview | ZhouYanbing proposed openstack/nova master: correct the wrong content in the notes https://review.opendev.org/c/openstack/nova/+/838205 | |
| 07:34:18 | bauzas | good morning Nova | |
| 07:34:26 | gibi | bauzas: o/ | |
| 07:38:35 | elodilles | o/ | |
| 07:55:15 | bauzas | gibi: damn, we forgot to ask to add the bp link for https://specs.openstack.org/openstack/nova-specs/specs/zed/approved/owner-nova-trait-usage.html | |
| 07:56:40 | gibi | just push a fup and I can approve it | |
| 07:57:07 | bauzas | https://blueprints.launchpad.net/nova/+specs?searchtext=trait | |
| 07:57:14 | bauzas | can't see any | |
| 07:57:27 | bauzas | maybe wenping forgot to create one | |
| 07:58:20 | bauzas | https://launchpad.net/~wenping1/+specs?role=assignee | |
| 07:58:34 | bauzas | none of them are for the spec | |
| 08:01:25 | bauzas | ok, then I'll create a bp and i'll assign it to wenping | |
| 08:01:54 | gibi | everybody loves paperwork ;) | |
| 08:07:14 | bauzas | gibi: should I say I'm French ? | |
| 08:07:29 | gibi | no need :) | |
| 08:10:49 | bauzas | ok, https://blueprints.launchpad.net/nova/+spec/owner-nova-trait-usage is created | |
| 08:16:40 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Add the BP link to owner-nova-trait-usage spec https://review.opendev.org/c/openstack/nova-specs/+/839159 | |
| 08:16:47 | bauzas | gibi: sean-k-mooney: ^ | |
| 08:19:14 | gibi | bauzas: +A | |
| 08:27:54 | bauzas | thanks | |
| 08:43:36 | opendevreview | Merged openstack/nova-specs master: Add the BP link to owner-nova-trait-usage spec https://review.opendev.org/c/openstack/nova-specs/+/839159 | |
| 09:10:26 | sean-k-mooney1 | ah already merged | |
| 09:11:45 | sean-k-mooney | bauzas: congratulations on still having a republic by the way | |