| openstackgerrit | Merged zuul/zuul master: docs: remove zuuldocs domain https://review.opendev.org/744648 | 00:11 |
|---|---|---|
| openstackgerrit | Merged zuul/zuul master: github status api docs : update https://review.opendev.org/744406 | 00:11 |
| openstackgerrit | Merged zuul/zuul master: github docs: clarify situation with checks/status API https://review.opendev.org/744577 | 00:11 |
| *** rfolco has quit IRC | 01:02 | |
| tristanC | corvus: thanks, that sounds good. By the way, we recently released software-factory 3.5 which also enables zk tls. | 01:23 |
| *** hamalq has quit IRC | 01:28 | |
| *** rlandy|bbl is now known as rlandy | 02:19 | |
| *** bhavikdbavishi has joined #zuul | 02:56 | |
| *** bhavikdbavishi1 has joined #zuul | 02:59 | |
| *** bhavikdbavishi has quit IRC | 03:01 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:01 | |
| *** wxy has quit IRC | 03:13 | |
| *** bhavikdbavishi has quit IRC | 03:31 | |
| *** bhavikdbavishi has joined #zuul | 03:45 | |
| *** rishabhhpe has joined #zuul | 04:10 | |
| *** rishabhhpe has quit IRC | 04:19 | |
| *** bhavikdbavishi has quit IRC | 04:24 | |
| *** bhavikdbavishi has joined #zuul | 04:25 | |
| *** raukadah is now known as chkumar|rover | 04:31 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** vishalmanchanda has joined #zuul | 04:44 | |
| *** saneax has joined #zuul | 05:54 | |
| *** bhavikdbavishi1 has joined #zuul | 05:58 | |
| *** sgw1 has quit IRC | 05:58 | |
| *** bhavikdbavishi has quit IRC | 05:59 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 05:59 | |
| *** bhavikdbavishi has quit IRC | 06:30 | |
| *** jcapitao has joined #zuul | 07:02 | |
| *** bhavikdbavishi has joined #zuul | 07:05 | |
| *** bhagyashris is now known as bhagyashris|off | 07:13 | |
| *** hashar has joined #zuul | 07:33 | |
| *** jpena|off is now known as jpena | 07:51 | |
| *** bhavikdbavishi has quit IRC | 07:54 | |
| *** saneax has quit IRC | 07:54 | |
| *** nils has joined #zuul | 07:59 | |
| *** tosky has joined #zuul | 08:04 | |
| *** dmellado has quit IRC | 08:09 | |
| *** dmellado has joined #zuul | 08:14 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 08:19 | |
| *** saneax has joined #zuul | 08:21 | |
| *** saneax has quit IRC | 08:27 | |
| *** saneax has joined #zuul | 08:39 | |
| *** saneax has quit IRC | 08:40 | |
| openstackgerrit | Carlos Goncalves proposed zuul/zuul-jobs master: configure-mirrors: add CentOS 8 Stream https://review.opendev.org/734787 | 09:54 |
| *** bhavikdbavishi has joined #zuul | 09:59 | |
| *** bhavikdbavishi1 has joined #zuul | 10:04 | |
| *** bhavikdbavishi has quit IRC | 10:06 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 10:06 | |
| *** bhavikdbavishi has quit IRC | 10:51 | |
| *** bhavikdbavishi has joined #zuul | 10:53 | |
| *** jpena is now known as jpena|lunch | 11:32 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.opendev.org/644557 | 11:41 |
| *** cloudnull has quit IRC | 11:42 | |
| *** cloudnull has joined #zuul | 11:43 | |
| *** hashar is now known as hasharLunch | 11:46 | |
| *** vishalmanchanda has quit IRC | 11:58 | |
| *** rfolco has joined #zuul | 12:00 | |
| *** bhavikdbavishi1 has joined #zuul | 12:03 | |
| *** bhavikdbavishi has quit IRC | 12:04 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 12:04 | |
| *** jcapitao is now known as jcapitao_lunch | 12:08 | |
| *** rlandy has joined #zuul | 12:16 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: ensure-pip: add instructions for RedHat system https://review.opendev.org/743750 | 12:27 |
| *** jpena|lunch is now known as jpena | 12:36 | |
| *** bhavikdbavishi has quit IRC | 12:50 | |
| *** bhavikdbavishi has joined #zuul | 13:10 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Improve typings in context of 630472 https://review.opendev.org/715247 | 13:13 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Required SQL reporters https://review.opendev.org/630472 | 13:13 |
| *** jcapitao_lunch is now known as jcapitao | 13:16 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Add information about Patternfly 4 to Zuul web developer documentation https://review.opendev.org/745325 | 13:18 |
| *** bhavikdbavishi has quit IRC | 13:19 | |
| *** chkumar|rover is now known as raukadah | 13:32 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Required SQL reporters https://review.opendev.org/630472 | 13:38 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Add information about Patternfly 4 to Zuul web developer documentation https://review.opendev.org/745325 | 14:00 |
| *** hasharLunch is now known as hashar | 14:17 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Improve typings in context of 630472 https://review.opendev.org/715247 | 14:28 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Required SQL reporters https://review.opendev.org/630472 | 14:28 |
| *** yolanda has quit IRC | 15:12 | |
| *** yolanda has joined #zuul | 15:13 | |
| *** sgw1 has joined #zuul | 15:36 | |
| *** bolg has quit IRC | 15:37 | |
| *** hashar has quit IRC | 15:46 | |
| *** jpena is now known as jpena|off | 16:10 | |
| *** hamalq has joined #zuul | 16:12 | |
| *** hamalq has quit IRC | 16:13 | |
| *** hamalq has joined #zuul | 16:13 | |
| *** bhavikdbavishi has joined #zuul | 16:51 | |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Loop over incomplete subunit files properly https://review.opendev.org/745382 | 18:06 |
| clarkb | people that know ansible better than me should double check ^ | 18:07 |
| clarkb | hrm ansible subelements expects a dictionairy only and not a list of dictionairies? the docs are wrong then? | 18:25 |
| clarkb | I'll have to look after my bike ride | 18:25 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Loop over incomplete subunit files properly https://review.opendev.org/745382 | 18:33 |
| clarkb | maybe ? really popping out now though | 18:33 |
| *** bhavikdbavishi has quit IRC | 19:09 | |
| *** nils has quit IRC | 19:22 | |
| *** hashar has joined #zuul | 20:01 | |
| clarkb | corvus: when you have a moment ^ has me very stumped | 20:22 |
| clarkb | I think I've read the docs properly. I guess my next step is to rtfs the lookup plugin source | 20:23 |
| corvus | clarkb: i would just mock that up locally | 20:28 |
| corvus | nothing zuul specific there | 20:28 |
| clarkb | ya but will ansible tell me anything more useful? I guess I can add all the -vvvv | 20:30 |
| corvus | clarkb: and some "debug: {var: zk_item}" | 20:31 |
| *** rfolco has quit IRC | 20:57 | |
| clarkb | I get different behavior using loop and with_subelements :/ | 20:59 |
| clarkb | good news is that it works if I use with_subelements | 20:59 |
| clarkb | I'll just push that (the skip isn't necessary becaus ethe files list is [] if there were no matches we still get the key) | 20:59 |
| corvus | tristanC, tobiash, mnaser, mordred: remote: https://gerrit-review.googlesource.com/c/zuul/ops/+/278177 Use TLS Zookeeper connections [NEW] | 21:00 |
| corvus | that should switch google's zuul to using tls zk with cert-manager | 21:00 |
| corvus | (it's the tip of a 4-patch stack) | 21:01 |
| corvus | paladox: ^ fyi | 21:01 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Loop over incomplete subunit files properly https://review.opendev.org/745382 | 21:01 |
| clarkb | based on local testing ^ should work | 21:02 |
| clarkb | I don't understand why yet | 21:02 |
| clarkb | woot ^ passes testing now so I think the whole stack is ready for review | 21:02 |
| clarkb | er ww meant that to be in #opdnev | 21:05 |
| clarkb | I odn't think that hcange is done testing yet :) | 21:06 |
| corvus | friday afternoon, i believed you | 21:06 |
| clarkb | thats what I get for having two different CI issues being sorted at the same time :) | 21:07 |
| clarkb | I mean the bugs were in my code CI was doing its job | 21:07 |
| *** rlandy has quit IRC | 21:18 | |
| paladox | Nice! | 21:25 |
| paladox | (I’m currently New Jersey time atm) | 21:25 |
| *** hashar has quit IRC | 21:27 | |
| fungi | hope you're visiting one of the parts of "the garden state" from which it derives its name, and not one of the parts buried under trash heaps and dotted with oil refineries | 21:31 |
| paladox | Heh, it’s new jersey | 22:02 |
| corvus | that's one thing about new jersey everyone can agree on: "it's new jersey" :) | 22:07 |
| clarkb | https://review.opendev.org/#/c/745382/ is happy now and I'm reasonably confident in it given my local testing | 22:08 |
| clarkb | between that testing and the testing zuul does hopeflly we're good | 22:08 |
| corvus | clarkb: those look like ansible words. +2. | 22:09 |
| tristanC | corvus: that looks correct | 22:11 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Fix typo in nodepool configuration when user provide a ZooKeeper https://review.opendev.org/745421 | 22:15 |
| clarkb | corvus: env: ZOO_PORT is still 2181. also is that env dict where you need to set the rotation of the on disk data? | 22:17 |
| corvus | clarkb: i don't know what ZOO_PORT is used for :( | 22:17 |
| corvus | and i don't know specifically about rotation; gerrit's zuul is so low volume i didn't bother tuning. | 22:18 |
| clarkb | I just call that one out because it fills disks | 22:19 |
| clarkb | but maybe if the volume is minimal thats a several years in the future problem | 22:19 |
| corvus | yeah, definitely worth looking into (and getting right for any future operator work) | 22:19 |
| clarkb | ah I think it may be SNAPRETAINCOUNT which is set to 3 so probably fine | 22:19 |
| corvus | autopurge.purgeInterval=6 | 22:20 |
| corvus | that's what our docs say to set | 22:20 |
| tristanC | corvus: for the root ca (i guess https://gerrit-review.googlesource.com/c/zuul/ops/+/278176/1/k8s/certmanager.yaml#15 ), i don't remember why but python tls library was not able to use the cert without those key usages : https://opendev.org/zuul/zuul-operator/src/branch/master/conf/zuul/resources.dhall#L260 | 22:21 |
| clarkb | ah ya I htink you need the interval to be >0 as welll | 22:21 |
| corvus | clarkb: so yeah, we should set that in values.yaml | 22:22 |
| corvus | tristanC: oh weird; i have not actually tried loading that in python yet (i only tested with zk quorum and openss s_client connections) | 22:22 |
| tristanC | corvus: something related to sslerror "unsupported certificate purpose", at least when using certificate from cert-manager v0.14.0 | 22:23 |
| paladox | Corvus: lol | 22:37 |
| *** tosky has quit IRC | 22:58 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use the cert-manager for ZooKeeper https://review.opendev.org/745430 | 23:04 |
| tristanC | corvus: btw we need to update the zuul-operator spec to define how the user can provide zookeeper tls client cert when it provides its own zookeeper service (currently only the ZUUL_ZK_HOST url is defined) | 23:09 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use the cert-manager for ZooKeeper https://review.opendev.org/745430 | 23:09 |
| tristanC | or perhaps we should not bother with supporting external zookeeper service and simply remove the existing input.zookeeper option. | 23:17 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Use the cert-manager for ZooKeeper https://review.opendev.org/745430 | 23:19 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!