| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-11 | |||
| 16:26:36 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Avoid unbound instance_uuid var during delete https://review.opendev.org/c/openstack/nova/+/828839 | |
| 16:52:29 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 16:52:29 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 18:49:48 | melwitt | elodilles: done | |
| 19:49:51 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 19:49:51 | opendevreview | Jonathan Race proposed openstack/nova master: object/notification for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828369 | |
| 19:49:52 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| #openstack-nova - 2022-02-12 | |||
| 17:11:12 | opendevreview | Merged openstack/nova stable/train: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/817830 | |
| #openstack-nova - 2022-02-13 | |||
| 22:25:08 | opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/victoria: libvirt: disable secure boot on non-q35 or with os secure_boot options https://review.opendev.org/c/openstack/nova/+/828978 | |
| 22:27:19 | opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/victoria: libvirt: disable secure boot on non-q35 or with os secure_boot options https://review.opendev.org/c/openstack/nova/+/828979 | |
| 22:28:24 | opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/ussuri: libvirt: disable secure boot on non-q35 or with os secure_boot options https://review.opendev.org/c/openstack/nova/+/828980 | |
| #openstack-nova - 2022-02-14 | |||
| 05:07:06 | opendevreview | Ghanshyam proposed openstack/nova master: Modify remaining APIs as per RBAC new guidelines https://review.opendev.org/c/openstack/nova/+/828994 | |
| 08:36:54 | kashyap | gibi: Morning; reading back. Don't worry, you've done a lot more already! Thank _you_ for the quick trials and tests | |
| 09:43:11 | gibi | kashyap: hi! | |
| 09:45:36 | gibi | good luck :) | |
| 10:25:09 | zigo | Would anyone have any idea what's going on with https://bugs.debian.org/1005632 ? | |
| 10:56:26 | gibi | zigo: looking... | |
| 10:58:29 | zigo | gibi: It's likely caused by an update of some Python dependencies in Unstable, but I can't figure it out ... :/ | |
| 11:04:09 | gibi | zigo: this seems relevant https://github.com/openstack/python-novaclient/blob/90525a1f5f55e206554da373dda9f26735e7c67f/novaclient/tests/unit/test_shell.py#L586-L599 | |
| 11:04:21 | gibi | so I guess it is prettytable | |
| 11:04:56 | gibi | from the debian build log: prettytable==0.0.0 | |
| 11:05:00 | gibi | that seems worng :) | |
| 11:05:28 | gibi | but also it says python3-prettytable all 2.5.0-1 | |
| 11:12:29 | gibi | it can be that the test code wrongly detects the version of prettytable | |
| 11:15:01 | gibi | dansmith: hi! I've pulled you into a review as I have some ovo / grenade questions https://review.opendev.org/c/openstack/nova/+/828369/7#message-fc72a0dda5b368c23a91f8a6b0244e10b8d511a5 | |
| 11:25:04 | zigo | gibi: Let me check what's the result of installing prettytable in Sid and see if the egginfo is wrong. | |
| 11:25:13 | gibi | zigo: ok | |
| 11:26:38 | zigo | Installed version really says 0.0.0 ... :/ | |
| 11:27:10 | zigo | So probably a packaging issue (I'm not the maintainer of that one ...). | |
| 11:27:41 | stephenfin | zigo: whenever you see 0.0.0, that usually means pbr hasn't been able to extract the version info from either git or sdist metadata | |
| 11:28:10 | stephenfin | I suspect whoever is building that package _might_ be using plain old tarballs or a shallow clone as opposed to an sdist or full clone | |
| 11:28:12 | bauzas | gmann: when you're up, I'm looking at https://review.opendev.org/c/openstack/nova/+/764292/34 | |
| 11:28:29 | zigo | ) | |
| 11:28:29 | zigo | use_scm_version={"local_scheme": local_scheme}, | |
| 11:28:29 | zigo | setup( | |
| 11:28:40 | zigo | So it's using scm, but the packaging hasn't set it, so it's wrong ... | |
| 11:28:42 | zigo | I'll try to fix it. | |
| 11:29:01 | zigo | gibi: I'll let you know the result when I'm done. | |
| 11:29:09 | gibi | zigo: cool thanks | |
| 11:29:47 | stephenfin | ah, not pbr but a similar issue. cool :) | |
| 12:37:59 | opendevreview | Merged openstack/nova master: Gracefull recovery when attaching volume fails https://review.opendev.org/c/openstack/nova/+/828516 | |
| 12:42:43 | opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/victoria: [stable-only] libvirt: disable secure boot on non-q35 or with os secure_boot options https://review.opendev.org/c/openstack/nova/+/828979 | |
| 12:45:26 | opendevreview | Mauricio Faria de Oliveira proposed openstack/nova stable/ussuri: [stable-only] libvirt: disable secure boot on non-q35 or with os secure_boot options https://review.opendev.org/c/openstack/nova/+/828980 | |
| 12:59:23 | zigo | gibi: The problem was indeed in the pyparsing package in Debian, fixed this way: https://salsa.debian.org/python-team/packages/prettytable/-/merge_requests/2/diffs | |
| 12:59:34 | zigo | s/pyparsing/prettytable/ | |
| 12:59:53 | gibi | zigo: good | |
| 13:00:04 | gibi | then it was easy | |
| 13:00:06 | gibi | :) | |
| 13:00:32 | zigo | Yeah, took me some times to realize how the current maintainer broke things though ... :P | |
| 13:01:29 | gibi | I don't see that part. :) Fortunately the test had a nice comment pointing at the pretty table version | |
| 13:07:36 | gibi | lyarwood: hi! do you know somebody planning to take ower tempest https://review.opendev.org/q/topic:wait_until_sshable_pingable series? | |
| 13:44:08 | bauzas | gibi: lyarwood stopped to work on Nova | |
| 13:49:22 | gibi | bauzas: yeah I know, hence my question | |
| 13:49:33 | gibi | as that series was pushed by lyarwood before | |
| 13:49:38 | gibi | and we still need it | |
| 13:50:47 | gibi | and I so that afazekas did a rebase on https://review.opendev.org/c/openstack/tempest/+/817635/ so wondering if he will take it over | |
| 13:50:57 | gibi | s/so/saw/ | |
| 14:08:32 | tobias-urdin | any have some seconds to review https://review.opendev.org/c/openstack/nova/+/825064 | |
| 14:08:38 | tobias-urdin | stable backport | |
| 14:09:01 | opendevreview | Tobias Urdin proposed openstack/nova stable/xena: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/828407 | |
| 15:04:32 | dmitriis | https://review.opendev.org/c/openstack/nova-specs/+/828177 lost a WF+1 during a rebase to fix a doc test failure - if anybody has time | |
| 15:11:32 | gibi | dmitriis: fixed | |
| 15:11:38 | dmitriis | gibi: tyvm | |
| 15:31:21 | chateaulav | dansmith: i imagine id want to just do a 'del primitive' if it meets thats case, correct? | |
| 15:31:48 | opendevreview | Merged openstack/nova-specs master: Late Amendments to the Off-path Backends Spec https://review.opendev.org/c/openstack/nova-specs/+/828177 | |
| 15:43:17 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 15:43:24 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 15:43:24 | opendevreview | Jonathan Race proposed openstack/nova master: object/notification for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828369 | |
| 15:43:25 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 15:50:49 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 15:51:57 | dansmith | chateaulav: you didn't add the field, you just added an enum value, so no, I don't think deleting the primitive entirely is really the right approach | |
| 15:52:22 | dansmith | chateaulav: I haven't looked, but did you determine why the field is being set to riscv32 if we're not running guests configured thusly? | |
| 15:53:43 | gibi | dansmith: I think the compute node has all the supported arch listed in supported_hv_specs field. And in grenade, during the live migration the old compute loads the new compute from the DB and sees a HVSpec with the new value | |
| 15:54:17 | dansmith | gibi: oh is this a list of all the possible supported specs? | |
| 15:54:35 | gibi | dansmith: I think so | |
| 15:54:37 | chateaulav | dansmith: yes. | |
| 15:54:59 | dansmith | ah, I see. So the thing that needs to happen is remove that object from the list, which you can't do in the backport of the offending object of course | |
| 15:55:15 | dansmith | this is one reason we have list objects, but no such container is used here I think | |
| 15:55:39 | dansmith | so if the list is on the compute node, you'll need to do it in the compute node's backport routine, which I assume is what gibi was originally suggesting | |
| 15:56:01 | gibi | dansmith: yeah, something like that | |
| 15:56:17 | dansmith | I thought this was representing a single guest, not expecting one to be set to riscv32, but now the tie to the compute node makes more sense :) | |
| 15:56:20 | chateaulav | dansmith: ok, that makes more sense. | |
| 15:58:45 | chateaulav | so then ill need to bump the version there in order to do a proper backport | |
| 15:58:46 | gibi | dansmith: yeah, that casuse my inital confusion too | |
| 15:59:11 | gibi | chateaulav: yes, I think so. bump the compute ovo version and in the backport code drop the HVSpec from the list that has new value | |
| 16:00:29 | dansmith | chateaulav: yes | |
| 16:18:09 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 16:44:04 | gmann | bauzas: sure, that series is in my list for today. | |
| 16:47:44 | melwitt | bauzas: if you get a chance, would appreciate your review on this backport that is two patches squashed together https://review.opendev.org/c/openstack/nova/+/825064 I did the squash so it would be best if someone else can review that | |
| 17:21:46 | bauzas | melwitt: easy peasy, done | |
| 17:21:58 | bauzas | gmann: I had a question about the approach for merging stuff | |
| 17:22:27 | bauzas | gmann: given the first patch has a microversion, we would hold this one until we review the whole series ? | |
| 17:29:02 | bauzas | ok, /me needs to leave | |
| 17:29:04 | bauzas | \o | |
| 17:33:44 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 17:34:01 | melwitt | bauzas: thanks :) | |
| 17:35:56 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 17:40:29 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 17:42:28 | opendevreview | David Hill proposed openstack/nova master: Fail on parse_dom if vif quota sizes > 32 bit unsigned int https://review.opendev.org/c/openstack/nova/+/829064 | |
| 17:46:02 | gmann | bauzas: yes, you are right. I am +2 on first patch but -W until we get the whole series ready to go. | |