| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-01 | |||
| 10:58:54 | openstackgerrit | Merged openstack/os-traits master: Remove translation sections from setup.cfg https://review.opendev.org/c/openstack/os-traits/+/728160 | |
| 11:04:04 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Drop use of deprecated collections classes https://review.opendev.org/c/openstack/os-vif/+/773360 | |
| 11:07:41 | hemanth_n | lyarwood: bug raised, for your reference, https://bugs.launchpad.net/neutron/+bug/1914037 | |
| 11:07:42 | openstack | Launchpad bug 1914037 in neutron "scenario tests tempest.scenario.test_network_v6.TestGettingAddress fails" [Undecided,New] | |
| 12:35:00 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/766976 | |
| 12:35:02 | openstack | bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Undecided,New] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood) | |
| 12:35:02 | openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/768472 | |
| 12:35:02 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380 | |
| 12:40:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380 | |
| 12:40:41 | lyarwood | stephenfin: https://review.opendev.org/c/openstack/nova/+/772869 - would you mind hitting this today, trivial fix needed downstream | |
| 12:40:50 | stephenfin | sure | |
| 12:41:20 | lyarwood | stephenfin: thanks | |
| 12:46:07 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/772869 - would you mind taking a look at this also, hopefully trivial but let me know if you have any issues with it. | |
| 12:48:47 | lyarwood | https://review.opendev.org/q/topic:nova-lvm - not urgent but if any cores have time, I'm trying to get nova-lvm voting again here | |
| 12:56:29 | sean-k-mooney | stephenfin: https://review.opendev.org/c/openstack/os-vif/+/773360 passed everthing but lower constraits | |
| 12:56:43 | sean-k-mooney | stephenfin: there is a patch to drop lc but i would prefer to fix it if we can | |
| 12:56:54 | sean-k-mooney | stephenfin: in this case it seams like its just hacking | |
| 12:57:15 | sean-k-mooney | stephenfin: do you have time to address that in a seperate patch if not ill take a crack at it later this week | |
| 12:57:23 | stephenfin | I can take a look | |
| 12:58:10 | sean-k-mooney | i think the decision was to only list direct deps in lc right | |
| 12:58:21 | sean-k-mooney | i was only partly following the ml thread | |
| 13:08:27 | gibi | elod: ack, on it | |
| 13:59:38 | gibi | lyarwood: I'm not familiar with rdb just wondering if user name is enough, is there some kind of password / key that also needs to be provided to access the encrypted disk? https://review.opendev.org/c/openstack/nova/+/772869 | |
| 14:07:39 | lyarwood | gibi: https://docs.ceph.com/en/latest/rbd/qemu-rbd/#usage - I can repsin with this link that explains this a little but tl;dr the username refers to the a local file (the keyring) used to authenticate the request | |
| 14:07:51 | lyarwood | to a* | |
| 14:08:14 | gibi | lyarwood: no need to respin, thanks | |
| 14:34:33 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Move the LibvirtDistroSupportMatrix wiki page into our docs https://review.opendev.org/c/openstack/nova/+/771981 | |
| 14:42:31 | dansmith | gibi: if you could +W this, it's blocking glance's ability to land the policy goal patches: https://review.opendev.org/c/openstack/nova/+/773192 | |
| 14:42:41 | dansmith | since we define the job, we're holding them hostage :) | |
| 14:42:56 | dansmith | (or anyone else ^) | |
| 14:43:27 | sean-k-mooney | ya that makes sense | |
| 14:44:21 | gibi | dansmith: looking | |
| 14:44:27 | sean-k-mooney | dansmith: i assume we are not doing that policy file upstae in [post-config| /etc/glance/policy.json] | |
| 14:44:29 | sean-k-mooney | for some reason | |
| 14:44:58 | sean-k-mooney | the default policy file shoudl be empty | |
| 14:45:29 | sean-k-mooney | also i guess at somepoint that need to be a policy.yaml file | |
| 14:46:04 | lyarwood | would post-config bounce the service? | |
| 14:46:17 | sean-k-mooney | no but it runs before its started | |
| 14:46:22 | sean-k-mooney | so it does not need too | |
| 14:46:43 | sean-k-mooney | post-config is after all the files are generated but before the services are started for the first time | |
| 14:47:03 | sean-k-mooney | excluddign rabbit and mariadb which start before post-config for reasons | |
| 14:47:04 | lyarwood | ah understood | |
| 14:47:32 | sean-k-mooney | if that was doing something more complicated post-cofnig would not work | |
| 14:47:52 | sean-k-mooney | but for this i think it would and they could then override it by inheriting form the job if they needed too | |
| 14:48:13 | sean-k-mooney | as a quick fix however that will work hence +1 | |
| 14:50:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/766976 | |
| 14:50:15 | openstack | bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Undecided,New] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood) | |
| 14:50:15 | openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/768472 | |
| 14:50:15 | openstackgerrit | Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380 | |
| 14:52:16 | gibi | dansmith: done | |
| 15:03:32 | openstackgerrit | sean mooney proposed openstack/nova master: [FUP] this change removes an unused paramater https://review.opendev.org/c/openstack/nova/+/773409 | |
| 15:03:40 | dansmith | gibi: thanks | |
| 15:03:59 | sean-k-mooney | FYI that ^ is really trivial if we can merge that quickly | |
| 15:04:51 | sean-k-mooney | lyarwood: i asked you about squash merging it into the backport of the original patch. are you still ok with that | |
| 15:05:10 | sean-k-mooney | lyarwood: if so ill repin those now | |
| 15:08:30 | lyarwood | sean-k-mooney: yup that's fine, lets land this first and then respin the backports | |
| 15:08:53 | sean-k-mooney | ok ill hold off then untill its merged on master | |
| 15:13:49 | sean-k-mooney | dansmith: are you still finding gerrit slow. currently im having the same multi second post behavior you were having when submitting comments | |
| 15:14:03 | dansmith | sean-k-mooney: yeah | |
| 15:14:15 | dansmith | sean-k-mooney: apparently that's just "how it is" with the new version of gerrit | |
| 15:14:19 | sean-k-mooney | ok so its not on my end so. | |
| 15:14:21 | sean-k-mooney | ack | |
| 15:14:27 | dansmith | suspected because they now use git as their db (!) | |
| 15:14:55 | sean-k-mooney | i wonder if that menas its more disk io sensitive then it used to be | |
| 15:15:30 | sean-k-mooney | im sure storing the data in git has proably resulted in more updates for that | |
| 15:15:33 | dansmith | it is, a lot | |
| 15:16:27 | sean-k-mooney | we need to get intel,samsung or micron to donate some fast ssds to infra :) | |
| 15:56:41 | openstackgerrit | Merged openstack/nova master: libvirt: Use specific user when probing encrypted rbd disks during extend https://review.opendev.org/c/openstack/nova/+/772869 | |
| 16:00:45 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Use specific user when probing encrypted rbd disks during extend https://review.opendev.org/c/openstack/nova/+/773443 | |
| 16:01:07 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Use specific user when probing encrypted rbd disks during extend https://review.opendev.org/c/openstack/nova/+/773444 | |
| 16:02:01 | supamatt | sean-k-mooney: I'd talk to Koxia and see if they want to sponsor our lab with storage ssd's lol | |
| 16:11:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Warn when starting services with older than N-1 computes https://review.opendev.org/c/openstack/nova/+/770764 | |
| 16:11:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Add upgrade check about old computes https://review.opendev.org/c/openstack/nova/+/770765 | |
| 16:26:27 | ganso | Hi Nova folks! Quick question: is there any way to migrate a VM that is in an affinity group without using the deprecated --live option? I have 2 VMs in the same host, they're in the same affinity group, and trying to move either (without --live) results in "No valid host". My main concern is, when that option is removed will there be no way to migrate those VMs? | |
| 16:28:14 | supamatt | ganso: try with --os-compute-api-version 2.11 | |
| 16:30:32 | ganso | supamatt: I'm actually already using --os-compute-api-version 2.30 to use the --host parameter. I still get "No valid host" | |
| 16:31:30 | supamatt | okay so try 2.11 | |
| 16:32:40 | ganso | supamatt: same error | |
| 16:33:05 | supamatt | you may want to check the nova scheduler logs and see whats going on | |
| 16:33:49 | ganso | supamatt: already did, "Filter ServerGroupAffinityFilter returned 0 hosts" | |
| 16:34:14 | ganso | which brings me back to my first question | |
| 17:59:36 | sean-k-mooney | ganso: you would need to force the migration | |
| 18:00:06 | sean-k-mooney | ganso: i dont think there is any supported way beyond that | |
| 18:00:06 | ganso | sean-k-mooney: with the deprecated option --live <host> ? | |
| 18:00:16 | sean-k-mooney | ganso: hat release of nova are you deploying | |
| 18:00:21 | sean-k-mooney | *what | |
| 18:00:36 | ganso | sean-k-mooney: stein | |
| 18:00:45 | sean-k-mooney | im pretty sure you would need to disable the compute node late affinity check | |
| 18:00:57 | sean-k-mooney | and i think you need to pass a --force option | |
| 18:01:39 | ganso | sean-k-mooney: it works with --live <host>. But my concern is that this option is deprecated. Has it been removed from a later release or planned to be removed? | |
| 18:01:40 | sean-k-mooney | ganso: so you would have to set this to true https://docs.openstack.org/nova/latest/configuration/config.html#workarounds.disable_group_policy_check_upcall | |
| 18:02:26 | sean-k-mooney | ganso: it has been removed in a later microversion yes | |
| 18:02:30 | sean-k-mooney | its still in the api | |
| 18:02:37 | sean-k-mooney | but not with the latest version | |
| 18:03:21 | sean-k-mooney | ganso: basically there no fully supported way to move vms in a server affinity group | |
| 18:03:35 | sean-k-mooney | we cant do a group migration for example | |
| 18:04:09 | sean-k-mooney | the best you could do without forcing the migration would likely be to shelve all vms in the server group | |
| 18:04:26 | sean-k-mooney | then unsleve them | |
| 18:04:54 | sean-k-mooney | thats a specualtion and not something i have tested by the way | |
| 18:05:11 | sean-k-mooney | i guess if you shelved all but 1 of the vms you could live migrate it too and then unshelve the rest | |