| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-05 | |||
| 15:23:35 | sean-k-mooney | either the host value does not match adn we create a new compute service entry | |
| 15:23:55 | sean-k-mooney | or the hypervior_hostname does not match and we delete and recreate teh compute node record | |
| 15:24:27 | dansmith | neither will match on a libvirt host that has renamed right? | |
| 15:24:32 | sean-k-mooney | so if the host value changes the service entry gets recreated and if the hypervior_hostname mismatches its the compute node | |
| 15:24:53 | sean-k-mooney | dansmith: if you have notihgn set in the config then yes | |
| 15:25:12 | dansmith | ah, right | |
| 15:25:26 | sean-k-mooney | ooo at least hardcodes the host value | |
| 15:25:39 | sean-k-mooney | so if the hostname change and the config is not updated then it will find the service | |
| 15:25:48 | sean-k-mooney | then delete the compute node and recreate it | |
| 15:27:36 | dansmith | sean-k-mooney: yeah I guess we need to make sure we have those cases covered in the tests.. "host renamed, no config set" and "host renamed, config set to something" | |
| 15:28:34 | sean-k-mooney | i think we can emulate that by mocking the driver method to get the host name and config flags at least at a unit test level | |
| 15:28:52 | sean-k-mooney | for functional tests we could proably do it too but only with one compute node sicne the config is shared | |
| 15:29:24 | sean-k-mooney | but yes i agree i dont know if we need to test all 4 combination but proably woudl not hurt | |
| 15:30:06 | admin1 | is there a good rescue image (os ) for openstack .. that kind of boots directly without login/pass for rescue ? | |
| 15:32:39 | sean-k-mooney | well by default the same image the vm is booted with is used for rescue | |
| 15:33:00 | sean-k-mooney | but any of the images you woudl use on a phsyical host will be good | |
| 15:33:16 | sean-k-mooney | like KNOPPIX | |
| 15:33:17 | opendevreview | Merged openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 15:34:37 | sean-k-mooney | it actully would proably have made sense to allow a keypair to be specifed for rescue | |
| 15:35:05 | sean-k-mooney | although the assumtion was the tenant would do rescue so that why we reuse the one the vm is booted with | |
| 15:36:13 | sean-k-mooney | admin1: if you are doing this on behalf of a tenatn what i would recommend is creating a iamge yourself and baking in a set of ssh keys and then usign that to rescue tenant vms | |
| 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 | 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 | |