| tristanC | there seems to be a bug in the jobs page when there is no base job | 00:04 |
|---|---|---|
| tristanC | zuul-maint: what is the next step for https://review.openstack.org/632620 (build-sshkey improvement for static node)? | 00:09 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: upgrade react and react-scripts to ^2.0.0 https://review.openstack.org/631902 | 00:41 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add ErrorBoundary https://review.openstack.org/647257 | 00:53 |
| *** dmsimard has joined #zuul | 01:13 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add ErrorBoundary https://review.openstack.org/647257 | 01:17 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: check if project has variant https://review.openstack.org/647260 | 01:26 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: check if job has parent https://review.openstack.org/647263 | 01:32 |
| SpamapS | pabelanger: shouldn't have a builds tab without a db | 01:50 |
| SpamapS | It was quite nice for me to experiment without having to set up mysql.. minor point, but score one for "keep it optional" | 01:51 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: do not display Build/Buildset menu when job_history is missing https://review.openstack.org/647264 | 02:08 |
| *** dmsimard has quit IRC | 02:10 | |
| *** swest has quit IRC | 02:17 | |
| *** swest has joined #zuul | 02:32 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: tests: be more gentle in the iterate timeout loop https://review.openstack.org/647267 | 03:02 |
| *** dmsimard has joined #zuul | 03:54 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: tests: be more gentle in the iterate timeout loop https://review.openstack.org/647267 | 04:12 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: tests: use a single build reference in stream test https://review.openstack.org/647281 | 04:12 |
| *** saneax has joined #zuul | 04:18 | |
| *** sshnaidm|rover has joined #zuul | 04:38 | |
| *** sshnaidm|rover has quit IRC | 04:49 | |
| *** swest has quit IRC | 05:34 | |
| *** swest has joined #zuul | 06:19 | |
| *** swest has quit IRC | 06:23 | |
| *** lennyb has quit IRC | 06:24 | |
| *** swest has joined #zuul | 06:37 | |
| *** quiquell has joined #zuul | 07:06 | |
| *** pcaruana has joined #zuul | 07:07 | |
| *** quiquell has quit IRC | 07:20 | |
| *** quiquell has joined #zuul | 07:20 | |
| openstackgerrit | William Huang proposed openstack-infra/zuul-jobs master: Add variable `npm_mirror` for change npm or yarn package registry https://review.openstack.org/647305 | 07:29 |
| openstackgerrit | William Huang proposed openstack-infra/zuul-jobs master: Add variable `npm_mirror` for change npm or yarn package registry https://review.openstack.org/647305 | 07:42 |
| *** themroc has joined #zuul | 07:42 | |
| *** badboy has joined #zuul | 08:00 | |
| *** gtema has joined #zuul | 08:05 | |
| *** sshnaidm|rover has joined #zuul | 08:38 | |
| *** zbr has quit IRC | 08:53 | |
| *** jpena|off is now known as jpena | 08:57 | |
| *** wxy-xiyuan has joined #zuul | 09:07 | |
| *** gtema has quit IRC | 09:11 | |
| *** hashar has joined #zuul | 09:12 | |
| *** zbr has joined #zuul | 09:19 | |
| *** sshnaidm|rover is now known as sshnaidm|pto | 10:34 | |
| *** sshnaidm|pto has quit IRC | 10:55 | |
| *** jpena is now known as jpena|off | 11:24 | |
| *** jpena|off is now known as jpena | 11:25 | |
| *** panda|drappt is now known as panda | 11:49 | |
| *** rfolco has joined #zuul | 12:00 | |
| *** rlandy has joined #zuul | 12:19 | |
| *** logan- has quit IRC | 12:27 | |
| *** sshnaidm|pto has joined #zuul | 12:30 | |
| *** logan- has joined #zuul | 12:31 | |
| *** rlandy is now known as rlandy|rover | 12:47 | |
| *** jpena is now known as jpena|lunch | 12:48 | |
| *** jamesmcarthur has joined #zuul | 12:50 | |
| *** quiquell is now known as quiquell|brb | 13:07 | |
| badboy | what can be the cause of "DEBUG zuul.web: Websocket close: 4011 Error with Gearman"? | 13:21 |
| badboy | that's in web-debug.log | 13:21 |
| *** sshnaidm|pto has quit IRC | 13:28 | |
| *** irclogbot_1 has joined #zuul | 13:29 | |
| *** altlogbot_2 has quit IRC | 13:31 | |
| *** altlogbot_2 has joined #zuul | 13:33 | |
| *** jamesmcarthur has quit IRC | 13:36 | |
| *** sshnaidm|pto has joined #zuul | 13:37 | |
| *** irclogbot_1 has quit IRC | 13:38 | |
| *** irclogbot_1 has joined #zuul | 13:40 | |
| *** jpena|lunch is now known as jpena | 13:46 | |
| *** sshnaidm|pto has quit IRC | 13:53 | |
| *** sshnaidm has joined #zuul | 13:53 | |
| *** sshnaidm is now known as sshnaidm|pto | 14:01 | |
| *** jamesmcarthur has joined #zuul | 14:29 | |
| corvus | badboy: it's from here: https://git.zuul-ci.org/cgit/zuul/tree/zuul/web/__init__.py#n137 | 14:34 |
| corvus | badboy: it probably means the build for that uuid wasn't running at the time | 14:35 |
| corvus | pabelanger, Shrews: i like Shrews's plan for secure.conf | 14:58 |
| tobiash | ++ | 15:05 |
| *** themroc has quit IRC | 15:14 | |
| *** saneax has quit IRC | 15:18 | |
| *** quiquell|brb is now known as quiquell|off | 15:23 | |
| tobiash | corvus, mordred: do you know the author of https://review.openstack.org/632498 ? | 15:29 |
| tobiash | I think this change makes sense so I like to move if forward if that's ok | 15:29 |
| corvus | fbo: ^ do you know? | 15:37 |
| pabelanger | tobiash: corvus: that is panda | 15:39 |
| tobiash | panda: do you want to fix the test cases of your change https://review.openstack.org/632498 or shall I take it over? | 15:40 |
| corvus | yeah, i guess now that we can specify a change+job, that's probably specific enough to set an autohold on success | 15:40 |
| tobiash | corvus: my use case would be to only hold on post_failure | 15:41 |
| corvus | ack | 15:41 |
| tobiash | I've had had a use case today to track down a network issue | 15:41 |
| panda | tobiash: Itried to discuss the change a bit on the mailing list, but the discussion wasn't moving forward. I didn't sped much time on it TBH. | 15:54 |
| tobiash | panda: ah ok, do you want to work on it further or shall I do it? | 15:54 |
| panda | tobiash: it may be easier for you to get consensus on a decision | 15:55 |
| tobiash | panda: ok | 15:56 |
| tobiash | thanks | 15:56 |
| corvus | looks like http://lists.zuul-ci.org/pipermail/zuul-discuss/2019-January/000715.html was previous discussion | 15:57 |
| corvus | Shrews asked some questions, but i don't see a followup | 15:57 |
| corvus | panda: ^ that's still a good question if you have a minute to answer. i mean, i think tobiash brought up a good use case, and i could see setting a hold on 'success' being useful too, but i don't see how that helps with the stepwise debugging you described (you could inject a fake "success" but that's the same as injecting a fake "failure") | 16:00 |
| panda | corvus: yeah, forcing a failure was working fine for us, that why I didn't move the change forward. What's really remains for me is that seesm more natural to alter the sequence and then request a autohold on success, than alter the sequence, inject a failure so we can trick zuul to autohold the node. | 16:13 |
| *** panda is now known as panda|ko | 16:27 | |
| *** jamesmcarthur has quit IRC | 17:26 | |
| *** jamesmcarthur has joined #zuul | 17:31 | |
| pabelanger | corvus: tobiash: were we going to do a zuul 3.7.1 release today? | 17:49 |
| tobiash | Fine for me, that default_ansinle_version fix finally landed after 9 rechecks | 17:50 |
| corvus | yep, i'll get started on that in just a few mins | 17:50 |
| pabelanger | cool | 17:51 |
| pabelanger | I am unsure if we have anything ready for nodepool, but a new release to pick up yaml.safe_load fix would help remove warning for nodepool CLI. | 17:53 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Organize documentation by subject area https://review.openstack.org/645955 | 17:57 |
| corvus | tobiash: i'll tag d283700b5dc075209570065e6a573d7d6f6ab53a as zuul 3.7.1 ? | 17:58 |
| *** jamesmcarthur has quit IRC | 17:59 | |
| tobiash | corvus: lgtm | 18:00 |
| *** jamesmcarthur has joined #zuul | 18:01 | |
| tobiash | corvus: did you re-decide to tag 6805f872? | 18:01 |
| tobiash | that just contains two test case fixes so lgtm too | 18:02 |
| corvus | tobiash: yep i messed that up sorry. but i also agree we can just let it run. | 18:03 |
| corvus | we'll set up reviewed tags some day :) | 18:04 |
| tobiash | :) | 18:04 |
| corvus | (openstack has all the machinery for it, we just need to figure out how to generalize it) | 18:04 |
| corvus | tobiash: gentle + hard! i never thought of that! what a great idea :) | 18:06 |
| tobiash | :) | 18:06 |
| openstackgerrit | Merged openstack-infra/zuul master: Fix deprecation warning around yaml.load https://review.openstack.org/644651 | 18:07 |
| *** jamesmcarthur has quit IRC | 18:09 | |
| mordred | corvus: I know - me neither! | 18:12 |
| corvus | i wish clarkb were here to see this | 18:12 |
| *** jpena is now known as jpena|off | 18:12 | |
| tobiash | corvus: I noticed that almost all of the slow runs of zuul tox py3* ran in ovh bhs1 | 18:12 |
| corvus | mordred, clarkb, i, and others have spent *hours and hours* talking about gentle vs hard timeouts... | 18:13 |
| tobiash | is this cloud just slower ir does it have a problem? | 18:13 |
| tobiash | really? | 18:13 |
| mordred | corvus: clearly not enough hours | 18:13 |
| mordred | corvus: or, more to the point - clearly we did not include the right people in those conversations :) | 18:13 |
| tobiash | :) | 18:14 |
| corvus | we seem to have made considerable progress on that point :) | 18:14 |
| corvus | tobiash: i've lost track of the ovh conversation; i think clarkb was leading that but maybe fungi knows if ovh-bhs1 is just slow or has a problem? | 18:15 |
| *** hashar is now known as hasharAway | 18:16 | |
| tobiash | ah, so this is known | 18:17 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Minor improvements to docker-image doc structure https://review.openstack.org/645897 | 18:17 |
| *** chandankumar has quit IRC | 18:17 | |
| *** chandankumar has joined #zuul | 18:18 | |
| corvus | tobiash: yeah, there was something about io access being slow and one of the regions needed to be corrected, but i don't remember which, so i can't say at the moment if we expect it to have been fixed in the past, or will be fixed in the future, or if that's just the way it will be | 18:20 |
| openstackgerrit | Merged openstack-infra/zuul master: Increase global test timeout https://review.openstack.org/645974 | 18:35 |
| openstackgerrit | Merged openstack-infra/zuul master: Make hard test timeouts more gentle https://review.openstack.org/645975 | 18:37 |
| openstackgerrit | Merged openstack-infra/zuul master: Add missing thread joins to tests https://review.openstack.org/645976 | 18:40 |
| fungi | corvus: i haven't heard anything specific to ovh-bhs1 lately. i know they were working on some openstack upgrades across all their regions in previous months and ran into some overloaded network gear as a result, but that was a while ago and i thought it was since mitigated | 18:43 |
| fungi | likely fixed in the past-future | 18:44 |
| fungi | er, was-will-be fixed i mean | 18:44 |
| * fungi did finally get his hands on a korean release of the non-dubbed french version of gandahar | 18:45 | |
| corvus | fungi: is that the Future Semiconditionally Modified Subinverted Plagal Past Subjunctive Intentional you used there? | 18:46 |
| fungi | um... yes, most definitely? | 18:48 |
| openstackgerrit | Merged openstack-infra/zuul master: Update component diagram to show statsd https://review.openstack.org/645798 | 18:55 |
| *** jamesmcarthur has joined #zuul | 19:21 | |
| *** hasharAway has quit IRC | 19:21 | |
| zbr | how can I make zuul/ansible bit more verbose when running a pre task? | 19:22 |
| zbr | i got a failure that looks like a bug in a module and default verbosity is kinda useless.also using debug statements does not help here. | 19:22 |
| tobiash | zbr: look into the job-output.json, that often contains more information | 19:23 |
| tobiash | zbr: you can also execute 'zuul-executor verbose' | 19:24 |
| tobiash | this will enable verbose logging into the executor log | 19:24 |
| zbr | tobiash: thanks! i got enough in json file. interestingly I tried ara before but it does not include the infor from there. | 19:25 |
| zbr | tobiash: json is enough, i already figured it out | 19:25 |
| tobiash | great | 19:25 |
| *** rlandy|rover is now known as rlandy|rover|mtg | 19:31 | |
| *** jamesmcarthur has quit IRC | 19:34 | |
| *** hasharAway has joined #zuul | 19:36 | |
| openstackgerrit | Merged openstack-infra/zuul master: Increase wait_timeout of test_playbook https://review.openstack.org/645982 | 19:41 |
| *** jamesmcarthur has joined #zuul | 19:49 | |
| openstackgerrit | Antoine Musso proposed openstack-infra/zuul master: gerrit: use change: when querying changes https://review.openstack.org/620328 | 20:01 |
| openstackgerrit | Merged openstack-infra/zuul master: Increase zk session timeout during tests https://review.openstack.org/645983 | 20:01 |
| *** hasharAway is now known as hashar | 20:04 | |
| *** rlandy|rover|mtg is now known as rlandy|rover | 20:10 | |
| corvus | jhesketh, tristanC: i reviewed several changes on the runner stack; i'm +2 on the early ones, but i think i need documentation on the later ones before i can fully understand them (and certainly before we merge them) | 20:14 |
| corvus | jhesketh, tristanC: also, it seems several still need rebasing | 20:14 |
| *** zbr has quit IRC | 20:22 | |
| *** zbr has joined #zuul | 20:22 | |
| *** pcaruana has quit IRC | 20:23 | |
| mordred | corvus: going through those - the api endpoint for frozen jobs has 2 +2's but no +A - are you wanting a wider set of +2 there? | 20:26 |
| openstackgerrit | Merged openstack-infra/zuul master: Fix test race in test_periodic_override https://review.openstack.org/645989 | 20:27 |
| corvus | mordred: good q -- i just left a WIP vote on 607078 to hold it and children until we get more votes | 20:28 |
| corvus | mordred: i think 607077 and 631703 can land now | 20:28 |
| mordred | corvus: yah - 607077 seemed very straightforward | 20:28 |
| mordred | corvus: and good re: WIP vote | 20:29 |
| openstackgerrit | Merged openstack-infra/zuul master: Increase event timeouts in git tests https://review.openstack.org/646001 | 20:32 |
| openstackgerrit | Merged openstack-infra/zuul master: Mock system load in executor governor tests https://review.openstack.org/646002 | 20:32 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: WIP: Fix for image build leaks https://review.openstack.org/647599 | 20:38 |
| Shrews | I need to come up with a way to test ^^, which is not easy :/ | 20:39 |
| *** dmsimard has quit IRC | 20:39 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: WIP: Fix for image build leaks https://review.openstack.org/647599 | 20:43 |
| openstackgerrit | Merged openstack-infra/zuul master: tests: use a single build reference in stream test https://review.openstack.org/647281 | 20:50 |
| openstackgerrit | Merged openstack-infra/zuul master: tests: be more gentle in the iterate timeout loop https://review.openstack.org/647267 | 20:50 |
| ianw | corvus: https://review.openstack.org/#/q/status:open+topic:opendev-gerrit-git+NOT+label:Verified=-1+NOT+label:Verified=%2B1+NOT+label:Verified=-2+NOT+label:Verified=%2B2+NOT+Code-Review:%2B+2 | 20:54 |
| ianw | corvus: ^ i have not dug deeper in any way, but i would have expected zuul to comment on those changes from yesterday | 20:55 |
| corvus | ianw: ack i'll look | 20:55 |
| openstackgerrit | Merged openstack-infra/zuul master: web: add /connections route https://review.openstack.org/631703 | 20:55 |
| corvus | ianw: oh, i see they all have stable branches in common | 20:55 |
| corvus | ianw: my guess is wacky zuul config | 20:56 |
| corvus | will continue to look | 20:56 |
| corvus | ianw: http://paste.openstack.org/show/748338/ | 20:57 |
| corvus | so yeah, there is no zuul configuration for those branches | 20:58 |
| ianw | yeah, that makes sense | 20:59 |
| ianw | i probably should have filtered out all those stable/XX.YY branches ... the scripts were just looking for stable/*, and then yeah, the rest is newton and mitaka ... tonyb might know more about why they're around | 21:02 |
| hashar | corvus: hello, you are very kind with my dummy change (to have the gerrit connection query changes with change:%s) ;) | 21:14 |
| hashar | and my 2nd patchset was just a guess to exercise CI and get test failure, I am quite pleased to see they all passed and the change got approved! | 21:15 |
| corvus | hashar: heh, it looked right to me :) thanks! | 21:16 |
| hashar | spotted that on our old v2.5 legacy zuul :/ | 21:18 |
| hashar | i guess we have enough changes/commits to eventually have collisions nowadays | 21:19 |
| *** jamesmcarthur_ has joined #zuul | 21:22 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Organize documentation by subject area https://review.openstack.org/645955 | 21:22 |
| *** jamesmcarthur_ has quit IRC | 21:23 | |
| *** jamesmcarthur_ has joined #zuul | 21:23 | |
| *** jamesmcarthur_ has quit IRC | 21:23 | |
| *** jamesmcarthur has quit IRC | 21:24 | |
| *** jamesmcarthur has joined #zuul | 21:24 | |
| *** jamesmcarthur_ has joined #zuul | 21:25 | |
| *** jamesmcarthur has quit IRC | 21:29 | |
| hashar | does anyone know whether zuul "container build resources" spec ended up implemented or is still something being considered ( https://zuul-ci.org/docs/zuul/developer/specs/container-build-resources.html#native-container-workflow ) ?;) | 21:37 |
| corvus | hashar: it merged | 21:43 |
| corvus | i mean, the implementation merged :) | 21:43 |
| corvus | hashar: https://zuul-ci.org/docs/nodepool/configuration.html#kubernetes-driver and https://zuul-ci.org/docs/nodepool/configuration.html#openshift-driver | 21:44 |
| hashar | ahhh openshift :] | 21:47 |
| hashar | corvus: thank you! | 21:47 |
| hashar | I am trying to wrap my mind around all those kubernetes / zuul v3 / executors / ansible / new nodepool etc | 21:47 |
| corvus | hashar: aiui, tristanC is using openshift as build resources. and several folks are running zuul *on* openshift. | 21:48 |
| openstackgerrit | Merged openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method https://review.openstack.org/633653 | 21:50 |
| *** brennen has joined #zuul | 21:52 | |
| hashar | corvus: thanks a lot for the pointer. I guess we will digest that and come back on the zuul mailing list with bunch of questoins eventually | 21:54 |
| *** jamesmcarthur_ has quit IRC | 22:06 | |
| *** hashar has quit IRC | 22:15 | |
| openstackgerrit | Merged openstack-infra/zuul master: gerrit: use change: when querying changes https://review.openstack.org/620328 | 22:21 |
| openstackgerrit | Merged openstack-infra/zuul master: Add API endpoint to get frozen jobs https://review.openstack.org/607077 | 22:25 |
| *** jamesmcarthur has joined #zuul | 22:32 | |
| *** jamesmcarthur has quit IRC | 22:56 | |
| *** jamesmcarthur has joined #zuul | 22:56 | |
| *** jamesmcarthur has quit IRC | 22:59 | |
| *** jamesmcarthur has joined #zuul | 23:01 | |
| *** jamesmcarthur has quit IRC | 23:40 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!