| -@gerrit:opendev.org- Simon Westphahl proposed: | 11:49 | |
| - [zuul/zuul] Simplified attribute API for ZKObjects https://review.opendev.org/c/zuul/zuul/+/809532 | ||
| - [zuul/zuul] wip: Make QueueItem a Zookeeper object https://review.opendev.org/c/zuul/zuul/+/809414 | ||
| - [zuul/zuul] wip: Store pipeline state in Zookeeper https://review.opendev.org/c/zuul/zuul/+/810658 | ||
| - [zuul/zuul] wip: Store queue state in Zookeeper https://review.opendev.org/c/zuul/zuul/+/810920 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 11:49 | |
| - [zuul/zuul] Add an API for ZK-backed objects https://review.opendev.org/c/zuul/zuul/+/809293 | ||
| - [zuul/zuul] Temporarily enqueue cycle changes for reporting https://review.opendev.org/c/zuul/zuul/+/810328 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: | 11:50 | |
| - [zuul/zuul] Simplified attribute API for ZKObjects https://review.opendev.org/c/zuul/zuul/+/809532 | ||
| - [zuul/zuul] Make QueueItem a Zookeeper object https://review.opendev.org/c/zuul/zuul/+/809414 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 11:50 | |
| - [zuul/zuul] Add an API for ZK-backed objects https://review.opendev.org/c/zuul/zuul/+/809293 | ||
| - [zuul/zuul] Temporarily enqueue cycle changes for reporting https://review.opendev.org/c/zuul/zuul/+/810328 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] Store pipeline state in Zookeeper https://review.opendev.org/c/zuul/zuul/+/810658 | 12:26 | |
| @fungicide:matrix.org | ianw: not urgent, but if you get time could you weigh in on https://review.opendev.org/809899 with your thoughts? trying to make sure we've got consensus that this is a reasonable path forward | 12:46 |
|---|---|---|
| @fungicide:matrix.org | ianw: https://review.opendev.org/810299 is related, updates the zuul-job docs to reflect that | 12:48 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] GUI: Add tenant dropdown to top menu https://review.opendev.org/c/zuul/zuul/+/810955 | 14:14 | |
| @mhuin:matrix.org | Hello zuul-maint, I'd like to move forward on the admin GUI changes ( see https://review.opendev.org/q/topic:%22GUI_admin%22 ). I've isolated the needed changes to zuul-web at the beginning of the patch chain; It'll make it easier to test and review the GUI changes if we get these merged first | 15:00 |
| @mhuin:matrix.org | so basically https://review.opendev.org/c/zuul/zuul/+/735586 and https://review.opendev.org/c/zuul/zuul/+/736968 | 15:12 |
| @clarkb:matrix.org | corvus: re https://review.opendev.org/c/zuul/zuul/+/810554 do we know if that is what happened with github3? Seems like a good check either way, I'm mostly wondering if we need more belts and suspenders for future restarts though | 16:07 |
| @jim:acmegating.com | Clark: i believe we confirmed that it was suspended | 16:08 |
| @jim:acmegating.com | Clark: i take that back, i can't find confirmation in the irc logs | 16:09 |
| @clarkb:matrix.org | I think that must be the suspicion because if the installation was removed it wouldn't show up in the installations request list | 16:09 |
| @clarkb:matrix.org | and I don't think you can change installation permissions? The app says "this is what I need" and you either agree to that or dont? | 16:10 |
| @jim:acmegating.com | Clark: yeah, so i think we just very strongly suspect it was suspended because the behavior matched | 16:10 |
| @jim:acmegating.com | right | 16:10 |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul] zk: change_cache: Delete bad JSON data from cache https://review.opendev.org/c/zuul/zuul/+/810468 | 16:59 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul] gitlabconnection: Use keyword argument for retries https://review.opendev.org/c/zuul/zuul/+/811204 | 17:01 | |
| @jim:acmegating.com | i tweeted about my ansiblefest zuul talk: https://twitter.com/acmegating/status/1442537059533479942 | 17:16 |
| @clarkb:matrix.org | "i tweeted" <- things I'm not sure I ever expected to hear corvus say :) | 17:17 |
| @jim:acmegating.com | well, er, acme gating tweeted :) | 17:18 |
| @cdesai:matrix.org | > <@jim:acmegating.com> i tweeted about my ansiblefest zuul talk: https://twitter.com/acmegating/status/1442537059533479942 | 18:23 |
| Is it done? If so, is there a replay somewhere and/or are slides available? | ||
| @jpew:matrix.org | I can't build zuul locally because the upstream repository for skopeo appears to be broken.... anyone else seen that? | 18:41 |
| @jpew:matrix.org | (zuul docker images that is) | 18:41 |
| @clarkb:matrix.org | https://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/stable/Debian_10/ that page loads at least | 18:43 |
| @jpew:matrix.org | Ya, it looks like the size (and maybe checksums?) in `Releases` doesn't match `Packages.gz` | 18:44 |
| @jim:acmegating.com | cde: oh sorry, it should be available starting wednesday morning | 18:45 |
| @jim:acmegating.com | for some reason all the on-demand sessions say "session ended" even though they haven't begun :/ | 18:45 |
| @clarkb:matrix.org | jpew: agreed a manual check the the size and sha1 in Packages.gz does not match the file at https://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/stable/Debian_10/arm64/skopeo_1.2.2-2_arm64.deb | 18:46 |
| @clarkb:matrix.org | The Packages file updated today. I wonder if they just haven't published the matching deb to go along with it | 18:47 |
| @clarkb:matrix.org | https://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/testing/Debian_Unstable/ seems to be a bit more up todate and might work as a fallback if this doesn't correct shortly | 18:48 |
| @jpew:matrix.org | Ok, I'll try that | 18:48 |
| @jim:acmegating.com | i just rechecked a change so we'll have a current upstream build shortly | 18:50 |
| @jim:acmegating.com | a build at 17:01 succeeded: https://zuul.opendev.org/t/zuul/build/971142e91097428d8a8669afbe134ecc | 18:51 |
| @clarkb:matrix.org | the timestamp on the packages file is 17:04 | 18:51 |
| @jim:acmegating.com | nice :) | 18:52 |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul] zk: change_cache: Delete bad JSON data from cache https://review.opendev.org/c/zuul/zuul/+/810468 | 18:55 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Remove time database https://review.opendev.org/c/zuul/zuul/+/808841 | 18:55 | |
| @jim:acmegating.com | either of those should get us a new build | 18:55 |
| @clarkb:matrix.org | Looks like the image buidls for those passee | 19:40 |
| @clarkb:matrix.org | * Looks like the image builds for those passed | 19:40 |
| @clarkb:matrix.org | One thing I notice is that there appears to be a mirror infrastructure in use for those repos. Maybe it is region dependent? | 19:41 |
| @jpew:matrix.org | Possibly... I seem to have worked around it locally anyway | 19:41 |
| @jpew:matrix.org | Thanks though | 19:41 |
| @clarkb:matrix.org | corvus: is that latest patchset on the times db just an update to aid in local performance testing? | 19:48 |
| @jim:acmegating.com | Clark: oh no sorry that was a rebase to deal with conflicts | 19:59 |
| @jim:acmegating.com | Clark: PS10 is my latest conceptual proposal ("no new tables, but an extra thread to handle lookups in the background"; i'm thinking that's the best way to move forward on SOS without adding risk to folks who are running at their limits; then we revisit that later). PS11 was a minor metric name change. PS12 was just a conflict resolution. | 20:02 |
| @jim:acmegating.com | maybe that helps selecting the inter-diffs | 20:02 |
| @clarkb:matrix.org | thanks | 20:03 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] Web UI: hover on build in buildset page for info https://review.opendev.org/c/zuul/zuul/+/811227 | 20:06 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/nodepool] kubernetes, openshift: Add volume mount support https://review.opendev.org/c/zuul/nodepool/+/810322 | 20:41 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/nodepool] kubernetes, openshift: Add kvm resource https://review.opendev.org/c/zuul/nodepool/+/811232 | 20:52 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/nodepool] kubernetes: Add startup-timeout https://review.opendev.org/c/zuul/nodepool/+/811234 | 20:56 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul-operator] Update spec if image version changes https://review.opendev.org/c/zuul/zuul-operator/+/811235 | 21:08 | |
| @jim:acmegating.com | tobiash: replied on the metastatic driver (sorry that took a while), thanks! | 21:38 |
| @spamaps:spamaps.ems.host | corvus: hi, you linked me to a thing that deploys zuul onto gcloud recently. I can't seem to find it. Halp again? ty | 21:40 |
| @jim:acmegating.com | spamaps: https://gerrit.googlesource.com/zuul/ops/ | 21:42 |
| @spamaps:spamaps.ems.host | Ah there it is. | 21:44 |
| @spamaps:spamaps.ems.host | I was looking on opendev. | 21:44 |
| @spamaps:spamaps.ems.host | Going to try and spin one up this week to eval against Team City, which is our blessed "high powered" alternative to the overly simplistic internal CI system. :) | 21:48 |
| @spamaps:spamaps.ems.host | (which is a thin wrapper around Jenkins) | 21:48 |
| @jim:acmegating.com | spamaps: good luck! it's mostly a straightforward k8s deployment, but with special sauce to use gcp service accounts instead of downloaded json crendentials | 21:50 |
| @jim:acmegating.com | zuul-operator would also be an option of course, but it doesn't have the gcp service account special sauce | 21:51 |
| @jim:acmegating.com | tobiash: is it time to unWIP and merge https://review.opendev.org/777479 ? | 21:56 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Allow configuring custom check names in github reporter https://review.opendev.org/c/zuul/zuul/+/811021 | 21:59 | |
| @jpew:matrix.org | I'm writing a plugin (basically) that builds with an upstream project and I want to test it with zuul. Should I add the upstream git repo as a project in my zuul and then have my plugin pull it in with a `required-project` or just write a task that clones down the upstream manually? | 22:08 |
| @jim:acmegating.com | jpew: using required-projects gets you the ability to use Depends-On with changes on the upstream project; so in most cases i'd say yes. | 22:09 |
| @jpew:matrix.org | Would I just use the Git driver then? upstream doesn't use zuul | 22:10 |
| @jim:acmegating.com | (also, uses the zuul executor git cache and may be faster/more network efficient) | 22:10 |
| @jim:acmegating.com | jpew: if upstream doesn't use one of the zuul-supported code review systems (github, gerrit, gitlab, pagure) then you can use the git driver; but that doesn't support dependencies, so you lose that advantage. you're down to only the executor git cache at that point. it may still be worthwhile (and may make your jobs nicer and more resilient), but it's less of a slam-dunk. | 22:12 |
| @jpew:matrix.org | corvus: zuul needs a mailing-list driver plugin 😛 | 22:13 |
| @jim:acmegating.com | jpew: actually not kidding: i think it's possible and i would be interested in a commission to work on that. :) | 22:17 |
| -@gerrit:opendev.org- Zuul merged on behalf of Joshua Watt: [zuul/zuul] zk: change_cache: Delete bad JSON data from cache https://review.opendev.org/c/zuul/zuul/+/810468 | 22:23 | |
| @jpew:matrix.org | corvus: Tempting :) Probably too much of a change for the project ATM (and I don't think I have to pull to get them to change anyway) | 22:24 |
| @jpew:matrix.org | * corvus: Tempting :) Probably too much of a change for the project ATM (and I don't think I have the pull to get them to change anyway) | 22:25 |
| @jim:acmegating.com | jpew: understood. could still be beneficial for your local use though, even without any buy-in or awareness from the upstream project (it would get you depends-on support, and you could start to slyly point people to your own local build results of the upstream project if you wanted to :) anyway, you can file that away for some future use. | 22:27 |
| @jpew:matrix.org | will do :) | 22:28 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Emit config warnings if shared queues on pipelines https://review.opendev.org/c/zuul/zuul/+/777479 | 22:44 | |
| @iwienand:matrix.org | > <@fungicide:matrix.org> ianw: not urgent, but if you get time could you weigh in on https://review.opendev.org/809899 with your thoughts? trying to make sure we've got consensus that this is a reasonable path forward | 22:49 |
| fungi: that all seems fine. no need to spend our limited attention spans on xenial now. i think the stack is all ready to go? | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Dequeue items after we're done with them https://review.opendev.org/c/zuul/zuul/+/811244 | 23:05 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: | 23:05 | |
| - [zuul/zuul] Make QueueItem a Zookeeper object https://review.opendev.org/c/zuul/zuul/+/809414 | ||
| - [zuul/zuul] Store pipeline state in Zookeeper https://review.opendev.org/c/zuul/zuul/+/810658 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] Sprinkle some set -e in the quickstart playbooks https://review.opendev.org/c/zuul/zuul/+/792489 | 23:29 | |
| @jim:acmegating.com | opendev is running master (well, as of a few commits ago), and so far the event queue is looking good. | 23:42 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] Codify branch matchers and refs https://review.opendev.org/c/zuul/zuul/+/794050 | 23:48 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!