| *** homeski has joined #zuul | 00:04 | |
| homeski | is it possible to pass Ansible tags to jobs? | 00:14 |
|---|---|---|
| *** michael-beaver has quit IRC | 00:32 | |
| *** corvus has quit IRC | 01:12 | |
| *** corvus has joined #zuul | 01:12 | |
| *** bhavikdbavishi has joined #zuul | 02:53 | |
| *** bhavikdbavishi1 has joined #zuul | 02:56 | |
| *** bhavikdbavishi has quit IRC | 02:58 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 02:58 | |
| *** bhavikdbavishi has quit IRC | 03:10 | |
| *** bhavikdbavishi has joined #zuul | 03:11 | |
| *** bhavikdbavishi1 has joined #zuul | 04:02 | |
| *** bhavikdbavishi has quit IRC | 04:04 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 04:04 | |
| *** swest has joined #zuul | 05:21 | |
| *** sshnaidm_ is now known as sshnaidm|ruck | 05:27 | |
| *** snapiri has joined #zuul | 06:08 | |
| *** AJaeger has quit IRC | 06:11 | |
| *** AJaeger has joined #zuul | 06:13 | |
| *** themroc has joined #zuul | 06:23 | |
| *** tosky has joined #zuul | 07:25 | |
| *** gtema_ has joined #zuul | 07:33 | |
| *** pcaruana has joined #zuul | 07:43 | |
| *** gtema_ has quit IRC | 08:04 | |
| *** gtema_ has joined #zuul | 08:05 | |
| *** jangutter has joined #zuul | 08:11 | |
| *** gtema_ has quit IRC | 08:14 | |
| tobiash | homeski: no, that's currently not supported | 08:42 |
| *** bolg has joined #zuul | 08:57 | |
| *** bolg has quit IRC | 08:58 | |
| *** bolg has joined #zuul | 08:59 | |
| *** bhavikdbavishi has quit IRC | 09:27 | |
| *** bhavikdbavishi has joined #zuul | 10:36 | |
| *** bhavikdbavishi has quit IRC | 10:38 | |
| *** hwangbo has quit IRC | 10:54 | |
| *** bhavikdbavishi has joined #zuul | 10:57 | |
| *** themroc has quit IRC | 11:18 | |
| *** themroc has joined #zuul | 11:25 | |
| *** pcaruana has quit IRC | 11:36 | |
| *** bhavikdbavishi has quit IRC | 11:37 | |
| *** bolg has quit IRC | 11:40 | |
| *** bolg has joined #zuul | 11:42 | |
| *** bhavikdbavishi has joined #zuul | 11:44 | |
| *** gtema_ has joined #zuul | 11:48 | |
| openstackgerrit | Simon Westphahl proposed zuul/nodepool master: Allow proceeding with requests on quota exceeded https://review.opendev.org/667371 | 11:51 |
| *** gtema_ has quit IRC | 12:09 | |
| *** rlandy has joined #zuul | 12:28 | |
| *** electrofelix has joined #zuul | 12:30 | |
| *** bhavikdbavishi has quit IRC | 13:39 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Prepare: Use PDK to build puppet module https://review.opendev.org/667590 | 13:53 |
| *** bolg has quit IRC | 13:56 | |
| *** swest has quit IRC | 14:05 | |
| *** pcaruana has joined #zuul | 14:06 | |
| openstackgerrit | Merged zuul/zuul-website master: Change git to opendev.org URLs https://review.opendev.org/667820 | 14:08 |
| *** jamesmcarthur has joined #zuul | 14:20 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Use PDK to build puppet module https://review.opendev.org/627534 | 14:28 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Spec: Add a Kubernetes Operator for Zuul https://review.opendev.org/659180 | 14:29 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Spec: Add a Kubernetes Operator for Zuul https://review.opendev.org/659180 | 14:30 |
| mordred | corvus: ^^ new version, and then a rebase patch to deal with merge conflict | 14:31 |
| corvus | mordred: \o/ thanks! | 14:32 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Test install-pdk-dependencies role https://review.opendev.org/668337 | 14:40 |
| AJaeger | corvus: should I add a job for project-config as well - or wait for your change https://review.opendev.org/668214 etc? | 14:41 |
| corvus | AJaeger: let's add it to p-c for now and move it over once we're running with 668214 | 14:42 |
| AJaeger | mordred: you might be interested in https://review.opendev.org/668214 and then https://review.opendev.org/#/c/668061/ (not yet ready) | 14:42 |
| AJaeger | corvus: ok, will propose... | 14:42 |
| *** themroc has quit IRC | 14:53 | |
| *** pcaruana has quit IRC | 14:56 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Spec: Add a Kubernetes Operator for Zuul https://review.opendev.org/659180 | 14:58 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Spec: Add a Kubernetes Operator for Zuul https://review.opendev.org/659180 | 15:01 |
| *** hashar has joined #zuul | 15:19 | |
| *** hashar has quit IRC | 15:51 | |
| *** weshay is now known as weshay_PTO | 15:52 | |
| *** hashar has joined #zuul | 15:53 | |
| *** pcaruana has joined #zuul | 15:53 | |
| homeski | I have a question about provides/requires attributes for jobs | 16:05 |
| homeski | I tried using them but I don't think I'm understanding how they get applied correctly ... | 16:05 |
| homeski | I'm trying to achieve the following: https://gist.githubusercontent.com/homeski/18b5d4333fc7e40b602db19fa7476384/raw/900c257524b6e42c69a820039566b01160fed21c/zuul.png | 16:05 |
| homeski | I want to have several jobs that "require" job-2, so that job-2 only runs once and then any jobs that require it will then run on the same node | 16:07 |
| *** mattw4 has joined #zuul | 16:20 | |
| *** mattw4 has quit IRC | 16:25 | |
| *** hwangbo has joined #zuul | 16:29 | |
| *** rlandy is now known as rlandy|brb | 16:30 | |
| *** chandankumar is now known as raukadah | 16:34 | |
| ofosos | Hey ho, I'm trying to run a Zuul job on a branch in repo A, but project config in the .zuul.yaml in repo A doesn't seem to match, instead I get a match for the default project config... what could be the reason? Branch protection? | 16:36 |
| tobiash | homeski: zuul runs each job on a new fresh node. | 16:40 |
| tobiash | ofosos: how does your scenario look like? | 16:42 |
| *** mattw4 has joined #zuul | 16:43 | |
| ofosos | Repository A had a master branch and my changes live on sth like SYS-2268 | 16:45 |
| corvus | homeski: you can use provides/requires so that job-2 can store its artifacts somewhere and job-3 can fetch those artifacts. but keep this in mind -- provides/requires is for connecting jobs across different *changes*. for jobs within a single change, use job dependencies and the zuul_return module to pass information between them. | 16:45 |
| ofosos | tobiash: what exactly so you mean by scenario? | 16:47 |
| tobiash | ofosos: and the job ran the wrong thing or didn't it run at all? | 16:47 |
| ofosos | Didn't run at all, the default job is empty. It's in zuul config | 16:48 |
| tobiash | ofosos: is the target branch protected? (assuming you're using github) | 16:49 |
| ofosos | Nope, I'm using bitbucket. The target branch has no .zuul.yaml | 16:50 |
| openstackgerrit | Kevin Carter (cloudnull) proposed zuul/zuul-jobs master: Fix docker install on fedora https://review.opendev.org/668527 | 16:52 |
| corvus | ofosos: is the only project-pipeline configuration for that project in the .zuul.yaml file on the master branch of that repo? | 16:53 |
| corvus | ofosos: https://zuul-ci.org/docs/zuul/user/config.html#configuration-loading and https://zuul-ci.org/docs/zuul/user/config.html#attr-job.branches describe the branch specific behavior and may be helpful | 17:00 |
| *** hashar has quit IRC | 17:03 | |
| *** mattw4 has quit IRC | 17:10 | |
| *** mattw4 has joined #zuul | 17:13 | |
| corvus | fbo: https://src.fedoraproject.org/rpms/python-gear/pull-request/5#_2__15 isn't the right release URL; you should still use tarballs.openstack.org. later, we'll probably change it to tarballs.opendev.org. but the tarball "archive" that gitea builds will never be correct; it won't have the right version info -- it's not a release, it's just an archive of the git repo at that point. | 17:15 |
| corvus | fbo: i would have left that as a comment, but i got some error about contributor agreements and gave up | 17:15 |
| ofosos | No there are probably multiple branches in zuul config | 17:15 |
| clarkb | corvus: fbo: also you can download the sdist from pypi as well | 17:16 |
| corvus | yes, that's probably even better than tarballs.o.o | 17:16 |
| fbo | corvus: clarkb thanks I'll fix | 17:20 |
| ofosos | corvus: So, the change is in an untrusted project and from the docs I would expect that these are included. | 17:21 |
| ofosos | corvus: The job itself is in an untrusted repo B (distinc from repo A and from repo zuul-config), that has a branch spec that reads `^.*$' | 17:22 |
| ofosos | Then again I get this: | 17:24 |
| corvus | ofosos: the scheduler debug logs should contain logs regarding the decisions zuul made about what jobs to run for a change; it might be good to look at that. also, when zuul performs a reconfiguration (ie, at startup), it logs the project-pipeline configuration for every project. that may be useful too. | 17:24 |
| ofosos | 2019-07-01T16:31:00.88929381Z 2019-07-01 16:31:00,889 DEBUG zuul.layout: Variant <Job cfn-lint branches: {MatchAny:{BranchMatcher:master}} source: SYS-IC/zuul-common/zuul.yaml@master#14> did not match <Change 0x7efffecfda90 IAC/puppetmaster-bootstrap 11,55ce59d3364af97e4655b017035946a5982b7d72> | 17:24 |
| ofosos | Hmm, in a untrusted-project, is the `zuul.yaml' read? Or does it need to be named `.zuul.yaml'? | 17:26 |
| corvus | ofosos: okay, so on line 14 of zuul-common/zuul.yaml there is some zuul config object with a branch matcher of 'master' -- what is it? | 17:26 |
| corvus | ofosos: see https://zuul-ci.org/docs/zuul/user/config.html#configuration-loading for which files are used | 17:27 |
| ofosos | https://www.irccloud.com/pastebin/SfxAdKOO/ | 17:27 |
| ofosos | That is the file. | 17:28 |
| ofosos | Pardon, it's the snippet at line 14 | 17:28 |
| ofosos | I'm a little out of luck, why this would not match/branch matcher is `master` | 17:29 |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix docker install on fedora https://review.opendev.org/668527 | 17:29 |
| corvus | ofosos: that seems in contradiction with the debug log -- are you sure zuul is running with that configuration? (does a config error show up in the status ui? do you need to run "zuul full-reconfigure" or restart the scheduler?) | 17:30 |
| ofosos | There's no config error in the UI. | 17:31 |
| corvus | ofosos: i'd run a "zuul full-reconfigure" and try again just to make sure it's really using that, because the debug log looks like what i would expect if the "branches: ^.*$" were not present. | 17:34 |
| *** rlandy|brb is now known as rlandy | 17:36 | |
| *** electrofelix has quit IRC | 17:38 | |
| tobiash | corvus: if you have time, it would be awesome to get a gear release this week to get the server name indication feature. I've also tested it in a test environment and it works nicely :) | 17:39 |
| corvus | tobiash: shouldn't be a problem, thanks for the reminder | 17:39 |
| tobiash | thanks :) | 17:39 |
| *** jamesmcarthur has quit IRC | 17:47 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Test install-pdk-dependencies role https://review.opendev.org/668337 | 17:54 |
| homeski | corvus: thanks for the info | 17:55 |
| homeski | I'll play with dependencies , I think that's where my mind is heading | 17:56 |
| ofosos | Ok that works, after some testing zuul does not pick up the changes in that to | 17:59 |
| ofosos | Repo | 17:59 |
| ofosos | But a full reconfigure does it | 17:59 |
| *** igordc has joined #zuul | 18:09 | |
| openstackgerrit | Merged zuul/zuul master: Add extra-config-paths tenant config option https://review.opendev.org/668214 | 18:09 |
| *** igordc has quit IRC | 18:14 | |
| *** igordc has joined #zuul | 18:15 | |
| *** igordc has quit IRC | 18:17 | |
| *** themroc has joined #zuul | 18:19 | |
| *** gtema_ has joined #zuul | 18:21 | |
| *** jamesmcarthur has joined #zuul | 18:24 | |
| *** gtema_ has quit IRC | 18:30 | |
| corvus | ofosos: okay, so we may have some work to do in the bitbucket driver; we should make sure there is a unit test for that (change to zuul.yaml in untrusted repo merges and configuration is updated). this is the bit that's supposed to cause zuul to update its config when a change like that merges: https://opendev.org/zuul/zuul/src/branch/master/zuul/scheduler.py#L1094-L1101 | 18:40 |
| hwangbo | I'm seeing "Job <job> in <repo> is not permitted to shadow job <job> in <other_repo>" as an error, what's the solution to this? | 18:41 |
| tobiash | hwangbo: that depends on what you want to do | 18:41 |
| tobiash | the error means that you defined the a job twice in different repos | 18:42 |
| hwangbo | If we want to run the same job in multiple repos, how would that look? | 18:43 |
| openstackgerrit | Graham Hayes proposed zuul/nodepool master: Implement an Azure driver https://review.opendev.org/554432 | 18:44 |
| tobiash | hwangbo: you only need to define it once, you can just use it in a different repo | 18:45 |
| AJaeger | hwangbo: we have a global namespace - so, whatevery you define in one repo as job, is visible everywhere | 18:46 |
| *** bolg has joined #zuul | 18:46 | |
| *** bolg has quit IRC | 18:50 | |
| hwangbo | tobiash AJaeger: So instead of job.run, we would specify it as a job.parent? | 18:51 |
| hwangbo | Or would the path just be the path of the playbook in the original repo? | 18:52 |
| AJaeger | hwangbo: if it's exactly the same job, just use it ;) | 18:54 |
| AJaeger | hwangbo: see what we do with openstack-zuul-jobs... | 18:55 |
| AJaeger | hwangbo: I'm not understanding what your problem is so far | 18:55 |
| ofosos | corvus: I thought test_pull_request_with_dyn_reconf in bitbucketfunctional would make sure it is | 18:56 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Reload config on extra config files changes https://review.opendev.org/668548 | 18:58 |
| corvus | ofosos: hrm, let me go look | 18:59 |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 18:59 | |
| hwangbo | AJaeger: Sorry, I think I figured it out. Thanks for your help! | 19:05 |
| corvus | ofosos: hrm -- test_branch_updated_and_tenant_reconfigure should cover that case, and when i run it locally, it does look it it works. it might be worth looking at the scheduler logs around the time that the update to add "branches: ^.*$" merged to see what the scheduler saw and why it didn't decide to reconfigure | 19:06 |
| ofosos | corvus: I think I disabled branch polling during debugging | 19:08 |
| ofosos | Doh | 19:08 |
| corvus | ofosos: doh :) | 19:08 |
| corvus | ofosos: that's almost certainly the issue. but since i already had this prepared -- here are the relevant log lines from the test run about reconfiguration: http://paste.openstack.org/show/753731/ | 19:09 |
| corvus | ofosos: (so you know what that looks like if there are problems in the future) | 19:09 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Reload config on extra config files changes https://review.opendev.org/668548 | 20:00 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Reload config on extra config files changes https://review.opendev.org/668548 | 20:22 |
| *** themroc has quit IRC | 20:40 | |
| *** pcaruana has quit IRC | 20:41 | |
| *** jamesmcarthur has quit IRC | 20:45 | |
| *** jamesmcarthur has joined #zuul | 21:03 | |
| *** mattw4 has quit IRC | 21:05 | |
| *** mattw4 has joined #zuul | 21:06 | |
| corvus | clarkb: can you +3 https://review.opendev.org/668548 ? it's a followup to the extras change we really ought to have in place before we try it out | 21:11 |
| corvus | (but once it merges, we can restart opendev and use it with zuul-jobs) | 21:12 |
| clarkb | looking | 21:12 |
| clarkb | corvus: should https://review.opendev.org/#/c/668548/3/tests/fixtures/config/tenant-parser/git/org_project2/extra.yaml be removed since it is overridden by the test? or is that there for all the other tests that will end up looking for that config file? That must actually be why it is there too | 21:17 |
| clarkb | ya ok | 21:18 |
| clarkb | the actual file sets one job name, then the test sets another and we check the other was used | 21:18 |
| clarkb | which implies we dynamically updated the config properly. I've approved it | 21:18 |
| corvus | clarkb: yep and thx | 21:18 |
| corvus | and yeah, i had to add the run playbook since that job is actually run now (it wasn't before; some tests did check that it was defined, but not that it ran) | 21:19 |
| *** mattw4 has quit IRC | 21:25 | |
| *** mattw4 has joined #zuul | 21:25 | |
| *** mattw4 has quit IRC | 21:28 | |
| openstackgerrit | Merged zuul/zuul master: Reload config on extra config files changes https://review.opendev.org/668548 | 22:00 |
| *** mattw4 has joined #zuul | 22:03 | |
| *** jamesmcarthur has quit IRC | 22:27 | |
| *** jamesmcarthur has joined #zuul | 22:27 | |
| *** jamesmcarthur_ has joined #zuul | 22:28 | |
| *** jamesmcarthur has quit IRC | 22:32 | |
| *** rlandy has quit IRC | 22:56 | |
| *** igordc has joined #zuul | 23:03 | |
| *** jamesmcarthur_ has quit IRC | 23:05 | |
| *** igordc has quit IRC | 23:05 | |
| *** igordc has joined #zuul | 23:06 | |
| *** tosky has quit IRC | 23:08 | |
| *** andreaf has quit IRC | 23:18 | |
| *** armstrongs has joined #zuul | 23:29 | |
| *** andreaf has joined #zuul | 23:34 | |
| *** armstrongs has quit IRC | 23:39 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
