| openstackgerrit | Akihiro Motoki proposed openstack/project-config master: Stop gerrit notification to neutron-stadium specific channels https://review.opendev.org/682771 | 01:40 |
|---|---|---|
| openstackgerrit | Mohammed Naser proposed zuul/zuul-operator master: Create zookeeper operator https://review.opendev.org/676458 | 02:08 |
| openstackgerrit | Merged openstack/project-config master: Add the gerrit checks plugin repo to Zuul https://review.opendev.org/682752 | 02:17 |
| openstackgerrit | Merged openstack/project-config master: Add upstream Gerrit projects to OpenDev Zuul tenant https://review.opendev.org/682754 | 02:21 |
| openstackgerrit | Merged openstack/project-config master: Add system-config and zuul to OpenDev tenant for jobs https://review.opendev.org/682755 | 02:21 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: zuul_console: fix python 3 support https://review.opendev.org/682556 | 03:59 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Support nodes setting 'auto' python-path https://review.opendev.org/682275 | 03:59 |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: logrotate: better rotation options https://review.opendev.org/682786 | 04:33 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Set default python-path to "auto" https://review.opendev.org/682797 | 05:57 |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: logrotate: better rotation options https://review.opendev.org/682786 | 06:02 |
| *** AJaeger has joined #opendev | 06:08 | |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Add horizon to translation jobs https://review.opendev.org/682808 | 06:54 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstack-zuul-jobs master: Use on-disk horizon dir for translations https://review.opendev.org/682809 | 07:03 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Add horizon to translation jobs https://review.opendev.org/682808 | 07:03 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstack-zuul-jobs master: Use on-disk horizon dir for translations https://review.opendev.org/682809 | 07:07 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Add horizon to translation jobs https://review.opendev.org/682808 | 07:08 |
| *** AJaeger has quit IRC | 08:14 | |
| *** recheck has quit IRC | 08:20 | |
| *** recheck has joined #opendev | 08:24 | |
| *** AJaeger has joined #opendev | 08:27 | |
| openstackgerrit | Merged zuul/zuul master: Add no-jobs reporter action https://review.opendev.org/681278 | 08:36 |
| openstackgerrit | Merged zuul/zuul master: Add report time to item model https://review.opendev.org/681323 | 08:50 |
| openstackgerrit | Merged openstack/cookiecutter master: Update URLs https://review.opendev.org/679998 | 08:52 |
| openstackgerrit | Merged openstack/project-config master: Add horizon to translation jobs https://review.opendev.org/682808 | 09:00 |
| openstackgerrit | Merged zuul/zuul master: Add Item.formatStatusUrl https://review.opendev.org/681324 | 09:11 |
| openstackgerrit | Merged openstack/openstack-zuul-jobs master: Use on-disk horizon dir for translations https://review.opendev.org/682809 | 09:19 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstack-zuul-jobs master: Fix horizon translation setup https://review.opendev.org/682850 | 09:49 |
| openstackgerrit | Merged openstack/project-config master: Announce opendev/project-config changes in channels https://review.opendev.org/682757 | 09:50 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Pagure - fix wrong commit gitweb url https://review.opendev.org/679946 | 10:02 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Pagure - handle initial comment change event https://review.opendev.org/680310 | 10:02 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Pagure - handle Pull Request tags (labels) metadata https://review.opendev.org/681050 | 10:02 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Pagure - reference pipelines add open: True requirement https://review.opendev.org/681252 | 10:02 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Pagure - handles pull-request.closed event https://review.opendev.org/681279 | 10:02 |
| *** openstackgerrit has quit IRC | 10:06 | |
| *** AJaeger has left #opendev | 10:08 | |
| *** openstackgerrit has joined #opendev | 10:10 | |
| openstackgerrit | Merged openstack/openstack-zuul-jobs master: Fix horizon translation setup https://review.opendev.org/682850 | 10:10 |
| *** zbr has quit IRC | 10:11 | |
| *** zbr has joined #opendev | 10:12 | |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Pagure - add support for git.tag.creation event https://review.opendev.org/679938 | 10:30 |
| openstackgerrit | Merged zuul/zuul master: Pagure - fix wrong commit gitweb url https://review.opendev.org/679946 | 10:57 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Revert "Remove accessbot from #openstack-kayobe" https://review.opendev.org/682881 | 12:06 |
| *** redrobot has joined #opendev | 12:27 | |
| openstackgerrit | Merged openstack/project-config master: Revert "Remove accessbot from #openstack-kayobe" https://review.opendev.org/682881 | 12:31 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Add scheduler config options for hold expiration https://review.opendev.org/682675 | 12:42 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Add scheduler config options for hold expiration https://review.opendev.org/682675 | 12:48 |
| openstackgerrit | Mark Goddard proposed opendev/irc-meetings master: Remove kayobe team meeting https://review.opendev.org/682894 | 13:22 |
| *** Green_Bird has quit IRC | 13:58 | |
| openstackgerrit | Merged openstack/project-config master: Stop gerrit notification to neutron-stadium specific channels https://review.opendev.org/682771 | 14:02 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Add scheduler config options for hold expiration https://review.opendev.org/682675 | 14:07 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Mark nodes as USED when deleting autohold https://review.opendev.org/664060 | 14:12 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Auto-delete expired autohold requests https://review.opendev.org/663762 | 14:12 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Add autohold delete/info commands to web API https://review.opendev.org/679057 | 14:12 |
| openstackgerrit | David Shrewsbury proposed zuul/zuul master: Remove outdated TODO https://review.opendev.org/682421 | 14:12 |
| *** openstackgerrit has quit IRC | 14:21 | |
| *** openstackgerrit has joined #opendev | 14:36 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: DNM: test prepare-workspace-git base-test https://review.opendev.org/682912 | 14:36 |
| openstackgerrit | Merged opendev/irc-meetings master: Remove kayobe team meeting https://review.opendev.org/682894 | 14:57 |
| openstackgerrit | Merged zuul/zuul master: Pagure - handle initial comment change event https://review.opendev.org/680310 | 14:59 |
| openstackgerrit | Merged zuul/zuul-website master: Update to page titles and Users https://review.opendev.org/680459 | 15:06 |
| openstackgerrit | Merged zuul/zuul master: zuul_console: fix python 3 support https://review.opendev.org/682556 | 15:17 |
| *** clarkb has joined #opendev | 15:24 | |
| clarkb | I mustve lost this channel last time I had to update weechat and it isnt in muscle memory yet to rejoin | 15:24 |
| corvus | pabelanger, mordred, fungi, clarkb: i'm continuing the work i started on using prepare-workspace-git | 15:24 |
| corvus | it looks like it's expecting a slightly different layout than what we cache on images | 15:25 |
| corvus | /opt/git/opendev.org/zuul/zuul-jobs is what it wants | 15:25 |
| corvus | /opt/git/zuul/zuul-jobs is what we have | 15:25 |
| corvus | do we want to adjust our caching to match the role? | 15:25 |
| clarkb | we should be able to modify the images to have symlinks for the old version, then switch zuul role, then delete symlinks (something like that) | 15:26 |
| corvus | (i'm guessing this answers the question "why didn't we get around to switching?") | 15:26 |
| corvus | good idea. i'll start on that | 15:26 |
| corvus | on the plus side, the "i don't have a cached image so i'll just init and push" code path works fine :) | 15:27 |
| corvus | (since that's what my test change ended up executing) | 15:27 |
| fungi | yeah, it could be done with a symlink from /opt/git/opendev.org to opt/git i suppose, but better if we populate /opt/git/ with a bunch of symlinks into /opt/git/opendev.org/ (as long as we don't create an org/namespace in opendev named "opendev.org" which seems unlikely) | 15:33 |
| fungi | that just needs to happen in our dib element right? | 15:33 |
| corvus | fungi: you said "just" does that mean you volunteer? :) | 15:34 |
| clarkb | fungi yes | 15:34 |
| fungi | sure, i can take a stab at it | 15:34 |
| corvus | (i'm completely baffled by dib elements and am currentry trying to figure out how all this works at all) | 15:34 |
| clarkb | corvus: fungi we wrote out a manifest file for a git repo managing element | 15:34 |
| corvus | yeah, it looks like "openstack-repos" does that | 15:35 |
| corvus | and then the dib upstream "source-repositories" reads it | 15:35 |
| clarkb | I think we need to update that manifest to change the canonical locations of the repos then for the symlinks a followup acript can create those | 15:35 |
| corvus | honestly, since this is a short-term thing, i think i'm in favor of the /opt/git/opendev.org -> /opt/git idea | 15:35 |
| corvus | that's probably something really simple we can stick in our local manifest | 15:35 |
| corvus | er in our local element | 15:36 |
| *** zbr has quit IRC | 15:36 | |
| fungi | well, the up side to a linkfarm is that once jobs are no longer using the old urls we can simply drop the routine which creates the symlinks | 15:36 |
| corvus | then make the role change, then update our manifest creation step to use the new names. | 15:36 |
| fungi | but i suppose it's a similar amount of work either way | 15:36 |
| fungi | there's a loop in 50-create-repo-list main() where i think we could add the link calls | 15:37 |
| fungi | and makedirs | 15:37 |
| corvus | fungi: yeah, but is that running in the right context? | 15:37 |
| fungi | (for the namespaces) | 15:37 |
| corvus | it's an extras.d - is the target mounted then? | 15:37 |
| *** zbr has joined #opendev | 15:38 | |
| fungi | oh, i see, that loop is just writing the repo list to a file | 15:38 |
| clarkb | ya you should do it during install.d probably to avoid a bunch of pote tially broken links later | 15:38 |
| corvus | fungi: back to the other issue -- i think the best ordering is 1) keep the real repos at the old location; 2) symlink to them (whether individually or en masse); 3) switch roles to use the new paths; 4) drop the links and write the repos to the new location | 15:38 |
| clarkb | since it will depend on what endsup on disk | 15:39 |
| corvus | fungi: the reason i think that's the direction we want the links is that we get better incremental testing. 3 is the first time the links get used, and it's in a controlled test change | 15:39 |
| *** paladox has joined #opendev | 15:39 | |
| fungi | indeed, looks like it's writing the repos list to os.path.join(TMP_HOOKS_PATH,'source-repository-projects-yaml') so that's probably outside the chroot | 15:39 |
| corvus | fungi: (if we pointed the links the other way, and we messed them up somehow, we'd fail all jobs instantly) | 15:39 |
| fungi | and yeah, whichever order we do it in, we need to be able to confirm jobs are using the new role before we get rid of the compatibility paths | 15:41 |
| corvus | fungi, clarkb: i think i grok this -- lemme try writing a change | 15:41 |
| fungi | i guess we can delete the role before we update the clone path in dib | 15:41 |
| corvus | yeah that should be fine | 15:41 |
| fungi | granted, there are probably also projects making direct references to the old path | 15:41 |
| fungi | so maybe we delay removing the symlink and give them some warning | 15:42 |
| corvus | fungi: if there are, they're bypassing zuul and i have no sympathy | 15:42 |
| fungi | yeah, we normally recommend they use teh variable for the base path to the repos | 15:42 |
| corvus | either the variable or the zuul path | 15:42 |
| corvus | but *no one* should be touching the cached repos | 15:43 |
| corvus | we don't even guarantee they are there | 15:43 |
| fungi | i suppose it's an opportunity to root out jobs which haven't done that and get them to do it properly | 15:43 |
| corvus | no -- no job *ever* should have done that | 15:43 |
| corvus | this isn't some migration thing | 15:43 |
| clarkb | ya they may be stale or not even present | 15:43 |
| fungi | ahh, right, i forgot the cached repo path is not the zuul repo path. so yeah the number of jobs referencing it should be zero | 15:43 |
| corvus | well, devstack-gate did that. but we did migrate that. a long time ago. | 15:43 |
| corvus | yeah | 15:43 |
| fungi | i had it in my head that zuul updated the cached copy rather than cloning from the cached copy and updating the clone | 15:44 |
| corvus | and i think the auto-migrated zuul-cloner stuff is working from the zuul-setup repos too | 15:44 |
| fungi | so if the zuul repositories path is not changing, then this should be fairly trivial to switch over | 15:44 |
| corvus | so our zuul-cloner shim shouldn't be touching these either -- but let's confirm that. | 15:44 |
| fungi | and in fact, if we get it wrong, jobs probably won't fail except insofar as they may timeout due to longer times for zuul to push repos onto nodes | 15:45 |
| openstackgerrit | Andreas Jaeger proposed openstack/infra-manual master: Document "clean check" requirement https://review.opendev.org/682933 | 15:46 |
| openstackgerrit | James E. Blair proposed openstack/project-config master: Symlink /opt/git to /opt/git/opendev.org https://review.opendev.org/682934 | 15:49 |
| openstackgerrit | James E. Blair proposed openstack/project-config master: Cache repos in /opt/git/opendev.org https://review.opendev.org/682935 | 15:49 |
| corvus | clarkb, fungi: ^ how's that look? | 15:49 |
| corvus | playbooks/legacy/pre.yaml: repo_src_dir: "/home/zuul/src/opendev.org" | 15:52 |
| corvus | that tells me that the zuul-cloner shim is cloning from the zuul homedir to wherever the job specifies, so that's all good. | 15:53 |
| corvus | (it does not touch the real cache dir) | 15:53 |
| openstackgerrit | James E. Blair proposed openstack/project-config master: Cache repos in /opt/git/opendev.org https://review.opendev.org/682935 | 15:54 |
| clarkb | its way to early in the morning to reason about whether or not that creates a loop | 15:59 |
| * clarkb tests it | 15:59 | |
| *** recheck has quit IRC | 16:00 | |
| *** recheck has joined #opendev | 16:00 | |
| clarkb | does not loop | 16:00 |
| fungi | lrwxrwxrwx 1 fungi fungi 15 Sep 18 16:00 /home/fungi/foo/bar -> /home/fungi/foo | 16:01 |
| fungi | works for me, yeah | 16:01 |
| fungi | i mean it does allow you to do silly things like reference /home/fungi/foo/bar/bar/bar/bar | 16:02 |
| fungi | but we're not in the business of stopping intentional silliness | 16:02 |
| clarkb | ya | 16:02 |
| clarkb | corvus: comment on the frist change for why it failed (and how I think it can be fixed) looks good otehrwise | 16:03 |
| *** zbr is now known as zbr|ruck | 16:07 | |
| openstackgerrit | Andreas Jaeger proposed openstack/infra-manual master: Document "clean check" requirement https://review.opendev.org/682933 | 16:08 |
| openstackgerrit | James E. Blair proposed openstack/project-config master: Symlink /opt/git to /opt/git/opendev.org https://review.opendev.org/682934 | 16:09 |
| openstackgerrit | James E. Blair proposed openstack/project-config master: Cache repos in /opt/git/opendev.org https://review.opendev.org/682935 | 16:09 |
| corvus | clarkb, fungi: ^ thanks! | 16:09 |
| openstackgerrit | Merged opendev/irc-meetings master: Updated the Diversity WG to monthly https://review.opendev.org/682727 | 16:15 |
| openstackgerrit | Merged openstack/openstack-zuul-jobs master: Switch project-config to promote jobs [5]: Update specs, cleanup https://review.opendev.org/682215 | 16:17 |
| openstackgerrit | Merged openstack/project-config master: Symlink /opt/git to /opt/git/opendev.org https://review.opendev.org/682934 | 16:30 |
| openstackgerrit | Merged zuul/zuul master: Add support for the Gerrit checks plugin https://review.opendev.org/680778 | 16:59 |
| openstackgerrit | Merged zuul/zuul master: Update gerrit pagination test fixtures https://review.opendev.org/682114 | 17:08 |
| openstackgerrit | Merged openstack/project-config master: Switch project-config to promote jobs [6]: Remove publish-openstack-specs https://review.opendev.org/682216 | 17:31 |
| openstackgerrit | Merged zuul/zuul master: Support HTTP-only Gerrit https://review.opendev.org/681936 | 17:35 |
| *** zbr|ruck is now known as zbr | 17:36 | |
| openstackgerrit | Paul Belanger proposed openstack/project-config master: Add windmill-ops / ansible-role-elasticsearch https://review.opendev.org/680830 | 17:40 |
| openstackgerrit | Paul Belanger proposed openstack/project-config master: Set requireContributorAgreement false for windmill roles https://review.opendev.org/682954 | 17:44 |
| openstackgerrit | Merged openstack/project-config master: Update gentoo image to current profile https://review.opendev.org/682639 | 17:47 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: DO NOT MERGE test cleanup phase playbook https://review.opendev.org/680178 | 17:49 |
| openstackgerrit | Merged openstack/project-config master: Enable Review-Priority flag for Blazar https://review.opendev.org/682613 | 17:54 |
| openstackgerrit | Merged openstack/project-config master: Fix release ACL for Kayobe https://review.opendev.org/682581 | 17:54 |
| openstackgerrit | Merged zuul/zuul master: Add autogenerated tag to Gerrit reviews https://review.opendev.org/682473 | 17:59 |
| openstackgerrit | Merged openstack/project-config master: Add windmill-ops / ansible-role-elasticsearch https://review.opendev.org/680830 | 18:18 |
| openstackgerrit | Merged opendev/base-jobs master: Add cleanup playbook to all base jobs https://review.opendev.org/681322 | 18:19 |
| openstackgerrit | Merged zuul/zuul master: Use robot_comments in Gerrit https://review.opendev.org/682487 | 18:19 |
| *** openstackgerrit has quit IRC | 18:37 | |
| *** openstackgerrit has joined #opendev | 19:18 | |
| openstackgerrit | Merged opendev/project-config master: Add record of 20190916 gerrit project renames https://review.opendev.org/681353 | 19:18 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Remove obsolete docs publish jobs https://review.opendev.org/682972 | 19:34 |
| openstackgerrit | Clark Boylan proposed opendev/system-config master: Restore setup-org.yaml https://review.opendev.org/682732 | 19:41 |
| openstackgerrit | David Shrewsbury proposed zuul/nodepool master: Reduce upload threads in tests from 4 to 1 https://review.opendev.org/682977 | 19:41 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Remove obsolete docs publish jobs https://review.opendev.org/682972 | 19:42 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Remove obsolete docs publish jobs https://review.opendev.org/682972 | 19:46 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: RFC: Generic cache implementation https://review.opendev.org/682992 | 20:38 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: RFC: Generic cache implementation https://review.opendev.org/682992 | 20:39 |
| openstackgerrit | Merged opendev/system-config master: Restore setup-org.yaml https://review.opendev.org/682732 | 21:11 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Store a list of held node per held build in hold request https://review.opendev.org/682466 | 21:47 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: WIP: Fix gerrit errors from production https://review.opendev.org/683006 | 22:53 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: DNM: Use http for all gerrit tests https://review.opendev.org/683007 | 22:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!