Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-03-25
15:34:08 sean-k-mooney ill make this configuable in a env var
15:34:46 sean-k-mooney tldr set this to false https://github.com/openstack/placement/blob/master/placement/tests/functional/fixtures/capture.py#L55
15:35:12 dansmith WAT
15:35:16 dansmith set to false?
15:35:35 sean-k-mooney yep to turn off the log capture and allow it to print to stderr
15:35:36 dansmith I have commented out the usage of that thing entirely and I still don't get logs
15:36:09 dansmith do you also have to disable the stdio capture?
15:36:10 sean-k-mooney i also invoked it liek this wiht a few other changes
15:36:12 sean-k-mooney OS_DEBUG=1 OS_LOG_CAPTURE=1 tox -e functional -- --failing
15:36:43 dansmith okay are you going to push up a change so I can see what all?
15:36:49 sean-k-mooney yep
15:37:03 dansmith aight
15:37:22 sean-k-mooney i mean its just this https://termbin.com/gj5q
15:37:39 sean-k-mooney but not all of that is relevent
15:38:11 sean-k-mooney ill past you the output so you can review
15:38:19 sean-k-mooney then ill figure out what bits actully matter
15:39:44 sean-k-mooney https://termbin.com/zacx
15:40:40 sean-k-mooney File "/home/smooney/repos/placement/placement/objects/allocation.py", line 262, in _check_capacity_exceeded
15:40:41 sean-k-mooney prev_used = prev_usage[alloc.resource_provider.id][rc_id]
15:40:43 sean-k-mooney ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^
15:40:43 opendevreview Dan Smith proposed openstack/placement master: Fix placement allocate while over-capacity https://review.opendev.org/c/openstack/placement/+/945465
15:40:43 opendevreview Dan Smith proposed openstack/placement master: Reproduce bug 2104040: allocate in over-capacity https://review.opendev.org/c/openstack/placement/+/945464
15:40:45 sean-k-mooney KeyError: 0
15:41:34 dansmith OS_LOG_CAPTURE=1 sets capture_logs=True on oslo right?
15:41:55 sean-k-mooney i need to see if thats actully needed but proably
15:42:17 dansmith very confusing if we need to set that on the parent *and* =False on the child.. I suspect one of those and not the other is to blame
15:42:36 sean-k-mooney ok so that is not relevnet
15:42:42 sean-k-mooney i can set it to 0 and it still work
15:42:48 sean-k-mooney i was gong to use that to contole it however
15:43:27 dansmith okay, isn't that capture_logs flag telling oslo to capture the logs in memory so they can be reported with the test failure?
15:43:29 sean-k-mooney so instead of hardcoding this to true defualt that to 0 or 1 i dont know what makes the most sense and allow you to override it
15:43:43 dansmith I'm not sure why we'd ever want that to be false unless the reporting is wrong
15:45:27 sean-k-mooney i need to read the related code to answer properly but this is the relevnet code https://github.com/openstack/oslotest/blob/1fe445863e124f8a10c1496fb8244c745e546b21/oslotest/log.py#L35-L91
15:46:13 sean-k-mooney this is why i have not pushed a patch yet im trying to load context and then figurout what the correct behavior should be before commiting and pushing
15:47:36 Uggla nova meeting in aound 15mn.
15:48:03 sean-k-mooney dansmith: by hardcodign it to ture we were alsways addign the fake loonger
15:48:25 sean-k-mooney which make shte logs aviable in the tests
15:48:37 dansmith right, but that's the intent so it gets reported to the test runner so the runner can report the test fail with logs right?
15:48:39 sean-k-mooney btu im not sure if that will attach them to the tests to print on output
15:48:47 dansmith yeah, idk
15:49:03 dansmith so many layers and years of crust, I'm not even sure how it's supposed to work
15:49:20 sean-k-mooney since we know what to flip we can easly expose that bool as an env var in any case
15:51:43 sean-k-mooney dansmith: so its wrapping a logging fixture form fixtures and nuke_handeler is set to true
15:51:45 sean-k-mooney https://github.com/testing-cabal/fixtures/blob/master/fixtures/_fixtures/logger.py#L39-L40
15:52:03 sean-k-mooney which prevent it going to stdout
15:53:15 sean-k-mooney dansmith: i have not trast all the code it possibel we ar attachign the logs as an addtional attachment to the test
15:53:27 dansmith right, but I thought things like testr were doing their own capturing of the logs, as long as they don't get swallowed so they can be reported per-test (in the "logging" section)
15:53:28 sean-k-mooney there is an stester flag to print addtional attachments
15:53:37 dansmith right, unittest does that with addSection() or whatever
15:53:54 sean-k-mooney ya let me see if passing that flag helps
15:54:52 sean-k-mooney oh that just for succesful test "--all-attachments"
15:55:00 dansmith yeah
15:56:03 opendevreview Merged openstack/nova master: Fix case sensitive comparison https://review.opendev.org/c/openstack/nova/+/933107
15:57:28 opendevreview Merged openstack/nova master: Fix case-sensitivity for metadata keys https://review.opendev.org/c/openstack/nova/+/873901
15:57:36 opendevreview Merged openstack/nova master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/nova/+/944890
16:00:20 opendevmeet The meeting name has been set to 'nova'
16:00:20 opendevmeet Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:20 opendevmeet Meeting started Tue Mar 25 16:00:20 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:20 Uggla #startmeeting nova
16:00:42 Uggla Hello everyone
16:01:41 masahito o/
16:02:02 elodilles o/
16:02:24 gibi o/
16:02:35 Uggla awaiting a moment for people to join.
16:02:43 dansmith o/
16:03:24 bauzas \o (in another meeting)
16:03:45 Uggla #topic Bugs (stuck/critical)
16:03:54 Uggla #info No Critical bug
16:04:02 Uggla #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster
16:04:34 Uggla anything about bugs ?
16:05:30 Uggla #topic Gate status
16:06:06 Uggla #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs
16:06:14 Uggla #link https://etherpad.opendev.org/p/nova-ci-failures-minimal
16:06:25 Uggla #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status
16:06:32 Uggla #info Please look at the gate failures and file a bug report with the gate-failure tag.
16:06:39 Uggla #info Please try to provide meaningful comment when you recheck
16:06:58 Uggla anything about the gate ?
16:07:46 Uggla seems not.
16:07:57 Uggla #topic Release Planning
16:08:07 Uggla #link https://releases.openstack.org/epoxy/schedule.html
16:08:15 Uggla #info Nova deadlines are set in the above schedule
16:08:21 Uggla #link https://etherpad.opendev.org/p/nova-epoxy-rc-potential
16:08:27 Uggla #link https://releases.openstack.org/flamingo/schedule.html
16:08:52 Uggla #info Remaining post RC1: update min servion version for a SLURP or non-SLURP release : https://review.opendev.org/c/openstack/nova/+/944018/
16:09:11 Uggla #info Nova Flamingo deadlines will be discussed at the PTG.
16:09:46 Uggla We are almost good for Epoxy release.
16:09:48 elodilles note that this week is the week of Final RC for 2025.1 Epoxy :)
16:10:36 Uggla #topic Review priorities
16:10:44 Uggla #info Flamingo priorities will be discussed at the PTG.
16:10:59 Uggla #topic PTG planning
16:11:06 Uggla #info Next PTG will be held on Apr 7-11
16:11:11 Uggla #link https://etherpad.opendev.org/p/nova-2025.2-ptg
16:11:45 Uggla Please add the topics you want to discuss in the pad.
16:12:24 Uggla #topic Stable Branches
16:12:30 Uggla #info stable gates seem to be healthy
16:12:41 Uggla #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci
16:13:03 Uggla elodilles, if you want to add something please go ahead.
16:13:14 elodilles yepp, one thing maybe:
16:13:17 elodilles #info stable/2025.1 branch was cut for nova as well last week
16:13:24 elodilles that's all from me

Earlier   Later