Monday, 2018-11-19

*** rcarrillocruz has quit IRC00:30
*** rcarrillocruz has joined #zuul00:42
openstackgerritPaul Belanger proposed openstack-infra/zuul master: WIP Add support for zones in executors  https://review.openstack.org/54919700:49
*** rcarrillocruz has quit IRC01:28
*** rcarrillocruz has joined #zuul01:49
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: executor: enable zuul_return to update Ansible inventory  https://review.openstack.org/59009203:17
*** bjackman_ has joined #zuul04:39
openstackgerritIan Wienand proposed openstack-infra/nodepool master: Add Fedora 29 testing  https://review.openstack.org/61867104:45
openstackgerritIan Wienand proposed openstack-infra/nodepool master: Logs stats for nodepool automated cleanup  https://review.openstack.org/61407405:04
*** chandankumar has joined #zuul05:09
*** chandankumar is now known as chkumar|rcuk05:11
*** bjackman_ has quit IRC05:11
*** chkumar|rcuk is now known as chkumar|ruck05:11
*** bjackman has joined #zuul05:24
*** bjackman_ has joined #zuul05:25
bjackmanDumb question incoming.. when I use ">" in yaml ("folded block scalar", like "|" but newlines don't get included in the string) my editor changes it to ">-".. anyone know what that means? There are examples that use it in the YAML spec but I can't see what difference it makes from ">"05:29
bjackman(I know this isn't05:30
bjackman(I know this isn't #yaml but I'm guessing you guys know your yaml!)05:30
tristanCbjackman: iiuc, ">" and ">-" are similar, except that ">-" remove the final new line05:38
tristanCbjackman: "|"  and "|-" follow the same rule too05:40
tristanChttp://yaml.org/spec/1.2/spec.html#id279453405:41
openstackgerritneilsun proposed openstack-infra/zuul master: Add type check for zuul conf  https://review.openstack.org/59191705:50
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: executor: add support for generic build resource  https://review.openstack.org/57066807:18
*** pcaruana has joined #zuul07:24
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: quick-start: remove ssh key command  https://review.openstack.org/61870707:50
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: quick-start: remove ssh key comment  https://review.openstack.org/61870707:51
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: quick-start: remove ssh key comment and for ce rsa type  https://review.openstack.org/61870708:17
*** gtema has joined #zuul08:18
*** bjackman has quit IRC08:22
*** fdegir has quit IRC08:26
*** fdegir has joined #zuul08:26
*** arxcruz|summit is now known as arxcruz08:31
openstackgerritIan Wienand proposed openstack-infra/nodepool master: Add Fedora 29 testing  https://review.openstack.org/61867108:36
*** goern has joined #zuul08:39
*** jpena|off is now known as jpena08:48
*** bjackman has joined #zuul08:52
bjackmantristanC, OK that makes sense, cheers09:02
*** nilashishc has joined #zuul09:22
*** hashar has joined #zuul09:23
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: quick-start: split the admin ssh key  https://review.openstack.org/61870709:40
*** sshnaidm has quit IRC09:55
*** sshnaidm has joined #zuul09:57
*** toabctl has quit IRC10:33
*** panda|rover|off is now known as panda|rover10:49
*** rfolco has joined #zuul10:57
tristanCnone of my quick-start job fix attempt worked in https://review.openstack.org/#/c/618707/ ...11:31
*** gtema has quit IRC11:45
*** quiquell has joined #zuul12:05
quiquellHello12:06
quiquellI am running zuul towards an openstack cloud and open the port to be able to run "shell" ansible command from there12:07
quiquellHave a task that takes around 25 minutes (deploy Standalone) it's getting stuck there12:07
quiquellCould be that the socket get closes at inactivity ?12:08
*** jpena is now known as jpena|lunch12:31
*** dkehn has quit IRC12:33
*** chkumar|ruck has quit IRC12:57
*** rlandy has joined #zuul12:57
*** gtema has joined #zuul13:13
*** bjackman has quit IRC13:13
pabelangerquiquell: we had a issue in the past if there was no traffic after 10 seconds, we'd close the socket, but believe we addressed that13:16
quiquellpabelanger: Maybe is my home connection or my personal tenant at RDO13:20
quiquellpabelanger: could be ?13:20
quiquellpabelanger: even my cable router at home13:20
pabelangerquiquell: where is your zuul running?13:20
pabelangerI could see it possible, if the port isn't getting any traffic that a firewall some place is closing it13:21
pabelangerI don't think we do any heartbeat across that stream13:21
quiquellpabelanger: Running at my laptop at home, very homie thing13:22
pabelangerquiquell: behind NAT?13:23
pabelangerI could see that closing ports on you13:23
quiquellpabelanger: yep13:23
pabelangeryah, I think we'd need to implement keep-alive in that case13:24
quiquellpabelanger: Just openned the port at a security group at my RDO cloud and run the thing from my laptop13:25
quiquellpabelanger: Inside a docker container though13:25
*** chandankumar has joined #zuul13:25
*** chandankumar is now known as chkumar|ruck13:28
*** jpena|lunch is now known as jpena13:29
*** quiquell has quit IRC14:00
*** chkumar|ruck has quit IRC14:10
*** quiquell has joined #zuul14:46
*** gtema has quit IRC14:48
*** quiquell is now known as quiquell|lunch14:49
*** sshnaidm is now known as sshnaidm|afk15:13
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Quick-start: set content-type header on ssh key upload  https://review.openstack.org/61876615:15
corvusdue to conference plague, serious timezone maladjustment, and a holiday later in the week, i may not be terribly useful this week.15:19
*** sshnaidm|afk is now known as sshnaidm15:36
*** quiquell|lunch is now known as quiquell|off15:39
openstackgerritPaul Belanger proposed openstack-infra/zuul master: Add support for zones in executors  https://review.openstack.org/54919715:46
pabelangershould provide a test now too15:47
corvusclarkb: can you +3 https://review.openstack.org/618766 ?  it's a gate blocker due to new gerrit release16:06
corvustristanC: ^ fyi16:06
clarkbI'll take a look16:07
*** pcaruana has quit IRC16:07
openstackgerritMerged openstack-infra/zuul master: Quick-start: set content-type header on ssh key upload  https://review.openstack.org/61876616:42
*** j^2 has joined #zuul17:26
*** pcaruana has joined #zuul17:44
*** nilashishc has quit IRC18:02
*** hashar has quit IRC18:02
*** jpena is now known as jpena|off18:03
fungicorvus: you contracted the plague too? i'm normally fairly resistant, so not sure what happened this time18:23
corvusfungi: yep.  i really "enjoyed" my flight back.  united obliged by making the trip even longer.18:27
fungiyeah, courtesy of fog blanketing cdg on friday i almost missed my connection in jfk by seconds18:28
fungii had to sprint across parking lots to get between terminal 1 and terminal 418:29
corvusberlin was fun though!  i keep trying to keep that in mind :)18:29
fungiexactly18:29
clarkbmy trip back was mostly uneventful thankfully. But do feel like I've been run over by a truck. My neck has decided that if I turn it just so that it should send pain signals to my brain18:30
fungiat least it wasn't like the trip out where i sprinted from one end of dtw to the other only to miss the barring of the doors by two minutes18:30
fungimy scheme of booking 3-hour layovers has ceased to be effective. i need to bump it to 4 now i guess18:31
clarkbfungi: so much of that depends on airport I find. jfk is particularly bad for clearing customs18:34
*** goern has quit IRC18:40
fungicustoms was way quick actually18:42
fungisecurity wasn't that terrible either18:42
fungisame for border control/immigration18:42
fungiproblem was we landed an hour late and then sat on the tarmac waiting for a parking crew and gate assignment for another hour18:43
clarkbouch18:43
fungiso i had right at one hour to get through everything and connect to the far end of the airport18:43
*** j^2 has quit IRC18:43
fungiand it really came down to seconds. if i'd broken pace and stopped to get out my jacket to cope with the cool night air i'd have missed my connection18:44
fungiit was above freezing anyway. the ice storm was well past by then18:46
fungigotta say though, i'm liking what cdg has done with their international terminal. the 80s arcade theme with all the upright and tabletop cabinets is great, also tons of comfy seating18:47
corvusaroo?  neato18:48
fungiif i wasn't zombified i think i'd really have enjoyed it more18:48
fungithey had some more modern console games too. playstation 4 kiosks and whatnot18:49
fungii think it was all free to play, though i didn't check. was too out of it at the time18:50
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Cache node request zNodes  https://review.openstack.org/61880618:50
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Update node request during locking  https://review.openstack.org/61880718:50
*** goern has joined #zuul19:04
openstackgerritMerged openstack-infra/zuul-jobs master: validate-host: retry network tests and include unbound logs  https://review.openstack.org/61569819:34
openstackgerritMerged openstack-infra/zuul-jobs master: Add prepare-workspace-git role  https://review.openstack.org/61303619:40
openstackgerritMerged openstack-infra/zuul master: Handle existing .ssh dir on merger  https://review.openstack.org/61795519:43
openstackgerritMerged openstack-infra/zuul-jobs master: mirror-workspace-git: use port when pushing git repo through ssh  https://review.openstack.org/61694119:48
openstackgerritMerged openstack-infra/zuul-jobs master: Add role git-prepare-nodecache  https://review.openstack.org/61694619:57
openstackgerritTobias Henkel proposed openstack-infra/zuul-jobs master: Remove mimetype check when calculating size  https://review.openstack.org/58815020:21
*** j^2 has joined #zuul21:00
pabelangercorvus: mordred: over the weekend I was trying to debug why I couldn't see yarn files in our wheels, both on pypi and tarballs.o.o. I believe https://review.openstack.org/618634/ and depends-on is the first step to correct our branch-tarballs. Today we are not installing required dependencies for the job21:08
corvuspabelanger: i thought pypi was fixed, but mordred was looking into that, so let's get him involved.21:10
pabelangerit is possible I am just looking in the wrong place inside the wheel21:10
pabelangercorvus: there was also some discussion between clarkb and I about executor zones in nodepool. Specifically, the first iteration on https://review.openstack.org/549197/ simply uses the nodepool provider_name in the gearman function. When your back to 100% your review would be awesome.21:17
corvuspabelanger: that's great!  i hope we can merge it soon; i'm fading now so i think i'll defer review a bit21:18
pabelanger++21:20
openstackgerritMerged openstack-infra/zuul-jobs master: Remove mimetype check when calculating size  https://review.openstack.org/58815021:29
*** j^2 has quit IRC21:31
logan-looks like https://github.com/openstack-infra/zuul-jobs/commit/7e00ba32dab04ac6a3f952d84dfd9eef1e021e47 broke zuul envs where zuul_site_traceroute_host is not set https://pasted.tech/pastes/6e7c0f0e650d464b1f8e69fc3eea29f0a1906f2e21:38
clarkbianw ^ fyi21:39
clarkbprobably worth a revert?21:39
pabelangeryes please, it has broken ansible-network zuul21:43
pabelangerjust became aware of it in the last 10mins21:43
clarkbthe issue is the role isnt nooping when the host isnt set?21:43
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Revert "validate-host: retry network tests and include unbound logs"  https://review.openstack.org/61884421:43
clarkbmaybe we should split that out of validate host enitrely?21:44
logan-the module args pass in traceroute_host: "{{ zuul_site_traceroute_host|default(omit) }}"21:44
pabelangerclarkb: right21:44
logan-but in the module it is a required arg21:44
logan-so the omit condition breaks it like the paste21:44
clarkbya I'm wondering if network validation should just be a different role21:44
clarkbthen you opt into it explicitly21:45
pabelangerI'd be okay with moving that out21:45
clarkbI've approved the revert21:46
pabelangerclarkb: logan-: thanks!21:47
openstackgerritLogan V proposed openstack-infra/zuul-jobs master: Skip traceroute when no host is specified  https://review.openstack.org/61884621:47
openstackgerritLogan V proposed openstack-infra/zuul-jobs master: Skip traceroute when no host is specified  https://review.openstack.org/61884621:50
*** pcaruana has quit IRC21:51
logan-thanks clarkb21:52
openstackgerritClark Boylan proposed openstack-infra/zuul-jobs master: Revert "mirror-workspace-git: use port when pushing git repo through ssh"  https://review.openstack.org/61884922:06
clarkbif anyone is around to review ^ that would be much appreciated as well22:07
clarkbother zuul-jobs change fallout22:07
openstackgerritClark Boylan proposed openstack-infra/zuul-jobs master: mirror-workspace-git: use port when pushing git repo through ssh  https://review.openstack.org/61885922:17
*** jesusaur has quit IRC22:39
*** goern has quit IRC22:39
*** pbrobinson has quit IRC22:39
*** manjeets has quit IRC22:39
openstackgerritMerged openstack-infra/zuul-jobs master: Revert "mirror-workspace-git: use port when pushing git repo through ssh"  https://review.openstack.org/61884922:41
*** jesusaur has joined #zuul22:41
clarkbhttps://review.openstack.org/618859 was pushed as a followup to ^ and should address the issue with ipv6 addresses as hostnames in the git push there22:42
ianwpabelanger: oh sorry, just catching up on the ping here ... hrm will re-evaluate that.  i guess we need more varied test cases!22:42
logan-ianw: possible fix here https://review.openstack.org/#/c/618846/22:43
ianwlogan-: that seems right, i'm open to the idea of moving it into a completely different role -- it has just grown there "organically"22:47
ianwalthough given policies around stable behaviour, that requires notifications etc which might not be worth it22:48

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!