| *** sanjayu_ has joined #zuul | 00:06 | |
| *** saneax has quit IRC | 00:08 | |
| *** tosky has quit IRC | 00:10 | |
| *** rlandy has quit IRC | 00:18 | |
| *** rfolco has quit IRC | 00:19 | |
| *** rfolco has joined #zuul | 00:19 | |
| *** icey has quit IRC | 00:23 | |
| *** rfolco has quit IRC | 00:25 | |
| *** zenkuro has quit IRC | 00:31 | |
| *** hamalq_ has quit IRC | 02:55 | |
| *** bhavikdbavishi has joined #zuul | 03:14 | |
| *** bhavikdbavishi1 has joined #zuul | 03:17 | |
| *** bhavikdbavishi has quit IRC | 03:19 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:19 | |
| *** bhavikdbavishi has quit IRC | 03:37 | |
| *** bhavikdbavishi has joined #zuul | 03:39 | |
| *** bhavikdbavishi has quit IRC | 03:42 | |
| *** bhavikdbavishi has joined #zuul | 03:43 | |
| *** bhavikdbavishi has quit IRC | 04:00 | |
| *** bhavikdbavishi has joined #zuul | 04:04 | |
| *** bhavikdbavishi has quit IRC | 04:11 | |
| *** bhavikdbavishi has joined #zuul | 04:34 | |
| *** vishalmanchanda has joined #zuul | 05:04 | |
| *** bhavikdbavishi has quit IRC | 05:31 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| vishalmanchanda | hi morning. | 05:41 |
|---|---|---|
| vishalmanchanda | Can I ask for another vote on this patch https://review.opendev.org/c/zuul/zuul-jobs/+/766744 ? | 05:41 |
| vishalmanchanda | It's unblock openstack/horizon gate. | 05:42 |
| *** bhavikdbavishi has joined #zuul | 06:01 | |
| *** jfoufas1 has joined #zuul | 06:05 | |
| *** bhavikdbavishi1 has joined #zuul | 06:19 | |
| *** bhavikdbavishi has quit IRC | 06:20 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 06:20 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Add Firefox browser role for horizon nodejs job https://review.opendev.org/c/zuul/zuul-jobs/+/766744 | 06:24 |
| *** bhavikdbavishi has quit IRC | 06:45 | |
| *** bhavikdbavishi has joined #zuul | 07:09 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/c/zuul/zuul/+/691253 | 07:25 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Allow refreshing changes in canMerge() check https://review.opendev.org/c/zuul/zuul/+/767084 | 07:25 |
| *** bhavikdbavishi1 has joined #zuul | 07:26 | |
| *** bhavikdbavishi has quit IRC | 07:28 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 07:28 | |
| *** icey has joined #zuul | 07:42 | |
| *** bhavikdbavishi has quit IRC | 07:51 | |
| *** jcapitao has joined #zuul | 07:57 | |
| *** bhavikdbavishi has joined #zuul | 08:01 | |
| *** mach1na has joined #zuul | 08:02 | |
| *** bhavikdbavishi has quit IRC | 08:04 | |
| *** bhavikdbavishi has joined #zuul | 08:04 | |
| *** rpittau|afk is now known as rpittau | 08:06 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Allow refreshing changes in canMerge() check https://review.opendev.org/c/zuul/zuul/+/767084 | 08:14 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Check cycle items are mergeable before reporting https://review.opendev.org/c/zuul/zuul/+/743450 | 08:14 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/c/zuul/zuul/+/691253 | 08:14 |
| *** nilsph has joined #zuul | 08:31 | |
| *** tosky has joined #zuul | 08:41 | |
| *** jpena|off is now known as jpena | 08:57 | |
| *** zenkuro has joined #zuul | 09:21 | |
| *** bhavikdbavishi has quit IRC | 09:26 | |
| *** bhavikdbavishi has joined #zuul | 09:27 | |
| *** bhavikdbavishi has quit IRC | 09:37 | |
| *** piotrowskim has joined #zuul | 10:14 | |
| *** vishalmanchanda has quit IRC | 10:22 | |
| *** wuchunyang has joined #zuul | 10:27 | |
| *** rpittau is now known as rpittau|bbl | 10:27 | |
| *** vishalmanchanda has joined #zuul | 10:30 | |
| *** wuchunyang has quit IRC | 10:32 | |
| openstackgerrit | Lida Liu proposed zuul/zuul master: Fix errors regarding parsing timestamps https://review.opendev.org/c/zuul/zuul/+/765990 | 10:34 |
| *** mach1na has quit IRC | 10:41 | |
| *** mach1na has joined #zuul | 11:00 | |
| tobiash | ianw: are there still centos related dib fixes in the pipeline? There are still two failing nodepool functional jobs, e.g. https://zuul.opendev.org/t/zuul/build/5ffce5f02bd74dfe810ad2615640375f | 11:04 |
| *** wuchunyang has joined #zuul | 11:12 | |
| avass | tobiash: I think there was a discussion in the issue a linked, or one of the links in there that explains that gitlab isn't always consistent with ISO format | 11:16 |
| avass | I think Lida happened to bump into that | 11:16 |
| openstackgerrit | Lida Liu proposed zuul/zuul master: Fix errors regarding parsing timestamps https://review.opendev.org/c/zuul/zuul/+/765990 | 11:30 |
| *** jcapitao is now known as jcapitao_lunch | 11:30 | |
| openstackgerrit | Lida Liu proposed zuul/zuul master: Fix errors regarding parsing timestamps https://review.opendev.org/c/zuul/zuul/+/765990 | 11:45 |
| *** rfolco has joined #zuul | 11:46 | |
| zbr|rover | can I add an additional role to existing jobs without being forced to create new jobs? I need to add ensure-yarn for a specific tox-docs job and apparently it does not work via https://review.opendev.org/c/zuul/zuul/+/766460/7/.zuul.yaml | 12:05 |
| zbr|rover | there are lots of cases where user may want to mix-and-match various roles w/o having to create extra jobs. | 12:06 |
| *** ikhan has quit IRC | 12:06 | |
| *** ikhan has joined #zuul | 12:08 | |
| *** ikhan has quit IRC | 12:08 | |
| *** ikhan has joined #zuul | 12:09 | |
| *** mach1na has quit IRC | 12:12 | |
| openstackgerrit | Sorin Sbârnea proposed zuul/project-config master: Add yarn to zuul-tox-docs https://review.opendev.org/c/zuul/project-config/+/767122 | 12:12 |
| *** ikhan has quit IRC | 12:14 | |
| openstackgerrit | Sorin Sbârnea proposed zuul/zuul master: Document tox environments https://review.opendev.org/c/zuul/zuul/+/766460 | 12:15 |
| openstackgerrit | Sorin Sbârnea proposed zuul/zuul master: Document tox environments https://review.opendev.org/c/zuul/zuul/+/766460 | 12:17 |
| zbr|rover | tobiash: any chance to look again at https://review.opendev.org/c/zuul/zuul/+/711002/ - the one on zuul_return overrides on retries. Are new changes needed? | 12:24 |
| *** ikhan has joined #zuul | 12:28 | |
| *** hashar has joined #zuul | 12:34 | |
| *** jpena is now known as jpena|lunch | 12:35 | |
| *** zenkuro has quit IRC | 12:43 | |
| *** zenkuro has joined #zuul | 12:44 | |
| openstackgerrit | Sorin Sbârnea proposed zuul/project-config master: Add yarn to zuul-tox-docs https://review.opendev.org/c/zuul/project-config/+/767122 | 12:51 |
| *** cloudnull has quit IRC | 12:59 | |
| *** mach1na has joined #zuul | 13:00 | |
| *** cloudnull has joined #zuul | 13:00 | |
| *** rlandy has joined #zuul | 13:02 | |
| *** jcapitao_lunch is now known as jcapitao | 13:04 | |
| *** frenzyfriday has joined #zuul | 13:04 | |
| *** rpittau|bbl is now known as rpittau | 13:12 | |
| *** frenzyfriday has quit IRC | 13:20 | |
| tobiash | zbr|rover: I'm now co-author of that so didn't vote, I think I've hopefully addressed all comments | 13:26 |
| zbr|rover | tobiash: i think is ok to vote if you are co-author, otherwise we would have a bunch of projects were it would be impossible to do anything :D | 13:27 |
| zbr|rover | usually I would refrain from voting in full if my contribution was signifiant, but most cases I do only fix minor bits. | 13:28 |
| tobiash | I'd like corvus to re-review that due to the earlier -1 | 13:28 |
| zbr|rover | tobiash: no worries, I was waiting for him to appear for the same reason, he had multiple questions, he should do make final decision on this one. | 13:29 |
| *** jpena|lunch is now known as jpena | 13:36 | |
| *** zenkuro has quit IRC | 13:37 | |
| *** zenkuro has joined #zuul | 13:38 | |
| *** vishalmanchanda has quit IRC | 13:52 | |
| openstackgerrit | Lida Liu proposed zuul/zuul master: Fix errors regarding parsing timestamps https://review.opendev.org/c/zuul/zuul/+/765990 | 14:07 |
| *** smyers has quit IRC | 14:18 | |
| *** smyers has joined #zuul | 14:20 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Don't bail out if change behind cannot be queried https://review.opendev.org/c/zuul/zuul/+/767162 | 14:28 |
| *** nilsph is now known as nils | 14:41 | |
| *** wuchunyang has quit IRC | 14:54 | |
| openstackgerrit | Lida Liu proposed zuul/zuul master: Fix errors regarding parsing timestamps https://review.opendev.org/c/zuul/zuul/+/765990 | 15:29 |
| *** jfoufas1 has quit IRC | 16:21 | |
| *** ikhan has quit IRC | 16:24 | |
| *** mach1na has quit IRC | 16:30 | |
| *** ikhan has joined #zuul | 16:31 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Don't bail out if change behind cannot be queried https://review.opendev.org/c/zuul/zuul/+/767162 | 16:34 |
| *** wuchunyang has joined #zuul | 16:55 | |
| *** wuchunyang has quit IRC | 16:59 | |
| *** piotrowskim has quit IRC | 17:02 | |
| *** hamalq has joined #zuul | 17:25 | |
| *** hamalq_ has joined #zuul | 17:27 | |
| *** hamalq has quit IRC | 17:31 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add allinone deployment and documentation https://review.opendev.org/c/zuul/zuul-operator/+/765630 | 17:33 |
| *** jcapitao has quit IRC | 17:38 | |
| *** rpittau is now known as rpittau|afk | 17:46 | |
| *** jpena is now known as jpena|off | 18:14 | |
| *** wuchunyang has joined #zuul | 18:57 | |
| *** wuchunyang has quit IRC | 19:01 | |
| *** nils has quit IRC | 19:20 | |
| *** hashar is now known as hasharAway | 19:47 | |
| avass | tristanC: the allinone deployment can probably generate sshkeys with a job pod to make it easier | 19:47 |
| tristanC | avass: good point, though it is just a mitigation until the issue is fixed by https://review.opendev.org/c/zuul/zuul/+/764584 | 19:50 |
| avass | oh :) | 19:51 |
| avass | tristanC: you could probably take parts of: https://review.opendev.org/c/zuul/zuul-helm/+/765554 to set up zookeeper as well | 19:52 |
| avass | I converted tools/zk-cert script to an ansible version to easily set up test environments in kubernetes | 19:53 |
| tristanC | avass: once https://github.com/pravega/zookeeper-operator/issues/141 is fixed we can also replace the provided zk resources with it | 19:55 |
| tristanC | so i think there will be 3 toggles, withCertManager, withZookeeperOperator and withDbOperator | 19:57 |
| avass | oh look at that | 19:58 |
| corvus | ++ to those toggles | 20:07 |
| openstackgerrit | Merged zuul/zuul master: Fix override variables in zuul_return https://review.opendev.org/c/zuul/zuul/+/711002 | 20:17 |
| corvus | infra-root: ^ we may want to consider a full zuul restart to pick up that bugfix at some point; however, i don't think it's super urgent since the incidences are generally caused by job retries due to gerrit instability. | 20:22 |
| avass | ooh good timing, I was planning a restart in a couple of minutes | 20:24 |
| fungi | corvus: that needs a full scheduler restart, not just executors? | 20:24 |
| corvus | fungi: yep | 20:25 |
| fungi | noted. i can try to do a full restart later today when things calm down | 20:25 |
| fungi | ahh, right, the structures won't be executor-local because retries don't necessarily end up on the same executor | 20:26 |
| fungi | result goes back to the scheduler and then it schedules a new build | 20:28 |
| corvus | fungi: yeah; changes to model.py in the change are a clue there that it's [probably] global impact | 20:31 |
| fungi | makes sense | 20:33 |
| *** hasharAway has quit IRC | 20:34 | |
| *** goneri has joined #zuul | 21:01 | |
| ianw | yeah i'm around all day to, so can look in on this afternoon | 21:03 |
| fungi | awesome, let's coordinate in that case | 21:05 |
| fungi | (in #opendev obviously) | 21:05 |
| *** rfolco has quit IRC | 21:10 | |
| corvus | ianw, infra-root, zuul-maint: https://gerrit-review.googlesource.com/admin/repos/plugins/zuul-results-summary exists and i added ianw, fungi, mordred, clarkb, fungi to the management group for it https://gerrit-review.googlesource.com/admin/groups/e56b738774680c0b696c20385399cc728291c473,members | 21:11 |
| ianw | corvus: awesome, thanks! i can import what we have | 21:12 |
| corvus | happy for more folks to be added, that was just the folks i could immediately find with existing accounts | 21:12 |
| ianw | i have our job producing screenshot artifacts like https://bf9fadcef442efcdf20f-b95957069fa081f0196bfab2502640d8.ssl.cf2.rackcdn.com/767079/2/check/system-config-run-review/d8ad74f/bridge.openstack.org/screenshots/ | 21:13 |
| ianw | however i need to et a bit better at selenium so it waits long enough for the page to load before taking the screenshot | 21:13 |
| corvus | i wonder if we could do something similar in gerrit's zuul | 21:13 |
| corvus | i expect we'd just run the jar instead of the docker container | 21:14 |
| corvus | but other than that difference, seems like it should be translatable | 21:14 |
| ianw | yes, the hardest part was working out the undocumented (for now, I'll propose patch :) --dev flag to init to setup an admin user | 21:15 |
| ianw | if you init with --dev *and* DEVELOPMENT_BECOME_ANY_ACCOUNT as the auth type, it creates a admin user for you | 21:16 |
| fungi | neat | 21:16 |
| ianw | the upstream container i think either does that somehow, or sets up the initial user in the in-container ldap it is connected to | 21:17 |
| ianw | anyway, easy to inject sample data after that -> https://review.opendev.org/c/opendev/system-config/+/765224/17/playbooks/test-review.yaml | 21:18 |
| ianw | it's easy enough to wait for document.readyState to be 'complete' ... but it seems 'complete' doesn't actually mean it's complete, in true javascript form | 21:20 |
| corvus | ianw: maybe you need to check that document.readyState =============== 'complete' ? :) | 21:27 |
| ianw | it would not surprise me in the slightest if that was valid javascript for the sign-extend-cast-to-float-left-shift-floor-ceiling-instantiate-new-object operator :) | 21:31 |
| corvus | lol | 21:31 |
| mordred | I really want to add a joke but I think ianw already won | 21:45 |
| tristanC | ianw: that's because readyState is a web api document property, framework like polymer or react extends that api and may need more time to settle | 21:52 |
| ianw | tristanC: yeah, it's hard to know what exactly to choose, especially with all this shadow-dom stuff polymer does | 22:07 |
| tristanC | ianw: perhaps you could register an extra polymerReadyState withing https://polymer-library.polymer-project.org/3.0/api/utils/render-status#afterNextRender ? | 22:09 |
| tristanC | or using a setTimeout without a delay should trigger at the next render cycle | 22:10 |
| ianw | actually, a big part of it might be a red-herring and the test actually working, because i think i've written the plugin installation wrong and broken all plugins | 22:14 |
| ianw | https://[2607:ff68:100:54:f816:3eff:fe4c:2752]/c/test-project/+/1 is a sample of the latest layout | 22:20 |
| ianw | the .js file has to be in the plugin directory, not a sub-directory. that's a bit annoying, because it means you bind mount in the file from outside; which means that you can't easily edit it (anything that changes the inode) | 22:22 |
| ianw | corvus: do you have an opinion on mounting the plugin into the container V building it into the container image? | 22:23 |
| corvus | ianw: probably an #opendev q; i'd build it in unless there's a reason not to. good to have complete docker images for easy testing/reuse. | 22:30 |
| fungi | that's the approach we already took with the theme plugin | 22:31 |
| * fungi finds his change for that | 22:31 | |
| ianw | it does mean that updates to the plugin require a restart | 22:31 |
| corvus | -> #opendev | 22:31 |
| *** wuchunyang has joined #zuul | 22:58 | |
| *** wuchunyang has quit IRC | 23:03 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!