| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-14 | |||
| 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. | |
| 17:53: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 | |
| 18:01:59 | 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:59 | 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:02:00 | 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:30 | 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:31 | 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:31 | 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 | |
| 21:14:50 | opendevreview | yuval proposed openstack/nova master: Lightbits LightOS driver https://review.opendev.org/c/openstack/nova/+/821606 | |
| 21:23:25 | opendevreview | Ade Lee proposed openstack/nova master: DNM - patch paramiko to get around ed25519 problem https://review.opendev.org/c/openstack/nova/+/829135 | |
| 23:55: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 | |
| #openstack-nova - 2022-02-15 | |||
| 00:00:02 | 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 | |
| 00:14: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 | |
| 00:34:02 | 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 | |
| 00:35:13 | 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 | |
| 00:40:07 | opendevreview | David Hill proposed openstack/nova master: Validate burst value is equal or smaller than 4194303 https://review.opendev.org/c/openstack/nova/+/829064 | |
| 07:00:29 | opendevreview | Takashi Kajinami proposed openstack/nova stable/xena: Fix the wrong exception used to retry detach API calls https://review.opendev.org/c/openstack/nova/+/829049 | |
| 08:27:23 | gibi | sean-k-mooney: hi! when you are up, regarding https://bugs.launchpad.net/neutron/+bug/1942329 I made some progress and added a proposal to the today's neutron meeting https://wiki.openstack.org/wiki/Network/Meetings#On_Demand_Agenda you might be interested | |
| 09:44:51 | opendevreview | alecorps proposed openstack/nova master: VMware: Support volumes backed by VStorageObject https://review.opendev.org/c/openstack/nova/+/808791 | |
| 11:14:54 | sean-k-mooney | gibi: oh ok cool so that is something we can fix entirly on the neutron side | |
| 11:15:03 | sean-k-mooney | * entirly on the nova side | |
| 11:15:14 | gibi | sean-k-mooney: nope we need both nova and neutron change | |
| 11:15:18 | sean-k-mooney | we just need to update the mac addres in the neutron port when we migrate | |
| 11:15:36 | sean-k-mooney | well you are thinking of extending the port bindings? | |
| 11:15:45 | sean-k-mooney | to avoid just doing the mac update? | |
| 11:16:01 | gibi | 1) in single binding case we need to update the mac on a bound port that is rejected today by neutron | |
| 11:16:13 | sean-k-mooney | ah ok | |
| 11:16:21 | gibi | 2) in multi binding case we might want to keep the MAC per binding to be able to switch | |
| 11:16:21 | sean-k-mooney | i didnt know that would be rejected | |
| 11:16:38 | sean-k-mooney | yep | |
| 11:17:05 | sean-k-mooney | the thing is the mac is not part of the port binding | |
| 11:17:19 | sean-k-mooney | which i guess is why we need neutron changes | |
| 11:17:24 | gibi | but then we need neturon to use the MAC from the active binding when returning a port | |
| 11:17:45 | gibi | yes, and that is a neutron change too | |
| 11:17:46 | sean-k-mooney | right but the mac is an atribute or the port extention | |
| 11:17:52 | gibi | yes, | |
| 11:18:26 | sean-k-mooney | rather then port binding extentions so ya if we do that we obviously need to handel this slight differently | |
| 11:18:27 | gibi | so we need to be carefully overlay the mac from the active binding top of the port.mac_address field | |