Friday, 2024-11-22

opendevreviewMarios Andreou proposed openstack/watcher master: WIP Add prometheus data source for watcher decision engine  https://review.opendev.org/c/openstack/watcher/+/93442314:20
opendevreviewsean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet  https://review.opendev.org/c/openstack/watcher/+/93389015:19
opendevreviewsean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime  https://review.opendev.org/c/openstack/watcher/+/93296315:19
sean-k-mooneydansmith: i added some testign to the fix patch. i unfortunetly was not able to create a clean repoducer but thats the best i could come up with15:21
dansmithsean-k-mooney: ack will look in a sec15:41
sean-k-mooneydansmith: thanks for looking clsoely at the testing. give me a minute to grab somethign to dring and ill look into why its not running16:25
sean-k-mooneymy guess is somethign to do with how stestr is configured ot look for tests?16:26
sean-k-mooneybut that very odd16:26
dansmithyeah idk16:27
sean-k-mooneyThe following tests exited without returning a status16:36
sean-k-mooneyand likely segfaulted or crashed Python:16:37
sean-k-mooney        * watcher.tests.common.test_scheduling.TestSchedulerMonkeyPatching.test_scheduler_monkey_patched_when_launched16:37
sean-k-mooneyso ya im goign to just remove that16:37
sean-k-mooneymy guess is it sometime doe snot exit cleanly when i pass wait=false16:37
sean-k-mooneyand that breaks the reportign for that worker16:37
sean-k-mooneyi didnt see that runign it locally when filtering but each test was runign in its own worker16:38
opendevreviewsean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet  https://review.opendev.org/c/openstack/watcher/+/93389016:49
opendevreviewsean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime  https://review.opendev.org/c/openstack/watcher/+/93296316:49
sean-k-mooneydansmith: ok locally that seams ot work16:50
sean-k-mooneythere is still room for improment but we wenn form 86% to 89%17:01
sean-k-mooneyim going to do one more rebase to resolve the commit message comment and add moen more test but ill likely finish soon after that17:01
sean-k-mooneydansmith: enjory your time off if we dont chat before then17:02
dansmithoh, that's why it wasn't showing as run because it was sigsegv?17:02
sean-k-mooneyya i think it killed the process in a way that it was not included in the report17:02
sean-k-mooneybut somehowe tox still tought it passed17:02
dansmithhrm, okay17:02
dansmithare you rebasing again or is this one going to run to completion?17:03
sean-k-mooneyit should run to completeion i was just goign to adress the typo in the commit17:04
sean-k-mooneyi can wait or just not17:04
dansmithI say go ahead and fix it to restart the timer,17:04
dansmiththen I'll circle back and hopefully it'll be good17:04
sean-k-mooneyack, thanks17:04
opendevreviewsean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet  https://review.opendev.org/c/openstack/watcher/+/93389017:09
opendevreviewsean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime  https://review.opendev.org/c/openstack/watcher/+/93296317:09
sean-k-mooneysigh.... under load the test can race on the job being requeue and it being cancled which only show up if you have enough load...17:19
sean-k-mooneyi can just set the job interval to 2 seconds instead of a 10th of a seocnd to mitigate that but i need to respine so ill do that now...17:19
opendevreviewsean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet  https://review.opendev.org/c/openstack/watcher/+/93389017:21
opendevreviewsean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime  https://review.opendev.org/c/openstack/watcher/+/93296317:21
sean-k-mooneyok unless that fails or i am asks to change something im going to leave that run to completion17:21
*** bryanfraschetti63 is now known as bryanfraschetti622:45

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!