| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-24 | |||
| 15:20:29 | sean-k-mooney | so service_obj.binary | |
| 15:20:33 | lpetrut | that works as well | |
| 15:21:16 | sean-k-mooney | that way if you have multiple service on the same host using the same file tirrger they wont overright | |
| 15:21:31 | sean-k-mooney | although the timestamp is unlikely to collide in anycase | |
| 15:24:03 | lpetrut | right, it's also more user friendly since it's easier to tell which is the originating service | |
| 15:25:24 | sean-k-mooney | well when not using the file backend it dumps to the service log so that not been an issue before but for dumping the GMR to a file its something we shoudl definetly address | |
| 15:41:31 | gibi | sean-k-mooney: btw, I tried your echo 0 > numa_node trick and it works like a charm. I can now confirm that live migration with SRIOV + NUMA works and the numa topology is properly recalculated | |
| 15:44:45 | sean-k-mooney | thats an old trick i have been using for ever | |
| 15:45:57 | gibi | this knowledge is gold | |
| 15:48:47 | sean-k-mooney | if you ignore the horrible hack that it is you can actully write udev rules to allwo you to assocaite the device with other numa nodes on the same socket if you enabel cluster on die or amds numa_per_socket>1 | |
| 15:48:47 | opendevreview | Lucian Petrut proposed openstack/nova master: api: enable oslo.reports when using uwsgi https://review.opendev.org/c/openstack/nova/+/810922 | |
| 15:55:04 | sean-k-mooney | ok going to finish there o/ talk to ye in a week | |
| 16:36:32 | fungi | elodilles: has there been any progress on discussions of whether to discontinue lower-constraints jobs on nova's stable/ussuri branch? at this point, nothing (including an outstanding security fix) can merge there, so it's probably time to start talking about early eol instead | |
| 16:37:44 | fungi | at least if we're up front with users that we're no longer fixing known vulnerabilities on that branch, the vmt can go forward with announcing fixes on the nova branches which are still receiving patches | |
| 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 | |