| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-22 | |||
| 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: block_device: Add DriverImageBlockDevice to block_device_info https://review.opendev.org/c/openstack/nova/+/826527 | |
| 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: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 | |
| 09:15:58 | bauzas | sean-k-mooney: well, indeed even if we have far-left and far-right for more than 60%... | |
| 09:17:02 | bauzas | actually, if we were not having a republic, then we would only have one party, I guess... | |
| 09:17:09 | sean-k-mooney | ya but fortunetly your electerit is able to put there difference aside when they know infiting will result in the extream end winning | |
| 09:18:47 | bauzas | sean-k-mooney: at least, for one month | |
| 09:19:09 | bauzas | sean-k-mooney: we will have a new election for parliament MPs in June | |
| 09:19:43 | bauzas | if the parliament changes, this would mean that we would have a new government | |
| 09:19:54 | bauzas | even if the president is the same | |
| 09:21:04 | sean-k-mooney | thats true well best of luck in june so | |
| 09:49:00 | Uggla | hello folks, little question, what is the difference between an instance in pause vs suspend ? | |
| 10:13:22 | vbel | good morning/evening, I'm looking into various ways to migrate VM disks from one storage to another (Ceph/LVM/NFS/...) and currently I see only two possibilities - 1) cinder retype of Available disks 2) cinder retype of In-use with VMs in Active state. I wonder why shut off VMs cannot be retyped | |
| 10:13:58 | gibi | Uggla: as a starting point: https://github.com/openstack/nova/blob/1ff89a09d45efadb592e6becab872ec1f7ac9894/nova/virt/driver.py#L779-L786 https://github.com/openstack/nova/blob/1ff89a09d45efadb592e6becab872ec1f7ac9894/nova/virt/driver.py#L812-L818 | |
| 10:16:00 | gibi | Uggla: I think in case of the libvirt virt driver pause is just stop the cpus but does not move stuff out from ram. In the other hand the suspend virt driver action will call libvirts managedSave capability https://libvirt.org/sources/virshcmdref/html/sect-managedsave.html | |
| 10:16:29 | gibi | Uggla: just to complicate thing the pause virt driver action calls libvirt's suspend call https://libvirt.org/docs/libvirt-appdev-guide-python/en-US/html/libvirt_application_development_guide_using_python-Guest_Domains-Lifecycle-Save.html | |
| 10:17:16 | sean-k-mooney | Uggla: tl;dr is susped is like hybernate. it suspends the vm to disk saving its ram to a file | |
| 10:17:27 | sean-k-mooney | Uggla: pause just stops the vcpu process | |
| 10:17:37 | sean-k-mooney | so the qemu isntastance is still running | |
| 10:17:55 | sean-k-mooney | but the cpus are not so all memory and devices are still attached to teh qemu process | |
| 10:18:18 | sean-k-mooney | Uggla: thinks of pause like your laptops sleep or "suspend to ram" in linux | |
| 10:19:19 | sean-k-mooney | Uggla: in general neither should be used in a cloud you shoud generally shelve the vm or just stop it | |
| 10:19:36 | sean-k-mooney | the usecase for using suspend and pause in a cloud exist but are very limitted | |
| 10:20:09 | vbel | commit b40d949b3137473227949b597b2a61da41752ee5 explicitly prohibits any swap volume of supspended/stopped/resized/soft-deleted | |
| 10:21:13 | Uggla | gibi, sean-k-mooney thanks | |
| 10:23:26 | sean-k-mooney | vbel: in some cases i coudl see it being a problem depenidn on if the connection mechanisuem changes | |
| 10:23:32 | sean-k-mooney | stopped i think shoudl be ok | |
| 10:23:50 | sean-k-mooney | im kind of suprised we allow live retype but not stopped | |
| 10:24:15 | sean-k-mooney | when you say resized do you mena resize_verify? | |
| 10:24:20 | sean-k-mooney | resized is nto a vm state | |
| 10:25:27 | sean-k-mooney | wehn we stop the vm i think the volume goes into reserved | |
| 10:25:37 | sean-k-mooney | which is likely why we cant retype then | |
| 10:25:46 | sean-k-mooney | same for suspend i suspect | |
| 12:34:47 | gibi | artom__: I'm +2 on https://review.opendev.org/c/openstack/nova/+/839091 now with a hint for an extra assert if you ever have to respin this | |
| 13:00:26 | elodilles | bauzas sean-k-mooney: just a heads up, victoria-em deadline is in 2 days & there's no new merged patches in stable/victoria (except the l-c job drop patch, but that is CI-only)... so I think we could just approve the transition patch as there is hardly any time to merge and release from stable/victoria | |
| 13:01:07 | bauzas | wfm | |
| 13:03:59 | gibi | bauzas: the neutron side of this bugfix now merged, could you look at the nova side? https://review.opendev.org/c/openstack/nova/+/829248 sean-k-mooney is already +2 on it | |
| 13:04:25 | bauzas | ouch. | |
| 13:04:37 | bauzas | gibi: going first to the coffee machine then :p | |
| 13:04:52 | gibi | make a strong one :) | |
| 13:05:16 | elodilles | bauzas: thanks, then please review the victoria-em patch o:) this one >>> https://review.opendev.org/c/openstack/releases/+/837953 | |
| 13:05:58 | kashyap | bauzas: gibi: I'm trying decaf (I know, I know) in the afternoons | |
| 13:06:14 | kashyap | Some days it works, other days not so. | |
| 13:06:28 | gibi | I just finished my cold brew so no more coffein to me today | |
| 13:06:29 | bauzas | kashyap: surely it can work for some people, but not me :) | |
| 13:06:53 | kashyap | gibi: I should learn to make cold brews, the for the reminder | |
| 13:06:57 | bauzas | I mean, I can pee for sure if I try decaf | |