mordred | jeblair: I've never waited myself :) | 00:01 |
---|---|---|
mordred | jeblair: I will now approve those patches | 00:01 |
jeblair | okiedokie | 00:01 |
anteaya | zaro: ah okay thank you | 00:02 |
anteaya | zaro: are you willing to create a patchset to put the its-storyboard plugin on review-dev yet? | 00:02 |
*** sarob has quit IRC | 00:02 | |
*** ddieterly has joined #openstack-infra | 00:05 | |
*** SumitNaiksatam has quit IRC | 00:05 | |
*** piet has quit IRC | 00:06 | |
zaro | anteaya: working on it | 00:06 |
JayF | Are apache logs stored for the gate anywhere, and I'm just missing it/ | 00:07 |
anteaya | zaro: oh lovely, thank you | 00:07 |
ianw | JayF: yes, they should be copied out | 00:10 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul: Enabled host_key_checking https://review.openstack.org/330827 | 00:10 |
pabelanger | Shrews: jeblair: mordred: I was thinking something like ^ for host_key_checking | 00:10 |
pabelanger | untested | 00:10 |
JayF | ianw: http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/ can you show me where? I've checked damn near everything :/ | 00:10 |
pabelanger | well, I tested it locally in a playbook, untested in zuul-launcher | 00:11 |
JayF | ianw: looks like apache *configs* are there but not logs | 00:11 |
ianw | JayF: ok ... for standard devstack-gate cleanup they *should* be there, but something might be going wrong ... looking | 00:12 |
JayF | ianw: honestly these failures look like some other service ninja'd use of port 8088 out from under ironic, but I can't tell with the gate logs provided :/ | 00:12 |
JayF | ianw: http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/logs/devstacklog.txt.gz#_2016-06-16_23_14_51_073 | 00:12 |
jeblair | JayF: oh, yep, we do that. | 00:12 |
*** sflanigan has joined #openstack-infra | 00:12 | |
ianw | JayF: port 8088 ... streaming logs via netcat with zuul ... ummm | 00:12 |
JayF | oh shit | 00:12 |
JayF | That's what's been breaking the ironic gate | 00:12 |
anteaya | please | 00:12 |
JayF | Sorry anteaya :( | 00:12 |
JayF | need to watch my language | 00:12 |
jeblair | JayF: we can move it fairly easily | 00:13 |
fungi | we could probably pick a more esoteric port | 00:13 |
jeblair | (what we can't do is predict what port a job is going to use) | 00:13 |
JayF | Either that or Ironic will have to change the port we configure in Ironic | 00:13 |
jeblair | pick a number | 00:13 |
JayF | 65534 | 00:13 |
JayF | :D | 00:13 |
fungi | heh | 00:13 |
fungi | wfm (and unlikely to have an ephemeral binding that early in the job) | 00:14 |
pabelanger | obligatory: 23 | 00:14 |
JayF | So you all are going to change the port you're using? It's just as simple for us to change the port as well | 00:14 |
mordred | jeblair: what about some of the really esoteric things below 80 | 00:14 |
fungi | pabelanger: spot on | 00:14 |
jeblair | i'd like to avoid below 1024 | 00:14 |
mordred | jeblair: but echo is 7 :) | 00:14 |
JayF | https://bugs.launchpad.net/ironic/+bug/1590139 is the Ironic bug related in question | 00:14 |
openstack | Launchpad bug 1590139 in Ironic "gate job fails when unable to start apache service during horizon setup" [Critical,Triaged] | 00:14 |
JayF | I'm going to put this information there | 00:14 |
fungi | though this isn't really a telnet server | 00:14 |
jeblair | someone who is not me pick a number :) | 00:14 |
fungi | just a streaming socket | 00:15 |
mordred | zomgzom | 00:15 |
mordred | jeblair: netrjs-1 71/tcp # Remote Job Service | 00:15 |
fungi | avoiding below 1024 | 00:15 |
JayF | 19885 | 00:15 |
fungi | 65535 sounded fine | 00:15 |
JayF | 1 + keypad numbers for 'zuul' | 00:15 |
fungi | er, 65534 | 00:15 |
JayF | fungi: it being in the source port range is probably nonideal, but unlikely to actually matter /famous_last_words | 00:16 |
fungi | 31337 | 00:16 |
*** sarob has joined #openstack-infra | 00:16 | |
fungi | ;) | 00:16 |
pabelanger | epic | 00:16 |
jeblair | but less esoteric | 00:17 |
*** roxanaghe has joined #openstack-infra | 00:17 | |
*** amotoki has quit IRC | 00:17 | |
fungi | might trip turn up false positives for rootkit scanners | 00:17 |
pabelanger | What was the issue with 8088? Breaking some jobs? | 00:17 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: Use port 19885 for console streaming https://review.openstack.org/330829 | 00:18 |
jeblair | JayF, fungi, mordred, pabelanger: ^ | 00:18 |
JayF | closes-bug 1590139 if you care | 00:18 |
openstack | bug 1590139 in Ironic "gate job fails when unable to start apache service during horizon setup" [Critical,Triaged] https://launchpad.net/bugs/1590139 | 00:18 |
JayF | I'll paste that review into the bug myself though | 00:18 |
clarkb | tests shoukd use port 0 imo | 00:18 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: Use port 19885 for console streaming https://review.openstack.org/330829 | 00:18 |
jeblair | clarkb: it's pretty hard to use port 0 for something and then go and configure apache to forward to it. possible, but it's a lot of work. | 00:19 |
ianw | JayF: hmm, so i can see around http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/logs/devstack-gate-cleanup-host.txt 2016-06-16 23:15:31.974 the apache logs are being copied, but don't end up in the final log output as you say | 00:19 |
clarkb | jeblair: ironic is fronting something with apache? | 00:19 |
jeblair | (i mean, you have to start your service, then configure proxy servers, which is backwards from the way people usually do something) | 00:20 |
JayF | We run apache to host our static ipxe rom | 00:20 |
JayF | for ipxe jobs | 00:20 |
clarkb | huh ehy nit port 80 forbthat | 00:20 |
JayF | Apache static serving is a dependency for Ironic | 00:20 |
JayF | I don't know, this well predates me | 00:20 |
JayF | I'm just troubleshooting :) | 00:20 |
clarkb | ugh phone typing. why not port 80 | 00:20 |
jeblair | JayF: "Huh, why not port 80 for that" | 00:20 |
* jeblair translates for clarkb | 00:20 | |
clarkb | jeblair: can read phone typing | 00:20 |
mordred | or 4011 which is the pxe port | 00:21 |
*** roxanaghe has quit IRC | 00:21 | |
ianw | JayF: and ansible seems to report rsyncing them ... http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/_zuul_ansible/logs/ansible.log | 00:21 |
jeblair | JayF: i wonder if ironic and i picked 8088 for the same reason (easy to type and remember intel nostalgia) | 00:22 |
pabelanger | review 330784 could use some love, removes jenkins from grafana | 00:22 |
*** amitgandhinz has joined #openstack-infra | 00:24 | |
*** ddieterly has quit IRC | 00:25 | |
JayF | jeblair: mordred: seriously no idea why; if 4011 is the standard we can use that. I think there was a reason we didn't want to host it on :80, but that was a long time ago before I worked upstream this much :) | 00:27 |
jeblair | i think what i would like to do to speed things up is to force-merge 330829, stop zuul, hard-stop all the launchers then restart | 00:27 |
jeblair | that will cleanup all the nodepool leaks at the same time as we switch ports | 00:28 |
*** pvaneck has quit IRC | 00:28 | |
JayF | if you don't mind putting a comment in the bug when it's all fixed up, I can be the recheck-fairy in the morning :) | 00:28 |
pabelanger | jeblair: want me to get up the system-config change for firewalls? | 00:28 |
*** zul has joined #openstack-infra | 00:28 | |
*** ddieterly has joined #openstack-infra | 00:28 | |
jeblair | pabelanger: oh, i forgot about those | 00:28 |
jeblair | pabelanger: that's an image update isn't it | 00:28 |
pabelanger | right | 00:28 |
*** amitgandhinz has quit IRC | 00:29 | |
jeblair | JayF: erm, it may actually be faster for you to change your port | 00:29 |
JayF | jeblair: are we measuring in hours? days? weeks? | 00:29 |
jeblair | JayF: i think infra may be multiple days. | 00:29 |
pabelanger | if things go well, 11+hours to update images | 00:29 |
jeblair | they never go well | 00:29 |
pabelanger | right | 00:29 |
jeblair | plan for a week | 00:29 |
JayF | lol | 00:29 |
JayF | works, I'll push up the ironic patch too | 00:30 |
pabelanger | actually, a week :) | 00:30 |
clarkb | oh right firewall updates | 00:30 |
JayF | 4011 is the official proper port for serving up pxe images? | 00:30 |
*** Sukhdev has quit IRC | 00:30 | |
clarkb | looks like it is for proxy dhcp | 00:31 |
jeblair | pabelanger: let's go ahead and put both ports in the iptables config so we can still change the port. just slowly. | 00:31 |
pabelanger | jeblair: ack | 00:31 |
*** isaacb has quit IRC | 00:32 | |
JayF | netboot-pxe 3928/tcp # PXE NetBoot Manager | 00:32 |
JayF | going for that. | 00:33 |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Use port 19885 for console streaming for zuul workers https://review.openstack.org/330831 | 00:34 |
*** lvdongbing has joined #openstack-infra | 00:38 | |
JayF | the Ironic side fix for that port overlap: Jay Faulkner proposed openstack/ironic: Change port used for Ironic static http to 3928 https://review.openstack.org/330832 | 00:39 |
pabelanger | An interesting argument could be made that zuul-launcher could check to see if the configured port for console streaming is open, and either give a warning or actually open it. | 00:39 |
jeblair | pabelanger: it opens it before the job starts | 00:40 |
pabelanger | jeblair: sorry, meaning the firewall port too | 00:40 |
jeblair | pabelanger: ah, that's pretty site-specific | 00:40 |
*** bpokorny has quit IRC | 00:41 | |
pabelanger | ya, I figure we're going to walk the line of bootstrapping the worker with DIB or ansible moving forward. | 00:41 |
*** ddieterly has quit IRC | 00:42 | |
*** lvdongbing has quit IRC | 00:43 | |
ianw | jeblair: ok, maybe i'm going bananas here. here's the rsync copy for the logs cleaned up -> http://paste.openstack.org/show/516752/ | 00:43 |
ianw | jeblair: the resulting log dir -> http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/logs/ | 00:44 |
*** lvdongbing has joined #openstack-infra | 00:44 | |
ianw | jeblair: what happened to "apache/*" ? | 00:44 |
jeblair | ianw: what job? | 00:44 |
jeblair | ianw: oh you just linked it :) | 00:44 |
JayF | ianw: I suspect it's also possible that apache never starts at all | 00:44 |
JayF | ianw: there probably should be error logs, but given the port conflicts wouldn't surprise me if it's just 100% empty | 00:45 |
ianw | JayF: but i see the rsync copying files? | 00:45 |
JayF | Just offering up other alternatives to being bananas :) | 00:45 |
jeblair | ianw: is that in both rsyncs? | 00:46 |
openstackgerrit | Sampath Priyankara proposed openstack-infra/project-config: Request to add new project Masakari https://review.openstack.org/330370 | 00:46 |
*** tphummel has quit IRC | 00:46 | |
ianw | jeblair: checking http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/_zuul_ansible/logs/ansible.log | 00:47 |
jeblair | drwxr-x--- 2 jenkins jenkins 4096 Jun 16 23:15 apache | 00:47 |
jeblair | ianw: ^ | 00:47 |
ianw | i wonder if we can get those "\n" to actually be a newline | 00:47 |
*** tphummel has joined #openstack-infra | 00:47 | |
jeblair | that's on the log server | 00:47 |
fungi | heh, we preserve perms? | 00:47 |
jeblair | we are faithfully copying the permissions | 00:47 |
fungi | niiiiiice | 00:47 |
ianw | ahhh, so i guess devstack-gate should fix this in the cleanup | 00:48 |
ianw | i can do that | 00:48 |
JayF | Thanks for the help! Was not expecting to get both things fixed and explained in a matter of minutes | 00:48 |
fungi | well, i'd argue that our publishers shouldn't publish files we can't... publish | 00:48 |
JayF | A++ would reccomend openstack infra to a friend | 00:48 |
jeblair | ianw: i think that's probably best... arguably we could have zuul-launcher not preserve perms, but requires more thought than the d-g change (and d-g *intends* to fix that stuff up anyway, so is a good one) | 00:49 |
*** thorst_ has joined #openstack-infra | 00:49 | |
ianw | jeblair / fungi : yeah, d-g is trying to get these files available, so it seems to make sense even not in a zuul-launcher context | 00:50 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul: Enabled host_key_checking https://review.openstack.org/330827 | 00:50 |
*** ddieterly has joined #openstack-infra | 00:51 | |
fungi | ianw: agreed. i think we have it possibly wrong in two places, but fixing one is enough and that one is easier | 00:51 |
*** xarses_ has joined #openstack-infra | 00:52 | |
fungi | i mean, maybe we _do_ want the flexibility of uploading files from a job and having them not be world-readable, but i'm not coming up with one | 00:52 |
ianw | ahh, we already do a chmod, just not enough -> https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n969 | 00:53 |
pabelanger | fungi: I can see that being usefully for somebody. | 00:54 |
openstackgerrit | Derek Higgins proposed openstack-infra/system-config: Add rh2 cloud credentials https://review.openstack.org/330833 | 00:54 |
*** thorst_ has quit IRC | 00:54 | |
openstackgerrit | Merged openstack-infra/system-config: Re-enable in-progress job url links https://review.openstack.org/330815 | 00:54 |
pabelanger | and think fixing devstack to setup the permissions properly is likely the right first | 00:54 |
openstackgerrit | Derek Higgins proposed openstack-infra/project-config: Add a temporary tripleo cloud "rh2" https://review.openstack.org/330514 | 00:54 |
openstackgerrit | Derek Higgins proposed openstack-infra/project-config: Add a experimental tripleo job on rh2 https://review.openstack.org/330515 | 00:54 |
*** thorst_ has joined #openstack-infra | 00:55 | |
* pabelanger quickly refreshes status.o.o/zuul waiting for 330815 to go live | 00:55 | |
* jeblair kicks static.openstack.org | 00:55 | |
pabelanger | \o/ | 00:56 |
jroll | that ironic port thing was an amazing bug, thanks friends for the help there :) | 00:57 |
jroll | was wondering why frequency just kept increasing | 00:57 |
jeblair | jroll: sorry about that! thanks for saying "8088" :) | 00:57 |
*** sdake has joined #openstack-infra | 00:58 | |
fungi | we should all say "8088" more often | 00:58 |
jroll | heh | 00:58 |
jeblair | oh, i didn't update system-config before kicking. oh well, i'll just wait for the cron. | 00:58 |
*** thorst_ has quit IRC | 00:59 | |
*** zhurong has joined #openstack-infra | 01:00 | |
JayF | clarkb: https://bugs.launchpad.net/ironic/+bug/1593510 filed to make our stuff play nicely on port :80 with the other pieces. | 01:01 |
openstack | Launchpad bug 1593510 in Ironic "[RFE] Host ipxe static assets in devstack on :80" [Undecided,New] | 01:01 |
*** tkelsey has joined #openstack-infra | 01:01 | |
*** Apoorva has quit IRC | 01:01 | |
*** linuxmaniac has quit IRC | 01:04 | |
jeblair | some non-afs servers have slipped into the afs ansible playbook | 01:05 |
jeblair | probably a grouping error | 01:05 |
*** samP has joined #openstack-infra | 01:05 | |
*** tkelsey has quit IRC | 01:05 | |
*** linuxmaniac has joined #openstack-infra | 01:06 | |
nibalizer | jeblair: which ones? | 01:07 |
jeblair | nibalizer: some afs clients -- mirror-update and release.slave | 01:08 |
openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: Set read/search perms recursively https://review.openstack.org/330835 | 01:08 |
jeblair | (the afs playbook was for afs servers so we can serialize them so we don't lose quorum) | 01:08 |
ianw | jeblair / fungi : ^ should do it. TIL about "X" (as opposed to "x") | 01:08 |
nibalizer | jeblair: hrm http://paste.openstack.org/show/516755/ | 01:08 |
JayF | ianw: that is amazing | 01:09 |
JayF | ianw: X, I mean (vs x, I just googled it) | 01:09 |
openstackgerrit | Merged openstack/diskimage-builder: Ironic agent kernel should be owned by user building image https://review.openstack.org/330783 | 01:09 |
jeblair | nibalizer: huh. that looks like the intent. | 01:09 |
ianw | JayF: it's often amazing what you find when you actually read the man pages | 01:09 |
nibalizer | jeblair: what are you seeing/looking at? | 01:09 |
jeblair | nibalizer: puppet run all log | 01:10 |
JayF | ianw: thanks for that fix too, I appreciate you digging into it | 01:10 |
JayF | leaving for the day, ty all for the help getting these things fixed up o/ | 01:10 |
jeblair | JayF: np, thank you! | 01:10 |
pabelanger | jeblair: nibalizer: I see the issue, we have an [afsadmin] group, and hosts: "afs*" in our playbook | 01:11 |
jeblair | pabelanger: and the group matches the regex? | 01:11 |
pabelanger | jeblair: looks like it | 01:11 |
fungi | ianw: great find! i too was oblivious to X | 01:11 |
jeblair | neat | 01:11 |
pabelanger | mirror-update and release.slave are in afsadmin | 01:12 |
jeblair | i guess we should call it adminafs :) | 01:12 |
jeblair | pabelanger: status.o.o updated | 01:12 |
pabelanger | YES | 01:13 |
pabelanger | now to setup telnet:// map in chrome | 01:13 |
dmsimard | CONSOLE LINKS | 01:13 |
dmsimard | \o/ | 01:13 |
nibalizer | jeblair: http://paste.openstack.org/show/516757/ looking at it its probably the * in 'afs*' | 01:13 |
nibalizer | oh pabelanger way ahead of me | 01:14 |
jeblair | #status log follow-up mail about zuul-related changes: http://lists.openstack.org/pipermail/openstack-dev/2016-June/097595.html | 01:14 |
*** sdake has quit IRC | 01:14 | |
openstackstatus | jeblair: finished logging | 01:14 |
jeblair | infra-root: ^ | 01:14 |
jhesketh | jeblair: I saw that... awesome stuff ! :-) | 01:15 |
jhesketh | amazing work to all | 01:15 |
jeblair | jhesketh: thank you too -- i hope to look at the testing stuff next week | 01:15 |
jeblair | and, you know, zuulv3 | 01:15 |
nibalizer | neat | 01:15 |
clarkb | the telnet links totally work on android with connectbot | 01:15 |
jeblair | clarkb: !!!!! | 01:15 |
clarkb | not really readable though | 01:16 |
*** sdake has joined #openstack-infra | 01:16 | |
nibalizer | clarkb: nice | 01:16 |
pabelanger | clarkb: nice! | 01:16 |
jhesketh | jeblair: that's cool, I haven't made it as far as I had hoped with the testing... nor did as much as I had hoped on v25 (or v3 yet for that matter) sorry :-s | 01:16 |
jeblair | np | 01:16 |
dmsimard | pabelanger: you figure out how to get chrome to handle those properly yet ? still looking myself | 01:16 |
pabelanger | dmsimard: hacking the googles chromes now | 01:17 |
clarkb | the issue is we dont use telnet line breaks | 01:17 |
clarkb | so the lines dont format well | 01:17 |
*** Goneri has joined #openstack-infra | 01:17 | |
*** roxanaghe has joined #openstack-infra | 01:17 | |
ianw | fungi: i think learning today that with rebase you can specify --exec to run tox in between each rebased change is still my "flag of the day" find :) | 01:18 |
clarkb | chrome has an ssh/telnet app | 01:19 |
clarkb | you can likely use that | 01:19 |
nibalizer | with bbsfox it seems to work | 01:19 |
nibalizer | sorta | 01:19 |
*** jsavak has joined #openstack-infra | 01:20 | |
clarkb | chrome secure shell | 01:20 |
fungi | ianw: yep, rebase --exec is fun, similar to using bisect | 01:21 |
ianw | dmsimard: ok, so i'm googling how to make readable and your name comes up ... -> http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/_zuul_ansible/logs/ansible.log | 01:22 |
*** roxanaghe has quit IRC | 01:22 | |
*** sdake_ has joined #openstack-infra | 01:22 | |
ianw | dmsimard: your "human_log" gist is best-practice? https://gist.github.com/dmsimard/cd706de198c85a8255f6 | 01:22 |
dmsimard | lol | 01:23 |
dmsimard | ianw: sooooooo | 01:24 |
dmsimard | ianw: it's what I found worked best | 01:24 |
*** Jeffrey4l has quit IRC | 01:24 | |
*** sdake has quit IRC | 01:24 | |
dmsimard | I forked the original version and made it a bit better and ansible 2 ready | 01:24 |
*** amitgandhinz has joined #openstack-infra | 01:24 | |
dmsimard | but it kind of sucks, because it doesn't suppress verbose output.. it prints the verbose output and then the "optimized" output | 01:25 |
dmsimard | so it makes for a super crowded shell | 01:25 |
dmsimard | this is the story of why I'm working on this: http://logs.openstack.org/76/330576/6/check/gate-ara-integration-ubuntu-trusty/6edcb0a/logs/build/ | 01:25 |
jamielennox | hey, so submitting a patch to oslo.context is telling me: ! [remote rejected] HEAD -> refs/publish/master/deprecate-vars (project is read only) and there is no project config at https://review.openstack.org/#/admin/projects/openstack/oslo.context,access | 01:26 |
lifeless | ianw: you can also poke in a per-stage test permanently with a little config | 01:26 |
jamielennox | known issue? something happening? | 01:26 |
jgriffith | clarkb: jeblair fungi et'al Well done, love the telnet access FWIW | 01:26 |
* jgriffith just needs to figure out how to implement all of it on his side :) | 01:27 | |
lifeless | ianw: https://github.com/testing-cabal/mock/blob/master/docs/index.txt#L147 | 01:27 |
jamielennox | https://review.openstack.org/gitweb?p=openstack/oslo.context.git;a=blobdiff;f=project.config;h=9e7a0f50b2759180ce6cc82773bdbbe998130279;hp=23af7117ff1ef93a9d20809fc97b1a15c6c005eb;hb=4dea369bdde28658a276e53bd6cb19b215055b94;hpb=15f83c3fc2ad5f6e098b7e5bdee490fa0dbb7901 | 01:28 |
*** baoli has joined #openstack-infra | 01:28 | |
eliqiao | hi AJaeger: thank you for helping update patch and give helps on irc renaming stuff :) , really appreciate of it? | 01:29 |
eliqiao | sorry s/?/. | 01:29 |
*** amitgandhinz has quit IRC | 01:30 | |
*** hongbin has joined #openstack-infra | 01:30 | |
ianw | it also seems colorization doesn't know when to stop -> http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/console.html | 01:31 |
*** kushal has joined #openstack-infra | 01:32 | |
fungi | jamielennox: dhellmann: i wonder if this is the problem? http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/projects.yaml#n2996 | 01:32 |
jamielennox | fungi: yep, just found that myself | 01:33 |
*** Daisy has joined #openstack-infra | 01:33 | |
jamielennox | fungi: is still pointing to the old incubator acls so has been shutdown - will propose something soon | 01:33 |
*** Apoorva has joined #openstack-infra | 01:33 | |
*** Goneri has quit IRC | 01:34 | |
fungi | jamielennox: yeah, missed in https://review.openstack.org/320680 | 01:35 |
*** sarob has quit IRC | 01:36 | |
openstackgerrit | Merged openstack-infra/system-config: Use port 19885 for console streaming for zuul workers https://review.openstack.org/330831 | 01:36 |
openstackgerrit | Merged openstack-infra/project-config: Ensure console-log is our last publisher https://review.openstack.org/329984 | 01:37 |
*** Jeffrey4l has joined #openstack-infra | 01:38 | |
dmsimard | ianw: let me know if you have any questions regarding human_log or ara, can show you how both of them look like in practice | 01:40 |
openstackgerrit | Jamie Lennox proposed openstack-infra/project-config: Add ACLs back for oslo.context https://review.openstack.org/330845 | 01:40 |
jamielennox | fungi: ^ | 01:41 |
dhellmann | jamielennox, fungi : looks good. harlowja_ ^^ | 01:41 |
*** ddieterly has quit IRC | 01:42 | |
ianw | dmsimard: well, i was just going to stick it into zuul, but i guess it's GPL so that's a no-go | 01:42 |
dmsimard | ianw: don't see how that's a problem, Zuul has GPL stuff in it already. | 01:42 |
dmsimard | https://github.com/openstack-infra/zuul/blob/master/zuul/ansible/plugins/callback_plugins/timeout.py | 01:43 |
dmsimard | There's an ongoing unclear debate as to whether or not stuff importing ansible has to be GPL or not | 01:43 |
*** yanyanhu has joined #openstack-infra | 01:44 | |
dmsimard | I have an ongoing thread with RH Legal and Ansible | 01:44 |
dmsimard | Ansible *is* GPL | 01:44 |
ianw | dmsimard: ok, well if we're doing that ... i guess just putting it in there would work and give us clearer output? | 01:44 |
*** mtanino has quit IRC | 01:45 | |
dmsimard | clearer is subjective, hang on, let me show you. | 01:45 |
dmsimard | ianw: https://ci.centos.org/job/weirdo-master-promote-packstack-scenario001/263/consoleText | 01:45 |
*** hongbin has quit IRC | 01:45 | |
ianw | dmsimard: ahh, so you get the two copies | 01:46 |
dmsimard | ianw: yeah, that's what I was saying. This is one task: https://etherpad.openstack.org/p/TvALrzw4ww | 01:46 |
*** claudiub has quit IRC | 01:47 | |
dmsimard | Now that everything is switched to zuul launcher, I was going to try to squeeze my ara project inside a job to see what it looks like | 01:47 |
*** Daisy has quit IRC | 01:47 | |
*** Goneri has joined #openstack-infra | 01:48 | |
*** jsavak has quit IRC | 01:48 | |
*** Daisy has joined #openstack-infra | 01:48 | |
*** Daisy has quit IRC | 01:52 | |
*** ddieterly has joined #openstack-infra | 01:52 | |
dmsimard | I was also wondering if zuul launcher would be affected by ansible configuration changes (such as exporting config variables and such) | 01:54 |
dmsimard | only one way to find out :) | 01:54 |
*** banix has joined #openstack-infra | 01:56 | |
*** Sukhdev has joined #openstack-infra | 01:57 | |
ianw | hmm, zuul, python3 + pep8 == ImportError: No module named 'extras' | 01:59 |
ianw | down the rabbit hole ... | 01:59 |
pabelanger | dmsimard: got it working | 02:00 |
dmsimard | pabelanger: oh, do tell | 02:00 |
*** Goneri has quit IRC | 02:00 | |
pabelanger | dmsimard: let me get pastebin | 02:00 |
pabelanger | dmsimard: https://etherpad.openstack.org/p/google-telnet might be better | 02:01 |
pabelanger | dmsimard: that should be it | 02:03 |
dmsimard | That's way more involved than than I thought it'd be, I'll try it | 02:03 |
pabelanger | dmsimard: there likely is a better way | 02:04 |
pabelanger | I came up with that just hacking | 02:04 |
*** SumitNaiksatam has joined #openstack-infra | 02:04 | |
dmsimard | wfm if it works | 02:04 |
*** Apoorva has quit IRC | 02:05 | |
pabelanger | calling it for tonight should have left hours ago | 02:05 |
*** Daisy has joined #openstack-infra | 02:06 | |
jeblair | ianw: re zuul py3 -- see https://review.openstack.org/325661 and related patches | 02:08 |
jeblair | sorry have to run | 02:08 |
ianw | lifeless: pbr -> testrepository -> subunit -> ImportError: No module named 'extras' (http://paste.openstack.org/show/516764/) ... suggestion welcome :) | 02:09 |
*** chem|off has quit IRC | 02:10 | |
ianw | ok, so extras is a python2 thing | 02:11 |
ianw | i am guessing this "basepython" set in -> http://git.openstack.org/cgit/openstack-infra/zuul/tree/tox.ini#n21 is not entirely safe | 02:12 |
*** SumitNaiksatam has quit IRC | 02:12 | |
*** Goneri has joined #openstack-infra | 02:15 | |
*** sdake has joined #openstack-infra | 02:16 | |
*** ddieterly has quit IRC | 02:17 | |
ianw | have we lost the "toggle CI" button? | 02:18 |
*** roxanaghe has joined #openstack-infra | 02:18 | |
*** sdake_ has quit IRC | 02:19 | |
*** piet has joined #openstack-infra | 02:22 | |
*** mtanino has joined #openstack-infra | 02:22 | |
*** roxanaghe has quit IRC | 02:23 | |
*** Goneri has quit IRC | 02:26 | |
*** amitgandhinz has joined #openstack-infra | 02:26 | |
ianw | the answer appears to be yes, and that we are looking for -> https://review.openstack.org/static/hideci.js?e=df833e6b9eb694b6da03c5d403d38a8a which doesn't exist | 02:27 |
phschwartz | mordred: fungi: nibalizer: so all of those reviews merged and I have verified in my downstream CI that all things are working onces again | 02:27 |
phschwartz | If I find any more that need to change I will open reviews | 02:27 |
tonyb | Is there a way to see all the jobs run on the post-release pipline in say the last 34hours? | 02:28 |
*** Goneri has joined #openstack-infra | 02:28 | |
ianw | ahh, seems to be the same issue as https://review.openstack.org/#/c/126482/ ... i'm guessing this isn't working | 02:29 |
ianw | jhesketh: ^ around maybe? | 02:29 |
*** amitgandhinz has quit IRC | 02:31 | |
*** jogo has quit IRC | 02:32 | |
*** ddieterly has joined #openstack-infra | 02:33 | |
jhesketh | ianw: right, sometimes that doesn't work and we don't know why | 02:33 |
jhesketh | ianw: the solution is to just touch the header | 02:34 |
jhesketh | I'll do that now | 02:34 |
openstackgerrit | zzxwill proposed openstack-infra/zuul: ename jenkins to zull https://review.openstack.org/330853 | 02:34 |
*** piet has quit IRC | 02:35 | |
jhesketh | ianw: the new link is https://review.openstack.org/static/hideci.js?e=3820a6091f0996731e3f0dd72527aa14 | 02:36 |
ianw | jhesketh: yay, the button is back! | 02:36 |
ianw | yak shaved | 02:36 |
dmsimard | Basic zuul runner/launcher question, where does Ansible actually run ? On "localhost" (the nodepool VM?) or elsewhere ? I can't find those files: https://github.com/openstack-infra/zuul/blob/aaa852e455a1b2135c53f4a6251b659cea2d41d4/zuul/launcher/ansiblelaunchserver.py#L102 | 02:37 |
dmsimard | http://logs.openstack.org/47/330847/3/check/gate-ara-integration-centos-7/b777804/console.html#_2016-06-17_02_26_26_824055 | 02:37 |
jhesketh | ianw: yeah I wonder if there is a race where jenkins continues to the notify command while the asset is still being syncd or something | 02:37 |
*** thorst_ has joined #openstack-infra | 02:38 | |
dmsimard | It looks like it would run from a "central" location (zuul launcher servers?) but at the same time I can see those servers getting quite busy with the amount of running jobs, even if it's just Ansible. No ? | 02:39 |
dmsimard | I guess we can still scale the zuul launchers fairly well horizontally | 02:40 |
*** thorst_ has quit IRC | 02:41 | |
*** thorst_ has joined #openstack-infra | 02:41 | |
*** somerville32 has quit IRC | 02:42 | |
jhesketh | dmsimard: yes, that's right.. the launchers hold the playbooks and then run them on the slave nodes | 02:44 |
jhesketh | and yeah, scalling is pretty horizontal | 02:45 |
openstackgerrit | zzxwill proposed openstack-infra/zuul: Rename jenkins to zull https://review.openstack.org/330853 | 02:45 |
*** thorst_ has quit IRC | 02:45 | |
*** jogo has joined #openstack-infra | 02:46 | |
*** Goneri has quit IRC | 02:46 | |
dmsimard | jhesketh: thought it ran locally, that would've allowed me to test if Zuul could've used another ansible callback successfully :( | 02:47 |
dmsimard | I would've submitted a patch to zuul to test it, but there's not really any integration test that'd allow me to see if it worked or not | 02:47 |
dmsimard | not sure how I'd test this tbh, I don't have a pocket zuul | 02:48 |
*** ddieterly has quit IRC | 02:48 | |
*** mdrabe has joined #openstack-infra | 02:49 | |
dmsimard | Hmm, I've never thought of that, but I guess Zuul also zuul-clones itself for code reviews | 02:50 |
dmsimard | Ah, it's not effective for the jobs that are ran in the review, though | 02:51 |
jhesketh | zuulv3 will allow for some of that stuff | 02:51 |
jhesketh | but yeah, it's a bit limited at the moment | 02:51 |
dmsimard | For example https://review.openstack.org/#/c/330827/ adds some ssh config | 02:51 |
dmsimard | but if you look at the ansible.cfg of the jobs, that ssh config isn't there | 02:51 |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul: Add human-log filter plugin https://review.openstack.org/330856 | 02:52 |
ianw | dmsimard: ^ something to argue over | 02:52 |
*** zhurong_ has joined #openstack-infra | 02:53 | |
dmsimard | ianw: eh I'm not going to fight for it | 02:53 |
dmsimard | ianw: I'd love to integrate ARA into Zuul but I don't know how to test it | 02:54 |
dmsimard | The way I'd see this would be: add ara to requirements.txt, add the ara callback path to callback_plugins and then in the post-job where the console logs are saved, we also generate a static version of the ansible run so we save something like this in the logs of the job: http://logs.openstack.org/34/330834/3/check/gate-ara-integration-ubuntu-trusty/d954777/logs/build/ | 02:55 |
nibalizer | phschwartz: http://codesearch.openstack.org/?q=provider%20%3D%3E%20%27pip%27&i=nope&files=&repos= and http://codesearch.openstack.org/?q=provider%20%3D%3E%20*pip&i=nope&files=&repos= seem to imply you hit them all | 02:56 |
nibalizer | at least under -infra | 02:56 |
dmsimard | Super simple but I'm not comfortable just hooking that up, I guess I'll send a review | 02:56 |
*** zhurong has quit IRC | 02:56 | |
*** Goneri has joined #openstack-infra | 02:57 | |
*** jamielennox is now known as jamielennox|away | 03:03 | |
openstackgerrit | zzxwill proposed openstack-infra/zuul: Rename jenkins to zuul https://review.openstack.org/330853 | 03:06 |
*** gyee has quit IRC | 03:07 | |
*** amotoki has joined #openstack-infra | 03:07 | |
*** Goneri has quit IRC | 03:08 | |
*** jcoufal has joined #openstack-infra | 03:08 | |
*** amotoki has quit IRC | 03:09 | |
*** gouthamr_ has quit IRC | 03:12 | |
*** tphummel has quit IRC | 03:13 | |
jlvillal | Would the openstackgerrit bot not announce patches to stable/mitaka for openstack/ironic on #openstack-ironic for some reason? | 03:17 |
*** r-mibu has quit IRC | 03:17 | |
*** jamielennox|away is now known as jamielennox | 03:19 | |
*** roxanaghe has joined #openstack-infra | 03:19 | |
mtreinish | jlvillal: it's not configured to: http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerritbot/channels.yaml#n948 | 03:20 |
jlvillal | mtreinish, Thanks. I was wandering around from project to project to figure out where that is configured :) | 03:20 |
*** Goneri has joined #openstack-infra | 03:21 | |
*** roxanaghe has quit IRC | 03:24 | |
*** baoli has quit IRC | 03:25 | |
*** jogo has quit IRC | 03:26 | |
openstackgerrit | John L. Villalovos proposed openstack-infra/project-config: Ironic: Have gerritbot announce for stable branches too https://review.openstack.org/330867 | 03:26 |
*** amitgandhinz has joined #openstack-infra | 03:27 | |
*** Goneri has quit IRC | 03:28 | |
*** jogo has joined #openstack-infra | 03:30 | |
*** amitgandhinz has quit IRC | 03:31 | |
pabelanger | dmsimard: the playbook currently live on the zuul-launchers (eg: zl01.openstack.org). We are dynamically building them when we see a new node. After the job is finished, we'll upload them into the _zuul_ansible folder for logs.o.o. Obviously moving forward, how we build and setup playbooks with change | 03:36 |
pabelanger | dmsimard: with the URL you linked above, here are the playbooks for that job. http://logs.openstack.org/47/330847/3/check/gate-ara-integration-centos-7/b777804/_zuul_ansible/ | 03:37 |
dmsimard | pabelanger: yeah I eventually sort of figured that out | 03:37 |
dmsimard | pabelanger: I'm going to submit a patch just to see what it'd look like, not expecting it to go it | 03:37 |
dmsimard | go in* | 03:38 |
*** r-mibu has joined #openstack-infra | 03:38 | |
dmsimard | that said, I don't see ansible in Zuul's requirement.txt | 03:38 |
dmsimard | hacks ? | 03:38 |
pabelanger | we've left it our for now | 03:38 |
pabelanger | and manage it from puppet | 03:38 |
dmsimard | ah, okay. I'll look in the manifests, just interested in knowing the version | 03:38 |
pabelanger | mostly because we don't want to support v25 for other users :) | 03:38 |
pabelanger | 2.1.0.0 | 03:38 |
dmsimard | alright | 03:39 |
*** furlongm has quit IRC | 03:41 | |
*** furlongm has joined #openstack-infra | 03:41 | |
*** Goneri has joined #openstack-infra | 03:46 | |
*** markvoelker has quit IRC | 03:47 | |
*** mdrabe has quit IRC | 03:48 | |
*** gema has quit IRC | 03:51 | |
*** mriedem has quit IRC | 03:51 | |
openstackgerrit | David Moreau Simard proposed openstack-infra/zuul: WIP: Integrate ARA reporting inside Zuul Launcher https://review.openstack.org/330874 | 03:54 |
dmsimard | ianw: ^ | 03:54 |
*** Daisy has quit IRC | 03:55 | |
*** Daisy has joined #openstack-infra | 03:55 | |
*** Goneri has quit IRC | 03:56 | |
*** Daisy has quit IRC | 03:58 | |
*** Daisy has joined #openstack-infra | 03:58 | |
*** amotoki has joined #openstack-infra | 04:00 | |
*** tkelsey has joined #openstack-infra | 04:03 | |
ianw | dmsimard: cool | 04:05 |
*** tkelsey has quit IRC | 04:07 | |
*** Daisy has quit IRC | 04:11 | |
*** Daisy has joined #openstack-infra | 04:11 | |
*** Daisy has quit IRC | 04:16 | |
*** amotoki has quit IRC | 04:18 | |
krotscheck | fungi: I'll try that, thanks | 04:19 |
*** links has joined #openstack-infra | 04:20 | |
*** _nadya_ has joined #openstack-infra | 04:22 | |
openstackgerrit | Rico Lin proposed openstack-infra/project-config: Fix error jenkins scripts path. https://review.openstack.org/330880 | 04:23 |
*** shahid_ has joined #openstack-infra | 04:23 | |
*** yamahata has joined #openstack-infra | 04:27 | |
*** amitgandhinz has joined #openstack-infra | 04:28 | |
*** ociuhandu has quit IRC | 04:29 | |
*** amitgandhinz has quit IRC | 04:33 | |
*** psachin has joined #openstack-infra | 04:35 | |
*** tqtran has quit IRC | 04:36 | |
openstackgerrit | Ian Wienand proposed openstack-infra/zuul: Add human-log filter plugin https://review.openstack.org/330856 | 04:38 |
*** yuanying has quit IRC | 04:41 | |
*** bhavik has quit IRC | 04:41 | |
*** ociuhandu has joined #openstack-infra | 04:42 | |
*** amotoki has joined #openstack-infra | 04:45 | |
*** yamamoto has quit IRC | 04:46 | |
*** markvoelker has joined #openstack-infra | 04:47 | |
*** sdake_ has joined #openstack-infra | 04:50 | |
*** markvoelker has quit IRC | 04:52 | |
*** sdake has quit IRC | 04:53 | |
*** Daisy has joined #openstack-infra | 04:53 | |
*** sflanigan is now known as sflanigan_afk | 04:59 | |
*** Daisy has quit IRC | 05:02 | |
*** Daisy has joined #openstack-infra | 05:02 | |
openstackgerrit | Merged openstack/diskimage-builder: Pre-install pip/virtualenv packages https://review.openstack.org/327472 | 05:05 |
*** Daisy has quit IRC | 05:07 | |
*** Daisy has joined #openstack-infra | 05:07 | |
*** maishsk has quit IRC | 05:07 | |
*** maishsk has joined #openstack-infra | 05:08 | |
*** salv-orl_ has joined #openstack-infra | 05:09 | |
*** Daisy has quit IRC | 05:10 | |
*** Daisy has joined #openstack-infra | 05:10 | |
*** salv-orlando has quit IRC | 05:12 | |
*** pgadiya has joined #openstack-infra | 05:12 | |
*** gema has joined #openstack-infra | 05:13 | |
*** pc_m has quit IRC | 05:16 | |
openstackgerrit | Merged openstack/diskimage-builder: Remove discover from test-requirements https://review.openstack.org/328170 | 05:19 |
*** banix has quit IRC | 05:19 | |
*** roxanagh_ has joined #openstack-infra | 05:20 | |
openstackgerrit | Merged openstack-infra/project-config: Move gate-manila-buildimage-docker job to check pipeline https://review.openstack.org/330604 | 05:22 |
*** roxanagh_ has quit IRC | 05:25 | |
*** sflanigan_afk is now known as sflanigan | 05:25 | |
*** yamamoto has joined #openstack-infra | 05:26 | |
*** links has quit IRC | 05:27 | |
*** sdake has joined #openstack-infra | 05:27 | |
*** links has joined #openstack-infra | 05:28 | |
*** amitgandhinz has joined #openstack-infra | 05:29 | |
*** sdake_ has quit IRC | 05:31 | |
*** Daisy has quit IRC | 05:32 | |
*** Daisy has joined #openstack-infra | 05:32 | |
*** _nadya_ has quit IRC | 05:33 | |
*** amitgandhinz has quit IRC | 05:33 | |
*** cfarquhar has quit IRC | 05:35 | |
*** _nadya_ has joined #openstack-infra | 05:36 | |
*** Daisy has quit IRC | 05:37 | |
*** sdake_ has joined #openstack-infra | 05:38 | |
*** sdake has quit IRC | 05:41 | |
*** Daisy has joined #openstack-infra | 05:42 | |
*** _nadya_ has quit IRC | 05:43 | |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Move element-info a standard entry-point https://review.openstack.org/330893 | 05:48 |
*** sdake_ has quit IRC | 05:50 | |
*** _nadya_ has joined #openstack-infra | 05:51 | |
*** amotoki has quit IRC | 05:56 | |
*** ccamacho has joined #openstack-infra | 06:00 | |
*** hashar has joined #openstack-infra | 06:01 | |
*** yaume has joined #openstack-infra | 06:02 | |
*** amotoki has joined #openstack-infra | 06:02 | |
*** yuanying has joined #openstack-infra | 06:02 | |
*** Daisy has quit IRC | 06:04 | |
*** Daisy has joined #openstack-infra | 06:04 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Add console-log validator https://review.openstack.org/329988 | 06:08 |
*** Daisy has quit IRC | 06:09 | |
*** YorikSar has quit IRC | 06:09 | |
*** yaume has quit IRC | 06:13 | |
*** Daisy has joined #openstack-infra | 06:15 | |
*** pc_m has joined #openstack-infra | 06:15 | |
*** Sukhdev has quit IRC | 06:17 | |
*** YorikSar has joined #openstack-infra | 06:19 | |
*** _nadya_ has quit IRC | 06:20 | |
*** aeng has quit IRC | 06:21 | |
openstackgerrit | Merged openstack-infra/project-config: Add ACLs back for oslo.context https://review.openstack.org/330845 | 06:21 |
openstackgerrit | Khai Do proposed openstack-infra/system-config: Refactor gerrit commentlinks parameter https://review.openstack.org/330902 | 06:22 |
*** rcernin has joined #openstack-infra | 06:23 | |
*** YorikSar has quit IRC | 06:23 | |
*** mtanino has quit IRC | 06:24 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config: Normalize projects.yaml https://review.openstack.org/330903 | 06:24 |
*** david-lyle has quit IRC | 06:26 | |
*** andreas_s has joined #openstack-infra | 06:28 | |
*** zeih has joined #openstack-infra | 06:28 | |
*** pcaruana has joined #openstack-infra | 06:29 | |
*** amitgandhinz has joined #openstack-infra | 06:30 | |
AJaeger | yolanda, https://github.com/openstack/fuel-plugin-datera-cinder has not been setup yet but change merged for it. Could you check it, please? | 06:31 |
*** YorikSar has joined #openstack-infra | 06:32 | |
*** amitgandhinz has quit IRC | 06:34 | |
*** _nadya_ has joined #openstack-infra | 06:35 | |
openstackgerrit | Merged openstack-infra/project-config: Add console-log validator https://review.openstack.org/329988 | 06:36 |
*** chem|off has joined #openstack-infra | 06:36 | |
*** ramishra has joined #openstack-infra | 06:39 | |
*** _nadya_ has quit IRC | 06:40 | |
*** _nadya_ has joined #openstack-infra | 06:42 | |
*** ramishra has quit IRC | 06:44 | |
*** ramishra has joined #openstack-infra | 06:47 | |
*** Daisy_ has joined #openstack-infra | 06:50 | |
*** dingyichen has quit IRC | 06:51 | |
*** Daisy has quit IRC | 06:52 | |
ttx | jeblair: nice! | 06:53 |
*** arxcruz has joined #openstack-infra | 06:55 | |
*** chem|off` has joined #openstack-infra | 06:56 | |
*** kushal has quit IRC | 06:56 | |
*** Daisy_ has quit IRC | 06:57 | |
*** _nadya_ has quit IRC | 06:57 | |
*** Daisy has joined #openstack-infra | 06:58 | |
*** mixos has quit IRC | 06:59 | |
*** chem|off has quit IRC | 06:59 | |
openstackgerrit | Tony Breeds proposed openstack-infra/irc-meetings: No-op Change to force a re-publish https://review.openstack.org/330914 | 06:59 |
*** yuanying has quit IRC | 07:02 | |
*** ihrachys has joined #openstack-infra | 07:03 | |
*** amotoki has quit IRC | 07:03 | |
*** ramishra has quit IRC | 07:04 | |
*** tkelsey has joined #openstack-infra | 07:05 | |
openstackgerrit | Sachi King proposed openstack-dev/pbr: Restore warnerrors behavior and support Sphinx 1.4 https://review.openstack.org/229951 | 07:07 |
*** ihrachys has quit IRC | 07:08 | |
*** javeriak has joined #openstack-infra | 07:09 | |
*** salv-orl_ has quit IRC | 07:09 | |
*** chem|off` has quit IRC | 07:09 | |
*** salv-orlando has joined #openstack-infra | 07:09 | |
*** tkelsey has quit IRC | 07:09 | |
*** yaume has joined #openstack-infra | 07:10 | |
*** Na3iL has joined #openstack-infra | 07:10 | |
*** tesseract has joined #openstack-infra | 07:10 | |
AJaeger | https://github.com/openstack/fuel-plugin-datera-cinder is ready now... | 07:11 |
*** hogepodge has quit IRC | 07:12 | |
*** saravana has joined #openstack-infra | 07:12 | |
*** pahuang has quit IRC | 07:13 | |
*** bauzas is now known as bauwser | 07:13 | |
*** bauwser is now known as bauzas | 07:14 | |
saravana | my zuul status page shows status.json: Proxy Error | 07:14 |
openstackgerrit | Khai Do proposed openstack-infra/puppet-gerrit: Add configuration section for its plugins https://review.openstack.org/330922 | 07:14 |
*** matrohon has joined #openstack-infra | 07:15 | |
*** dtantsur|afk is now known as dtantsur | 07:15 | |
openstackgerrit | Merged openstack-infra/irc-meetings: No-op Change to force a re-publish https://review.openstack.org/330914 | 07:16 |
*** bauzas has left #openstack-infra | 07:18 | |
*** jlanoux has joined #openstack-infra | 07:20 | |
*** dtardivel has joined #openstack-infra | 07:20 | |
*** otherwiseguy has quit IRC | 07:20 | |
*** ramishra has joined #openstack-infra | 07:21 | |
*** otherwiseguy has joined #openstack-infra | 07:21 | |
*** esikachev has joined #openstack-infra | 07:22 | |
*** roxanaghe has joined #openstack-infra | 07:22 | |
*** vincentll has joined #openstack-infra | 07:23 | |
*** rossella_s has joined #openstack-infra | 07:25 | |
openstackgerrit | Khai Do proposed openstack-infra/system-config: Configure its-storyboard plugin https://review.openstack.org/330925 | 07:25 |
openstackgerrit | Merged openstack-infra/project-config: Normalize projects.yaml https://review.openstack.org/330903 | 07:26 |
openstackgerrit | Babu Shanmugam proposed openstack-infra/project-config: Adding a new JOB for networking-ovn project https://review.openstack.org/330926 | 07:27 |
*** roxanaghe has quit IRC | 07:27 | |
*** bauwser has joined #openstack-infra | 07:28 | |
openstackgerrit | Jeffrey Zhang proposed openstack-infra/project-config: Enable the Ubuntu and CentOS gate voting for Kolla https://review.openstack.org/330927 | 07:30 |
*** amitgandhinz has joined #openstack-infra | 07:30 | |
*** javeriak has quit IRC | 07:31 | |
*** javeriak has joined #openstack-infra | 07:31 | |
*** dmk0202 has joined #openstack-infra | 07:32 | |
*** dmk0202 has quit IRC | 07:33 | |
*** sorantis has joined #openstack-infra | 07:35 | |
*** salv-orlando has quit IRC | 07:35 | |
*** amitgandhinz has quit IRC | 07:35 | |
*** _ioni is now known as ioni | 07:36 | |
*** ioni is now known as Guest27179 | 07:36 | |
*** claudiub has joined #openstack-infra | 07:37 | |
openstackgerrit | zzxwill proposed openstack-infra/zuul: Rename jenkins to zuul https://review.openstack.org/330853 | 07:38 |
*** dingyichen has joined #openstack-infra | 07:39 | |
*** amoralej|off is now known as amoralej | 07:39 | |
*** javeriak has quit IRC | 07:42 | |
*** netsin has quit IRC | 07:45 | |
*** javeriak has joined #openstack-infra | 07:45 | |
*** rcernin has quit IRC | 07:45 | |
*** Daisy has quit IRC | 07:46 | |
*** Daisy has joined #openstack-infra | 07:46 | |
*** jpich has joined #openstack-infra | 07:50 | |
*** Daisy has quit IRC | 07:51 | |
*** Daisy has joined #openstack-infra | 07:51 | |
openstackgerrit | Masayuki Igawa proposed openstack/os-testr: Make unexpected success as fail https://review.openstack.org/330937 | 07:51 |
*** camunoz has quit IRC | 07:52 | |
*** tkelsey has joined #openstack-infra | 07:53 | |
*** ramishra has quit IRC | 07:53 | |
*** saravana has quit IRC | 07:53 | |
*** yolanda has joined #openstack-infra | 07:55 | |
*** ramishra has joined #openstack-infra | 07:55 | |
yolanda | good morning | 07:56 |
*** ramishra has quit IRC | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-infra | 08:00 | |
*** yamamoto has quit IRC | 08:01 | |
*** mikelk has joined #openstack-infra | 08:03 | |
*** ramishra has joined #openstack-infra | 08:04 | |
*** yuanying has joined #openstack-infra | 08:05 | |
openstackgerrit | Dmitry Tantsur proposed openstack-infra/project-config: Make ironic-{pxe,agent}_ipmitool jobs voting https://review.openstack.org/312596 | 08:06 |
*** eezhova has joined #openstack-infra | 08:06 | |
dtantsur | lucasagomes, updated this a bit while you were out ^^^ | 08:07 |
*** yamahata has quit IRC | 08:07 | |
*** rcernin has joined #openstack-infra | 08:10 | |
*** ramishra has quit IRC | 08:16 | |
*** dizquierdo has joined #openstack-infra | 08:18 | |
*** ameade has quit IRC | 08:21 | |
*** dteselkin has quit IRC | 08:21 | |
*** ramishra has joined #openstack-infra | 08:21 | |
*** BobBall_AWOL is now known as BobBall | 08:21 | |
*** redrobot has quit IRC | 08:21 | |
*** jlvillal has quit IRC | 08:22 | |
*** jlvillal has joined #openstack-infra | 08:22 | |
*** tobiash_ has quit IRC | 08:22 | |
*** ameade has joined #openstack-infra | 08:22 | |
*** igormarnat has quit IRC | 08:22 | |
*** igormarnat has joined #openstack-infra | 08:22 | |
*** hferenc has quit IRC | 08:22 | |
*** Daviey_ has quit IRC | 08:23 | |
*** rbergeron has quit IRC | 08:23 | |
*** roxanaghe has joined #openstack-infra | 08:23 | |
*** Daviey has joined #openstack-infra | 08:23 | |
*** tobiash has joined #openstack-infra | 08:23 | |
*** Guest27179 is now known as ioni | 08:23 | |
*** ioni is now known as Guest62342 | 08:24 | |
*** redrobot has joined #openstack-infra | 08:24 | |
*** redrobot is now known as Guest31553 | 08:24 | |
*** rbergeron has joined #openstack-infra | 08:24 | |
*** javeriak has quit IRC | 08:24 | |
*** Daisy has quit IRC | 08:26 | |
*** Daisy has joined #openstack-infra | 08:26 | |
*** dmk0202 has joined #openstack-infra | 08:27 | |
*** dteselkin has joined #openstack-infra | 08:27 | |
*** roxanaghe has quit IRC | 08:28 | |
*** daemontool has joined #openstack-infra | 08:28 | |
*** gema has quit IRC | 08:29 | |
*** Daisy_ has joined #openstack-infra | 08:29 | |
openstackgerrit | Masayuki Igawa proposed openstack/os-testr: Fix to enable '--no-failure-debug' option for success tests https://review.openstack.org/330952 | 08:30 |
*** amitgandhinz has joined #openstack-infra | 08:31 | |
*** Daisy has quit IRC | 08:33 | |
ttx | Looks like we have an issue with Launchpad auth on Gerrit | 08:34 |
*** SlickNik has quit IRC | 08:35 | |
*** SlickNik has joined #openstack-infra | 08:35 | |
*** icey has quit IRC | 08:35 | |
*** amitgandhinz has quit IRC | 08:35 | |
ttx | signing in does not redirect to UbuntuOne auth but to some "Sign In to Gerrit Code Review at review.openstack.org" canned page | 08:36 |
*** trash has joined #openstack-infra | 08:36 | |
*** jokke__ has joined #openstack-infra | 08:36 | |
*** mestery_ has joined #openstack-infra | 08:36 | |
*** andreas_s_ has joined #openstack-infra | 08:36 | |
*** tristanC_ has joined #openstack-infra | 08:37 | |
ttx | "Provider is not supported, or was incorrectly entered." | 08:37 |
*** kaisers has quit IRC | 08:37 | |
*** pgadiya_ has joined #openstack-infra | 08:37 | |
*** Jeffrey4l has quit IRC | 08:37 | |
*** flepied1 has joined #openstack-infra | 08:37 | |
*** jlvillal has quit IRC | 08:37 | |
*** mandre_ has joined #openstack-infra | 08:38 | |
ttx | AJaeger, yolanda ^ | 08:38 |
*** kaisers has joined #openstack-infra | 08:38 | |
*** bswartz has quit IRC | 08:39 | |
*** yamamoto has joined #openstack-infra | 08:39 | |
*** LinuxJedi_ has joined #openstack-infra | 08:39 | |
*** coreyob has quit IRC | 08:39 | |
*** rodrigods has quit IRC | 08:40 | |
*** andymccr_ has joined #openstack-infra | 08:40 | |
*** alazarev_ has joined #openstack-infra | 08:40 | |
yolanda | ttx, looking | 08:40 |
*** xwizard___ has joined #openstack-infra | 08:40 | |
ttx | FWIW logging into LP works ok | 08:40 |
*** rodrigods has joined #openstack-infra | 08:40 | |
yolanda | going to check logs | 08:41 |
*** harlowja_ has quit IRC | 08:41 | |
ttx | yolanda: can you reproduce ? | 08:41 |
*** pkoniszewski has quit IRC | 08:41 | |
*** electrofelix has joined #openstack-infra | 08:41 | |
yolanda | there may be some connectivity problem, i cannot ssh to gerrit | 08:41 |
ttx | Ah. Interesting | 08:42 |
*** jlvillal has joined #openstack-infra | 08:42 | |
*** Daisy_ has quit IRC | 08:42 | |
*** jmccrory has quit IRC | 08:42 | |
*** daemontool_ has joined #openstack-infra | 08:42 | |
*** anteaya has quit IRC | 08:42 | |
*** waynr has quit IRC | 08:42 | |
*** HeOS has joined #openstack-infra | 08:42 | |
*** Daisy has joined #openstack-infra | 08:42 | |
*** andreas_s has quit IRC | 08:42 | |
*** xwizard has quit IRC | 08:42 | |
*** tristanC has quit IRC | 08:42 | |
*** ekarlso has quit IRC | 08:42 | |
*** LinuxJedi has quit IRC | 08:42 | |
*** LinuxJedi_ is now known as LinuxJedi | 08:42 | |
*** alazarev has quit IRC | 08:42 | |
*** alazarev_ is now known as alazarev | 08:42 | |
*** m3m0 has quit IRC | 08:42 | |
*** Na3iL has quit IRC | 08:42 | |
*** shahid_ has quit IRC | 08:42 | |
*** shahid__ has joined #openstack-infra | 08:42 | |
*** furlongm has quit IRC | 08:42 | |
*** pkoniszewski has joined #openstack-infra | 08:43 | |
*** trash_ has quit IRC | 08:43 | |
*** flepied has quit IRC | 08:43 | |
*** mestery has quit IRC | 08:43 | |
*** mattymo has quit IRC | 08:43 | |
*** jokke_ has quit IRC | 08:43 | |
*** pgadiya has quit IRC | 08:43 | |
*** andymccr has quit IRC | 08:43 | |
*** tlbr has quit IRC | 08:43 | |
*** mestery_ is now known as mestery | 08:43 | |
*** zeroDivisible has quit IRC | 08:43 | |
*** mandre has quit IRC | 08:43 | |
*** kaisers1 has quit IRC | 08:43 | |
*** mandre_ is now known as mandre | 08:43 | |
*** rcarrillocruz has quit IRC | 08:43 | |
*** ruhe has quit IRC | 08:43 | |
*** SergeyLukjanov2 has joined #openstack-infra | 08:43 | |
*** asselin_ has joined #openstack-infra | 08:43 | |
*** SergeyLukjanov has quit IRC | 08:43 | |
*** dingyichen has quit IRC | 08:43 | |
*** persia has quit IRC | 08:43 | |
*** furlongm has joined #openstack-infra | 08:43 | |
*** zeroDivisible has joined #openstack-infra | 08:43 | |
*** jmccrory has joined #openstack-infra | 08:43 | |
*** SergeyLukjanov2 is now known as SergeyLukjanov | 08:43 | |
*** ramishra has quit IRC | 08:43 | |
*** waynr has joined #openstack-infra | 08:43 | |
*** persia has joined #openstack-infra | 08:43 | |
*** schang has quit IRC | 08:44 | |
*** rcarrillocruz has joined #openstack-infra | 08:44 | |
*** ruhe has joined #openstack-infra | 08:44 | |
yolanda | ttx, works now | 08:44 |
*** asselin__ has quit IRC | 08:44 | |
*** mattymo has joined #openstack-infra | 08:44 | |
*** mtreinish has quit IRC | 08:44 | |
ttx | indeed | 08:44 |
*** daemontool has quit IRC | 08:44 | |
*** schang has joined #openstack-infra | 08:44 | |
*** tlbr has joined #openstack-infra | 08:44 | |
*** e0ne has joined #openstack-infra | 08:44 | |
ttx | false alarm | 08:44 |
*** ihrachys has joined #openstack-infra | 08:44 | |
ttx | still the error was weird, must be some kind of fallback | 08:44 |
*** tristanC_ is now known as tristanC | 08:44 | |
*** dingyichen has joined #openstack-infra | 08:45 | |
*** kaisers has left #openstack-infra | 08:45 | |
*** m3m0 has joined #openstack-infra | 08:45 | |
*** kaisers1 has joined #openstack-infra | 08:45 | |
*** anteaya has joined #openstack-infra | 08:45 | |
*** kaisers1 has left #openstack-infra | 08:45 | |
*** kaisers1 has joined #openstack-infra | 08:45 | |
*** mtreinish has joined #openstack-infra | 08:46 | |
*** coreyob has joined #openstack-infra | 08:46 | |
*** MarkAtwood has quit IRC | 08:46 | |
openstackgerrit | Yaroslav Lobankov proposed openstack-infra/project-config: [Rally] Enable Ceilometer for one of the 'rally verify' jobs https://review.openstack.org/327157 | 08:46 |
*** Ng has quit IRC | 08:47 | |
*** tpeoples has quit IRC | 08:47 | |
*** dham1 has quit IRC | 08:47 | |
*** teran has quit IRC | 08:47 | |
*** sflanigan has quit IRC | 08:47 | |
*** Daisy_ has joined #openstack-infra | 08:47 | |
*** Jeffrey4l has joined #openstack-infra | 08:48 | |
*** teran has joined #openstack-infra | 08:49 | |
*** dham1 has joined #openstack-infra | 08:49 | |
*** tpeoples has joined #openstack-infra | 08:49 | |
*** MarkAtwood has joined #openstack-infra | 08:49 | |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/system-config: Replace hpuseast name for chocolate https://review.openstack.org/330580 | 08:49 |
*** Ng has joined #openstack-infra | 08:49 | |
*** kzaitsev_mb has joined #openstack-infra | 08:50 | |
*** Daisy has quit IRC | 08:50 | |
*** ekarlso has joined #openstack-infra | 08:51 | |
*** dizquierdo has quit IRC | 08:54 | |
*** ramishra has joined #openstack-infra | 08:54 | |
openstackgerrit | osman proposed openstack-infra/jenkins-job-builder: Adds support rerun matrix part to the naginator https://review.openstack.org/330959 | 08:56 |
*** Na3iL has joined #openstack-infra | 08:56 | |
*** Daisy has joined #openstack-infra | 08:58 | |
*** Daisy_ has quit IRC | 08:58 | |
*** ramishra has quit IRC | 08:59 | |
*** ramishra has joined #openstack-infra | 09:00 | |
*** markusry has joined #openstack-infra | 09:01 | |
openstackgerrit | Merged openstack-infra/project-config: Ironic: Have gerritbot announce for stable branches too https://review.openstack.org/330867 | 09:01 |
*** YorikSar has quit IRC | 09:01 | |
*** kushal has joined #openstack-infra | 09:01 | |
*** javeriak has joined #openstack-infra | 09:02 | |
openstackgerrit | Merged openstack-infra/project-config: Enable storyboard for js-openstack-lib https://review.openstack.org/330779 | 09:02 |
*** yanyanhu has quit IRC | 09:03 | |
*** YorikSar has joined #openstack-infra | 09:03 | |
*** yanyanhu has joined #openstack-infra | 09:04 | |
*** Daisy_ has joined #openstack-infra | 09:04 | |
*** yanyanhu has quit IRC | 09:04 | |
*** Daisy_ has quit IRC | 09:06 | |
*** Daisy has quit IRC | 09:06 | |
*** Na3iL has quit IRC | 09:06 | |
*** Na3iL has joined #openstack-infra | 09:07 | |
*** dtantsur is now known as dtantsur|bbl | 09:07 | |
*** _nadya_ has joined #openstack-infra | 09:11 | |
*** sambetts|afk is now known as sambetts | 09:12 | |
openstackgerrit | Merged openstack-infra/project-config: Remove Jenkins masters from grafana dashboard https://review.openstack.org/330784 | 09:12 |
*** jyuso1 has quit IRC | 09:12 | |
openstackgerrit | Merged openstack-infra/project-config: New Project Quark Plugin https://review.openstack.org/319368 | 09:12 |
*** e0ne has quit IRC | 09:13 | |
openstackgerrit | Merged openstack-infra/project-config: Adds API reference publishing jobs for Swift https://review.openstack.org/313015 | 09:14 |
bauwser | congrats folks for getting rid of Jenkins ! | 09:14 |
*** openstackgerrit has quit IRC | 09:18 | |
*** openstackgerrit has joined #openstack-infra | 09:18 | |
*** eezhova has quit IRC | 09:19 | |
*** samP has quit IRC | 09:20 | |
openstackgerrit | Merged openstack-infra/project-config: Add a functional job for neutron-lbaas https://review.openstack.org/322812 | 09:20 |
*** chem|off` has joined #openstack-infra | 09:21 | |
*** chem|off` is now known as chem | 09:21 | |
*** chem has quit IRC | 09:22 | |
*** acoles_ is now known as acoles | 09:22 | |
*** chem has joined #openstack-infra | 09:22 | |
*** acoles is now known as acoles_ | 09:23 | |
*** eezhova has joined #openstack-infra | 09:24 | |
*** Guest62342 is now known as ioni | 09:24 | |
*** ioni is now known as Guest58184 | 09:25 | |
*** binbincong has quit IRC | 09:25 | |
*** binbincong has joined #openstack-infra | 09:25 | |
*** alex_xu has quit IRC | 09:26 | |
*** AJaeger has quit IRC | 09:26 | |
openstackgerrit | Joshua Hesketh proposed openstack-infra/system-config: Fix the destination for eavesdrop site https://review.openstack.org/330986 | 09:27 |
*** alex_xu has joined #openstack-infra | 09:28 | |
*** acoles_ is now known as acoles | 09:29 | |
*** flepied1 is now known as flepied | 09:31 | |
*** amitgandhinz has joined #openstack-infra | 09:32 | |
*** mhickey has joined #openstack-infra | 09:33 | |
*** _milan_ has joined #openstack-infra | 09:33 | |
_milan_ | guys, I want to introduce inspector experimental grenade gate. I want to base it on the ironic job from devstack-gate.yaml jenkins definitnion in infra. I'm wondering how to make the job enable inspector's grenade plugin (under review now) without having to insert ref spec in the grenade plugin line. | 09:33 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/system-config: Fix the destination for eavesdrop site https://review.openstack.org/330986 | 09:33 |
jhesketh | any other infra-root's around to help merge ^ to unblock eavesdrop publishers? | 09:33 |
openstackgerrit | Elena Ezhova proposed openstack-infra/project-config: Run neutron-lbaas-dsvm-functional-nv job only on master https://review.openstack.org/330996 | 09:34 |
_milan_ | e.g https://github.com/openstack-infra/project-config/blob/d09a1a898d34931f7fc49e027bf665950e66a2b2/jenkins/jobs/devstack-gate.yaml#L2096 | 09:35 |
_milan_ | vs https://github.com/dparalen/devstack-gate-test/blob/with_inspector/ansible/roles/devstack-gate/files/inspector-grenade.sh#L99 | 09:35 |
*** amitgandhinz has quit IRC | 09:36 | |
eezhova | yolanda, Hi! It just came up to me that the new lbaas functional job should be run only on master, but the patch has already merged | 09:37 |
eezhova | yolanda, so I created a follow-up patch https://review.openstack.org/#/c/330996 | 09:37 |
eezhova | yolanda, could you please take a look at it? | 09:38 |
*** jokke__ is now known as jokke_ | 09:38 | |
*** Guest58184 has quit IRC | 09:40 | |
*** Guest58184 has joined #openstack-infra | 09:40 | |
*** Guest58184 is now known as ioni | 09:40 | |
*** kzaitsev_mb has quit IRC | 09:41 | |
*** kzaitsev_mb has joined #openstack-infra | 09:42 | |
*** dizquierdo has joined #openstack-infra | 09:42 | |
yolanda | eezhova, sure | 09:44 |
openstackgerrit | Merged openstack-infra/project-config: New repo for murano plugin https://review.openstack.org/329931 | 09:44 |
*** megm_ has quit IRC | 09:44 | |
chem | hi, on the tripleo-ci side we have all jobs that timeout, and it looks like something changed as we have new ansible logs, is there somebody that could help me find out ? | 09:44 |
openstackgerrit | Merged openstack-infra/system-config: Add compute014.useast metadata for better tracking https://review.openstack.org/330421 | 09:45 |
eezhova | yolanda, thanks! | 09:45 |
jhesketh | yolanda: Howdy.. If you have a moment to look at https://review.openstack.org/#/c/330986/ that'd be appreciated | 09:48 |
yolanda | hi jhesketh | 09:48 |
*** mjb has quit IRC | 09:48 | |
*** megm has joined #openstack-infra | 09:49 | |
openstackgerrit | Milan Kováčik proposed openstack-infra/project-config: Introduce experimental Inspector grenade job https://review.openstack.org/331001 | 09:50 |
*** saravana has joined #openstack-infra | 09:52 | |
_milan_ | dtantsur|bbl, ^ | 09:55 |
*** savihou has quit IRC | 09:55 | |
*** mjb has joined #openstack-infra | 09:55 | |
*** mjb has quit IRC | 09:56 | |
*** gouthamr has joined #openstack-infra | 09:59 | |
*** mjb has joined #openstack-infra | 09:59 | |
*** ramishra has quit IRC | 09:59 | |
*** gouthamr_ has joined #openstack-infra | 10:00 | |
*** eezhova has quit IRC | 10:01 | |
*** zeih has quit IRC | 10:03 | |
*** openstackgerrit has quit IRC | 10:03 | |
*** openstackgerrit has joined #openstack-infra | 10:03 | |
jhesketh | thanks yolanda | 10:03 |
jhesketh | yolanda: any objections to me self-merging? | 10:04 |
*** gouthamr has quit IRC | 10:04 | |
yolanda | jhesketh, no problem | 10:04 |
*** ramishra has joined #openstack-infra | 10:05 | |
jhesketh | cool, will do | 10:05 |
*** rcernin has quit IRC | 10:05 | |
*** eezhova has joined #openstack-infra | 10:06 | |
*** zhurong_ has quit IRC | 10:06 | |
*** kzaitsev_mb has quit IRC | 10:06 | |
*** esikachev has quit IRC | 10:07 | |
*** lvdongbing has quit IRC | 10:08 | |
*** zeih has joined #openstack-infra | 10:09 | |
odyssey4me | jhesketh I'm seeing an odd result from time to time where our 'linters' job log shows that it succeeded, but the job result shows as failed. An example: https://review.openstack.org/#/c/330973/ | 10:10 |
*** e0ne has joined #openstack-infra | 10:12 | |
jhesketh | odyssey4me: it looks like we're not reporting timeouts correctly... that's likely a bug | 10:14 |
jhesketh | in zuul that is | 10:14 |
jhesketh | yolanda: so I self-merged but it occurred to me I'll need to restart the launchers to apply the change :-( | 10:14 |
jhesketh | as far as I can tell that's a pause, wait then stop on each of the launchers | 10:14 |
odyssey4me | jhesketh should I note this somewhere, or just recheck and move on? | 10:15 |
yolanda | jhesketh, so you can do it gradually? | 10:15 |
jhesketh | odyssey4me: recheck because I'm curious to see if it'll pass, and I'll look into zuul | 10:15 |
jhesketh | yolanda: I /think/ so if I do the stepping through with pauses etc | 10:16 |
jhesketh | yolanda: might take a while | 10:16 |
odyssey4me | jhesketh I think that I had one of these yesterday, but I saw the transitioning away from jenkins and thought it might be a transitional failure, so I rechecked and it worked | 10:16 |
odyssey4me | I'll do the same here | 10:16 |
openstackgerrit | Thomas Goirand proposed openstack-infra/project-config: Fix upload path to tarballs.o.o for packaging https://review.openstack.org/331008 | 10:16 |
yolanda | jhesketh, can I help somehow? i'll need your guidance for that, because I have not been very involved with that changes lately | 10:16 |
zigo | odyssey4me: Can you review that one liner path fix patch? | 10:17 |
jhesketh | yolanda: maybe... let me do a little more digging first :-) | 10:17 |
odyssey4me | zigo looking | 10:17 |
zigo | Cheers. | 10:17 |
openstackgerrit | Merged openstack-infra/system-config: Fix the destination for eavesdrop site https://review.openstack.org/330986 | 10:18 |
*** mrunge has quit IRC | 10:18 | |
yolanda | jhesketh, thanks | 10:18 |
*** Adri2000 has quit IRC | 10:18 | |
yolanda | i need to find some time to study about how zuul 2.5 works now, to be able to help | 10:18 |
odyssey4me | zigo I'd recommend linking to the eavesdrop logs referencing the conversation for anyone reviewing to provide context | 10:18 |
*** esikachev has joined #openstack-infra | 10:21 | |
openstackgerrit | Tony Breeds proposed openstack-infra/irc-meetings: Revert "No-op Change to force a re-publish" https://review.openstack.org/331010 | 10:21 |
odyssey4me | zigo FYI, as I understand it, the general idea with the macros is to provide a generalised facility for multiple projects to use | 10:22 |
jhesketh | tonyb: don't do that thing yet | 10:22 |
jhesketh | tonyb: we need to restart the zuul launchers for the new config to apply | 10:22 |
odyssey4me | zigo in the case where only one project is doing something you may not have to have a macro at all, and can probably just implement the publisher in your job definition instead | 10:22 |
jhesketh | tonyb: after that your revert will be a good test to check it's actually working | 10:22 |
jhesketh | tonyb: but at the moment we're not ready | 10:23 |
tonyb | jhesketh: okay | 10:23 |
odyssey4me | zigo let me take a peek at your job definition | 10:23 |
*** ociuhandu has quit IRC | 10:23 | |
tonyb | jhesketh: feel free to +W that revert once the restart has happened if I'm not around | 10:23 |
jhesketh | tonyb: thanks | 10:23 |
*** e0ne has quit IRC | 10:24 | |
*** roxanaghe has joined #openstack-infra | 10:24 | |
jhesketh | yolanda: I'm going to gracefully restart zl01' | 10:27 |
*** tosky has joined #openstack-infra | 10:27 | |
*** mrunge has joined #openstack-infra | 10:28 | |
*** roxanaghe has quit IRC | 10:29 | |
*** Daviey has quit IRC | 10:29 | |
*** mohankumar has joined #openstack-infra | 10:29 | |
*** Daviey has joined #openstack-infra | 10:29 | |
*** e0ne has joined #openstack-infra | 10:29 | |
odyssey4me | zigo I've provided some feedback. | 10:31 |
odyssey4me | (in the review) | 10:31 |
*** amitgandhinz has joined #openstack-infra | 10:32 | |
*** ramishra has quit IRC | 10:36 | |
*** amitgandhinz has quit IRC | 10:37 | |
*** ramishra has joined #openstack-infra | 10:37 | |
*** gouthamr_ is now known as gouthamr | 10:40 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard: Allow tags to be filtered by name https://review.openstack.org/331027 | 10:41 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/system-config: OpenStack-CI Apt mirror for Ubuntu Cloud Archive https://review.openstack.org/330152 | 10:41 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Make the Tags resource look for existing tags to filter by https://review.openstack.org/331030 | 10:43 |
*** ociuhandu has joined #openstack-infra | 10:43 | |
*** ramishra has quit IRC | 10:45 | |
*** permalac has joined #openstack-infra | 10:47 | |
*** rossella_s has quit IRC | 10:47 | |
*** e0ne has quit IRC | 10:47 | |
*** sree has joined #openstack-infra | 10:48 | |
*** ramishra has joined #openstack-infra | 10:49 | |
*** AJaeger has joined #openstack-infra | 10:50 | |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/infra-ansible: Make services optional https://review.openstack.org/331036 | 10:50 |
*** ramishra has quit IRC | 10:50 | |
*** mescanef has joined #openstack-infra | 10:51 | |
*** dizquierdo has quit IRC | 10:53 | |
*** hogepodge has joined #openstack-infra | 10:54 | |
openstackgerrit | Vladyslav Drok proposed openstack-infra/project-config: Do not run any grenade jobs on branches older than mitaka https://review.openstack.org/331046 | 10:56 |
*** ramishra has joined #openstack-infra | 10:56 | |
*** binbincong has quit IRC | 10:57 | |
*** binbincong has joined #openstack-infra | 10:57 | |
AJaeger | infra-root, there'S since 10 hours a change in the post queue for openstack-manuals - two out of the 4 jobs are still queued. Could anybody check these, please? | 10:58 |
*** Adri2000 has joined #openstack-infra | 10:59 | |
yolanda | AJaeger, let me see logs. Unfortunately, i cannot help so much with new zuul-launcher, until I get more knowledge about it | 11:00 |
*** sree has quit IRC | 11:01 | |
AJaeger | yolanda: might as well be a bug - so feel free to wait for jeblair or pabelanger to check.... Just wanted to point it out... | 11:02 |
openstackgerrit | Vladyslav Drok proposed openstack-infra/project-config: Do not run any grenade jobs on mitaka and older branches https://review.openstack.org/331046 | 11:03 |
*** eezhova has quit IRC | 11:03 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Make the Tags resource look for existing tags to filter by https://review.openstack.org/331030 | 11:03 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/system-config: OpenStack-CI Apt mirror for MariaDB https://review.openstack.org/307831 | 11:04 |
*** andreas_s_ has quit IRC | 11:04 | |
*** dtantsur|bbl is now known as dtantsur | 11:04 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Use tag icon when tags are used as filter criteria https://review.openstack.org/331054 | 11:06 |
yolanda | AJaeger, i see nodepool working fine, picking jobs. No stale ready nodes, no errors in log | 11:07 |
openstackgerrit | Vladyslav Drok proposed openstack-infra/project-config: Do not run any grenade jobs on mitaka and older branches https://review.openstack.org/331046 | 11:08 |
*** dizquierdo has joined #openstack-infra | 11:08 | |
openstackgerrit | Merged openstack-infra/project-config: Add a build for the gerrit its-storyboard plugin https://review.openstack.org/330796 | 11:10 |
*** sree has joined #openstack-infra | 11:13 | |
*** linuxmaniac has quit IRC | 11:15 | |
*** linuxmaniac has joined #openstack-infra | 11:15 | |
*** esikachev has quit IRC | 11:21 | |
*** esikachev has joined #openstack-infra | 11:22 | |
*** thorst_ has joined #openstack-infra | 11:23 | |
*** saravana has quit IRC | 11:23 | |
*** kzaitsev_mb has joined #openstack-infra | 11:25 | |
*** roxanaghe has joined #openstack-infra | 11:25 | |
*** amoralej is now known as amoralej|lunch | 11:26 | |
*** kzaitsev_mb has quit IRC | 11:29 | |
*** roxanaghe has quit IRC | 11:29 | |
*** hashar is now known as hasharAway | 11:30 | |
zigo | odyssey4me: Replied. It will be used by other repos. | 11:33 |
*** amitgandhinz has joined #openstack-infra | 11:33 | |
*** kzaitsev_mb has joined #openstack-infra | 11:35 | |
*** aysyd has joined #openstack-infra | 11:35 | |
*** ldnunes has joined #openstack-infra | 11:37 | |
*** amitgandhinz has quit IRC | 11:38 | |
*** lucasagomes is now known as lucas-hungry | 11:39 | |
*** sree__ has joined #openstack-infra | 11:41 | |
*** sree has quit IRC | 11:42 | |
*** esikachev has quit IRC | 11:43 | |
*** gouthamr has quit IRC | 11:43 | |
*** DevBox has joined #openstack-infra | 11:44 | |
*** esikachev has joined #openstack-infra | 11:45 | |
pabelanger | morning | 11:46 |
*** zhurong has joined #openstack-infra | 11:47 | |
*** sree has joined #openstack-infra | 11:47 | |
*** rcernin has joined #openstack-infra | 11:47 | |
*** Jeffrey4l has quit IRC | 11:48 | |
*** zhurong has quit IRC | 11:48 | |
*** Jeffrey4l has joined #openstack-infra | 11:48 | |
*** esikachev has quit IRC | 11:50 | |
*** gouthamr has joined #openstack-infra | 11:50 | |
*** sree__ has quit IRC | 11:51 | |
*** eezhova has joined #openstack-infra | 11:52 | |
*** eharney has joined #openstack-infra | 11:55 | |
*** DevBox has quit IRC | 11:56 | |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/puppet-openstackci: Add puppetdb class https://review.openstack.org/328228 | 11:58 |
*** jsavak has joined #openstack-infra | 11:58 | |
jhesketh | yolanda: so zl01 restarted gracefully but it took a while... I'm going to start zl02, but I don't think I'll get through them all tonight... the fix for eavesdrop isn't critical though so I can pick it up tomorrow | 11:59 |
pabelanger | jhesketh: this should be of interest: https://review.openstack.org/#/c/330184/ | 12:00 |
jhesketh | Morning pabelanger :-) | 12:01 |
*** psilvad has joined #openstack-infra | 12:01 | |
pabelanger | jhesketh: which fix are you reverting too? | 12:01 |
jhesketh | pabelanger: huh, that is handy... I was about to write my own. Thakns! | 12:01 |
jhesketh | pabelanger: https://review.openstack.org/#/c/330986/ | 12:01 |
*** gouthamr_ has joined #openstack-infra | 12:02 | |
*** ramishra has quit IRC | 12:02 | |
*** gouthamr_ has quit IRC | 12:02 | |
*** gouthamr_ has joined #openstack-infra | 12:03 | |
openstackgerrit | Merged openstack-infra/release-tools: fix the pattern matching in the constraint update logic https://review.openstack.org/330542 | 12:03 |
pabelanger | jhesketh: Ah, ya. Didn't realize we'd need to graceful restart zuul to pick up new conf changes | 12:03 |
ikhudoshyn | AJaeger: aroung? | 12:04 |
jhesketh | pabelanger: yeah, we could look at reloading zuul.conf on reconfigure | 12:04 |
pabelanger | jhesketh: Ya, about to say the same. Seems like a good things to add | 12:04 |
*** gouthamr_ has quit IRC | 12:04 | |
jhesketh | pabelanger: yeah I'll work on that | 12:04 |
*** markvoelker has joined #openstack-infra | 12:04 | |
*** gouthamr_ has joined #openstack-infra | 12:04 | |
*** dmk0202 has quit IRC | 12:05 | |
*** gouthamr has quit IRC | 12:05 | |
pabelanger | jhesketh: nice | 12:05 |
*** gouthamr_ is now known as gouthamr | 12:05 | |
jhesketh | pabelanger: small nit on your playbook if you want to fix it | 12:05 |
pabelanger | jhesketh: sure, looking | 12:06 |
*** weshay has joined #openstack-infra | 12:06 | |
*** pradk has joined #openstack-infra | 12:06 | |
*** pradk has quit IRC | 12:07 | |
*** psilvad has quit IRC | 12:07 | |
*** e0ne has joined #openstack-infra | 12:07 | |
*** ramishra has joined #openstack-infra | 12:07 | |
*** mtanino has joined #openstack-infra | 12:07 | |
AJaeger | ikhudoshyn: now yes, what's up? | 12:08 |
*** sree has quit IRC | 12:08 | |
*** rfolco has joined #openstack-infra | 12:08 | |
AJaeger | yolanda: thanks for looking. | 12:08 |
ikhudoshyn | AJaeger: it's about the patch, https://review.openstack.org/#/c/328249/. It's in a merge conflict | 12:08 |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Add playbook to graceful restart zuul-launchers https://review.openstack.org/330184 | 12:08 |
AJaeger | pabelanger: is the openstack-manuals job in the post queue something for jeblair to check? | 12:08 |
ikhudoshyn | i'm about to rebase it, yet I promised you not to touch it | 12:08 |
AJaeger | ikhudoshyn: go ahead and rebase ;) | 12:09 |
ikhudoshyn | AJaeger: tnx) | 12:09 |
pabelanger | jhesketh: odd, it didn't have the latest version in the review. I've updated it to use async for the wait_for command | 12:09 |
pabelanger | AJaeger: Sure, I can look | 12:09 |
openstackgerrit | Illia Khudoshyn proposed openstack-infra/project-config: Add Rally job against deployment with existing users https://review.openstack.org/328249 | 12:09 |
ikhudoshyn | AJaeger: submitted new one ^^ | 12:10 |
jhesketh | pabelanger: are you sure we need the timeout on the wait_for? won't the async timeout fail and then the any-errors-fatal kicks in? | 12:11 |
*** javeriak has quit IRC | 12:11 | |
pabelanger | jhesketh: Ya, I think we need to setup a timeout for it too, otherwise it defaults to 30 seconds. But I agree, I'd be interested to see if that is actually the case. When I tested locally, the syntax was valid | 12:12 |
pabelanger | jhesketh: I should ask in #ansible | 12:12 |
*** javeriak has joined #openstack-infra | 12:12 | |
*** javeriak has quit IRC | 12:12 | |
*** pradk has joined #openstack-infra | 12:13 | |
jhesketh | pabelanger: ah okay.. | 12:13 |
pabelanger | AJaeger: http://paste.openstack.org/show/516834/ | 12:13 |
pabelanger | thats from zuul.o.o | 12:13 |
pabelanger | I think we recently fixed that | 12:13 |
jhesketh | pabelanger: yeah, there's a default of 300 http://docs.ansible.com/ansible/wait_for_module.html | 12:13 |
*** maishsk has quit IRC | 12:13 | |
openstackgerrit | Merged openstack-infra/storyboard-webclient: New Favicon https://review.openstack.org/327639 | 12:14 |
pabelanger | AJaeger: jeblair: I believe we need to restart zuul.o.o to fix the openstack-manuals post pipeline | 12:14 |
jhesketh | pabelanger: I'm doing graceful on zl02 | 12:15 |
pabelanger | jhesketh: okay | 12:15 |
pabelanger | AJaeger: https://review.openstack.org/#/c/327274/ was the fix | 12:15 |
jhesketh | pabelanger: yeah, that'll need a restart :-( | 12:15 |
AJaeger | pabelanger: thanks for checking | 12:16 |
pabelanger | AJaeger: np | 12:16 |
*** ddieterly has joined #openstack-infra | 12:16 | |
*** kzaitsev_mb has quit IRC | 12:17 | |
*** rhallisey has joined #openstack-infra | 12:17 | |
jhesketh | pabelanger: so the reconfigure for launchers is currently about reconfiguring jjb... should we provide two different types of reconfigures? the current one for jjb and anotehr one for launcher config (ie zuul.conf)? | 12:18 |
jhesketh | otherwise do them both at once maybe? | 12:18 |
pabelanger | jhesketh: I think we could get away with doing 2 zuul-launchers at a time for restarts, but should wait to see what other infra-roots say. I've see each zuul-launcher top out at 150 jobs each, so with 6 online we should be able to do 900 concurrent jobs | 12:18 |
*** jsavak has quit IRC | 12:18 | |
jhesketh | pabelanger: I agree and would be more keen to do so if it was a more urgent change, but this isn't too bad | 12:18 |
pabelanger | jhesketh: I think reconfigure is fine, but maybe jeblair has some thoughts | 12:19 |
openstackgerrit | osman proposed openstack-infra/jenkins-job-builder: Adds matrix restriction to trigger-builds plugin https://review.openstack.org/331112 | 12:19 |
jhesketh | okay, let me play with that.. | 12:19 |
*** dprince has joined #openstack-infra | 12:19 | |
pabelanger | jhesketh: ya, I think in the case of urgent changes, we have the ability to use stop. that will simply terminal all active jobs, and requeue them into zuul. Which simply means, the current patch will just longer over actually failing the job | 12:20 |
pabelanger | terminate* | 12:20 |
jhesketh | yep | 12:20 |
pabelanger | I like the fact, it will be hard for us to actually fail a job because of something we do to zuul-launcher configuration | 12:21 |
*** ramishra has quit IRC | 12:25 | |
*** roxanaghe has joined #openstack-infra | 12:26 | |
*** ddieterly has quit IRC | 12:26 | |
jhesketh | :-) | 12:29 |
*** roxanaghe has quit IRC | 12:30 | |
AJaeger | ;) | 12:30 |
*** ramishra has joined #openstack-infra | 12:31 | |
*** kzaitsev_mb has joined #openstack-infra | 12:31 | |
openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add support for standard zuul signals to launcher https://review.openstack.org/331123 | 12:34 |
*** jsavak has joined #openstack-infra | 12:34 | |
jhesketh | pabelanger: first pass at reconfiguring ^ | 12:34 |
*** edmondsw has joined #openstack-infra | 12:34 | |
*** amitgandhinz has joined #openstack-infra | 12:34 | |
*** kushal has quit IRC | 12:34 | |
*** baoli has joined #openstack-infra | 12:34 | |
pabelanger | jhesketh: looking | 12:35 |
*** kushal has joined #openstack-infra | 12:35 | |
*** baoli_ has joined #openstack-infra | 12:36 | |
*** kzaitsev_mb has quit IRC | 12:36 | |
*** piet has joined #openstack-infra | 12:37 | |
openstackgerrit | Joshua Hesketh proposed openstack-infra/puppet-zuul: Reconfigure via sending a HUP https://review.openstack.org/331127 | 12:37 |
openstackgerrit | Merged openstack-infra/release-tools: treat missing metadata as a warning not an error https://review.openstack.org/330764 | 12:38 |
*** jsavak has quit IRC | 12:38 | |
openstackgerrit | Raildo Mascena proposed openstack-infra/project-config: Adds ironic v3 only job https://review.openstack.org/304306 | 12:39 |
*** amitgandhinz has quit IRC | 12:39 | |
*** jsavak has joined #openstack-infra | 12:39 | |
openstackgerrit | Raildo Mascena proposed openstack-infra/project-config: Adds ironic v3 only job https://review.openstack.org/304306 | 12:40 |
*** rlandy has joined #openstack-infra | 12:40 | |
*** alaski is now known as lascii | 12:40 | |
*** baoli has quit IRC | 12:40 | |
pabelanger | jhesketh: left a comment | 12:41 |
*** kzaitsev_mb has joined #openstack-infra | 12:41 | |
*** baoli_ has quit IRC | 12:41 | |
*** kdas__ has joined #openstack-infra | 12:42 | |
phschwartz | So I am running nodepool on a network that is in a private scope(192.168). I order to not try and get floating ip's assigned (we were out of floatings) I changed the networks name to public. | 12:42 |
phschwartz | How bad will it make things die if I rename the network since I now have floating ip's | 12:42 |
phschwartz | And of course change my nodepool config | 12:42 |
*** bswartz has joined #openstack-infra | 12:43 | |
*** Srinu has quit IRC | 12:44 | |
pabelanger | phschwartz: I wouldn't expect running nodes to be affected. We recently did that to OSIC in nodepool.yaml and didn't need to schedule an outage | 12:44 |
jhesketh | pabelanger: thanks, replied | 12:44 |
phschwartz | pabelanger: perfect. I can do the update then :) | 12:44 |
*** kushal has quit IRC | 12:44 | |
*** kdas__ is now known as kushal | 12:45 | |
*** kushal has joined #openstack-infra | 12:46 | |
mordred | pabelanger: you figured out telnet:// handling in chrome? any chance you accidentally figured out firefox too? | 12:46 |
*** gordc has joined #openstack-infra | 12:46 | |
pabelanger | mordred: ya, only (badly) hacked chrome last night | 12:47 |
mordred | pabelanger: I spent some time trolling the internet and so far have learned no useful information | 12:48 |
*** ramishra has quit IRC | 12:48 | |
*** shahid__ has quit IRC | 12:49 | |
*** pradk has quit IRC | 12:50 | |
openstackgerrit | osman proposed openstack-infra/jenkins-job-builder: Adds matrix restriction to trigger-builds plugin https://review.openstack.org/331112 | 12:50 |
*** lucas-hungry is now known as lucasagomes | 12:50 | |
pabelanger | mordred: if you can get firefox to use xdg-open for telnet://, I think you could use the same logic I do with chrome. | 12:51 |
pabelanger | will give a test in a few minutes | 12:51 |
jhesketh | pabelanger: I've got to head off... zl02 is shutting down still, could you please bring it back up once its done? | 12:51 |
pabelanger | jhesketh: sure | 12:51 |
jhesketh | only 10 more nodes to go | 12:52 |
openstackgerrit | Merged openstack-infra/storyboard-webclient: Add a way to make time-moment not use am-time-ago https://review.openstack.org/329478 | 12:52 |
*** maishsk has joined #openstack-infra | 12:52 | |
jhesketh | I can do the other launchers tomorrow | 12:52 |
jhesketh | pabelanger: thanks :-) | 12:52 |
*** ddieterly has joined #openstack-infra | 12:53 | |
*** ccamacho is now known as ccamacho|lunck | 12:53 | |
*** matt-borland has joined #openstack-infra | 12:53 | |
*** ccamacho|lunck is now known as ccamacho|lunch | 12:54 | |
jhesketh | Night all! | 12:55 |
AJaeger | good night, jhesketh ! | 12:55 |
phschwartz | is there a command to remove 1 job from zuul? I have never had a need to do it before, but it looks like an abandon'ed review has borked my check queue as it is enqueue without a jenkins job to run | 12:59 |
*** psachin has quit IRC | 13:00 | |
pabelanger | mordred: got it working, let me get directions | 13:00 |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/puppet-openstackci: Add puppetdb class https://review.openstack.org/328228 | 13:01 |
*** dizquierdo has quit IRC | 13:02 | |
phschwartz | hmm, puppetdb added to openstackci... Interesting. | 13:02 |
rcarrillocruz | very rough | 13:02 |
rcarrillocruz | i'm wip'ing | 13:03 |
rcarrillocruz | i need systtem-config module to be bumped to 4.1 first | 13:03 |
rcarrillocruz | cos that version exposes ssl params | 13:03 |
pabelanger | mordred: https://etherpad.openstack.org/p/firefox-telnet _should_ work | 13:03 |
*** amoralej|lunch is now known as amoralej | 13:03 | |
*** trown|outtypewww is now known as trown | 13:04 | |
*** zhs_ss has quit IRC | 13:05 | |
*** inc0 has joined #openstack-infra | 13:07 | |
mordred | pabelanger: woot. that worked - I was missing expose = False before | 13:07 |
AJaeger | dims: could you review 329440, please? I wonder whether all projects need these oslo-jobs jobs now - or what's the proper set of projects to run them | 13:08 |
*** zhurong has joined #openstack-infra | 13:08 | |
pabelanger | mordred: nice | 13:09 |
EmilienM | it seems like TripleO CI is really down | 13:09 |
fungi | phschwartz: https://review.openstack.org/95035 but i believe it still needs some polish | 13:09 |
*** mdrabe has joined #openstack-infra | 13:09 | |
mordred | pabelanger: we should maybe write this down somewhere for people | 13:09 |
mordred | pabelanger: infra-manual perhaps? | 13:09 |
fungi | EmilienM: they were relocating the hardware, right? has that started yet? | 13:09 |
EmilienM | fungi: ok, I missed some info here thx | 13:10 |
*** xyang1 has joined #openstack-infra | 13:10 | |
pabelanger | mordred: Ya, we likely should do that. | 13:10 |
*** thiagop has joined #openstack-infra | 13:10 | |
phschwartz | fungi: I will see if I can get that rolling again, till then, I take it there is no way to remove the borked job, right? | 13:10 |
fungi | phschwartz: restart zuul? | 13:11 |
mordred | pabelanger: I'll start a patch | 13:11 |
phschwartz | fungi: that will make us loose all queued behind the borked one which I was trying to prevent | 13:11 |
EmilienM | fungi: I'm not sure it's related in fact, it fails in the middle of overcloud deployment | 13:11 |
openstackgerrit | Merged openstack-infra/storyboard-webclient: Rework timeline events into a timeline https://review.openstack.org/327231 | 13:11 |
EmilienM | fungi: but nevermind, probably something on our side | 13:11 |
*** Hal has joined #openstack-infra | 13:11 | |
*** ddieterly has quit IRC | 13:11 | |
*** kgiusti has joined #openstack-infra | 13:12 | |
*** Hal is now known as Guest28113 | 13:12 | |
fungi | EmilienM: oh, sorry, i interpreted "down" in a more literal sense | 13:12 |
EmilienM | fungi: sorry | 13:12 |
pabelanger | EmilienM: fungi: agreed, I still see tripleo-centos-7 nodes online | 13:13 |
AJaeger | EmilienM, fungi: See https://review.openstack.org/#/c/330514/ - so there's a planned move | 13:13 |
EmilienM | AJaeger: can we land it? | 13:14 |
EmilienM | slagle, dprince ^ is it ok to switch to rh2 ? ^ | 13:14 |
AJaeger | https://review.openstack.org/#/c/330833/ is the system-config change that needs to land first. fungi, pabelanger, do you want to review? | 13:15 |
dprince | EmilienM: I think that is a question for derekh. I believe he has good results on that cloud but unclear to me that switching today (when derek is on PTO) is the plan | 13:17 |
dprince | sorry, best plan | 13:17 |
EmilienM | dprince: well, tripleo.org/cistatus.html | 13:17 |
EmilienM | there is a problem atm ^ | 13:17 |
EmilienM | dprince: is it related to the move or something? | 13:17 |
dprince | EmilienM: switching to an entirely new cloud because we have a CI failure on the old one may break more than it fixes :) | 13:18 |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/puppet-openstackci: Add puppetdb class https://review.openstack.org/328228 | 13:18 |
dprince | EmilienM: or not. let me look at the failures | 13:18 |
EmilienM | dprince: http://logs.openstack.org/85/330785/2/check-tripleo/gate-tripleo-ci-centos-7-nonha/18f9b94/logs/postci.txt.gz#_2016-06-17_04_01_58_000 | 13:18 |
EmilienM | AJaeger: can we hold this one please? https://review.openstack.org/#/c/330514/ | 13:21 |
EmilienM | AJaeger: until dprince and I find out why our CI is broken first - thanks a lot | 13:21 |
AJaeger | EmilienM: sure... | 13:22 |
EmilienM | thanks | 13:22 |
AJaeger | EmilienM: I set a WIP | 13:22 |
EmilienM | perfect | 13:22 |
*** akshai has joined #openstack-infra | 13:23 | |
rcarrillocruz | so folks | 13:23 |
rcarrillocruz | any recommendation for a chromium extension for telnet | 13:24 |
rcarrillocruz | mordred , pabelanger ^ | 13:24 |
rcarrillocruz | you using it already via browser ? | 13:24 |
pabelanger | rcarrillocruz: https://etherpad.openstack.org/p/google-telnet | 13:24 |
pabelanger | mind you, pretty hacky | 13:24 |
rcarrillocruz | thx | 13:24 |
openstackgerrit | Luigi Toscano proposed openstack-infra/project-config: Enable Tempest test job also for sahara-tests https://review.openstack.org/329590 | 13:26 |
fungi | chromium doesn't just have a protocol to application mapping you can configure? | 13:26 |
*** roxanaghe has joined #openstack-infra | 13:27 | |
*** binbincong has quit IRC | 13:27 | |
fungi | (map telnet protocol to "xterm -c nc $host $port" or whatever?) | 13:27 |
*** abregman|afk has joined #openstack-infra | 13:27 | |
Shrews | doesn't look like the launchers are running with all the latest goodies | 13:27 |
pabelanger | fungi: it might, but that's what I got it working last night. It was pretty late | 13:27 |
fungi | Shrews: "latest goodies" meanining job config updates? zuul patches? something else? | 13:30 |
mordred | fungi: well, something has to split the url into $host $port | 13:30 |
Shrews | fungi: zuul patches | 13:30 |
fungi | mordred: fair point. wrapper script to |sed 's/:/ /' | 13:30 |
mordred | yah | 13:31 |
fungi | (mailercode excuse) | 13:31 |
*** inc0 has quit IRC | 13:31 | |
*** mriedem has joined #openstack-infra | 13:31 | |
*** roxanaghe has quit IRC | 13:31 | |
*** inc0 has joined #openstack-infra | 13:31 | |
*** jamesmcarthur has joined #openstack-infra | 13:32 | |
fungi | Shrews: all have "zuul==2.1.1.dev321 # git sha aaa852e" installed but maybe some need restarts? | 13:33 |
fungi | i saw jhesketh mention something about working through graceful restarts before i awoke | 13:33 |
pabelanger | Shrews: zl01 was restarted my jhesketh this morning | 13:35 |
dims | AJaeger : asked a question and added Josh to the review | 13:35 |
pabelanger | zl02 is in graceful mode now | 13:35 |
*** mtanino has quit IRC | 13:35 | |
AJaeger | thanks, dims | 13:36 |
openstackgerrit | Merged openstack-infra/system-config: Add rh2 cloud credentials https://review.openstack.org/330833 | 13:37 |
*** ddieterly has joined #openstack-infra | 13:38 | |
EmilienM | pabelanger: we noticed that tripleo jobs start to fail when decommissionning all jenkins slaves | 13:38 |
*** nwkarsten has joined #openstack-infra | 13:39 | |
EmilienM | we're wondering if some ENV used by zuul or something might break us | 13:39 |
pabelanger | EmilienM: Hmm, I thought I see tripleo-ci passing on zuul-launcher before | 13:39 |
dprince | yes, it would seem so. Does the zuul runner perhaps have different timeouts, env variables or something? | 13:40 |
pabelanger | seen* | 13:40 |
EmilienM | this is an example of console.log of failing job: http://logs.openstack.org/94/330694/4/check-tripleo/gate-tripleo-ci-centos-7-nonha/8fff5e5/console.html | 13:40 |
*** maishsk_ has joined #openstack-infra | 13:40 | |
*** rbrndt has joined #openstack-infra | 13:40 | |
*** mixos has joined #openstack-infra | 13:41 | |
*** dansmith is now known as superdan | 13:41 | |
pabelanger | EmilienM: dprince: I don't see the job timing out, but simply failing: http://logs.openstack.org/94/330694/4/check-tripleo/gate-tripleo-ci-centos-7-nonha/8fff5e5/_zuul_ansible/logs/ansible.log | 13:41 |
pabelanger | atleast from zuul-launcher POV | 13:41 |
EmilienM | http://logs.openstack.org/94/330694/4/check-tripleo/gate-tripleo-ci-centos-7-nonha/8fff5e5/console.html#_2016-06-16_21_22_37_284281 | 13:42 |
EmilienM | is it a big deal ^? | 13:42 |
dprince | pabelanger: true, just noticed the failed jobs running faster | 13:42 |
pabelanger | EmilienM: no, we are still using jenkins, but will be switching to zuul in the future | 13:42 |
dprince | pabelanger: perhaps new ENV variables are changing our job behavior somehow? | 13:42 |
pabelanger | dprince: oh, that is possible. We aren't using java anymore to run the process on the remote host. Commands are directly over SSH | 13:43 |
*** amitgandhinz has joined #openstack-infra | 13:43 | |
*** maishsk has quit IRC | 13:43 | |
*** maishsk_ is now known as maishsk | 13:43 | |
pabelanger | dprince: possible, but I am sure the I've seen tripleo CI passing on zuul-launcher over the last week | 13:43 |
dprince | pabelanger: link to the job? | 13:44 |
*** _ari_ has joined #openstack-infra | 13:44 | |
dprince | pabelanger: perhaps that was experimental and we there is a slightly different setting that made it work or something | 13:44 |
EmilienM | dprince: did timeout changed ? | 13:45 |
EmilienM | http://logs.openstack.org/94/330694/4/check-tripleo/gate-tripleo-ci-centos-7-nonha/8fff5e5/console.html#_2016-06-16_21_32_45_024721 | 13:45 |
EmilienM | on a successful job: http://logs.openstack.org/14/329714/2/check-tripleo/gate-tripleo-ci-centos-7-nonha/5f84c27/console.html#_2016-06-16_19_21_13_882 | 13:45 |
*** HeOS has quit IRC | 13:45 | |
EmilienM | from 153 to 97 min? | 13:45 |
*** piet has quit IRC | 13:45 | |
*** links has quit IRC | 13:46 | |
pabelanger | dprince: looking for a job | 13:47 |
*** hongbin has joined #openstack-infra | 13:47 | |
dprince | pabelanger: no worries if you can't find it. Just be good to have something to compare too | 13:47 |
*** amitgandhinz has quit IRC | 13:48 | |
pabelanger | EmilienM: there was an issue where build-timeout wasn't being setup properly under zuul. but I thought we fixed that | 13:48 |
*** piet has joined #openstack-infra | 13:48 | |
*** sarob has joined #openstack-infra | 13:48 | |
*** piet has quit IRC | 13:49 | |
*** dingyichen has quit IRC | 13:49 | |
*** PollyZ has joined #openstack-infra | 13:50 | |
pabelanger | zl02 restarted | 13:50 |
*** Goneri has joined #openstack-infra | 13:50 | |
*** e0ne has quit IRC | 13:51 | |
openstackgerrit | Alexander Evseev proposed openstack-infra/zuul: Use ssh-agent instead of ugly ssh-wrapper scripts https://review.openstack.org/331148 | 13:52 |
*** e0ne has joined #openstack-infra | 13:52 | |
EmilienM | it sounds like a timeout thing really | 13:52 |
*** rodrigods has quit IRC | 13:52 | |
*** rodrigods has joined #openstack-infra | 13:52 | |
EmilienM | when we used jenkins: Job timeout set to: 153 minutes | 13:52 |
EmilienM | and now Job timeout set to: 97 minutes | 13:53 |
EmilienM | looking at http://tripleo.org/cistatus.html | 13:53 |
pabelanger | maybe we're not exporting BUILD_TIMEOUT correctly now | 13:53 |
EmilienM | all jobs fail at ~ 1h30 (90 min) | 13:53 |
*** sarob has quit IRC | 13:53 | |
EmilienM | dprince: the thing is, I'm really surprised to see jobs > 1h30 | 13:53 |
EmilienM | dprince: nonha takes usually less time | 13:53 |
EmilienM | ~1h or so | 13:53 |
EmilienM | but looking at http://tripleo.org/cistatus.html it seems it took more recently | 13:54 |
*** armax has joined #openstack-infra | 13:54 | |
EmilienM | if you scroll down, you'll see some jobs taking 1h20 or so | 13:54 |
EmilienM | so we're a bit longer now, probably because we add more services etc | 13:54 |
EmilienM | pabelanger: I have no idea who we can increase the timeout in zuul | 13:54 |
*** zhurong has quit IRC | 13:55 | |
*** Goneri has quit IRC | 13:55 | |
*** pradk has joined #openstack-infra | 13:56 | |
*** zhurong has joined #openstack-infra | 13:56 | |
pabelanger | we should be passing BUILD_TIMEOUT into the env for our subprocess | 13:56 |
pabelanger | EmilienM: we use the build-timeout wrapper | 13:57 |
*** xarses_ has quit IRC | 13:57 | |
*** mriedem has quit IRC | 13:58 | |
*** mriedem has joined #openstack-infra | 13:59 | |
*** burgerk has joined #openstack-infra | 13:59 | |
openstackgerrit | Paul Belanger proposed openstack-infra/tripleo-ci: DO NOT MERGE https://review.openstack.org/331151 | 13:59 |
*** zhurong has quit IRC | 14:00 | |
EmilienM | pabelanger: we'll know in ~12 min | 14:02 |
EmilienM | pabelanger: the time to run this script | 14:02 |
openstackgerrit | Paul Belanger proposed openstack-infra/devstack-gate: DO NOT MERGE https://review.openstack.org/331156 | 14:03 |
*** PollyZ has quit IRC | 14:03 | |
*** inc0 has quit IRC | 14:04 | |
*** inc0 has joined #openstack-infra | 14:04 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config: Promote existing OSA non-voting Xenial tests to voting https://review.openstack.org/331157 | 14:06 |
*** ddieterly is now known as ddieterly[away] | 14:06 | |
*** sigmavirus24_ is now known as sigmavirus24 | 14:06 | |
*** zhurong has joined #openstack-infra | 14:07 | |
*** PollyZ has joined #openstack-infra | 14:07 | |
pabelanger | EmilienM: Yup, BUILD_TIMEOUT is not being passed properly http://logs.openstack.org/56/331156/1/check/gate-tempest-dsvm-full-mitaka-nv/1f7b5e0/console.html#_2016-06-17_14_06_18_270604 | 14:07 |
*** eezhova has quit IRC | 14:07 | |
openstackgerrit | Paul Belanger proposed openstack-infra/devstack-gate: DO NOT MERGE https://review.openstack.org/331156 | 14:07 |
EmilienM | damn | 14:08 |
EmilienM | pabelanger: nice catch | 14:08 |
pabelanger | EmilienM: checking to see why | 14:08 |
AJaeger | strange, what happend with https://review.openstack.org/#/c/328298/ ? | 14:09 |
openstackgerrit | Alexander Evseev proposed openstack-infra/zuul: Add ability to use SSH instead of HTTP to get refs https://review.openstack.org/246433 | 14:10 |
*** andreykurilin has joined #openstack-infra | 14:10 | |
andreykurilin | hi everyone! | 14:10 |
EmilienM | pabelanger: can it be in zuul/launcher/ansiblelaunchserver.py ? | 14:11 |
pabelanger | EmilienM: yes, this will be in zuul-launcher | 14:11 |
*** rbergeron has quit IRC | 14:11 | |
*** rbergeron has joined #openstack-infra | 14:11 | |
andreykurilin | Could someone look at novaclient's change ( https://review.openstack.org/#/c/291370/ )? It had W+1, but verify check was not launched. Rebase did not help. | 14:11 |
EmilienM | AJaeger: yeah I was looking at this one | 14:11 |
*** PollyZ has quit IRC | 14:12 | |
AJaeger | andreykurilin: could you remove the +A and add it back again? | 14:12 |
*** sdake has joined #openstack-infra | 14:12 | |
andreykurilin | AJaeger: done | 14:12 |
pabelanger | EmilienM: I think I see the issue | 14:12 |
AJaeger | andreykurilin: wrong solution, found it. The job is stacked on top of https://review.openstack.org/#/c/240277/5 is | 14:13 |
*** ddieterly[away] is now known as ddieterly | 14:13 | |
AJaeger | And as long as 240277 is not merged, it will not merge either... | 14:14 |
*** eezhova has joined #openstack-infra | 14:14 | |
andreykurilin | oh | 14:14 |
AJaeger | andreykurilin: see "Related Changes" in upper left corner for the stacked changes | 14:14 |
andreykurilin | AJaeger: yeah... I missed it sometimes in new gerrit ui:( | 14:14 |
AJaeger | Yeah ;( | 14:14 |
*** Jeffrey4l has quit IRC | 14:15 | |
andreykurilin | AJaeger: thanks | 14:15 |
*** zul has quit IRC | 14:16 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul: Fix typo with timeout wrappers https://review.openstack.org/331170 | 14:16 |
*** zul has joined #openstack-infra | 14:16 | |
EmilienM | what? | 14:17 |
EmilienM | the wrapper name is build-timeout | 14:17 |
EmilienM | pabelanger: ^ | 14:17 |
openstackgerrit | Merged openstack-infra/devstack-gate: Set read/search perms recursively https://review.openstack.org/330835 | 14:17 |
*** jamesmcarthur has quit IRC | 14:17 | |
*** amitgandhinz has joined #openstack-infra | 14:17 | |
pabelanger | EmilienM: I am testing | 14:17 |
*** amitgandhinz has quit IRC | 14:18 | |
*** jamesmcarthur has joined #openstack-infra | 14:18 | |
*** jamesmcarthur has quit IRC | 14:18 | |
*** amitgandhinz has joined #openstack-infra | 14:18 | |
*** xarses_ has joined #openstack-infra | 14:18 | |
EmilienM | pabelanger: ok I think you're right, we're already in 'wrappers' and we're looking for the next key, not the wrapper name | 14:20 |
fungi | yep, it's in wrappers[build-timeout][timeout] | 14:20 |
*** hemanthm|afk is now known as hemanthm | 14:20 | |
pabelanger | I've abandoned for now | 14:20 |
pabelanger | because we are setting up timeout some how | 14:20 |
pabelanger | want to understand that first | 14:21 |
*** zul_ has joined #openstack-infra | 14:21 | |
fungi | pabelanger: no, i think your change looks correct | 14:21 |
*** mtanino has joined #openstack-infra | 14:21 | |
fungi | we're doing "for wrapper in wrappers..." | 14:21 |
EmilienM | yes, I think it's ok too | 14:21 |
fungi | so wrapper inside the loop is wrappers[built-timeout] already, hence we're looking for wrapper[timeout] | 14:22 |
*** ccamacho|lunch is now known as ccamacho | 14:22 | |
pabelanger | my concern is, ANSIBLE_DEFAULT_TIMEOUT is 7200 | 14:22 |
*** zul has quit IRC | 14:23 | |
EmilienM | it's 120 min | 14:23 |
pabelanger | but I see -e timeout=10800 -v in our process | 14:23 |
pabelanger | where is 10800 coming from | 14:23 |
fungi | oh, er, your change isn't quite correct, but i think the code there is wrong nonetheless | 14:23 |
pabelanger | ya | 14:23 |
EmilienM | I found where it comes from | 14:23 |
pabelanger | something is a little weird, let me see if I can better understand | 14:24 |
EmilienM | http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/elements/testenv-worker/testenv-worker#n180 | 14:24 |
*** maishsk has quit IRC | 14:24 | |
pabelanger | EmilienM: no, this in on zuul-launcher | 14:24 |
EmilienM | also http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/testenv-client#n141 | 14:24 |
*** kushal has quit IRC | 14:26 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config: Promote existing OSA non-voting CentOS-7 tests to voting https://review.openstack.org/331191 | 14:27 |
pabelanger | I think I see the issue | 14:27 |
*** roxanaghe has joined #openstack-infra | 14:28 | |
fungi | basically, line 1089 is looking for wrappers[built-timeout][build-timeout] which won't exist and ends up with build_timeout set to {} | 14:28 |
pabelanger | is it possible that code is looking at the job_template wrappers section? | 14:28 |
pabelanger | which does have a wrappers: build-timeout | 14:28 |
jeblair | o/ | 14:28 |
pabelanger | and a timeout var | 14:28 |
*** Jeffrey4l has joined #openstack-infra | 14:28 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config: Promote existing OSA non-voting CentOS-7 tests to voting https://review.openstack.org/331191 | 14:28 |
pabelanger | but, does not have timeout-var | 14:28 |
fungi | pabelanger: it's looking at the job, which will be formed from the job-template, which yes | 14:28 |
pabelanger | jeblair: oah | 14:28 |
EmilienM | jeblair: hey, we're looking https://bugs.launchpad.net/tripleo/+bug/1593740 | 14:29 |
openstack | Launchpad bug 1593740 in tripleo "CI jobs timeout at 97 minutes instead of 153" [Critical,Confirmed] - Assigned to Paul Belanger (pabelanger) | 14:29 |
pabelanger | fungi: ya, I was assuming it was some how using the wrapper in macro.yaml | 14:29 |
fungi | pabelanger: so anyway, i think it may make more sense to move the timeout processing out of that for loop entirely | 14:29 |
pabelanger | which is where timeout-var is setup | 14:29 |
*** flepied has quit IRC | 14:29 | |
fungi | wrappers = jjb_job.get('wrappers', []) | 14:30 |
openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/puppet-openstackci: Add puppetdb class https://review.openstack.org/328228 | 14:31 |
fungi | or something | 14:31 |
jeblair | /win 21 | 14:31 |
pabelanger | jjb_job['wrappers']['wrapper']['build-timeout']['wrappers] ? | 14:32 |
sc68cal | found the other irssi user | 14:32 |
*** david-lyle has joined #openstack-infra | 14:32 | |
fungi | pabelanger: no, wait, i'm brainparsing yaml incorrectly | 14:32 |
pabelanger | _shugs_ my brains can't parse | 14:32 |
AJaeger | jeblair, fungi, pabelanger : Could you restart zuul later today? We have some changes in post queue that have no jobs run - according to pabelanger this is fixed since last night and needs zuul restart | 14:32 |
*** roxanaghe has quit IRC | 14:32 | |
fungi | now i see wrappers is a list, whose elements are nested dicts, so i think the original code is correct | 14:33 |
*** e0ne has quit IRC | 14:34 | |
pabelanger | fungi: I think this is what it is parsing: http://paste.openstack.org/show/516872/ | 14:34 |
pabelanger | which lacks timeout-var | 14:34 |
*** PollyZ has joined #openstack-infra | 14:34 | |
jeblair | fungi, pabelanger: http://paste.openstack.org/show/516873/ produces timeout=600 for me | 14:34 |
fungi | basically one of the elements of the 'wrappers' list will be { 'build-timeout': { 'timeout': 1234 } } | 14:34 |
pabelanger | fungi: I was expecting: http://paste.openstack.org/show/516875/ | 14:34 |
dprince | pabelanger: so there is no switch back to Jenkins just for #tripleo jobs... while we sort this out option? | 14:35 |
*** e0ne has joined #openstack-infra | 14:35 | |
pabelanger | dprince: no, we've delete jenkins | 14:35 |
fungi | AJaeger: pabelanger: what was the fix merged for the jobs stuck queued in post? | 14:35 |
fungi | dprince: there are no jenkins servers | 14:35 |
dprince | boom | 14:35 |
fungi | dprince: the last of them were deleted yesterday | 14:35 |
dprince | got it | 14:35 |
pabelanger | fungi: yes, we need to restart zuul | 14:35 |
fungi | pabelanger: that's not what i asked | 14:36 |
fungi | AJaeger: pabelanger: what was the fix merged for the jobs stuck queued in post? | 14:36 |
*** zz_dimtruck is now known as dimtruck | 14:36 | |
*** zhurong has quit IRC | 14:36 | |
fungi | (change number or hyperlink, or project and title, something i can query) | 14:36 |
fungi | (git sha) | 14:37 |
AJaeger | fungi https://review.openstack.org/330725 | 14:37 |
fungi | was it "Ansible launcher: delay node assignment under load" ? | 14:37 |
AJaeger | fungi, sorry, wrong link ;( | 14:38 |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config: Add driver compliance checks to check pipeline https://review.openstack.org/331192 | 14:38 |
pabelanger | fungi: sorry, confused now. Is we are talking about post pipeline issue with manual jobs, I think it is hitting https://review.openstack.org/#/c/327274/ | 14:38 |
* AJaeger tries again https://review.openstack.org/#/c/327274/ | 14:38 | |
jeblair | why would that have caused something to get stuck in post? | 14:39 |
jeblair | oh i think i see how it could be possible | 14:39 |
jeblair | why do we think the tripleo problem is related to zuul launcher timeouts? | 14:40 |
AJaeger | have to leave now, might be back later... | 14:40 |
pabelanger | jeblair: I don't believe we are setting BUILD_TIMEOUT correctly in the env now | 14:41 |
jeblair | did anyone push up a test that echos $BUILD_TIMEOUT? | 14:41 |
pabelanger | yes | 14:42 |
*** dprince has quit IRC | 14:42 | |
pabelanger | 331156 is a quick test | 14:42 |
smcginnis | Has there been any talk/exploration of using containers for some gate tests? Seems like things like pylint and pep8 jobs could be a lot faster if they just all ran on one instance in their own containers. | 14:42 |
pabelanger | added env and echo | 14:42 |
*** bhavik has joined #openstack-infra | 14:42 | |
pabelanger | I currently believe, http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/launcher/ansiblelaunchserver.py#n1091 will always be None | 14:42 |
pabelanger | because I didn't properly understand our jjb_job.get('wrappers', []) code | 14:43 |
jeblair | oh i know the problem | 14:43 |
rcarrillocruz | smcginnis: there was, but nothing concrete... | 14:43 |
*** dprince has joined #openstack-infra | 14:43 | |
pabelanger | jeblair: great | 14:43 |
smcginnis | rcarrillocruz: OK, cool. Just something that popped in my head. :) | 14:43 |
fungi | smcginnis: those already run in a virtualenv. what more isolation is a container going to provide there? | 14:44 |
smcginnis | fungi: Just thinking of the cost of spinning up VM instances. Or are they in their own venv on one instance? | 14:44 |
*** ihrachys has quit IRC | 14:45 | |
*** baoli has joined #openstack-infra | 14:45 | |
fungi | smcginnis: we used to do that a long time ago. it turned out to be a lot more costly in sysadmin time to maintain those servers than it was to discard them and let projects run whatever they want in a test instead | 14:45 |
fungi | smcginnis: so you could certainly design a job that ran both unit tests and static analysis if you wanted | 14:45 |
fungi | pabelanger: i understand the wrappers loop now. i was misparsing the yaml for it in my mind | 14:46 |
fungi | pabelanger: and the code looks right to me | 14:46 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: set timeout env var in ms https://review.openstack.org/331199 | 14:46 |
jeblair | pabelanger, fungi: ^ | 14:46 |
*** flepied has joined #openstack-infra | 14:48 | |
*** jamesmcarthur has joined #openstack-infra | 14:48 | |
mordred | jeblair: yay for units! | 14:49 |
mordred | jeblair: at least we didn't mess up space flight due to metric/imperial | 14:49 |
rcarrillocruz | lol | 14:49 |
pabelanger | jeblair: I still have a concern that BUILD_TIMEOUT isn't actually in our env however | 14:49 |
pabelanger | is it possible line 1091 is always None? | 14:50 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config: Promote existing OSA experimental Xenial jobs to non-voting https://review.openstack.org/331203 | 14:50 |
*** hasharAway is now known as hashar | 14:52 | |
*** sorantis has quit IRC | 14:52 | |
*** mescanef has quit IRC | 14:52 | |
fungi | pabelanger: you have to look into jjb for that | 14:52 |
pabelanger | ya, that is what I am testing now | 14:53 |
*** ajmiller has quit IRC | 14:53 | |
*** ajmiller has joined #openstack-infra | 14:53 | |
*** krtaylor has joined #openstack-infra | 14:53 | |
*** ramishra has joined #openstack-infra | 14:54 | |
*** pcaruana has quit IRC | 14:54 | |
fungi | pabelanger: so looking at the macro, if we provide a build-timeout wrapper, we set timeout to whatever we passed as timeout in the job(-template) and then we set timeout-var to BUILD_TIMEOUT http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml#n875 | 14:54 |
fungi | pabelanger: oh, hrm, so thinking back through this, 331170 may be necessary after all | 14:55 |
fungi | jeblair: ^ | 14:55 |
fungi | our macro name is build-timeout, but the actual wrapper is called timeout | 14:56 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: ensure timeout values are the correct type https://review.openstack.org/331210 | 14:56 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config: Promote existing OSA experimental CentOS-7 jobs to non-voting https://review.openstack.org/331211 | 14:56 |
jeblair | fungi, pabelanger: without 331210 i was able to cause jobs to not run at all, but did not see them run successfully without a build_timeout var | 14:57 |
jeblair | it may be that we are still not setting it correctly, and once we figure it out, we would need 331210 | 14:57 |
*** inc0 has quit IRC | 14:57 | |
pabelanger | http://paste.openstack.org/show/516882/ | 14:57 |
jeblair | i will look into what fungi is suggesting by pointing my local copy at the full job set and introspecting | 14:57 |
*** flepied has quit IRC | 14:57 | |
*** inc0 has joined #openstack-infra | 14:57 | |
pabelanger | that is what wrappers contains | 14:57 |
pabelanger | which is the wrappers from the job_template | 14:58 |
pabelanger | not macros | 14:58 |
pabelanger | and we set BUILD_TIMEOUT in timeout-var | 14:58 |
fungi | pabelanger: is that from parsing the yaml directly, or echoing the expanded data structure from jjb? | 14:58 |
*** roxanaghe has joined #openstack-infra | 14:59 | |
pabelanger | fungi: Ya, echo. I did print self.jobs in loadLoads() | 14:59 |
pabelanger | no where in the jobs, is timeout-var or BUILD_TIMEOUT exposed | 15:00 |
pabelanger | self.jobs | 15:00 |
*** gouthamr has quit IRC | 15:01 | |
*** gouthamr has joined #openstack-infra | 15:01 | |
fungi | there was some work done to make jjb libified enough to be able to extract the parsed and expanded tree, but i wonder if the routine that substitutes builders and publishers with macros got moved while the routine which does the same for wrappers didn't | 15:01 |
fungi | will take a few minutes to reacquaint myself with that part of jjb | 15:02 |
*** vincentll has quit IRC | 15:02 | |
openstackgerrit | Merged openstack-infra/zuul: Ansible launcher: set timeout env var in ms https://review.openstack.org/331199 | 15:02 |
*** inc0 has quit IRC | 15:03 | |
*** roxanaghe has quit IRC | 15:03 | |
jeblair | fungi: oh, i think you've got it | 15:04 |
jeblair | fungi: i think we're missing wrapper expansion in zuul | 15:04 |
jeblair | fungi: (because jjb actually can't do the thing you just said) | 15:05 |
*** tesseract has quit IRC | 15:05 | |
fungi | that explains why i couldn't find the relocation i was expecting within jjb | 15:06 |
fungi | didn't remember that ended up reimplemented in zuul | 15:06 |
*** rcernin has quit IRC | 15:07 | |
pabelanger | okay cool | 15:07 |
*** dprince has quit IRC | 15:08 | |
*** amotoki has joined #openstack-infra | 15:09 | |
jeblair | that may be at issue, but i'm not convinced that's it yet | 15:09 |
*** dprince has joined #openstack-infra | 15:09 | |
jeblair | just looking at gate-tempest-dsvm-full -- it defines its own wrappers. it has a build-timeout wrapper with no var | 15:09 |
openstackgerrit | Dong Ma proposed openstack-infra/jenkins-job-builder: Use 'false' and 'true' in default docstrings https://review.openstack.org/331220 | 15:10 |
jeblair | iow, i believe that jobs from particular job template will correctly not have a timeout var set | 15:10 |
*** dprince has quit IRC | 15:11 | |
jeblair | fungi, pabelanger: oh, wait -- where are we on the 'build-timout' vs 'timeout' name? :) | 15:12 |
pabelanger | jeblair: confused :D | 15:12 |
pabelanger | As I understand it, we should be passing BUILD_TIMEOUT for every job that is using the build-timeout wrapper from macros.yaml | 15:13 |
pabelanger | I believe that was the functionality in jenkins | 15:13 |
jeblair | i think the wrapper is called timout, and the macro is called build-timeout | 15:14 |
pabelanger | I believe that is correct | 15:14 |
*** electrofelix has quit IRC | 15:15 | |
pabelanger | in the case of 'gate-dg-hook-dsvm' in jenkins/jobs/devstack-gate.yaml. I believe zuul-launcher is using that wrappers to extract our timeout and timeout-var. And because timeout-var is missing, we are never setting it in zuul-launcher | 15:16 |
jeblair | so i think we are not substituting the macros and we have the wrong wrapper name (weirdly, the combination of those things causes this to partially work -- just not setting the env var) | 15:16 |
pabelanger | definitely weirdness | 15:17 |
openstackgerrit | Julien Danjou proposed openstack-infra/project-config: Disable Tempest plugins gate for Telemetry in stable/liberty https://review.openstack.org/331225 | 15:19 |
fungi | jeblair: yes, i agree. pabelanger's 331170, your 331199 and a fix for expanding wrappers macros are all going to be needed to solve this | 15:20 |
*** sdake has quit IRC | 15:20 | |
*** flepied has joined #openstack-infra | 15:21 | |
*** mdrabe has quit IRC | 15:24 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: Fix timeout processing https://review.openstack.org/331210 | 15:25 |
*** afazekas|dentist is now known as afazekas | 15:25 | |
*** zeih has quit IRC | 15:25 | |
jeblair | pabelanger, fungi: ^ please look closely at that | 15:25 |
pabelanger | jeblair: looking | 15:26 |
fungi | jeblair: i think there's still a missing level | 15:27 |
fungi | jeblair: the parsed wrapper will actually be { 'timeout': { 'timeout': 12345 } } | 15:27 |
jeblair | fungi: ? | 15:27 |
*** mixos has quit IRC | 15:27 | |
*** vgridnev has joined #openstack-infra | 15:28 | |
fungi | at least looking at the macro we have | 15:28 |
jeblair | fungi: line 1089 is the 1st and 1092 is the 2nd | 15:28 |
vgridnev | AJaeger, could you please take another look on https://review.openstack.org/#/c/330125/ ? | 15:28 |
fungi | jeblair: d'oh, yep. misread | 15:28 |
pabelanger | this is what I am seeing now: http://paste.openstack.org/show/516890/ which looks better | 15:29 |
pabelanger | checking patch now | 15:29 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: Fix timeout processing https://review.openstack.org/331210 | 15:29 |
jeblair | fungi, pabelanger: that's just a style change ^ (dropping the None default values) | 15:30 |
fungi | yeah, .get() will default to None on its own | 15:30 |
fungi | i was just confirming that | 15:30 |
jeblair | we had a mixture of both styles together which is potentially confusing as to intent | 15:31 |
pabelanger | okay, I think that is correct | 15:31 |
pabelanger | seems to line up with what I am expecting now | 15:31 |
jeblair | i don't normally mind that sort of thing, but two lines next to each other doing the same thing different ways is weird | 15:31 |
jeblair | (and OrderedDict behaves the same way with .get()) | 15:31 |
pabelanger | jeblair: +2 | 15:32 |
*** rajinir has joined #openstack-infra | 15:32 | |
*** pgadiya_ has quit IRC | 15:32 | |
jeblair | i love that the input to the build timeout plugin is seconds and the output is ms | 15:33 |
*** dizquierdo has joined #openstack-infra | 15:34 | |
*** mriedem has quit IRC | 15:35 | |
*** andreww has joined #openstack-infra | 15:35 | |
*** esikachev has joined #openstack-infra | 15:36 | |
*** xarses_ has quit IRC | 15:36 | |
*** hashar is now known as hasharAway | 15:36 | |
*** sree has joined #openstack-infra | 15:36 | |
fungi | i thought the input to the build timeout plugin was in minutes? | 15:37 |
jeblair | fungi: er, yes it is :) | 15:37 |
fungi | (we have timeout * 60 in there) | 15:37 |
jeblair | even better | 15:37 |
*** vhosakot has joined #openstack-infra | 15:38 | |
jeblair | fungi, pabelanger: when 210 lands, i'm thinking we should go ahead and do a full system restart | 15:38 |
jeblair | because we have a scheduler fix that needs to go in | 15:38 |
*** mriedem has joined #openstack-infra | 15:38 | |
pabelanger | sure, we should test out our playbooks | 15:38 |
jeblair | so might as well down all the launchers at the same time to speed that up | 15:38 |
jeblair | pabelanger: no, i don't mean graceful | 15:39 |
pabelanger | Oh | 15:39 |
*** yamahata has joined #openstack-infra | 15:39 | |
pabelanger | actually, that would be fun to watch | 15:39 |
jeblair | pabelanger: because we need to restart zuul, which is not graceful | 15:39 |
jeblair | (well, it can be, but it would take all day) | 15:39 |
pabelanger | right | 15:39 |
jesusaur | I seem to have hit a strange error with my log_processor workers, most of them crashed without writing any error to log; has anyone else seen this happen? | 15:39 |
pabelanger | would could update the playbook to by dynamic in that case | 15:40 |
pabelanger | either way | 15:40 |
*** tphummel has joined #openstack-infra | 15:40 | |
pabelanger | change for another day | 15:40 |
krotscheck | What is the purpose of yasfb in the infra specs module? I'm trying to figure out if I need it. | 15:40 |
krotscheck | (for javascript docs) | 15:40 |
*** ajmiller_ has joined #openstack-infra | 15:40 | |
*** hosanai has joined #openstack-infra | 15:41 | |
mordred | krotscheck: rss feed generation | 15:41 |
krotscheck | mordred: Ahhhh, that makes sense | 15:41 |
krotscheck | mordred: Do we have an idea of how many people use that? | 15:42 |
mordred | krotscheck: I do not | 15:42 |
fungi | krotscheck: makes some sense for specs, less sense for mostly static docs | 15:43 |
krotscheck | fungi: I agree. | 15:43 |
rcarrillocruz | hmm | 15:43 |
rcarrillocruz | nibalizer: | 15:43 |
rcarrillocruz | around | 15:43 |
*** bhavik has quit IRC | 15:43 | |
rcarrillocruz | ? | 15:43 |
*** ajmiller has quit IRC | 15:43 | |
rcarrillocruz | https://docs.puppet.com/puppetdb/3.2/release_notes.html , shows puppetDB only supports puppet 4.x | 15:44 |
rcarrillocruz | we still on puppet 3.x on nodes, no? | 15:44 |
*** flwang has quit IRC | 15:44 | |
krotscheck | fungi: That suggestion of yours yesterday workd. | 15:45 |
krotscheck | One question: pip install Sphinx -> put that in the makefile? I'm thinking yes. | 15:45 |
njohnston | Hi! Could I get a second +2 for https://review.openstack.org/329570 please? Thanks much! | 15:45 |
*** inc0 has joined #openstack-infra | 15:46 | |
*** ddieterly is now known as ddieterly[away] | 15:47 | |
*** flwang has joined #openstack-infra | 15:48 | |
smcginnis | Anyone have a moment to tell me what I'm doing wrong here: https://review.openstack.org/#/c/331192/ | 15:48 |
fungi | krotscheck: again, i would probably use distro packages for tools you need in your docs build, document that they're needed, and just assume they're preinstalled | 15:48 |
fungi | krotscheck: i would be surprised if i ran a makefile and it downloaded things from the internet | 15:49 |
smcginnis | Looks to me like it should be fine, but job fails with a message saying the job I added isn't defined. | 15:49 |
*** bpokorny has joined #openstack-infra | 15:49 | |
openstackgerrit | Merged openstack-infra/zuul: Ansible launcher: Fix timeout processing https://review.openstack.org/331210 | 15:49 |
*** esikachev has quit IRC | 15:49 | |
*** Guest31553 is now known as redrobot | 15:50 | |
*** mdrabe has joined #openstack-infra | 15:51 | |
*** bhavik has joined #openstack-infra | 15:51 | |
smcginnis | Hmm, and it is listed as a job earlier in the output. Confused. | 15:51 |
*** sdake has joined #openstack-infra | 15:51 | |
anteaya | smcginnis: there has to be a coresponding job definition for the jenkins job builder which draws from http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/projects.yaml#n763 | 15:52 |
jeblair | fungi, pabelanger: i'll restart zuul and all launchers after the next puppet run -- should be around 16:10 | 15:52 |
smcginnis | anteaya: Ah, thanks. Too many moving parts I guess. | 15:52 |
anteaya | smcginnis: and I've never heard of this job name before, you might have to build the definition | 15:52 |
pabelanger | fungi: jeblair: ack, I need to step away from my desk but back at 16:30 | 15:52 |
anteaya | smcginnis: there is a lot yes, is this a brand new job? | 15:53 |
*** matrohon has quit IRC | 15:54 | |
*** sree has quit IRC | 15:55 | |
anteaya | smcginnis: if this is a brand new job specific to cinder you are going to have to build a job-template for it, I'd suggest putting a new job template for cinder in here: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/cinder.yaml | 15:55 |
*** _milan_ has quit IRC | 15:57 | |
hosanai | Hello, I have a question about jenkins configuration. I would like to install non-python package for python27 checking. And I tried to use bindep but it seems to work for only packages which provide from debian or other OS distributors. | 15:57 |
hosanai | The package is a tgz file "http://apache.claz.org/spark/spark-1.6.1/spark-1.6.1.tgz" so I would like to download it and then extract it for jenkins. Do you know how to do it? | 15:57 |
mordred | hosanai: you'll just have to write a script to download and install it in your build scripts | 15:58 |
mordred | hosanai: jobs in openstack infra have sudo access on the hosts they run on, so you can do kindof whatever you want to | 15:58 |
*** tonytan4ever has joined #openstack-infra | 15:59 | |
mordred | hosanai: what project is this for? | 15:59 |
*** sree has joined #openstack-infra | 15:59 | |
hosanai | mordred: thanks for quick response. Monasca (Monasca-Analytics) | 15:59 |
*** sree has quit IRC | 15:59 | |
*** lanakars has joined #openstack-infra | 16:00 | |
*** _nadya_ has quit IRC | 16:01 | |
hosanai | mordred: I already have the script. where should I register it? | 16:01 |
*** bpokorny has quit IRC | 16:01 | |
*** winggundamth_ has joined #openstack-infra | 16:02 | |
*** bpokorny has joined #openstack-infra | 16:02 | |
*** permalac has quit IRC | 16:03 | |
lanakars | Hi folks, can anyone please help to fix this gate-fuel-docs-docs job: https://review.openstack.org/#/c/329587/ | 16:03 |
*** Na3iL has quit IRC | 16:03 | |
mordred | hosanai: you really just need to run it in your job - I'm guessing as a devstack plugin, or as part of the devstack plugin monasca already uses | 16:03 |
mordred | hosanai: be aware that downloading things from the internet during your jobs will increase your exposure to the internet being the internet | 16:04 |
openstackgerrit | Sean M. Collins proposed openstack-infra/devstack-gate: [WIP] Switch to new Neutron devstack https://review.openstack.org/278415 | 16:04 |
nibalizer | rcarrillocruz: hiya | 16:05 |
rcarrillocruz | hey, so, it seems puppetdb 3.x is for puppet 4?? | 16:06 |
rcarrillocruz | at least 3.2, the link i pasted you | 16:06 |
*** roxanaghe has joined #openstack-infra | 16:06 | |
*** jlanoux has quit IRC | 16:06 | |
jeblair | hosanai: as an official project, monasca should probably try to use dependencies that are already packaged or working with upstream distros to get them packaged | 16:06 |
mordred | ++ | 16:07 |
nibalizer | rcarrillocruz: so I think that means they didn't bother solving the packaging issue, not that there is anything wrong | 16:07 |
*** sdake has quit IRC | 16:07 | |
nibalizer | especially when you consider thta we own the script that is going to post to puppetdb, 'not being compatible' is basically impossible | 16:07 |
jeblair | zuul is updated everywhere. i'm going to restart it now | 16:08 |
*** dtantsur is now known as dtantsur|afk | 16:08 | |
rcarrillocruz | K | 16:08 |
anteaya | lanakars: looking in this file: http://logs.openstack.org/87/329587/3/check/gate-fuel-docs-docs/6bef22e/_zuul_ansible/logs/ansible.log | 16:09 |
*** dmk0202 has joined #openstack-infra | 16:09 | |
anteaya | lanakars: IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/zuul/ansible/library/zuul_runner.py' | 16:09 |
hosanai | mordred: thanks! sorry i still don't clear... i wrote devstack plugin for monasca-analytics but i don't know who kick this script from jenkins. maybe jenkins/jobs/projects.yaml? | 16:09 |
*** eezhova has quit IRC | 16:10 | |
anteaya | lanakars: I'm guessing having ansbile find a zuul_runner.py file is kind of important for the job to complete | 16:10 |
anteaya | I'm not sure what is supposed to put it where ansible can find it | 16:10 |
*** javeriak has joined #openstack-infra | 16:10 | |
jeblair | anteaya: recheck it; it's a known rare problem | 16:11 |
anteaya | jeblair: thank you | 16:11 |
anteaya | lanakars: ^^ | 16:11 |
*** jogo has quit IRC | 16:12 | |
jroll | hey friends, do you want to know about: gate-ironic-specs-docs POST_FAILURE in 47s | 16:12 |
hosanai | jeblair: thanks for the advise. monasca-analytics is a sub component of monasca and it will introduce new dependencies (there is no pip/apt-get packages) | 16:12 |
jroll | I assume that's a log post failure, but given all the changes, thought I'd let you know | 16:12 |
anteaya | jroll: link to patch? | 16:13 |
jeblair | jroll: it is, and we can find a clue as to what caused it by looking at the ansible log published along with the job | 16:13 |
jroll | anteaya: jeblair: https://review.openstack.org/#/c/188370/18 | 16:13 |
jeblair | (i can't look right now, i'm in the middle of a zuul restart, but can in a minute) | 16:13 |
*** tkelsey has quit IRC | 16:14 | |
pleia2 | good morning | 16:14 |
*** ddieterly[away] is now known as ddieterly | 16:14 | |
anteaya | jroll: hmm, File Not Found for the logs link is less helpful that I had hoped | 16:14 |
anteaya | morning pleia2 | 16:14 |
jroll | anteaya: :) | 16:14 |
anteaya | pleia2: welcome to a brand new zuul world | 16:14 |
lanakars | anteaya, jeblair: thank you! =) | 16:14 |
anteaya | lanakars: welcome | 16:14 |
jeblair | anteaya, jroll: oh, then we would have to look on the launcher. i'll do that when finished here | 16:14 |
jroll | jeblair: anteaya: I'm not concerned about the actual results, I rechecked and assumed it will past (just specs linting), but wanted to let you know. thanks :) | 16:15 |
anteaya | jeblair jroll thanks | 16:15 |
anteaya | jroll: yep, thanks for letting us know | 16:15 |
anteaya | jroll: I have to relearn how to read logs again | 16:15 |
jeblair | what's the link that 404s? | 16:15 |
jroll | jeblair: http://logs.openstack.org/70/188370/18/check/gate-ironic-specs-docs/8629a84/ | 16:15 |
*** roxanaghe has quit IRC | 16:15 | |
pleia2 | anteaya: indeed! | 16:16 |
anteaya | pleia2: getting some log groking practice in | 16:16 |
jeblair | oh, the launchers take a long time to stop because they still run their post playbooks | 16:16 |
jeblair | pabelanger: what's the status of the don't run post on abort change? | 16:17 |
pleia2 | anteaya: great :) | 16:17 |
anteaya | :) | 16:18 |
EmilienM | zuul launchers were restarted already? | 16:20 |
jeblair | EmilienM: in the process | 16:20 |
EmilienM | ack | 16:20 |
*** inc0 has quit IRC | 16:20 | |
*** degorenko is now known as _degorenko|afk | 16:21 | |
pabelanger | jeblair: I believe that is 329631 | 16:21 |
*** mrhillsman has joined #openstack-infra | 16:21 | |
*** ramishra has quit IRC | 16:22 | |
*** roxanaghe has joined #openstack-infra | 16:22 | |
fungi | lgtm | 16:23 |
jeblair | sorry i expected a hard stop to be a bit faster than this. hopefully 631 will fix that (we should test that it does) | 16:23 |
jeblair | should be just a couple more mins | 16:23 |
fungi | good point, 329631 doesn't add a test for that | 16:25 |
jeblair | fungi: we have no tests for zuul launcher | 16:25 |
*** jsavak has quit IRC | 16:25 | |
pabelanger | I did check -9 is the exit code of an aborted job. But yes, I've not actually tested it with zuul-launcher yet | 16:26 |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Add the ability to permalink to a comment https://review.openstack.org/289078 | 16:26 |
*** jsavak has joined #openstack-infra | 16:26 | |
openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Remove pagination for timeline events https://review.openstack.org/327682 | 16:26 |
jeblair | mostly i mean we should issue a zuul-launcher stop and make sure it takes that code path | 16:26 |
pabelanger | agreed | 16:26 |
fungi | yeah, their exit codes aren't clearly documented that i can find | 16:26 |
*** e0ne has quit IRC | 16:27 | |
jeblair | i think -9 may come from us because of the way we kill the pgrop | 16:27 |
jeblair | pgroup | 16:27 |
*** mixos has joined #openstack-infra | 16:27 | |
jeblair | okay, all launchers have stopped | 16:27 |
fungi | aha, was about to say i don't see it at https://github.com/ansible/ansible/blob/devel/bin/ansible#L99 either | 16:28 |
*** dprince has joined #openstack-infra | 16:28 | |
pabelanger | Right, I didn't check if stop will actually get there. I was looking from abortRunningJob | 16:28 |
jeblair | wow | 16:28 |
jeblair | zuul-launcher leaked 0 nodes :) | 16:28 |
fungi | um | 16:28 |
pabelanger | impressive | 16:28 |
jeblair | like, there are 0 nodes in nodepool assigned to zl* that are not in delete state | 16:28 |
fungi | jenkins looks worse every day | 16:28 |
pabelanger | that is very nice | 16:29 |
mordred | \o/ | 16:29 |
jeblair | so i think with everything in nodepool in either building or delete, i will go ahead and start zuul without touching nodepool | 16:29 |
jeblair | (i was prepared to do a db query to force delete nodes but do not think that is necessary) | 16:29 |
mordred | I agree | 16:29 |
fungi | seems not anyway | 16:29 |
*** mriedem is now known as mriedem_lunch | 16:30 | |
clarkb | did we remove the jenkins restart cron? it will fire today if not | 16:30 |
jeblair | zuul is started | 16:30 |
nibalizer | clarkb: haha good point | 16:30 |
fungi | also did we remove jenkins masters from dns? if not, i can do that now | 16:30 |
fungi | looks like we did not | 16:31 |
mordred | fungi: they still resolve for me | 16:31 |
fungi | (no idea why i asked when it was easier to query) | 16:31 |
fungi | yep | 16:31 |
mordred | fungi: :) | 16:31 |
jeblair | status notice zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time | 16:31 |
jeblair | how does that look ^ ? | 16:31 |
fungi | any objections to me removing them from dns now? | 16:31 |
fungi | jeblair: lgtm | 16:31 |
jeblair | fungi: no objection | 16:31 |
anteaya | looks good | 16:31 |
jeblair | #status notice zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time | 16:31 |
openstackstatus | jeblair: sending notice | 16:31 |
pabelanger | fungi: none here | 16:32 |
anteaya | I have no objection | 16:32 |
-openstackstatus- NOTICE: zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time | 16:33 | |
smcginnis | anteaya: Oh, duh. I'm obviously not paying enough attention today. | 16:34 |
anteaya | smcginnis: it is Friday, it is common | 16:34 |
smcginnis | anteaya: It should have struck me that I at least need to say what to run for the job somewhere. ;) | 16:34 |
openstackstatus | jeblair: finished sending notice | 16:34 |
pabelanger | used nodes are almost 100% equal across zuul-launchers now | 16:34 |
anteaya | smcginnis: well we did just change everything, so I totally understand the wishful thinking approach to coding | 16:34 |
*** jpich has quit IRC | 16:34 | |
jeblair | pabelanger: woo! | 16:34 |
anteaya | smcginnis: who knows, maybe that will work someday | 16:35 |
pabelanger | drift of about 2 nodes I am seeing | 16:35 |
smcginnis | anteaya: haha, it was definitely wishful thinking on my part that it would be that easy. | 16:35 |
jlvillal | Is the data that "CI Watch" (http://ci-watch.tintri.com/project?project=ironic&time=7+days) uses available as a CSV file or something? If I want to generate some statistics. Or some other way for me to relatively easily generate statistics. | 16:35 |
anteaya | smcginnis: ha ha ha | 16:35 |
anteaya | asselin_: mmedvede ^^ | 16:35 |
*** tkelsey has joined #openstack-infra | 16:35 | |
Shrews | pabelanger: yeah, i was watching that myself | 16:35 |
*** Apoorva has joined #openstack-infra | 16:36 | |
mmedvede | jlvillal: no, it is not | 16:36 |
*** Apoorva has quit IRC | 16:37 | |
mmedvede | jlvillal: ciwatch uses sql database, and there is no rest api implemented for backend | 16:37 |
pabelanger | jeblair: I would be interested to see the steps you just did in etherpad to mass restart zuul. With nodepool other of the picture, it might be a straightforward playbook to automate that process | 16:37 |
jlvillal | mmedvede: Darn :( | 16:37 |
jlvillal | mmedvede: And thanks. | 16:37 |
EmilienM | jeblair: is there a plan to have a graphic realtime console? like we had in jenkins, that we can see in web browser, instead of using nc? | 16:38 |
mordred | EmilienM: yes | 16:39 |
jeblair | pabelanger: save gate/check queue; service zuul stop; on all launchers: zuul-launcher stop; wait; service zuul start; service zuul-launcher start | 16:39 |
mordred | EmilienM: I've got it half written, hope to finish it soon | 16:39 |
pabelanger | jeblair: great, thanks | 16:39 |
EmilienM | mordred: it's in gerrit? | 16:39 |
*** Goneri has joined #openstack-infra | 16:39 | |
jeblair | jroll: ah, that job failed due to a known, but rare issue | 16:39 |
mordred | EmilienM: https://review.openstack.org/#/c/320563/ | 16:39 |
EmilienM | mordred: awesome | 16:39 |
jeblair | jroll: our ansible module briefly disappears from the file system when we reinstall zuul (which we do every time we land a patch) | 16:40 |
clarkb | I am pretty sure if you install the chrome ssh plugin it will just work | 16:40 |
*** tkelsey has quit IRC | 16:40 | |
mordred | EmilienM: I'm waiting for the 2.5 stuff to be mostly done so I can avoid lots of rebases and stuff - but I should be able to start working on it again either today or early next week | 16:40 |
pabelanger | clarkb: which one did you used on your phone again? | 16:40 |
clarkb | pabelanger: connectbot on phone | 16:40 |
pabelanger | clarkb: thanks | 16:40 |
jeblair | mordred: i keep saying this is "rare" often enough that i'm thinking we may need to fix it soon. | 16:41 |
EmilienM | mordred: awesome feature | 16:41 |
mordred | jeblair: I agree | 16:41 |
EmilienM | mordred: for impatient people like us :P | 16:41 |
*** mohankumar has quit IRC | 16:41 | |
*** PollyZ has quit IRC | 16:41 | |
pabelanger | what I'm currently doing for chrome: https://etherpad.openstack.org/p/google-telnet | 16:42 |
fungi | mmedvede: jlvillal: we could perhaps consider doing what we did for the subunut2sql db used by health.openstack.org and just provide a query:query read-only user available through a socket proxy | 16:42 |
pabelanger | warning, pretty hacky | 16:42 |
fungi | in theory there's no sensitive data in there? | 16:42 |
*** gyee has joined #openstack-infra | 16:42 | |
EmilienM | pabelanger: /me trying | 16:43 |
mmedvede | fungi: should be no sensitive data there, it is just an aggregate from jenkins event stream | 16:43 |
mmedvede | *gerrit event stream | 16:44 |
fungi | mmedvede: or if it's a very small dataset, just hack a csv export call into ciwatch available from the web interface | 16:44 |
*** mikelk has quit IRC | 16:44 | |
fungi | speaking of which, i should plug https://github.com/harelba/q again | 16:45 |
fungi | it's awesomely useful | 16:45 |
*** trown is now known as trown|lunch | 16:45 | |
*** PollyZ has joined #openstack-infra | 16:45 | |
pabelanger | fungi: jeblair: just confirmed BUILD_TIMEOUT is passed correctly now | 16:45 |
mmedvede | fungi: that is awesome, I was just recently googling for something that does it | 16:46 |
pabelanger | fungi: jeblair: thanks for helping out | 16:46 |
fungi | excellent | 16:46 |
AJaeger | great, pabelanger ! | 16:46 |
fungi | dprince: ^ | 16:46 |
fungi | the tripleo crowd will be relieved | 16:46 |
mordred | \o/ | 16:46 |
AJaeger | smcginnis: do you now know what to do? | 16:46 |
mmedvede | fungi: so ciwatch instance is currently maintained outside infra team. Any chance there is now some compute available to get a VM up and deploy it within infra? | 16:47 |
EmilienM | fungi: thx | 16:47 |
smcginnis | AJaeger: I think so. Working on it now. Seems mostly straightforward, so I'll give it a shot. | 16:47 |
fungi | mmedvede: shouldn't be a problem. usually we ask for a (very small) spec describing the service and its needs | 16:47 |
smcginnis | AJaeger: Thanks | 16:47 |
EmilienM | pabelanger: it works lol | 16:47 |
zaro | morning | 16:48 |
AJaeger | smcginnis: just commented on your review and gave an example what to do... | 16:48 |
smcginnis | AJaeger: Oh, awesome. Thanks! | 16:48 |
dprince | fungi: ack, thanks for this | 16:48 |
AJaeger | smcginnis: add that to jenkins/jobs/projects.yaml | 16:48 |
mmedvede | fungi: http://specs.openstack.org/openstack-infra/infra-specs/specs/deploy-ci-dashboard.html, the spec landed a while back, so might need revision | 16:48 |
fungi | dprince: jeblair and pabelanger did most of the work, but you're welcome! | 16:48 |
dprince | thanks jeblair and pabelanger :) | 16:49 |
jeblair | dprince: sorry for the inconvenience | 16:49 |
fungi | mmedvede: oh, already most of the way there then | 16:49 |
openstackgerrit | Raildo Mascena proposed openstack-infra/project-config: Run novaclient functional tests in Identity v3-only https://review.openstack.org/313588 | 16:49 |
EmilienM | pabelanger: just missed the chmod +x | 16:49 |
*** ddieterly is now known as ddieterly[away] | 16:50 | |
pabelanger | EmilienM: update etherpad :) | 16:50 |
*** _degorenko|afk has quit IRC | 16:50 | |
EmilienM | pabelanger: done | 16:50 |
pabelanger | EmilienM: but ya, hacked on that last night at 10:30pm EST | 16:50 |
pabelanger | should revisit it this weekend | 16:50 |
smcginnis | AJaeger: Should that be the jenkins/jobs/projects.yaml file or jenkins/jobs/cinder.yaml since this is Cinder specific (at least for now). | 16:50 |
AJaeger | clarkb: could you review https://review.openstack.org/330821 - Revert "Remove skip test logic for project-config" , please? | 16:51 |
AJaeger | smcginnis: you instantiate templates in projects.yaml | 16:51 |
EmilienM | oops I see "gear.NoConnectedServersError: No connected Gearman servers" | 16:51 |
EmilienM | dprince: ^ | 16:51 |
EmilienM | dprince: in tripleo jobs | 16:51 |
AJaeger | smcginnis: just search for tox in that file | 16:51 |
smcginnis | AJaeger: OK, got it. Thanks again. | 16:51 |
pabelanger | clarkb: would connectbot open, and connect to the host once you used telnet:// directly from the browser? Or do you copy and paste? | 16:51 |
EmilienM | dprince: not sure that's something we expect | 16:51 |
*** lucasagomes is now known as lucas-afk | 16:51 | |
*** daemontool_ has quit IRC | 16:52 | |
zaro | pleia2: can you advice about users required for this change? | 16:53 |
zaro | https://review.openstack.org/#/c/330925/ | 16:53 |
*** harlowja has joined #openstack-infra | 16:53 | |
fungi | okay, all jenkins*.openstack.org dns records have been deleted | 16:54 |
mordred | \o/ | 16:54 |
*** degorenko has joined #openstack-infra | 16:54 | |
pleia2 | zaro: looking | 16:55 |
zaro | pleia2: we basically need a bot user in storyboard for gerrit. wondering if we might already have one a general bot user that gerrit can use? | 16:55 |
pleia2 | zaro: gotcha, let's see what we use for updating launchpad | 16:56 |
openstackgerrit | zzxwill proposed openstack-infra/zuul: Rename jenkins to zull https://review.openstack.org/330853 | 16:57 |
fungi | pleia2: in lp it's the hudson-openstack account | 16:57 |
fungi | i don't think we need to duplicate that username in sb though | 16:57 |
*** pvaneck has joined #openstack-infra | 16:58 | |
pleia2 | fungi: right, but what user on review.o.o interacts with that account? | 16:58 |
*** _ari_ has quit IRC | 16:58 | |
*** timello has quit IRC | 16:58 | |
dprince | EmilienM: other jobs are running, which log did you see the connection error in? | 16:58 |
fungi | pleia2: none. it's a gerrit hook script calling an entrypoint in jeepyb | 16:58 |
dprince | EmilienM: are they consistent, or just a one off | 16:59 |
pleia2 | oh I see, this is looking for a user *in storyboard* | 16:59 |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config: Add driver compliance checks to check pipeline https://review.openstack.org/331192 | 16:59 |
*** inc0 has joined #openstack-infra | 17:00 | |
pleia2 | zaro: we do seem to have an OpenStack Gerrit user | 17:00 |
EmilienM | dprince: all | 17:00 |
EmilienM | dprince: telnet://66.187.229.81:8088 | 17:00 |
pleia2 | zaro: user 240 for review.o.o, user 616 for review-dev | 17:01 |
pabelanger | ha, need to update weechat to parse telnet:// | 17:01 |
zaro | pleia2: cool, would you mind comment on my change what user ino i should retrieve from heirs? | 17:01 |
EmilienM | dprince: http://paste.openstack.org/show/518471/ | 17:01 |
pleia2 | zaro: will do | 17:01 |
*** inc0 has quit IRC | 17:01 | |
EmilienM | dprince: I tested different jobs on different patches | 17:01 |
*** inc0 has joined #openstack-infra | 17:02 | |
zaro | pleia2: thanks. | 17:02 |
*** Apoorva has joined #openstack-infra | 17:02 | |
*** jsavak has quit IRC | 17:02 | |
pabelanger | EmilienM: dprince: I suspect your local gearman server is offline | 17:02 |
*** jsavak has joined #openstack-infra | 17:02 | |
zaro | pleia2: now i’m wondering if review-dev.o.o has access to storyboard-dev.o.o? i mean can you post to sb-dev using secure curl? | 17:03 |
EmilienM | dprince: do you have access to it? | 17:03 |
dprince | EmilienM: yes... just have to remember how I get in again | 17:03 |
pleia2 | zaro: I don't know, review-dev's account is on storyboard.o.o (not storyboard-dev), noting in change | 17:04 |
*** inc0 has quit IRC | 17:04 | |
SotK | we intend to populate storyboard-dev with data from storyboard.o.o at some point ftr | 17:05 |
clarkb | pabelanger: it opened automagically | 17:05 |
pabelanger | clarkb: thanks, I'll dig deeper why that's not working for me | 17:05 |
anteaya | SotK: sure put that likely won't happen by next week's sprint | 17:06 |
anteaya | s/put/but | 17:06 |
*** bpokorny has quit IRC | 17:06 | |
hosanai | mordred: sorry for bother you. could you point out a script (devstack plugin) which is called by jenkins? | 17:06 |
zaro | pleia2: i’m concerned because sb-dev doesn’t have real certs so making requests might be an issue. | 17:07 |
anteaya | SotK: I'd like to get this interaction in place on the dev servers for next week's sprint | 17:07 |
*** lanakars has left #openstack-infra | 17:07 | |
zaro | pleia2: would we be able to test a POST to sb-dev from review-dev? | 17:07 |
anteaya | and I'm so grateful to zaro and pleia2 for helping here | 17:07 |
pleia2 | zaro: could be | 17:07 |
*** javeriak_ has joined #openstack-infra | 17:07 | |
*** javeriak has quit IRC | 17:07 | |
*** tonytan4ever has quit IRC | 17:07 | |
*** xyang1 has quit IRC | 17:08 | |
dprince | EmilienM: not all our machines have public IPs, so I'll go in through the mirror server | 17:08 |
*** xyang1 has joined #openstack-infra | 17:08 | |
zaro | anteaya: gerrit sb plugin is unlikely to make it in by next week. i’m finding that there will need to quite a few changes required for install and configuration | 17:09 |
anteaya | zaro: okay fair enough, is there anything I can do to help, I've looked at as many changes as I can find | 17:09 |
*** yamamoto has quit IRC | 17:10 | |
pleia2 | zaro or anteaya: have a suggestion as to what to post? I haven't used the API much yet | 17:10 |
anteaya | zaro: are you using a common topic for all the patches needed to get this interaction in place? | 17:10 |
anteaya | pleia2: yes, let me get you a command in a paste | 17:10 |
zaro | anteaya: not atm, i’ll let you know when the changes are all ready for review. | 17:10 |
anteaya | pleia2: I didn't want to interrupt your conversation with zaro, you were making such good progress | 17:10 |
anteaya | zaro: okay can you link them with a gerrit topic in the meantime, so I can at least be aware of status? | 17:11 |
zaro | anteaya: sure can. | 17:11 |
pleia2 | anteaya: thanks :) | 17:11 |
anteaya | zaro: thank you | 17:12 |
*** e0ne has joined #openstack-infra | 17:12 | |
*** amotoki has quit IRC | 17:13 | |
*** timello has joined #openstack-infra | 17:13 | |
*** ddieterly[away] is now known as ddieterly | 17:13 | |
*** arxcruz has quit IRC | 17:13 | |
AJaeger | there's a change to remove node-devstack from nodepool. Syntax looks fine but I'm not sure whether we want that. fungi, jeblair, pabelanger: Could either of you comment on https://review.openstack.org/318994 , please? | 17:16 |
*** bpokorny has joined #openstack-infra | 17:16 | |
anteaya | pleia2: hopefully http://paste.openstack.org/show/518474/ gets you at least part of the way there | 17:17 |
pabelanger | anteaya: left comment | 17:19 |
fungi | AJaeger: yep, that's some good cleanup | 17:19 |
pabelanger | anteaya: apologies, tabbed incorrectly | 17:19 |
anteaya | pabelanger: sorry, left comment on what | 17:19 |
pleia2 | anteaya: thanks | 17:19 |
anteaya | oh okay | 17:19 |
pabelanger | AJaeger: I have left comments | 17:19 |
anteaya | pleia2: let me know if you need more | 17:19 |
zaro | pleia2: i think we want to test without the ‘-key' | 17:20 |
pabelanger | fungi: I think we should also remove nodepool/elements/node-devstack ? | 17:20 |
fungi | pabelanger: yes, though i'm not particular about whether it has to be in that change | 17:21 |
AJaeger | thanks, pabelanger | 17:21 |
AJaeger | thanks, fungi | 17:21 |
pabelanger | fungi: sure, followup works too | 17:21 |
fungi | if you're -1 on it, i'm happy to drop my workflow +1 | 17:21 |
pleia2 | zaro: yeah, I don't have a local key for this | 17:21 |
pabelanger | fungi: sounds like work. I'll do a follow up | 17:22 |
anteaya | pleia2: you probably don't need one, that was just what I needed for this to work on my local gerrit | 17:22 |
anteaya | pleia2: I think zaro wants to test POSTing without that bit to see if certs on both sides will allow the POST to happend | 17:23 |
anteaya | happen* | 17:23 |
*** thiagop has quit IRC | 17:23 | |
pleia2 | anteaya: exactly | 17:23 |
anteaya | cool | 17:23 |
pleia2 | trying to figure out why this command isn't returning, I might have a stray ' somewhere | 17:24 |
*** e0ne has quit IRC | 17:24 | |
anteaya | or I could have given you a bad paste | 17:24 |
pleia2 | anteaya: in the Authorization section, should it be 'Authorization: My Name <3243232>' ? | 17:24 |
anteaya | s/or// | 17:24 |
anteaya | no | 17:24 |
dprince | EmilienM: our tripleo testenv geard is restarted. Watching it now | 17:24 |
anteaya | you have to get a token and paste it in | 17:25 |
pleia2 | anteaya: well yes :) I was using a fake one in that example | 17:25 |
pleia2 | but pretend 3243232 is real, I have 'Authorization: Elizabeth K. Joseph <3243232>' | 17:25 |
anteaya | pleia2: https://storyboard-dev.openstack.org/#!/profile/tokens | 17:25 |
anteaya | no, just the token | 17:25 |
pleia2 | anteaya: I know, I have my token, I was asking about syntax | 17:26 |
anteaya | sorry | 17:26 |
pleia2 | anteaya: what is "Bearer" ? is that part of the command? | 17:26 |
anteaya | not the username | 17:26 |
anteaya | yes Bearer needs to be there | 17:26 |
pleia2 | gotcha, I thought it was a placeholder for name, thanks | 17:26 |
AJaeger | mriedem_lunch: is https://review.openstack.org/#/c/313588 now fine? | 17:26 |
*** PollyZ has quit IRC | 17:26 | |
anteaya | -H 'Authorization: Bearer 1234567' | 17:26 |
pleia2 | ah, no brackets either | 17:27 |
anteaya | no brackets | 17:27 |
anteaya | sorry the brackets were my attempt to say fill in this space | 17:27 |
pleia2 | makes sense | 17:27 |
anteaya | fill in this space and then let's talk about it, is what I should have said | 17:28 |
pleia2 | so curl definitely doesn't like it without a key, using --insecure gets me a 500 error | 17:29 |
anteaya | :( | 17:29 |
pleia2 | zaro: ^^ | 17:29 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Ansible launcher: add support for node revocation https://review.openstack.org/331273 | 17:30 |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Revoke nodes via gearman on deletion https://review.openstack.org/331274 | 17:30 |
jeblair | pabelanger, mordred, fungi: ^ that is not urgent but may come in handy. | 17:31 |
*** piet has joined #openstack-infra | 17:31 | |
*** nwkarsten has quit IRC | 17:32 | |
*** dizquierdo has quit IRC | 17:32 | |
*** nwkarsten has joined #openstack-infra | 17:32 | |
mordred | jeblair: agree - looks good ... although fwiw when I read the commit messages at first I thought you were describing the opposite direction | 17:32 |
mordred | that's likely just the way i processed it rather than a real issue | 17:33 |
AJaeger | project-config cores, the dependent job is currently in gate queue, would be nice to have install-guides jobs soon. Could you review https://review.openstack.org/330725 , please? | 17:33 |
*** nwkarsten has quit IRC | 17:33 | |
jeblair | mordred: yeah, they're "assigned" by gearman, so i think they should also be "revoked" by gearman. the commit message prose could probably be better, but i think the jobs are named right | 17:34 |
*** milan has joined #openstack-infra | 17:34 | |
AJaeger | pabelanger: do you want to abandon 326006, or is this still needed? | 17:34 |
mordred | jeblair: I agree | 17:34 |
jeblair | er, i think 's/gearman/nodepool' above | 17:34 |
pleia2 | anteaya: so the 500 error seems to be "ValueError: No JSON object could be decoded" (and python traceback along with it) | 17:34 |
jeblair | mordred: do you have any ideas how we might deal with the missing zuul_runner thing? | 17:35 |
jeblair | mordred: it seems like the pip install makes the file disappear for a significant enough amount of time | 17:36 |
zaro | curl -v -H "Authorization:Bearer a6ya1ndTkEaDHqfp5aJsqjVP3VUahp" -H "Content-Type: application/json" -X POST -d '{"story_id”:7,"content":"mama!"}' https://storyboard.openstack.org/api/v1/stories/7/comments | 17:36 |
zaro | pleia2: try this with your token ^ | 17:36 |
pleia2 | zaro: thanks | 17:36 |
*** dtardivel has quit IRC | 17:37 | |
mordred | jeblair: I'm trying _really_ hard to not say AFS | 17:37 |
zaro | pleia2: opps that should be storyboard-dev.o.o | 17:37 |
jeblair | mordred: i admire your restraint | 17:37 |
mordred | jeblair: of course, it seems that atomic releasing of a directory worth of contents is a thing this would benefit from | 17:38 |
openstackgerrit | Merged openstack-infra/project-config: Remove node-devstack from nodepool https://review.openstack.org/318994 | 17:38 |
EmilienM | dprince: it looks better | 17:38 |
pleia2 | zaro: breaks without ssl certificate, running with --insecure I get the same json error as with anteaya's command | 17:39 |
*** banix has joined #openstack-infra | 17:39 | |
*** sarob has joined #openstack-infra | 17:39 | |
pleia2 | zaro: json error is in the logs, 500 error on the user side from my curl command | 17:39 |
zaro | pleia2: well i would expect —insecure to work though | 17:40 |
mordred | jeblair: do we have a good handle anywhere that could trap for the "this file isn't here" error and retry? | 17:40 |
*** sarob has quit IRC | 17:40 | |
*** ajmiller has joined #openstack-infra | 17:40 | |
*** sdake has joined #openstack-infra | 17:40 | |
*** hosanai has quit IRC | 17:40 | |
jeblair | mordred: well, it's in the middle of the playbook | 17:40 |
*** sarob has joined #openstack-infra | 17:40 | |
mordred | darn. | 17:40 |
jeblair | mordred: idea: | 17:41 |
fungi | pleia2: make sure you use straight quotes? zaro's example had a smart quote mixed in according to my font | 17:41 |
jeblair | mordred: we could, when setting up the jobdir, copy the needed modules into it. that's something we can retry-loop. | 17:41 |
*** yaume has quit IRC | 17:41 | |
pleia2 | fungi: thanks, I'll check | 17:41 |
mordred | jeblair: that makes me a little sad, but not excessively sad | 17:42 |
jeblair | mordred: it means every job would get their own copy of the modules which is silly, and it's a bit extra io, but i guess they're not that big. | 17:42 |
jeblair | (we would want to make sure we don't put them into the directory that gets uploaded, because we really do not need a copy of zuul_runner for every job on our logserver) | 17:42 |
mordred | ++ | 17:42 |
*** ajmiller_ has quit IRC | 17:43 | |
zaro | first time i’ve ever heard those things called 'smart' | 17:43 |
jeblair | mordred: only other thing that jumps to mind is having puppet copy them to some location on the host. | 17:43 |
mordred | jeblair: we could also do the pip install and then rsync the contents of lib/python to another location which is the path we actually run from | 17:43 |
*** PollyZ has joined #openstack-infra | 17:43 | |
mordred | jeblair: so as that we can avoid the delete-then-create behavior | 17:43 |
mordred | yah | 17:43 |
mordred | what you said | 17:43 |
jeblair | mordred: oh, that makes me think of a combination of the two: | 17:43 |
jeblair | mordred: what if we had *zuul* do that, not puppet | 17:44 |
fungi | zaro: ” vs " | 17:44 |
mordred | ah - interesting. yeah. so we only have one extra copy - but it's a safe extra copy | 17:44 |
*** sambetts is now known as sambetts|afk | 17:44 | |
jeblair | mordred: so zuul makes a sort of global tmpdir for its ansible deps, and periodically checks that it's up to date from its install location | 17:44 |
*** bpokorny has quit IRC | 17:44 | |
mordred | yes | 17:44 |
asselin_ | rcarrillocruz, https://ansible-role-cloud-launcher.readthedocs.org link gets forwarded to https://ansible-role-cloud-launcher.readthedocs.io/ which doesn't exist yet. Is that expected? | 17:44 |
anteaya | fungi: pleia2 for context zaro was able to get the storyboard-plugin to work: https://storyboard-dev.openstack.org/#!/story/7 | 17:44 |
fungi | zaro: what was the outstanding stack of changes for the sb/its plugin work for review-dev? happy to do some reviewing on those | 17:45 |
*** jed56 has quit IRC | 17:45 | |
anteaya | we just have to get it configured correctly on review-dev | 17:45 |
*** rbrndt has quit IRC | 17:45 | |
fungi | anteaya: zaro: ooh, excellent | 17:45 |
pleia2 | anteaya: got your command to work, but only by using --insecure | 17:45 |
anteaya | yeah, so the fault was mine, but I don't know what I did wrong yet | 17:45 |
anteaya | pleia2: okay thanks | 17:45 |
fungi | pleia2: the json decoding error was quoting-related then? | 17:45 |
zaro | that came straight out of my text editor, must be irccloud mucking it up on paste | 17:46 |
jeblair | mordred: so like: every time a job starts, make sure /var/lib/zuul/ansible is up to date. and use that for the ansible plugins/modules location. | 17:46 |
krotscheck | I've got a couple of old-ish maniphest patches still in my queue, do we still care or shall I abandon them? https://review.openstack.org/#/q/status:open+project:openstack-infra/puppet-phabricator+branch:master+topic:maniphest | 17:46 |
zaro | pleia2: cool, so we need it to work without -insecure. i thinkg | 17:46 |
jeblair | mordred: or, you know what, it's not critical they be up to date | 17:47 |
anteaya | krotscheck: we still care as the ux folks will need the functionality there | 17:47 |
jeblair | mordred: let's just do it on startup. | 17:47 |
anteaya | krotscheck: we just haven't had the review time for them to make the top priority currently | 17:47 |
jeblair | (in fact, it's slightly weird that they change when the launcher doesn't) | 17:47 |
krotscheck | anteaya: Alright, can you take over those patches? I'm refocusing on non-infra things. | 17:47 |
anteaya | ah sure, or you can just put them in any state you like | 17:48 |
anteaya | if you abandon them a gerrit admin can take them over | 17:48 |
anteaya | krotscheck: thanks for letting us know | 17:48 |
krotscheck | s/infra/task-tracking/ | 17:48 |
anteaya | krotscheck: yep fair enough | 17:48 |
*** pblaho has quit IRC | 17:48 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config: Add auto interval to nodepool dashboard https://review.openstack.org/331281 | 17:48 |
anteaya | krotscheck: thanks for working on those patches | 17:49 |
pabelanger | jeblair: AJaeger: ^ I'd like to give that a try, it should give the users the ability to adjust the interval ranges based on https://www.youtube.com/watch?v=CLXKlgHvOuo | 17:49 |
zaro | pleia2: ohh maybe not. i think we only need to allow java access to do the post. | 17:49 |
*** _nadya_ has joined #openstack-infra | 17:49 | |
pabelanger | AJaeger: checking on review you linked | 17:50 |
AJaeger | thanks, pabelanger. | 17:50 |
*** markusry has quit IRC | 17:51 | |
zaro | pleia2: this is how i gave java access from my local machine http://paste.openstack.org/show/518481/ | 17:51 |
*** ccamacho has quit IRC | 17:52 | |
zaro | pleia2: i’m not sure if that’s legit to do on review-dev or not. fungi? ^ | 17:52 |
pleia2 | zaro: ah, this creates the storyboard-dev.crt that anteaya had in her command | 17:52 |
mordred | jeblair: ++ to doing it on startup | 17:52 |
*** tosky has quit IRC | 17:52 | |
anteaya | pleia2: yeah, zaro had me run those commands on my local machine | 17:53 |
pleia2 | anteaya: nods | 17:53 |
anteaya | pleia2: here was what he had given to me: http://paste.openstack.org/show/516325/ | 17:53 |
zaro | yep, curl can accept that key as well | 17:53 |
*** yamahata has quit IRC | 17:53 | |
anteaya | note the last command is run with sudo | 17:53 |
pleia2 | anteaya: yeah, since it's editing the global java keystore | 17:54 |
pleia2 | I can run the first and just reference the key without touching global stuff | 17:54 |
anteaya | I had missed noticing that myself on my first pass | 17:54 |
anteaya | or making that connection I should say | 17:55 |
*** trown|lunch is now known as trown | 17:55 | |
*** gyee has quit IRC | 17:55 | |
fungi | zaro: we ought to be able to puppet that onto review-dev | 17:56 |
pleia2 | zaro: ok, got your command to work by using that ssl cert (I didn't put it in the global config) | 17:56 |
fungi | assuming paste.o.o ever responds to me | 17:57 |
pleia2 | fungi: yeah | 17:57 |
*** tonytan4ever has joined #openstack-infra | 17:57 | |
* fungi is assuming what is in it without being able to look at it yet | 17:57 | |
zaro | fungi: ok. i just wanted to verify that it’s not going to conpromise any security thingys | 17:57 |
fungi | oh, not at all. also it's just review-dev | 17:57 |
fungi | for production we wouldn't need to do this because review will be able to trust the signature on storyboard.o.o | 17:58 |
anteaya | yes, we must consider the security thingys | 17:58 |
pleia2 | yeah, anything with a real cert is fine | 17:58 |
*** mriedem_lunch is now known as mriedem | 17:59 | |
*** cody-somerville has joined #openstack-infra | 17:59 | |
zaro | ok. i’ll puppet it then. thanks all. | 17:59 |
pleia2 | you're welcome | 17:59 |
anteaya | zaro: awesome, thanks | 17:59 |
fungi | zaro: we can just put the cert contents in public hiera, no need for that first command to be run _on_ review-dev | 17:59 |
*** nwkarsten has joined #openstack-infra | 18:00 | |
*** nwkarsten has quit IRC | 18:00 | |
zaro | +1 dat | 18:00 |
*** nwkarsten has joined #openstack-infra | 18:00 | |
fungi | the second command, because it's updating the jvm trust store, does need to run on review-dev obviously | 18:00 |
mriedem | AJaeger: yeah, i don't know if it's going to pass, but it's non-voting to start | 18:01 |
mriedem | so seems ok to me | 18:01 |
AJaeger | mriedem: thanks - let's try it out ;) | 18:01 |
*** ddieterly is now known as ddieterly[away] | 18:03 | |
*** harlowja has quit IRC | 18:04 | |
fungi | zaro: pleia2: anteaya: so basically put the results of that first command in public hiera for review-dev.o.o, and then create a file resource with that content somewhere and then subscribe the exec for that second command to it | 18:04 |
fungi | should be pretty simple | 18:04 |
anteaya | yay simple! | 18:04 |
pleia2 | java couldn't just have a file listing of certs | 18:05 |
*** kzaitsev_mb has quit IRC | 18:06 | |
pleia2 | I guess it's faster this way? | 18:06 |
*** dprince has quit IRC | 18:06 | |
*** dprince has joined #openstack-infra | 18:06 | |
fungi | pleia2: i'm guessing keytool -storepass changeit also notifies the running jvm | 18:07 |
pleia2 | fungi: ah, perhaps | 18:07 |
fungi | though you can likely directly edit ${JAVA_JDK_HOME}/jre/lib/security/cacerts if you want (no clue what format it's in, i haven't looked) | 18:07 |
pleia2 | it's a binary file | 18:08 |
fungi | why of course it is! | 18:08 |
zaro | pleia2: i’m not sure actually. it’s been a while but last i tried this was the only solution i found on the internet | 18:08 |
fungi | probably the java equivalent of python pickle | 18:08 |
*** ccamacho has joined #openstack-infra | 18:08 | |
*** javeriak_ has quit IRC | 18:09 | |
*** yamamoto has joined #openstack-infra | 18:10 | |
*** javeriak has joined #openstack-infra | 18:10 | |
*** thorst_ has quit IRC | 18:13 | |
*** xiangxinyong has quit IRC | 18:13 | |
*** thorst_ has joined #openstack-infra | 18:14 | |
*** xiangxinyong has joined #openstack-infra | 18:14 | |
nwkarsten | i'm trying to port the pyton-jobs unit tests into my ci system, but i am having issues. i have the jenkins/jobs/macros.yaml file copied exactly, but the zuul-git-prep-upper-constraints builder is failing. Here's the console output http://paste.openstack.org/show/518485/ does anyone know why that i can't access that git repo? | 18:14 |
*** sarob has quit IRC | 18:15 | |
clarkb | nwkarsten: probably a firewall on your end blocking port 9418 | 18:16 |
*** cody-somerville has quit IRC | 18:17 | |
*** jamesmcarthur has quit IRC | 18:18 | |
*** thorst_ has quit IRC | 18:18 | |
*** thorst_ has joined #openstack-infra | 18:19 | |
openstackgerrit | Andreas Florath proposed openstack/diskimage-builder: Refactor: block-device handling (Local Loop) https://review.openstack.org/319591 | 18:22 |
*** yamamoto has quit IRC | 18:24 | |
fungi | nwkarsten: you could try using https:// instead of git:// for the url passed to zuul-cloner | 18:24 |
fungi | (or even just http://) | 18:24 |
*** yamahata has joined #openstack-infra | 18:25 | |
nwkarsten | @fungi,@clarkb: i will give https a shot | 18:26 |
*** jamesmcarthur has joined #openstack-infra | 18:27 | |
anteaya | nwkarsten: it isn't necessary to use @ in front of an irc nick | 18:27 |
anteaya | that is hipchat or twitter | 18:27 |
*** ajmiller has quit IRC | 18:29 | |
*** ajmiller_ has joined #openstack-infra | 18:29 | |
*** Sukhdev has joined #openstack-infra | 18:32 | |
*** jamesmcarthur has quit IRC | 18:32 | |
*** hasharAway is now known as hashar | 18:33 | |
zxiiro | anyone know if it's possible to edit the build description from within a jenkins job? | 18:35 |
*** jamesmcarthur has joined #openstack-infra | 18:36 | |
pabelanger | zxiiro: see jenkins/jobs/defaults.yaml in project-config | 18:37 |
*** jsavak has quit IRC | 18:37 | |
*** jamesmca_ has joined #openstack-infra | 18:37 | |
*** jsavak has joined #openstack-infra | 18:38 | |
zxiiro | pabelanger: no i don't mean the job description. I mean the specific build description. Like if you run a job and it creates build #5, you can edit the description of that build run. | 18:39 |
zxiiro | I have a variable from a job i want to attach to the build #'s description | 18:39 |
zxiiro | trying to find if there's a plugin to handle that but no luck yet... maybe have to do a groovy script or something | 18:39 |
*** inc0 has joined #openstack-infra | 18:40 | |
mordred | zxiiro: maybe with groovy | 18:40 |
mordred | yah | 18:40 |
*** simondodsley has joined #openstack-infra | 18:40 | |
*** sarob has joined #openstack-infra | 18:40 | |
mordred | zxiiro: you already said that :) | 18:40 |
*** jamesmcarthur has quit IRC | 18:41 | |
*** hosanai has joined #openstack-infra | 18:41 | |
*** amitgandhinz has quit IRC | 18:41 | |
*** amitgandhinz has joined #openstack-infra | 18:42 | |
jroll | jeblair: ansible disappearing sounds fun :) | 18:43 |
mordred | jroll: I mean ... fun is one word for it | 18:44 |
*** hosanai has quit IRC | 18:45 | |
*** jamesmca_ has quit IRC | 18:46 | |
zaro | zxiiro: i think that requires a plugin | 18:46 |
pabelanger | mordred: jroll: where did we see that? | 18:46 |
zxiiro | zaro: as you said that I think i just found it https://wiki.jenkins-ci.org/display/JENKINS/Description+Setter+Plugin | 18:47 |
jroll | pabelanger: "our ansible module briefly disappears from the file system when we reinstall zuul (which we do every time we land a patch)" | 18:47 |
jroll | known issues according to jim | 18:47 |
*** andreww has quit IRC | 18:47 | |
*** jamesmcarthur has joined #openstack-infra | 18:47 | |
pabelanger | Eep | 18:47 |
jeblair | pabelanger: i have an idea to fix it, i'll write a patch shortly | 18:48 |
*** ccamacho has quit IRC | 18:50 | |
fungi | pabelanger: yeah, non-atomic replacement of files ansible is relying on, so it just sort of crumbles at that point | 18:50 |
*** jamesmcarthur has quit IRC | 18:51 | |
*** ccamacho has joined #openstack-infra | 18:52 | |
*** jsavak has quit IRC | 18:52 | |
*** cody-somerville has joined #openstack-infra | 18:52 | |
*** jsavak has joined #openstack-infra | 18:53 | |
anteaya | fungi: jeblair storyboard has grown utc timestamps again: https://storyboard-dev.openstack.org/#!/story/8 | 18:53 |
fungi | anteaya: i saw that patch streak by... thanks SotK! | 18:53 |
anteaya | yay thanks SotK | 18:53 |
*** _nadya_ has quit IRC | 18:54 | |
anteaya | and Zara for reviewing and mergeing | 18:54 |
pabelanger | jeblair: when you have a moment, would be interested in our feedback on 330827 | 18:54 |
fungi | absolutely | 18:54 |
fungi | i need to step away and get some yardwork done before the rains come. bbiab | 18:54 |
anteaya | enjoy sunshine | 18:54 |
anteaya | and fresh air | 18:54 |
fungi | more like grass clippings up my nose, but thanks! | 18:55 |
anteaya | wooo, even better | 18:56 |
anteaya | chlorophyl is good for you | 18:56 |
*** aysyd has quit IRC | 18:58 | |
jeblair | pabelanger: we should ask notmorgan to look at 330827 since he helped us make it so that we *don't* store host keys for workers :) | 18:58 |
zaro | zxiiro: cool, there’s also one that sets a build number, https://wiki.jenkins-ci.org/display/JENKINS/Next+Build+Number+Plugin | 18:59 |
notmorgan | jeblair: looking now | 18:59 |
*** javeriak has quit IRC | 18:59 | |
zxiiro | zaro: neat! | 18:59 |
lifeless | ianw: a distro installed testtools or something perhaps? Got a pip install output to look at? | 18:59 |
zaro | zxiiro: python-jenkins has support for that one. | 19:00 |
*** sarob_ has joined #openstack-infra | 19:00 | |
jeblair | pabelanger, notmorgan: we did that because if we share a single known_hosts file, it will grow forever, and it will have old host keys from previous jobs. so i think this approach of using a per-job file in 330827 is good. | 19:00 |
notmorgan | jeblair: sometimes i'm paying attention :) | 19:00 |
jeblair | notmorgan: i mostly said that to remind myself :) | 19:01 |
anteaya | jeblair: fungi so timeline will now look like this: https://storyboard-dev.openstack.org/#!/story/7 instead of https://storyboard.openstack.org/#!/story/2000407 since this patch has merged: https://review.openstack.org/#/c/327231/ can we call happiness on timelines then? | 19:01 |
notmorgan | hehehe | 19:01 |
notmorgan | so we are storing known hosts here - and we want to store them but per job? | 19:02 |
jeblair | notmorgan: i just put a comment in gerrit about *why* we want to do this | 19:02 |
jeblair | notmorgan: not sure if you saw that "excitement" from yesterday | 19:02 |
*** pvaneck has quit IRC | 19:02 | |
notmorgan | right | 19:02 |
*** pvaneck has joined #openstack-infra | 19:02 | |
*** ddieterly[away] has quit IRC | 19:03 | |
*** amoralej is now known as amoralej|off | 19:03 | |
*** aysyd has joined #openstack-infra | 19:03 | |
*** sarob has quit IRC | 19:03 | |
notmorgan | jeblair: that makes sense. the question is do we need to even store known hosts at all for the worker? | 19:04 |
jeblair | notmorgan: good question -- we are speculating that we were seeing problems with ansible logging into different instances competing for the same ip | 19:04 |
jeblair | notmorgan: and hoping/assuming that known_hosts would cause that to blow up if it happens again | 19:05 |
jeblair | notmorgan: but i do wonder about controlmaster | 19:05 |
notmorgan | jeblair: ahh | 19:05 |
notmorgan | ok then. | 19:05 |
jeblair | notmorgan: like -- even if ansible does a bunch of ssh calls -- shouldn't controlmaster make them use the same connection anyway? | 19:05 |
notmorgan | this makes a lot of sense :) | 19:05 |
jeblair | but we're off in the weeds here, so i dunno | 19:05 |
notmorgan | jeblair: yes. | 19:05 |
*** jamesmcarthur has joined #openstack-infra | 19:05 | |
notmorgan | it should, but this is a good immediate metric of "oh hey, yeah this isn't happening" | 19:06 |
*** andreww has joined #openstack-infra | 19:06 | |
jeblair | (but maybe something was slow and controlmaster terminated and so we got a new connection) | 19:06 |
*** jamesmca_ has joined #openstack-infra | 19:06 | |
jeblair | notmorgan, pabelanger, fungi, mordred: response on ticket: http://paste.openstack.org/raw/518495/ | 19:07 |
*** e0ne has joined #openstack-infra | 19:08 | |
notmorgan | hmm | 19:08 |
jeblair | there's some definite misunderstanding of the question there. i don't think i am completely confident they have eliminated our hypothesis. | 19:08 |
*** javeriak has joined #openstack-infra | 19:08 | |
notmorgan | ok - well i have zero issue (just checking the code one more time) with this fix. | 19:08 |
mordred | jeblair: I don't either | 19:08 |
notmorgan | jeblair: and we can continue to poke at the tickets / vms / etc | 19:08 |
jeblair | i will ask if they can say whether the host that the old instance was on had any problems, or was removed from/added to service | 19:08 |
*** jamesmcarthur has quit IRC | 19:10 | |
notmorgan | jeblair: oh ... uhm make sure HashKnowHosts is false | 19:10 |
notmorgan | jeblair: just in case we want to check the IP and not guess | 19:11 |
notmorgan | jeblair: otherwise this looks good. | 19:11 |
* notmorgan will add a comment | 19:11 | |
notmorgan | i assume we're going to upload the knownhost file as well at the success/fail of the job? | 19:11 |
jeblair | notmorgan: yeah, the way the patch is written, that will happen | 19:12 |
odyssey4me | if anyone has a moment to help me figure out what's wrong with a tox config patch ( https://review.openstack.org/331251 ) I'd appreciate it - the zuul log says it can't open the upper-constraints file, even though it also appears from http://logs.openstack.org/51/331251/1/check/gate-openstack-ansible-memcached_server-linters/22f608c/console.html#_2016-06-17_18_06_30_738546 that it should be there | 19:12 |
jeblair | notmorgan: (uploading will happen) | 19:12 |
*** sarob_ has quit IRC | 19:12 | |
odyssey4me | hrm, I just thought of the idea that perhaps it's not there even if an env var is set for it | 19:12 |
notmorgan | jeblair: +1 with the knownhosts comment - shouldn't matter, but hey... | 19:13 |
notmorgan | sometimes suspenders and a belt is the right answer | 19:13 |
jeblair | notmorgan: ah, your old patch put hash false for the zuul user in its .ssh/config | 19:15 |
jeblair | so i think this is sufficient, but it would be good to make zuul more self-contained here | 19:16 |
notmorgan | jeblair: ++ | 19:16 |
*** jamesmca_ has quit IRC | 19:16 | |
*** jamesmcarthur has joined #openstack-infra | 19:17 | |
*** gyee has joined #openstack-infra | 19:17 | |
jeblair | mordred: this would be so much nicer with your nova key thing | 19:20 |
*** jamesmcarthur has quit IRC | 19:22 | |
*** sarob has joined #openstack-infra | 19:23 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config: Enable linters job to use upper-constraints https://review.openstack.org/331317 | 19:23 |
odyssey4me | heh, found the problem - a speedy review by project-config cores on https://review.openstack.org/331317 would be appreciated | 19:24 |
sc68cal | oh yahoo. why u do dis? http://paste.openstack.org/show/518501/ | 19:24 |
openstackgerrit | Merged openstack-infra/zuul: Enabled host_key_checking https://review.openstack.org/330827 | 19:25 |
*** piet has quit IRC | 19:25 | |
*** piet has joined #openstack-infra | 19:26 | |
rcarrillocruz | asselin_: no, should not be broken, not sure | 19:26 |
rcarrillocruz | i'll take a look next week | 19:27 |
*** sarob has quit IRC | 19:27 | |
*** ddieterly has joined #openstack-infra | 19:27 | |
openstackgerrit | Khai Do proposed openstack-infra/system-config: run git gc instead of repack for git repos https://review.openstack.org/329566 | 19:28 |
pabelanger | jeblair: notmorgan: Right, I too am hoping SSH blows up. | 19:28 |
anteaya | sc68cal: does invoking the word yahoo autmatically force the invoker to use text speak | 19:30 |
anteaya | sc68cal: I had given you credit for actually being able to speak in real sentences | 19:30 |
sc68cal | anteaya: heh. it's an internet meme / joke | 19:30 |
AJaeger | anteaya: I think the zuul-git-prep-upper-constraints is fine | 19:30 |
AJaeger | it just sets up a few more files - projects still need to use it | 19:31 |
zaro | any infra-core want to enable git gc so we can see how gerrit perfoms next week? https://review.openstack.org/#/c/329566 | 19:31 |
anteaya | sc68cal: ah | 19:31 |
anteaya | AJaeger: does project-config and system-config use constraints? | 19:31 |
anteaya | I didn't think we did | 19:31 |
AJaeger | anteaya: they don't - but no problem | 19:31 |
*** harlowja has joined #openstack-infra | 19:32 | |
AJaeger | the change *allows* use to use constraints there. | 19:32 |
AJaeger | but that is still a choice in tox.ini | 19:32 |
AJaeger | the change just makes the file available | 19:32 |
AJaeger | And then tox.ini needs to use it - and infra will just not use it | 19:32 |
*** gouthamr has quit IRC | 19:32 | |
anteaya | oh okay | 19:33 |
*** inc0 has quit IRC | 19:33 | |
*** Jeffrey4l has quit IRC | 19:34 | |
*** ociuhandu has quit IRC | 19:34 | |
*** dimtruck is now known as zz_dimtruck | 19:34 | |
odyssey4me | AJaeger I haven't confirmed whether that job still does the work of the other job too... ie is it a wholesale replacement, just with some extras | 19:34 |
*** sarob has joined #openstack-infra | 19:35 | |
AJaeger | odyssey4me: pep8 and linters should be the same, just different tox invocation and name | 19:37 |
AJaeger | odyssey4me: so, your change looks fine to me. | 19:37 |
odyssey4me | AJaeger yeah, that's what I thought | 19:37 |
odyssey4me | ok cool | 19:37 |
anteaya | odyssey4me: yeah that is what I am looking at: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml#n13 | 19:37 |
mordred | jeblair: many things would be nicer with my nova key thing | 19:38 |
pabelanger | interesting spike in zl05 used nodes: http://grafana.openstack.org/dashboard/db/nodepool | 19:38 |
odyssey4me | anteaya ok, it seems pretty clear to me that zuul-git-prep and gerrit-git-prep are equivalent... they do the same thing in different ways | 19:38 |
pabelanger | not sure what is happening there | 19:38 |
*** sarob has quit IRC | 19:39 | |
odyssey4me | anteaya and zuul-git-prep-upper-constraints is much the same, with the extra bit for upper constraints | 19:39 |
*** infra-red has joined #openstack-infra | 19:39 | |
*** infra-red has quit IRC | 19:39 | |
*** tonytan4ever has quit IRC | 19:39 | |
AJaeger | yep | 19:39 |
odyssey4me | I also see now why my zuul clone mapping wasn't working. :p | 19:39 |
AJaeger | as long as this is not run in a *post* job, it's fine - and that should be the case | 19:40 |
waynr | zaro: any chance we can get a python-jenkins release any time soon? | 19:40 |
AJaeger | btw. could a project-config core review https://review.openstack.org/330725 , please? | 19:40 |
mordred | pabelanger: the nodepool graph page is amazing | 19:40 |
*** hosanai has joined #openstack-infra | 19:41 | |
pabelanger | mordred: ya, we're pretty even for used nodes across all our zuul-launchers. Which is awesome | 19:42 |
waynr | also, does anyone know how we can get released versions of python-jenkins documentation on readthedocs? all I see (http://readthedocs.org/projects/python-jenkins/) is "master" and "stable" | 19:42 |
mordred | it's also just pretty with the different streams crossing and stuff | 19:42 |
pabelanger | but, don't know why zl05 jumped by 30nodes at 19:00UTC | 19:42 |
anteaya | odyssey4me AJaeger yes, I just needed to take the time to look at this | 19:43 |
pabelanger | mordred: https://review.openstack.org/#/c/331281/ should give us better control on the interval for the graphs too. Right now we're doing 1m summarize, that adds up to 1d summarized | 19:43 |
mordred | woot | 19:44 |
*** rajinir has quit IRC | 19:44 | |
*** hosanai has quit IRC | 19:46 | |
*** mriedem has quit IRC | 19:46 | |
jeblair | pabelanger: wait, why do we need that? | 19:46 |
pabelanger | jeblair: There is a video in the backscroll explaining what is does. Let me find it | 19:47 |
pabelanger | https://www.youtube.com/watch?v=CLXKlgHvOuo | 19:47 |
jeblair | pabelanger: i know what it does which is why i'm asking :) | 19:47 |
*** Jeffrey4l has joined #openstack-infra | 19:47 | |
odyssey4me | thanks AJaeger & anteaya :) | 19:47 |
pabelanger | basically, as you zoom out, I'd like to summarize to something other then secods | 19:48 |
rcarrillocruz | jesusaur: hipchat died for me | 19:48 |
rcarrillocruz | so | 19:48 |
pabelanger | seconds* | 19:48 |
rcarrillocruz | http://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/update_puppet.yaml | 19:48 |
*** jsavak has quit IRC | 19:48 | |
anteaya | odyssey4me: thank you | 19:48 |
rcarrillocruz | good thing is that you can pass to ansible-galaxy a yaml, containing what roles you want it to install | 19:48 |
jeblair | pabelanger: those graphs are not a rate, so it does not make sense to summarize them | 19:48 |
anteaya | zaro: have you selected a gerrit topic you would like to use for the gerrit storyboard plugin patches? | 19:48 |
jesusaur | rcarrillocruz: oh interesting | 19:49 |
*** samueldmq has quit IRC | 19:49 | |
rcarrillocruz | and the roles.yaml | 19:49 |
rcarrillocruz | http://git.openstack.org/cgit/openstack-infra/system-config/tree/roles.yaml | 19:49 |
rcarrillocruz | so that installs under /etc/ansible/roles | 19:49 |
rcarrillocruz | cloud-launcher | 19:49 |
rcarrillocruz | and puppet | 19:49 |
*** Apoorva has quit IRC | 19:49 | |
rcarrillocruz | you could probably have a cron or something on the gozer ansible control machine | 19:49 |
hashar | mordred: jeblair: congratulations on having phased out Jenkins and solely relying on ansible :] | 19:49 |
rcarrillocruz | to have an up to date roles all the time | 19:49 |
pabelanger | jeblair: Is there another function that allows use to change the frequency at which we plot on the graph? I find 10sec to busy when I'm viewing over a day period | 19:50 |
anteaya | zaro: like to you consider the patch to fix gerrit plugin installation to be critial to getting the storyboard plugin on review-dev? https://review.openstack.org/#/c/259653/ | 19:50 |
jeblair | pabelanger: in fact, one of the nice things about grafana is that as you zoom out, those graphs aggregate automatically as needed | 19:50 |
jeblair | pabelanger: it's based on the size of the graph and the time period being graphed | 19:50 |
jeblair | pabelanger: change the time period in the top to, say 24 hours and you'll get something like 5 minutes for each point | 19:50 |
pabelanger | jeblair: right, I was looking to adjust that data point more, so say 1 hour | 19:51 |
jesusaur | rcarrillocruz: does galaxy add /etc/ansible/roles to the roles_path, or do I need to manage that myself? | 19:51 |
jeblair | pabelanger: why do you want that? | 19:52 |
*** mriedem has joined #openstack-infra | 19:53 | |
pabelanger | jeblair: personal preference I guess. | 19:53 |
*** e0ne has quit IRC | 19:53 | |
jeblair | pabelanger: you know the default aggrecation function for summarize is 'sum'. which means you will add all the values together. | 19:55 |
jeblair | pabelanger: so that graph might look nicer, but will be wrong. | 19:55 |
*** markvoelker has quit IRC | 19:55 | |
*** sarob has joined #openstack-infra | 19:55 | |
jeblair | pabelanger: so i think if you want to do that, you'll need to set it to avg | 19:56 |
jeblair | pabelanger: https://graphite.readthedocs.io/en/0.9.10/functions.html#graphite.render.functions.summarize | 19:56 |
*** ddieterly is now known as ddieterly[away] | 19:57 | |
jeblair | (i like my reference documentation in the form of documentation, not videos :) | 19:57 |
pabelanger | jeblair: right, it is possible I have it incorrect. Let me explain what I was hoping for, and see if there is a better function to use | 19:57 |
*** raildo is now known as raildo-afk | 19:57 | |
jeblair | pabelanger: (also, consider smartSummarize over summarize) | 19:57 |
rcarrillocruz | jesusaur: i believe the default of roles path is already /etc/ansible/roles | 19:57 |
rcarrillocruz | but regardless | 19:57 |
rcarrillocruz | that can be set on ansible.cfg | 19:57 |
openstackgerrit | Merged openstack-infra/project-config: Enable linters job to use upper-constraints https://review.openstack.org/331317 | 19:58 |
*** sarob has quit IRC | 19:58 | |
rcarrillocruz | http://docs.ansible.com/ansible/intro_configuration.html#roles-path | 19:58 |
nwkarsten | clarkb, fungi: i think my issue was that the $ZUUL_PROJECT was being set to "nova" not "openstack/nova". do you know where to change that? | 19:58 |
*** sarob has joined #openstack-infra | 19:59 | |
rcarrillocruz | jesusaur: yeah, that's the default http://docs.ansible.com/ansible/galaxy.html#roles-path | 19:59 |
rcarrillocruz | so it will just work as you expect | 19:59 |
pabelanger | Using Last 24 hours quick range, I would like to see the data plotted in the In Use panel to have a frequency of 1hr between data points. That value, is the number of used nodes, we totaled for the hour | 19:59 |
pabelanger | if that makes sense | 19:59 |
rcarrillocruz | should you need further locations for roles, just change roles_path on ansible.cfg | 19:59 |
*** piet has quit IRC | 20:00 | |
jeblair | pabelanger: you won't be able to get that from this metric -- you'll need a metrice that's a counter (number of nodes used incrementing over time) or a rate (number of nodes used in 10 second intervals) | 20:00 |
jeblair | pabelanger: this metric is a guage, which means it's only updated when the number changes -- so you can't tell when an individual node is 'consumed' | 20:01 |
*** ddieterly[away] is now known as ddieterly | 20:01 | |
pabelanger | jeblair: okay, let me test in graphite.o.o using a different metric | 20:01 |
jeblair | pabelanger: (for instance, if inside of a 10 second statsd interval, a node went from ready to used and a different node went from used to delete, this number will stay the name, and there's no way to tell what happened) | 20:02 |
jeblair | pabelanger: yeah, i bet we have what you're looking for in a different metric :) | 20:02 |
*** nwkarsten has quit IRC | 20:06 | |
pabelanger | jeblair: Right, it looks like I incorrectly used that metric, because that was the only one (gauge) we used the node state. We'd have to revisit nodepool and see if it made sense to add another data type | 20:06 |
jeblair | pabelanger: i'd probably ask at this point what question you're trying to answer.... it's starting to sound like '# jobs zuul ran' which we have -- do you want '# of jobs zuul ran by launcher'? | 20:08 |
*** zz_dimtruck is now known as dimtruck | 20:08 | |
pabelanger | jeblair: I mostly was curious how many used nodes we had it a hour, in a day | 20:09 |
pabelanger | that how this all started | 20:09 |
*** nwkarsten has joined #openstack-infra | 20:09 | |
jeblair | pabelanger: that is fairly close to # jobs zuul ran (which does use summarize)... do you want to update that graph with the interval selector? | 20:09 |
pabelanger | jeblair: but you are right, I likely can use another metric to see that data | 20:10 |
*** markvoelker has joined #openstack-infra | 20:10 | |
jeblair | (i mean the difference between those numbers is only the difference between two different ways of measuring the same thing. they should be equivalent) | 20:10 |
jeblair | (minus jobs run on static nodes) | 20:10 |
pabelanger | jeblair: let me use graphite.o.o and see what that looks like, you are likely right | 20:11 |
*** piet has joined #openstack-infra | 20:13 | |
*** ewindisch has joined #openstack-infra | 20:13 | |
ewindisch | just here to @mordred. | 20:14 |
*** ddieterly has quit IRC | 20:14 | |
apuimedo | dhellmann: mriedem: If you could review https://review.openstack.org/#/c/331333/ | 20:14 |
*** nwkarsten has quit IRC | 20:14 | |
*** gouthamr has joined #openstack-infra | 20:14 | |
apuimedo | it would be very helpful | 20:14 |
apuimedo | kuryr is kind of blocked in its fullstack tests until this is fixed | 20:15 |
*** winggundamth_ has quit IRC | 20:15 | |
mriedem | apuimedo: you must have me confused with a requirements core | 20:16 |
mriedem | those guys told me i couldn't be part of the club | 20:16 |
mriedem | that my ideas were too 'radical' | 20:16 |
apuimedo | oh, sorry | 20:16 |
pabelanger | jeblair: right, so I believe Zuul Jobs Launcher (per hour): http://git.openstack.org/cgit/openstack-infra/project-config/tree/grafana/zuul-status.yaml#n161 provides that information I was looking for. Minus the granularity of which launcher it happen on | 20:17 |
apuimedo | mriedem: I thouoght you were Matt | 20:17 |
mriedem | apuimedo: no problem :) i'm joking about 1/2 of the above. | 20:17 |
jeblair | 💩ewindisch: hi there! | 20:17 |
*** akshai has quit IRC | 20:18 | |
*** Apoorva has joined #openstack-infra | 20:18 | |
* jeblair starts a new nick prefix style | 20:19 | |
apuimedo | dims: could you check https://review.openstack.org/#/c/331333/ | 20:19 |
dims | apuimedo : ack. will do | 20:19 |
apuimedo | dims: thanks a lot | 20:19 |
apuimedo | :-) | 20:19 |
pabelanger | mtreinish: it looks like stackviz failed again, breaking our nodepool DIBs for the day. Do you mind reviewing nodepool.o.o to see if we need to address anything? | 20:20 |
pabelanger | all images were affected | 20:21 |
*** amitgandhinz has quit IRC | 20:21 | |
timothyb89 | pabelanger: the cause was a broken dependency, I have already pushed a workaround: https://review.openstack.org/#/c/331256/ | 20:21 |
mtreinish | pabelanger: https://review.openstack.org/#/c/331256/ | 20:22 |
timothyb89 | pabelanger: I'm testing DIB right now to be sure, but that patch got gate tests working again | 20:22 |
pabelanger | timothyb89: thanks for the quick reply | 20:22 |
mtreinish | oh, timothyb89 was faster | 20:22 |
*** ihrachys has joined #openstack-infra | 20:22 | |
*** banix has quit IRC | 20:22 | |
*** thorst_ has quit IRC | 20:23 | |
mordred | ewindisch: yay me :) | 20:24 |
smcginnis | Any zuul issues right now? | 20:30 |
smcginnis | Had a patch get +A earlier and it never showed up in gate queue. | 20:30 |
smcginnis | Figured it was the earlier restart. | 20:30 |
smcginnis | https://review.openstack.org/#/c/327285/ | 20:30 |
smcginnis | Had them do a reverify on it and it kicked off in the check queue again. | 20:30 |
smcginnis | Passed but never picked up in the gate queue again. | 20:30 |
smcginnis | Not a dependent patch either. | 20:31 |
*** acoles is now known as acoles_ | 20:31 | |
anteaya | smcginnis: you figured out early that we programmed zuul to prevent your patches | 20:31 |
smcginnis | anteaya: Haha :) | 20:31 |
*** chem` has joined #openstack-infra | 20:32 | |
smcginnis | if cinder: sys.exit() | 20:32 |
pabelanger | zuul was restarted between 16:08 and 16:30 UTC, so events from gerrit during that time would not have been picked up by zuul | 20:32 |
anteaya | that's the line! | 20:32 |
anteaya | check results posted as success at 19:41 | 20:33 |
*** chem has quit IRC | 20:33 | |
anteaya | queue lengths are 0, I don't see it on the status page | 20:34 |
odyssey4me | I've seen a bit of chatter about some jobs passing tests as far as the log is concerned, but failing from a job standpoint... An example is http://logs.openstack.org/00/331100/1/gate/gate-openstack-ansible-repo_server-linters/54bbdb5/console.html / https://review.openstack.org/331100 | 20:34 |
odyssey4me | It seems that it's a known issue, and the workaround at this point is simply to recheck. | 20:35 |
odyssey4me | Any further pointers? Can we help in any way? | 20:35 |
anteaya | 2 conflicts with other patches, but no parents and no Depends-On: | 20:35 |
anteaya | smcginnis: I think we are going to have to impose on an infra-root to check zuul logs for https://review.openstack.org/#/c/327285/ | 20:36 |
pabelanger | odyssey4me: this looks to be a timeout issue. If you look at http://logs.openstack.org/00/331100/1/gate/gate-openstack-ansible-repo_server-linters/54bbdb5/_zuul_ansible/logs/ansible.txt you can see that our async task did complete | 20:36 |
odyssey4me | pabelanger ah: fatal: [node]: FAILED! => {"changed": false, "failed": true, "msg": "async task did not complete within the requested time"} | 20:37 |
odyssey4me | I saw a patch go in recently to change a setting related to timeouts | 20:37 |
jeblair | it kind of looks like that job took about 2 minutes to run though? | 20:37 |
openstackgerrit | Scott DAngelo proposed openstack-infra/project-config: Add experimental Cinder job for multibackend https://review.openstack.org/330678 | 20:38 |
anteaya | smcginnis: nikesh's reverify comment didn't help, but I don't think it hurt anything | 20:38 |
openstackgerrit | Mikhail S Medvedev proposed openstack-infra/ciwatch: Cleanup before Jenkins to Zuul rename https://review.openstack.org/331346 | 20:38 |
openstackgerrit | Mikhail S Medvedev proposed openstack-infra/ciwatch: Support Jenkins to Zuul rename https://review.openstack.org/331347 | 20:38 |
odyssey4me | s/go in/be submitted or perhaps merge/ | 20:38 |
jeblair | 2016-06-17 20:24:02,589 DEBUG zuul.NodeWorker.ubuntu-trusty-rax-ord-1777421: Ansible command: ['ansible-playbook', '/tmp/tmp3gMsXA/ansible/playbook', '-e', 'timeout=1800', '-v'] | 20:38 |
*** ihrachys has quit IRC | 20:38 | |
jeblair | odyssey4me, pabelanger: ^ so it had an 1800 second timeout (30 minutes) | 20:39 |
smcginnis | anteaya: Yeah, strange that it kicked off check again, but no luck with gate. | 20:39 |
odyssey4me | I mentioned the same thing this morning: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2016-06-17.log.html#t2016-06-17T10:10:08 | 20:40 |
jeblair | smcginnis, anteaya: i can look at it after i track this one down | 20:40 |
pabelanger | I would be interested to see what timeout we passed into async for that task | 20:40 |
asselin_ | rcarrillocruz, read the scroll back on ansible & system-config. Thanks for the links. Will continue looking into it. | 20:40 |
anteaya | jeblair: yup, thanks | 20:40 |
odyssey4me | another bit I think might be related: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2016-06-17.log.html#t2016-06-17T12:11:01 | 20:40 |
jeblair | odyssey4me: this is a better case because we had much incorrect about timeouts this morning. but this is recent enough it should have all fixes | 20:40 |
anteaya | smcginnis: in case that reverify did interrupt things, do you want to try a +! workflow on it? | 20:40 |
anteaya | +1 | 20:40 |
smcginnis | anteaya: Worth a shot. | 20:41 |
anteaya | smcginnis: we are waiting anyway | 20:41 |
jeblair | pabelanger: agree, though i don't know a good way to get that. we could do a zuul_log of "Remaining timeout {{ timeout }}" | 20:42 |
smcginnis | anteaya: And there it is, happily waiting in the gate queue. | 20:42 |
odyssey4me | ok, I see the various tons of chatter from 13:00 onwards | 20:42 |
anteaya | smcginnis: there it goes | 20:42 |
smcginnis | anteaya: Thanks! | 20:42 |
anteaya | smcginnis: welcome, looks like the reverify comment did short circuit things | 20:42 |
*** hosanai has joined #openstack-infra | 20:42 | |
jeblair | anteaya, smcginnis: did it already have a +1 or +2 before the recheck by any chance? | 20:43 |
jeblair | (from jenkins) | 20:43 |
pabelanger | jeblair: I think zuul_log would be a good location. Our only other way would be to add a debug task because each zuul_runner with the msg={{ timeout }} | 20:43 |
pabelanger | which grows our playbooks | 20:43 |
smcginnis | jeblair: Yeah, +2/+A, so I thought the reverify should give it a kick. | 20:43 |
smcginnis | jeblair: But only moved after I removed my vote, then did +2/+A again. | 20:44 |
jeblair | smcginnis: it had a +2 from jenkins? | 20:44 |
anteaya | jeblair: it had a +1 Verifying from jenkins yes | 20:44 |
anteaya | then a +2 from smcginnis, then +2 +A from xing | 20:44 |
jeblair | anteaya: ah, a +1 makes more sense. in that case, gerrit does not emit an event that zuul can process (because "nothing changed"). clarkb has a patch to gerrit to change that. i don't know the status. | 20:44 |
anteaya | then 'reverify' | 20:44 |
*** rhochmuth has joined #openstack-infra | 20:45 | |
anteaya | thank you | 20:45 |
jeblair | anteaya, smcginnis: in this case, -W +W is the way to get things going | 20:45 |
anteaya | thanks | 20:45 |
smcginnis | jeblair: OK, thanks. | 20:45 |
anteaya | smcginnis: and please tell nikesh to ease back on the recheck/reverify | 20:45 |
pabelanger | jeblair: I'd be in favor of adding http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n165 by zuul_log for each zuul_runner task we do | 20:45 |
jeblair | pabelanger: debug task might be good -- | 20:46 |
pabelanger | jeblair: let me do a quick test to see if -v will output it | 20:46 |
jeblair | pabelanger: it'll end up in the ansible log, so people can grab it | 20:46 |
jeblair | pabelanger: but if we want it in the console log, that works too | 20:47 |
jeblair | pabelanger: i'm kind of on the fence (mostly wondering whether it's useful or noise in the console log) | 20:47 |
*** hosanai has quit IRC | 20:47 | |
clarkb | jeblair zaro got it in iirc but would be 2.12 | 20:47 |
jeblair | anteaya: woot ^ | 20:47 |
pabelanger | jeblair: same, I mean, it is useful now because there is an issue. But in that case, ansible.log makes more sense | 20:48 |
clarkb | I am not sure any of the code I fiddled with made it but the fix is there for the future so yay | 20:48 |
jeblair | pabelanger: cool, let's start there | 20:48 |
*** banix has joined #openstack-infra | 20:48 | |
jeblair | pabelanger: (and actually, now that we upload the log, maybe we should drop task exit code line | 20:48 |
anteaya | jeblair: clarkb zaro yay! | 20:48 |
*** thorst_ has joined #openstack-infra | 20:49 | |
odyssey4me | hmm, perhaps to ask the stupid question - why is the task async at all? | 20:49 |
jeblair | odyssey4me: without it, we hit ssh timeout errors | 20:49 |
jeblair | odyssey4me: (on, eg, huge devstack jobs) | 20:49 |
anteaya | clarkb: sometimes fiddling with the code is enough to get the attention of the folks who will actually fix it | 20:49 |
*** mhickey has quit IRC | 20:49 | |
anteaya | clarkb: so thanks for that | 20:49 |
jeblair | odyssey4me: (also, bonus, async seems to be the best way to have ansible time out a task) | 20:50 |
jeblair | odyssey4me: (current situation notwithstanding :) | 20:50 |
openstackgerrit | Peter Stachowski proposed openstack-infra/project-config: [trove] Add trove-dashboard to PROJECTS https://review.openstack.org/331352 | 20:51 |
jeblair | odyssey4me, pabelanger: yeah, i don't have an answer for this other than it sure does look like ansible decided to abort the task after 24 seconds. i would feel a lot better about proceeding with debugging this if we had a log line right above that that either did or did not say that we set the timeout to 24 seconds. | 20:52 |
notmorgan | jeblair: so, i havea question for you re: the py3 (encode/decode/encode) things. if we're still replicating all the py3 conversions to the zuulV3 branch, I'll do that cleanup there - if not i can do the more intelligent encode/decode stuff here. | 20:52 |
odyssey4me | jeblair hmm, yeah I guess if you're doing huge monolithic scripts executed by the command/shell module then that would be an issue unless you have something external to kill the task | 20:52 |
notmorgan | jeblair: i am thinking of adding a | 20:52 |
notmorgan | "auto-encode/decode" interface directly in gear so I don't need to battle with gear looking at something different/receiving different output that zuul does. | 20:53 |
pabelanger | jeblair: odyssey4me: this is what we'd see in the ansible.log for example: http://paste.openstack.org/show/518616/ | 20:53 |
pabelanger | are we okay with that? | 20:53 |
rcarrillocruz | asselin_: if by scrollback you mean hipchat, i'll check it out on Monday, cos I can't connect... | 20:53 |
odyssey4me | jeblair the ssh timeout jobs, though, might be a different problem - we've seen that from time to time depending on various settings... perhaps with a little more info on that one we can find an alternative solution | 20:53 |
notmorgan | where a subclass is going to have to look a lot like that anyway. | 20:53 |
notmorgan | mordred: ^ cc re jeblair's comment on https://review.openstack.org/#/c/325661/ | 20:54 |
*** trown is now known as trown|outtypewww | 20:54 | |
asselin_ | rcarrillocruz, no I mean here (and the start over there). hipchat is down. | 20:54 |
notmorgan | Shrews: ^ cc as well. | 20:54 |
rcarrillocruz | ah ok | 20:54 |
* rcarrillocruz checks | 20:54 | |
odyssey4me | pabelanger jeblair in terms of noise versus value, a few lines to give more debugging capacity versus no info at all... I'd rather have more lines | 20:54 |
jeblair | notmorgan: okay, i like the idea of being able to make this easier for everyone in gear -- i worry about what that interface will look like (which i don't have to worry as much about in a zuul subclass)... maybe you can sketch it out before doing all the work? | 20:54 |
notmorgan | jeblair: basically i was going to add supplemental interface that was an @property for the name/unique/etc with setter/getter that encodes/decodes | 20:55 |
notmorgan | jeblair: haven't set on the "name" | 20:55 |
*** kgiusti has left #openstack-infra | 20:56 | |
asselin_ | rcarrillocruz, too bad those os_network ansible roles don't use my clouds.yaml config for cert verify=false | 20:56 |
notmorgan | jeblair: so if you reference .name, it's default like it is today - or make .name the auto-encode/decode and an internal property gear references | 20:56 |
notmorgan | jeblair: i haven't decided yet. was mulling on that already actually :) | 20:56 |
notmorgan | specifically because of this patch. | 20:57 |
jeblair | notmorgan: yeah, i like that gear is very 'pythonic' [aside: the fact that it isn't in python3 suggests something to me about how 'pythonic' python3 is] | 20:57 |
rcarrillocruz | asselin_: yeah, that's a bug | 20:58 |
jeblair | notmorgan: so i feel like just being able to say 'job.name' is important. so figuring out how to make that sensible in the most cases is important. | 20:58 |
rcarrillocruz | i would have sworn i hit that and mordred fixed it on shade... | 20:58 |
rcarrillocruz | i will search on git | 20:58 |
rcarrillocruz | asselin_: feel free to open a bug on ansible | 20:58 |
rcarrillocruz | i'll take it | 20:59 |
asselin_ | launchpad? | 20:59 |
rcarrillocruz | github | 20:59 |
notmorgan | jeblair: right so i'll plan job.name to be public interface and have job._gear_thing_whatever_it's_called [though also public if someone wants to poke at it] be the way gear manages the byte-requirement of the policy. but this does imply we'll be breaking anyone (only us?) with that upgrade and did it the old way. | 20:59 |
odyssey4me | pabelanger jeblair I have a few more sample of data if you'd like them? | 20:59 |
notmorgan | s/policy/properties | 20:59 |
asselin_ | https://github.com/ansible/ansible ? | 20:59 |
rcarrillocruz | i think should be here | 21:00 |
rcarrillocruz | https://github.com/ansible/ansible-modules-core/issues | 21:00 |
jeblair | notmorgan: maybe we should have an email thread | 21:00 |
notmorgan | jeblair: or, i could just make a new class. anyway, I'll look at the options. But the auto-encode/decode should absolutely be part of gear itself. | 21:00 |
notmorgan | jeblair: ++. | 21:00 |
asselin_ | rcarrillocruz, ok, still getting up to speed on ansible and seeing how the pieces fit together. | 21:00 |
jeblair | notmorgan: well, i think doing this in gear can be good -- but i'm not 100% convinced it's the right/only way. gear also wants to be a low-level protocol interface. | 21:01 |
rcarrillocruz | yep | 21:01 |
rcarrillocruz | let me know how the launcher works for you | 21:01 |
rcarrillocruz | also | 21:01 |
rcarrillocruz | https://review.openstack.org/#/c/320466/11/playbooks/launch_node.yml | 21:01 |
rcarrillocruz | to replace the launch_node with the cloud launcher | 21:01 |
rcarrillocruz | in case you want to look at more examples | 21:01 |
notmorgan | jeblair: sure. but it feels like this is a usability thing. we're not really changing the protocol, just how someone interfaces with the bytes vs string requirements in py3. In py2 this "just works", and i like that. | 21:02 |
notmorgan | jeblair: i'll get an email thread going on this though. | 21:02 |
*** davidlenwell has quit IRC | 21:02 | |
jeblair | notmorgan: yeah, we spent a lot of time trying to get this right. i'm surprised it's so terrible in python3. | 21:03 |
mordred | jeblair: I like a ton of the new stuff in python3 - the bytes/string handling is not one of the things | 21:03 |
notmorgan | jeblair: the difference between py2 and py3 bytesvstext seems to be py3 is consistently bad in the same way - py2 was inconsistently bad. | 21:03 |
asselin_ | rcarrillocruz, I like the separation from infra-ansible. I'm getting stuck on creating the sub-network. http://paste.openstack.org/show/518619/ | 21:03 |
notmorgan | jeblair: so it was much harder to "Get right" in py2 | 21:03 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul: Add debug task before zuul_runner https://review.openstack.org/331355 | 21:03 |
odyssey4me | pabelanger jeblair ok, I'm out for the weekend... have a great one everyone - catch you next week! | 21:04 |
rcarrillocruz | paste me your resources.yml | 21:04 |
pabelanger | jeblair: odyssey4me: ^ for our debug command | 21:04 |
notmorgan | jeblair: also.. b'abdc'[0] => 97 | 21:04 |
mordred | notmorgan: yah - I think in py2 it's almost impossible to get non-ascii right. in py3 it's ALSO hard to get ascii right, but it's consistently hard no matter whether you're doing ascii or not | 21:04 |
*** javeriak has quit IRC | 21:04 | |
notmorgan | jeblair: in py3 | 21:04 |
notmorgan | mordred: exactly | 21:04 |
odyssey4me | pabelanger FYI https://review.openstack.org/#/c/331076/ and https://review.openstack.org/#/c/331100/ both have the timeout issue in their logs if that matters | 21:05 |
pabelanger | odyssey4me: thanks, I will look at them | 21:05 |
asselin_ | rcarrillocruz, nice patch....I essentially did the same thing in a less elegant way. I'll try it out | 21:05 |
jeblair | notmorgan: i think we have some assumptions in gear that "strings" in py2 are to be encoded as utf8 | 21:06 |
jeblair | notmorgan: we may be in a slightly better position than we think to do automatic decoding | 21:06 |
*** burgerk has quit IRC | 21:06 | |
jeblair | notmorgan: see the "convert_to_bytes" function in gear | 21:06 |
rcarrillocruz | i want to put the generate tmpkeypair on the role itself | 21:06 |
notmorgan | jeblair: yeah. | 21:06 |
notmorgan | jeblair: i noticed that. | 21:07 |
rcarrillocruz | if a server does not have a keypair passed as param, create a temp one and return it after the task finishe | 21:07 |
rcarrillocruz | s | 21:07 |
jeblair | notmorgan: anyway, this is complex, and an email will i think give the space to help page everything in :) | 21:07 |
notmorgan | jeblair: which is why i just did encode to utf-8 directly vs trying to do platform locale lookups. | 21:07 |
rcarrillocruz | need to get to it | 21:07 |
notmorgan | jeblair: ++ | 21:07 |
mordred | jeblair: ++ | 21:07 |
notmorgan | jeblair: i'll rebase the current patch and get that email sent- so we can keep working on moving forward. | 21:07 |
notmorgan | regardless, the rebase has to happen :) | 21:08 |
jeblair | ok | 21:08 |
jeblair | odyssey4me: yes, i'd love more sample data! :) | 21:08 |
jeblair | odyssey4me: ah i see you noted them for pabelanger, got it, thanks | 21:09 |
fungi | anteaya: SotK: Zara: new timelines on storyboard-dev.o.o lgtm. awesome! | 21:09 |
fungi | nibalizer: crinkle: regarding https://review.openstack.org/329566 if you rename a cron resource do you need to ensure=>absent the old name? | 21:09 |
pabelanger | jeblair: I need to step away for a bit, please take over 331355 if you need too | 21:10 |
*** croelandt has quit IRC | 21:10 | |
*** hashar has quit IRC | 21:10 | |
jeblair | pabelanger, odyssey4me: it seems strange this happens so often on the linters job | 21:11 |
nibalizer | fungi: yea i think so | 21:11 |
anteaya | fungi: yay, thank you for looking | 21:11 |
jeblair | mordred: can you +3 https://review.openstack.org/331355 ? | 21:11 |
anteaya | Zara: SotK just knocking them out of the park, you are | 21:12 |
SotK | fungi: woo, just need to get rid of pagination now! | 21:12 |
*** daemontool has joined #openstack-infra | 21:12 | |
jeblair | mordred: i just tried 355 and it doesn't look right to me | 21:12 |
anteaya | SotK: you have a patch up for that | 21:12 |
*** thorst_ has quit IRC | 21:12 | |
anteaya | SotK: and it looks good to me | 21:12 |
*** davidlenwell has joined #openstack-infra | 21:13 | |
jeblair | 2016-06-17 14:12:19,778 DEBUG zuul.NodeWorker.test: Ansible output: "msg": "Async" | 21:13 |
jeblair | mordred, pabelanger: i got that ^ | 21:13 |
jeblair | i think we need to use data structures, not strings. respinning. | 21:13 |
anteaya | https://review.openstack.org/#/c/327682/4 | 21:14 |
mordred | jeblair: I agree | 21:14 |
jeblair | i'm just testing now with actual timeout values to make sure it dtrt | 21:15 |
*** Guest28113 has quit IRC | 21:15 | |
*** piet has quit IRC | 21:15 | |
*** andreww has quit IRC | 21:15 | |
*** andreww has joined #openstack-infra | 21:16 | |
jeblair | mordred: hrm. it outputs the passed timeout value every time, not the one that's updated by the timeout callback plugin | 21:17 |
jeblair | (which is a "nonpersistent fact") | 21:18 |
jeblair | mordred: i *thought* i had ensured that async was picking up the updated timeout value, now i am not certain of that either. | 21:18 |
*** ldnunes has quit IRC | 21:18 | |
*** chem` has quit IRC | 21:19 | |
*** cody-somerville has quit IRC | 21:20 | |
*** gordc has quit IRC | 21:21 | |
*** markvoelker has quit IRC | 21:21 | |
*** amitgandhinz has joined #openstack-infra | 21:22 | |
mordred | jeblair: that testing implies the same concern to me | 21:22 |
*** gouthamr has quit IRC | 21:22 | |
crinkle | fungi: yes | 21:24 |
fungi | crinkle: thanks! | 21:24 |
jeblair | mordred: the async_wrapper command line has a number on it which looks like it is the timeout. it doesn't appear to change from one task to the next, so it also seems to be using the original rather than updated value | 21:26 |
*** amitgandhinz has quit IRC | 21:27 | |
jeblair | this is annoying, because it means our timeouts are inaccurate -- but only in the direction of being too long. so it should not account for the error odyssey4me saw | 21:28 |
*** lascii is now known as alaski | 21:28 | |
*** rfolco has quit IRC | 21:32 | |
*** dmk0202 has quit IRC | 21:33 | |
mordred | jeblair: yes. that is doubly annoying | 21:33 |
anteaya | <-- going for a walk | 21:34 |
*** dmk0202 has joined #openstack-infra | 21:34 | |
jeblair | mordred: do you have any ideas? | 21:36 |
*** thorst has joined #openstack-infra | 21:36 | |
*** mdrabe has quit IRC | 21:36 | |
*** erlon has quit IRC | 21:36 | |
*** dprince has quit IRC | 21:36 | |
asselin_ | rcarrillocruz, do you know if creating and attaching volumes works? I didn't see it in the sample, and having issues with it. "msg": "Volume {'name': 'gfs02', 'volume_type': '3par-vol', 'size': 500} is not a valid volume in gpc-asselin:region1"} | 21:37 |
*** thorst has quit IRC | 21:39 | |
*** thorst has joined #openstack-infra | 21:40 | |
*** abregman|afk has quit IRC | 21:41 | |
*** bhavik1 has joined #openstack-infra | 21:42 | |
*** bhavik has quit IRC | 21:44 | |
*** bhavik1 is now known as bhavik | 21:44 | |
*** PollyZ has quit IRC | 21:45 | |
*** xyang1 has quit IRC | 21:48 | |
*** thorst has quit IRC | 21:49 | |
*** dmk0202 has quit IRC | 21:50 | |
zaro | waynr: i’ll do python-jenkins release early next week. | 21:51 |
zaro | waynr: i thought RTD automatically builds docs from all the tags but it doesn’t look like it. took a quick look but don’t see how i can change it to do that. | 21:51 |
*** daemontool has quit IRC | 21:53 | |
zaro | anteaya: none of my patches for storyboard plugin are ready for review yet so i have not assigned a topic yet. | 21:54 |
zaro | anteaya: what was your question regarding change https://review.openstack.org/#/c/259653/ | 21:55 |
mordred | jeblair: I do not. I also stepped away for just a moment- back in a few | 21:56 |
jeblair | mordred: i'm also not certain that the pre_tasks thing we did in 330827 is working | 21:56 |
*** mriedem has quit IRC | 21:59 | |
*** jcoufal has quit IRC | 21:59 | |
openstackgerrit | Scott DAngelo proposed openstack-infra/project-config: Add experimental Cinder job for multibackend https://review.openstack.org/330678 | 22:00 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul: Revert "Enabled host_key_checking" https://review.openstack.org/331362 | 22:01 |
jeblair | pabelanger, mordred, notmorgan: ^ i think the ansible is not right for that -- i don't see the keyscan running | 22:01 |
jeblair | i'm going to push that through until we can be more sure of it | 22:01 |
*** esberglu has quit IRC | 22:02 | |
openstackgerrit | Morgan Fainberg proposed openstack-infra/zuul: Python 3 Fixes: Bundled Fixes https://review.openstack.org/325661 | 22:05 |
openstackgerrit | Morgan Fainberg proposed openstack-infra/zuul: Update gear requirement https://review.openstack.org/326173 | 22:05 |
openstackgerrit | Morgan Fainberg proposed openstack-infra/zuul: Python 3 fix: GC final state inspection https://review.openstack.org/327436 | 22:06 |
*** vhosakot has quit IRC | 22:07 | |
notmorgan | jeblair: hmm. | 22:11 |
asselin_ | rcarrillocruz, per your request: https://github.com/ansible/ansible-modules-core/issues/3981 | 22:11 |
notmorgan | jeblair: i don't know | 22:11 |
*** baoli has quit IRC | 22:13 | |
openstackgerrit | Khai Do proposed openstack-infra/system-config: run git gc instead of repack for git repos https://review.openstack.org/329566 | 22:13 |
*** dimtruck is now known as zz_dimtruck | 22:13 | |
*** banix has quit IRC | 22:13 | |
*** pradk has quit IRC | 22:13 | |
*** bhavik has quit IRC | 22:16 | |
*** matt-borland has quit IRC | 22:18 | |
*** zz_dimtruck is now known as dimtruck | 22:18 | |
*** dimtruck is now known as zz_dimtruck | 22:20 | |
*** tphummel has quit IRC | 22:23 | |
*** amitgandhinz has joined #openstack-infra | 22:23 | |
*** tphummel has joined #openstack-infra | 22:25 | |
*** amitgandhinz has quit IRC | 22:27 | |
*** tpsilva has quit IRC | 22:27 | |
*** jcoufal has joined #openstack-infra | 22:30 | |
anteaya | zaro: do you require 259653 to be merged for the storyboard plugin to be on review-dev | 22:32 |
*** banix has joined #openstack-infra | 22:32 | |
zaro | no, that’s to fix upgrading plugins. for example if a sb-1.0 already exists and we upgrade to sb-2.0 | 22:33 |
anteaya | zaro: okay well I still would really like the storyboard plugin to be on review-dev by the sprint next week, and am very keen to know what patches are required so I can review and help you plus recruit other reviewers | 22:33 |
anteaya | ah okay thanks | 22:33 |
anteaya | regardless of the shape of them if I can at least envision how many are required, that you know of at this point, that would help me | 22:33 |
*** mixos has quit IRC | 22:35 | |
zaro | anteaya: topic is "its-storyboard" | 22:36 |
anteaya | thank you | 22:37 |
anteaya | thanks so much: https://review.openstack.org/#/q/topic:its-storyboard | 22:38 |
*** sdake has quit IRC | 22:38 | |
*** rhochmuth has quit IRC | 22:41 | |
*** sdake has joined #openstack-infra | 22:41 | |
*** rlandy has quit IRC | 22:43 | |
*** hosanai has joined #openstack-infra | 22:44 | |
*** thorst has joined #openstack-infra | 22:45 | |
*** aysyd has quit IRC | 22:45 | |
*** hosanai has quit IRC | 22:48 | |
mordred | jeblair: I have a wild idea on the timeout issue | 22:49 |
pabelanger | jeblair: sure. I'm going to test it again locally now | 22:49 |
mordred | jeblair: perhaps there is a timing/scope issue with the non-persistent facts - and maybe they're not in-scope at the right time | 22:49 |
mordred | jeblair: by maybe if we register the timeout value at the end of each task, the next task will pick up the value | 22:50 |
*** thorst has quit IRC | 22:53 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul: Register the timeout variable to pass to the next task https://review.openstack.org/331371 | 22:54 |
mordred | jeblair: something like that ^^ | 22:54 |
*** edmondsw has quit IRC | 22:59 | |
dmsimard | mordred: you can persist the fact, no need to keep it in-memory | 23:02 |
mordred | dmsimard: what do you mean? | 23:03 |
dmsimard | if the issue is that you don't want to lose the variable from task to task | 23:03 |
dmsimard | you can persist it | 23:03 |
*** jcoufal has quit IRC | 23:03 | |
mordred | in this case the variable is something that's injected by a callback plugin | 23:03 |
mordred | we start with a total timeout value, and then with each task we run it through a callback plugin and inject the new timeout value based on the elapsed time in the previous task | 23:04 |
dmsimard | hm, I've never done something like that before, not sure my solution makes sense in that context | 23:05 |
mordred | it's a pretty edge case :) | 23:05 |
*** rfolco has joined #openstack-infra | 23:05 | |
dmsimard | but basically you can have /etc/ansible/facts.d/whatever.fact and then have ini-style fact declaration in there | 23:06 |
mordred | yah. in this case it's the rewriting between tasks that's killing us I think | 23:06 |
dmsimard | mordred: btw: https://review.openstack.org/#/c/330874/ | 23:07 |
dmsimard | I'm out for the weekend now though, catch you later | 23:08 |
openstackgerrit | David Moreau Simard proposed openstack-infra/zuul: WIP: Integrate ARA reporting inside Zuul Launcher https://review.openstack.org/330874 | 23:11 |
*** IlyaG has joined #openstack-infra | 23:13 | |
*** ffrank has joined #openstack-infra | 23:16 | |
*** markvoelker has joined #openstack-infra | 23:16 | |
pabelanger | jeblair: I'll wait for you to debug locally ssh-keyscan, the only thing I can think of is maybe a path issue. I was able to extract the code for a locally test and the playbook looked to be correct | 23:16 |
*** bmjen has joined #openstack-infra | 23:19 | |
*** sarob has quit IRC | 23:19 | |
*** sarob has joined #openstack-infra | 23:20 | |
openstackgerrit | Spencer Krum proposed openstack-infra/vinz: Test commit https://review.openstack.org/331377 | 23:23 |
*** amitgandhinz has joined #openstack-infra | 23:24 | |
*** rfolco has quit IRC | 23:24 | |
fungi | heh | 23:27 |
*** hongbin has quit IRC | 23:27 | |
*** amitgandhinz has quit IRC | 23:28 | |
mordred | wow | 23:28 |
nibalizer | i just wanted to test | 23:31 |
nibalizer | haha the tests are failing on that repo with no content | 23:31 |
*** andreww has quit IRC | 23:35 | |
*** claudiub has quit IRC | 23:38 | |
waynr | zaro: cool | 23:43 |
*** denisra has quit IRC | 23:43 | |
*** hosanai has joined #openstack-infra | 23:44 | |
*** yamahata has quit IRC | 23:45 | |
*** eharney has quit IRC | 23:46 | |
*** gyee has quit IRC | 23:46 | |
*** hosanai has quit IRC | 23:49 | |
*** asselin_ has quit IRC | 23:49 | |
*** zz_dimtruck is now known as dimtruck | 23:50 | |
*** thorst has joined #openstack-infra | 23:51 | |
*** weshay has quit IRC | 23:52 | |
*** Sukhdev has quit IRC | 23:55 | |
*** thorst has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!