| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-22 | |||
| 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 | |
| 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 | |