| *** saneax_ has joined #zuul | 00:24 | |
| *** saneax_ has quit IRC | 00:43 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: edit-json-file: add role to combine values into a .json https://review.opendev.org/746834 | 00:46 |
|---|---|---|
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: only run docker-setup.yaml when installed https://review.opendev.org/747062 | 00:46 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: Linaro MTU workaround https://review.opendev.org/747063 | 00:46 |
| *** yolanda has quit IRC | 00:55 | |
| *** yolanda has joined #zuul | 01:00 | |
| *** saneax has joined #zuul | 01:12 | |
| *** saneax has quit IRC | 02:01 | |
| *** rlandy|bbl is now known as rlandy | 02:13 | |
| *** saneax has joined #zuul | 02:39 | |
| *** rlandy has quit IRC | 03:07 | |
| *** bhavikdbavishi has joined #zuul | 03:07 | |
| *** bhavikdbavishi has quit IRC | 03:37 | |
| *** bhavikdbavishi has joined #zuul | 03:56 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ara-report: add option for artifact prefix https://review.opendev.org/747100 | 04:11 |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** vishalmanchanda has joined #zuul | 04:33 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ara-report: add option for artifact prefix https://review.opendev.org/747100 | 04:39 |
| *** raukadah is now known as chkumar|rover | 04:43 | |
| *** sgw1 has quit IRC | 04:50 | |
| openstackgerrit | Merged zuul/zuul master: Doc updates for Github interaction https://review.opendev.org/746815 | 05:25 |
| openstackgerrit | Merged zuul/zuul master: Separate slow rpc operations https://review.opendev.org/744679 | 05:36 |
| *** reiterative has quit IRC | 05:43 | |
| *** reiterative has joined #zuul | 05:43 | |
| *** bhavikdbavishi has quit IRC | 05:55 | |
| *** bhavikdbavishi has joined #zuul | 05:55 | |
| *** webknjaz has quit IRC | 05:57 | |
| *** webknjaz has joined #zuul | 06:00 | |
| felixedel | mhu: Yes, this one https://review.opendev.org/#/c/741385/ :) It removes the TableFilters class (to get rid of the inheritance) and defines a FilterToolbar component that is reused in both, the builds and buildsets pages | 06:12 |
| *** y2kenny has quit IRC | 06:18 | |
| *** bhavikdbavishi has quit IRC | 06:30 | |
| *** sanjayu_ has joined #zuul | 06:46 | |
| *** saneax has quit IRC | 06:46 | |
| *** hashar has joined #zuul | 07:04 | |
| *** bhavikdbavishi has joined #zuul | 07:32 | |
| *** maxamillion has quit IRC | 07:40 | |
| *** ericsysmin has quit IRC | 07:41 | |
| *** johnsom has quit IRC | 07:41 | |
| *** kklimonda has quit IRC | 07:41 | |
| *** maxamillion has joined #zuul | 07:42 | |
| *** kklimonda has joined #zuul | 07:44 | |
| *** kklimonda has quit IRC | 07:44 | |
| AJaeger | mhu, do you then want to review https://review.opendev.org/#/c/746423/ , please? clarkb wanted somebody with software factory knowledge to review it | 07:44 |
| *** maxamillion has quit IRC | 07:45 | |
| *** rpittau has quit IRC | 07:47 | |
| *** ericsysmin has joined #zuul | 07:50 | |
| *** kklimonda has joined #zuul | 07:55 | |
| *** jpena|off is now known as jpena | 07:55 | |
| *** maxamillion has joined #zuul | 07:55 | |
| *** rpittau has joined #zuul | 07:56 | |
| *** johnsom has joined #zuul | 07:57 | |
| mhu | AJaeger, done, although I'm no specialist | 08:11 |
| *** tosky has joined #zuul | 08:18 | |
| openstackgerrit | yatin proposed zuul/zuul-jobs master: Ensure git is installed in prepare-workspace-git role https://review.opendev.org/747121 | 08:21 |
| *** bhavikdbavishi has quit IRC | 08:27 | |
| *** vishalmanchanda has quit IRC | 08:30 | |
| *** vishalmanchanda has joined #zuul | 08:30 | |
| *** ericsysmin has quit IRC | 08:30 | |
| *** ericsysmin has joined #zuul | 08:30 | |
| *** rpittau has quit IRC | 08:30 | |
| *** rpittau has joined #zuul | 08:30 | |
| *** johnsom has quit IRC | 08:30 | |
| *** johnsom has joined #zuul | 08:30 | |
| mhu | AJaeger, tobiash can I get the last +2 on https://review.opendev.org/#/c/746680/ ? It removes the useless "trigger" arg for the enqueue REST API | 08:30 |
| tobiash | lgtm | 08:31 |
| *** mgoddard has quit IRC | 08:32 | |
| mhu | felixedel, thanks, I'll have a look and see if I can rebase on your change | 08:32 |
| mhu | tobiash, thanks! | 08:33 |
| *** nils has joined #zuul | 08:37 | |
| tobiash | mhu: I've commented on the web-cli change: https://review.opendev.org/729236 | 08:38 |
| mhu | tobiash, I agree with the project split, should we resurrect the zuul-discuss thread about it? http://lists.zuul-ci.org/pipermail/zuul-discuss/2020-June/001257.html | 08:39 |
| *** mgoddard has joined #zuul | 08:40 | |
| *** bhavikdbavishi has joined #zuul | 08:43 | |
| * tobiash is strugging to let outlook do a correct text format reply on an html formatted mail | 08:45 | |
| tobiash | mhu: revived | 08:49 |
| mhu | \o/ | 08:49 |
| openstackgerrit | Merged zuul/zuul-jobs master: Ensure git is installed in prepare-workspace-git role https://review.opendev.org/747121 | 09:02 |
| *** sanjayu__ has joined #zuul | 09:03 | |
| *** sanjayu_ has quit IRC | 09:06 | |
| *** sanjayu_ has joined #zuul | 09:09 | |
| *** sanjayu__ has quit IRC | 09:12 | |
| tobiash | remote: https://review.opendev.org/747127 Create zuul/zuul-cli | 09:13 |
| tobiash | mhu, corvus: I went ahead and proposed the creation of a new repo for that ^ | 09:14 |
| AJaeger | mhu: thanks | 09:15 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Rework quick-start and prepare for other tutorials https://review.opendev.org/732066 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate your first patch" https://review.opendev.org/732067 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs" https://review.opendev.org/732068 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline" https://review.opendev.org/732069 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets" https://review.opendev.org/732070 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies" https://review.opendev.org/732071 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Rename quick-start to zuul-tutorial-quick-start https://review.opendev.org/737656 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] TEST run zuul tutorials to test stream+callback (+ zuul-jobs change) https://review.opendev.org/735477 | 09:24 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] Test: run multiple tutorials ('job dependencies' 2 times) https://review.opendev.org/741558 | 09:24 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality https://review.opendev.org/709735 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Separate connection registries in tests https://review.opendev.org/712958 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler https://review.opendev.org/717269 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests https://review.opendev.org/721254 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Driver event ingestion https://review.opendev.org/717299 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper https://review.opendev.org/716221 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper https://review.opendev.org/716875 | 09:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 09:33 |
| 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 | 09:33 |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once https://review.opendev.org/740328 | 09:53 |
| *** bhavikdbavishi has quit IRC | 10:11 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve promote endpoint testing https://review.opendev.org/728489 | 10:13 |
| mhu | felixedel, regarding line 66 of https://review.opendev.org/#/c/741385/4/web/src/pages/Buildsets.jsx is there some doc I could check? I'd like to add checkbox filtering for boolean attributes (example: whether a build was held or not) | 10:16 |
| *** bhavikdbavishi has joined #zuul | 10:28 | |
| openstackgerrit | Merged zuul/zuul master: REST API: remove deprecated trigger arg in enqueue endpoint https://review.opendev.org/746680 | 10:32 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Annotate getBranch with event id https://review.opendev.org/747142 | 11:02 |
| *** gtema has joined #zuul | 11:12 | |
| *** bhavikdbavishi1 has joined #zuul | 11:13 | |
| *** bhavikdbavishi has quit IRC | 11:14 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 11:14 | |
| 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 | 11:14 |
| *** bhavikdbavishi has quit IRC | 11:28 | |
| *** bhavikdbavishi has joined #zuul | 11:29 | |
| *** bhavikdbavishi has quit IRC | 11:34 | |
| *** jpena is now known as jpena|lunch | 11:35 | |
| *** rfolco has joined #zuul | 11:36 | |
| *** rlandy has joined #zuul | 11:51 | |
| *** gtema has quit IRC | 11:54 | |
| *** bhavikdbavishi has joined #zuul | 11:55 | |
| *** bhavikdbavishi has quit IRC | 12:11 | |
| *** bhavikdbavishi has joined #zuul | 12:14 | |
| felixedel | In line 135 of https://review.opendev.org/#/c/741385/4/web/src/containers/FilterToolbar.jsx I differentiate between the different types of filter (so far I have only "search"). But we could add another "checkbox" type which renders a checkbox. For the checkbox itself you might check how it is done here https://www.patternfly.org/v4/documentation/react/demos/filtertabledemo (Is that only a single state we have here or | 12:15 |
| felixedel | could there be more later on? In case it's only one, a dropdown wouldn't make much sense IMOH). In that case you have to look at the checkbox documentation https://www.patternfly.org/v4/documentation/react/components/checkbox and see how this can be incorporated into the toolbar | 12:15 |
| felixedel | mhu ^ | 12:15 |
| felixedel | Another questions would be if we put this state also in the dropdown or if we just render the checkbox next to the dropdown/input combination. I'm not sure how "prominent" this filter should be | 12:16 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: UI: Highlight pipeline names on status page https://review.opendev.org/747152 | 12:27 |
| *** jpena|lunch is now known as jpena | 12:42 | |
| *** hashar has quit IRC | 12:50 | |
| mhu | felixedel, thanks I'll have a look. when I added the checkboxes, I made it independent from the filter box, so that's a possibility too | 12:54 |
| *** Goneri has joined #zuul | 13:13 | |
| *** bhavikdbavishi has quit IRC | 13:20 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Prevent Git GC issue between merger and executor https://review.opendev.org/747160 | 13:22 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Prevent Git GC issue between merger and executor https://review.opendev.org/747160 | 13:24 |
| *** hashar has joined #zuul | 13:35 | |
| openstackgerrit | Benjamin Schanzel proposed zuul/zuul master: GitHub Reporter: Fix User Email in Merge Commit Message https://review.opendev.org/741157 | 13:49 |
| *** Shrews has left #zuul | 13:58 | |
| openstackgerrit | Radosław Piliszek proposed zuul/zuul-jobs master: Fix git install on Debian distro family https://review.opendev.org/747170 | 14:10 |
| *** chkumar|rover is now known as raukadah | 14:33 | |
| *** arxcruz is now known as arxcruz|ruck | 14:38 | |
| *** sgw1 has joined #zuul | 15:21 | |
| *** tosky_ has joined #zuul | 15:35 | |
| *** tosky has quit IRC | 15:36 | |
| *** tosky_ is now known as tosky | 15:37 | |
| *** rlandy is now known as rlandy|afk | 15:59 | |
| *** jpena is now known as jpena|off | 16:05 | |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Revert "Ensure git is installed in prepare-workspace-git role" https://review.opendev.org/747238 | 16:08 |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Revert "Ensure git is installed in prepare-workspace-git role" https://review.opendev.org/747238 | 16:13 |
| *** nils has quit IRC | 17:15 | |
| *** sgw1 has quit IRC | 17:26 | |
| *** bhavikdbavishi has joined #zuul | 17:28 | |
| *** sgw has joined #zuul | 17:30 | |
| *** rlandy|afk is now known as rlandy | 17:37 | |
| *** hashar has quit IRC | 17:41 | |
| *** sgw1 has joined #zuul | 17:42 | |
| zbr | corvus: do we still need to wait for someone else on https://review.opendev.org/#/c/739482/ ? | 18:11 |
| *** Goneri has quit IRC | 18:21 | |
| *** hashar has joined #zuul | 18:30 | |
| *** Goneri has joined #zuul | 18:52 | |
| *** bhavikdbavishi has quit IRC | 18:55 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Load diskimage configs before building them https://review.opendev.org/747277 | 18:57 |
| clarkb | fungi: corvus ^ I think that may make nodepool builders more responsive in this pause after deleting all images case | 18:58 |
| *** dustinc has joined #zuul | 19:16 | |
| *** nils has joined #zuul | 19:24 | |
| *** hashar has quit IRC | 19:52 | |
| *** yoctozepto2 has joined #zuul | 20:06 | |
| *** yoctozepto has quit IRC | 20:07 | |
| *** yoctozepto2 is now known as yoctozepto | 20:07 | |
| *** zenkuro has joined #zuul | 20:24 | |
| corvus | clarkb: i left some thoughts on that | 20:24 |
| zenkuro | corvus: hi, regarding issue with ara-report, after turning back 'upstream' playbooks software factory failed with <the role 'merge-output-to-logs' was not found> | 20:26 |
| corvus | zbr: you said "fixed in followup" on that; what's the followup? | 20:26 |
| corvus | zenkuro: that sounds like a SF problem i don't understand; https://zuul-ci.org/docs/zuul-jobs/log-roles.html#role-merge-output-to-logs exists in zuul-jobs | 20:27 |
| zenkuro | corvus: in general sf goes with pre-defined jobs(including ara). Also there is(in sf) an option to use most recent playbooks from opendev.org. | 20:31 |
| zenkuro | corvus: in my case this option was enabled. Because of it I faces isses with Error 30(read-only file system) on ara related job. This problem been solved by disabling upstream playbooks in sf. | 20:32 |
| corvus | zenkuro: it sounds like we'll need someone who understands the sf jobs to look into this. | 20:33 |
| zenkuro | why Im posting all this stuff here? -> Just in case other person with software factory face this issues and start dig it to zuul. | 20:33 |
| zenkuro | corvus: yes, I will try to contact sf team and tell this story to them. Hopefully in the next release they will fix it. | 20:34 |
| zenkuro | hm... I do not know if there is a search over zuul irc logs... hm... | 20:35 |
| corvus | zenkuro: they are archived here http://eavesdrop.openstack.org/irclogs/%23zuul/ | 20:36 |
| zenkuro | corvus: cool so it is not in vain >_< | 20:36 |
| *** sshnaidm is now known as sshnaidm|afk | 20:47 | |
| clarkb | corvus: ya I think I prefer checking if the ocnfig has updated to handle the builds taking longer than an rebuild interval (that seems very possible depending on the setup, iirc bmw has some very large builds) | 20:48 |
| clarkb | I'll work on a new ps | 20:48 |
| corvus | kk | 20:49 |
| *** vishalmanchanda has quit IRC | 20:52 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Load diskimage configs before building them https://review.opendev.org/747277 | 21:13 |
| clarkb | corvus: something like that maybe? | 21:13 |
| corvus | clarkb: yeah, though i was thinking if *anything* is different in the config, short circuit | 21:26 |
| corvus | clarkb: (since maybe we want other things to update faster too) | 21:26 |
| clarkb | ah something like if nodepool_config.loadConfig() != self._config: return? | 21:28 |
| clarkb | thats actually simpler | 21:28 |
| corvus | ++ | 21:29 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Load diskimage configs before building them https://review.opendev.org/747277 | 21:33 |
| clarkb | __eq__ is implemented on the config type too so I think that will just work | 21:33 |
| corvus | yeah i think that was the hope :) | 21:33 |
| *** Goneri has quit IRC | 21:35 | |
| *** tosky has quit IRC | 22:07 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Bump dib requirement to 3.2.1 https://review.opendev.org/747303 | 22:10 |
| ianw | clarkb/fungi: ^ | 22:10 |
| *** Goneri has joined #zuul | 22:25 | |
| *** Goneri has quit IRC | 22:32 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add image-pause CLI command https://review.opendev.org/747306 | 22:37 |
| corvus | clarkb, fungi, ianw: ^ that should improve the pause workflow | 22:37 |
| clarkb | looking | 22:37 |
| fungi | corvus: neat! i was wondering if that might not be a thing we could add | 22:38 |
| fungi | the other related thing i wonder was if there was a safe way to abort diskimage-builder then i wonder if dib-image-delete could be made to also kill images in building state | 22:39 |
| corvus | we'd need another thread to watch for that | 22:40 |
| fungi | ahh, good point, this is all sync | 22:41 |
| ianw | fungi: aborting dib is usually "safe". unless it isn't. which is sometimes, and it leaves behind mounts. all instances of that i would consider bugs, but they're often difficult to make sure things unwind properly depending on what's happening | 22:41 |
| fungi | so really what we want is for a building image to transition to deleting if its been paused, i guess, which may already be the case once clarkb's fix goes in | 22:42 |
| corvus | i might have shown my age in my review of 747303 | 22:42 |
| fungi | ianw: yeah, i did test one by killing the diskimage-create parent process and the builder cleaned up as expected | 22:43 |
| fungi | corvus: now i'm going to have the theme song for that stuck in my head all evening | 22:44 |
| ianw | fungi: this will *usually* be the case. i think there's some corner cases, particularly in the minimal builds where they do mounts | 22:45 |
| clarkb | corvus: on the pausing cli command change does zk client.create() succeed if the node already exists? | 22:45 |
| corvus | fungi: yeah, i mean, they programmed us pretty thoroughly didn't they? :) | 22:45 |
| clarkb | corvus: wondering if we need to handle the case of pausing an already puased image differently | 22:45 |
| corvus | clarkb: oh lemme test | 22:45 |
| fungi | extended disco edit! https://www.youtube.com/watch?v=-AuLmwoH-1Y | 22:45 |
| clarkb | corvus: the change looks good other than that | 22:45 |
| corvus | clarkb: yeah, it's a prob will fix | 22:46 |
| ianw | corvus/fungi: i guess we didn't get that down under. we had https://en.wikipedia.org/wiki/The_Curiosity_Show | 22:46 |
| fungi | though i expect we can all enjoy this marvellous spoof of those horrible 80s educational program(me)s: https://en.wikipedia.org/wiki/Look_Around_You | 22:48 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add image-pause CLI command https://review.opendev.org/747306 | 22:48 |
| corvus | ianw: wikipedia says 321c was based on that. til :) | 22:48 |
| ianw | haha the circle is closed | 22:49 |
| *** zenkuro has quit IRC | 23:07 | |
| fungi | aprpos of nothing in particular, my six-year-old nephew has just declared he's named his new betta fish "zuul the gatekeeper" | 23:13 |
| fungi | i guess now i have to mail him some project logo stickers | 23:16 |
| openstackgerrit | Merged zuul/nodepool master: Bump dib requirement to 3.2.1 https://review.opendev.org/747303 | 23:20 |
| corvus | that's a good name | 23:20 |
| fungi | he's been watching ghostbusters a lot lately, according to my brother | 23:21 |
| *** dcastellani has quit IRC | 23:54 | |
| *** webknjaz has quit IRC | 23:55 | |
| *** iamweswilson has quit IRC | 23:56 | |
| *** aprice has quit IRC | 23:56 | |
| *** webknjaz has joined #zuul | 23:57 | |
| *** PrinzElvis has quit IRC | 23:57 | |
| *** iamweswilson has joined #zuul | 23:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!