| opendevreview | Ghanshyam proposed openstack/nova master: Modify remaining APIs as per RBAC new guidelines https://review.opendev.org/c/openstack/nova/+/828994 | 05:07 |
|---|---|---|
| *** amoralej|off is now known as amoralej | 07:01 | |
| kashyap | gibi: Morning; reading back. Don't worry, you've done a lot more already! Thank _you_ for the quick trials and tests | 08:36 |
| gibi | kashyap: hi! | 09:43 |
| * kashyap waves; is fighting an embarassing networking issue on an internal OSP cloud | 09:44 | |
| gibi | good luck :) | 09:45 |
| zigo | Would anyone have any idea what's going on with https://bugs.debian.org/1005632 ? | 10:25 |
| gibi | zigo: looking... | 10:56 |
| zigo | gibi: It's likely caused by an update of some Python dependencies in Unstable, but I can't figure it out ... :/ | 10:58 |
| gibi | zigo: this seems relevant https://github.com/openstack/python-novaclient/blob/90525a1f5f55e206554da373dda9f26735e7c67f/novaclient/tests/unit/test_shell.py#L586-L599 | 11:04 |
| gibi | so I guess it is prettytable | 11:04 |
| gibi | from the debian build log: prettytable==0.0.0 | 11:04 |
| gibi | that seems worng :) | 11:05 |
| gibi | but also it says python3-prettytable all 2.5.0-1 | 11:05 |
| gibi | it can be that the test code wrongly detects the version of prettytable | 11:12 |
| 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:15 |
| zigo | gibi: Let me check what's the result of installing prettytable in Sid and see if the egginfo is wrong. | 11:25 |
| gibi | zigo: ok | 11:25 |
| zigo | Installed version really says 0.0.0 ... :/ | 11:26 |
| zigo | So probably a packaging issue (I'm not the maintainer of that one ...). | 11:27 |
| 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:27 |
| 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 |
| bauzas | gmann: when you're up, I'm looking at https://review.opendev.org/c/openstack/nova/+/764292/34 | 11:28 |
| zigo | setup( | 11:28 |
| zigo | use_scm_version={"local_scheme": local_scheme}, | 11:28 |
| zigo | ) | 11:28 |
| zigo | So it's using scm, but the packaging hasn't set it, so it's wrong ... | 11:28 |
| zigo | I'll try to fix it. | 11:28 |
| zigo | gibi: I'll let you know the result when I'm done. | 11:29 |
| gibi | zigo: cool thanks | 11:29 |
| stephenfin | ah, not pbr but a similar issue. cool :) | 11:29 |
| opendevreview | Merged openstack/nova master: Gracefull recovery when attaching volume fails https://review.opendev.org/c/openstack/nova/+/828516 | 12:37 |
| 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:42 |
| 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:45 |
| *** amoralej is now known as amoralej|lunch | 12:58 | |
| 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 |
| zigo | s/pyparsing/prettytable/ | 12:59 |
| gibi | zigo: good | 12:59 |
| gibi | then it was easy | 13:00 |
| gibi | :) | 13:00 |
| zigo | Yeah, took me some times to realize how the current maintainer broke things though ... :P | 13:00 |
| gibi | I don't see that part. :) Fortunately the test had a nice comment pointing at the pretty table version | 13:01 |
| gibi | lyarwood: hi! do you know somebody planning to take ower tempest https://review.opendev.org/q/topic:wait_until_sshable_pingable series? | 13:07 |
| bauzas | gibi: lyarwood stopped to work on Nova | 13:44 |
| gibi | bauzas: yeah I know, hence my question | 13:49 |
| gibi | as that series was pushed by lyarwood before | 13:49 |
| gibi | and we still need it | 13:49 |
| 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 |
| gibi | s/so/saw/ | 13:50 |
| *** dasm|off is now known as dasm | 14:01 | |
| *** amoralej|lunch is now known as amoralej | 14:03 | |
| tobias-urdin | any have some seconds to review https://review.opendev.org/c/openstack/nova/+/825064 | 14:08 |
| tobias-urdin | stable backport | 14:08 |
| opendevreview | Tobias Urdin proposed openstack/nova stable/xena: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/828407 | 14:09 |
| 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:04 |
| gibi | dmitriis: fixed | 15:11 |
| dmitriis | gibi: tyvm | 15:11 |
| chateaulav | dansmith: i imagine id want to just do a 'del primitive' if it meets thats case, correct? | 15:31 |
| opendevreview | Merged openstack/nova-specs master: Late Amendments to the Off-path Backends Spec https://review.opendev.org/c/openstack/nova-specs/+/828177 | 15:31 |
| 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 |
| 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 |
| 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 |
| 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:43 |
| 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:50 |
| 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:51 |
| 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:52 |
| 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:53 |
| dansmith | gibi: oh is this a list of all the possible supported specs? | 15:54 |
| gibi | dansmith: I think so | 15:54 |
| chateaulav | dansmith: yes. | 15:54 |
| 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:54 |
| dansmith | this is one reason we have list objects, but no such container is used here I think | 15:55 |
| 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:55 |
| gibi | dansmith: yeah, something like that | 15:56 |
| *** hemna2 is now known as hemna | 15:56 | |
| 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 |
| chateaulav | dansmith: ok, that makes more sense. | 15:56 |
| chateaulav | so then ill need to bump the version there in order to do a proper backport | 15:58 |
| gibi | dansmith: yeah, that casuse my inital confusion too | 15:58 |
| 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 | 15:59 |
| dansmith | chateaulav: yes | 16:00 |
| *** amoralej is now known as amoralej|off | 16:12 | |
| 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:18 |
| *** akekane_ is now known as abhishekk | 16:43 | |
| gmann | bauzas: sure, that series is in my list for today. | 16: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 | 16:47 |
| bauzas | melwitt: easy peasy, done | 17:21 |
| bauzas | gmann: I had a question about the approach for merging stuff | 17:21 |
| bauzas | gmann: given the first patch has a microversion, we would hold this one until we review the whole series ? | 17:22 |
| * bauzas is about to leave in a very few | 17:23 | |
| bauzas | ok, /me needs to leave | 17:29 |
| bauzas | \o | 17: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:33 |
| melwitt | bauzas: thanks :) | 17:34 |
| 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:35 |
| 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 |
| 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 |
| gmann | bauzas: yes, you are right. I am +2 on first patch but -W until we get the whole series ready to go. | 17:46 |
| 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:53 |
| 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 | 18:01 |
| 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:01 |
| 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 | 18:02 |
| 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 | 20:26 |
| 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 | 20:26 |
| 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 | 20:26 |
| opendevreview | yuval proposed openstack/nova master: Lightbits LightOS driver https://review.opendev.org/c/openstack/nova/+/821606 | 21:14 |
| opendevreview | Ade Lee proposed openstack/nova master: DNM - patch paramiko to get around ed25519 problem https://review.opendev.org/c/openstack/nova/+/829135 | 21:23 |
| *** dasm is now known as dasm|off | 23:19 | |
| 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 | 23:55 |
| *** thelounge551 is now known as thelounge55 | 23:56 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!