| *** threestrands has joined #zuul | 00:04 | |
| *** mrhillsman has quit IRC | 00:51 | |
| *** threestrands has quit IRC | 01:41 | |
| *** jhesketh has quit IRC | 02:12 | |
| *** jhesketh has joined #zuul | 02:13 | |
| *** smyers_ has joined #zuul | 02:18 | |
| *** smyers has quit IRC | 02:19 | |
| *** smyers_ is now known as smyers | 02:19 | |
| *** sgw has quit IRC | 02:33 | |
| *** bhavikdbavishi has joined #zuul | 02:45 | |
| *** bhavikdbavishi1 has joined #zuul | 03:34 | |
| *** bhavikdbavishi has quit IRC | 03:35 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:35 | |
| *** saneax has joined #zuul | 04:43 | |
| *** chandankumar has quit IRC | 05:06 | |
| *** chandankumar has joined #zuul | 05:07 | |
| *** swest has joined #zuul | 05:16 | |
| *** swest has quit IRC | 05:34 | |
| *** swest has joined #zuul | 05:35 | |
| *** swest has quit IRC | 05:37 | |
| *** swest has joined #zuul | 05:38 | |
| *** swest has quit IRC | 05:40 | |
| *** swest has joined #zuul | 05:41 | |
| *** AJaeger has joined #zuul | 05:57 | |
| *** swest has quit IRC | 06:03 | |
| *** swest has joined #zuul | 06:05 | |
| *** iamweswilson_ has joined #zuul | 06:07 | |
| *** daniel3 has joined #zuul | 06:07 | |
| *** evgenyl_ has joined #zuul | 06:08 | |
| *** samccann_ has joined #zuul | 06:08 | |
| *** aprice_ has joined #zuul | 06:08 | |
| *** ChrisShort_ has joined #zuul | 06:08 | |
| *** zer0c00l has joined #zuul | 06:10 | |
| *** swest has quit IRC | 06:13 | |
| *** swest has joined #zuul | 06:13 | |
| *** gouthamr_ has joined #zuul | 06:14 | |
| *** evgenyl has quit IRC | 06:15 | |
| *** samccann has quit IRC | 06:15 | |
| *** ChrisShort has quit IRC | 06:15 | |
| *** smcginnis has quit IRC | 06:15 | |
| *** zer0c00l_ has quit IRC | 06:15 | |
| *** aprice has quit IRC | 06:15 | |
| *** iamweswilson has quit IRC | 06:15 | |
| *** gouthamr has quit IRC | 06:15 | |
| *** daniel2 has quit IRC | 06:15 | |
| *** samccann_ is now known as samccann | 06:15 | |
| *** iamweswilson_ is now known as iamweswilson | 06:15 | |
| *** aprice_ is now known as aprice | 06:15 | |
| *** ChrisShort_ is now known as ChrisShort | 06:15 | |
| *** evgenyl_ is now known as evgenyl | 06:15 | |
| *** irclogbot_3 has quit IRC | 06:18 | |
| *** panda has quit IRC | 06:18 | |
| *** irclogbot_0 has joined #zuul | 06:20 | |
| *** panda has joined #zuul | 06:20 | |
| *** pcaruana has joined #zuul | 06:25 | |
| *** badboy has joined #zuul | 06:36 | |
| *** gtema has joined #zuul | 06:48 | |
| *** gtema has quit IRC | 07:06 | |
| *** gtema has joined #zuul | 07:09 | |
| *** swest has quit IRC | 07:32 | |
| *** hashar has joined #zuul | 07:34 | |
| *** flepied has joined #zuul | 07:38 | |
| *** sshnaidm|off has quit IRC | 07:48 | |
| *** sshnaidm has joined #zuul | 07:49 | |
| *** swest has joined #zuul | 07:53 | |
| *** swest has quit IRC | 08:06 | |
| *** themroc has joined #zuul | 08:12 | |
| badboy | anybody can tell me what's causing this error? | 09:51 |
|---|---|---|
| badboy | Zuul encountered an error while accessing the repo MYREPO The error was: type object 'EllipticCurvePublicKey' has no attribute 'from_encoded_point' | 09:52 |
| *** bstinson has quit IRC | 09:52 | |
| *** bstinson has joined #zuul | 09:53 | |
| *** gtema has quit IRC | 09:54 | |
| *** gtema has joined #zuul | 09:54 | |
| *** jpena|off is now known as jpena | 10:00 | |
| *** gtema_ has joined #zuul | 10:06 | |
| *** gtema has quit IRC | 10:06 | |
| badboy | so the problem is that paramiko-2.6.0 (latest) is causing this error, after downgrading to paramiko-2.4.2 everything is working | 10:08 |
| *** bhavikdbavishi has quit IRC | 10:34 | |
| *** hashar has quit IRC | 10:43 | |
| *** jpena is now known as jpena|lunch | 11:20 | |
| *** rfolco has joined #zuul | 11:46 | |
| *** rlandy has joined #zuul | 11:55 | |
| *** hashar has joined #zuul | 11:59 | |
| tobiash | corvus, fungi: I just found a build leak in zuul that leads to builds that never end if they will pause at the end. And as far as I can see that would also leat to a node leak. | 12:01 |
| tobiash | corvus, fungi: that might be related to the node leak you were seeing last week | 12:01 |
| tobiash | corvus: fungi: see this https://opendev.org/zuul/zuul/src/branch/master/zuul/executor/client.py#L401 | 12:01 |
| tobiash | corvus, fungi: I've seen this also in my environment where the reason was that the build url was only set a few minutes after we wanted to cancel the job | 12:02 |
| tobiash | but I don't know yet how to solve this | 12:03 |
| tobiash | I've also seen the message 'Build foo is not in the current build set' defined here: https://opendev.org/zuul/zuul/src/branch/master/zuul/scheduler.py#L1287 | 12:05 |
| tobiash | no, here: https://opendev.org/zuul/zuul/src/branch/master/zuul/scheduler.py#L1185 | 12:06 |
| tobiash | we just return here and don't resume/cancel the job in this case, so I think we just could cancel the job in this state which should resolve that leak | 12:08 |
| *** jangutter has joined #zuul | 12:12 | |
| *** hashar has quit IRC | 12:14 | |
| *** hashar has joined #zuul | 12:14 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Fix build leak with paused jobs https://review.opendev.org/667072 | 12:15 |
| tobiash | corvus, fungi: I think this would solve that leak but I don't know how to test this ^ | 12:17 |
| *** gundalow has quit IRC | 12:36 | |
| *** gtema_ has quit IRC | 12:38 | |
| *** gundalow has joined #zuul | 12:39 | |
| *** AJaeger has quit IRC | 12:41 | |
| *** AJaeger has joined #zuul | 12:53 | |
| *** zbr is now known as zbr|ruck | 12:53 | |
| *** jpena|lunch is now known as jpena | 13:02 | |
| *** NBorg_ has quit IRC | 13:09 | |
| pabelanger | tobiash: does that mean you've upgraded to 3.9.0? | 13:12 |
| pabelanger | also, have you had a change to do any ansible 2.8 zuul jobs testing? | 13:12 |
| pabelanger | we flipped all jobs to 2.8 over the weekend, and haven't seen any zuul-jobs syntax issues so far | 13:13 |
| tobiash | pabelanger: we run 2.7 by default atm and I personally didn't test 2.8 yet | 13:13 |
| tobiash | but we run most of the time from master + x ;) | 13:13 |
| pabelanger | ack. Mostly trying to collect stats on 2.8 issues, to decided when to make the default change | 13:13 |
| pabelanger | tobiash: ah, cool. So you are running the refactor of retries for github then | 13:14 |
| tobiash | yes | 13:15 |
| pabelanger | okay cool | 13:15 |
| pabelanger | tobiash: I suspect, I'll have some time this week to help provide feedback, but need to review your parallel PR too | 13:15 |
| tobiash | oh not yet, after the next restart tomorrow morning I will | 13:15 |
| *** jeliu_ has joined #zuul | 13:16 | |
| *** hashar has quit IRC | 13:17 | |
| *** hashar has joined #zuul | 13:25 | |
| pabelanger | Another thing, I know I asked before, why does github driver require depends-on to be the first comment of a PR? Can we not get it from each commit message? | 13:26 |
| *** wxy-xiyuan has quit IRC | 13:29 | |
| tobiash | the thing is that a pr is complicated and can contain hundreds of commits and merges back and forth | 13:32 |
| tobiash | so the only workable place is the pr description | 13:33 |
| tobiash | and that's the only place where the user can see it easily (as github hides the commit messages themselves pretty good) | 13:34 |
| pabelanger | isn't commit message found under: https://github.com/ansible/ansible/pull/58273/commits for example | 13:36 |
| tobiash | yes, but you need to walk through all commit messages then and it's hidden from the main pr page | 13:38 |
| tobiash | and you cannot easily change a depends-on that's buried in the 345th commit of a pull request | 13:38 |
| pabelanger | Right, the bonus to that, is not having to update github UI, when you make a depends on change, via commit message. But, I hear what you are saying | 13:39 |
| tobiash | check out the hub tool, I create most pull requests via commandline | 13:40 |
| tobiash | and with that you don't need the ui either | 13:40 |
| tobiash | https://github.com/github/hub | 13:41 |
| tobiash | I've an alias 'pr' that creates a pr with the current remote tracking branch | 13:41 |
| pabelanger | tobiash: yah, I use git pull-request from jd. It helps a lot, but with ansible/ansible, there are also https://github.com/ansible/ansible/blob/devel/.github/PULL_REQUEST_TEMPLATE.md | 13:42 |
| pabelanger | which, doesn't seem to update each git push that is done | 13:42 |
| tobiash | ah ok | 13:42 |
| pabelanger | I wonder how depends-on works with pagure now that it is merged | 13:43 |
| pabelanger | it would be neat, if it work the same way across all drivers | 13:43 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: URLTrigger driver time based https://review.opendev.org/635567 | 13:46 |
| fungi | badboy: it looks like that error means you need a newer release of pyca/cryptography | 13:51 |
| *** bhavikdbavishi has joined #zuul | 14:03 | |
| *** bhavikdbavishi1 has joined #zuul | 14:06 | |
| *** bhavikdbavishi has quit IRC | 14:07 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 14:07 | |
| *** smcginnis has joined #zuul | 14:12 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: WIP: new devstack-based functional job https://review.opendev.org/665023 | 14:22 |
| *** hashar has quit IRC | 14:24 | |
| *** themroc has quit IRC | 14:32 | |
| *** jamesmcarthur has joined #zuul | 14:58 | |
| *** panda has quit IRC | 15:09 | |
| *** bhavikdbavishi has quit IRC | 15:10 | |
| *** panda has joined #zuul | 15:11 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Split ansible output into its own sub logger https://review.opendev.org/667131 | 15:22 |
| *** sgw has joined #zuul | 15:23 | |
| *** daniel3 is now known as daniel2 | 15:29 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: WIP: new devstack-based functional job https://review.opendev.org/665023 | 15:32 |
| *** jpena is now known as jpena|off | 15:48 | |
| *** sgw has quit IRC | 15:53 | |
| *** jamesmcarthur has quit IRC | 15:56 | |
| *** gtema_ has joined #zuul | 16:00 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Allow config projects to override allowed-projects https://review.opendev.org/666733 | 16:05 |
| *** sgw has joined #zuul | 16:05 | |
| corvus | SpamapS: https://review.opendev.org/666733 should be ready | 16:15 |
| SpamapS | corvus: reviewed (note there are some errant spaces-only chunks in the YAML.. I forgot to comment on them.. I don't actually care.. but.. if you make another rev. :) | 16:23 |
| clarkb | /query fungi | 16:23 |
| clarkb | derp | 16:23 |
| *** gtema_ has quit IRC | 16:24 | |
| corvus | oops, yeah, i'll get those if there's another ps | 16:24 |
| tobiash | Lgtm, do you want to clear those whitespaces before landing? | 16:30 |
| *** mattw4 has joined #zuul | 16:32 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Allow config projects to override allowed-projects https://review.opendev.org/666733 | 16:32 |
| corvus | sure :) | 16:32 |
| *** hashar has joined #zuul | 16:37 | |
| *** bhavikdbavishi has joined #zuul | 16:38 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Switch functional testing to a devstack consumer job https://review.opendev.org/665023 | 16:41 |
| *** igordc has joined #zuul | 16:42 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: WIP: remove devstack plugin functional test jobs https://review.opendev.org/667156 | 16:45 |
| clarkb | corvus: ^ one small suggestion for further optimizing the openstack cloud install | 16:47 |
| *** mattw4 has quit IRC | 16:50 | |
| *** hwangbo has joined #zuul | 16:51 | |
| *** mattw4 has joined #zuul | 16:52 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add install-devstack role https://review.opendev.org/667157 | 16:53 |
| corvus | clarkb: good point, thx -- i was about to move that role into zuul-jobs too, so i'll do that as part of the move and next patchset ^ | 16:54 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add install-devstack role https://review.opendev.org/667157 | 16:55 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Switch functional testing to a devstack consumer job https://review.opendev.org/665023 | 16:56 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: WIP: remove devstack plugin functional test jobs https://review.opendev.org/667156 | 16:56 |
| *** hashar has quit IRC | 17:06 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Use password lookup for run-buildset-registry role https://review.opendev.org/663119 | 17:11 |
| *** michael-beaver has joined #zuul | 17:18 | |
| *** panda has quit IRC | 17:18 | |
| *** panda has joined #zuul | 17:20 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add install-devstack role https://review.opendev.org/667157 | 18:12 |
| evgenyl | Hi everyone, I'm trying to get `zuul-base-jobs` and `zuul-jobs` to work for my Zuul installation. When the job is triggered it cannot find `add-build-sshkey` https://paste.ubuntu.com/p/wbfW6HK37J/ , I specified `zuul/zuul-jobs` in `untrusted-projects` as it is described in the docs https://paste.ubuntu.com/p/QDwwkwTnKj/ , any ideas how to debug that? Thanks! | 18:54 |
| *** jangutter has quit IRC | 18:58 | |
| pabelanger | evgenyl: did you add zuul/zuul-jobs to your base? eg: https://zuul-ci.org/docs/zuul/user/config.html#attr-job.roles | 19:00 |
| evgenyl | pabelanger: Hmm, let me check that, have not noticed this config, thanks! | 19:02 |
| pabelanger | I am guessing there is an issue with: https://opendev.org/zuul/zuul-base-jobs/src/branch/master/zuul.yaml#L14 | 19:02 |
| pabelanger | if you are consuming that directly | 19:02 |
| *** themroc has joined #zuul | 19:04 | |
| evgenyl | pabelanger: Yes, I see now! What options do I have? I can probably add a repo with name `git.zuul-ci.org` and project `zuul-jobs`, or am I expected to override it for every job? | 19:05 |
| pabelanger | evgenyl: you should use opendev.org/zuul/zuul-jobs | 19:05 |
| pabelanger | opendev.org is optional, depending on your connection settings in zuul.conf | 19:05 |
| pabelanger | evgenyl: for example, https://github.com/ansible-network/windmill-config/blob/master/zuul/zuul.conf.j2#L88 is how we set up the connection to opendev (via gerrit) | 19:06 |
| evgenyl | pabelanger: Thanks, let me try that. | 19:11 |
| *** bhavikdbavishi has quit IRC | 19:18 | |
| *** themroc has quit IRC | 19:29 | |
| *** gtema_ has joined #zuul | 19:33 | |
| corvus | i guess we should update the base-jobs repo with the new url | 19:42 |
| *** saneax has quit IRC | 19:44 | |
| corvus | clarkb: i think https://review.opendev.org/665023 is ready for realz | 19:45 |
| pabelanger | Hmm, we just had a log fail to upload to swift: http://paste.openstack.org/show/753324/ | 20:00 |
| pabelanger | I wonder if it is safe to register task variable and loop to upload again | 20:01 |
| clarkb | corvus: reviewing it now and I think the one piece I'm not seeing is what runs zookeeper? | 20:01 |
| corvus | clarkb: i'm guessing the bindep does it? | 20:03 |
| clarkb | corvus: yup http://logs.openstack.org/23/665023/38/check/nodepool-functional-openstack/9168c9a/job-output.txt.gz#_2019-06-24_18_17_49_391929 looks like that is the case | 20:05 |
| clarkb | both changes lgtm | 20:05 |
| clarkb | pabelanger: looks like you got an error from vexxhost | 20:08 |
| clarkb | 504 gateway timeout | 20:08 |
| clarkb | that would be on the vexxhost side that failed | 20:08 |
| pabelanger | clarkb: yah, about the first time that has happened. I think adding it some sort of retry option, will help to remove the POST_FAILURE zuul reports | 20:13 |
| clarkb | if the servers are timing out on the backend a retry may only make the problem worse though | 20:13 |
| clarkb | we may want to check with mnaser to see if he thinks that would help | 20:13 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: WIP: remove devstack plugin functional test jobs https://review.opendev.org/667156 | 20:26 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add env var customization to openstack func job https://review.opendev.org/667217 | 20:26 |
| pabelanger | clarkb: good point | 20:29 |
| *** pcaruana has quit IRC | 20:44 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Remove devstack plugin functional test jobs https://review.opendev.org/667156 | 20:46 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Remove devstack plugin functional test jobs https://review.opendev.org/667156 | 20:51 |
| clarkb | The parallel github requests with one thread per installation id change lgtm https://review.opendev.org/#/c/666982/1. I did have one minor thing inline if it needs a respin | 20:51 |
| clarkb | tobiash: ^ | 20:51 |
| tobiash | clarkb: thanks | 20:52 |
| *** gtema_ has quit IRC | 20:53 | |
| *** gtema has joined #zuul | 20:56 | |
| *** rfolco has quit IRC | 20:57 | |
| corvus | zuul-maint: https://review.opendev.org/667228 Add zuul-operator project | 20:59 |
| corvus | zuul-maint: i think there is general consensus that we'd like to have an operator; even though the details are still being worked out in the spec, i think we can go ahead and import that repo and start working on the scaffolding. | 21:00 |
| clarkb | corvus: fwiw https://review.opendev.org/#/c/641474/ exists too | 21:00 |
| corvus | well that would have saved some time | 21:01 |
| clarkb | that change definitely needs updating so maybe easiest to abandon and go with the new one | 21:01 |
| clarkb | but one will want ot be abandoned I think | 21:01 |
| tobiash | Yay :) | 21:01 |
| corvus | oh yeah | 21:01 |
| corvus | let's go with mine, since i'm importing tristan's poc | 21:01 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add env var customization to openstack func job https://review.opendev.org/667217 | 21:05 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Remove devstack plugin functional test jobs https://review.opendev.org/667156 | 21:05 |
| *** gouthamr_ is now known as gouthamr | 21:16 | |
| *** ianychoi_ has joined #zuul | 21:19 | |
| *** ianychoi has quit IRC | 21:22 | |
| *** michael-beaver has quit IRC | 21:25 | |
| *** ianychoi_ has quit IRC | 21:30 | |
| *** ianychoi_ has joined #zuul | 21:31 | |
| *** gtema has quit IRC | 21:40 | |
| *** PaulSheldon has joined #zuul | 21:45 | |
| *** PaulSheldon has quit IRC | 21:49 | |
| clarkb | corvus: looking at https://review.opendev.org/#/c/666733/3 seems like that allows for a lot of new footgunning but https://review.opendev.org/#/c/666733/3/doc/source/user/config.rst isn't updated to warn people of the implications of this. Maybe we should add something like "a config project may still add this job to any project's pipeline. Apply caution when doing so as other projects may expose the source | 21:57 |
| clarkb | project's secrets. | 21:57 |
| clarkb | or something | 21:57 |
| *** panda has quit IRC | 21:58 | |
| *** panda has joined #zuul | 22:03 | |
| *** tobiash has quit IRC | 22:10 | |
| corvus | clarkb: good point. it's basically a third party stepping in and saying that the second party is trustworthy -- we should convey that. | 22:25 |
| fungi | maybe with a big bold admonition callout box | 22:26 |
| corvus | fungi: above or below the existing one? :) | 22:26 |
| fungi | heh | 22:26 |
| fungi | yeah, i guess inside | 22:27 |
| clarkb | or maybe even a separate "things to consider with secerts" section | 22:30 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Strengthen the caution about allowed-projects https://review.opendev.org/667241 | 22:31 |
| corvus | clarkb, fungi: ^ implemented as a followup so we can bikeshed with abandon | 22:31 |
| clarkb | that looks good for now. I think any things to consider seciont would be a bigger followup | 22:33 |
| *** jeliu_ has quit IRC | 22:39 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Expand documentation of test-setup role https://review.opendev.org/667244 | 22:54 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Switch functional testing to a devstack consumer job https://review.opendev.org/665023 | 22:55 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add env var customization to openstack func job https://review.opendev.org/667217 | 22:55 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Remove devstack plugin functional test jobs https://review.opendev.org/667156 | 22:55 |
| *** ianw_pto is now known as ianw | 23:08 | |
| *** saneax has joined #zuul | 23:41 | |
| *** rlandy has quit IRC | 23:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!