| *** dkehn has quit IRC | 00:46 | |
| *** dkehn has joined #zuul | 00:56 | |
| *** rlandy|bbl is now known as rlandy | 01:43 | |
| *** rlandy has quit IRC | 02:13 | |
| tristanC | is there a trick to dequeue a PullRequest from a pipeline? What I tried doesn't seems to work: https://storyboard.openstack.org/#!/story/2003747 | 02:31 |
|---|---|---|
| tobiash | tristanC: dequeue doesn't seem to work on github so close or update the pr | 03:35 |
| tristanC | tobiash: well i asked the pr owner to do that, but it would be more convenient if this can be done from zuul directly | 03:42 |
| tristanC | thanks for confirming | 03:42 |
| tobiash | tristanC: yes, dequeue should be fixed for github | 03:43 |
| openstackgerrit | Merged openstack-infra/zuul master: Autohold on timeout https://review.openstack.org/602134 | 06:26 |
| openstackgerrit | Merged openstack-infra/zuul master: Use tox-docs directly https://review.openstack.org/602131 | 06:33 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-sphinx master: Use tox-docs directly https://review.openstack.org/602252 | 07:24 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/nodepool master: Use tox-docs directly https://review.openstack.org/602253 | 07:25 |
| *** jpena|off is now known as jpena | 07:44 | |
| openstackgerrit | Benoît Bayszczak proposed openstack-infra/zuul master: replace dict.update by a dict merge in zuul_return https://review.openstack.org/602054 | 07:54 |
| openstackgerrit | Benoît Bayszczak proposed openstack-infra/zuul master: replace dict.update by a dict merge in zuul_return https://review.openstack.org/602054 | 07:54 |
| *** snapiri has quit IRC | 08:04 | |
| openstackgerrit | Benoît Bayszczak proposed openstack-infra/zuul master: replace dict.update by a dict merge in zuul_return https://review.openstack.org/602054 | 08:13 |
| *** hashar has joined #zuul | 08:18 | |
| openstackgerrit | Benoît Bayszczak proposed openstack-infra/zuul master: replace dict.update by a dict merge in zuul_return https://review.openstack.org/602054 | 08:30 |
| *** nchakrab has joined #zuul | 08:35 | |
| *** snapiri has joined #zuul | 08:41 | |
| *** jiapei has quit IRC | 08:52 | |
| *** nchakrab has quit IRC | 08:57 | |
| *** nchakrab has joined #zuul | 08:58 | |
| *** nchakrab_ has joined #zuul | 08:59 | |
| *** nchakrab has quit IRC | 09:03 | |
| *** nchakrab_ has quit IRC | 09:05 | |
| *** nchakrab has joined #zuul | 09:06 | |
| *** nchakrab has quit IRC | 09:10 | |
| *** electrofelix has joined #zuul | 09:21 | |
| *** gtema has joined #zuul | 10:00 | |
| *** hashar is now known as hasharNap | 10:41 | |
| *** xbend has joined #zuul | 10:53 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: emit-job-header: fix print node information https://review.openstack.org/602290 | 11:09 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: wip: Assume config change if file list is incomplete https://review.openstack.org/602293 | 11:10 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: emit-job-header: fix print node information https://review.openstack.org/602290 | 11:15 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: emit-job-header: fix print node information https://review.openstack.org/602290 | 11:21 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: emit-job-header: fix print node information https://review.openstack.org/602290 | 11:33 |
| *** ianychoi has quit IRC | 12:04 | |
| *** bbayszczak has joined #zuul | 12:07 | |
| *** ianychoi has joined #zuul | 12:11 | |
| *** shachar has joined #zuul | 12:19 | |
| *** shachar has quit IRC | 12:20 | |
| *** shachar has joined #zuul | 12:20 | |
| *** snapiri has quit IRC | 12:22 | |
| *** hasharNap is now known as hashar | 12:23 | |
| tristanC | pabelanger: emit-job-header doesn't seems to work anymore with network_cli node which doesn't have ansible_hostname | 12:25 |
| *** rlandy has joined #zuul | 12:35 | |
| *** gtema has quit IRC | 13:07 | |
| *** electrofelix has quit IRC | 13:11 | |
| pabelanger | right, nodes with facts missing will fail | 13:16 |
| pabelanger | we can likey default to none if missing | 13:17 |
| *** bbayszczak has quit IRC | 13:23 | |
| *** samccann has joined #zuul | 13:26 | |
| rcarrillocruz | yup, cos otherwise it would get too snowflakey | 13:36 |
| rcarrillocruz | meaning | 13:36 |
| rcarrillocruz | right now, if you want to get facts for a network device in ansible | 13:36 |
| rcarrillocruz | you have <platform>_facts | 13:36 |
| rcarrillocruz | i.e., gather_facts doesn't invoke those | 13:36 |
| pabelanger | rcarrillocruz: can you link the docs for those? I haven't used them yet | 13:37 |
| *** bbayszczak has joined #zuul | 13:37 | |
| rcarrillocruz | for example https://docs.ansible.com/ansible/2.6/modules/ios_facts_module.html | 13:38 |
| rcarrillocruz | iirc, bcoca had a proposal 'somewhere' can't remember where or which that would allow at some point to hint gather_facts which module to use | 13:38 |
| rcarrillocruz | i.e. work not just in linux/windows | 13:38 |
| dmsimard | rcarrillocruz: I remember that discussion | 13:38 |
| dmsimard | rcarrillocruz: but I don't remember the conclusion | 13:39 |
| pabelanger | okay cool, so we could write a pre-run playbooks, that does a conditional look up then do platform_facts | 13:39 |
| bbayszczak | pabelanger: Hi, now with test :) https://review.openstack.org/#/c/602054/ | 13:39 |
| pabelanger | bbayszczak: thanks! I'll look today | 13:40 |
| bbayszczak | pabelanger: thanks ! | 13:41 |
| dmsimard | rcarrillocruz, pabelanger: ah, it wasn't exactly the same thing but SpamapS had a sort of related issue with how zuul gathered facts before handing the node over https://github.com/ansible/ansible/issues/19605#issuecomment-417160811 | 13:42 |
| dmsimard | The issue was that ansible would not try /usr/bin/python3 if /usr/bin/python was missing | 13:43 |
| rcarrillocruz | https://github.com/ansible/proposals/issues/17 | 13:45 |
| rcarrillocruz | dmsimard , pabelanger ^ | 13:45 |
| rcarrillocruz | found it | 13:45 |
| rcarrillocruz | so inh theory, we could put as a hostvars in nodepool | 13:45 |
| rcarrillocruz | the action for ios | 13:45 |
| rcarrillocruz | ios_facts | 13:45 |
| rcarrillocruz | problem solved | 13:45 |
| * rcarrillocruz being handwavy | 13:45 | |
| pabelanger | https://github.com/ansible/ansible/pull/31783 | 13:46 |
| pabelanger | code has merged | 13:46 |
| pabelanger | okay, I'll look into that today | 13:46 |
| rcarrillocruz | bear in mind thatt's in devel | 13:47 |
| rcarrillocruz | check it out the branches, if it's been backported, etc | 13:47 |
| pabelanger | right | 13:48 |
| dmsimard | pabelanger: it's also been reverted | 13:48 |
| pabelanger | zuul is also using 2.5 still, so we'd have to bump that too | 13:49 |
| dmsimard | doesn't say why though https://github.com/ryansb/ansible/commit/41129ce8fdb237bf0b53e91848bc9536cb770a84 | 13:49 |
| *** electrofelix has joined #zuul | 13:49 | |
| pabelanger | the best kinda reverts | 13:50 |
| pabelanger | https://github.com/ansible/ansible/pull/40022 | 13:51 |
| *** shachar is now known as snapiri | 13:58 | |
| openstackgerrit | Merged openstack/pbrx master: Correct small typo https://review.openstack.org/602151 | 14:05 |
| *** annabelleB has joined #zuul | 14:11 | |
| *** bbayszczak has quit IRC | 14:11 | |
| *** annabelleB has quit IRC | 14:15 | |
| *** annabelleB has joined #zuul | 14:18 | |
| *** pcaruana has joined #zuul | 14:28 | |
| *** jimi|ansible has quit IRC | 14:39 | |
| *** annabelleB has quit IRC | 14:52 | |
| *** annabelleB has joined #zuul | 15:01 | |
| *** jiapei has joined #zuul | 15:04 | |
| *** sshnaidm|ptg has quit IRC | 15:14 | |
| *** sshnaidm|ptg has joined #zuul | 15:14 | |
| *** yolanda has joined #zuul | 15:55 | |
| SpamapS | FYI, the problem with pbrx that the revert solves seems to be that bindep doesn't work | 16:06 |
| dkehn | clarkb: https://pastebin.com/QLkLUE6j | 16:07 |
| SpamapS | Looks like the problem is the lack of 'apk update' from that patch | 16:11 |
| clarkb | dkehn: I don't think zuulv2 + github was ever supported. I'm wasn't familiar with how that worked then and definitely am not now | 16:11 |
| dkehn | clarkb: great, thanks anyways, its obviously comparing two unlike things, trying to get the ZuulV3 stood up as fast as possible, but in the meantime… | 16:14 |
| pabelanger | oh, I see https://review.openstack.org/602172/ reverted some apk update things | 16:14 |
| pabelanger | SpamapS: so, ^ doesn't help? | 16:14 |
| SpamapS | the revert fixes the build | 16:15 |
| SpamapS | because it puts back the `apk update` | 16:15 |
| SpamapS | We may also need to fix bindep | 16:15 |
| SpamapS | it should probably raise an error if the apk database is empty | 16:15 |
| SpamapS | rather than printing *nothing* | 16:15 |
| SpamapS | http://paste.openstack.org/show/730041/ | 16:17 |
| pabelanger | SpamapS: okay, I'll have to read up again on apk update, because if we do that, we'll bake indexes into a layer of the image I think | 16:18 |
| pabelanger | not a huge deal, honestly | 16:19 |
| SpamapS | pabelanger: I don't think it's possible to avoid | 16:28 |
| SpamapS | you can't apk add without it | 16:28 |
| SpamapS | I'm looking at why bindep didn't complain | 16:29 |
| SpamapS | oh wow yeah | 16:35 |
| SpamapS | the apk driver is.... optimistic | 16:35 |
| clarkb | SpamapS: I think bindep is optimistic | 16:45 |
| clarkb | it turns out that distros don't even identify themselves consistently across the same distro | 16:45 |
| SpamapS | the problem is that you get WARNING in the output | 16:54 |
| SpamapS | and it just assumes it if has 4 words split by whitespace, then the 4th one is the version. | 16:54 |
| dmsimard | clarkb: the code in ansible that determines the ansible_os_family and ansible_os_distribution facts is not very pretty | 17:04 |
| dmsimard | it works but it's not very pretty | 17:04 |
| clarkb | SpamapS: neat | 17:06 |
| *** jpena is now known as jpena|off | 17:07 | |
| *** hashar is now known as hasharDinner | 17:19 | |
| SpamapS | anyway it looks like there's simply no way to use bindep without an `apk update` | 17:41 |
| SpamapS | actually no | 17:41 |
| SpamapS | I'm wrong | 17:41 |
| SpamapS | if I filter the WARNING's out then the version is None instead of 'random-word-from-warnings' and then we'll print out the missing packages we need | 17:42 |
| *** electrofelix has quit IRC | 18:13 | |
| *** sshnaidm|ptg has quit IRC | 18:27 | |
| *** yolanda has quit IRC | 18:30 | |
| *** annabelleB has quit IRC | 18:55 | |
| *** hasharDinner is now known as hashar | 18:56 | |
| *** annabelleB has joined #zuul | 19:10 | |
| *** annabelleB has quit IRC | 19:11 | |
| *** jimi|ansible has joined #zuul | 19:38 | |
| *** jimi|ansible has quit IRC | 19:38 | |
| *** jimi|ansible has joined #zuul | 19:38 | |
| *** annabelleB has joined #zuul | 20:00 | |
| *** yolanda has joined #zuul | 20:05 | |
| *** sshnaidm|ptg has joined #zuul | 20:25 | |
| *** myoung has quit IRC | 20:31 | |
| *** pcaruana has quit IRC | 20:38 | |
| *** samccann has quit IRC | 20:47 | |
| *** hashar has quit IRC | 20:55 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Doc: executor operations document pause, remove graceful https://review.openstack.org/602455 | 21:08 |
| *** annabelleB has quit IRC | 21:22 | |
| *** annabelleB has joined #zuul | 21:22 | |
| *** annabelleB has quit IRC | 21:34 | |
| *** rlandy is now known as rlandy|brb | 21:34 | |
| *** yolanda has quit IRC | 21:38 | |
| *** annabelleB has joined #zuul | 21:39 | |
| *** yolanda has joined #zuul | 21:39 | |
| *** annabelleB has quit IRC | 22:07 | |
| *** annabelleB has joined #zuul | 22:10 | |
| *** rlandy|brb is now known as rlandy | 22:11 | |
| *** annabelleB has quit IRC | 22:49 | |
| *** annabelleB has joined #zuul | 22:52 | |
| *** annabelleB has quit IRC | 23:08 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Indent add-fileserver docs more https://review.openstack.org/602496 | 23:13 |
| corvus | i think the new version of sphinx may be adding an extra title undor our logo | 23:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Update docs theme logo https://review.openstack.org/602500 | 23:24 |
| *** annabelleB has joined #zuul | 23:25 | |
| openstackgerrit | Merged openstack-infra/zuul master: Correct Last-Modified header for status API https://review.openstack.org/601832 | 23:27 |
| *** sshnaidm|ptg has quit IRC | 23:28 | |
| *** sshnaidm|ptg has joined #zuul | 23:32 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Update docs theme logo https://review.openstack.org/602500 | 23:49 |
| *** annabelleB has quit IRC | 23:49 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Update docs theme logo https://review.openstack.org/602500 | 23:57 |
| *** annabelleB has joined #zuul | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!