| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-06 | |||
| 09:12:37 | sean-k-mooney | sahid_: i dont know if i will get to them today but i will get to them before i go on pto for the year | |
| 09:12:52 | sean-k-mooney | im here until friday next week | |
| 09:13:03 | sean-k-mooney | i suspect we can likely get this merged before then | |
| 09:15:00 | bauzas | yup, I'll try too | |
| 09:15:16 | bauzas | I personnally need to rush on implementing the cpu offlining feature | |
| 09:15:22 | bauzas | before the xmas period | |
| 09:21:56 | sean-k-mooney | if its aviable next week ill have time to start reviewing | |
| 09:22:12 | zigo | sahid_: Hi there! Would you have the URL of the bp handy? | |
| 09:24:48 | zigo | It appears cinderclient has the same exact bug (the 2 broken tests are looking very similar if not completely identical) | |
| 09:24:48 | zigo | Back to the bug report from yesterday: https://bugs.launchpad.net/python-novaclient/+bug/1998856 | |
| 09:37:58 | sahid_ | Hey zigo, https://github.com/openstack/nova-specs/blob/master/specs/2023.1/approved/allowing-target-state-for-evacuate.rst | |
| 09:54:07 | sahid_ | sean-k-mooney: hey! no worries, just that more we can have it soon in the tree more we can fix any bugs it could have :-) (and also safe am I we have it merged for antelope :p) | |
| 10:52:25 | opendevreview | Sylvain Bauza proposed openstack/nova stable/wallaby: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866157 | |
| 10:52:40 | opendevreview | Sylvain Bauza proposed openstack/nova stable/wallaby: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866158 | |
| 10:59:25 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/865334 | |
| 10:59:26 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/865336 | |
| 10:59:27 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/865571 | |
| 10:59:28 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/865572 | |
| 10:59:29 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866157 | |
| 10:59:31 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866158 | |
| 11:11:01 | zigo | sahid_: If I understand correctly, the goal is to have a instance go from ACTIVE to SHUTOFF during the evacuate ? | |
| 11:18:18 | sahid_ | zigo: yes exactly | |
| 11:18:41 | zigo | Thanks. Though I don't really understand why one would do that ... :P | |
| 11:18:48 | sahid_ | now when we do evacuate (maintenance operation) instances on dest get stopped | |
| 11:19:49 | sahid_ | yea.. there are several use-cases, also that for operators that do prefer kept the previous behavior they can use microversion | |
| 11:20:37 | zigo | I'm not a fan of using microversions to have change of behavior, tbh... but ok. | |
| 11:20:40 | sahid_ | sean-k-mooney: regarding that tiny one, https://review.opendev.org/c/openstack/os-vif/+/866102 i would tend to accept it, i comment on it, are you ok or do you feel like we should find an other solution | |
| 11:22:31 | sahid_ | zigo: yeah, it's really for operators to handle resources and things like that, I think that has been discussed previously during PTG | |
| 11:23:42 | sean-k-mooney | so i kind of disagree on the parsablity argument as if you cant parse multiline output you cant actully parse logs properly | |
| 11:24:41 | sahid_ | sean-k-mooney: sure, fair enough | |
| 11:24:42 | sean-k-mooney | sahid_: we also use multi line strings in nova for the deprecated_reason extensivly and im not infavor fo changing it in nova | |
| 11:25:28 | sean-k-mooney | for example https://github.com/openstack/nova/blob/6ba6d86e9c2ab44752d87f070ce5e1ce00a48301/nova/conf/glance.py#L34-L39 | |
| 11:26:25 | sahid_ | yes i understand your point no worries | |
| 11:27:45 | sean-k-mooney | with all that said we could actully remove those deprecated optoins this cycle | |
| 11:27:48 | sahid_ | I was seeing that more like, we could keep some sort of coherence with the other multiline strings we have in this set of options | |
| 11:27:55 | sean-k-mooney | its been several years at thsi point | |
| 11:28:03 | sahid_ | interesting point | |
| 11:28:19 | sean-k-mooney | we could be if we were to do that i woudl change all the rest to multi line | |
| 11:28:32 | sean-k-mooney | that woudl be more consitent with the wider code base | |
| 11:28:35 | sahid_ | good point :-) | |
| 11:28:51 | sean-k-mooney | we have moved away form sting conatination like that over the last few years | |
| 11:29:05 | sahid_ | but that would not help the contributor which have noticed that multiline thing in its logs | |
| 11:29:51 | sahid_ | i would vote in favor of removing that deprecating message if i may | |
| 11:29:53 | sean-k-mooney | by the way https://review.opendev.org/c/openstack/os-vif/+/854486 i generally see a a nuisance commit | |
| 11:30:50 | sean-k-mooney | im going to merge it to not have to deal with it again but i have seen a bunch of people submiting patch to remove the u prfix and it is somethign we shoudl discurage | |
| 11:34:55 | sahid_ | yes i was considering to see whether we could or not add a hacking rule for that sort of change | |
| 11:35:23 | sahid_ | yes you are certainly dealing with such changes often that me :-) | |
| 11:40:45 | opendevreview | Merged openstack/os-vif master: remove unicode from code https://review.opendev.org/c/openstack/os-vif/+/854486 | |
| 13:15:59 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/865572 | |
| 13:16:00 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866157 | |
| 13:16:01 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866158 | |
| 14:12:40 | opendevreview | Sylvain Bauza proposed openstack/nova stable/wallaby: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866157 | |
| 14:13:00 | opendevreview | Sylvain Bauza proposed openstack/nova stable/wallaby: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866158 | |
| 14:22:06 | kgube | sean-k-mooney, melwitt, Regarding the assisted volume extend API: I have now proposed an alternative spec for an async solution to Cinder: https://blueprints.launchpad.net/cinder/+spec/extend-volume-completion-action | |
| 14:24:35 | opendevreview | alecorps proposed openstack/nova master: Add mksproxy files https://review.opendev.org/c/openstack/nova/+/863090 | |
| 14:24:43 | kgube | sean-k-mooney, this does not require an API change in Nova, just a change in how "volume-extended" events are processed, should I create a nova-spec anyway? | |
| 14:27:09 | kgube | sean-k-mooney, the changes for Nova are described in a "Nova" subsection of the "Proposed change" section of the cinder-spec | |
| 15:49:49 | bauzas | reminder: nova meeting in 10 mins | |
| 16:01:28 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:28 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:28 | opendevmeet | Meeting started Tue Dec 6 16:01:28 2022 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:28 | bauzas | #startmeeting nova | |
| 16:01:33 | bauzas | sorry, was late | |
| 16:01:53 | bauzas | howdy everyone | |
| 16:02:04 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:02:23 | elodilles | o/ | |
| 16:03:18 | Kirill_ | o/ | |
| 16:03:26 | bauzas | looks like we maybe not have a lot of quorum | |
| 16:03:30 | bauzas | but let's start | |
| 16:03:35 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:40 | bauzas | #info No Critical bug | |
| 16:03:45 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 18 new untriaged bugs (+2 since the last meeting) | |
| 16:03:50 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:03:57 | bauzas | elodilles: anything to discuss about bugs ? | |
| 16:04:07 | elodilles | maybe one question | |
| 16:04:21 | elodilles | had a bug: explicit fixed ip request assigns random ip: https://bugs.launchpad.net/nova/+bug/1998927 | |
| 16:04:32 | elodilles | this is i think python-openstackclient | |
| 16:04:50 | elodilles | how can i reroute this? o:) | |
| 16:06:07 | bauzas | elodilles: yup, looks to me an OSC issue | |
| 16:06:16 | bauzas | "the fixed ip command assigns random IP even if we ask an IP explicitly" | |
| 16:06:26 | bauzas | you mean, because of OSC ? | |
| 16:06:39 | bauzas | if so, we need to change the project in Launchpad | |
| 16:06:40 | elodilles | i think so | |
| 16:06:43 | bauzas | lemme do it | |
| 16:06:58 | bauzas | "also affects project" > openstackclient | |
| 16:07:12 | elodilles | oh, cool, thanks | |
| 16:07:29 | bauzas | oh snap | |
| 16:07:42 | bauzas | "python-openstackclient doesn't use Launchpad to track its bugs. If you know this bug has been reported in another bug tracker, you can link to it; Launchpad will keep track of its status for you. " | |
| 16:07:52 | bauzas | StoryBoard my love | |
| 16:08:08 | bauzas | then you should close the bug | |
| 16:08:23 | bauzas | and ask the report to create a Storyboard issue for osc | |
| 16:08:26 | bauzas | lemme find it | |
| 16:08:28 | elodilles | bauzas: ack, will do that | |
| 16:08:42 | elodilles | bauzas: i'll do that after the meeting | |
| 16:08:57 | elodilles | bauzas: no need to spend time with it now :) | |
| 16:09:00 | bauzas | https://storyboard.openstack.org/#!/project/975 | |
| 16:09:08 | elodilles | thx :) | |
| 16:09:14 | bauzas | it was simple to find https://docs.openstack.org/python-openstackclient/latest/#contributing | |
| 16:09:43 | bauzas | elodilles: ok, I'll then let you triage this bug | |
| 16:09:50 | bauzas | and thanks for it | |