| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 964694: Replace slash when normalizing metrics in launcher https://review.opendev.org/c/zuul/zuul/+/964694 | 07:15 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 964694: Replace slash when normalizing metrics in launcher https://review.opendev.org/c/zuul/zuul/+/964694 | 07:16 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 965097: Pin Ansible google-api-core dependency to <2.28.0 https://review.opendev.org/c/zuul/zuul/+/965097 | 11:56 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 965097: Pin google-api-core dependency to <2.28.0 https://review.opendev.org/c/zuul/zuul/+/965097 | 12:03 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 965097: Pin google-api-core dependency to <2.28.0 https://review.opendev.org/c/zuul/zuul/+/965097 | 14:27 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 965117: Re-enable upgrade job https://review.opendev.org/c/zuul/zuul/+/965117 | 14:27 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 964893: Fix missing update in node reassignment https://review.opendev.org/c/zuul/zuul/+/964893 | 14:27 | |
| @clarkb:matrix.org | the failure here https://2457b8f5cd310a5219de-e44509a4e2cdd0df87d44b1b5e050fc0.ssl.cf1.rackcdn.com/zuul/ff6a7884036547a7a5f75e95ed80defe/testr_results.html is going to prevent the google-api-core fix from merging (and the launcher fix that depends on that) | 16:06 |
|---|---|---|
| @clarkb:matrix.org | looks like a kazoo bad version error | 16:07 |
| @clarkb:matrix.org | is that a concurrency issue with writing to the db? | 16:07 |
| @jim:acmegating.com | Clark: yes, i'll look into it to make sure it's not a real issue, but given the name of the test, there's a good chance it's a test sequencing issue. | 16:54 |
| @clarkb:matrix.org | ah. I've gone ahead and rechecked the changes for now trying to get lucky and have them merge | 16:54 |
| @jim:acmegating.com | Clark: i've got test-only fix for that locally, just running it through a test battery to make sure it statistically improves the pass rate. | 17:12 |
| @clarkb:matrix.org | I'll review it when ready | 17:15 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 965165: Fix BadVersionError in static tests https://review.opendev.org/c/zuul/zuul/+/965165 | 17:27 | |
| @jim:acmegating.com | Clark: ^ made the same fix to a bunch of tests just in case. | 17:27 |
| @clarkb:matrix.org | +2 thanks. Not sure if we need to squash that with the google package fix to get things stable. We shall see if the current gate run for the google package fix is happy | 17:31 |
| @jim:acmegating.com | 17:52 | |
| #15 5.656 error Error: https://registry.yarnpkg.com/@softwarefactory-project/re-ansi/-/re-ansi-0.7.4.tgz: Request failed "500 Internal Server Error" | ||
| @jim:acmegating.com | Clark: ^ it is going to be one of those days | 17:53 |
| @clarkb:matrix.org | when it rains it pours. I just got the estimate from the mechanic after car engine light lit up yesterday too... | 17:53 |
| @fungicide:matrix.org | oof | 17:58 |
| @fungicide:matrix.org | so not anything simple | 17:58 |
| @clarkb:matrix.org | transmission valve control body needs replacing. Our specific model has a weird extended warranty situation on the transmission specifically and we're trying to sort out now if this is covered or not | 18:01 |
| @clarkb:matrix.org | they sent us a letter years ago about this that I doubt I can find today. But its probably not a good sign when the manufacturer says out of the blue the warranty is being extended on a particular part of the vehicle | 18:01 |
| @fungicide:matrix.org | yeah, we have a 2019 outback that has a similar situation with a class action warranty extension | 18:03 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 965097: Pin google-api-core dependency to <2.28.0 https://review.opendev.org/c/zuul/zuul/+/965097 | 19:17 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 965165: Fix BadVersionError in static tests https://review.opendev.org/c/zuul/zuul/+/965165 | 20:43 | |
| @clarkb:matrix.org | the launcher fix hit the yarn/npm request errors so I've rechecked the remaining two changes | 20:49 |
| @clarkb:matrix.org | hopefully with those two other fixes in place the error rate is low enough that we're able to land these changes next | 20:49 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 964807: WIP: Switch quickstart to zuul-launcher https://review.opendev.org/c/zuul/zuul/+/964807 | 21:41 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 965117: Re-enable upgrade job https://review.opendev.org/c/zuul/zuul/+/965117 | 22:29 | |
| @clarkb:matrix.org | corvus: the current run of the launcher test failed in tests.unit.test_launcher.TestLauncher.test_failed_node_reassignment it hasn't reported yet though | 22:48 |
| @clarkb:matrix.org | `RuntimeError: Another context is already active <zuul.zk.zkobject.ZKContext object at 0x7f5c643a5f10>` is this a new variant of the error you just fixed? | 22:49 |
| @clarkb:matrix.org | (basically we need to grab the lock and ensure we're the lock owner before proceeding?) | 22:49 |
| @jim:acmegating.com | yeah but the fix in this case is to add a lock. i'll update | 22:53 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 964893: Fix missing update in node reassignment https://review.opendev.org/c/zuul/zuul/+/964893 | 22:57 | |
| @jim:acmegating.com | can't lock the node in that test because the launcher holds it the whole time, so we need to just stop the launcher | 22:58 |
| @clarkb:matrix.org | I went ahead and reapproved the change with that update since it is only affecting the test case | 23:02 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!