| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-24 | |||
| 17:08:24 | opendevreview | Rodolfo Alonso proposed openstack/nova master: Set "cache_ok=True" in "TypeDecorator" inheriting classes https://review.opendev.org/c/openstack/nova/+/807359 | |
| 17:58:17 | opendevreview | Artom Lifshitz proposed openstack/nova master: Gracefully power off guest on instance delete https://review.opendev.org/c/openstack/nova/+/808474 | |
| 17:58:17 | opendevreview | Artom Lifshitz proposed openstack/nova master: "Regression" test for server delete https://review.opendev.org/c/openstack/nova/+/810951 | |
| 17:58:18 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Make os_shutdown_timer a proper image property https://review.opendev.org/c/openstack/nova/+/810952 | |
| 18:02:57 | opendevreview | Artom Lifshitz proposed openstack/nova master: "Regression" test for server delete https://review.opendev.org/c/openstack/nova/+/810951 | |
| 18:02:57 | opendevreview | Artom Lifshitz proposed openstack/nova master: Gracefully power off guest on instance delete https://review.opendev.org/c/openstack/nova/+/808474 | |
| 18:02:58 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Make os_shutdown_timer a proper image property https://review.opendev.org/c/openstack/nova/+/810952 | |
| 18:57:41 | elodilles | fungi: there is a patch that sets the lower-constraints job non-voting, but unfortunately it needed an impressive amount of rechecks and it is still not merged so far ( https://review.opendev.org/c/openstack/nova/+/809955 ) | |
| 18:59:06 | elodilles | also there is another patch from gibi that pins the setuptools ( https://review.opendev.org/c/openstack/nova/+/810461 ) | |
| 19:04:22 | fungi | note that the lower-constraints job was already not passing on stable/ussuri when setuptools updated | |
| 19:12:41 | elodilles | how do you mean? | |
| 19:45:32 | elodilles | what I remember is that the gate was not blocked until the gate started to use setuptools 58.0.4. | |
| 19:45:50 | elodilles | even lower-constraints jobs were passing | |
| 19:48:23 | elodilles | and the passing setuptools pinning patch also proves that there is no other issue (for now) than the one that was introduced by the latest setuptools release | |
| 20:06:53 | fungi | elodilles: oh, you're right, we got setuptools 58 in tox jobs when virtualenv 20.8.0 was released on 2021-09-16 at 12:52 utc and change 806628 was unlucky enough to be approved a few hours later at 16:23 utc that same day | |
| 20:10:16 | fungi | change 810461 could probably instead do requires = virtualenv<20.8 if you wanted to be more flexible | |
| #openstack-nova - 2021-09-25 | |||
| 05:40:57 | opendevreview | Lucian Petrut proposed openstack/nova master: api: enable oslo.reports when using uwsgi https://review.opendev.org/c/openstack/nova/+/810922 | |
| 05:50:43 | opendevreview | norman shen proposed openstack/nova master: Recreate mdev devices according to placement https://review.opendev.org/c/openstack/nova/+/810220 | |
| 06:02:13 | opendevreview | norman shen proposed openstack/nova master: Recreate mdev devices according to placement https://review.opendev.org/c/openstack/nova/+/810220 | |
| 09:11:57 | alecorps__ | hello guys | |
| 09:12:05 | alecorps__ | I'm looking to have a review on https://review.opendev.org/c/openstack/nova/+/808791 | |
| 09:12:11 | alecorps__ | Is someone could tell me please how to proceed ? | |
| 09:58:06 | opendevreview | norman shen proposed openstack/nova master: Recreate mdev devices according to placement https://review.opendev.org/c/openstack/nova/+/810220 | |
| #openstack-nova - 2021-09-26 | |||
| 07:24:26 | opendevreview | wang proposed openstack/nova master: Add armv6l to arch list https://review.opendev.org/c/openstack/nova/+/799964 | |
| 11:42:59 | opendevreview | Merged openstack/nova stable/ussuri: [stable-only] Set lower-constraints job as non-voting https://review.opendev.org/c/openstack/nova/+/809955 | |
| 12:04:37 | opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | |
| 12:05:48 | opendevreview | Federico Ressi proposed openstack/nova master: Check Nova project changes with Tobiko scenario test cases https://review.opendev.org/c/openstack/nova/+/806853 | |
| 16:05:31 | opendevreview | Merged openstack/nova stable/ussuri: address open redirect with 3 forward slashes https://review.opendev.org/c/openstack/nova/+/806628 | |
| #openstack-nova - 2021-09-27 | |||
| 06:33:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: [stable-only][WIP] Pin virtualenv and setuptools https://review.opendev.org/c/openstack/nova/+/810461 | |
| 06:34:25 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: [stable-only]Pin virtualenv and setuptools https://review.opendev.org/c/openstack/nova/+/810461 | |
| 06:34:46 | gibi | fungi, elodilles: made this more flexible ^^ | |
| 07:38:22 | gibi | bauzas: I'm going to propose the RC2 patches for both nova and placement | |
| 07:38:33 | bauzas | good morning Noba | |
| 07:38:36 | bauzas | Nova even ;) | |
| 07:38:38 | bauzas | gibi: ack, ok | |
| 07:38:48 | bauzas | I was thinking about creating them for tomorrow | |
| 07:39:13 | gibi | I will do it now, and we can hold it until tomorrow's meeting | |
| 07:56:58 | bauzas | ++ | |
| 07:57:46 | bauzas | gibi: I will also ask if people want to mentor for Outreachy | |
| 07:58:05 | bauzas | the deadline being pushed until Wed, people can do this | |
| 09:20:43 | opendevreview | Stephen Finucane proposed openstack/nova master: tools: Ignore bot-generated branch creation patches https://review.opendev.org/c/openstack/nova/+/810285 | |
| 10:03:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810909 | |
| 10:43:27 | lyarwood | gibi: https://zuul.opendev.org/t/openstack/build/b607db4018bb4753839d6fd1202322b4/log/controller/logs/screen-q-svc.txt#29241 - nova-grenade-multinode appears to be failing 100% of the time due to a missing extension in Neutron, any ideas if anything has changed recently? | |
| 10:43:38 | gibi | lyarwood: looking | |
| 10:52:16 | gibi | lyarwood: this feels relevant https://review.opendev.org/c/openstack/neutron/+/809550 | |
| 10:52:44 | gibi | the revert landed ~1 day ago | |
| 10:53:04 | gibi | I have a grenade job running on master as we speak so we will see if the problem is still exists | |
| 10:53:55 | gibi | https://zuul.opendev.org/t/openstack/status#810909 | |
| 10:53:58 | lyarwood | ack thanks | |
| 10:54:23 | lyarwood | it failed right at the end after the upgrade to master for me | |
| 10:54:33 | lyarwood | likely before that revert landed | |
| 11:10:16 | gibi | lyarwood: https://zuul.opendev.org/t/openstack/status#810909 still fails with trunks 404 :/ so there is something else | |
| 11:11:15 | gibi | so far I only found one small change that is in openstack/grenade switching to test upgrade from xena -> master instead of wallaby -> master | |
| 11:48:27 | gibi | lyarwood: quick summary. It seems devstack decides to generate [network-feature-enabled]api_extensions = all to the tempest.conf but the trunk extenison is not in neutrons [DEFAULT]service_plugins config | |
| 11:48:43 | lyarwood | nice | |
| 11:49:06 | gibi | on stable/wallaby the [network-feature-enabled]api_extensions did listed extensions explicity, but did not list the trunk extension so greande skips the trunk testing | |
| 11:50:00 | gibi | on master nova-next has api_extensions=all but trunk is in the service_plugins list | |
| 11:50:08 | gibi | so trunk passing there | |
| 12:01:32 | lyarwood | enable_service neutron-trunk is missing from the job | |
| 12:01:39 | lyarwood | I think that's it | |
| 12:02:09 | gibi | lyarwood: but how that got dropped? | |
| 12:02:40 | gibi | we did not run trunk on stable/wallaby in grenade but now on master we try | |
| 12:03:39 | gibi | I think the problem is not that we forgot to add neutron-trunk on master, but that somehow the trunk testing got enabled in grenade on master | |
| 12:03:49 | gibi | implicitly | |
| 12:04:12 | gibi | but sure one way to fix this is to add neutron-trunk to grenade on master | |
| 12:04:59 | lyarwood | https://github.com/openstack/tempest/blob/ae41052a51f5dbb748eb6bf4f23e9145853f4639/tempest/api/compute/admin/test_live_migration.py#L252 yeah there's a skip on the test but I guess the tempest conf has changed? | |
| 12:05:34 | gibi | yepp | |
| 12:05:56 | gibi | in the tempest conf [network-feature-enabled]api_extensions instructs tempest what to run or skip | |
| 12:06:22 | gibi | and that value change from an explicit list without trunk to a simple 'all' value | |
| 12:07:47 | gibi | in our grenade testing | |
| 12:07:59 | gibi | and actually it seems in every testing on mater | |
| 12:08:00 | gibi | master | |
| 12:08:21 | gibi | in nova-next we also get 'all' but in nova-next neutron-trunk is enabled | |
| 12:08:26 | gibi | so the test passes there | |
| 12:08:58 | gibi | bottom line our grenade tempest config got inconsistent as it is configured to run trunk tests without neutron-trunk | |
| 12:10:55 | lyarwood | kk I'll add the service in the parent grenade job | |
| 12:14:13 | gibi | OK, lets unblock the gate with that change | |
| 12:14:35 | gibi | is it enough to merge that to master or we need to backport that to stable/xena as well? | |
| 12:15:36 | lyarwood | that should be enough on master | |
| 12:15:41 | gibi | ack | |
| 12:15:42 | lyarwood | it's the zuul job itself after all | |
| 12:15:55 | lyarwood | https://review.opendev.org/c/openstack/grenade/+/811117 | |
| 12:17:17 | opendevreview | Lee Yarwood proposed openstack/nova master: DNM testing grenade neutron-trunk fix https://review.opendev.org/c/openstack/nova/+/811118 | |
| 12:17:56 | gibi | lyarwood: thanks. | |
| 12:31:18 | lyarwood | ttps://review.opendev.org/q/topic:bug/1943431 is ready for reviews if anyone has time | |
| 12:31:29 | lyarwood | https://review.opendev.org/q/topic:bug/1943431 even | |
| 12:39:46 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810911 | |
| 12:41:03 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810913 | |
| 12:47:04 | spatel | sean-k-mooney does Intel Corporation Ethernet 10G 2P X550-t Adapter support SRIOV ? | |
| 12:47:18 | spatel | its copper 10G | |
| 12:49:52 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/810915 | |
| 12:50:06 | spatel | Its saying but i can't see in lspci -vv command out for any SRIOV reference. let me check in BIOS | |
| 13:08:23 | spatel | sean-k-mooney nevermind, after enable in BIOS > NIC > Config (enable:SRIOV) | |
| 13:08:47 | spatel | i can see SRIOV support in lspci -vv | |
| 14:00:22 | bauzas | gibi: +2d for https://review.opendev.org/c/openstack/nova/+/810461 but I'd like to first tell about this during tomorrow's meeting before merging it | |
| 14:00:55 | bauzas | it looks to me we have a consensus but I'd prefer to see if anyone has a concern | |
| 14:08:50 | gibi | bauzas: sure | |
| 14:09:28 | bauzas | (and I could miss one=) | |
| 14:10:29 | gibi | the third is the stable setuptools fix | |
| 14:10:31 | gibi | I guess :D | |