| pabelanger | fear maybe | 00:00 |
|---|---|---|
| *** JasonCL has joined #zuul | 00:06 | |
| *** JasonCL has quit IRC | 00:13 | |
| *** JasonCL has joined #zuul | 00:33 | |
| *** JasonCL has quit IRC | 01:06 | |
| *** dmsimard is now known as dmsimard|off | 01:06 | |
| dmsimard|off | Apparently this is a thing: https://wiki.jenkins.io/display/JENKINS/NodePool+Agents+Plugin | 01:06 |
| *** JasonCL has joined #zuul | 01:08 | |
| pabelanger | dmsimard|off: that is out of rackspace, IIRC | 01:09 |
| pabelanger | https://github.com/jenkinsci/nodepool-agents-plugin | 01:09 |
| *** JasonCL_ has joined #zuul | 01:11 | |
| *** JasonCL has quit IRC | 01:12 | |
| *** harlowja has quit IRC | 01:16 | |
| *** JasonCL_ has quit IRC | 01:16 | |
| *** JasonCL has joined #zuul | 01:17 | |
| *** JasonCL has quit IRC | 01:25 | |
| *** JasonCL has joined #zuul | 01:25 | |
| *** spsurya has joined #zuul | 01:54 | |
| *** rlandy|bbl is now known as rlandy | 01:55 | |
| *** rlandy has quit IRC | 01:56 | |
| *** JasonCL has quit IRC | 02:35 | |
| *** harlowja has joined #zuul | 03:43 | |
| *** Wei_Liu has quit IRC | 03:51 | |
| *** harlowja has quit IRC | 04:12 | |
| *** pwhalen_ has joined #zuul | 04:28 | |
| *** pwhalen has quit IRC | 04:29 | |
| *** hashar has joined #zuul | 06:01 | |
| tobiash | tristanC, corvus: I have a use case with static nodes which I think we don't support yet | 06:10 |
| tobiash | our customers can provide static nodes (the config is owned by the operations team) | 06:11 |
| tobiash | and they need to be able to take out nodes for maintenance without interacting with the operations team | 06:11 |
| tobiash | I think currently this will trigger failed builds as nodepool checks for connectivity only at reconfigurations | 06:12 |
| tristanC | tobiash: well the first versions of the driver did the ssh validation in the handler... | 06:14 |
| tobiash | tristanC: do you know the reason for removing the ssh validation from the handler? | 06:15 |
| tobiash | was it performance? | 06:15 |
| tobiash | maybe we have to re-add (optional) ssh validation into the handler (with fallback to the next valid host if validation fails) | 06:16 |
| tristanC | tobiash: not sure how this was decided, all i can found is those reviews: https://review.openstack.org/#/c/468624/7/nodepool/config.py and https://review.openstack.org/#/c/468624/7/nodepool/driver/static/provider.py | 06:20 |
| tristanC | tobiash: oh nevermind, the logic was only used at reconfigurations | 06:23 |
| tristanC | so yes, an option to validate ssh connection before setting a static node ready would be useful | 06:24 |
| *** elyezer has quit IRC | 06:24 | |
| *** elyezer has joined #zuul | 06:37 | |
| *** mordred has quit IRC | 06:45 | |
| *** mordred has joined #zuul | 06:48 | |
| *** zigo has quit IRC | 07:11 | |
| *** zigo has joined #zuul | 07:14 | |
| *** jpena|off is now known as jpena | 07:47 | |
| *** mugsie has quit IRC | 08:09 | |
| *** electrofelix has joined #zuul | 08:11 | |
| *** mugsie has joined #zuul | 08:21 | |
| *** mugsie has quit IRC | 08:21 | |
| *** mugsie has joined #zuul | 08:21 | |
| *** ssbarnea_ has joined #zuul | 08:27 | |
| *** aluria has quit IRC | 10:21 | |
| *** aluria has joined #zuul | 10:25 | |
| *** aluria has quit IRC | 10:27 | |
| *** aluria has joined #zuul | 10:27 | |
| *** jpena is now known as jpena|lunch | 12:00 | |
| *** rlandy has joined #zuul | 12:30 | |
| *** ssbarnea_ has quit IRC | 13:00 | |
| *** jpena|lunch is now known as jpena | 13:01 | |
| *** pwhalen_ is now known as pwhalen | 13:03 | |
| *** pwhalen has quit IRC | 13:03 | |
| *** pwhalen has joined #zuul | 13:03 | |
| *** ssbarnea_ has joined #zuul | 13:05 | |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Localize build time in build overview https://review.openstack.org/562273 | 13:19 |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Localize build time in build overview https://review.openstack.org/562273 | 13:23 |
| *** JasonCL has joined #zuul | 13:31 | |
| *** dkranz has joined #zuul | 13:39 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Update test instructions https://review.openstack.org/562289 | 14:12 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Update test instructions https://review.openstack.org/562289 | 14:13 |
| *** elyezer has quit IRC | 15:04 | |
| *** elyezer has joined #zuul | 15:12 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Don't store references to secret objects from jobs https://review.openstack.org/553596 | 15:31 |
| openstackgerrit | Merged openstack-infra/nodepool master: Refactor run_handler to be generic https://review.openstack.org/535554 | 15:38 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: Proposed spec: tenant-scoped admin web API https://review.openstack.org/562321 | 15:46 |
| Shrews | https://techcrunch.com/2018/04/17/mirantis-bets-on-spinnaker-netflixs-open-source-continuous-delivery-platform/ | 15:57 |
| openstackgerrit | Merged openstack-infra/zuul master: Make gearman queries asynchronous in github driver https://review.openstack.org/560044 | 15:57 |
| corvus | tristanC: would you mind trying to avoid rebasing when you update changes? i'm trying to diff between patchsets of the mqtt driver change, and it's difficult because there's so much other stuff going on in the rebases | 16:00 |
| corvus | tristanC: (or if you do have to rebase, can you do it in a separate patchset? ie, make changes first, then rebase) | 16:02 |
| *** harlowja has joined #zuul | 16:06 | |
| clarkb | corvus: question for you on the secrets freezing change when you return | 16:10 |
| *** acozine1 has joined #zuul | 16:10 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: mqtt: add basic reporter https://review.openstack.org/535543 | 16:11 |
| corvus | clarkb: thx; i'll dig into that when i get back. | 16:14 |
| tobiash | clarkb: responded on the secrets question | 17:01 |
| clarkb | tobiash: thanks | 17:02 |
| openstackgerrit | Merged openstack-infra/nodepool master: Support winrm hosts in static driver https://review.openstack.org/561256 | 17:10 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Switch to http://security.debian.org/ for debian https://review.openstack.org/562349 | 17:11 |
| *** jpena is now known as jpena|off | 17:12 | |
| *** electrofelix has quit IRC | 17:33 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Switch to http://security.debian.org/ for debian https://review.openstack.org/562349 | 17:38 |
| *** harlowja has quit IRC | 17:44 | |
| *** harlowja has joined #zuul | 18:18 | |
| *** harlowja_ has joined #zuul | 18:23 | |
| *** harlowja has quit IRC | 18:24 | |
| *** pwhalen has quit IRC | 18:48 | |
| *** pwhalen has joined #zuul | 18:58 | |
| *** pwhalen has joined #zuul | 18:58 | |
| *** ssbarnea_ has quit IRC | 19:10 | |
| *** acozine1 has quit IRC | 19:14 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Perform late validation of secrets https://review.openstack.org/553041 | 19:32 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Perform late validation of nodesets https://review.openstack.org/553088 | 19:32 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: late bind pipelines https://review.openstack.org/553618 | 19:32 |
| corvus | rebases, since parents are passing tests now | 19:32 |
| corvus | tobiash, clarkb, fungi: when you have a sec, do you mind going ahead and reviewing https://review.openstack.org/561750 and its parents? that's everything except the 2 changes which add the video | 19:33 |
| fungi | sure, will take a peek after the sb meeting wraps up | 19:37 |
| *** ssbarnea_ has joined #zuul | 19:47 | |
| pabelanger | does shadowing of projects work across connections? | 20:00 |
| pabelanger | I would think so | 20:01 |
| corvus | pabelanger: should; you may need to specify the canonical name | 20:01 |
| corvus | ie: git.zuul-ci.org/zuul-base-jobs | 20:01 |
| pabelanger | k, let me try that | 20:01 |
| corvus | (but probably only if there's another zuul-base-jobs in the system) | 20:02 |
| openstackgerrit | Merged openstack-infra/zuul-website-media master: Add license and zuul.yaml https://review.openstack.org/561747 | 20:12 |
| pabelanger | corvus: okay, figured it out. miss match configuration for main.yaml between scheduler and merger. We can reload scheduler today, but have to stop / start merger to pickup main.yaml changes. | 20:15 |
| corvus | pabelanger: what does a merger use main.yaml for? | 20:16 |
| corvus | pabelanger: do you mean zuul.conf? | 20:16 |
| pabelanger | not sure, let me see if that statement is correct. as I had to reload both | 20:16 |
| pabelanger | this was main.yaml, let me confirm | 20:16 |
| corvus | pabelanger: okay. nothing but the scheduler should use main.yaml. what you said would be correct for zuul.conf though. | 20:17 |
| openstackgerrit | Merged openstack-infra/zuul-website master: Merge zuul-website-media when publishing site https://review.openstack.org/561749 | 20:18 |
| pabelanger | corvus: okay, that is what I initially though. Working on reproducing issue | 20:19 |
| *** JasonCL has quit IRC | 20:30 | |
| pabelanger | corvus: ah, okay. So you are right, zuul-merger doesn't need to be restarted, but it does appear to be an order issue in main.yaml. Which was part of my process before stating the zuul-merger reload is needed above. | 20:31 |
| pabelanger | http://paste.openstack.org/show/719515/ | 20:31 |
| pabelanger | shows the issue | 20:31 |
| corvus | pabelanger: can you show me the error? | 20:36 |
| *** ssbarnea_ has quit IRC | 20:37 | |
| pabelanger | corvus: sure: http://paste.openstack.org/show/719516/ | 20:37 |
| pabelanger | I need to AFK for a few minutes, will return | 20:37 |
| *** ssbarnea_ has joined #zuul | 20:38 | |
| corvus | pabelanger: that shouldn't matter either. dictionaries are unordered. | 20:39 |
| *** dmellado has quit IRC | 20:45 | |
| *** gouthamr has quit IRC | 20:46 | |
| *** JasonCL has joined #zuul | 20:51 | |
| pabelanger | not sure, I can try to debug why it fails later. But seems to be working now. I plan on wiping this setup later today and trying from scratch | 20:52 |
| corvus | pabelanger: k. would be good to get to the bottom of the issue if there is one. | 20:59 |
| pabelanger | corvus: yah, I'll start tracking it into an etherpad. I want to give a report after I get this 3pci working and get some feedback | 21:01 |
| *** dkranz has quit IRC | 21:03 | |
| *** eandersson has quit IRC | 21:10 | |
| *** eandersson has joined #zuul | 21:13 | |
| *** hashar has quit IRC | 21:32 | |
| corvus | zuul.configloader.ConfigurationSyntaxError: Zuul encountered a syntax error while parsing its configuration in the | 22:14 |
| corvus | repo common-config on branch master. The error was: | 22:14 |
| corvus | 2 | 22:14 |
| corvus | i got a chuckle out of that. i clearly have a ways to go. :) | 22:15 |
| *** rlandy has quit IRC | 22:18 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: late bind pipelines https://review.openstack.org/553618 | 22:25 |
| corvus | that now passes at least one test | 22:25 |
| corvus | i think it's going to make thinking about projects/templates/branches a lot simpler | 22:25 |
| *** yolanda has quit IRC | 22:32 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: late bind pipelines https://review.openstack.org/553618 | 22:46 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: mqtt: add basic reporter https://review.openstack.org/535543 | 22:47 |
| *** andreaf has quit IRC | 23:18 | |
| *** andreaf has joined #zuul | 23:18 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: late bind pipelines https://review.openstack.org/553618 | 23:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: late bind pipelines https://review.openstack.org/553618 | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!