| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-05 | |||
| 17:03:54 | opendevreview | Merged openstack/nova stable/xena: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/864932 | |
| 17:04:00 | opendevreview | Merged openstack/nova stable/xena: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/864933 | |
| 17:04:07 | opendevreview | Merged openstack/nova stable/xena: Remove double mocking https://review.opendev.org/c/openstack/nova/+/864934 | |
| 17:04:13 | opendevreview | Merged openstack/nova stable/xena: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/864935 | |
| 17:04:19 | opendevreview | Merged openstack/nova stable/xena: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/864936 | |
| 17:04:24 | opendevreview | Merged openstack/nova stable/xena: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/864937 | |
| 17:56:04 | sean-k-mooney | :) | |
| 17:56:14 | sean-k-mooney | ok ill update the wallaby ones now | |
| 18:02:38 | melwitt | sean-k-mooney: fyi this is the strange py27 failure on the stable/train backport that I could not figure out what is going on https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_80f/866090/1/check/openstack-tox-py27/80f58ef/testr_results.html | |
| 18:03:10 | sean-k-mooney | yep i have that open in a tab somewhere | |
| 18:03:26 | melwitt | ok cool :) | |
| 18:04:59 | sean-k-mooney | im wondering if this is related to a call to super or something like that | |
| 18:05:23 | sean-k-mooney | im not exactly sure where TypeError: __init__() takes at least 2 arguments (1 given) | |
| 18:05:32 | sean-k-mooney | is coming form or if this is a tully the issue | |
| 18:06:26 | melwitt | hmm, I will check | |
| 18:09:01 | sean-k-mooney | my guess is either this is related to the python 3 only verison fo super or we require an adtional positional arg in train for a class | |
| 18:09:18 | sean-k-mooney | the one arg that was given is presumabley self | |
| 18:10:05 | sean-k-mooney | althugh it could be the context | |
| 18:13:02 | melwitt | I feel like it has to be something to do with creation of the ClientException objects in the unit tests, there's really not much in the patch | |
| 18:14:00 | melwitt | *not much else | |
| 18:14:37 | sean-k-mooney | whats weired is it only fails on py27 | |
| 18:14:58 | sean-k-mooney | i expect most or all of that to not care | |
| 18:15:16 | melwitt | yeah, same. so weird | |
| 21:01:53 | zigo | Hi there! About https://bugs.debian.org/1025110 to me it feels like novaclient is calling suparser.addparser twice with the same subparser name, which now raises an error in Py 3.11. | |
| 21:02:07 | zigo | Though I'm having trouble finding the 2 places ... | |
| 21:02:25 | zigo | (I'm not familliar with novaclient...) | |
| 21:02:33 | zigo | Can someone have a look? | |
| 21:04:44 | zigo | In novaclient/tests/unit/test_shell.py, I can see shell._find_actions() called twice with fake_actions_module as argument ... | |
| 21:05:53 | zigo | That's both in test_load_versioned_actions() and test_load_actions_with_versioned_args(). | |
| 21:06:02 | zigo | So this looks like a test issue, not a novaclient issue. | |
| 21:10:19 | zigo | https://bugs.launchpad.net/python-novaclient/+bug/1998856 | |
| #openstack-nova - 2022-12-06 | |||
| 08:32:51 | opendevreview | Hiroki Narukawa proposed openstack/nova master: libvirt: add sftp driver https://review.opendev.org/c/openstack/nova/+/866672 | |
| 09:08:33 | sahid_ | o/ sorry to annoy guys any chance to get somes reviews regarding the impl of evacuate spec? https://review.opendev.org/c/openstack/nova/+/858383 there are lot of files updated but just small changes | |
| 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 | Back to the bug report from yesterday: https://bugs.launchpad.net/python-novaclient/+bug/1998856 | |
| 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: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 | bauzas | #startmeeting nova | |
| 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 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:28 | opendevmeet | The meeting name has been set to '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/ | |