| *** rcarrillocruz has quit IRC | 00:50 | |
| *** rcarrillocruz has joined #zuul | 00:52 | |
| *** openstackgerrit has quit IRC | 01:23 | |
| *** openstackgerrit has joined #zuul | 01:34 | |
| openstackgerrit | K Jonathan Harker proposed openstack-infra/zuul feature/zuulv3: Add support for 'repos' job configuration https://review.openstack.org/436712 | 01:34 |
|---|---|---|
| jesusaur | and now with tests :) | 01:34 |
| *** saneax is now known as saneax-_-|AFK | 01:35 | |
| *** isaacb has joined #zuul | 05:09 | |
| *** isaacb has quit IRC | 05:19 | |
| *** saneax-_-|AFK is now known as saneax | 05:48 | |
| *** bhavik1 has joined #zuul | 06:20 | |
| *** bhavik1 has quit IRC | 06:23 | |
| *** isaacb has joined #zuul | 07:11 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Fix getInfoRefs call from getProjectBranches https://review.openstack.org/436801 | 07:35 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Use ssh for git-upload-pack https://review.openstack.org/436802 | 07:35 |
| *** hashar has joined #zuul | 08:07 | |
| *** abregman_ has joined #zuul | 08:24 | |
| *** abregman_ is now known as abregman | 08:47 | |
| *** rzetikx has quit IRC | 08:54 | |
| *** isaacb has quit IRC | 11:20 | |
| *** hashar has quit IRC | 12:05 | |
| *** hashar has joined #zuul | 13:27 | |
| *** yolanda has quit IRC | 13:29 | |
| *** jamielennox|away is now known as jamielennox | 14:16 | |
| SpamapS | jeblair: 'morning. I'm here today mostly so I can hack with zuul people. Let me know if/where we can create an impromptu Zuul hacking space today. :) | 14:19 |
| *** yolanda has joined #zuul | 14:26 | |
| pabelanger | o/ | 14:30 |
| pabelanger | checking zuulv3 topic for open reviews | 14:31 |
| pabelanger | currently on level 1 | 14:31 |
| jeblair | SpamapS, pabelanger: clarkb and i found a space in salon c level 3 | 14:37 |
| SpamapS | jeblair: lovely. I'll be down in a few. | 14:38 |
| pabelanger | sure, heading up | 14:38 |
| jeblair | jesusaur: neat! | 14:41 |
| jeblair | jesusaur: that change looks good; i think we still need to do the zuul-cloner style branch selection at some point (if the change under test is stable/foo, try to checkout stable/foo of all the "repos", but fall back to master if it doesn't exist) | 14:48 |
| jeblair | jesusaur: oh, i think i see something related to that which we should change in your patch... i'll leave a comment | 14:53 |
| mordred | jeblair: lib/ansible/plugins/__init__.py | 15:05 |
| rbergeron | https://github.com/willthames/ansible-review/blob/master/README.md | 15:18 |
| rbergeron | http://ndemengel.github.io/2015/01/20/ansible-modules-and-action-plugins/ | 15:31 |
| *** saneax is now known as saneax-_-|AFK | 15:43 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/nodepool master: Clarification about the rate parameter https://review.openstack.org/436993 | 15:45 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/nodepool master: Clarification about the rate parameter https://review.openstack.org/436993 | 15:46 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add hello_world test job https://review.openstack.org/436544 | 15:51 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add hello_world test job https://review.openstack.org/436544 | 15:54 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add hello_world test job https://review.openstack.org/436544 | 15:56 |
| *** bhavik1 has joined #zuul | 16:09 | |
| dmsimard | that looks super amazing ^ | 16:09 |
| bhavik1 | here, I was looking for mail notification on failure, I couldn't find way to setup "TO" mail list per repo? any help how to setupt that ? | 16:11 |
| clarkb | bhavik1: I think reporters are per pipeline and so you can't do a different one per repo unless you have a pipeline per repo? | 16:12 |
| bhavik1 | clarkb: pipeline per repo doesn't seem right way (just for mail), I'll skip mail thing in that case | 16:17 |
| dmsimard | bhavik1: just an idea -- a publisher on jobs that knows who to notice ? | 16:18 |
| dmsimard | rather than at the pipeline level | 16:18 |
| bhavik1 | dmsimard: not jobs, but project do .. mistakenly I use "repo" instead of mentioning "project" name | 16:21 |
| *** isaacb has joined #zuul | 16:23 | |
| mordred | clarkb: https://github.com/emonty/ansible/tree/fix-recursive-super | 16:49 |
| *** hashar has quit IRC | 16:52 | |
| *** abregman has quit IRC | 16:59 | |
| *** jamielennox is now known as jamielennox|away | 17:03 | |
| *** yolanda has quit IRC | 17:10 | |
| *** isaacb has quit IRC | 17:16 | |
| *** bhavik1 has quit IRC | 17:49 | |
| jesusaur | how's the ptg going? | 18:17 |
| pabelanger | Good, we have jobs running on zuulv3-dev.o.o: https://review.openstack.org/#/c/436544/ | 18:22 |
| pabelanger | but still working on a few hiccups | 18:22 |
| pabelanger | more with Ansible then zuul | 18:22 |
| jesusaur | cool | 18:25 |
| SotK | that looks exciting :D | 18:26 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 18:43 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 18:45 |
| *** isaacb has joined #zuul | 18:48 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix infinite recursion on action module import https://review.openstack.org/437066 | 18:51 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Import ansible synchronize correctly https://review.openstack.org/437067 | 18:56 |
| jeblair | jesusaur: ohai! thanks for the patch, i left some comments :) | 18:57 |
| *** jamielennox|away is now known as jamielennox | 18:57 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 19:02 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Revert "Run everything in secure mode while debugging" https://review.openstack.org/437072 | 19:02 |
| jesusaur | jeblair: ya, thanks for the comments. I'm looking through ZuulTestCase to see how to check the jobdir; it looks like self.test_root might be what I want | 19:08 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_dependent_behind_dequeue https://review.openstack.org/437076 | 19:12 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add host ssh keys to known hosts https://review.openstack.org/436634 | 19:12 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix infinite recursion on action module import https://review.openstack.org/437066 | 19:17 |
| jeblair | jesusaur: it'll be something deep under there that is per-job. you may need to hold a job while it's running and then maybe see if you can get the ansiblejob from the launcher to find the jobdir for that job... or maybe add a playbook that inspects the git directory in the jobdir (like the playbooks that set the 'flag' files in test_v3) | 19:20 |
| jesusaur | jeblair: ah, ok. I was just noticing that the test I was looking at (TestMergerRepo) is actually doing the cloning itself with a git.Repo object | 19:23 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 19:23 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix infinite recursion on action module import https://review.openstack.org/437066 | 19:32 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Import ansible synchronize correctly https://review.openstack.org/437067 | 19:34 |
| jlk | huh, I can't load gerrit | 19:35 |
| jlk | oh it finally went. | 19:36 |
| clarkb | jlk: you are at ptg ya? | 19:36 |
| jlk | no :( | 19:37 |
| clarkb | ok we thought maybe local net issues but jeblair says high load on gerrit now | 19:37 |
| jlk | nod | 19:37 |
| jlk | jeblair: 437067 patch set 2 uses a new function, did that new function get added in a different review? | 19:38 |
| *** jamielennox is now known as jamielennox|away | 19:38 | |
| jeblair | jlk: yes, its parent is 437066 | 19:39 |
| jlk | n/m I see it | 19:39 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Fix all action plugins to import safely https://review.openstack.org/437097 | 19:43 |
| jeblair | clarkb, jlk: ^ | 19:46 |
| jeblair | i've blocked my synchronize change since SpamapS included it in his | 19:46 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Fix all action plugins to import safely https://review.openstack.org/437097 | 19:47 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix infinite recursion on action module import https://review.openstack.org/437066 | 19:48 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Revert "Run everything in secure mode while debugging" https://review.openstack.org/437072 | 19:48 |
| *** jamielennox|away is now known as jamielennox | 19:48 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Fix all action plugins to import safely https://review.openstack.org/437097 | 19:49 |
| *** nibz is now known as nibalizer | 19:52 | |
| *** abregman has joined #zuul | 19:57 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 19:58 |
| *** abregman has quit IRC | 20:01 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix all action plugins to import safely https://review.openstack.org/437097 | 20:01 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:06 |
| *** hashar has joined #zuul | 20:07 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:07 |
| *** isaacb has quit IRC | 20:08 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:08 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:10 |
| *** abregman has joined #zuul | 20:11 | |
| *** abregman has quit IRC | 20:11 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:12 |
| *** yolanda has joined #zuul | 20:13 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:18 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:19 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Expose the jobdir git root as a variable https://review.openstack.org/437118 | 20:20 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:22 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Expose the jobdir git root as a variable https://review.openstack.org/437118 | 20:30 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:32 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:34 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 20:36 |
| *** jamielennox is now known as jamielennox|away | 20:39 | |
| *** jamielennox|away is now known as jamielennox | 20:44 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Expose the jobdir git root as a variable https://review.openstack.org/437118 | 20:49 |
| jesusaur | jeblair: I think I've become confused about the various copies of the git repos again. What's the purpose of the launcher git_root? | 21:05 |
| jesusaur | is that primarily for compiling the .zuul.yaml files? | 21:06 |
| jesusaur | jeblair: in your review comment you say that each job gets a new copy of their repos in a temporary jobdir, but I'm not seeing that happen in any tests | 21:09 |
| jeblair | jesusaur: the launcher has a directory where it keeps copies of all the repos it sees | 21:22 |
| jeblair | jesusaur: that's LaunchServer.merge_root | 21:22 |
| jeblair | jesusaur: and yeah, the internal merger that it has uses that to collect .zuul.yaml files | 21:23 |
| jeblair | jesusaur: then, there's an AnsibleJob for every job it runs | 21:23 |
| jeblair | jesusaur: AnisibleJob.jobdir is a temporary directory that exists only as long as the job is running | 21:24 |
| jeblair | jesusaur: AnsibleJob.jobdir.git_root is a subdirectory of that where we check out the changes being tested | 21:24 |
| jeblair | jesusaur: that's the thing we want to check on here | 21:24 |
| jesusaur | jeblair: ah, ok, thanks | 21:35 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 21:41 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 21:44 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 21:49 |
| *** jamielennox is now known as jamielennox|away | 21:52 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Add pre-run playbook to copy git repos to worker https://review.openstack.org/437059 | 21:55 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Add some lock debug lines and an exception handler https://review.openstack.org/437179 | 22:36 |
| openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Support launch failures in FakeProviderManager https://review.openstack.org/436574 | 22:39 |
| *** hashar has quit IRC | 22:42 | |
| openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Send min-ready requests only when images are ready https://review.openstack.org/436038 | 22:43 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Remove color from callback logs https://review.openstack.org/436039 | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Change 'secure' to 'trusted' https://review.openstack.org/435969 | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Consume display_args_to_stdout in zuul_stream https://review.openstack.org/435962 | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Add initial callback plugin to handle log streaming https://review.openstack.org/435961 | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Add daemon subprocess to read remote log content https://review.openstack.org/436040 | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Add placeholder for doing the actual streaming https://review.openstack.org/436014 | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Reorganize the job dir to include a work dir https://review.openstack.org/437182 | 22:53 |
| Shrews | clarkb: fyi, i -1'd 437179 after your +3. If it gets merged, a followup is fine. | 22:53 |
| clarkb | I can remoce approval | 22:54 |
| *** saneax-_-|AFK is now known as saneax | 23:18 | |
| SpamapS | mordred: I was just looking at the way the normal plugin works and I think it can be got around if any non-127 IP of the server is known by doing set_fact: hostvars['target']['ansible_ssh_hostname']=1.2.3.4 | 23:42 |
| SpamapS | hm no | 23:42 |
| SpamapS | it will still not be able to ssh | 23:42 |
| SpamapS | though we should be careful to block SSH from ourselves just in case. | 23:43 |
| SpamapS | (or maybe we can somehow freeze the variables that feed into the ssh connection plugin | 23:43 |
| SpamapS | jeblair, mordred: FYI, I added this story so we don't forget to test the ansible plugins. :) https://storyboard.openstack.org/#!/story/2000878 | 23:47 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!