| openstackgerrit | Merged openstack-infra/zuul-jobs master: Fix sphinx publishing https://review.openstack.org/530790 | 00:05 |
|---|---|---|
| *** jlk has quit IRC | 00:14 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Slack driver https://review.openstack.org/530521 | 00:39 |
| SpamapS | tobish, jhesketh: ^ I rewrote the test there to be a ZuulTestCase | 00:39 |
| SpamapS | and added some features | 00:39 |
| SpamapS | so warrants a full re-review. | 00:39 |
| jhesketh | SpamapS: cool, will take a look shortly | 00:47 |
| corvus | SpamapS: i left an admin -2 on that since there's a non-trivial configuration expansion we should carefully consider after the v3.0 release. hope that's okay. | 00:47 |
| SpamapS | corvus: totally fine. I'm using it locally but it can wait until we're ready for it. | 00:52 |
| SpamapS | Just finding SMTP inadqeuate as GoDaddy is very slack-centric. | 00:53 |
| corvus | SpamapS: ya -- no objections in principle. i'm sure we'll merge it after we hash out the per-project config stuff. | 00:54 |
| SpamapS | corvus: I'm glad there's some thought happening in that direction. I dislike mentioning projects in pipeline configs. :) | 00:55 |
| tristanC | couldn't the driver load a separate config file? for example a irc reporter could load the gerritbot configuration | 00:56 |
| mordred | there's definitely interesting things to think about in that area - I know we've had people curious in openstack about how they could add project-specific reporting | 00:57 |
| SpamapS | Happy to help on any remaining 3.0 items btw, I just don't see any babies that can be sped up being born there. | 00:57 |
| SpamapS | Maybe "finish git driver" ? | 00:57 |
| SpamapS | I thought that was done tho | 00:57 |
| corvus | yeah, i think that merged and just needs updating | 00:58 |
| *** rlandy|ruck is now known as rlandy|ruck|bbl | 01:06 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Slack driver https://review.openstack.org/530521 | 01:47 |
| SpamapS | silly unpythonic slackclient | 01:48 |
| SpamapS | None should always be "pretend I didn't pass this" in method args. :-/ | 01:48 |
| * SpamapS wonders if that is in a PEP somewhere | 01:48 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Add a plugin interface for drivers https://review.openstack.org/524620 | 01:49 |
| *** xinliang has quit IRC | 02:57 | |
| tristanC | fwiw, here is the fedora review request for nodepool feature/zuulv3: https://bugzilla.redhat.com/show_bug.cgi?id=1530431 | 03:07 |
| openstack | bugzilla.redhat.com bug 1530431 in Package Review "Review Request: nodepool - nodes management for a distributed test infrastructure" [Unspecified,New] - Assigned to nobody | 03:07 |
| *** xinliang has joined #zuul | 03:10 | |
| *** xinliang has quit IRC | 03:10 | |
| *** xinliang has joined #zuul | 03:10 | |
| *** rlandy|ruck|bbl is now known as rlandy|ruck | 03:38 | |
| *** jaianshu has joined #zuul | 03:53 | |
| *** jaianshu_ has joined #zuul | 03:56 | |
| *** jaianshu has quit IRC | 03:59 | |
| *** jaianshu__ has joined #zuul | 04:03 | |
| *** jaianshu_ has quit IRC | 04:06 | |
| *** rlandy|ruck has quit IRC | 04:29 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: Add linters job and role https://review.openstack.org/530682 | 04:49 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: python-upload-pypi: only run on success https://review.openstack.org/530174 | 05:38 |
| *** dmsimard has quit IRC | 05:42 | |
| *** dmsimard has joined #zuul | 05:43 | |
| *** hashar has joined #zuul | 08:29 | |
| *** jaianshu_ has joined #zuul | 10:14 | |
| *** jaianshu__ has quit IRC | 10:18 | |
| *** jaianshu__ has joined #zuul | 10:26 | |
| *** jaianshu_ has quit IRC | 10:30 | |
| *** sshnaidm|afk is now known as sshnaidm | 10:56 | |
| *** jkilpatr has quit IRC | 11:00 | |
| *** hashar has quit IRC | 11:05 | |
| *** jkilpatr has joined #zuul | 11:49 | |
| *** haint has quit IRC | 11:56 | |
| *** jkilpatr has quit IRC | 12:28 | |
| *** rlandy has joined #zuul | 13:00 | |
| *** rlandy is now known as rlandy|ruck|mtg | 13:00 | |
| *** jaianshu__ has quit IRC | 13:01 | |
| *** jaianshu__ has joined #zuul | 13:01 | |
| *** yolanda__ has joined #zuul | 13:25 | |
| *** jaianshu__ has quit IRC | 13:26 | |
| *** yolanda has quit IRC | 13:29 | |
| *** dkranz has joined #zuul | 13:32 | |
| *** jkilpatr has joined #zuul | 13:42 | |
| *** dkranz has quit IRC | 13:52 | |
| *** rlandy|ruck|mtg is now known as rlandy|ruck | 13:53 | |
| *** dkranz has joined #zuul | 13:53 | |
| *** sshnaidm is now known as sshnaidm|mtg | 14:42 | |
| *** rlandy|ruck is now known as rlandy|ruck|mtg | 14:56 | |
| *** JasonCL has quit IRC | 15:45 | |
| *** sshnaidm|mtg is now known as sshnaidm | 15:46 | |
| *** JasonCL has joined #zuul | 15:47 | |
| *** JasonCL has quit IRC | 15:55 | |
| *** JasonCL has joined #zuul | 16:01 | |
| *** jkilpatr has quit IRC | 16:06 | |
| *** rlandy|ruck|mtg is now known as rlandy | 16:07 | |
| *** JasonCL has quit IRC | 16:08 | |
| *** JasonCL has joined #zuul | 16:13 | |
| *** JasonCL has quit IRC | 16:19 | |
| *** openstackgerrit has quit IRC | 16:19 | |
| *** jkilpatr has joined #zuul | 16:19 | |
| *** jkilpatr has quit IRC | 16:22 | |
| *** jkilpatr has joined #zuul | 16:23 | |
| *** openstackgerrit has joined #zuul | 16:40 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Handle invalid build UUID in finger gateway https://review.openstack.org/530963 | 16:40 |
| *** JasonCL has joined #zuul | 16:45 | |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Remove name from project stanza https://review.openstack.org/530970 | 17:04 |
| corvus | i think we're running with that change now -- that excercises it ^ | 17:04 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Strip /r from build UUID in fingergw https://review.openstack.org/530973 | 17:09 |
| SpamapS | harlowja: I was going to add rate limiting to the slack driver but then I ran into https://github.com/slackapi/python-slackclient/issues/169 | 17:21 |
| SpamapS | corvus: nice | 17:21 |
| SpamapS | makes a lot of sense. | 17:21 |
| corvus | Shrews: do we have a test that exercises the finger gw? | 17:23 |
| Shrews | corvus: yes | 17:27 |
| *** dkranz has quit IRC | 17:27 | |
| corvus | Shrews: why didn't it catch 530973? | 17:29 |
| Shrews | corvus: not entirely sure. I had to run out real quick after I found that for an appointment. Can look when I get back. | 17:31 |
| clarkb | SpamapS: re None should always be pretend I didn't pass this its often used as a proxy for empty table types that you don't want a singleton of generated at compile time when dealing with the function args | 17:31 |
| clarkb | SpamapS: so None -> [] or {} or set() that then gets used | 17:31 |
| corvus | Shrews: oh, it looks like the test just sends "\n", so i guess it's "unrealistic test input" :) | 17:32 |
| corvus | Shrews: maybe we should just change that to \r\n in the test | 17:32 |
| SpamapS | clarkb: Right, I missed something in my statement. My problem wasn't that a weird or unexpected default was chosen. slackclient passes None through as a json null, and the Slack API rejects null as a value for the field, so it generates an error. | 17:42 |
| *** JasonCL has quit IRC | 17:48 | |
| SpamapS | I had a thought today. I have my 'deploy stuff' pipeline as an independent pipeline. But I wonder if it shoudl be dependent. If I land 5 things, seems like it would be better to try and run that post job with all of them together than run it 5 times. | 18:01 |
| *** jkilpatr has quit IRC | 18:03 | |
| *** jkilpatr has joined #zuul | 18:03 | |
| harlowja | SpamapS i should of fixed that | 18:10 |
| corvus | SpamapS: after 3.0 i'm going to write a new pipeline manager for post | 18:10 |
| harlowja | SpamapS at least partially i guess fixed maybe/hopefully @ https://github.com/slackapi/python-slackclient/commit/51b7fa1b748459f88622 | 18:10 |
| harlowja | maybe its not 100% fixed though, ha | 18:11 |
| SpamapS | corvus: yeah seems like you need something like dependent, but that fails all the things behind it if 1 fails. | 18:11 |
| mordred | SpamapS: the idea is to add a coalesce-like pipeline manager so that multiple triggers can only cause one instance of the job to run | 18:12 |
| SpamapS | harlowja: oh cool so you can just grab the reply from the exception. That works. | 18:12 |
| harlowja | ya, may not be 100% working, but should be easier to fix the other places (if they exist) | 18:12 |
| SpamapS | mordred: yeah that's what I need. :) I'm just dealing with my ansible running over and over for now (which hasn't happened, just theoretical) | 18:12 |
| rcarrillocruz | mordred , corvus : heya, as discussed weeks ago, pushed https://review.openstack.org/#/c/530265/ to have setup_inventory , so we can blacklist connection types not expected to have python on the other end | 18:13 |
| *** clienthax__511 has joined #zuul | 18:13 | |
| clienthax__511 | â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSvjoqzmbm: pleia2 persia leifmadsen â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â– | 18:13 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSleodhjwsm: hogepodge rlandy SotK ▄▄▄▄▄▄▄▄▄▄▄▄▄ | 18:13 |
| harlowja | niceeee | 18:13 |
| harlowja | lol | 18:13 |
| clienthax__511 | â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSyuruozgnk: jianghuaw_ mgagne openstackgerrit â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â– | 18:13 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILStzgsziped: rcarrillocruz jkilpatr zhuli ▄▄▄▄▄▄▄▄▄▄ | 18:13 |
| harlowja | lol | 18:13 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSinyvm: patrickeast cinerama` jamielennox ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSowvefe: jianghuaw_ mrhillsman persia ▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| *** yolanda__ is now known as yolanda | 18:14 | |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSjlinhlkpf: adam_g jtanner timrc ▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| leifmadsen | oh fun | 18:14 |
| rcarrillocruz | here we go again | 18:14 |
| rcarrillocruz | geez | 18:14 |
| clienthax__511 | â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSprzfasinuy: mgagne yolanda__ fungi â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â–„â– | 18:14 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSwqhoydnd: weshay patriciadomin zhuli ▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSmjqagz: harlowja leifmadsen persia ▄▄▄▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSgqkrgqzgxh: jianghuaw_ mnaser patriciadomin ▄▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| harlowja | oh man, i got invited | 18:14 |
| harlowja | lol | 18:14 |
| mnaser | sigh | 18:14 |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSwxxlp: mgagne patriciadomin timrc ▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| *** ChanServ sets mode: +o mnaser | 18:14 | |
| clienthax__511 | ▄▄▄▄▄▄▄▄▄▄ A BUSY MEETING IS GOING ON NOW IN #/JOIN ITS A JOINT MEETING WITH THE DISCUSSION OF RE-ENSLAVEMENT OF NIGGERS..MESSAGE CHRONO OR VAP0R FOR DETAILSvuvgk: fdegir SotK tristanC ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄ | 18:14 |
| *** mnaser sets mode: +b clienthax__511!*@* | 18:14 | |
| *** clienthax__511 was kicked by mnaser (clienthax__511) | 18:14 | |
| *** mnaser sets mode: -o mnaser | 18:14 | |
| SpamapS | need kline | 18:14 |
| mnaser | yeah i figure the freenode staff will get around | 18:15 |
| *** JasonCL has joined #zuul | 18:25 | |
| clarkb | pabelanger: Shrews corvus fyi, I am going to mark the nodepool issue with locked nodes on the zuulv3-issues etherpad as "fixed" because it isn't currently actionable and we are trying to distill the info there down into bugs we can actually work | 18:40 |
| *** jkilpatr has quit IRC | 18:40 | |
| *** dkranz has joined #zuul | 18:43 | |
| *** jlk has joined #zuul | 18:48 | |
| *** jlk has joined #zuul | 18:48 | |
| *** hashar has joined #zuul | 19:00 | |
| *** Bob123 has joined #zuul | 19:04 | |
| *** jhesketh has quit IRC | 19:08 | |
| *** jhesketh has joined #zuul | 19:11 | |
| *** Bob123 has left #zuul | 19:15 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Strip \r from build UUID in fingergw https://review.openstack.org/530973 | 19:15 |
| Shrews | corvus: yeah, good idea on test modification | 19:17 |
| *** NespaLa has joined #zuul | 19:17 | |
| *** NespaLa has left #zuul | 19:17 | |
| clarkb | corvus: if I start converting some of these zuulv3 issues to storyboard stories is there a tag I should use? (or set of tags depending?) | 19:17 |
| corvus | clarkb: zuulv3 i think. if it needs to be a 3.0 release blocker we can add zuulv3.0 | 19:18 |
| *** hashar has quit IRC | 19:25 | |
| Shrews | clarkb: and yes, that is fine. was fairly certain at the time that the referenced bug was the cause, just couldn't prove it then | 19:26 |
| *** hashar has joined #zuul | 19:34 | |
| openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Test that -1 works for infinite quota https://review.openstack.org/529231 | 19:45 |
| openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: requirements: remove paramiko <2.0 cap https://review.openstack.org/528719 | 19:49 |
| openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Remove name from project stanza https://review.openstack.org/530970 | 20:01 |
| clarkb | corvus: do you know what is up with the this was due to a broken git repo after a shutdown item? | 20:12 |
| corvus | clarkb: i don't have any more info than what's there. i'm not sure that's a bug. there was data corruption. the only thing zuul could do differently would be to identify it was corrupted data and report a nicer error message. if we could identify the data were corrupted, we wouldn't report an error message at all. | 20:15 |
| *** jkilpatr has joined #zuul | 20:21 | |
| *** hashar has quit IRC | 20:32 | |
| clarkb | pabelanger: heads up I'm moving https://storyboard.openstack.org/#!/story/2000809 to nodepool | 20:35 |
| pabelanger | ack | 20:37 |
| pabelanger | I'm not sure that is valid anymore, I can look shortly | 20:38 |
| SpamapS | TIL if you put an ansible-vault encrypted file in <role>/files and use copy: to try and copy it up to the destination, ansible will insist on trying to decrypt that file for you. | 20:42 |
| SpamapS | HAh, and 2.4 adds a flag to turn that off | 20:45 |
| SpamapS | Shrews: weren't you working on 2.4? | 20:45 |
| SpamapS | IIRC it broke the streaming | 20:45 |
| harlowja | don't touch my file | 20:45 |
| harlowja | lol | 20:45 |
| Shrews | SpamapS: 2.4 is cold busted. 2.4.1 is the minimum we can use | 20:45 |
| SpamapS | harlowja: YOU WANT Ansible to touch your files. You NEED Ansible to touch your files. You can't handle the decryption truth. | 20:45 |
| Shrews | err, minimum in the 2.4 genre | 20:46 |
| SpamapS | Shrews: zuul is still pinned at <2.4 | 20:46 |
| SpamapS | Wondering when we can remove that pin. | 20:46 |
| SpamapS | I've run into a few .. awww... not in 2.3 | 20:46 |
| Shrews | SpamapS: yes, but my changes to make it work have merged. there *may* be something still with the modules that mordred found last time we checked, but we should totally check again | 20:47 |
| SpamapS | Would be nice if that happened pre 3.0 release actually.. since we'll probably have to stay pinned to whatever 3.0 releases with for a long time. | 20:47 |
| Shrews | SpamapS: i can throw up a change to unpin it and see what breaks. | 20:47 |
| Shrews | been meaning to do that anyway | 20:47 |
| SpamapS | Yeah that would be cool.. if we have like, ways to run extra playbooks too.. I would guess more stuff will break. | 20:48 |
| * SpamapS will ponder this whilst enjoying a sandwich from Jersey Mike's | 20:48 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: WIP: Test Ansible 2.4 again https://review.openstack.org/531009 | 20:49 |
| Shrews | SpamapS: ^^^ | 20:49 |
| clarkb | pabelanger: on zuulv3-issues etherpad there is entry for zuul doing weird stuff after the command socket merged. Any idea if that is an ongoing problem? | 20:52 |
| mordred | SpamapS, Shrews: I'd love to get up on to 2.4 *and* have a zuul/ansible test job that runs zuul's tests with ansible devel and on changes to ansible itself - so that we've got at least an information pipeline for knowing if ansible.next is going to break us | 20:52 |
| Shrews | mordred: that's crazy talk | 20:53 |
| mordred | Shrews: I know. Im a bad person | 20:53 |
| openstackgerrit | Antoine Musso proposed openstack-infra/zuul master: Do not import paramiko before daemonization https://review.openstack.org/531012 | 20:55 |
| Shrews | oh, speaking of command socket stuff... did we want our init scripts to still shutdown the zuul processes? b/c the one we added for zuul-fingergw doesn't seem to work (have to do 'zuul-fingergw stop') | 20:55 |
| openstackgerrit | Antoine Musso proposed openstack-infra/zuul master: Do not import paramiko before daemonization https://review.openstack.org/531012 | 20:55 |
| mordred | Shrews: yah - I think we want the init scripts to call zuul-fingergw stop when they get the 'stop' argument | 20:56 |
| pabelanger | mordred: Shrews: https://review.openstack.org/526463/ | 20:57 |
| pabelanger | we can update it there | 20:57 |
| pabelanger | clarkb: looking | 20:57 |
| Shrews | pabelanger: oh ossum | 20:58 |
| mordred | corvus: ^^ that hashar patch is definitely a review/judgement call for you | 20:58 |
| pabelanger | clarkb: I am unsure what the 'weird stuff' was. I think we had mismatch of scheduler / executor running | 20:59 |
| pabelanger | clarkb: so, don't think an issue anymore | 20:59 |
| clarkb | pabelanger: ok I'm moving that to fixed | 20:59 |
| Shrews | pabelanger: you want to tackle the one for the fingergw, or do you want me to throw a patchset on top of that for you? | 21:00 |
| pabelanger | Shrews: sure, I can do it. | 21:00 |
| Shrews | or maybe even a separate review | 21:00 |
| Shrews | cool.thx | 21:00 |
| pabelanger | yah, 2nd review is fine I think | 21:00 |
| tobiash | mordred, corvus: what about uncapping paramiko instead like in nodepool? | 21:02 |
| pabelanger | should also be a review up for that | 21:02 |
| pabelanger | https://review.openstack.org/528729/ | 21:03 |
| tobiash | mordred, corvus: if I'm reading the commit message right, uncapping paramiko should s | 21:04 |
| tobiash | Also solve the issue | 21:05 |
| mordred | tobiash: yah - maybe we should suggest that and see if that works for him too | 21:05 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Handle invalid build UUID in finger gateway https://review.openstack.org/530963 | 21:05 |
| mordred | tobiash: k. I left a comment | 21:06 |
| tobiash | Oh, that patch is against master | 21:07 |
| mordred | yah. that's why I figured it was a corvus patch to look at - I don't see any harm in landing it for hashar, but we're also not running it, so it's sort of landing something blind | 21:09 |
| clarkb | jhesketh: if you can look at https://review.openstack.org/#/c/530820/ when you wake that would be much appreciated (you had reviewed older patchset) | 21:09 |
| Shrews | SpamapS: mordred: oh nice, ansible 2.4.1 works nicely for us now. https://review.openstack.org/531009 | 21:10 |
| * Shrews moves to un-WIP | 21:10 | |
| *** JasonCL has quit IRC | 21:10 | |
| Shrews | is this correct? ansible>=2.3.0.0,!=2.4.0,<2.5 | 21:13 |
| Shrews | or does it need to reference 2.4.0.0 ? | 21:13 |
| clarkb | 2.4.0 == 2.4.0.0 | 21:13 |
| clarkb | so that should work | 21:13 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Allow Ansible 2.4 https://review.openstack.org/531009 | 21:14 |
| Shrews | clarkb: thx | 21:15 |
| clarkb | Shrews: pip/setuptools/whatever will internally convert all versions to the desired format it uses for comparison and ya those two should be equivalent | 21:20 |
| *** JasonCL has joined #zuul | 21:27 | |
| *** JasonCL has quit IRC | 21:40 | |
| *** JasonCL has joined #zuul | 21:48 | |
| *** JasonCL has quit IRC | 21:53 | |
| *** JasonCL has joined #zuul | 21:54 | |
| leifmadsen | mordred: pabelanger: https://gist.github.com/leifmadsen/93b9283d10dfddba096e32fb172cf569 | 22:20 |
| leifmadsen | updated docs based on manual testing again today (thanks Paul for helping with a couple issues that popped up since last time I ran through this) | 22:20 |
| leifmadsen | next steps are going through this again, and converting this into some actual documentation | 22:21 |
| leifmadsen | where are the logs stored again for the actual job runs? | 22:24 |
| leifmadsen | oh, in /tmp :) | 22:31 |
| leifmadsen | or maybe not... anyways, dinner! | 22:32 |
| clarkb | leifmadsen: they are pat of the build dir under logs/ iirc | 22:33 |
| clarkb | leifmadsen: so wherever you've told the executor to store the builds | 22:33 |
| leifmadsen | what is the default location for that? | 22:33 |
| leifmadsen | my zuul.conf only has private_key_file and default_username under [executor] | 22:33 |
| leifmadsen | can check the code later... gotta run and eat some fajitas now :D | 22:34 |
| clarkb | leifmadsen: /var/lib/zuul/builds is where we have them and I'm guessing we use the dfeault location | 22:34 |
| corvus | that's not where they're *stored* -- it's where they are during a build | 22:38 |
| corvus | the base job post playbook is responsible for storing them after the build | 22:39 |
| *** rlandy is now known as rlandy|bbl | 22:58 | |
| leifmadsen | last time I was going through this, I did seem to have some sort of output from the run on the server so that I could at least go and debug some stuff, and that was without anything magical from the ansible playbooks itself | 23:00 |
| leifmadsen | I don't actually have a /var/lib/zuul/builds/ directory either | 23:00 |
| leifmadsen | I wonder if there was a configuration option I added to keep the data after the run or something | 23:01 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: Support cross-source dependencies https://review.openstack.org/530806 | 23:14 |
| *** sshnaidm is now known as sshnaidm|afk | 23:22 | |
| corvus | leifmadsen: starting the executor with '--keep' will do that | 23:29 |
| corvus | it's intended as a debug aid | 23:29 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!