Monday, 2020-06-29

*** wuchunyang has joined #zuul01:05
*** wuchunyang has quit IRC01:15
*** wuchunyang has joined #zuul01:32
*** swest has quit IRC02:00
*** swest has joined #zuul02:14
*** wuchunyang has quit IRC02:27
*** wuchunyang has joined #zuul02:28
*** johanssone has quit IRC02:43
*** johanssone has joined #zuul02:45
*** jamesmcarthur has joined #zuul02:49
*** bhavikdbavishi has joined #zuul03:13
*** bhavikdbavishi1 has joined #zuul03:16
*** bhavikdbavishi has quit IRC03:18
*** bhavikdbavishi1 is now known as bhavikdbavishi03:18
*** jamesmcarthur has quit IRC03:28
*** jamesmcarthur_ has joined #zuul03:28
*** jamesmcarthur_ has quit IRC03:31
*** jamesmcarthur has joined #zuul03:33
*** wuchunyang has quit IRC03:34
*** jamesmcarthur has quit IRC03:37
*** wuchunyang has joined #zuul03:49
*** ysandeep|away is now known as ysandeep04:00
*** wuchunyang has quit IRC04:05
*** jamesmcarthur has joined #zuul04:15
*** bhavikdbavishi has quit IRC04:20
*** bhavikdbavishi has joined #zuul04:21
*** jamesmcarthur has quit IRC04:25
*** evrardjp has quit IRC04:33
*** evrardjp has joined #zuul04:33
*** bhavikdbavishi1 has joined #zuul04:44
*** bhavikdbavishi has quit IRC04:46
*** bhavikdbavishi1 is now known as bhavikdbavishi04:46
*** wuchunyang has joined #zuul04:57
*** jamesmcarthur has joined #zuul05:02
*** jamesmcarthur has quit IRC05:07
*** wuchunyang has quit IRC05:10
*** wuchunyang has joined #zuul05:15
*** bhagyashris is now known as bhagyashris|pto05:16
*** wuchunyang has quit IRC05:28
*** wuchunyang has joined #zuul05:29
*** dpawlik4 has joined #zuul06:21
*** wuchunyang has quit IRC06:23
*** bhavikdbavishi has quit IRC06:26
*** sgw has quit IRC06:33
*** ysandeep is now known as ysandeep|brb06:45
*** rpittau|afk is now known as rpittau06:51
*** jcapitao has joined #zuul07:02
*** ysandeep|brb is now known as ysandeep07:06
*** bhavikdbavishi has joined #zuul07:20
*** sshnaidm is now known as sshnaidm|ruck07:24
*** vishalmanchanda has joined #zuul07:31
*** jpena|off is now known as jpena07:56
*** nils has joined #zuul07:57
*** ysandeep is now known as ysandeep|lunch08:17
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper  https://review.opendev.org/70571608:46
*** ysandeep|lunch is now known as ysandeep08:50
*** jamesmcarthur has joined #zuul09:03
*** jamesmcarthur has quit IRC09:08
*** harrymichal has joined #zuul09:42
*** dennis_effa has joined #zuul09:47
*** hashar has joined #zuul09:50
*** rpittau is now known as rpittau|bbl10:21
openstackgerritTobias Henkel proposed zuul/zuul master: Log missing required status checks  https://review.opendev.org/71811410:43
*** jcapitao is now known as jcapitao_lunch10:45
*** fbo is now known as fbo|lunch10:48
openstackgerritTobias Henkel proposed zuul/zuul master: Optionally allow zoned executors to process unzoned jobs  https://review.opendev.org/67384010:53
*** tosky has joined #zuul10:57
*** bhavikdbavishi has quit IRC10:57
*** ysandeep is now known as ysandeep|brb11:02
*** fbo|lunch is now known as fbo11:07
*** wuchunyang has joined #zuul11:18
*** wuchunyang has joined #zuul11:19
*** jpena is now known as jpena|lunch11:32
*** wuchunyang has quit IRC11:36
*** bhavikdbavishi has joined #zuul11:40
*** ysandeep|brb is now known as ysandeep11:43
*** jamesmcarthur has joined #zuul11:44
*** rfolco has joined #zuul11:56
*** rlandy has joined #zuul11:59
*** Defolos_ has joined #zuul12:00
*** rpittau|bbl is now known as rpittau12:00
*** jamesmcarthur has quit IRC12:01
*** Defolos_ has quit IRC12:05
*** jcapitao_lunch is now known as jcapitao12:08
*** jamesmcarthur has joined #zuul12:08
*** ianw is now known as ianw_pto12:13
zbrhow can know if a specific job was a retry or not? i want not able to find that info inside manifest json file.12:19
avasszbr: https://zuul-ci.org/docs/zuul/reference/jobs.html#var-zuul.attempts12:24
avasszbr: is that what you want?12:24
*** weshay_ is now known as weshay_ruck12:26
*** harrymichal has quit IRC12:27
*** rlandy is now known as rlandy|training12:29
*** Goneri has joined #zuul12:48
zbravass: that is where you configure it, the question is where on a job logs you can find how many attempts were needed before it ended as failure or success.13:00
clarkbzbr: in the inventory file as that is where we include the zuul dict13:00
corvuszbr: the next time we restart opendev's zuul, we'll have retry attempts recorded in the database and queryable in the web ui (a patch recently landed for that)13:08
zbrcorvus: that is great news, especially after recent issues.13:13
zbrmy guess is that the failure rate on centos 8.2 is quite high, but it gets hidden due to the retries13:13
fungiin opendev's zuul, i assume you mean13:14
*** sgw has joined #zuul13:29
*** bschanzel has joined #zuul13:33
bschanzelHey tristanC, may I have a review on https://review.opendev.org/#/c/736984/ ? In the OpenShift nodepool driver you can set resource limits (cpu/mem) for pod nodes. This was missing in the k8s driver, so I added the equivalent there.13:35
tristanCbschanzel: thank you for reminding me, the change is looking good to me13:40
bschanzelThat was fast. Many thanks!13:41
corvusbschanzel: +313:41
tristanCbschanzel: i actually reviewed it previously, but somehow forgot to reply with the vote :-)13:43
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: ensure-dhall: update to latest version  https://review.opendev.org/73844813:44
*** masterpe has quit IRC13:46
*** holser_ has quit IRC13:48
openstackgerritJan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper  https://review.opendev.org/71622113:48
openstackgerritJan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper  https://review.opendev.org/71626213:48
openstackgerritJan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper  https://review.opendev.org/71687513:48
*** ysandeep is now known as ysandeep|afk13:50
*** holser has joined #zuul13:52
*** bhavikdbavishi has quit IRC13:52
zbrfungi: yes, opendev, i posted on the right channel as zuul specific issue is going to be fixed on restart.13:52
zbreven so, i am still wondering if we cannot improve the UX on such failures where zuul believes that node is out. currently the logs are lost, and a normal user will never know what happened that got the node down.13:53
fungizbr: sure, i wasn't questioning your choice of channel to ask in, just getting additional context14:00
zbrfungi: no worries, well other than the fact that this kind of node failure makes it very hard to debug.14:02
zbrideally zuul should be able to report such failures in a public place, seeing node console could prove very useful.14:03
fungigetting to the instance console requires permissions which zuul doesn't normally have14:03
fungi(nodepool launchers have it, zuul executors don't though)14:04
fungiat least in an openstack nova based cloud context14:04
clarkband in this case didn't provide any additional data (so while I agree would be need to figure out, it won't help with this particular bug)14:05
*** masterpe has joined #zuul14:07
fungiyeah, only kmesg entries on the instances we caught were iptables blocking unrelated traffic and some benign selinux enforcement messages14:12
fungicurrent assumption is it could be network configuration changes sometimes making the instances completely unreachable without crashing the server14:12
*** dennis_effa has quit IRC14:17
openstackgerritJames E. Blair proposed zuul/zuul master: Add the openafs-client package to the container image  https://review.opendev.org/73845214:17
corvusclarkb, fungi, mordred: ^  (re #opendev)14:17
openstackgerritSimon Westphahl proposed zuul/zuul master: Ensure refs for recent branches are not GCed  https://review.opendev.org/73845414:25
openstackgerritMerged zuul/nodepool master: Kubernetes Driver: Allow cpu/mem resource limits  https://review.opendev.org/73698414:26
*** jpena|lunch is now known as jpena14:29
*** dennis_effa has joined #zuul14:31
*** sgw1 has joined #zuul14:56
*** ysandeep|afk is now known as ysandeep|away14:57
openstackgerritTobias Henkel proposed zuul/zuul master: Improve typings in context of 630472  https://review.opendev.org/71524715:00
openstackgerritTobias Henkel proposed zuul/zuul master: Required SQL reporters  https://review.opendev.org/63047215:00
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: add i18n support, french translations  https://review.opendev.org/73729015:12
*** bschanzel has quit IRC15:36
*** hashar has quit IRC15:36
openstackgerritMerged zuul/zuul master: Add the openafs-client package to the container image  https://review.opendev.org/73845215:39
*** hashar has joined #zuul15:52
openstackgerritElod Illes proposed zuul/zuul-jobs master: DNM: ensure-pip to upgrade pip3 if version is 8.X.X  https://review.opendev.org/73847016:00
*** jcapitao has quit IRC16:01
openstackgerritMatthieu Huin proposed zuul/zuul master: Builds: add "held" attribute  https://review.opendev.org/72899416:03
openstackgerritMatthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations  https://review.opendev.org/72811816:05
openstackgerritMatthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST  https://review.opendev.org/72841016:09
openstackgerritMatthieu Huin proposed zuul/zuul master: REST API: add promote endpoint  https://review.opendev.org/72848916:10
openstackgerritMatthieu Huin proposed zuul/zuul master: CLI: support promote via the REST API  https://review.opendev.org/72890716:10
openstackgerritMatthieu Huin proposed zuul/zuul master: web UI: add Autoholds Requests page  https://review.opendev.org/72930716:10
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: add checkbox filters to TableFilters component, filter builds by held status  https://review.opendev.org/72926516:11
*** bhavikdbavishi has joined #zuul16:13
*** bhavikdbavishi1 has joined #zuul16:16
*** vishalmanchanda has quit IRC16:17
*** bhavikdbavishi has quit IRC16:18
*** bhavikdbavishi1 is now known as bhavikdbavishi16:18
*** Goneri has quit IRC16:20
*** hamalq has joined #zuul16:24
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: add i18n support, french translations  https://review.opendev.org/73729016:24
*** Goneri has joined #zuul16:33
*** jamesmcarthur has quit IRC16:34
*** dennis_effa has quit IRC16:37
openstackgerritMatthieu Huin proposed zuul/zuul master: Web UI: add i18n support, french translation  https://review.opendev.org/73729016:54
zbrcorvus: any idea when zuul will be restarted next time? is there a planned moment or is when forced to do it by other event.16:58
clarkbzbr: we tend to do it as we are able17:02
clarkbmy hunch is we'll restart at some point soon to switch zk tls back on now that the fix upstream has merged but that may depend on a kazoo release?17:02
tobiashwe either need a kazoo release for this or pull it from master depending on how frequent they do releases :)17:03
fungi2.7.0 was in march, the last release before that (2.6.1) was january 201917:05
fungiso it does not see frequent releases17:05
fungi3 releases in the past two years17:06
*** jpena is now known as jpena|off17:09
*** nils has quit IRC17:12
*** hashar has quit IRC17:12
tobiashthere were quite a few releases that were requested via issues so I created one: https://github.com/python-zk/kazoo/issues/62117:22
fungigreat idea17:38
corvustobiash: ++ and even better not coming from me, so they know there's more than one person affected :)17:43
tobiash:)17:43
*** bhavikdbavishi has quit IRC18:00
openstackgerritMerged zuul/nodepool master: doc: openshiftpods handles python-path too  https://review.opendev.org/73137318:25
openstackgerritElod Illes proposed zuul/zuul-jobs master: Upgrade pip3 if version is 8.X.X  https://review.opendev.org/73847018:30
corvusavass: are you sure we need the cert installed system-wide (ie, 'update-ca-certificates') for the publication registry?18:30
corvusoh that might be for podman18:30
corvusnm18:32
*** Goneri has quit IRC18:43
*** Goneri has joined #zuul18:55
*** jamesmcarthur has joined #zuul19:07
openstackgerritMatthieu Huin proposed zuul/zuul master: REST API, Web UI: implement nodes filtering  https://review.opendev.org/73604219:10
*** sshnaidm|ruck is now known as sshnaidm|afk19:12
*** hashar has joined #zuul19:25
*** Goneri has quit IRC20:06
*** y2kenny has joined #zuul20:25
*** tobiash has quit IRC20:51
*** sshnaidm|afk has quit IRC20:51
*** tobiash has joined #zuul20:51
*** sshnaidm|afk has joined #zuul20:53
*** jamesmcarthur has quit IRC21:05
corvusavass, mordred: fyi, i've been plugging away at the temp registry thing; making good progress.21:09
*** hashar has quit IRC21:29
avasscorvus: I believe it should be21:30
*** rlandy|training is now known as rlandy21:30
avasscorvus: but I guess the test playbooks would only work on debian atm21:31
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Use a temporary registry with buildx  https://review.opendev.org/73851721:33
corvusavass, mordred: okay, let's see how that goes :) -- a few notes: i don't think we need ssl for either of these uses if we just use 127.0.0.1, so i simplified it a bit.  and if this works and we like it, i propose we squash it into the parent change21:34
avasscorvus: hmm, I thought I read that it requires ssl for for authentication to work21:36
avassoh but not for the temporary registry21:36
*** Goneri has joined #zuul21:37
corvusavass: i think there's an exception for 'insecure' registries, and 127.0.0.1 is automatically classified as insecure21:45
avasscorvus: yeah I guess there is. this is what I was thinking about: https://docs.docker.com/registry/deploying/#native-basic-auth which says 'cannot'21:53
avassbut should probably be 'cannot unless you configure it as an insecure registry'21:53
corvusavass: yeah, i'm guessing they just omitted that; i did test it out on that held node we've been using22:06
corvusoh, haha, i should really not make a root level tasks directory22:08
*** armstrongs has joined #zuul22:08
corvuszuul switched from thinking it was a collection-of-roles repo to a single-role-repo22:08
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Use a temporary registry with buildx  https://review.opendev.org/73851722:09
avasscorvus: yeah that might be good :)22:10
*** armstrongs has quit IRC22:18
avasscorvus: right, use-buildset-registry sets up /etc/buildkit/buildkit.toml as well22:20
*** Goneri has quit IRC22:22
*** tosky has quit IRC22:46
*** Goneri has joined #zuul22:47
*** rlandy has quit IRC23:00

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!