| *** goneri has quit IRC | 00:04 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Install podman in container for container-based builds https://review.opendev.org/c/zuul/nodepool/+/756643 | 00:34 |
|---|---|---|
| fungi | ianw: yes, the backports sources will have a low enough priority that nothing should auto-install from them unless you explicitly request a package which is only in backports or provide a version constraint for a version only in backports or directly request the package from the backports suite instead of the default | 00:37 |
| mordred | ianw: ++ | 00:51 |
| *** jamesmcarthur has joined #zuul | 01:05 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Update to Python 3.9 https://review.opendev.org/c/zuul/zuul/+/789654 | 01:15 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Update to Python 3.9 https://review.opendev.org/c/zuul/nodepool/+/789655 | 01:15 |
| *** jamesmcarthur has quit IRC | 01:28 | |
| *** jamesmcarthur has joined #zuul | 01:46 | |
| *** jamesmcarthur has quit IRC | 01:50 | |
| *** jamesmcarthur has joined #zuul | 02:00 | |
| *** jamesmcarthur has quit IRC | 02:26 | |
| *** jamesmcarthur has joined #zuul | 02:27 | |
| *** evrardjp has quit IRC | 02:33 | |
| *** evrardjp has joined #zuul | 02:33 | |
| *** ikhan has joined #zuul | 02:48 | |
| *** jamesmcarthur has quit IRC | 03:25 | |
| *** bhavikdbavishi has joined #zuul | 03:52 | |
| *** ikhan has quit IRC | 04:17 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Update to Python 3.9 https://review.opendev.org/c/zuul/nodepool/+/789655 | 04:17 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Update to Python 3.9 https://review.opendev.org/c/zuul/zuul/+/789654 | 04:18 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Update to Python 3.9 https://review.opendev.org/c/zuul/nodepool/+/789655 | 04:20 |
| *** ykarel has joined #zuul | 04:20 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Update to Python 3.9 https://review.opendev.org/c/zuul/zuul/+/789654 | 04:27 |
| *** jfoufas1 has joined #zuul | 04:28 | |
| *** saneax has quit IRC | 04:29 | |
| *** ikhan has joined #zuul | 04:48 | |
| *** vishalmanchanda has joined #zuul | 04:51 | |
| *** ikhan has quit IRC | 04:52 | |
| *** bhavikdbavishi has quit IRC | 05:12 | |
| *** bhavikdbavishi has joined #zuul | 05:30 | |
| *** ikhan has joined #zuul | 06:02 | |
| *** ikhan has quit IRC | 06:06 | |
| *** jcapitao has joined #zuul | 06:29 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Fix resolveBranch working on outdated data https://review.opendev.org/c/zuul/zuul/+/789253 | 06:29 |
| *** bhavikdbavishi has quit IRC | 06:43 | |
| *** bhavikdbavishi has joined #zuul | 06:44 | |
| *** avass has joined #zuul | 06:51 | |
| *** bhavikdbavishi has quit IRC | 06:57 | |
| *** nils has joined #zuul | 07:07 | |
| *** rpittau|afk is now known as rpittau | 07:12 | |
| *** hashar has joined #zuul | 07:19 | |
| *** bhavikdbavishi has joined #zuul | 07:23 | |
| *** tosky has joined #zuul | 07:30 | |
| *** jpena|off is now known as jpena | 07:57 | |
| *** harrymichal has joined #zuul | 08:05 | |
| *** ianychoi_ has quit IRC | 08:13 | |
| lyr | Hi there | 08:27 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: add checkboxes to filter toolbar, filter builds by held, voting https://review.opendev.org/c/zuul/zuul/+/729265 | 08:28 |
| lyr | Is there a way to encrypt a secret globaly instead of on project level ? | 08:29 |
| lyr | Like encrypting my mattermost webhook globaly instead of once for each project where I want a notificaiton | 08:29 |
| avass | lyr: no but I believe you can ecrypt it and put the job config in a config project and then use that job in other projects | 08:31 |
| lyr | Ah you mean a job with only a post step, running the mattermost task | 08:32 |
| lyr | Hmm will that work ? Using the encryt cli I've to specify a project | 08:32 |
| mhu | encrypt it for the config project, and then use this job defined in the config project elsewhere | 08:36 |
| *** ikhan has joined #zuul | 08:36 | |
| avass | lyr: or are you running that in a post/cleanup step? | 08:40 |
| *** ikhan has quit IRC | 08:41 | |
| avass | lyr: but yeah it would work. you encrypt it and store it in a config project. I do something like that to get around github not allowing self approvals :) | 08:41 |
| avass | lyr: I think this is public: https://github.com/albinvass/zuul-config/blob/master/zuul.d/approve.yaml that sets up a project-template to make zuul approve PRs when a CODEOWNER comments /approve. In other projects I just apply that template to make it work | 08:46 |
| avass | but if you need to run something with secrets in a post-run then it would be easier to create a job with only a post-run like you say and then use that as a parent. (or you could add that post-run to the base job if everything should run it ) | 08:49 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Store tenants in unparsed abide as dict https://review.opendev.org/c/zuul/zuul/+/771456 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Always request all required project configuration https://review.opendev.org/c/zuul/zuul/+/788962 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Fix missing call to updateFromDict for mgmt events https://review.opendev.org/c/zuul/zuul/+/789574 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Stream API for sharded Zookeeper data https://review.opendev.org/c/zuul/zuul/+/771454 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add Zookeeper API to get the current logical time https://review.opendev.org/c/zuul/zuul/+/789575 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add a logical timestamp to management events https://review.opendev.org/c/zuul/zuul/+/789576 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Implement API for caching raw config files in ZK https://review.opendev.org/c/zuul/zuul/+/789577 | 08:50 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Cache unparsed config files in Zookeeper https://review.opendev.org/c/zuul/zuul/+/771455 | 08:50 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/c/zuul/zuul/+/734082 | 09:00 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Fix resolveBranch working on outdated data https://review.opendev.org/c/zuul/zuul/+/789253 | 09:01 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/c/zuul/zuul/+/735586 | 09:02 |
| *** bhavikdbavishi has quit IRC | 09:15 | |
| *** bhavikdbavishi has joined #zuul | 09:16 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | 09:18 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | 09:19 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold https://review.opendev.org/c/zuul/zuul/+/768115 | 09:21 |
| *** ykarel has quit IRC | 09:22 | |
| *** ykarel has joined #zuul | 09:22 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API, Web UI: add pipelines' manager, triggers data in status https://review.opendev.org/c/zuul/zuul/+/736968 | 09:30 |
| *** harrymichal has quit IRC | 09:31 | |
| *** saneax has joined #zuul | 09:43 | |
| *** holser has joined #zuul | 09:49 | |
| *** jcapitao is now known as jcapitao_lunch | 10:23 | |
| *** bhavikdbavishi has quit IRC | 10:25 | |
| *** bhavikdbavishi has joined #zuul | 10:30 | |
| *** hashar has quit IRC | 10:32 | |
| *** sshnaidm|afk is now known as sshnaidm | 10:34 | |
| *** ikhan has joined #zuul | 10:37 | |
| *** tosky has quit IRC | 10:38 | |
| *** tosky has joined #zuul | 10:38 | |
| *** ikhan has quit IRC | 10:42 | |
| *** tosky_ has joined #zuul | 10:45 | |
| *** tosky has quit IRC | 10:47 | |
| *** tosky_ is now known as tosky | 10:49 | |
| *** bhavikdbavishi1 has joined #zuul | 11:03 | |
| *** bhavikdbavishi has quit IRC | 11:05 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 11:05 | |
| *** Shrews has joined #zuul | 11:14 | |
| *** ikhan has joined #zuul | 11:17 | |
| *** jpena is now known as jpena|lunch | 11:26 | |
| *** ikhan has quit IRC | 11:38 | |
| *** jcapitao_lunch is now known as jcapitao | 11:57 | |
| *** jamesmcarthur has joined #zuul | 12:16 | |
| *** hashar has joined #zuul | 12:25 | |
| *** jpena|lunch is now known as jpena | 12:30 | |
| *** jamesmcarthur has quit IRC | 12:31 | |
| *** jamesmcarthur has joined #zuul | 12:33 | |
| *** saneax has quit IRC | 12:49 | |
| *** ssmashnuk has joined #zuul | 12:58 | |
| *** nhicher_ has quit IRC | 13:21 | |
| *** dpawlik has quit IRC | 13:22 | |
| *** hashar has quit IRC | 13:25 | |
| *** hashar has joined #zuul | 13:25 | |
| *** nhicher has joined #zuul | 13:33 | |
| *** jamesmcarthur has quit IRC | 13:36 | |
| *** ssmashnuk has quit IRC | 13:42 | |
| *** josefwells has joined #zuul | 13:43 | |
| *** bhavikdbavishi has quit IRC | 13:46 | |
| *** jamesmcarthur has joined #zuul | 13:52 | |
| openstackgerrit | Mahdi Hammouda proposed zuul/nodepool master: Revert "docker: don't daemonize when starting images" https://review.opendev.org/c/zuul/nodepool/+/789814 | 13:57 |
| corvus | does anyone have any context for that ^ ? | 13:59 |
| fungi | first i've seen it | 14:05 |
| fungi | i don't recall anyone mentioning in here having trouble with the docker images after the switch to running foreground | 14:05 |
| fungi | and the commit it's reverting merged over 2 years ago | 14:06 |
| fungi | looks like the revert was created by gerrit revert button on 635584 | 14:07 |
| fungi | so maybe an accident? | 14:07 |
| * Shrews wonders why he is a reviewer on that and thus getting emails | 14:07 | |
| fungi | Shrews: gerrit helpfully adds approvers from the reverted change on the revert | 14:07 |
| Shrews | so helpful | 14:08 |
| fungi | i mean, i guess that might be helpful if it were reverting a change which merged yesterday | 14:08 |
| * Shrews clicks the 'X' to remove himself | 14:08 | |
| *** ssmashnuk has joined #zuul | 14:27 | |
| *** ssmashnuk has quit IRC | 14:27 | |
| *** dpawlik5 has joined #zuul | 14:27 | |
| *** ssmashnuk has joined #zuul | 14:28 | |
| ssmashnuk | Good morning everyone, thanks again for all your help last week. My pilot project to move our dev/qa infrastructure from TFS into Zuul/Gerrit/Jira is looking promising however there are still a few things I need to grok, mostly around logging. Because we build on Windows I get that the realtime console streaming won't work out of the box. What | 14:29 |
| ssmashnuk | I'd appreciate some assistance with is 1) getting the ansible playbook output from the build nodes exposed in zuul/gerrit, and 2) understanding the ansible vars available to devs and typical workflows for capturing artifacts. | 14:29 |
| ssmashnuk | For #1 the trusted output comes through fine but the untrusted only shows the finger:// url. The logs are present on the executor in the work/logs/job-output.txt though. How can I get this propagated? I've also got an ARA container available. I see references to ARA in the docs, but nothing specific on getting it working. I've set ARA_API_CLIENT | 14:29 |
| ssmashnuk | and ARA_API_SERVER env vars set on the executor but I'm still missing something. | 14:29 |
| ssmashnuk | For #2 Are there any examples I can have a look at? Are there zuul helper roles which can be exposed here, or should these playbooks be straight up vanilla ansible? | 14:29 |
| dmsimard | ssmashnuk: good morning o/ | 14:30 |
| dmsimard | the implementation of ara from the zuul executor perspective is out of date right now, it's still using 0.x so the variables like ARA_API_CLIENT are not taken into account -- these are for 1.x | 14:30 |
| *** dpawlik5 is now known as dpawlik | 14:32 | |
| *** avass has quit IRC | 14:32 | |
| dmsimard | I had a very rough PoC a while back that I didn't follow through, in part due to lack or time and because I wasn't satisfied with it: https://review.opendev.org/c/zuul/zuul/+/694622 | 14:33 |
| ssmashnuk | dmsimard: thanks, ARA was more of a nice to have. | 14:35 |
| dmsimard | My zuul is rusty nowadays but I think the right approach would be to decouple it from zuul and make it so operators can enable arbitrary callbacks (like say, Ansible's profile_tasks) with a way to set the config | 14:36 |
| *** snktparik has joined #zuul | 14:53 | |
| corvus | dmsimard: ++ | 14:53 |
| snktparik | Hi, We are facing an issue in zuul. after restarting eks pod zuul reset our key pair. | 15:06 |
| snktparik | as a result's of it we are unable to decrypt our secrets. | 15:06 |
| snktparik | any one know how we can resolve this ? | 15:07 |
| tobiash | snktparik: you need to store the keys on a pvc | 15:07 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API, Web UI: add pipelines' manager, triggers data in status https://review.opendev.org/c/zuul/zuul/+/736968 | 15:07 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to promote a change https://review.opendev.org/c/zuul/zuul/+/781858 | 15:10 |
| *** avass has joined #zuul | 15:10 | |
| snktparik | tobiash you have any document for refrence. | 15:12 |
| tobiash | snktparik: more specifically you need to have the state dir on a pvc: https://zuul-ci.org/docs/zuul/discussion/components.html#attr-scheduler.state_dir | 15:13 |
| tobiash | the keys are stored within that directory and need to be persisted between restarts | 15:14 |
| snktparik | tobiash thanks. | 15:14 |
| *** ykarel has quit IRC | 15:28 | |
| *** patrick34 has joined #zuul | 15:40 | |
| patrick34 | Hi IRC people, quick question, do you know if enqueue/dequeue CLI features of zuul should be working for drivers other than gerrit? I use a git server, specifically gitea and seems the enqueue and dequeue cli commands are not working. (I opened this storyboard not long ago https://storyboard.openstack.org/#!/story/2008819) | 15:43 |
| mhu | have you tried with the admin CLI that comes with zuul? It uses the gearman protocol. If it works this way, then there's a problem with the REST API | 15:44 |
| patrick34 | the "admin" cli? humm.. checking | 15:44 |
| mhu | if it doesn't, it's a problem with the RPC server | 15:44 |
| mhu | patrick34, try "zuul --help" where zuul is running (server or container) | 15:46 |
| patrick34 | I did exactly that as you wrote it lol :D | 15:46 |
| mhu | great minds etc :) | 15:46 |
| patrick34 | also note I use zuul 3.19.1-1 | 15:47 |
| patrick34 | I don't see any thing related to an 'admin' in the help | 15:47 |
| lyr | Hi there. I've this error stack in zuulm-scheduler logs https://paste.garrigue.re/?f21f8d95a23631b1#9ccnpKbVA7KK4pqW8uycAyGVigPdfBEnJEr6DA9oBE3B. I don't get it, our Zuul's github app is configured for this organization, but this repository isn't defined in the resources.yaml (the projects), there's no zuul.yaml or zuul.d ... | 15:47 |
| clarkb | patrick34: you may need to use enqueue-ref? since git doesn't have a concept of changes? | 15:47 |
| lyr | This example happens on a PR after merge, dunno if it's systematic | 15:47 |
| clarkb | patrick34: iirc the enqueue command is very change centric | 15:48 |
| clarkb | (which could be a PR too) | 15:48 |
| fungi | clarkb: the example traceback in the linked story was for dequeue though | 15:48 |
| patrick34 | clarkb oh I haven't tried this indeed. I would also interested in a "dequeue-ref" tough? | 15:48 |
| mhu | patrick34, it's just that the CLI that comes with zuul is meant to be used as an admin. You don't need an auth token to use it. It'll get the gearman connection specifications from /etc/zuul/zuul.conf directly | 15:48 |
| mhu | hence "admin" | 15:48 |
| patrick34 | mhu ok yeah I run the commands from the server directly. Note I run this on a kubernetes cluster | 15:49 |
| clarkb | I think there is a corresponding dequeue-ref? | 15:49 |
| mhu | ok I don't think it should impact anything | 15:49 |
| fungi | patrick34: there's no dequeue-ref, dequeue should in theory handle all types of queue items | 15:49 |
| clarkb | ah dequeue-ref is rest api specific | 15:49 |
| fungi | i've used zuul dequeue in the past with a branch ref as well (with our periodic pipeline) | 15:50 |
| clarkb | lyr: is that project in your tenant config? (is that what resources.yaml is? will zuul read from there?) | 15:50 |
| clarkb | lyr: another thought is maybe that repo has your app configured on it so it is sending events to zuul | 15:50 |
| clarkb | (not sure how much filtering zuul does early) | 15:50 |
| lyr | resources.yaml is a software factory file. What I meant is, this repository ain't in the projects list | 15:52 |
| *** rpittau is now known as rpittau|afk | 15:52 | |
| lyr | My app is configured for all repos, which could explain that | 15:53 |
| clarkb | lyr: I would grep the event id (44487936-adb5-11eb-9362-b4fc8aadf4e7 in your example) and see what the genesis is | 15:54 |
| clarkb | but I suspect that the web hooks are firing and you are seeing this as a result | 15:54 |
| *** hashar has quit IRC | 15:54 | |
| lyr | ok | 15:56 |
| lyr | well I've updated my app installation for this org to only the relevant repositories instead of all of them | 15:56 |
| lyr | it's cleaner anyway, even if it requires one more step to add more later on | 15:56 |
| patrick34 | I just typed this and get some RPCfailure fileschanges unique https://gist.github.com/plaurin84/792c9140a571aaf8dee3df1bc660e2ba | 15:57 |
| clarkb | patrick34: that indicates the merge job called fileschanges failed when running that command. If you look in your merger logs you should find why (executors also act as mergers) | 15:59 |
| patrick34 | oh yes I see something in merger logs | 16:01 |
| patrick34 | stderr: 'fatal: Invalid revision range 0000000000000000000000000000000000000000..d37445c45090f5128abe5c15524a8996d76a23ca | 16:01 |
| patrick34 | detailed... https://gist.github.com/plaurin84/b445f1a5cfa51a978f7576ce94062916 | 16:02 |
| clarkb | I wonder if this is a sideffect of the consistent buildset ref changes | 16:03 |
| corvus | it looks like zuul doesn't handle dequeing refs like that | 16:03 |
| corvus | clarkb: no, this is the dequeue command not supplying an oldrev | 16:04 |
| patrick34 | ops I updated the gist I missed a part | 16:04 |
| clarkb | ah | 16:05 |
| corvus | and it's specific to the git driver | 16:05 |
| corvus | which attempts to supply a files list for branch updates; i don't think the gerrit driver does that. | 16:06 |
| corvus | so i think changes to zuul would be needed to make that work | 16:06 |
| patrick34 | ah! glad that might be a real bug and not just me misusing the zuul cli '=D | 16:08 |
| patrick34 | I'll be back a bit later lunchtime for me, thx for you help | 16:11 |
| *** snktparik has quit IRC | 16:12 | |
| *** hashar has joined #zuul | 16:22 | |
| *** jcapitao has quit IRC | 16:31 | |
| *** ssmashnuk has quit IRC | 16:36 | |
| *** jpena is now known as jpena|off | 16:46 | |
| *** bhavikdbavishi has joined #zuul | 16:55 | |
| *** bhavikdbavishi has quit IRC | 16:59 | |
| *** josefwells has quit IRC | 17:05 | |
| *** jamesmcarthur_ has joined #zuul | 17:08 | |
| *** jamesmcarthur has quit IRC | 17:11 | |
| corvus | tobiash: qq on https://review.opendev.org/789253 | 17:14 |
| tobiash | oh shit, overlooked that | 17:16 |
| tobiash | just a sec | 17:16 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Fix resolveBranch working on outdated data https://review.opendev.org/c/zuul/zuul/+/789253 | 17:17 |
| corvus | clarkb: ^ is ready for your re-review (fixes a regression from global repo state) | 17:18 |
| clarkb | looking | 17:18 |
| *** nils has quit IRC | 17:22 | |
| patrick34 | Hi I'm back. So, I guess the conversation he had earlier makes this valid? https://storyboard.openstack.org/#!/story/2008819 | 17:23 |
| patrick34 | If needed let me know if you want me to add data to this story and how I can help | 17:23 |
| *** jamesmcarthur_ has quit IRC | 17:23 | |
| *** hashar has quit IRC | 18:06 | |
| *** hashar has joined #zuul | 18:07 | |
| openstackgerrit | Merged zuul/zuul master: Fix implicit role repos missing in repo-state https://review.opendev.org/c/zuul/zuul/+/789252 | 18:14 |
| *** josefwells has joined #zuul | 18:22 | |
| *** vishalmanchanda has quit IRC | 18:46 | |
| corvus | clarkb, fungi, mordred: the yarn retry thing at https://review.opendev.org/789401 did not appear to work: https://zuul.opendev.org/t/zuul/build/00101e6628e84b38b6789ce4d9fb2b62/console | 18:49 |
| mordred | corvus: I agree with that assessment | 18:51 |
| clarkb | am I reading it correctly that it doesn't seem to have retried? | 18:51 |
| corvus | clarkb: that's how i read it (especially with timestamps continuously increasing) | 18:52 |
| corvus | the proximity of the last timestamp to 180s makes me wonder if there's a timeout i don't know about | 18:53 |
| corvus | but i can't find anything about a tox install_command timeout | 18:54 |
| mordred | is it because we're set -e in the script? | 18:54 |
| corvus | oh huh | 18:54 |
| mordred | so instead of the until operating the error, it's treating it as a bash fail? | 18:54 |
| mordred | because it definitely doesn't seem to have retried | 18:55 |
| corvus | yep | 18:55 |
| corvus | i just reprod that with a simple bin/false script | 18:55 |
| corvus | so the retry above doesn't work either i guess? | 18:55 |
| mordred | I love set -e - but I really do hate that it subverts all flow control in scripts | 18:55 |
| mordred | corvus: I'm gonna go with "no it doesn't work" | 18:56 |
| corvus | okay, so i'll remove -e and add in some strategic "|| exit 1" ? | 18:56 |
| mordred | maybe we shoudl just wrap those loops with set +e / set -e | 18:56 |
| mordred | or that | 18:57 |
| corvus | hrm. that's an idea... | 18:57 |
| mordred | whichever seems the least ugly to your eyes honestly | 18:57 |
| corvus | i think i like yours | 18:57 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Really retry yarn installs https://review.opendev.org/c/zuul/zuul/+/789935 | 18:58 |
| corvus | wait, tabs | 18:59 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Really retry yarn installs https://review.opendev.org/c/zuul/zuul/+/789935 | 18:59 |
| corvus | clarkb, fungi, mordred: ^ | 19:00 |
| clarkb | yup that should work since you explicitly exit 1 if you want to fail | 19:00 |
| mordred | +A | 19:01 |
| mordred | I wish bash had a "set -e execpt for flow control" | 19:01 |
| *** jfoufas1 has quit IRC | 19:01 | |
| *** hashar has quit IRC | 19:02 | |
| *** josefwells has quit IRC | 19:18 | |
| *** patrick34 has quit IRC | 19:21 | |
| *** josefwells has joined #zuul | 19:22 | |
| *** jamesmcarthur has joined #zuul | 19:24 | |
| *** nils has joined #zuul | 19:27 | |
| *** jamesmcarthur has quit IRC | 19:29 | |
| openstackgerrit | Merged zuul/zuul master: Really retry yarn installs https://review.opendev.org/c/zuul/zuul/+/789935 | 20:01 |
| *** fsvsbs has joined #zuul | 20:04 | |
| fsvsbs | Hi Tristan, qq we have the operator up and running almost ready to use in production... but I see that the persistent volumes are not being mounted | 20:06 |
| tristanC | fsvsbs: hi, you mean for postgres and zookeeper? | 20:08 |
| fsvsbs | scheduler | 20:11 |
| fsvsbs | The claim appears but there is no mount point | 20:11 |
| tristanC | fsvsbs: oh i see, it seems like they are not correctly assigned | 20:14 |
| fsvsbs | so when we restart the scheduler to get new tenants and repos we loose all of our secrets :( | 20:15 |
| tristanC | yep, let me fix data volumes to use the claim | 20:19 |
| fsvsbs | Thanks, sorry not much help on this one Dhall is not my friend, give me an operator like the awx one I could help, or in golang | 20:20 |
| corvus | fsvsbs: there is a rewrite to kopf/python in review starting at https://review.opendev.org/785039 | 20:21 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use the volume claim for the scheduler data https://review.opendev.org/c/zuul/zuul-operator/+/789941 | 20:27 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Use kopf operator framework https://review.opendev.org/c/zuul/zuul-operator/+/785039 | 20:31 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Bump API version to v1alpha2 https://review.opendev.org/c/zuul/zuul-operator/+/785047 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Run a git server in k8s for functional tests https://review.opendev.org/c/zuul/zuul-operator/+/785738 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Move ingress to functional test https://review.opendev.org/c/zuul/zuul-operator/+/785757 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Support externally managed Zookeeper and DB https://review.opendev.org/c/zuul/zuul-operator/+/785273 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Pass through extra scheduler config options https://review.opendev.org/c/zuul/zuul-operator/+/785277 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Add merger support https://review.opendev.org/c/zuul/zuul-operator/+/785278 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Support imagePrefix and versions https://review.opendev.org/c/zuul/zuul-operator/+/785279 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Support fingergw https://review.opendev.org/c/zuul/zuul-operator/+/785300 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Add docs https://review.opendev.org/c/zuul/zuul-operator/+/785083 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Support zuul-preview https://review.opendev.org/c/zuul/zuul-operator/+/785760 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Add support for zuul-registry https://review.opendev.org/c/zuul/zuul-operator/+/785761 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Remove extra 2 minute wait from tests https://review.opendev.org/c/zuul/zuul-operator/+/785762 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Add allowUnsafeConfig database setting https://review.opendev.org/c/zuul/zuul-operator/+/785764 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Pass through environment to scheduler, web and launcher https://review.opendev.org/c/zuul/zuul-operator/+/785988 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Allow terminationGracePeriodSeconds to be configurable https://review.opendev.org/c/zuul/zuul-operator/+/785989 | 20:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Flake8 cleanups https://review.opendev.org/c/zuul/zuul-operator/+/786349 | 20:33 |
| tristanC | fsvsbs: https://review.opendev.org/c/zuul/zuul-operator/+/789941 should make the scheduler use the volume claim. Though as corvus said, the zuul-operator is being rewritten with kopf, so you might want to use that instead | 20:33 |
| corvus | tristanC: +2; i just double checked the kopf stack, and i think it does already correctly have a volume + claim for /var/lib/zuul | 20:34 |
| corvus | (sorry for the patchset spam; i just noticed some review comments on the first patch, so addressed them and rebased) | 20:35 |
| fsvsbs | I would agree and I am sure we will swing back to use it and to contribute to it, as we are github enterprise (and bit bucket) org (FIS) | 20:35 |
| corvus | fsvsbs: just fyi, we're probably not going to have a good upgrade story between the current version and the kopf version; it might be a case of exporting the data and then re-importing it | 20:39 |
| fsvsbs | makes sense thanks | 20:40 |
| corvus | but i think saving a copy of the keys and doing a db export/import are probably all that would be required. so hopefully not too onerous | 20:40 |
| *** jamesmcarthur has joined #zuul | 20:41 | |
| fsvsbs | We use an external DB (AWS RDS) and so it should just be a backup of the keys hopefully | 20:41 |
| corvus | ++ | 20:42 |
| corvus | incidentally, we're moving those into zk right now; so we won't actually need the state volume pvc much longer. we'll make a utility to export/import keys from zk for backup purposes, but that doesn't exist yet. | 20:43 |
| *** jamesmcarthur has quit IRC | 20:54 | |
| *** jamesmcarthur has joined #zuul | 20:54 | |
| *** patrick34 has joined #zuul | 21:06 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use the volume claim for the scheduler data https://review.opendev.org/c/zuul/zuul-operator/+/789941 | 21:11 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add ensure-keystore-secret role https://review.opendev.org/c/zuul/zuul-operator/+/789943 | 21:11 |
| *** jamesmcarthur has quit IRC | 21:12 | |
| *** jamesmcarthur has joined #zuul | 21:13 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add ensure-keystore-secret role https://review.opendev.org/c/zuul/zuul-operator/+/789943 | 21:42 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use the volume claim for the scheduler data https://review.opendev.org/c/zuul/zuul-operator/+/789941 | 21:42 |
| *** nils has quit IRC | 21:47 | |
| *** nils has joined #zuul | 21:50 | |
| *** jamesmcarthur has quit IRC | 21:52 | |
| *** nils has quit IRC | 21:59 | |
| *** CrayZee has joined #zuul | 22:00 | |
| *** snapiri has quit IRC | 22:03 | |
| *** jamesmcarthur has joined #zuul | 22:06 | |
| *** jamesmcarthur has quit IRC | 22:10 | |
| *** jamesmcarthur has joined #zuul | 22:14 | |
| *** jamesmcarthur has quit IRC | 22:29 | |
| *** jamesmcarthur has joined #zuul | 22:30 | |
| *** holser has quit IRC | 22:33 | |
| *** rlandy is now known as rlandy|bbl | 22:33 | |
| *** tosky has quit IRC | 23:03 | |
| *** jamesmcarthur has quit IRC | 23:16 | |
| *** Shrews has quit IRC | 23:26 | |
| *** jamesmcarthur has joined #zuul | 23:36 | |
| *** jamesmcarthur has quit IRC | 23:45 | |
| *** jamesmcarthur has joined #zuul | 23:47 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!