| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-23 | |||
| 18:28:42 | sean-k-mooney | gibi: im not sure that it really has changed | |
| 18:28:57 | sean-k-mooney | gibi: i think they still have not fixed it | |
| 18:30:42 | sean-k-mooney | gibi: bauzas by the way if eitehr of ye can approve https://review.opendev.org/c/openstack/nova/+/865031 it will simplfy ralonsoh life and help with the trunk port issue | |
| 18:31:12 | sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/837780 cuurntly need a cofnig option because of our min version | |
| 18:31:26 | sean-k-mooney | but that can be removed if we increase the min version of os-vif | |
| 19:59:35 | opendevreview | Merged openstack/nova master: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/850673 | |
| #openstack-nova - 2022-11-24 | |||
| 03:04:47 | opendevreview | Jorhson Deng proposed openstack/nova master: Remove the redundant code in HostState._update_from_compute_node https://review.opendev.org/c/openstack/nova/+/864824 | |
| 05:17:23 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863806 | |
| 05:17:24 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 05:56:49 | opendevreview | Jorhson Deng proposed openstack/nova master: Remove the redundant code in HostState._update_from_compute_node https://review.opendev.org/c/openstack/nova/+/864824 | |
| 06:31:14 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 06:32:14 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 07:02:52 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 07:02:53 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 08:16:03 | han-guangyu | hello, cloud I ask a question | |
| 08:16:48 | han-guangyu | Could I use a windows ISO image to create instance in openstack | |
| 08:19:10 | han-guangyu | Or do I have to make an image with the virtio driver first | |
| 08:47:07 | gibi | sean-k-mooney: but we fixed it (worked around it) in tempest by waiting for the guest to boot up before we attach / detach volumes. | |
| 08:47:29 | gibi | so if tempest is failing again on master then someting is changed | |
| 09:04:40 | bauzas | gibi: yeah I need to do my homework on the flakey gate | |
| 09:05:22 | bauzas | I'm impacted for my vgpu series, I'm quite in a position where I should take time digging into the ci issue for my own egoistic sake :) | |
| 09:05:48 | bauzas | and I'm done with preparing my mock presentation for tonight's grenoble meetup :p | |
| 09:17:46 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/864006 | |
| 09:17:47 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864007 | |
| 09:25:46 | gibi | bauzas: you are in a good spot then :) | |
| 09:33:01 | bauzas | gibi: on a side note, that's fun to prescribe testing patterns to an audience with does and donts when your own project isn't that much following those testing patterns :) | |
| 09:33:27 | bauzas | like, piling our mock decorators in every single UT | |
| 09:33:59 | bauzas | I just feel schizophrenic : | |
| 09:34:00 | bauzas | :) | |
| 09:34:05 | gibi | you learned things in our project you dont want to repeate | |
| 09:34:06 | gibi | that is normal | |
| 10:04:37 | opendevreview | Merged openstack/nova stable/train: func: Add _live_migrate helper to InstanceHelperMixin https://review.opendev.org/c/openstack/nova/+/865381 | |
| 10:23:30 | opendevreview | Merged openstack/nova master: Bump minimum version of os-vif to 3.1.0 https://review.opendev.org/c/openstack/nova/+/865031 | |
| 10:44:42 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Refactor volume connection cleanup out of _post_live_migration https://review.opendev.org/c/openstack/nova/+/864670 | |
| 10:44:43 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863806 | |
| 10:44:44 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 11:19:45 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863806 | |
| 11:19:46 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 14:37:26 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/865571 | |
| 14:37:27 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/865572 | |
| 14:38:00 | sean-k-mooney | gibi: just an fyi i took a slightly diffent approch with xena->wallaby | |
| 14:38:19 | sean-k-mooney | for yoga and xena i included the double mocking patch | |
| 14:38:25 | sean-k-mooney | well patches | |
| 14:38:36 | gibi | ack | |
| 14:38:38 | sean-k-mooney | they were pretty ok to backport | |
| 14:38:55 | sean-k-mooney | but xena to wallaby it was getting a bit messy so i dropped them | |
| 14:39:12 | sean-k-mooney | i didnt really have many conflict when i did that so it shoudl be fine | |
| 14:40:01 | gibi | OK | |
| 14:40:03 | bauzas | don't get me into the ML thread about OSC/SDK and the project clients :D | |
| 14:40:08 | gibi | I added it to my review queue | |
| 14:40:30 | bauzas | IMHO, OSC : profiled for UX | |
| 14:40:44 | bauzas | SDK : profiled for powerusers | |
| 14:41:32 | bauzas | powerusers being in my terminology 'seasoned users of openstack', and not powerusers in terms of role | |
| 14:41:57 | bauzas | so, having the OSC defaulting to latest sounds to me right | |
| 14:41:58 | sean-k-mooney | so i attened some of the very early design summit sessions on osc and talked to dean troyer alot about it in the halway track | |
| 14:42:18 | bauzas | while we could tell that the SDK defaults to the bare min version | |
| 14:42:24 | sean-k-mooney | bauzas: we coudl default to latest but if we do it need to eb a new microverion | |
| 14:42:31 | sean-k-mooney | *new majory version | |
| 14:42:41 | bauzas | OSC or the SDK ? | |
| 14:42:50 | sean-k-mooney | technially both | |
| 14:43:00 | sean-k-mooney | sdk recently broke all of the ansible module integration | |
| 14:43:24 | sean-k-mooney | osc need to have a majory version if its going to enabel auto negociation | |
| 14:43:54 | sean-k-mooney | the sdk i think did but they broke ansible recently with a change in this area | |
| 15:00:52 | bauzas | sean-k-mooney oh when you say major version, you were talking about the semver of OSC releases, lol | |
| 15:02:51 | sean-k-mooney | yes | |
| 15:04:10 | sean-k-mooney | osc has a backward incompatible change dock and follows semver for the client releases | |
| 15:04:12 | sean-k-mooney | https://docs.openstack.org/python-openstackclient/pike/cli/backwards-incompatible.html | |
| 15:04:47 | sean-k-mooney | so if they want to turn on auto negociation there should be a major version of the client | |
| 15:05:37 | sean-k-mooney | https://github.com/openstack/ansible-collections-openstack#breaking-backward-compatibility-warning | |
| 15:05:58 | bauzas | I'm not opposed to signal it thru a .x | |
| 15:06:05 | bauzas | x. rather | |
| 15:06:46 | bauzas | this is not like we have an open bug for 1 year due to a libvirt behavioural breaking change in a .y bump | |
| 15:06:56 | sean-k-mooney | :) | |
| 15:07:16 | sean-k-mooney | i like to think that we have learn for other projects mistakes | |
| 15:25:01 | bauzas | that port_active_state failure gives me headaches https://zuul.opendev.org/t/openstack/build/c6dee3859e354e6d8ed69c7ac7826761 | |
| 15:31:34 | auniyal | bauzas, and Push the CA certificate to me https://zuul.opendev.org/t/openstack/build/e3bf378e2f8249feb1beec96fcde2802 | |
| 15:35:17 | sean-k-mooney | bauzas: that will hopefully get fixed soon althoguh we coudl skip those test for now until that happens | |
| 15:35:47 | sean-k-mooney | bauzas: oh that is a differnt one | |
| 15:36:13 | sean-k-mooney | oh no its the turnk one | |
| 15:36:27 | sean-k-mooney | so ya we can exclude it for now until the neutron patch lands | |
| 15:37:09 | sean-k-mooney | bauzas: ralonsoh has updated a patch form mlavalle to adress it | |
| 15:37:23 | ralonsoh | sean-k-mooney, not yet | |
| 15:37:30 | ralonsoh | I've pushed a patch to test the backends | |
| 15:37:41 | ralonsoh | but I'll remove the conditional branch now Nova patch is merged | |
| 15:37:43 | ralonsoh | and thanks! | |
| 15:37:54 | sean-k-mooney | ralonsoh: your still having issue with ovn right | |
| 15:38:03 | ralonsoh | yes but I found the issue | |
| 15:38:08 | sean-k-mooney | cool | |
| 15:38:29 | ralonsoh | (what a week with trunks a live migration, both OVS and OVN hehehe) | |
| 15:38:42 | sean-k-mooney | ralonsoh: would you object if we dropped the test until its fixed or show we just wait a few days | |
| 15:39:56 | ralonsoh | sean-k-mooney, maybe you can use "tempest_exclude_regex" for some days | |
| 15:40:02 | ralonsoh | not to block the CI | |
| 15:40:10 | ralonsoh | same as we did in https://review.opendev.org/c/openstack/neutron/+/865295/1/zuul.d/tempest-multinode.yaml#171 | |
| 15:42:44 | bauzas | sean-k-mooney: yup, this is a networking issue, I'm happy to hear a fix is coming up | |
| 15:43:00 | bauzas | in order to hold my rechecks, any gerrit link I should take care of ? | |
| 15:46:09 | sean-k-mooney | ralonsoh: ya that is what i was going to propose | |
| 15:46:20 | sean-k-mooney | bauzas: we dont have a patch currently but i can create one or you can | |
| 15:46:38 | sean-k-mooney | we only run the trunk live migration tests in a couple of jobs | |
| 15:46:55 | ralonsoh | but a patch for tempest or nova? I would suggest for nova only, as in | |
| 15:46:55 | sean-k-mooney | so we can juse exculde them for a few days and revert it when the fix lands | |