opendevreview | James Page proposed openstack/project-config master: Setup/update Sunbeam single charm repos to noop https://review.opendev.org/c/openstack/project-config/+/917827 | 05:03 |
---|---|---|
opendevreview | James Page proposed openstack/project-config master: Setup/update Sunbeam single charm repos to noop https://review.opendev.org/c/openstack/project-config/+/917827 | 05:12 |
clarkb | I added a new etherpad_admin_api_secret to group vars for prod | 15:11 |
clarkb | fungi: not sure if you think now is still a good time for https://review.opendev.org/c/opendev/system-config/+/914119 seems like your vote got reset? | 15:11 |
clarkb | I am going to pop out for an early lunch today (people are in town randomly) but I'm around until then and after | 15:12 |
fungi | clarkb: oops! it was gertty async-racing its commit of my earlier vote, i think. approved just now | 15:19 |
fungi | i'm around most of the day, semi-distracted by packing and last-minute yardwork, but will keep an eye on it | 15:20 |
opendevreview | Merged openstack/diskimage-builder master: ConfigDrive: Disable cloud-init auto dhcp by default https://review.opendev.org/c/openstack/diskimage-builder/+/899886 | 15:22 |
tonyb | Is there a way to have zuul work through a list of fallback branches is the target branch doesn't exist? | 15:30 |
tonyb | I have a requirements change on stable/zed which clones masakari, as masakari doesn't have a stable/zed it defaulted to the master branch but it'd be better if it tried unmaintained/zed first | 15:31 |
clarkb | you can specify the branch to checkout for each project on the job. You can also have the project specify a branch pragma rule that maps stable/foo to unmaintained/foo | 15:35 |
tonyb | Can you point me at docs for the pragma rules? | 15:37 |
tonyb | https://zuul-ci.org/docs/zuul/latest/config/pragma.html got it | 15:38 |
clarkb | https://zuul-ci.org/docs/zuul/latest/config/pragma.html#attr-pragma.implied-branch-matchers | 15:39 |
clarkb | ya that | 15:39 |
clarkb | specifically the last config item I think | 15:39 |
tonyb | Thanks. I can for sure make override-checkout work, but I think given the highly OpenStack specific usecase the pagma wont work as I want. | 15:44 |
tonyb | I'll just use override-checkout for now | 15:45 |
fungi | tonyb: i think the job transitions going on in openstack-zuul-jobs are using a pragma for the exact situation you described, if you want a clear example | 15:47 |
tonyb | fungi: I think I'm missing something. I can't see any changes that are doing that. | 16:02 |
tonyb | fungi: and: for d in opendev/*jobs* openstack/*jobs* ; do echo $d ; git -C $d remote update ; git -C $d grep pragma origin/master -- ; done | 16:02 |
tonyb | is also empty | 16:02 |
fungi | mmm, lemme find what i was thinking of, maybe they did use overrides | 16:05 |
tonyb | fungi: thanks | 16:06 |
clarkb | the etherpad change should be landing soon and then will end up behind hourly jobs for a short bit | 16:08 |
opendevreview | Merged opendev/system-config master: Update etherpad to v2.0.3 https://review.opendev.org/c/opendev/system-config/+/914119 | 16:08 |
tonyb | clarkb: noted | 16:08 |
fungi | tonyb: mmm, i'm not finding it. maybe gmann remembers where/how that was done | 16:11 |
tonyb | fungi: Okay Thanks for confirming | 16:12 |
fungi | or maybe it was a suggestion but the problem never got solved, now i don't recall | 16:12 |
tonyb | gmann: Any pointers to stable/foo -> unmaintained/foo -> master fallback logic in zuul jobs would be great | 16:12 |
opendevreview | Brian Haley proposed openstack/project-config master: Improve terminology in project-config tree https://review.opendev.org/c/openstack/project-config/+/917863 | 16:17 |
fungi | etherpad image upload seems to have succeeded, but the service upgrade is waiting behind hourly jobs | 16:17 |
gmann | tonyb: fungi: I do not think we have any case of fallback branches, what we did was generic regex with */x so that it match stable/x or unmaintained/x | 16:17 |
tonyb | gmann: Can you point me at where? | 16:18 |
fungi | gmann: okay, so as far as projects with unmaintained/foo branches doing integration testing with projects that still have stable/foo branches, we never resolved that challenge i suppose | 16:19 |
fungi | tonyb: it was for branch selection in determining which jobs to run | 16:19 |
tonyb | Ahhh okay so releated but different | 16:19 |
fungi | so simply matching ^.*/zed where explicit branch matchers used to have ^stable/foo | 16:20 |
fungi | er, used to have ^stable/zed i meant | 16:20 |
gmann | tonyb: this one, but yes to select the job to run https://github.com/openstack/tempest/blob/550784f306df22a83dc1894da18c64b7494c6521/zuul.d/stable-jobs.yaml#L86 | 16:20 |
clarkb | etherpad is restarting now | 16:21 |
fungi | yep | 16:21 |
clarkb | docker ps reports it is healthy | 16:21 |
clarkb | https://etherpad.opendev.org/p/opendev-project-renames-20240422 and https://etherpad.opendev.org/p/gerrit-upgrade-3.9 still load for me | 16:21 |
fungi | start time of the container was 16:20 | 16:21 |
fungi | yep, loading for me too | 16:22 |
fungi | will keep an eye out for any reports of problems | 16:22 |
gmann | tonyb: I think what you are facing is what we face in devstack when not all project were ready on new stable/ and we have to wait for all those stable branch setup before everything run fine | 16:22 |
clarkb | fungi: have we written a change to update file perms on settings.json yet? | 16:24 |
tonyb | Possibly similar situation yes | 16:24 |
opendevreview | Jeremy Stanley proposed opendev/system-config master: Tighten permissions on Etherpad settings file https://review.opendev.org/c/opendev/system-config/+/917865 | 16:28 |
fungi | clarkb: jinx! | 16:28 |
fungi | curious to see if it passes tests or actually breaks something | 16:29 |
clarkb | fungi: the file in the Dockerfile is there just to keep in sync with upstream's dockerfile. We actually bind mount the production file over that one | 16:43 |
clarkb | so I don't think we need to change the dockerfile but need to update the file perms on the file that we bind mount? | 16:43 |
fungi | oh, got it. thanks, i'll find it | 16:49 |
opendevreview | Jeremy Stanley proposed opendev/system-config master: Tighten permissions on Etherpad settings file https://review.opendev.org/c/opendev/system-config/+/917865 | 16:52 |
clarkb | fungi: and CI should tell us if that is a problem for etherpad the service reading the file | 16:55 |
clarkb | we may need to set owner and group to whatever uid:gid pair the dockerfile is using | 16:55 |
clarkb | but CI should have that well covered especially due to the testinfra tests that rely on the jwt token generation | 16:55 |
fungi | yep | 17:09 |
clarkb | CI did pass. I don't fully understand how/why after looking more closely since the file is owned by root:root and etherpad runs as 5001 | 17:29 |
fungi | need to go run a quick pre-travel errand, but can work on it some more shortly | 18:09 |
opendevreview | Jeremy Stanley proposed opendev/system-config master: Tighten permissions on Etherpad settings file https://review.opendev.org/c/opendev/system-config/+/917865 | 19:20 |
clarkb | https://review.opendev.org/c/opendev/afsmon/+/917470 is a straightforward change to update afsmon's CI for xenial removal and also to test something closer to what we're deploying in production | 20:17 |
fungi | ltgm | 20:26 |
opendevreview | Merged opendev/afsmon master: Update afsmon python testing to modern versions https://review.opendev.org/c/opendev/afsmon/+/917470 | 20:39 |
*** dmellado0334 is now known as dmellado033 | 21:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!