| *** marios is now known as marios|ruck | 05:51 | |
| opendevreview | Merged zuul/zuul master: Fix race when canceling a node request https://review.opendev.org/c/zuul/zuul/+/800994 | 06:50 |
|---|---|---|
| *** rpittau|afk is now known as rpittau | 07:49 | |
| *** bhagyashris_ is now known as bhagyashris | 08:05 | |
| *** bhagyashris_ is now known as bhagyashris | 09:35 | |
| opendevreview | Matthieu Huin proposed zuul/zuul master: [api][cors] Add CORS configuration https://review.opendev.org/c/zuul/zuul/+/767691 | 09:59 |
| opendevreview | Simon Westphahl proposed zuul/zuul master: Set runtime related system attributes on reconfig https://review.opendev.org/c/zuul/zuul/+/801749 | 10:05 |
| opendevreview | Matthieu Huin proposed zuul/zuul master: UI: add pagination to builds, buildsets search https://review.opendev.org/c/zuul/zuul/+/793467 | 10:33 |
| *** jcapitao is now known as jcapitao_lunch | 10:40 | |
| *** jcapitao_lunch is now known as jcapitao | 12:17 | |
| opendevreview | Simon Westphahl proposed zuul/zuul master: wip: Cache system config in Zookeeper https://review.opendev.org/c/zuul/zuul/+/801859 | 13:48 |
| opendevreview | James E. Blair proposed zuul/zuul master: Add commands to export/import keys to/from ZK https://review.opendev.org/c/zuul/zuul/+/800854 | 15:07 |
| opendevreview | James E. Blair proposed zuul/project-config master: Add gating.dev publish job https://review.opendev.org/c/zuul/project-config/+/801899 | 15:58 |
| *** rpittau is now known as rpittau|afk | 16:02 | |
| opendevreview | James E. Blair proposed zuul/gating.dev master: Add initial job framework https://review.opendev.org/c/zuul/gating.dev/+/801901 | 16:02 |
| corvus | mordred: ^ maybe we can get that moving when you have a minute | 16:02 |
| TylerPearce[m] | Hello! I'm trying to do a fresh install of Zuul on an Ubuntu EC2 instance, following the steps at https://zuul-ci.org/docs/zuul/howtos/zuul_install.html, but I'm running into issues with re2. Does anyone know how to resolve this? | 16:16 |
| TylerPearce[m] | Details: Initially I hit "fatal error: re2/re2.h: No such file or directory" when running `sudo pip3 install .`, so I tried installing it manually by cloning & installing libre2 and pyre2. I had to make some minor changes to re2.cpp in this step, but managed to successfully run `sudo pip3 install .`. However, I'm still running into an error when running `sudo zuul-manage-ansible`: 'File "re2.pyx", line 1, in init re2 (src/re2.cpp:13681) | 16:16 |
| TylerPearce[m] | NameError: basestring' | 16:16 |
| corvus | Tyler Pearce: we can probably fix that -- but first -- have you considered running Zuul from containers instead? many of us are finding it much simpler | 16:18 |
| TylerPearce[m] | corvus: Are there instructions on how to do this? I'm a novice so I could probably use some help heh | 16:19 |
| corvus | Tyler Pearce: (if you do want to install from source, since you're on an ubuntu system, i'd suggest "apt install libre2-dev libre2-9" -- i think that should get the required dependencies -- though the numbers might vary depending on the distro) | 16:19 |
| corvus | Tyler Pearce: yes, 1 sec and i'll get a link | 16:20 |
| TylerPearce[m] | corvus: Thank you so much! | 16:20 |
| corvus | Tyler Pearce: this should get you up and going quickly with containers: https://zuul-ci.org/docs/zuul/tutorials/quick-start.html | 16:20 |
| corvus | one advantage that has is that we literally run the instructions in that tutorial on every commit to Zuul, so we're pretty sure it works. it's also very similar to how we run zuul's own zuul. | 16:22 |
| corvus | installing from source and running the components manually should still work of course, you'll just learn a lot more along the way, possibly more than you want :) | 16:22 |
| *** marios|ruck is now known as marios|out | 16:24 | |
| TylerPearce[m] | Nice. I'm planning to use GitHub instead of Gerrit, I'm assuming I can look through the docker file to figure out how to spin up the containers minus gerrit | 16:24 |
| corvus | Tyler Pearce: exactly. also, if the tutorial content is useful for you, you can start with the built-in gerrit and then add github and remove gerrit. which system is used for code review isn't that important to zuul. | 16:25 |
| fungi | yeah, gerrit is included in the quickstart so that you have some self-contained system to be able to exercise zuul end to end, but a bit of surgery on the list of containers and zuul configuration and you can connect it to any or as many code review systems as you want | 16:25 |
| TylerPearce[m] | Thank you so much! | 16:26 |
| corvus | Tyler Pearce: see https://zuul-ci.org/docs/zuul/reference/drivers/github.html for how to configure zuul for github (including on the github side and the pipeline configuration) | 16:26 |
| fungi | so if you want at some point you could connect your zuul not only to github but also gitlab.com and several different gerrits and a pagure... and have dependencies between changes in all of those | 16:26 |
| TylerPearce[m] | Perfect, thank you for the link (And for being so responsive and helpful :) ) | 16:28 |
| TylerPearce[m] | Seems like Zuul is pretty configurable! Very neat | 16:28 |
| opendevreview | Merged zuul/zuul master: Copy tenants dictionary before modification https://review.opendev.org/c/zuul/zuul/+/801439 | 16:49 |
| clarkb | corvus: I'm going to try and catch back up on reviews after lunch today. https://review.opendev.org/c/zuul/zuul/+/799127 https://review.opendev.org/c/zuul/zuul/+/800854 https://review.opendev.org/c/zuul/zuul/+/800800 looks like important ones. Any others? | 18:11 |
| corvus | clarkb: those are important; https://review.opendev.org/801422 and https://review.opendev.org/801658 are easy | 18:14 |
| clarkb | ok | 18:14 |
| opendevreview | Merged zuul/zuul master: Sort tenant list by name https://review.opendev.org/c/zuul/zuul/+/801442 | 18:20 |
| opendevreview | James E. Blair proposed zuul/gating.dev master: Add initial content https://review.opendev.org/c/zuul/gating.dev/+/801919 | 18:20 |
| corvus | mordred: and i definitely want your ack on ^ since i'm copying your text :) | 18:21 |
| opendevreview | James E. Blair proposed zuul/gating.dev master: Add initial content https://review.opendev.org/c/zuul/gating.dev/+/801919 | 18:23 |
| clarkb | pabelanger[m]: I also meant to ask if the nodepool release has solved the problem you had (just to double check) | 18:38 |
| mordred | corvus: +2 on the project-config change - the other one isn't happy without that one landed | 19:11 |
| mordred | corvus: but it lgtm | 19:12 |
| corvus | mordred: thanks! clarkb can you look at https://review.opendev.org/801899 too? it's a config-project change, so it's a blocker | 19:30 |
| opendevreview | Merged zuul/zuul master: Log result payload size of merger jobs https://review.opendev.org/c/zuul/zuul/+/801422 | 19:37 |
| *** sshnaidm is now known as sshnaidm|afk | 20:10 | |
| pabelanger[m] | clarkb: yes, it did! nodes booting properly again | 20:25 |
| pabelanger[m] | only thing left to work on, is the new gather facts logic on the executor side, for nodes that come online but don't have routing setup properly | 20:26 |
| clarkb | corvus: can do. Just sitting down now and pulling changes up | 20:26 |
| clarkb | corvus: does https://review.opendev.org/c/zuul/project-config/+/801899/1/zuul.d/jobs.yaml need a parent job specified to do something usefl? It just seems to assume there is information locally? | 20:31 |
| corvus | clarkb: nope, it needs a child job. it's a duplicate of the job above | 20:45 |
| clarkb | oh its the base job | 20:45 |
| corvus | ya | 20:45 |
| clarkb | ok I don't know why that didn't occur to me | 20:45 |
| opendevreview | Merged zuul/project-config master: Add gating.dev publish job https://review.opendev.org/c/zuul/project-config/+/801899 | 20:45 |
| clarkb | but makes perfect sense once I've flipped the perspective around | 20:45 |
| opendevreview | James E. Blair proposed zuul/gating.dev master: Add initial job framework https://review.opendev.org/c/zuul/gating.dev/+/801901 | 20:53 |
| clarkb | corvus: swest comment on https://review.opendev.org/c/zuul/zuul/+/800800 | 20:55 |
| clarkb | it feels a bit weird to undermine the constant in that class and then override the vars set by __init__ with the constnat | 20:55 |
| corvus | clarkb: wfm. | 20:58 |
| clarkb | corvus: for the import export of secrets the serialized form of that is a private key itself? | 21:19 |
| corvus | clarkb: it's the password-protected private key + metadata; exactly what's in zk | 21:32 |
| corvus | clarkb: you can see it in the added test fixtures actually | 21:32 |
| corvus | i obtained them by running an export of the existing private keys we use for testing. | 21:33 |
| clarkb | ya I guess I'm mostly surprised that is how openssl records them | 21:34 |
| clarkb | but I guess that ensures you have all the metadata necessary to decrypt | 21:34 |
| clarkb | corvus: ok left some thoughts on that one | 21:43 |
| clarkb | more thinking about operating mechanisms and test coverage than specific code improvements. THe code seems to be correct | 21:43 |
| corvus | clarkb: replied with followup q's | 22:09 |
| opendevreview | Merged zuul/zuul master: Remove OrderedDict for tenants https://review.opendev.org/c/zuul/zuul/+/801658 | 22:11 |
| opendevreview | James E. Blair proposed zuul/gating.dev master: Add initial content https://review.opendev.org/c/zuul/gating.dev/+/801919 | 22:11 |
| clarkb | corvus: responded | 22:16 |
| corvus | mordred: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_516/801919/3/check/gating.dev-build/516d875/html/ it's a start :) | 22:19 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!