| *** jamesmcarthur has quit IRC | 00:07 | |
| *** igordc has quit IRC | 00:28 | |
| jhesketh | corvus, tobiash: added some comments about the multi-connection SQL discussion to 621479. Let me know if it makes sense or if I can help further :-) | 00:29 |
|---|---|---|
| *** rfolco has joined #zuul | 00:55 | |
| *** igordc has joined #zuul | 00:59 | |
| *** rfolco has quit IRC | 01:29 | |
| *** jamesmcarthur has joined #zuul | 01:47 | |
| *** jamesmcarthur has quit IRC | 02:06 | |
| *** jamesmcarthur has joined #zuul | 02:11 | |
| *** jamesmcarthur has quit IRC | 02:18 | |
| *** jamesmcarthur has joined #zuul | 02:18 | |
| *** jamesmcarthur has quit IRC | 02:24 | |
| *** jamesmcarthur has joined #zuul | 02:49 | |
| *** jamesmcarthur has quit IRC | 02:55 | |
| *** 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 | |
| *** jamesmcarthur has joined #zuul | 03:50 | |
| *** armstrongs has joined #zuul | 03:54 | |
| *** armstrongs has quit IRC | 04:04 | |
| *** rlandy|bbl is now known as rlandy | 04:08 | |
| *** igordc has quit IRC | 04:08 | |
| *** jamesmcarthur has quit IRC | 04:25 | |
| *** jamesmcarthur has joined #zuul | 04:28 | |
| *** jamesmcarthur has quit IRC | 04:33 | |
| *** jamesmcarthur has joined #zuul | 04:56 | |
| *** jamesmcarthur has quit IRC | 05:02 | |
| *** mattw4 has quit IRC | 05:21 | |
| *** swest has joined #zuul | 05:26 | |
| *** reiterative has quit IRC | 05:41 | |
| *** reiterative has joined #zuul | 05:41 | |
| *** raukadah is now known as chkumar|rover | 05:57 | |
| *** jamesmcarthur has joined #zuul | 05:58 | |
| *** jamesmcarthur has quit IRC | 06:03 | |
| *** jamesmcarthur has joined #zuul | 06:38 | |
| *** jamesmcarthur has quit IRC | 06:42 | |
| *** bolg has joined #zuul | 06:57 | |
| *** bolg has quit IRC | 07:10 | |
| *** yolanda has quit IRC | 07:32 | |
| *** jamesmcarthur has joined #zuul | 07:39 | |
| *** jamesmcarthur has quit IRC | 07:44 | |
| *** jamesmcarthur has joined #zuul | 07:55 | |
| *** jamesmcarthur has quit IRC | 07:59 | |
| *** jpena|off is now known as jpena | 08:10 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Protect prime_installation_map with mutex https://review.opendev.org/704932 | 08:12 |
| *** sshnaidm|afk is now known as sshnaidm | 08:12 | |
| *** dmsimard5 has joined #zuul | 08:31 | |
| *** dmsimard has quit IRC | 08:33 | |
| *** dmsimard5 is now known as dmsimard | 08:33 | |
| *** hashar has joined #zuul | 08:39 | |
| *** tosky has joined #zuul | 08:46 | |
| *** jpena is now known as jpena|off | 09:00 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: fetch-output-openshift: initial role https://review.opendev.org/682044 | 09:11 |
| AJaeger | tristanC: docs work now ^ | 09:34 |
| *** bhavikdbavishi has quit IRC | 09:42 | |
| *** bhavikdbavishi has joined #zuul | 09:43 | |
| *** bhavikdbavishi has quit IRC | 09:47 | |
| *** zxiiro has quit IRC | 10:03 | |
| *** pcaruana has quit IRC | 10:46 | |
| *** zbr has quit IRC | 10:52 | |
| *** zbr has joined #zuul | 11:23 | |
| *** zbr has quit IRC | 11:35 | |
| *** zbr has joined #zuul | 11:35 | |
| *** pcaruana has joined #zuul | 11:48 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Report retried builds in a build set via mqtt. https://review.opendev.org/632727 | 12:07 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Test that retries don't trigger fail-fast https://review.opendev.org/704983 | 12:07 |
| *** rfolco has joined #zuul | 12:08 | |
| *** bolg has joined #zuul | 12:08 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Report retried builds via sql reporter. https://review.opendev.org/633501 | 12:09 |
| *** pcaruana has quit IRC | 12:21 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Test that retries don't trigger fail-fast https://review.opendev.org/704983 | 12:24 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Protect prime_installation_map with mutex https://review.opendev.org/704932 | 12:35 |
| *** bhavikdbavishi has joined #zuul | 12:41 | |
| tristanC | AJaeger: thanks! | 12:43 |
| *** bhavikdbavishi1 has joined #zuul | 12:44 | |
| *** bhavikdbavishi has quit IRC | 12:46 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 12:46 | |
| *** rlandy has joined #zuul | 12:59 | |
| hashar | good morning | 13:24 |
| hashar | tristanC: it seems you wrote the the zuul react page and I have a tiny feature request for it ;] | 13:25 |
| hashar | hopefully it is very simple, else I will fill an issue/story or something | 13:25 |
| hashar | when listing the playbook console and all the play / tasks, it would be very helpful to add a column for the duration of the plays / tasks | 13:26 |
| hashar | for example on https://zuul.opendev.org/t/zuul/build/86d17d1791b94d1aacebd8617799f372/console | 13:26 |
| hashar | ;) | 13:26 |
| tristanC | hashar: sure, i can help for that. note that i wrote a guide to get started in react: https://www.softwarefactory-project.io/react-for-python-developers.html | 13:31 |
| *** bhavikdbavishi has quit IRC | 13:31 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add spec for scale out scheduler https://review.opendev.org/621479 | 13:31 |
| * hashar looks for py2react | 13:32 | |
| hashar | tristanC: I gave it a try last night but all of that react/jsx is way over my brain capacity right now unfortunately. Though I only wasted a couple hours on it | 13:32 |
| hashar | will read that doc for sure | 13:32 |
| tristanC | hashar: i can help you find your way in the zuul implementation. I didn't wrote the console interface, but i may be able to figure out how to add a column | 13:34 |
| hashar | it was just a wild idea I had yesterday, nothing blocking ;] | 13:37 |
| hashar | I am trying to make those tests faster, notably the AnsibleZuulTestCase ones | 13:38 |
| openstackgerrit | Antoine Musso proposed zuul/zuul master: executor: avoid duplicate debug logs https://review.opendev.org/705010 | 13:45 |
| tristanC | hashar: adding something like `ai.push(<p>{task.duration}</p>)` to https://opendev.org/zuul/zuul/src/branch/master/web/src/containers/build/Console.jsx#L200 should add the column you want | 13:46 |
| hashar | tristanC: I will give it a try locally :] | 13:48 |
| hashar | I at least managed to run it locally. Thank you! | 13:48 |
| tristanC | hashar: nice, let me know if you have issue with react/jsx, i'm happy to help | 13:54 |
| *** harrymichal has joined #zuul | 14:01 | |
| *** pcaruana has joined #zuul | 14:03 | |
| *** zxiiro has joined #zuul | 14:05 | |
| *** sshnaidm is now known as sshnaidm|afk | 14:07 | |
| *** sshnaidm|afk is now known as sshnaidm | 14:32 | |
| *** pcaruana has quit IRC | 14:34 | |
| zbr | can we change the summary page top include a link to job history for specific project? | 14:41 |
| zbr | now you have to click to job, which has history, but is history on entire tenant, which is not of much use | 14:42 |
| zbr | if we have direct link, we can include both job name and project name | 14:42 |
| tristanC | zbr: what is `the summary page` ? | 14:43 |
| zbr | https://zuul.opendev.org/t/openstack/build/a8ce4c9611d2426a89d81524717bebd8 | 14:43 |
| tristanC | zbr: oh, in the build page, then to add a new link, this would be in that file: https://opendev.org/zuul/zuul/src/branch/master/web/src/containers/build/Summary.jsx#L33 | 14:47 |
| tristanC | zbr: adding something like `<Link ...>build history for that project</Link>` to https://opendev.org/zuul/zuul/src/branch/master/web/src/containers/build/Summary.jsx#L52 should do the trick | 14:49 |
| *** swest has quit IRC | 14:50 | |
| *** migi has joined #zuul | 14:51 | |
| tristanC | zuul-maint: would it be possible to get review on https://review.opendev.org/#/q/topic:zuul-crd . it would be really useful to get the operator image published so that we can start using it! | 14:52 |
| *** pcaruana has joined #zuul | 15:02 | |
| *** bhavikdbavishi has joined #zuul | 15:02 | |
| *** swest has joined #zuul | 15:03 | |
| *** swest has quit IRC | 15:05 | |
| *** rishabhhpe has joined #zuul | 15:05 | |
| rishabhhpe | HI All, Can anyone check the error i am getting here http://paste.openstack.org/show/788980/ .. while creating an instance from nodepool | 15:05 |
| Shrews | rishabhhpe: that does not look like a nodepool error. you might want to ask in the neutron channel | 15:07 |
| *** harrymichal has quit IRC | 15:07 | |
| rishabhhpe | Shrews: actually this one is generated by neutron while nodepool is calling to create the instance | 15:08 |
| Shrews | rishabhhpe: that may be, but the error occurs within neutron operations. you'll need to start there to see what may cause it | 15:09 |
| rishabhhpe | can u please tell me the channel name for neutron | 15:10 |
| Shrews | rishabhhpe: fwiw, nodepool doesn't call any neutron operations directly. | 15:10 |
| Shrews | #openstack-neutron, iirc | 15:10 |
| rishabhhpe | shrews: thanks | 15:11 |
| *** bhavikdbavishi1 has joined #zuul | 15:16 | |
| *** bhavikdbavishi has quit IRC | 15:17 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 15:17 | |
| *** mattw4 has joined #zuul | 15:22 | |
| *** chkumar|rover is now known as raukadah | 15:24 | |
| corvus | tobiash, bolg: i thought of one more problem with the scale out scheduler spec overnight; left a comment on 621479 | 15:26 |
| *** hashar has quit IRC | 15:29 | |
| bolg | corvus: will check it out | 15:34 |
| *** rishabhhpe has quit IRC | 15:46 | |
| tobiash | corvus: thanks, regarding sharding of uncached files. I think it can add quite a bit of complexity and maybe it's easier to just start without and support a max file size of 1mb? | 15:48 |
| zbr | tristanC: thanks for the hints on jsx. i am now writing the PR, i only need to figure out which var contains the project name | 15:49 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: POC: Add build history link to summary https://review.opendev.org/705049 | 15:54 |
| corvus | tobiash: i think if we just do it by size it should be easy -- basicaly jobs.yaml/1 is up to 1mb, and anything > is in jobs.yaml/2 | 15:56 |
| corvus | tobiash: we will probably almost never use more than /1, but if someone does, that way it's not an error | 15:57 |
| tobiash | corvus: ok, we should just not forget to test this :) | 16:03 |
| corvus | hah yes | 16:04 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add gcloud_service auth option for Gerrit driver https://review.opendev.org/704904 | 16:09 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Gerrit: poll for merged changes if no stream events https://review.opendev.org/705053 | 16:09 |
| *** plaurin has joined #zuul | 16:16 | |
| plaurin | gmorning irc people | 16:16 |
| *** mattw4 has quit IRC | 16:22 | |
| *** mattw4 has joined #zuul | 16:31 | |
| *** rishabhhpe has joined #zuul | 16:33 | |
| *** hashar has joined #zuul | 16:37 | |
| *** michael-beaver has quit IRC | 16:50 | |
| plaurin | tristanC: Hi! I have couple of questions regarding the fetch-output-openshift module | 16:51 |
| *** jamesmcarthur has joined #zuul | 16:54 | |
| tristanC | plaurin: hello, what's your questions? | 16:55 |
| clarkb | TIL about .dockerignore and how it can break assumptions in opendev/python-builder | 16:55 |
| corvus | plaurin: (you can usually just go ahead and ask your question without preamble and people will get around to responding when they check irc) | 16:56 |
| corvus | clarkb: how can i learn what you learned? | 16:56 |
| tristanC | clarkb: yeah i'm curious too :) | 16:56 |
| plaurin | tristanC: There is a similar task in the main.yml and rsync.yaml. "Ensure local output dirs" and "Ensure local directory exists". The latter fails for some unexpected reason for me. But also I don't why the zuul.executor.work_root are created on the target pod? | 16:57 |
| plaurin | '' I don't get why [...] | 16:57 |
| plaurin | corvus: right, this is not slack lol | 16:57 |
| clarkb | corvus: tristanC I'm trying to build a docker image for refstack that follows some of our assumptions and practices. So I've ported a simple dockerfile to it that basically just attempts to build and install python things and build the js things. Except I kept getting pip errors about not a valid git repo when trying to build an sdist for refstack in opendev/python-builder. I fired up the build container and | 16:58 |
| clarkb | looked in /tmp/src and sure enough there is no .git dir. Googling around I discoverd that .dockerignore is a thing and refstack set that to not copy the .git dir contents. https://opendev.org/openstack/refstack/src/branch/master/.dockerignore | 16:58 |
| tristanC | plaurin: https://review.opendev.org/#/c/682044/6/roles/fetch-output-openshift/tasks/main.yaml@11 is running on the executor, that's to prevent rsync issue | 16:58 |
| tristanC | plaurin: https://review.opendev.org/#/c/682044/6/roles/fetch-output-openshift/tasks/rsync.yaml@2 is running on the pod, that's also to prevent rsync issue (iirc it fails if source or target dir are absent) | 16:59 |
| corvus | clarkb: wow, that is "neat" | 16:59 |
| clarkb | corvus: tristanC basically the COPY for . into /tmp/src that we use with python-builder is somewhat at odds with a .dockerignore | 16:59 |
| clarkb | I've just deleted the file and now I get real build errors like gcc not found (now to update bindep) | 16:59 |
| corvus | clarkb: yeah, i think the .dockerignore should be removed and "RUN rm" added as desired to compensate | 16:59 |
| tristanC | plaurin: `zuul.executor` is not meant to reference pod path, it's just data related to the executor env | 17:00 |
| Shrews | oh, that reminds me of things i learned yesterday... we don't seem to use it anymore, but this outlines reasons we should avoid alpine for python images: https://pythonspeed.com/articles/alpine-docker-python/ | 17:00 |
| corvus | Shrews: ++ | 17:00 |
| tristanC | plaurin: could you share the failure log? | 17:00 |
| plaurin | tristanC: so, hum where do I put my log files? in {{ zuul_output_dir }}/logs ? | 17:00 |
| *** rishabhhpe has quit IRC | 17:01 | |
| plaurin | tristanC: https://gist.github.com/plaurin84/06a955666cb06ddd450f0a0afe5441bc | 17:01 |
| plaurin | I should add more debug logging I guess | 17:02 |
| tristanC | plaurin: hmm, that second task looks odd indeed, it shouldn't be necessary to create zuul.executor.work_root on the pod. that seems like a typo | 17:02 |
| tristanC | plaurin: but yes, a job should put log files in "{{ zuul_output_dir }}/logs" | 17:02 |
| plaurin | the loop could only use - "{{ log_path }}" maybe | 17:03 |
| tristanC | plaurin: seems like the error got removed by the no_log, could you comment that line please: https://review.opendev.org/#/c/682044/6/roles/fetch-output-openshift/tasks/rsync.yaml@22 | 17:04 |
| plaurin | sure | 17:05 |
| plaurin | That doesn't change anything because I don't get to the "Fetch zuul-output from the pod" task | 17:08 |
| plaurin | exactly the same log as previously | 17:08 |
| plaurin | let me check if I can get some debug output fomr the file task | 17:09 |
| *** bhavikdbavishi has quit IRC | 17:09 | |
| tristanC | plaurin: hmm, perhaps also remove the file task from the rsync.yaml file too | 17:09 |
| plaurin | okay I was thinking about trying that, let me test | 17:09 |
| *** hashar has quit IRC | 17:13 | |
| plaurin | okay I get a loop of missing directories in "Fetch zuul-output from the pod". I'll ensure to create the one I need before and only keep that one in the loop | 17:13 |
| *** mattw4 has quit IRC | 17:14 | |
| plaurin | tristanC: I also realise this module uses {{ ansible_user_dir }} instead of the default Workdir of /tmp, so it's a bit confusing, because now it's trying to use /home/ubuntu (as I expected to be in /tmp/zuul-output) | 17:15 |
| plaurin | tristanC: Okay it works now!! Here's my current rsync.yaml https://gist.github.com/plaurin84/d0a959b7b94866e8817171d166e62c7f | 17:21 |
| tristanC | plaurin: nice! | 17:21 |
| plaurin | I create and put files in the {{ ansible_user_dir }}/zuul-output/logs during my normal job run | 17:21 |
| plaurin | so then it works with jobs that directory in the loop | 17:22 |
| plaurin | err '' so then it works with just that directory in the loop >> {{ zuul_output_dir }}/logs | 17:23 |
| tristanC | plaurin: thank you for the feedback, i'll have a look to fix https://review.opendev.org/682044 later today | 17:25 |
| plaurin | tristanC: glad I could help, thx for your help too | 17:25 |
| Shrews | tristanC: was it intentional to remove the zuul-operator-build-image job from gate too? (re: https://review.opendev.org/#/c/702103/7/.zuul.yaml) | 17:26 |
| Shrews | if it passes check, not seeing the reason to remove from gate | 17:28 |
| tristanC | Shrews: no it was not, i guess that will prevent merging changes | 17:28 |
| Shrews | i'm genuinely curious how zuul would deal with that :) | 17:28 |
| corvus | Shrews: i think it would sit there and ignore it | 17:29 |
| corvus | http://zuul.opendev.org/t/zuul/project/opendev.org/zuul/zuul-operator shows no other jobs | 17:29 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Update sdk to v0.13.0 and install dhall-to-json https://review.opendev.org/702103 | 17:29 |
| *** evrardjp has joined #zuul | 17:33 | |
| *** armstrongs has joined #zuul | 18:02 | |
| *** igordc has joined #zuul | 18:09 | |
| *** armstrongs has quit IRC | 18:12 | |
| *** sshnaidm is now known as sshnaidm|afk | 18:20 | |
| *** adamw has quit IRC | 18:27 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Gerrit: poll for merged changes if no stream events https://review.opendev.org/705053 | 18:30 |
| *** hashar has joined #zuul | 18:46 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 18:53 | |
| *** mattw4 has joined #zuul | 18:58 | |
| hashar | tristanC: thank you for your React guide :] Looks like magic now on my machine! | 18:59 |
| tristanC | hashar: that's great to hear thanks | 19:00 |
| hashar | they even found out a proper way to reload everything as soon as I saving a file modification | 19:03 |
| hashar | very handy | 19:03 |
| mnaser | i have a very trivial change here that only needs +W: https://review.opendev.org/#/c/704756/ | 19:10 |
| *** hashar has quit IRC | 19:23 | |
| *** felixedel has joined #zuul | 19:33 | |
| Shrews | mnaser: seems reasonable to me. +3 | 19:39 |
| corvus | mnaser: out of curiosity, do you happen to have a cache-evection event json handy? | 19:43 |
| mnaser | corvus: i dont because its not logged i guess, but i can try and capture one | 19:50 |
| mnaser | corvus: have something up right now and ill grab you a copy when i have one | 19:51 |
| mnaser | corvus: `{"cacheName":"web_sessions","key":"aS2hpHsFTJilOkNB-c-urn5dJPNkBOKnaq","type":"cache-eviction","eventCreatedOn":1580413898}` | 19:51 |
| corvus | mnaser: thanks! that looks very boring indeed. :) | 19:53 |
| mnaser | :) | 19:53 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: fetch-output-openshift: initial role https://review.opendev.org/682044 | 20:00 |
| *** hashar has joined #zuul | 20:04 | |
| tristanC | plaurin: ^ seems to work as excepted, https://review.opendev.org/#/c/682049/ ran with it successfully | 20:12 |
| *** bolg has quit IRC | 20:12 | |
| plaurin | 😁️ | 20:13 |
| hashar | tristanC: I got it working thank you very much ;] Will polish up something worth a review tomorrow! | 20:14 |
| *** felixedel has quit IRC | 20:23 | |
| openstackgerrit | Merged zuul/zuul master: Start ignoring certain Gerrit events https://review.opendev.org/704756 | 20:35 |
| *** jamesmcarthur has quit IRC | 20:39 | |
| *** jamesmcarthur has joined #zuul | 20:49 | |
| *** jamesmcarthur has quit IRC | 20:53 | |
| *** jamesmcarthur has joined #zuul | 20:57 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Setup Prelude and Kubernetes binding https://review.opendev.org/702104 | 21:00 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new dhall function https://review.opendev.org/702106 | 21:00 |
| *** jamesmcarthur has quit IRC | 21:02 | |
| *** jamesmcarthur has joined #zuul | 21:03 | |
| *** jamesmcarthur has quit IRC | 21:10 | |
| *** jamesmcarthur has joined #zuul | 21:10 | |
| openstackgerrit | Antoine Musso proposed zuul/zuul master: web: show duration of each task in console https://review.opendev.org/705120 | 21:17 |
| *** rfolco has quit IRC | 21:24 | |
| *** rfolco has joined #zuul | 21:24 | |
| *** plaurin has quit IRC | 21:24 | |
| *** jamesmcarthur has quit IRC | 21:25 | |
| *** jamesmcarthur has joined #zuul | 21:27 | |
| *** jamesmcarthur has quit IRC | 21:29 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new dhall function https://review.opendev.org/702106 | 21:32 |
| *** jamesmcarthur has joined #zuul | 21:35 | |
| hashar | tristanC: I got my zuul-web patch done for react using momentjs for some human() duration ;] | 21:41 |
| hashar | I wanted to use https://github.com/codebox/moment-precise-range/ to have them formatted as 3 minutes 12 seconds 125 miliseconds but could not figure out ;) | 21:41 |
| tristanC | hashar: it seems easier to use something like that https://review.opendev.org/#/c/704191/ | 21:43 |
| corvus | we can make that a util function and import it in multiple components | 21:44 |
| hashar | tristanC: but it is not moment() friendly ! :D | 21:45 |
| tristanC | hashar: the patch looks good, i suggested some hardcoded alignment, the console i picked randomly had multiple hosts with different name, ( build/84627509738444928a702ba51b5401ff/console ) | 21:45 |
| *** armstrongs has joined #zuul | 21:45 | |
| hashar | thanks will catch up tomorrow | 21:46 |
| hashar | and give a try at factoring out the code to some common util function | 21:46 |
| tristanC | i think it would be nice to keep the moment() output, and add a tooltip overlay with the precise notation (e.g. using the `title` dom attribute) | 21:48 |
| *** jamesmcarthur has quit IRC | 21:49 | |
| *** jamesmcarthur has joined #zuul | 21:51 | |
| *** sgw has quit IRC | 21:54 | |
| *** armstrongs has quit IRC | 21:55 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new dhall function https://review.opendev.org/702106 | 21:56 |
| *** jamesmcarthur has quit IRC | 21:57 | |
| *** jamesmcarthur has joined #zuul | 22:03 | |
| *** jamesmcarthur_ has joined #zuul | 22:05 | |
| *** jamesmcarthur has quit IRC | 22:08 | |
| *** jamesmcarthur_ has quit IRC | 22:08 | |
| *** jamesmcarthur has joined #zuul | 22:09 | |
| *** jamesmcarthur has quit IRC | 22:11 | |
| *** sgw has joined #zuul | 22:11 | |
| ssbarnea | i need some feedback re the new job history link i want to add to summary pages, see it at https://7526c4c70fcb636961b8-9585b566393241b76329bc2140bbefac.ssl.cf5.rackcdn.com/705049/1/check/zuul-build-dashboard/f94d613/npm/html/build/6012014d91f747d1830545c99ddbd60d | 22:34 |
| ssbarnea | is after the build url (missing a space) | 22:34 |
| ssbarnea | link works, space is should be easy to add. | 22:35 |
| ssbarnea | i wanted to avoid cluttering the table with an extra row | 22:35 |
| corvus | ssbarnea: seems good. you might say "build history for this job" or at least "build history" since it's not entirely clear what "history" means without more context. | 22:37 |
| ssbarnea | i can also put tooltip info. i guess use of icons is not encouraged. | 22:38 |
| ssbarnea | i was also considering implementing a feature that reports relative job duration compared with average, to measure speed up/down, but that seems quite advanced for the moment. | 22:39 |
| corvus | should be possible, but for performance, probably the best way to do that would be to add it into the api result for the build endpoint. the time database is about to change significantly, so it might be best to wait a bit. | 22:45 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Replace existing operator tasks with the new dhall function https://review.opendev.org/702106 | 22:46 |
| *** hashar has quit IRC | 22:51 | |
| *** paladox is now known as paladox|UKInEU | 23:00 | |
| *** zxiiro has quit IRC | 23:05 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Generate TLS certificats for the gearman service https://review.opendev.org/702716 | 23:08 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Handle service restart when connections are changed https://review.opendev.org/703624 | 23:08 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add tenant reconfiguration when main.yaml changed https://review.opendev.org/703631 | 23:08 |
| corvus | tristanC: do we have a way to add packages to the standard executor image? ie, it seems like the on-executor dependencies in zuul-jobs should be there (openstacksdk, google, boto, etc) | 23:43 |
| openstackgerrit | James E. Blair proposed zuul/zuul-helm master: Set ClusterIP to None for executor logs https://review.opendev.org/705132 | 23:45 |
| corvus | mnaser: ^ that seemed to be a bug in the helm charts (was causing streaming logs not to work because the dns name of the executor did not resolve. that seemed to fix it) | 23:45 |
| corvus | tristanC: oh, i guess for the *python* requirements we add them to the ansible config | 23:47 |
| tristanC | corvus: the k8s task failed because of a missing openshift library on the job nodeset node | 23:52 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add OpenShift SCC and functional test https://review.opendev.org/702758 | 23:54 |
| corvus | tristanC: sorry i was asking because of a problem of my own making, but i think i have refreshed my memory now :) | 23:54 |
| tristanC | corvus: i'm not entirely sure what needs to be installed on the executor or the test instance, but i guess if we want to enable using k8s tasks on localhost, then `openshift` needs to be installed there | 23:58 |
| tristanC | for the zuul-operator integration job, running `python3 -mpip install --user openshift` on the node seems enough to enable using k8s task | 23:59 |
| corvus | tristanC: i believe openshift is installed in the zuul-executor image | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!