Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-09-24
14:46:45 lpetrut yeah, it's hard to label it as a bug in order to allow backports but that's ok. a release note makes sense, I can also mention the fact that uwsgi may intercept SIGUSR2, in which case a file trigger may be configured
14:47:35 sean-k-mooney lpetrut: ya if you add a release note and maybe add a doc for the intercept i would be +1 on it
14:48:07 lpetrut awesome, thanks. is there a specific doc that you have in mind?
14:48:28 sean-k-mooney we have a doc for GMR i think in the contibutor section
14:49:17 sean-k-mooney https://github.com/openstack/nova/blob/50fdbc752a9ca9c31488140ef2997ed59d861a41/doc/source/reference/gmr.rst
14:49:20 sean-k-mooney ah its in refernce
14:49:44 sean-k-mooney can you update https://github.com/openstack/nova/blob/master/doc/source/reference/gmr.rst#generating-a-gmr
14:50:07 sean-k-mooney with the exmaple of the file trigger
14:50:14 lpetrut definitely, thanks for the link
14:51:00 lpetrut the "TextGuruMeditation.setup_autorun(version)" hook sample should also be updated. if we don't pass the config, gmr will not be aware of the [oslo_reports] config opts
14:56:32 sean-k-mooney ya i dont think we have really updated it since it was added
14:57:26 sean-k-mooney i would suggest updating the existig singal based exmaple to use nova-compute and then adding the file example for nova-api and makeign any other changes that you think are needed
14:59:54 lpetrut sounds good
15:15:07 bauzas gibi: ack, so we need to discuss this during the PTG
15:15:52 gibi yepp
15:15:57 gibi added notes to the bad
15:16:00 gibi pad
15:16:29 lpetrut one minor nit: gmr.setup_autorun takes a "service_name" parameter which is used when constructing the report filename. when missing, it's trying to retrieve it from the stack trace but it seems to always end up with "thread.py", so the reports are named something like "thread.py_gurumeditation_20210924141722". since none of the other service pass this parameter, I'm thinking about doing the same for nova-uwsgi for consistency reasons.
15:19:54 sean-k-mooney lpetrut: or you could fix them all
15:20:05 sean-k-mooney lpetrut: you should be able to use the service binary name
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?

Earlier   Later