| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-18 | |||
| 15:54:47 | gmann | stephenfin: lyarwood if that broken on fedora having py3.9 ? but we do have py3.9 job running successfully though those are n-v | |
| 15:56:32 | gmann | is that | |
| 16:22:21 | opendevreview | Merged openstack/nova master: Handle OPERATION_FAILED error during detach https://review.opendev.org/c/openstack/nova/+/796255 | |
| 17:15:25 | stephenfin | lyarwood: Good thing you asked for that test. This code is doing nothing currently 😇 | |
| 17:15:54 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Fix VIFMigrateData.supports_os_vif_delegation setter https://review.opendev.org/c/openstack/nova/+/797142 | |
| 17:15:59 | stephenfin | sean-k-mooney: ^ | |
| 17:16:24 | stephenfin | I haven't run those tests locally. I want to push it to the gate and see if the interfaces are correctly created in the Tempest job | |
| 17:16:53 | stephenfin | gmann: The main issue I was seeing is that the deps in lower-constraints don't work with Python 3.9 | |
| 17:17:20 | stephenfin | gmann: However, in the past the functional tests didn't work with Python 3.9. It could be possible that things have been fixed since | |
| 17:18:05 | stephenfin | gmann: However, regardless, there's no reason we should be running different things locally and in the CI. Something could conceivably pass locally (where we're using Python 3.9) but fail in the gate (using Python 3.8) | |
| 17:18:36 | stephenfin | gibi: done | |
| 17:18:40 | gmann | stephenfin: yeah, l-c can be dropped :) which i am not much worried about. | |
| 17:19:36 | gmann | stephenfin: cases like passing py3.9 and failing py3.8 should not be much right as at next cycle we want all code to run on both | |
| 17:20:12 | gmann | stephenfin: i am not against of that change to test py3.8 as default locally but it just add extra work you mentioned in commit msg | |
| 17:20:18 | stephenfin | Sure, but what about when Fedora introduces Python 3.10 | |
| 17:20:41 | stephenfin | Fedora is bleeding edge, and there will always be a delay between when Fedora introduces a Python version and when nova supports it | |
| 17:21:07 | stephenfin | we already have to update setup.cfg to state our supported versions so this is minimal extra work IMO | |
| 17:21:42 | gmann | yeah that is automated in release script i think and may we can add tox basepython update also.. | |
| 17:22:02 | stephenfin | that would be helpful | |
| 17:25:16 | gmann | stephenfin: we can merge that I am not -1 on that. I will see if we can automate in release script sometime later | |
| 17:27:51 | opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 17:30:27 | opendevreview | Merged openstack/osc-placement master: default to max version when no session https://review.opendev.org/c/openstack/osc-placement/+/794276 | |
| 17:39:00 | opendevreview | Stephen Finucane proposed openstack/nova stable/victoria: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797144 | |
| 17:39:31 | stephenfin | hmm, why can't I leave -W on stable/wallaby? | |
| 17:39:36 | stephenfin | but I can on stable/victoria | |
| 17:48:15 | gmann | stephenfin: because of owner? | |
| 17:49:47 | stephenfin | ohhh | |
| 17:49:51 | stephenfin | yeah, that's it | |
| 17:49:58 | stephenfin | whoops :) | |
| #openstack-nova - 2021-06-19 | |||
| 01:03:59 | opendevreview | Merged openstack/nova master: db: Remove 'nova.db.sqlalchemy.utils' https://review.opendev.org/c/openstack/nova/+/786292 | |
| 01:04:24 | opendevreview | Merged openstack/nova master: db: Reintroduce validation of shadow table schema https://review.opendev.org/c/openstack/nova/+/796814 | |
| 01:04:41 | opendevreview | Merged openstack/nova stable/wallaby: Honor [neutron]http_retries in the manual client https://review.opendev.org/c/openstack/nova/+/794186 | |
| 13:28:32 | opendevreview | Merged openstack/nova stable/wallaby: Stop leaking ceph df cmd in RBD utils https://review.opendev.org/c/openstack/nova/+/791939 | |
| 13:28:50 | opendevreview | Merged openstack/nova master: db: Remove unused DB methods https://review.opendev.org/c/openstack/nova/+/786293 | |
| 18:48:11 | opendevreview | Merged openstack/nova stable/wallaby: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791577 | |
| #openstack-nova - 2021-06-20 | |||
| 15:10:18 | opendevreview | Merged openstack/nova stable/stein: Validate id as integer for os-aggregates https://review.opendev.org/c/openstack/nova/+/764308 | |
| #openstack-nova - 2021-06-21 | |||
| 01:46:34 | opendevreview | jiang guodao proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 01:55:06 | swp20 | gmann: hi, our cyborg-tempest-plugin test failed, seems there are no accelerator resources on the zuul server, please help us: https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/796977 | |
| 03:23:56 | opendevreview | jiang guodao proposed openstack/nova stable/victoria: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787252 | |
| 05:58:29 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 05:58:30 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 05:58:31 | opendevreview | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 05:58:32 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 05:58:33 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 07:09:51 | opendevreview | Eric Xie proposed openstack/nova master: Fix error '404 Not Found' https://review.opendev.org/c/openstack/nova/+/797233 | |
| 08:32:34 | slaweq | elodilles hi, can You +W patch https://review.opendev.org/c/openstack/nova/+/787252 ? It was rebased recently | |
| 08:34:19 | elodilles | slaweq: +W'd | |
| 08:36:41 | elodilles | I hope it really merges now... the gate is in surprisingly bad shape according to the rechecks on this patch :/ | |
| 08:37:55 | elodilles | despite that a number of gate improvements were merged :X | |
| 08:56:15 | lyarwood | sean-k-mooney / gibi / artom / bauzas ; https://review.opendev.org/c/openstack/nova/+/797142 - can you all take a look at this when you get a chance? Blocks the already in progress backport of https://review.opendev.org/q/I11fb5d3ada7f27b39c183157ea73c8b72b4e672e | |
| 08:56:21 | lyarwood | oh and morning btw :) | |
| 08:56:30 | bauzas | fer sur | |
| 08:56:44 | bauzas | and bonjour | |
| 09:27:22 | slaweq | elodilles thx a lot | |
| 09:37:20 | elodilles | slaweq: np | |
| 10:52:28 | stephenfin | sean-k-mooney: It's working for real this time now https://review.opendev.org/c/openstack/nova/+/797142 | |
| 10:53:32 | sean-k-mooney | stephenfin: yep reviewing it again | |
| 10:54:16 | sean-k-mooney | i did not have a chance to test that since i added the detection mechanium in a real mixed env only with the ci | |
| 10:54:35 | sean-k-mooney | its annoying that this sliped though but im about to +1 that | |
| 10:55:16 | stephenfin | lyarwood: I'd rather avoid addressing those nits on ^, since I've already prepared the backports (with a -1 to prevent anyone merging yet) and don't want to drag more work on myself :-D | |
| 10:55:31 | stephenfin | I could do a follow-up but they seem too minor to bother, for once | |
| 10:56:21 | sean-k-mooney | i dont think the new=... | |
| 10:56:24 | sean-k-mooney | is needed | |
| 10:56:31 | sean-k-mooney | its defenetly nice to prefer to do that | |
| 10:56:38 | sean-k-mooney | if the mock is not required in the test | |
| 10:57:04 | sean-k-mooney | but we didnt do that until about a year or so ago because we did not know we could | |
| 10:57:22 | sean-k-mooney | so its not inconsitnet with thte exsiting test just not the optimal way | |
| 11:05:09 | stephenfin | yeah, my thinking too | |
| 11:05:22 | stephenfin | I'd respin if I hadn't already started on the backports, but ... | |
| 11:05:32 | stephenfin | backports appear to be working as expected too | |
| 11:05:45 | stephenfin | I'll tackle stable/ussuri and stable/train after lunch | |
| 11:06:27 | sean-k-mooney | yeah!!! i really wasnt looking forward to debuging that if it didint | |
| 11:07:20 | sean-k-mooney | /yeah/yay/ | |
| 11:22:26 | sean-k-mooney | by the way http://paste.openstack.org/show/806817/ | |
| 11:22:36 | sean-k-mooney | we have q35 testing right? | |
| 11:23:48 | sean-k-mooney | actully this is more userful http://paste.openstack.org/show/806818/ | |
| 11:24:50 | sean-k-mooney | wait i wonder did i uppercase Q35 instead of q35 | |
| 11:41:57 | sean-k-mooney | lyarwood: stephenfin bauzas https://bugs.launchpad.net/nova/+bug/1933097 this is what i get form deploying openstack at home, more work | |
| 12:24:12 | bauzas | sean-k-mooney: *me* facepalms | |
| 12:28:11 | sean-k-mooney | i never write it with an uppercase but this time i did... | |
| 12:37:07 | opendevreview | Merged openstack/nova master: api: Log correct client IP if load balancer in use https://review.opendev.org/c/openstack/nova/+/786766 | |
| 12:50:13 | sean-k-mooney | bauzas: can you take a look at https://review.opendev.org/q/topic:%22bug%252F1910466%22+(status:open%20OR%20status:merged) | |
| 12:50:30 | bauzas | ack, will look | |
| 12:50:42 | sean-k-mooney | we suspect the customer might start escalating that soonish so just trying to get ahead fo that | |
| 13:09:06 | opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797291 | |
| 13:46:34 | lyarwood | stephenfin: yeah no issues | |
| 13:46:50 | lyarwood | sean-k-mooney: haha nice, I did think about that when I was doing the nova-manage stuff that does validate that | |
| 14:11:45 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797316 | |
| 14:13:09 | stephenfin | sean-k-mooney: ^ will wait and see how the stable/ussuri change goes (I can't run all tests locally in my env) but that _should_ be it | |
| 14:17:30 | sean-k-mooney | ok good | |
| 14:17:40 | sean-k-mooney | did you hit any issue with python2 compatiablity | |
| 14:17:53 | sean-k-mooney | or other funcitonal test issue | |
| 14:19:22 | opendevreview | Merged openstack/nova master: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/769601 | |
| 14:22:56 | stephenfin | sean-k-mooney: not that I can tell. The modified tests did pass | |
| 14:23:04 | stephenfin | (I could run those) | |
| 14:23:42 | stephenfin | gibi: Are you around today? Care to look at https://review.opendev.org/c/openstack/nova/+/797142 if so? | |
| 14:23:42 | sean-k-mooney | i was ok cool from thje commit it looks like the conflict were not as bad as i was fearing | |
| 14:24:06 | sean-k-mooney | s/i was// | |
| 16:11:44 | opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797291 | |
| 16:20:11 | opendevreview | Stephen Finucane proposed openstack/nova stable/train: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797316 | |