| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-24 | |||
| 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 | |
| 10:15:26 | elodilles | lyarwood stephenfin : ok, thanks for the info :) | |
| 10:20:16 | gibi | stephenfin: I did not find the meeting where we discussed that change | |
| 10:22:28 | stephenfin | hmm, maybe it was here so | |
| 10:22:36 | gibi | and also I dont see a os-vif stable group in gerrit | |
| 10:22:38 | stephenfin | we started discussing it here https://meetings.opendev.org/irclogs/%23openstack-nova/%23openstack-nova.2021-06-02.log.html#t2021-06-02T12:42:10 | |
| 10:23:41 | stephenfin | we also discussed it here https://meetings.opendev.org/irclogs/%23openstack-nova/%23openstack-nova.2021-06-14.log.html#t2021-06-14T11:09:39 | |
| 10:23:53 | stephenfin | now did we discuss it in the interim :) | |
| 10:24:13 | gibi | OK so it is a bit more complicated | |
| 10:24:30 | gibi | as there is no stable group for os-vif in gerrit | |
| 10:25:00 | gibi | or I dont see it | |
| 10:25:27 | gibi | elodilles: do you know who has approve rights now on os-vif stable? | |
| 10:27:13 | stephenfin | gibi: looks like it's nova-stable-maint https://github.com/openstack/project-config/blob/master/gerrit/acls/openstack/os-vif.config | |
| 10:27:31 | stephenfin | plus stable-maint-core | |
| 10:27:44 | stephenfin | so if we wanted to do this, we'd simply add os-vif-core to that list | |
| 10:27:55 | stephenfin | gibi: will I post to openstack-discuss about this, or do you want to? | |
| 10:30:06 | gibi | stephenfin: go ahead and post this on ML and then I reply that we should do it | |
| 10:30:45 | gibi | can we simply propose the acl change in gerrit review against the linked project config? | |
| 10:31:46 | stephenfin | I think so, yes | |
| 10:32:21 | opendevreview | Lee Yarwood proposed openstack/nova master: DNM: Test libvirtd with increased keepalive_interval https://review.opendev.org/c/openstack/nova/+/798069 | |
| 10:32:43 | gibi | stephenfin: cool, then could you propose the patch too/ | |
| 10:32:44 | gibi | ? | |
| 10:33:00 | stephenfin | make sense. Will do | |
| 10:34:04 | gibi | thanks | |
| 10:38:39 | lyarwood | gibi: ^ the LM with volumes attached failures continue even after the os-vif workaround in devstack, I'm going to see if playing around with the keepalive_interval helps in libvirtd | |
| 10:46:49 | lyarwood | gibi / stephenfin ; https://review.opendev.org/q/topic:nova-centos-8-stream - could I get some reviews on this series, tl;dr we are trying to use more upstream zuul infra downstream and given centos 8 stream is a supported base OS we wanted to get it covered in the upstream gate | |
| 10:47:10 | stephenfin | sure, I'll look shortly | |
| 10:51:50 | lyarwood | ta | |
| 12:10:05 | opendevreview | Merged openstack/nova master: db: Use module-level imports for sqlalchemy https://review.opendev.org/c/openstack/nova/+/786295 | |