| *** cdearborn has quit IRC | 00:01 | |
| *** jamesmcarthur has joined #zuul | 00:27 | |
| *** jamesmcarthur has quit IRC | 00:36 | |
| *** jamesmcarthur has joined #zuul | 00:58 | |
| *** swest has quit IRC | 01:16 | |
| *** jamesmcarthur has quit IRC | 01:26 | |
| *** jamesmcarthur has joined #zuul | 01:26 | |
| *** swest has joined #zuul | 01:32 | |
| *** jamesmcarthur has quit IRC | 01:32 | |
| *** jamesmcarthur has joined #zuul | 02:04 | |
| *** jamesmcarthur_ has joined #zuul | 02:07 | |
| *** jamesmcarthur has quit IRC | 02:11 | |
| *** jamesmcarthur_ has quit IRC | 02:13 | |
| *** jamesmcarthur has joined #zuul | 02:41 | |
| *** jamesmcarthur has quit IRC | 02:43 | |
| *** jamesmcarthur has joined #zuul | 02:44 | |
| *** jamesmcarthur_ has joined #zuul | 03:02 | |
| *** jamesmcarthur has quit IRC | 03:05 | |
| openstackgerrit | Merged zuul/nodepool master: config_validator: refactor the schema to a static method https://review.opendev.org/718582 | 03:15 |
|---|---|---|
| *** threestrands has joined #zuul | 03:29 | |
| *** zxiiro has joined #zuul | 03:56 | |
| *** jamesmcarthur has joined #zuul | 04:06 | |
| *** jamesmcarthur_ has quit IRC | 04:09 | |
| *** bhavikdbavishi has joined #zuul | 04:21 | |
| *** jamesmcarthur has quit IRC | 04:22 | |
| *** jamesmcarthur has joined #zuul | 04:23 | |
| *** bhavikdbavishi1 has joined #zuul | 04:26 | |
| *** bhavikdbavishi has quit IRC | 04:26 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 04:26 | |
| *** jamesmcarthur has quit IRC | 04:28 | |
| *** jamesmcarthur has joined #zuul | 04:30 | |
| *** jamesmcarthur has quit IRC | 04:30 | |
| *** jamesmcarthur has joined #zuul | 04:36 | |
| *** evrardjp has quit IRC | 04:37 | |
| *** evrardjp has joined #zuul | 04:37 | |
| *** jamesmcarthur has quit IRC | 04:38 | |
| *** jamesmcarthur has joined #zuul | 04:39 | |
| *** jamesmcarthur has quit IRC | 04:47 | |
| *** jamesmcarthur has joined #zuul | 05:14 | |
| *** sgw has quit IRC | 05:34 | |
| *** ysandeep is now known as ysandeep|brb | 05:38 | |
| *** bhavikdbavishi has quit IRC | 05:38 | |
| *** reiterative has quit IRC | 05:43 | |
| *** reiterative has joined #zuul | 05:43 | |
| *** bhavikdbavishi has joined #zuul | 05:49 | |
| *** saneax has joined #zuul | 06:01 | |
| *** jamesmcarthur_ has joined #zuul | 06:16 | |
| *** jamesmcarthur has quit IRC | 06:20 | |
| *** jamesmcarthur_ has quit IRC | 06:21 | |
| *** ysandeep|brb is now known as ysandeep | 06:34 | |
| *** gtema has joined #zuul | 06:45 | |
| *** dpawlik has joined #zuul | 06:49 | |
| *** jamesmcarthur has joined #zuul | 06:54 | |
| *** yolanda has quit IRC | 07:08 | |
| *** yolanda has joined #zuul | 07:09 | |
| *** jcapitao has joined #zuul | 07:14 | |
| *** bhavikdbavishi has quit IRC | 07:14 | |
| *** tflink has quit IRC | 07:29 | |
| *** tflink_ has joined #zuul | 07:29 | |
| *** rpittau|afk is now known as rpittau | 07:34 | |
| *** tosky has joined #zuul | 07:38 | |
| *** jpena|off is now known as jpena | 07:53 | |
| *** zxiiro has quit IRC | 07:55 | |
| *** bhavikdbavishi has joined #zuul | 08:08 | |
| *** bhavikdbavishi has quit IRC | 08:18 | |
| *** bhavikdbavishi has joined #zuul | 08:23 | |
| *** threestrands has quit IRC | 08:45 | |
| *** harrymichal has joined #zuul | 08:49 | |
| *** ysandeep is now known as ysandeep|lunch | 08:53 | |
| *** jamesmcarthur has quit IRC | 08:55 | |
| *** newbie999 has joined #zuul | 09:11 | |
| newbie999 | Hi guys! | 09:11 |
| newbie999 | Does the Zuul timer driver use UTC time? | 09:12 |
| newbie999 | https://zuul-ci.org/docs/zuul/reference/drivers/timer.html | 09:12 |
| *** gtema has left #zuul | 09:12 | |
| newbie999 | does not say, but it seems it uses whatever local time has been configured on the scheduler/executor | 09:12 |
| tobiash | newbie999: zuul uses apscheduler in its default settings: https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/timer/__init__.py#L35 | 09:34 |
| tobiash | which probably defaults to the timezone defined by the locale | 09:34 |
| newbie999 | thanks! That is a good starting point to investigate it | 09:36 |
| newbie999 | https://apscheduler.readthedocs.io/en/stable/userguide.html | 09:37 |
| newbie999 | it seems you could tell apscheduler to use UTC, but that is not the default | 09:37 |
| *** bhavikdbavishi has quit IRC | 09:39 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Separate connection registries in tests https://review.opendev.org/712958 | 10:03 |
| *** bhavikdbavishi has joined #zuul | 10:09 | |
| *** harrymichal has quit IRC | 10:19 | |
| *** harrymichal has joined #zuul | 10:19 | |
| *** rpittau is now known as rpittau|bbl | 10:19 | |
| *** ysandeep|lunch is now known as ysandeep | 10:22 | |
| *** harrymichal has quit IRC | 10:24 | |
| *** harrymichal has joined #zuul | 10:24 | |
| *** yolanda has quit IRC | 10:26 | |
| msuszko | Hi! I'm looking for a clue: I'd like to prepare check and gate project jobs do an app in docker. Dockerfile has build, and release stages. Release stage would be uploaded published - obviously, but i'd like to run tests in build stage. Maybe add test stage with extra tools and run tests there. | 10:34 |
| msuszko | Should I use buildset registry for this? | 10:35 |
| *** yolanda has joined #zuul | 10:38 | |
| msuszko | I just got it working, but run-buildset-registry and use-buildset-registry alone take over 4 minutes with tiny image. | 10:38 |
| *** rfolco has joined #zuul | 10:50 | |
| *** jamesmcarthur has joined #zuul | 10:53 | |
| *** jcapitao is now known as jcapitao_lunch | 10:53 | |
| *** newbie999 has quit IRC | 10:56 | |
| *** avass has quit IRC | 11:06 | |
| *** ysandeep is now known as ysandeep|afk | 11:15 | |
| *** jpena is now known as jpena|lunch | 11:30 | |
| *** cdearborn has joined #zuul | 11:37 | |
| *** ysandeep|afk is now known as ysandeep | 11:47 | |
| *** dpawlik has quit IRC | 11:50 | |
| *** dpawlik has joined #zuul | 11:50 | |
| *** Goneri has joined #zuul | 11:59 | |
| *** bhavikdbavishi has quit IRC | 12:04 | |
| *** jcapitao_lunch is now known as jcapitao | 12:07 | |
| *** rpittau|bbl is now known as rpittau | 12:07 | |
| *** harrymichal has quit IRC | 12:07 | |
| *** hashar has joined #zuul | 12:09 | |
| *** dpawlik has quit IRC | 12:09 | |
| *** dpawlik has joined #zuul | 12:10 | |
| *** harrymichal has joined #zuul | 12:17 | |
| zbr | corvus: clarkb: are you pleased with new <pre> output on https://review.opendev.org/#/c/681532/ ? | 12:19 |
| zbr | i was finally able to get it to wrap correctly | 12:20 |
| *** bhavikdbavishi has joined #zuul | 12:24 | |
| *** zxiiro has joined #zuul | 12:27 | |
| *** jpena|lunch is now known as jpena | 12:31 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler https://review.opendev.org/717269 | 12:36 |
| *** harrymichal has quit IRC | 12:39 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: dhall-diff: add new job https://review.opendev.org/718694 | 12:44 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Detach head before reset https://review.opendev.org/720458 | 12:51 |
| *** yolanda has quit IRC | 12:55 | |
| *** yolanda has joined #zuul | 12:55 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Detach head before reset https://review.opendev.org/720458 | 12:56 |
| *** ysandeep is now known as ysandeep|mtg | 13:14 | |
| *** gtema has joined #zuul | 13:21 | |
| *** gtema_ has joined #zuul | 13:31 | |
| *** gtema has quit IRC | 13:34 | |
| *** sgw has joined #zuul | 13:40 | |
| *** bhavikdbavishi has quit IRC | 13:48 | |
| mordred | corvus: idea - add a config setting to zuul to communicate what the log retention period is - so that on old builds in the dashboard instead of throwing a "can't load file, maybe you have an ad blocker" error, we can throw a "can't load logs, the log data is beyond the retention period" | 14:23 |
| mordred | corvus: since the build record has a date associated with it - so we should be able to know "yup, this build is too old to have logs" | 14:24 |
| corvus | mordred: yeah, that's something we could return via zuul_return and stick in mysql | 14:25 |
| mordred | corvus: oh, that's even better than a config setting :) | 14:25 |
| corvus | (we could actually cheat, and do that as metadata on the manifest today with no schema change, but this seems like something to make a bit more fundamental) | 14:25 |
| mordred | yeah - it seems like a piece of data an admin would want to communicate | 14:27 |
| mordred | msuszko: you don't need the buildset registry for a self-contained docker build like that. where it's more useful is if you have subsequent jobs in the buildset that want to use the image you built | 14:29 |
| mordred | msuszko: with the buildset registry instead of making a test stage in the dockerfile you could instead make a second test job that runs the release image and verifies that it's working like it would in production - but it's possible that's overkill for your use case | 14:30 |
| *** harrymichal has joined #zuul | 14:49 | |
| *** panda is now known as panda|ruck | 14:49 | |
| *** harrymichal has quit IRC | 14:57 | |
| *** ysandeep|mtg is now known as ysandeep | 14:59 | |
| msuszko | mordred: It seems I have two cases: 1) one job with playbook running roles build-docker-image and test-in-docker, 2) build-docker-image + run-buildset-registry and other jobs test-in-docker + use-buildset-registry with dependencies set to first job | 14:59 |
| msuszko | the latter one is required to run tests in parallel | 15:00 |
| openstackgerrit | Monty Taylor proposed zuul/nodepool master: Add yappi and objgraph to an extra https://review.opendev.org/720499 | 15:02 |
| msuszko | BTW: is anyone using S3 for log storage? | 15:02 |
| AJaeger | corvus: want to review https://review.opendev.org/#/c/717663/ (the pip ensure zuul-jobs changes) again, please? With clarkb's change the 3rd party CI is passing now. | 15:02 |
| *** gtema_ has left #zuul | 15:02 | |
| corvus | msuszko: i think some folks are; there's a wip change in zuul-jobs to support it, though i think it could use an update to be more like the gcp log storage role | 15:08 |
| *** ysandeep is now known as ysandeep|away | 15:11 | |
| *** bhavikdbavishi has joined #zuul | 15:25 | |
| *** dpawlik has quit IRC | 15:26 | |
| *** dpawlik has joined #zuul | 15:26 | |
| msuszko | upload-logs-gcs is a copy of older upload-logs-swift with few changes | 15:34 |
| fungi | and upload-logs-swift is the one which gets the most testing and is most widely-deployed, so i expect consistency with it is preferable | 15:35 |
| *** ttx has quit IRC | 15:38 | |
| *** ttx has joined #zuul | 15:38 | |
| zbr | fungi: can you do a quick look at https://review.opendev.org/#/c/681532/ ? (ignore WIP prefix, is ready for review afaik). -- the <pre> change. | 15:46 |
| zbr | https://sbarnea.com/ss/Screen-Shot-2020-04-16-16-46-33.42.png | 15:46 |
| zbr | no more vertical scrolling | 15:46 |
| fungi | i assume you mean no more horizontal scrolling, but looks good! https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_633/681532/10/check/zuul-build-dashboard-multi-tenant/63376a0/npm/html/t/local/build/ad1ef4edadca43d38b90cd784f121c1d | 16:01 |
| fungi | zbr: are you interested in similar work on the console view, or is that less of a concern? | 16:02 |
| fungi | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_633/681532/10/check/zuul-build-dashboard-multi-tenant/63376a0/npm/html/t/local/build/ad1ef4edadca43d38b90cd784f121c1d/console | 16:02 |
| zbr | fungi: yes, I am (fyi, these links are not shareable). | 16:03 |
| zbr | i happen to care a lot about UX of the console output, so I will be glad to make changes to improve it. | 16:04 |
| zbr | my plan is to also enable ANSI support, but i need to wait for the react app version upgrade to happen, apparently there is something wrong with it. | 16:07 |
| fungi | zbr: d'oh, yep i didn't notice those links don't work unless you browse to them starting from https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_633/681532/10/check/zuul-build-dashboard-multi-tenant/63376a0/npm/html/ | 16:07 |
| corvus | we could probably use zuul-preview to make those work | 16:07 |
| openstackgerrit | Sorin Sbarnea proposed zuul/zuul master: Improve display of stdout/stderr task errors https://review.opendev.org/681532 | 16:09 |
| *** jcapitao has quit IRC | 16:10 | |
| *** sshnaidm has joined #zuul | 16:17 | |
| *** rpittau is now known as rpittau|afk | 16:18 | |
| *** evrardjp has quit IRC | 16:37 | |
| *** evrardjp has joined #zuul | 16:37 | |
| corvus | i plan to focus on upgrading the opendev installation and using the new TLS feature there for the next few days | 16:42 |
| corvus | i'd like to make sure that gets some real-world usage before we release it | 16:43 |
| corvus | i apologize if that delays reviews for a bit | 16:43 |
| tobiash | k, sounds good | 17:18 |
| *** jpena is now known as jpena|off | 17:18 | |
| *** sshnaidm has quit IRC | 17:37 | |
| *** avass has joined #zuul | 17:57 | |
| *** hashar is now known as hasharAway | 18:03 | |
| mordred | corvus: https://review.opendev.org/#/c/720499 if you get a sec | 18:07 |
| mordred | corvus: we should land that before the nodepol launcher change in opendev | 18:07 |
| *** yolanda has quit IRC | 18:14 | |
| corvus | done | 18:15 |
| *** yolanda has joined #zuul | 18:28 | |
| *** bhavikdbavishi has quit IRC | 18:33 | |
| *** jamesmcarthur has quit IRC | 18:44 | |
| *** jamesmcarthur has joined #zuul | 18:44 | |
| *** tflink_ is now known as tflink | 18:54 | |
| openstackgerrit | Merged zuul/nodepool master: Add yappi and objgraph to an extra https://review.opendev.org/720499 | 18:54 |
| *** rfolco is now known as rfolco|bbl | 19:01 | |
| *** harrymichal has joined #zuul | 19:04 | |
| *** hasharAway is now known as hashar | 19:05 | |
| *** saneax has quit IRC | 19:05 | |
| *** jamesmcarthur has quit IRC | 19:29 | |
| *** jamesmcarthur has joined #zuul | 19:33 | |
| *** jamesmcarthur has quit IRC | 19:37 | |
| *** jamesmcarthur has joined #zuul | 19:38 | |
| *** harrymichal has quit IRC | 19:45 | |
| *** rfolco|bbl is now known as rfolco | 19:46 | |
| *** jamesmcarthur has quit IRC | 20:14 | |
| *** jamesmcarthur has joined #zuul | 20:20 | |
| *** jamesmcarthur has quit IRC | 20:20 | |
| *** rfolco has quit IRC | 20:36 | |
| *** saneax has joined #zuul | 20:42 | |
| *** sassyn has quit IRC | 20:57 | |
| *** rfolco has joined #zuul | 21:10 | |
| *** hashar has quit IRC | 21:17 | |
| jkt | I switched my Zuul 3.18.0 to use HTTP reporting (I wanted to get access to file comments), but now I'm getting AttributeError: 'GerritChange' object has no attribute 'commit' in gerritconnection.py", line 1108, in review_http | 21:48 |
| jkt | Gerrit 3.1.4 | 21:49 |
| jkt | I see that this is probably introduced in 947b7b1dcb6699ac3e5cfe2c046d18599d0ba8fe with introduction of GerritChangeData, because before that the change.commit was taken from ps['revision'] | 21:50 |
| clarkb | jkt: I think its still doing ps['revision'] in gerritmodel.py | 21:57 |
| clarkb | jkt: https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/gerrit/gerritmodel.py#L114 there | 21:59 |
| clarkb | jkt: looking at that code, is it possible ti was trying to comment on an old patchset? | 21:59 |
| clarkb | we seem to not set the commit revision if the patchset isn't current there | 21:59 |
| openstackgerrit | Merged zuul/zuul master: Filter secret ZUUL_ env variables from ansible env https://review.opendev.org/720238 | 22:03 |
| jkt | clarkb: digging into this (with printfs as usual), but now I'm getting a rejectionf when I try to re-run via zuul enqueue, and that exception eating code in rpclistener.py is less than helpful | 22:09 |
| jkt | clarkb: ah!, so that error was caused by enqueueuing a wrong version (there was no PS2, just PS1, and I manually injected PS2) | 22:27 |
| jkt | clarkb: looks like something which should not be allowed, though | 22:27 |
| fungi | sounds like it wasn't allowed, it raised an exception? | 22:30 |
| fungi | or are you suggesting the rpc interface should do some sort of lookahead to find out if stuff is in gerrit? | 22:31 |
| fungi | and bubble that back up to the rpc request interface? | 22:31 |
| jkt | fungi: the problem is that I can enqueue a non-existing PS, and when this non-existing PS gets "reported" to gerrit, it results in an exception | 22:33 |
| fungi | got it | 22:33 |
| jkt | fungi: as I was debugging this, I added extra print()s into the code and that resulted in the RPC layer showing me something which confused me | 22:33 |
| fungi | did it report an exception into gerrit though? | 22:33 |
| fungi | or just short-circuit to the reporter since there was no valid item to enqueue? | 22:34 |
| jkt | nope, it resulted in a build and everything, but nothing hit Gerrit | 22:34 |
| jkt | it started (and finished) building stuff | 22:34 |
| fungi | neat, i wonder what it built | 22:34 |
| fungi | (what ref, i mean) | 22:34 |
| jkt | https://gerrit.cesnet.cz/c/CzechLight/netconf-cli/+/2452 is that change, and https://zuul.gerrit.cesnet.cz/t/public/buildset/0f162a8a7cac427b860dd5e17f32d2d0 is one of these build sets | 22:34 |
| jkt | let me dig through the logs | 22:35 |
| jkt | fungi: my build log says it built PS1 of that change | 22:35 |
| jkt | I wonder what would have happened if someone uploaded PS2 while this PS1 treated as-if-it-was-PS2 was getting built, prior to Zuul reporting result of actual PS1 | 22:36 |
| fungi | an interesting race condition to test, i suppose | 22:37 |
| jkt | zuul.GerritConnection: Updating <Change 0x7feafc466748 CzechLight/netconf-cli 2452,2>: Getting git-dependent change 2451,2 | 22:39 |
| jkt | zuul.GerritConnection: Updating <Change 0x7feafc466748 CzechLight/netconf-cli 2452,2>: Getting git-needed change 2452,1 | 22:39 |
| jkt | zuul.GerritConnection: Updating <Change 0x7feafc466748 CzechLight/netconf-cli 2452,2>: Running query message:{I9c1f039e7e054f84559a3d57812d0254db183a99} to find changes needed-by | 22:39 |
| jkt | here's how it ended up there I think | 22:40 |
| *** jamesmcarthur has joined #zuul | 22:48 | |
| openstackgerrit | Merged zuul/zuul-jobs master: ensure-pip: export ensure_pip_virtualenv_command https://review.opendev.org/718224 | 23:01 |
| *** tosky has quit IRC | 23:09 | |
| *** threestrands has joined #zuul | 23:14 | |
| *** threestrands has quit IRC | 23:15 | |
| *** threestrands has joined #zuul | 23:15 | |
| *** jamesmcarthur has quit IRC | 23:19 | |
| *** rfolco has quit IRC | 23:21 | |
| openstackgerrit | Merged zuul/zuul-jobs master: fetch-zuul-cloner: use ensure-pip https://review.opendev.org/717882 | 23:25 |
| *** armstrongs has joined #zuul | 23:32 | |
| *** armstrongs has quit IRC | 23:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
