| *** rfolco has quit IRC | 00:07 | |
| *** hamalq has quit IRC | 00:18 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: ConfigItem*Exception : add failure context https://review.opendev.org/744090 | 00:50 |
|---|---|---|
| *** rlandy has quit IRC | 01:19 | |
| *** rfolco has joined #zuul | 01:34 | |
| *** rfolco has quit IRC | 01:49 | |
| *** bhavikdbavishi has joined #zuul | 03:12 | |
| *** bhavikdbavishi1 has joined #zuul | 03:17 | |
| *** bhavikdbavishi has quit IRC | 03:19 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:19 | |
| *** adam_g has quit IRC | 03:32 | |
| *** adam_g has joined #zuul | 03:37 | |
| tobiash | ianw: oh yes, pre is the right thing then | 03:43 |
| tobiash | I wonder if switching from a drawer to a dialog would be better then (given creates problema with scrolling anyway) | 03:45 |
| ianw | maybe? would that be like a modal dialog that overlays the page? | 03:46 |
| tobiash | Yes that would be the idea, not as fancy as the drawer but with more space for the errors and less problems generally | 03:50 |
| tobiash | felixedel: what do you think? ^ | 03:52 |
| *** saneax has joined #zuul | 04:00 | |
| *** bhavikdbavishi has quit IRC | 04:25 | |
| *** bhavikdbavishi has joined #zuul | 04:27 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** sgw1 has quit IRC | 04:40 | |
| *** raukadah is now known as chkumar|rover | 05:06 | |
| *** reiterative has quit IRC | 05:43 | |
| *** reiterative has joined #zuul | 05:43 | |
| *** bhavikdbavishi1 has joined #zuul | 06:07 | |
| *** bhavikdbavishi has quit IRC | 06:08 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 06:08 | |
| *** vishalmanchanda has joined #zuul | 06:09 | |
| *** bhavikdbavishi has quit IRC | 06:31 | |
| felixedel | tobiash: Not sure. What I (personally) don't like about the modal is that it kind of blocks the page as long as it's open. Also IIRC we would still have the "you need to click on the page content first to scroll via keys" issue even without the drawer as this seems to be related to the fixed navbar. I will try another approach which might even serve our purpose without being much of a workaround, but I didn't came to | 06:42 |
| felixedel | that yet | 06:42 |
| tobiash | or can the drawer made be wider so we can see more of the error messages? | 06:44 |
| felixedel | good question. Need to check this out | 06:47 |
| felixedel | There is a "Width for drawer panel at various breakpoints" setting that can be applied to the drawer | 06:48 |
| felixedel | setting -> property | 06:48 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Dequeue changes via github checks API https://review.opendev.org/709135 | 07:02 |
| *** jcapitao has joined #zuul | 07:07 | |
| *** bhavikdbavishi has joined #zuul | 07:10 | |
| *** jpena|off is now known as jpena | 07:13 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper https://review.opendev.org/716221 | 07:41 |
| *** tosky has joined #zuul | 07:46 | |
| *** sugaar has joined #zuul | 07:54 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Add releasenote about PF4 migration https://review.opendev.org/745085 | 07:57 |
| *** nils has joined #zuul | 08:13 | |
| *** hashar has joined #zuul | 08:14 | |
| *** arxcruz|off is now known as arxcruz | 08:42 | |
| *** iurygregory has quit IRC | 08:51 | |
| *** bhavikdbavishi has quit IRC | 08:53 | |
| *** iurygregory has joined #zuul | 09:02 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Remove unneeded api requests when commenting in github https://review.opendev.org/744194 | 10:17 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Move reports from FakeGithubConnection to github data https://review.opendev.org/745107 | 10:17 |
| *** bhavikdbavishi has joined #zuul | 10:24 | |
| *** bhavikdbavishi1 has joined #zuul | 10:27 | |
| *** bhavikdbavishi has quit IRC | 10:28 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 10:29 | |
| *** saneax has quit IRC | 10:42 | |
| *** bhavikdbavishi1 has joined #zuul | 10:48 | |
| *** bhavikdbavishi has quit IRC | 10:48 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 10:48 | |
| *** bhavikdbavishi1 has joined #zuul | 10:51 | |
| *** bhavikdbavishi has quit IRC | 10:53 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 10:53 | |
| sshnaidm | hi, I'm trying to set up pair of jobs that parent waits for children to finish with "pause", but parent job exits w/o waiting, what do I do wrong? http://paste.openstack.org/show/796628/ | 11:29 |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** armstrongs has joined #zuul | 11:31 | |
| *** sshnaidm is now known as sshnaidm|afk | 11:33 | |
| *** rfolco has joined #zuul | 11:37 | |
| *** jcapitao is now known as jcapitao_lunch | 11:45 | |
| *** sshnaidm|afk is now known as sshnaidm | 11:49 | |
| sshnaidm | tristanC, maybe you know ^ | 11:51 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper https://review.opendev.org/716221 | 12:08 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper https://review.opendev.org/716875 | 12:08 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 12:08 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: Switch to using zookeeper instead of gearman for jobs (keep gearman for mergers) https://review.opendev.org/744416 | 12:08 |
| *** saneax has joined #zuul | 12:08 | |
| *** armstrongs has quit IRC | 12:16 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Driver event ingestion https://review.opendev.org/717299 | 12:17 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper https://review.opendev.org/705716 | 12:17 |
| *** rlandy has joined #zuul | 12:17 | |
| tristanC | sshnaidm: that looks correct to me. Perhaps the pause needs to happen in the run phase, not post? | 12:35 |
| *** jpena|lunch is now known as jpena | 12:36 | |
| sshnaidm | tristanC, ack, will try | 12:36 |
| *** cloudnull has quit IRC | 12:43 | |
| *** cloudnull has joined #zuul | 12:44 | |
| *** rishabhhpe has joined #zuul | 12:46 | |
| rishabhhpe | Hi , I am facing issue while streaming the events from gerrit. can anyone please help me on this . | 12:46 |
| rishabhhpe | packet_write_wait: connection to unknown port 65535 : Broken pipe ----- getting this error | 12:49 |
| *** rlandy is now known as rlandy|mtg | 13:01 | |
| *** jcapitao_lunch is now known as jcapitao | 13:17 | |
| *** rishabhhpe has quit IRC | 13:19 | |
| *** rishabhhpe has joined #zuul | 13:20 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Avoid to use 'length' filter with null value https://review.opendev.org/742316 | 13:36 |
| tobiash | rishabhhpe: could it be that you have a firewall in between that kills the connection after some time? | 13:46 |
| *** bhavikdbavishi has quit IRC | 13:51 | |
| *** rlandy|mtg is now known as rlandy | 13:56 | |
| rishabhhpe | tobiash: firewall status is inactive | 13:57 |
| *** sgw1 has joined #zuul | 13:59 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: ensure-pip: add instructions for RedHat system https://review.opendev.org/743750 | 14:00 |
| *** rishabhhpe has quit IRC | 14:07 | |
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use (upload|promote)-docker-image roles in periodic jobs https://review.opendev.org/740560 | 15:03 |
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use (upload|promote)-docker-image roles in periodic jobs https://review.opendev.org/740560 | 15:04 |
| *** bhavikdbavishi has joined #zuul | 15:13 | |
| *** hashar has quit IRC | 15:19 | |
| *** chkumar|rover is now known as raukadah | 15:20 | |
| *** jcapitao has quit IRC | 15:28 | |
| clarkb | is there a canonical way to identify the root location of all the git repos on a test node? I know we supply a specific value for each repo but what provides /home/zuul/src/ ? | 15:31 |
| clarkb | hrm zuul.project.src_dir is relative to the assumed $HOME I think | 15:32 |
| *** jpena is now known as jpena|off | 15:38 | |
| *** rlandy is now known as rlandy|brb | 15:55 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add event queue sizes to stats reporting https://review.opendev.org/745175 | 15:56 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add event queue sizes to stats reporting https://review.opendev.org/745175 | 16:04 |
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use *-docker-image roles in periodic jobs https://review.opendev.org/740560 | 16:05 |
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use *-docker-image roles in periodic jobs https://review.opendev.org/740560 | 16:10 |
| *** iurygregory has quit IRC | 16:10 | |
| *** iurygregory has joined #zuul | 16:11 | |
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use *-docker-image roles in periodic jobs https://review.opendev.org/740560 | 16:12 |
| *** hamalq has joined #zuul | 16:17 | |
| *** hamalq has quit IRC | 16:19 | |
| *** hamalq has joined #zuul | 16:20 | |
| *** rlandy|brb is now known as rlandy | 16:22 | |
| openstackgerrit | Benjamin Schanzel proposed zuul/zuul master: Log complete output of kubectl port-forward https://review.opendev.org/745182 | 16:26 |
| openstackgerrit | Andrii Ostapenko proposed zuul/zuul-jobs master: Add ability to use *-docker-image roles in periodic jobs https://review.opendev.org/740560 | 16:34 |
| clarkb | zuulians if anyone else has a moment for https://review.opendev.org/#/c/744565/ that would be great as we have at least one user that would help out (thanks tobiash for the review) | 17:49 |
| *** nils has quit IRC | 17:55 | |
| *** gmann is now known as gmann_afk | 18:35 | |
| corvus | clarkb: +2 with comment -- i'll leave it to you to self-approve (with or without a followup) or revise | 18:36 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Fix partial subunit stream logging https://review.opendev.org/744565 | 18:40 |
| clarkb | corvus: ^ something like that? | 18:40 |
| corvus | clarkb: yes! +3 | 18:41 |
| corvus | i'm doing some zk tls experiments -- i just wanted to share this error code from zk: SSL_NULL_WITH_NULL_NULL | 18:43 |
| clarkb | hopefully NULLs are NULL | 18:44 |
| *** sshnaidm is now known as sshnaidm|afk | 18:47 | |
| fungi | that's... an awesome error | 18:50 |
| fungi | enough of those and you don't have to worry about a null lull | 18:51 |
| *** vishalmanchanda has quit IRC | 18:57 | |
| *** tosky has quit IRC | 19:20 | |
| tobiash | who needs useful error codes :) | 20:37 |
| clarkb | I'm debugging the opensuse 15.1 package instaslls error affecting zuul-jobs | 20:37 |
| clarkb | (holding the node in order to see what zypper configs look like) | 20:37 |
| *** bhavikdbavishi has quit IRC | 20:49 | |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Drop suse mirror types in our zypper repo configs https://review.opendev.org/745225 | 21:00 |
| clarkb | I think ^ will fix the opensuse issues but we should let testing confirm | 21:00 |
| clarkb | yup zuul has given it a +1 | 21:19 |
| openstackgerrit | Merged zuul/zuul-jobs master: Drop suse mirror types in our zypper repo configs https://review.opendev.org/745225 | 21:41 |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix partial subunit stream logging https://review.opendev.org/744565 | 21:56 |
| *** rlandy is now known as rlandy|bbl | 22:16 | |
| ianw | corvus: if you get a sec to look at the other two changes in the stack @ https://review.opendev.org/#/q/status:open+project:zuul/zuul+branch:master+topic:zuul-github-status that would be great. just doc changes | 22:29 |
| *** gmann_afk is now known as gmann | 22:35 | |
| corvus | ianw: ack | 22:35 |
| corvus | tobiash, tristanC, mnaser: i opened a PR to add ssl support for the zk helm chart: https://github.com/helm/charts/pull/23472 | 22:36 |
| corvus | i have that working locally with certs created with cert-manager | 22:36 |
| mnaser | corvus: wow that’s awesome. I wonder if we can add to required projects and then deploy cert manager and nodepool with it and ssl | 22:37 |
| mnaser | So we can do depends on | 22:37 |
| mnaser | I mean, we can :) I don’t think I can find the time to help with that immediately | 22:38 |
| corvus | wow, you can't see the circleci results without making a circleci account | 22:54 |
| corvus | ianw: qq on 744577 | 23:08 |
| *** armstrongs has joined #zuul | 23:08 | |
| ianw | corvus: oh yeah, right, that was what led me to the first change changing the domain :) | 23:09 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: github docs: clarify situation with checks/status API https://review.opendev.org/744577 | 23:10 |
| *** armstrongs has quit IRC | 23:17 | |
| *** saneax has quit IRC | 23:50 | |
| openstackgerrit | Merged zuul/zuul master: Dequeue changes via github checks API https://review.opendev.org/709135 | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!