| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-23 | |||
| 17:27:29 | sean-k-mooney | that was quick | |
| 17:39:33 | ganso | lyarwood: hi! is there anything pending on https://review.opendev.org/c/openstack/nova/+/795432 so it can be merged? That patch was the latest one to pass CI. There are several patches (including one with +W) that haven't had a single positive CI run | |
| 19:37:58 | lyarwood | elodilles: https://review.opendev.org/c/openstack/nova/+/795432 - can you hit this in the morning? I've upgraded my +1 to +2 to move it along. | |
| 19:38:17 | lyarwood | ganso: apologies, I'll work with elodilles and the other stable cores to unblock things in the morning | |
| 19:39:47 | ganso | lyarwood: np! thank you very much! I was mostly wondering if something else was pending because I had rebased on top of it and it still failed. See latest comments in https://review.opendev.org/c/openstack/nova/+/796719 | |
| #openstack-nova - 2021-06-24 | |||
| 07:25:05 | opendevreview | Slawek Kaplonski proposed openstack/nova stable/ussuri: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787253 | |
| 07:47:10 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/794675 | |
| 07:51:55 | elodilles | lyarwood: uhh, I almost messed up the supermegasquash :S | |
| 07:52:19 | elodilles | lyarwood: btw, do we really need there the "Replace nova-live-migration with zuulv3 jobs" ? | |
| 07:52:26 | elodilles | :/ | |
| 08:18:32 | elodilles | lyarwood: anyway, I've +2+W'd the supermegasquash | |
| 08:23:16 | elodilles | lyarwood: I also abandoned the ussuri patches that were squashed into the supermegasquash | |
| 08:23:40 | elodilles | just to clean up a bit | |
| 08:40:05 | lyarwood | ganso: ah apologies I see now, the nova-next failure in your stable/ussuri change looks like a known issue with the bionic version of QEMU | |
| 08:40:59 | lyarwood | ganso: we used the zuulv3 change to the live migration jobs to fix this in later releases by moving to Focal but that isn't an option on Ussuri, Train etc. | |
| 08:41:20 | lyarwood | ganso: so tl;dr the zuulv3 changes might not resolve this issue | |
| 08:41:42 | lyarwood | ganso: however we could skip the test if it's failing constantly across the remaining Bionic based branches | |
| 08:42:35 | lyarwood | elodilles: as above I'm not sure that we do need that back on stable/ussuri but at the same time it shouldn't hurt | |
| 08:43:18 | lyarwood | elodilles: the label is defined in the parent tempest job and everything in our job definition should be generic (with the edits I made to the evacuation test role) to run on Bionic | |
| 08:46:34 | elodilles | lyarwood: ok, thanks, meanwhile I came to the same conclusion that it probably doesn't hurt but on the other hand fixes the gate, so i +w'd the patch :) | |
| 08:47:04 | lyarwood | that's the thing, I've not seen an example where it clearly does just yet but hopefully I've missed something | |
| 08:48:00 | lyarwood | the paused instance live migration failure is a known bionic issue and something that we saw back during Victoria that prompted us to move the jobs to Focal as part of the zuulv3 migration | |
| 09:38:38 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Avoid calling detach with src connection_info during LM rollback https://review.opendev.org/c/openstack/nova/+/797725 | |
| 09:42:19 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Avoid calling detach with src connection_info during LM rollback https://review.opendev.org/c/openstack/nova/+/797725 | |
| 10:14:21 | opendevreview | Lee Yarwood proposed openstack/nova master: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/790519 | |
| 10:20:20 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Create qcow2 disks with the correct size without extending https://review.opendev.org/c/openstack/nova/+/779275 | |
| 11:24:58 | opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797878 | |
| 11:24:59 | opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797879 | |
| 11:35:20 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797880 | |
| 11:35:21 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797881 | |
| 13:23:15 | ganso | lyarwood: thanks for the response! I've rechecked your patch. failed on grenade job | |
| 13:49:13 | opendevreview | Elod Illes proposed openstack/nova stable/queens: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/c/openstack/nova/+/751249 | |
| 16:10:07 | sean-k-mooney | stephenfin: by the way do we want to have comments enabeld in the github mirror? | |
| 16:10:23 | stephenfin | wdym? | |
| 16:10:27 | sean-k-mooney | stephenfin: im not sure how you saw that tread intitally | |
| 16:10:41 | sean-k-mooney | stephenfin: they way we were talking about the client change via github | |
| 16:10:57 | sean-k-mooney | stephenfin: i was wondering if we should talk to infra and see if we can disable that | |
| 16:11:06 | stephenfin | ahh, gotcha | |
| 16:11:13 | stephenfin | I don't think you can disable that | |
| 16:11:31 | stephenfin | and I don't mind, really. I could always ignore the request if I wanted to | |
| 16:13:08 | sean-k-mooney | you can disable Discussions | |
| 16:13:13 | sean-k-mooney | im not sure if that is what that was | |
| 16:13:40 | sean-k-mooney | no not that | |
| 16:18:30 | sean-k-mooney | stephenfin: ya i think its part of the code review feature and cant be disable | |
| 16:18:52 | sean-k-mooney | it was more signeling to people that we do not use it for comunication | |
| 16:20:05 | stephenfin | eh, like I said, I don't think it's a huge deal. I'm happy to be pinged that way. I'll just prod them to other channels | |
| 16:20:43 | opendevreview | Stephen Finucane proposed openstack/nova stable/stein: libvirt: Start checking compute usage in functional tests https://review.opendev.org/c/openstack/nova/+/797974 | |
| 16:20:45 | opendevreview | Stephen Finucane proposed openstack/nova stable/stein: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797975 | |
| 16:20:46 | opendevreview | Stephen Finucane proposed openstack/nova stable/stein: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797976 | |
| 16:21:03 | stephenfin | oh, that first patch wasn't supposed to be there | |
| 16:21:08 | sean-k-mooney | yep i just think i have my github notificaiton turned off | |
| 16:21:29 | sean-k-mooney | well no i have them on by they go to my personal email | |
| 16:21:43 | sean-k-mooney | so ill see it eventually | |
| 16:23:59 | opendevreview | Stephen Finucane proposed openstack/nova stable/stein: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797975 | |
| 16:24:00 | opendevreview | Stephen Finucane proposed openstack/nova stable/stein: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797976 | |
| 16:47:05 | opendevreview | Stephen Finucane proposed openstack/nova stable/rocky: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797981 | |
| 16:47:06 | opendevreview | Stephen Finucane proposed openstack/nova stable/rocky: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797982 | |
| 16:57:57 | opendevreview | Stephen Finucane proposed openstack/nova stable/queens: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/797987 | |
| 16:57:58 | opendevreview | Stephen Finucane proposed openstack/nova stable/queens: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/797988 | |
| 17:49:47 | sean-k-mooney | gmann: i should proably know this but is there a config option or something to disable the nova legacy api i.e. v2.0 | |
| 17:58:29 | sean-k-mooney | gmann: i just removed the endpoint definitions form keystone so that will more or less have the same effect so dont worry about it | |
| 21:29:48 | gmann | sean-k-mooney: only via api-paste.ini otherwise there is no config option as such | |
| #openstack-nova - 2021-06-25 | |||
| 02:39:58 | opendevreview | Merged openstack/nova stable/victoria: rbd: Get rbd_utils unit tests running again https://review.opendev.org/c/openstack/nova/+/794624 | |
| 02:40:12 | opendevreview | Merged openstack/nova stable/victoria: Initialize global data separately and run_once in WSGI app init https://review.opendev.org/c/openstack/nova/+/785059 | |
| 03:30:44 | opendevreview | Merged openstack/nova stable/train: Add missing exception https://review.opendev.org/c/openstack/nova/+/763393 | |
| 03:55:16 | sean-k-mooney | gmann: thanks i just did not want it showing up in my endpoint list so i just delete the endpoint and tweaked my gloabl.yaml so that kolla would not recreate it | |
| 05:15:29 | opendevreview | sean mooney proposed openstack/os-vif master: update os-vif ci to account for devstack default changes https://review.opendev.org/c/openstack/os-vif/+/798038 | |
| 05:16:35 | opendevreview | sean mooney proposed openstack/os-vif master: update os-vif ci to account for devstack default changes https://review.opendev.org/c/openstack/os-vif/+/798038 | |
| 05:19:03 | opendevreview | sean mooney proposed openstack/os-vif master: update os-vif ci to account for devstack default changes https://review.opendev.org/c/openstack/os-vif/+/798038 | |
| 06:28:43 | opendevreview | Jorhson Deng proposed openstack/nova master: recheck the attachment_id after the reschedule successful https://review.opendev.org/c/openstack/nova/+/796209 | |
| 07:13:30 | opendevreview | Jorhson Deng proposed openstack/nova master: To deal instance with soft-deleting in _init_instance https://review.opendev.org/c/openstack/nova/+/761264 | |
| 07:16:20 | opendevreview | Jorhson Deng proposed openstack/nova master: To deal instance with soft-deleting in _init_instance https://review.opendev.org/c/openstack/nova/+/761264 | |
| 07:49:53 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 07:49:54 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 07:49:55 | opendevreview | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 07:49:56 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 07:49:57 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 07:49:57 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 07:55:22 | opendevreview | Jorhson Deng proposed openstack/nova master: process instance in soft-deleting task state during _init_instance https://review.opendev.org/c/openstack/nova/+/761264 | |
| 07:57:39 | opendevreview | Jorhson Deng proposed openstack/nova master: process instance in soft-deleting task state during _init_instance https://review.opendev.org/c/openstack/nova/+/761264 | |
| 07:57:49 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 08:06:18 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 08:06:22 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 08:06:28 | opendevreview | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 08:06:33 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 08:06:44 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 08:06:55 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 08:08:30 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 09:00:05 | opendevreview | Lee Yarwood proposed openstack/nova master: compute: Avoid calling detach with src connection_info during LM rollback https://review.opendev.org/c/openstack/nova/+/797725 | |
| 09:44:11 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [CI] Fix gate by using zuulv3 live migration and grenade jobs https://review.opendev.org/c/openstack/nova/+/795432 | |
| 09:48:48 | elodilles | lyarwood: as the ussuri gate fixing supermegasquash did not pass I decided to update the commit message (maybe it's a bit easier to track what happened in it) | |
| 09:49:00 | opendevreview | Jorhson Deng proposed openstack/nova master: process instance in soft-deleting task state during _init_instance https://review.opendev.org/c/openstack/nova/+/761264 | |
| 09:49:32 | elodilles | lyarwood: it's really just a commit message update. so if you will have time then please review it o:) | |
| 09:49:36 | elodilles | melwitt: ^^^ | |
| 10:08:43 | lyarwood | elodilles: ack will do, melwitt is out this week btw | |
| 10:08:44 | stephenfin | elodilles: mel is out all this week | |
| 10:08:49 | stephenfin | jinx | |
| 10:10:18 | stephenfin | gibi: Can we update the os-vif stable core team to include os-vif core now? We discussed during the team meeting a few weeks back and had no disagreements from what I can recall | |
| 10:10:34 | stephenfin | assuming you have the power to do so | |
| 10:14:05 | gibi | stephenfin: let me see | |