opendevreview | Marios Andreou proposed openstack/watcher master: WIP Add prometheus data source for watcher decision engine https://review.opendev.org/c/openstack/watcher/+/934423 | 14:20 |
---|---|---|
opendevreview | sean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet https://review.opendev.org/c/openstack/watcher/+/933890 | 15:19 |
opendevreview | sean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/watcher/+/932963 | 15:19 |
sean-k-mooney | dansmith: 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 with | 15:21 |
dansmith | sean-k-mooney: ack will look in a sec | 15:41 |
sean-k-mooney | dansmith: thanks for looking clsoely at the testing. give me a minute to grab somethign to dring and ill look into why its not running | 16:25 |
sean-k-mooney | my guess is somethign to do with how stestr is configured ot look for tests? | 16:26 |
sean-k-mooney | but that very odd | 16:26 |
dansmith | yeah idk | 16:27 |
sean-k-mooney | The following tests exited without returning a status | 16:36 |
sean-k-mooney | and likely segfaulted or crashed Python: | 16:37 |
sean-k-mooney | * watcher.tests.common.test_scheduling.TestSchedulerMonkeyPatching.test_scheduler_monkey_patched_when_launched | 16:37 |
sean-k-mooney | so ya im goign to just remove that | 16:37 |
sean-k-mooney | my guess is it sometime doe snot exit cleanly when i pass wait=false | 16:37 |
sean-k-mooney | and that breaks the reportign for that worker | 16:37 |
sean-k-mooney | i didnt see that runign it locally when filtering but each test was runign in its own worker | 16:38 |
opendevreview | sean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet https://review.opendev.org/c/openstack/watcher/+/933890 | 16:49 |
opendevreview | sean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/watcher/+/932963 | 16:49 |
sean-k-mooney | dansmith: ok locally that seams ot work | 16:50 |
sean-k-mooney | there is still room for improment but we wenn form 86% to 89% | 17:01 |
sean-k-mooney | im going to do one more rebase to resolve the commit message comment and add moen more test but ill likely finish soon after that | 17:01 |
sean-k-mooney | dansmith: enjory your time off if we dont chat before then | 17:02 |
dansmith | oh, that's why it wasn't showing as run because it was sigsegv? | 17:02 |
sean-k-mooney | ya i think it killed the process in a way that it was not included in the report | 17:02 |
sean-k-mooney | but somehowe tox still tought it passed | 17:02 |
dansmith | hrm, okay | 17:02 |
dansmith | are you rebasing again or is this one going to run to completion? | 17:03 |
sean-k-mooney | it should run to completeion i was just goign to adress the typo in the commit | 17:04 |
sean-k-mooney | i can wait or just not | 17:04 |
dansmith | I say go ahead and fix it to restart the timer, | 17:04 |
dansmith | then I'll circle back and hopefully it'll be good | 17:04 |
sean-k-mooney | ack, thanks | 17:04 |
opendevreview | sean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet https://review.opendev.org/c/openstack/watcher/+/933890 | 17:09 |
opendevreview | sean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/watcher/+/932963 | 17:09 |
sean-k-mooney | sigh.... 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-mooney | i 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 |
opendevreview | sean mooney proposed openstack/watcher master: Fix incompatiablity between apscheduler and eventlet https://review.opendev.org/c/openstack/watcher/+/933890 | 17:21 |
opendevreview | sean mooney proposed openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/watcher/+/932963 | 17:21 |
sean-k-mooney | ok unless that fails or i am asks to change something im going to leave that run to completion | 17:21 |
*** bryanfraschetti63 is now known as bryanfraschetti6 | 22:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!