| mnasiadka | frickler: not really, everything was fine for me :) | 07:44 |
|---|---|---|
| *** jroll05 is now known as jroll0 | 08:57 | |
| Clark[m] | Johnson: fyi https://github.com/openstack/octavia/pull/8 | 12:40 |
| Clark[m] | Gah autocorrect is so annoying sometimes. johnsom ^ | 12:40 |
| gthiemonge | Clark[m]: interesting, this is an update of a patch that was already proposed on gerrit | 12:44 |
| fungi | Clark[m]: sorry, still not really around until tomorrow afternoon, but is 358954 a typo? looks like an abandoned change from 7 years ago... | 12:51 |
| fungi | oh! it's on gerrit's gerrit, not ours | 12:52 |
| fungi | should be fine, openstack election tooling uses anonymous queries, so had to switch to only relying on the gerrit "preferred" address years ago | 12:54 |
| Clark[m] | fungi: ya I'm slowly trying to work through the gerrit breaking change list and keeping track on that etherpad | 12:56 |
| opendevreview | Merged openstack/project-config master: Add infrawatch/sg-core to available repos https://review.opendev.org/c/openstack/project-config/+/898314 | 13:09 |
| clarkb | I think ara has broken our system-config-run-* jobs | 14:42 |
| clarkb | not sure yet if that was a one off failuer or something more persistent. I'm going to recheck (since that is easy) and then if it hapepns again look at debugging after PTG stuff | 14:43 |
| frickler | clarkb: does it look like this? https://paste.openstack.org/show/b441YNDGVTDSizdpec90/ that's apparently due to a new ruamel version | 14:47 |
| clarkb | hrm no it failed on sqlite3.OperationalError: no such table: playbooks but looking closer I think this was because ansible didn't run in the main job | 14:48 |
| clarkb | so the failure is actually elsewhere and ara fails beacuse we don't have any data generated for it to process into a report | 14:49 |
| clarkb | https://zuul.opendev.org/t/openstack/build/2f5672507d4c4aaaa7251430726873fe/logs is the build that failed | 14:50 |
| frickler | "Install root key" is what failed there, though no_log is set https://zuul.opendev.org/t/openstack/build/2f5672507d4c4aaaa7251430726873fe/console#2/0/46/bridge99.opendev.org | 14:51 |
| frickler | same failure though in two jobs in the buildset, strange | 14:53 |
| clarkb | ya it does look like that task records its log onto disk though so in theory I can hold the nodes and check for that log file to see why it broke | 14:54 |
| clarkb | that will be the next step I think | 14:55 |
| clarkb | my held nodes for gerrit yesterday hit this and didn't deploy gerrit things | 15:08 |
| clarkb | not sure what is going on based on the logs yet though. Might need to -vvv the ansible run | 15:08 |
| clarkb | on a held server I reran the command that failed manually but switched -v to -vvv unfortunately it passed | 15:25 |
| clarkb | return code 0 | 15:25 |
| clarkb | I think a DNM change to remove the no log is in order as well as a bump to -vvv there? | 15:25 |
| clarkb | oh maybe this command isn't what failed though and something else in the included role broke? | 15:26 |
| opendevreview | Clark Boylan proposed opendev/system-config master: DO NOT MERGE This is debugging only and if merged would be problematic for production https://review.opendev.org/c/opendev/system-config/+/899186 | 15:29 |
| clarkb | lets see what that shows | 15:29 |
| clarkb | oops forgot -vvv update. Maybe we'll see enough as is though | 15:33 |
| clarkb | heh of course that triggers a bunch of jobs | 15:37 |
| clarkb | oh and I didn't set pipefail so maybe these will "succeed" on that task | 15:42 |
| clarkb | frickler: yes now that we have logs it does appear to be that ruamel problem. Any idea what the fix is? | 15:47 |
| clarkb | frickler: do we disable ara or maybe pin ruamel? | 15:47 |
| opendevreview | Clark Boylan proposed opendev/system-config master: DO NOT MERGE This is debugging only and if merged would be problematic for production https://review.opendev.org/c/opendev/system-config/+/899186 | 15:48 |
| clarkb | "If you only read this after your program has stopped working: I am sorry to hear that, but that also means you, or the person developing your program, has not tested with warnings on" | 15:50 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Cap ruamel.yaml install for ARA https://review.opendev.org/c/opendev/system-config/+/899190 | 15:55 |
| clarkb | 899190 will have to be rebased off of the DNM fix change if it fixes things | 15:55 |
| clarkb | it isn't clear to me hwy it didn't break when I ran things by hand on the held nodes | 15:55 |
| clarkb | maybe ara is doing somethign stateful and skipping running on a second pass because the first ran failed so it disables itself? | 16:02 |
| opendevreview | Clark Boylan proposed opendev/system-config master: DO NOT MERGE This is debugging only and if merged would be problematic for production https://review.opendev.org/c/opendev/system-config/+/899186 | 16:22 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Cap ruamel.yaml install for ARA https://review.opendev.org/c/opendev/system-config/+/899190 | 16:22 |
| clarkb | this little corner of code is hitting all of the corner case gotchas. Yaml block string quoting fun, shell executable being sh not bash, pypi dependency problems | 16:22 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Cap ruamel.yaml install for ARA https://review.opendev.org/c/opendev/system-config/+/899190 | 16:44 |
| clarkb | ok based on test results from the previous stack I think ^ does fix things for us so I have rebased it off of the DNM change to make it mergable | 16:44 |
| opendevreview | Merged opendev/system-config master: Cap ruamel.yaml install for ARA https://review.opendev.org/c/opendev/system-config/+/899190 | 17:35 |
| clarkb | I have abandoned https://review.opendev.org/c/opendev/system-config/+/899186 to make it much less likely it will ever merge | 17:44 |
| opendevreview | Merged opendev/system-config master: Add zk test to check myid is set in service https://review.opendev.org/c/opendev/system-config/+/898614 | 18:38 |
| *** diablo_rojo is now known as Guest4496 | 19:30 | |
| opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: Introduce logreduce roles https://review.opendev.org/c/zuul/zuul-jobs/+/899212 | 20:24 |
| opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: Introduce logreduce roles https://review.opendev.org/c/zuul/zuul-jobs/+/899212 | 20:51 |
| opendevreview | Clark Boylan proposed opendev/system-config master: WIP Update gitea to 1.21 https://review.opendev.org/c/opendev/system-config/+/897679 | 20:55 |
| clarkb | there was a minor template difference between rc1 and rc2 but that was it | 20:56 |
| clarkb | https://149.202.181.121/c/x/test-project/+/3 shows comment links work for everything but the gitsha comment link. I'll have to dig into that more I guess | 21:01 |
| clarkb | oh maybe this is simple. It appears to need to be surrounded by whitespace or <p> tags. My test case didn't do that. | 21:03 |
| clarkb | I think we can probably update the regex to make the test case work though and that might be worthwile but I've shown the commentlink works generally which means i think the commentlinks update change here: https://review.opendev.org/c/opendev/system-config/+/898989 is safe to merge whenever we want to restart gerrit to pick that up | 21:04 |
| opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: Introduce logreduce roles https://review.opendev.org/c/zuul/zuul-jobs/+/899212 | 21:06 |
| ianw | obviously it got figured out, @ https://meetings.opendev.org/irclogs/%23opendev/%23opendev.2023-06-13.log.html#t2023-06-13T00:57:28 i did put the linaro steps into a file bridge:/root/tmp/linaro-renew-steps.txt (not that this was discoverable in any way) | 21:28 |
| ianw | i just ran out of time trying to split up the base install bits so we could have our users, deploy a script or similar on linaro but not do our full base playbook, which was wary would probably not be compatible with kolla | 21:29 |
| Clark[m] | The notes you left in the email were more than plenty to figure it out. Thank you for the writing those down. And ya avoiding conflicts between Kolla/openstack and our base playbooks seems like a good thing | 21:55 |
| opendevreview | James E. Blair proposed zuul/zuul-jobs master: Switch ensure-openshift to quay.io https://review.opendev.org/c/zuul/zuul-jobs/+/899217 | 22:33 |
| opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: Introduce logreduce roles https://review.opendev.org/c/zuul/zuul-jobs/+/899212 | 22:37 |
| opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: Introduce logreduce roles https://review.opendev.org/c/zuul/zuul-jobs/+/899212 | 23:02 |
| opendevreview | Julia Kreger proposed opendev/glean master: Add support for tinycore linux https://review.opendev.org/c/opendev/glean/+/899219 | 23:13 |
| opendevreview | Tristan Cacqueray proposed zuul/zuul-jobs master: Introduce logreduce roles https://review.opendev.org/c/zuul/zuul-jobs/+/899212 | 23:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!