clarkb | jamesmcarthur: http://www.openstack.org/summit/openstack-summit-hong-kong-2013/agenda/ gives me a message about needing to login to the schedule site and may be trying to talk to the last summit (in portland) | 00:03 |
---|---|---|
clarkb | jamesmcarthur: thought that may be related to the work you just did so FYI :) | 00:03 |
jamesmcarthur | thanks! i'll pass that along. | 00:04 |
*** michchap has joined #openstack-infra | 00:05 | |
*** Steap has joined #openstack-infra | 00:05 | |
*** adrian_otto1 has joined #openstack-infra | 00:05 | |
jamesmcarthur | clarkb: no, but it's still something to be looked at. | 00:05 |
*** mriedem has joined #openstack-infra | 00:05 | |
jamesmcarthur | will pass along to the foundation staff | 00:06 |
*** jesusaurus has joined #openstack-infra | 00:06 | |
*** adrian_otto has quit IRC | 00:07 | |
anteaya- | -a | 00:09 |
*** adrian_otto1 has quit IRC | 00:10 | |
*** jamesmcarthur has left #openstack-infra | 00:10 | |
*** SergeyLukjanov has joined #openstack-infra | 00:12 | |
*** markwash has quit IRC | 00:14 | |
*** smarcet has left #openstack-infra | 00:16 | |
*** rakhmerov has quit IRC | 00:20 | |
*** fbo is now known as fbo_away | 00:23 | |
*** senk1 has quit IRC | 00:24 | |
*** melwitt has quit IRC | 00:26 | |
*** slong has quit IRC | 00:30 | |
*** michchap_ has joined #openstack-infra | 00:30 | |
*** matsuhashi has joined #openstack-infra | 00:30 | |
*** hogepodge has quit IRC | 00:32 | |
*** reed has quit IRC | 00:33 | |
*** michchap has quit IRC | 00:33 | |
*** oubiwann has quit IRC | 00:36 | |
*** melwitt has joined #openstack-infra | 00:41 | |
*** yaguang has joined #openstack-infra | 00:43 | |
*** michchap_ has quit IRC | 00:44 | |
fungi | sdague: looks like grenade's consistently failing after putting https://review.openstack.org/53940 through | 00:46 |
clarkb | fungi: I think that was a concern since we were asymettric before | 00:47 |
*** michchap has joined #openstack-infra | 00:47 | |
fungi | euca test failures in devstack for the ones i've looked at so far | 00:47 |
sdague | urg, it broke again ey? Do we have one that's made it all the way throught | 00:48 |
sdague | actually, euca I think is a different fail | 00:48 |
fungi | oh? | 00:48 |
fungi | mmm | 00:48 |
*** Ryan_Lane has quit IRC | 00:49 | |
sdague | fungi: yuo got log url for one of these? | 00:49 |
sdague | give me a few minutes to look before we straight revert it | 00:49 |
fungi | https://jenkins02.openstack.org/job/check-grenade-devstack-vm/3724/console | 00:49 |
sdague | I need the logs server url | 00:49 |
fungi | i was just looking at console logs in jenkins for the ones i spotted in the zuul status page | 00:50 |
fungi | i can go hunting | 00:50 |
sdague | right, but I need the other artifact logs to figure out what's going on | 00:50 |
clarkb | for that one it is http://logs.openstack.org/37/50537/7/check/check-grenade-devstack-vm/10ce925/ | 00:50 |
sdague | perfect, thanks | 00:50 |
clarkb | you can do copy pasta with the scp copy paths to figure it out | 00:50 |
fungi | http://logs.openstack.org/13/53513/4/check/check-grenade-devstack-vm/e3d8fc7/console.html | 00:51 |
fungi | oh, clarkb beat me to one | 00:51 |
*** ericw has quit IRC | 00:53 | |
sdague | ok, so I think I know what broke. | 00:54 |
sdague | this was actually the thing I was concerned would break, which is how does zuulifying "old" work | 00:54 |
*** SergeyLukjanov is now known as _SergeyLukjanov | 00:55 | |
sdague | that review is a nova change, and the logic in devstack-gate means that it actually set the old nova tree to the zuul master change | 00:55 |
*** jamesmcarthur has joined #openstack-infra | 00:55 | |
*** _SergeyLukjanov has quit IRC | 00:55 | |
fungi | right, when we want to override it to look in havana^Wgrizzly | 00:56 |
fungi | got any corresponding zuul ref, or fall back to branch tip | 00:56 |
fungi | er, for any | 00:56 |
*** jamesmcarthur has quit IRC | 00:57 | |
*** julim has quit IRC | 00:57 | |
sdague | so do we just need another $ZUUL_BRANCH=$BRANCH check? | 00:58 |
fungi | i think so, and at this point it should be self-testable since we're past the pre-reexec setup stuff | 00:59 |
fungi | right? | 00:59 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Only zuulify the tree if we are on the right branch https://review.openstack.org/54453 | 01:00 |
*** ericw has joined #openstack-infra | 01:00 | |
sdague | I'm not sure | 01:01 |
*** ^demon has joined #openstack-infra | 01:01 | |
*** ^demon has joined #openstack-infra | 01:01 | |
sdague | this would however explain the reverify bug I saw | 01:01 |
clarkb | sdague: you missed a closing ] which isn't technically needed, but nice to have | 01:01 |
*** ^demon is now known as ^d | 01:01 | |
sdague | also, seriously, havana nova works with grizzly services except for ec2? | 01:01 |
fungi | also, we do need to zuulify both branches i think, old and new, since there could be stable backport and master changes ahead of you in the queue | 01:01 |
*** sarob has quit IRC | 01:01 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Only zuulify the tree if we are on the right branch https://review.openstack.org/54453 | 01:02 |
sdague | fungi: we are | 01:02 |
*** sarob has joined #openstack-infra | 01:02 | |
fungi | [disclaimer: hadn't looked at the code yet, only the commit title] | 01:02 |
fungi | ;) | 01:02 |
sdague | 511 is the new tree, and the old tree happens later | 01:02 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/config: Remove use of undefined console-log-post publisher https://review.openstack.org/54454 | 01:03 |
clarkb | does there need to be whitespace before the ]? bash whitespace rules always trip me up | 01:03 |
sdague | clarkb: probably :) | 01:03 |
*** nosnos has joined #openstack-infra | 01:03 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Only zuulify the tree if we are on the right branch https://review.openstack.org/54453 | 01:03 |
sdague | ok, lets see what that looks like | 01:03 |
sdague | though, sadly, it's hard to tell until we can see a nova change on top of it | 01:04 |
fungi | there needs to be whitespace after the [ in case you need portability (most systems symlink .../bin/[ to .../bin/test) and so i'm in the habit of just whitespace padding square brackets | 01:04 |
*** dprince has quit IRC | 01:04 | |
clarkb | sdague: you can always reverify/approve a nova change then -2 it afterwards >_> | 01:04 |
*** ericw has quit IRC | 01:04 | |
clarkb | or remove the +A | 01:04 |
mgagne | clarkb: what fungi said ^ | 01:04 |
sdague | true, though that only helps if this is in the gate queue | 01:05 |
clarkb | sdague: right and it is probably a bad thing to get into a habit of doing | 01:05 |
sdague | yep :) | 01:05 |
clarkb | fungi: mgagne: right but bash has [/test as a built in | 01:05 |
sdague | I wonder if there is a way to test a d-g change on a whacky zuul ref scenario sanely | 01:05 |
fungi | clarkb: which is why i referred to it as a habit. years and years of patching spaghetti shell on sco unix :( | 01:06 |
sdague | fungi: sco eh... | 01:06 |
mgagne | clarkb: won't work anyway | 01:06 |
clarkb | fungi: before I started hacking on d-g and friends I always tried to write portable bourne scripts, now I think bash has won | 01:06 |
clarkb | mgagne: ah | 01:06 |
*** sarob has quit IRC | 01:06 | |
clarkb | mgagne: yup local test show it derps | 01:07 |
*** oubiwann has joined #openstack-infra | 01:07 | |
sdague | one of the guys in the office actually bought 30 shares of sco common stock before they went under, and paid to have it turned into an actual stock certificate (and hung it up in his office) | 01:07 |
mgagne | $ bash -c 'if ["1"=="1"]; then echo "works"; fi' | 01:07 |
mgagne | bash: [1==1]: command not found | 01:07 |
fungi | nice | 01:07 |
clarkb | its weird that it complains about ] missing since it actually doesn't need it | 01:07 |
clarkb | but maybe the built in does need it and I am thinking of /bin/[ | 01:07 |
*** SergeyLukjanov has joined #openstack-infra | 01:08 | |
*** ericw has joined #openstack-infra | 01:08 | |
sdague | oh, crap, it's not an == is it | 01:08 |
mgagne | sdague: they should be the same | 01:08 |
sdague | ok | 01:08 |
mgagne | sdague: http://www.tldp.org/LDP/abs/html/comparison-ops.html | 01:08 |
clarkb | sdague: = is bourne == is bash iirc | 01:09 |
fungi | yeah, i usually try to stick to = but we have both all over | 01:09 |
fungi | and of course = gets you in trouble in other languages | 01:09 |
sdague | that would actually be a good bash8 check.... :) | 01:09 |
fungi | actually gets you in trouble in shell too if you're not paying attentiom | 01:09 |
sdague | so many december tasks | 01:10 |
clarkb | sdague: its like a stack where you can push in the middle | 01:10 |
*** jhesketh__ has quit IRC | 01:10 | |
sdague | heh | 01:10 |
sdague | my november is so broken for getting anything done | 01:10 |
*** krotscheck has quit IRC | 01:10 | |
mgagne | fungi: if (current->uid = 0) ... =) | 01:11 |
fungi | yes, fond memories of staring at c for hours trying to figure out what i did wrong, only to discover i meant == | 01:12 |
sdague | clarkb: so this actually completely would explain why d-g was -2ed in the gate originally | 01:12 |
sdague | because there were nova changes in front of it | 01:12 |
sdague | and we had our crazy nova on master, everything else on grizzly | 01:12 |
sdague | which apparently passes all devstack exercises besides euca | 01:13 |
sdague | so kind of "go us" | 01:13 |
sdague | because I didn't think it would work that well | 01:13 |
sdague | then when I finally got the rebase up, there was nothing in front of it in the gate | 01:13 |
sdague | and, honestly most of the ec2 tests actually passed | 01:14 |
*** yamahata has joined #openstack-infra | 01:15 | |
mordred | sdague: awesome | 01:15 |
sdague | so there is still an issue here.... it occurs to me | 01:16 |
sdague | or, maybe not | 01:16 |
sdague | I'm trying to get my head around what happens when we come in on stable/grizzly, but with a novaclient change, which is actually a master tree | 01:16 |
*** rakhmerov has joined #openstack-infra | 01:17 | |
clarkb | sdague: I think you test the havana -> ice house upgrade and call it good | 01:17 |
clarkb | which is what it should do now right? | 01:17 |
*** jhesketh__ has joined #openstack-infra | 01:18 | |
sdague | actually not, I think I need to check ZUUL_BRANCH against $branch, not against $BRANCH | 01:18 |
sdague | because we first try to find BRANCH on the project, and if it's not found, we set it to master | 01:18 |
sdague | which is how we can roll the clients to master | 01:19 |
sdague | but then the ZUUL_BRANCH check on BRANCH is no good | 01:19 |
sdague | so then we just default to checking out master | 01:19 |
sdague | and the novaclient issue that happened before can happen again | 01:19 |
*** sdake has joined #openstack-infra | 01:19 | |
sdague | someone want to double check that train of thought before I rev the patch | 01:19 |
clarkb | my brain is pretty well cooked, but I will give it a shot | 01:20 |
sdague | mordred: how cooked is your brain? | 01:20 |
clarkb | sdague: wouldn't *client always default to checking out master? | 01:21 |
clarkb | because there are no branches | 01:21 |
sdague | clarkb: right | 01:21 |
clarkb | regardless of BRANCH or branch | 01:21 |
sdague | yes... but | 01:21 |
*** rakhmerov has quit IRC | 01:22 | |
sdague | the trick is how do we get to L141 - https://review.openstack.org/#/c/54453/3/devstack-vm-gate-wrap.sh | 01:22 |
sdague | L141 is "check out to the magic zuul ref" | 01:22 |
sdague | so if we ever bump out of there we get BRANCH instead | 01:22 |
clarkb | right so don't we want to compare BRANCH and ZUUL_BRANCH? | 01:23 |
clarkb | oh wait | 01:23 |
clarkb | I think I grok now. BRANCH can carry the fallback which may be something different | 01:23 |
*** ^d has quit IRC | 01:23 | |
sdague | I think we want ZUUL_BRANCH==$branch | 01:23 |
sdague | which is the branch setup_workspace was called with | 01:24 |
clarkb | yeah | 01:24 |
sdague | right, because of the fallback | 01:24 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Only zuulify the tree if we are on the right branch https://review.openstack.org/54453 | 01:25 |
sdague | ok, so I think that's right | 01:25 |
*** ericw has quit IRC | 01:26 | |
* fungi is now where he can check it out | 01:27 | |
mordred | sdague: my brain is a jelly donut | 01:28 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Don't attempt to zuulify the old tree https://review.openstack.org/54458 | 01:28 |
sdague | so that last change is a fail safe | 01:28 |
mordred | we really need jeblair to not be on vacation don't we | 01:28 |
sdague | yeh, though I've almost grokked this now | 01:29 |
sdague | but it would have been a ton easier with him around | 01:29 |
sdague | so, I think 54453 is the right fix, and should be put through the gate, and rainbows and unicorns | 01:29 |
sdague | but... if it's not, 54458 should take us back to basically the old behavior | 01:30 |
*** ericw has joined #openstack-infra | 01:30 | |
*** vipul is now known as vipul-away | 01:31 | |
sdague | my brain is also starting to go into mush land, as I started my day driving 1.5hrs to white plains for federal district jury duty | 01:31 |
sdague | but on the upside, going to hong kong on friday got me dismissed off the jury, and exempt for the next 4 years :) | 01:32 |
clarkb | jury duty! I haven't done that in a while. Watch I am going to get a summons now | 01:32 |
sdague | the federal district one is a pain | 01:32 |
sdague | as it's a month | 01:32 |
sdague | and white plains, which is 1:15 => 1:30 depending on traffic from my place | 01:32 |
clarkb | I have done two different counties | 01:33 |
sdague | but at least I only had to go down physically twice | 01:33 |
sdague | clarkb: when you getting to HK? | 01:35 |
*** bingbu has joined #openstack-infra | 01:35 | |
*** vipul-away is now known as vipul | 01:36 | |
clarkb | sdague: Sunday at 1720 | 01:37 |
sdague | ok, cool. mtreinish and I are getting in Sat at 1900. Hoping to get a couple of days to time adjust. Wonder if there will be a Sunday night gathering of folks | 01:38 |
mordred | sdague I arrive Sat too | 01:38 |
* mgagne hopes there will be gallons of coffee as he only arrives Monday night | 01:40 | |
sdague | heh | 01:40 |
sdague | mordred: what part of town are you in pre-summit? | 01:40 |
mordred | sdague: kowloon | 01:40 |
mgagne | you all make me worry about jet lag | 01:41 |
sdague | ok we're over near victoria peak | 01:41 |
sdague | dansmith is over those parts as well, different hotel | 01:41 |
sdague | also getting in Sat | 01:41 |
*** SergeyLukjanov has quit IRC | 01:42 | |
*** ericw has quit IRC | 01:43 | |
clarkb | sdague: there is talk of doing the monday registration check in thing at 3pm and meeting folks there and scheming for monday afternoon/evening/night | 01:43 |
sdague | yep, jog0 pinged me about that | 01:43 |
clarkb | so if you show up for that you will probably find a gaggle of us | 01:43 |
sdague | mkoderer was kicking off a qa core gathering, I'm going to try to get it to merge with the infra gathering :) | 01:44 |
sdague | we should all be one happy family anyway | 01:44 |
*** markwash has joined #openstack-infra | 01:44 | |
clarkb | ++ | 01:44 |
*** ericw has joined #openstack-infra | 01:45 | |
*** nati_ueno has joined #openstack-infra | 01:46 | |
fungi | yeah, i get there sunday around 1930 local clocks | 01:46 |
fungi | er, saturday | 01:46 |
fungi | just noticed we don't gate devstack-gate on grenade... grr | 01:47 |
clarkb | whoops :/ | 01:47 |
clarkb | fungi: we should lookat using layout templates so that we can avoid those problems | 01:47 |
*** esker has joined #openstack-infra | 01:50 | |
sdague | fungi: are you sure? | 01:51 |
sdague | I'm actually pretty sure we do | 01:51 |
fungi | sdague: oh, heh. it's clearly getting late here ;) | 01:51 |
sdague | heh | 01:51 |
fungi | in fact, that's what the patch failed on originally, so obviously we were | 01:52 |
sdague | I can't wait to see jeblair's reaction to all my failed d-g work this week, and the reverts when he looks at the git logs | 01:52 |
sdague | heh | 01:52 |
sdague | yeh, I was going to say | 01:52 |
fungi | i think he'll just be glad it's still running in his absence | 01:53 |
sdague | so realistically I don't think the test results on - https://review.openstack.org/#/c/54453/ are actually going to tell us anything, so my suggestion is that if anyone is feeling adventurous to push it through, then recheck a nova change that failed in the last hour and see if it passes | 01:53 |
sdague | and if everything is still screwed, 54458 will make it better, by giving up on trying to update "old" | 01:54 |
*** adalbas has quit IRC | 01:54 | |
sdague | but I need to wonder away from the computer and sleep | 01:54 |
*** nati_ueno has quit IRC | 01:55 | |
fungi | i'll forego sleep a while longer | 01:55 |
sdague | see folks in the mornign | 01:55 |
*** ericw has quit IRC | 01:56 | |
*** nati_ueno has joined #openstack-infra | 01:57 | |
anteaya | night sdague | 01:57 |
*** dims has quit IRC | 02:01 | |
*** dcramer_ has joined #openstack-infra | 02:02 | |
*** sarob has joined #openstack-infra | 02:13 | |
*** sarob has quit IRC | 02:13 | |
*** sarob_ has joined #openstack-infra | 02:13 | |
*** sarob has joined #openstack-infra | 02:14 | |
*** ericw has joined #openstack-infra | 02:17 | |
*** rakhmerov has joined #openstack-infra | 02:18 | |
*** sarob_ has quit IRC | 02:18 | |
*** guohliu has joined #openstack-infra | 02:20 | |
*** ericw has quit IRC | 02:21 | |
*** nati_ueno has quit IRC | 02:23 | |
*** nati_ueno has joined #openstack-infra | 02:24 | |
*** zul has quit IRC | 02:24 | |
*** UtahDave has quit IRC | 02:27 | |
*** nati_ueno has quit IRC | 02:28 | |
*** ljjjustin has joined #openstack-infra | 02:31 | |
*** ericw has joined #openstack-infra | 02:36 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Only zuulify the tree if we are on the right branch https://review.openstack.org/54453 | 02:45 |
*** senk has joined #openstack-infra | 02:45 | |
*** yamahata has quit IRC | 02:46 | |
*** michchap has quit IRC | 02:48 | |
*** esheffield_ is now known as esheffield | 02:48 | |
*** rakhmerov has quit IRC | 02:48 | |
*** rakhmerov has joined #openstack-infra | 02:49 | |
*** gyee has quit IRC | 02:49 | |
*** rakhmerov has quit IRC | 02:53 | |
*** xchu has joined #openstack-infra | 02:55 | |
fungi | rechecked https://jenkins02.openstack.org/job/check-grenade-devstack-vm/3741/console a few minutes ago, watching to see if it passes now | 02:56 |
*** dcramer_ has quit IRC | 02:57 | |
*** lnxnut has joined #openstack-infra | 02:58 | |
*** yamahata has joined #openstack-infra | 02:59 | |
*** mriedem has quit IRC | 02:59 | |
*** anteaya has quit IRC | 03:00 | |
*** guohliu has quit IRC | 03:01 | |
fungi | no such luck | 03:02 |
*** sarob has quit IRC | 03:05 | |
*** dcramer_ has joined #openstack-infra | 03:10 | |
*** wenlock has joined #openstack-infra | 03:17 | |
fungi | rechecked https://jenkins01.openstack.org/job/check-grenade-devstack-vm/3429/console to the same end | 03:19 |
clarkb | fungi: so it isn't happy? | 03:20 |
clarkb | should we revert? | 03:20 |
*** vipul has quit IRC | 03:20 | |
*** melwitt has quit IRC | 03:20 | |
fungi | sdague left us https://review.openstack.org/54458 for this occasion | 03:21 |
*** vipul has joined #openstack-infra | 03:21 | |
clarkb | that has my +2 | 03:21 |
fungi | about to approve, just collecting some links to include in the comment so he can have a jumpstart on investigating it tomorrow | 03:22 |
clarkb | k | 03:23 |
SlickNik | lifeless: Sorry, didn't get your messages until now. | 03:33 |
SlickNik | lifeless: Ideally we'd want the trove elements to live somewhere else too; we were thinking a different repo so that we can build the image every so often and use a "golden" image for integration test runs. | 03:34 |
lifeless | SlickNik: sure, heat are going to do that too | 03:35 |
lifeless | SlickNik: they'll be building it out of tripleo-image-elements though | 03:35 |
lifeless | SlickNik: I don't particularly care either way FWIW :) | 03:35 |
SlickNik | lifeless: ah, we were thinking of putting it into a different repo, but I think tripleo-image-elements makes sense. | 03:36 |
SlickNik | lifeless: also I hear you about moving away from first-boot.d. Hope to get some time to look into it soon… :) | 03:37 |
fungi | adrian_otto: if you wind up reading the logs, i was finally able to get manage-projects to run clean with no github api errors and stackforge/solum seems to clone successfully from the expected places now | 03:38 |
*** lnxnut has quit IRC | 03:42 | |
*** lnxnut has joined #openstack-infra | 03:43 | |
*** lnxnut has quit IRC | 03:47 | |
*** rakhmerov has joined #openstack-infra | 03:47 | |
fungi | 54458 looks like it's going to pass. at least succeeded grenade so i think we're in the clear with it | 03:51 |
clarkb | woot | 03:51 |
fungi | i'll recheck the same nova changes once it merges | 03:52 |
*** beagles has quit IRC | 03:56 | |
*** adrian_otto has joined #openstack-infra | 04:02 | |
*** vipul is now known as vipul-away | 04:05 | |
*** harlowja has quit IRC | 04:07 | |
fungi | crap, random failure on the last running job for it | 04:08 |
fungi | needs to be reverified | 04:08 |
clarkb | :( | 04:09 |
*** vipul-away is now known as vipul | 04:10 | |
*** senk has quit IRC | 04:17 | |
fungi | seems it hit bug 1218582 | 04:17 |
fungi | lucky us | 04:18 |
*** rakhmerov has quit IRC | 04:18 | |
*** yaguang has quit IRC | 04:24 | |
*** nati_ueno has joined #openstack-infra | 04:29 | |
*** Adri2000 has quit IRC | 04:32 | |
*** oubiwann has quit IRC | 04:32 | |
*** Adri2000 has joined #openstack-infra | 04:33 | |
*** yaguang has joined #openstack-infra | 04:44 | |
*** DinaBelova has joined #openstack-infra | 04:45 | |
*** changbl has joined #openstack-infra | 04:50 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Don't attempt to zuulify the old tree https://review.openstack.org/54458 | 04:55 |
fungi | yay. second try's a charm | 04:55 |
*** vipul is now known as vipul-away | 05:02 | |
*** SergeyLukjanov has joined #openstack-infra | 05:05 | |
*** oubiwann has joined #openstack-infra | 05:09 | |
*** jpeeler has quit IRC | 05:16 | |
*** jpeeler has joined #openstack-infra | 05:18 | |
*** vipul-away is now known as vipul | 05:19 | |
fungi | success on the rechecked nova changes, so we're in the clear | 05:21 |
clarkb | woot | 05:22 |
*** zul has joined #openstack-infra | 05:27 | |
*** nicedice__ has quit IRC | 05:29 | |
*** zul has quit IRC | 05:32 | |
mrda | hey all, I'm seeing some failures from check-grenade-devstack-vm (ref https://review.openstack.org/#/c/54472). I don't think it's me. Is there a known issue out there? | 05:35 |
mordred | SlickNik, lifeless: did we ever sort out the heat-cfntools pbr issue? | 05:35 |
lifeless | mordred: no | 05:35 |
lifeless | mordred: also ?good morning? | 05:35 |
mordred | lifeless: morning! | 05:36 |
*** oubiwann has quit IRC | 05:36 | |
mordred | lifeless: any better luck finding a reproducible test case for mordred? | 05:36 |
clarkb | mrda: I think fungi just finished it with his comments at 0523 UTC | 05:36 |
mrda | ok, thanks | 05:37 |
lifeless | mordred: I have not had a time slice on it sorry | 05:39 |
lifeless | mordred: one of those days | 05:39 |
mordred | lifeless: it's ok. I haven't either | 05:39 |
* mordred just wanting to make sure people are blocking on him | 05:39 | |
*** adrian_otto has quit IRC | 05:44 | |
*** sdake has quit IRC | 05:46 | |
fungi | mrda: yeah, should be resolved for now | 05:47 |
mrda | thanks fungi, I've just "recheck no bug"d and we'll see how it goes | 05:47 |
lifeless | https://pypi.python.org/pypi/radon | 05:49 |
clarkb | neat | 05:50 |
clarkb | I was just reading https://www.dropbox.com/s/wnzqidngrtj8y2l/Bookout_v_Toyota_Barr_REDACTED.pdf so radon seems relevant | 05:51 |
mordred | neat | 05:51 |
mordred | also- flake8 has a mccabe plugin | 05:51 |
mrda | lifeless: that's pretty cool | 05:53 |
mordred | radon's mi command mostly likes nova | 05:53 |
mrda | I like it that "mccabe" is close to "macabre" | 05:55 |
clarkb | the tex (I think it is tex) in the docs didn't format properly and makes the interesting bits hard to read | 05:56 |
*** zul has joined #openstack-infra | 06:01 | |
clarkb | ttx: is it still to early to stop running the folsom jobs? | 06:01 |
clarkb | ttx: I semi hope that we might be able to pull them out before the summit (but not a huge deal if it happens after) | 06:01 |
*** denis_makogon has joined #openstack-infra | 06:06 | |
*** wenlock has quit IRC | 06:08 | |
*** blamar has quit IRC | 06:14 | |
*** mrda has quit IRC | 06:43 | |
lifeless | mordred: since you are around - https://review.openstack.org/#/c/53646/ | 06:49 |
*** mrda has joined #openstack-infra | 06:55 | |
lifeless | hey | 07:05 |
lifeless | why is https://review.openstack.org/#/c/54426/1 failing ? | 07:05 |
*** yaguang has quit IRC | 07:05 | |
*** yaguang has joined #openstack-infra | 07:05 | |
*** vipul is now known as vipul-away | 07:06 | |
clarkb | requirements are changed and don't match contents of the global list | 07:06 |
lifeless | clarkb: it's being changed by the global list sync job | 07:07 |
lifeless | clarkb: so I don't understand how that can happen | 07:07 |
lifeless | clarkb: also, the iso8601 change is broken because 0.5/0.6/0.7 are broken | 07:07 |
clarkb | oh I wonder if a change to a stable branch fired that off | 07:09 |
clarkb | mordred ^ because it seems to be reverting versions | 07:09 |
*** mattymo has joined #openstack-infra | 07:12 | |
*** SergeyLukjanov is now known as _SergeyLukjanov | 07:12 | |
*** _SergeyLukjanov has quit IRC | 07:13 | |
openstackgerrit | Peter Liljenberg proposed a change to openstack-infra/jenkins-job-builder: Added support for Jenkins plugin Blame upstream committers https://review.openstack.org/54085 | 07:28 |
*** AJaeger has left #openstack-infra | 07:32 | |
*** jcoufal has joined #openstack-infra | 07:38 | |
*** dkliban has joined #openstack-infra | 07:42 | |
*** flaper87|afk is now known as flaper87 | 07:44 | |
*** hashar has joined #openstack-infra | 07:50 | |
*** osanchez has joined #openstack-infra | 07:53 | |
*** hashar has quit IRC | 07:56 | |
*** pblaho has joined #openstack-infra | 08:18 | |
*** guohliu has joined #openstack-infra | 08:19 | |
*** flaper87 is now known as flaper87|afk | 08:22 | |
*** mrda has quit IRC | 08:25 | |
openstackgerrit | Oleg Gelbukh proposed a change to openstack-infra/config: Add rubick project to stackforge https://review.openstack.org/53858 | 08:26 |
*** denis_makogon has quit IRC | 08:26 | |
*** dkliban has quit IRC | 08:28 | |
*** tkammer has joined #openstack-infra | 08:29 | |
*** ljjjustin has quit IRC | 08:30 | |
*** flaper87|afk is now known as flaper87 | 08:32 | |
openstackgerrit | Oleg Gelbukh proposed a change to openstack-infra/config: Add rubick project to stackforge https://review.openstack.org/53858 | 08:33 |
*** osanchez has quit IRC | 08:35 | |
ttx | clarkb: I wanted to formally kill them during the stable branch session, so that we can address complaints (if any) directly and in-person | 08:37 |
ttx | clarkb: but if it blocks useful work, I guess a thread on stable-maint would do | 08:38 |
*** dcramer_ has quit IRC | 08:39 | |
*** mrda has joined #openstack-infra | 08:45 | |
tsufiev | mordred, clarkb: could you please take a look at https://review.openstack.org/#/c/54370/ ? | 08:46 |
*** ljjjustin has joined #openstack-infra | 08:48 | |
*** fbo_away is now known as fbo | 08:51 | |
*** dcramer_ has joined #openstack-infra | 08:51 | |
*** matsuhashi has quit IRC | 09:03 | |
*** matsuhashi has joined #openstack-infra | 09:03 | |
*** hashar has joined #openstack-infra | 09:04 | |
*** derekh has joined #openstack-infra | 09:05 | |
*** matsuhas_ has joined #openstack-infra | 09:08 | |
*** matsuhashi has quit IRC | 09:08 | |
*** jpich has joined #openstack-infra | 09:09 | |
*** hashar_ has joined #openstack-infra | 09:11 | |
*** hashar has quit IRC | 09:12 | |
*** hashar_ is now known as hashar | 09:12 | |
*** yassine has joined #openstack-infra | 09:13 | |
*** ljjjustin has quit IRC | 09:14 | |
*** mrda has quit IRC | 09:21 | |
openstackgerrit | Dzmitry Horbach proposed a change to openstack-infra/jenkins-job-builder: Jenkins Dynamic Parameter Plug-in support https://review.openstack.org/53884 | 09:22 |
*** plomakin has quit IRC | 09:23 | |
*** plomakin has joined #openstack-infra | 09:23 | |
*** dkliban has joined #openstack-infra | 09:23 | |
*** CaptTofu has quit IRC | 09:24 | |
*** rakhmerov has joined #openstack-infra | 09:25 | |
rakhmerov | hi, could you please give some tips on why the newly created repo at stackforge may be empty? The change request: https://review.openstack.org/#/c/51205/. Repo: https://github.com/stackforge/mistral, seed repo: https://github.com/rakhmerov/mistral | 09:25 |
*** che-arne has joined #openstack-infra | 09:33 | |
*** tkammer has quit IRC | 09:36 | |
*** mugsie has quit IRC | 09:37 | |
*** johnthetubaguy has joined #openstack-infra | 09:40 | |
*** johnthetubaguy1 has joined #openstack-infra | 09:41 | |
*** johnthetubaguy has quit IRC | 09:42 | |
*** bingbu has quit IRC | 09:44 | |
*** romcheg has joined #openstack-infra | 09:47 | |
*** boris-42_ is now known as boris-42 | 10:00 | |
*** tkammer has joined #openstack-infra | 10:00 | |
*** xchu has quit IRC | 10:01 | |
*** osanchez has joined #openstack-infra | 10:02 | |
*** alexpilotti has quit IRC | 10:05 | |
*** fifieldt has quit IRC | 10:10 | |
*** pcm_ has joined #openstack-infra | 10:12 | |
*** pcm_ has quit IRC | 10:13 | |
*** pcm_ has joined #openstack-infra | 10:13 | |
openstackgerrit | SlickNik proposed a change to openstack-dev/pbr: Fixed pbr install to not error out and fail if git is not installed https://review.openstack.org/54510 | 10:14 |
*** tkammer has quit IRC | 10:15 | |
SlickNik | ^^mordred / lifeless: This was the issue we were running into while building the trove guestagent image | 10:16 |
*** afazekas has joined #openstack-infra | 10:33 | |
*** plomakin has quit IRC | 10:35 | |
*** plomakin has joined #openstack-infra | 10:35 | |
sdague | clarkb / fungi: did we fail things besides the stable / havana change ? | 10:41 |
openstackgerrit | A change was merged to openstack-dev/hacking: Updated from global requirements https://review.openstack.org/54434 | 10:42 |
sdague | gah, I put the conditional in the wrong spot.... | 10:43 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Correct test for zuulifying the tree https://review.openstack.org/54518 | 10:48 |
*** jhesketh has quit IRC | 10:48 | |
*** markmc has joined #openstack-infra | 10:54 | |
jog0 | sdague: btw I am with you about RPC testing has to be done in grenade | 10:59 |
jog0 | you convinced me pretty quickly | 11:00 |
*** matsuhas_ has quit IRC | 11:01 | |
*** matsuhashi has joined #openstack-infra | 11:02 | |
*** dims has joined #openstack-infra | 11:03 | |
sdague | heh, nice | 11:05 |
sdague | and not only because I keep breaking the freaking gate? | 11:05 |
jog0 | sdague: I always thought long term it belonged there but short term it does too because of the zuulifcation of git stuff | 11:07 |
jog0 | sdague: http://paste.openstack.org/show/50204/ | 11:07 |
jog0 | the email draft for hacking 0.8.0 (which I am getting ready to cut momentarily) | 11:07 |
sdague | cool | 11:08 |
*** DinaBelova has quit IRC | 11:09 | |
*** matsuhashi has quit IRC | 11:11 | |
jog0 | cool, cutting release now | 11:12 |
*** DinaBelova has joined #openstack-infra | 11:14 | |
*** yamahata has quit IRC | 11:20 | |
*** flaper87 is now known as flaper87|afk | 11:24 | |
*** yaguang has quit IRC | 11:25 | |
openstackgerrit | Joe Gordon proposed a change to openstack/requirements: Switch to Hacking 0.8.0 https://review.openstack.org/54523 | 11:34 |
*** CaptTofu has joined #openstack-infra | 11:37 | |
*** alexpilotti has joined #openstack-infra | 11:44 | |
*** hashar has quit IRC | 11:54 | |
*** dcramer_ has quit IRC | 11:58 | |
*** matsuhashi has joined #openstack-infra | 12:02 | |
*** afazekas has quit IRC | 12:03 | |
*** adalbas has joined #openstack-infra | 12:14 | |
*** dprince has joined #openstack-infra | 12:14 | |
fungi | rakhmerov: github was having serious issues when that change merged, and it appears it didn't get imported successfully. i've cleared that repository at the gerrit end and am re-running the import script now manually | 12:16 |
*** dhellmann is now known as dhellmann-afk | 12:16 | |
fungi | sdague: i linked a couple failing changes on that approval | 12:16 |
fungi | ahh, new change i see | 12:17 |
*** jpeeler has quit IRC | 12:19 | |
*** jpeeler has joined #openstack-infra | 12:19 | |
*** esker has quit IRC | 12:23 | |
*** dims has quit IRC | 12:25 | |
*** dims has joined #openstack-infra | 12:25 | |
*** tkammer has joined #openstack-infra | 12:26 | |
*** nosnos has quit IRC | 12:27 | |
rakhmerov | fungi: thanks a lot, I hope it'll work this time | 12:29 |
*** anteaya- is now known as anteaya | 12:30 | |
fungi | rakhmerov: i'll test when it finishes, troubleshoot if not, and let you know when it's usable | 12:30 |
rakhmerov | sure | 12:31 |
*** hashar has joined #openstack-infra | 12:33 | |
jog0 | just got devstack diablo running wow things looked different back then | 12:33 |
*** yamahata has joined #openstack-infra | 12:33 | |
*** anteaya has quit IRC | 12:34 | |
*** flaper87|afk is now known as flaper87 | 12:35 | |
*** anteaya has joined #openstack-infra | 12:37 | |
*** yamahata has quit IRC | 12:39 | |
*** afazekas has joined #openstack-infra | 12:40 | |
*** CaptTofu has quit IRC | 12:44 | |
*** CaptTofu has joined #openstack-infra | 12:45 | |
*** yamahata has joined #openstack-infra | 12:46 | |
*** thomasem has joined #openstack-infra | 12:46 | |
*** beagles has joined #openstack-infra | 12:46 | |
*** julim has joined #openstack-infra | 12:46 | |
zul | *cough* sorry https://review.openstack.org/#/c/54213/ | 12:48 |
*** oubiwann has joined #openstack-infra | 13:02 | |
fungi | i think i remember jgriffith saying he was cool with that | 13:04 |
*** SergeyLukjanov has joined #openstack-infra | 13:04 | |
fungi | if not we can always revert i guess | 13:05 |
zul | yep | 13:05 |
*** hashar has quit IRC | 13:05 | |
zul | ill bug jgriffith today about it just to make sure | 13:05 |
*** hashar has joined #openstack-infra | 13:05 | |
fungi | thanks! | 13:05 |
openstackgerrit | A change was merged to openstack-infra/config: Make cinderclient-py33 gating https://review.openstack.org/54213 | 13:05 |
*** matsuhashi has quit IRC | 13:07 | |
*** matsuhashi has joined #openstack-infra | 13:08 | |
*** matsuhas_ has joined #openstack-infra | 13:09 | |
*** matsuhashi has quit IRC | 13:10 | |
*** guohliu has quit IRC | 13:10 | |
*** weshay has joined #openstack-infra | 13:12 | |
*** Bada has joined #openstack-infra | 13:13 | |
*** mriedem has joined #openstack-infra | 13:13 | |
*** tkammer_ has joined #openstack-infra | 13:15 | |
*** tkammer has quit IRC | 13:18 | |
*** guohliu has joined #openstack-infra | 13:23 | |
*** tkammer has joined #openstack-infra | 13:23 | |
*** odyssey4me has joined #openstack-infra | 13:27 | |
*** ryanpetrello has joined #openstack-infra | 13:36 | |
*** esker has joined #openstack-infra | 13:38 | |
*** esker has quit IRC | 13:38 | |
*** esker has joined #openstack-infra | 13:39 | |
*** sandywalsh_ has joined #openstack-infra | 13:39 | |
*** dstanek has joined #openstack-infra | 13:41 | |
*** rcleere has joined #openstack-infra | 13:41 | |
*** sandywalsh has quit IRC | 13:42 | |
*** pblaho has quit IRC | 13:43 | |
openstackgerrit | Arx Cruz proposed a change to openstack-infra/zuul: Add the possibility to pass the zuul url https://review.openstack.org/54196 | 13:44 |
ArxCruz | lifeless: ^ | 13:46 |
ArxCruz | hope you're happy now with all the changes. You make me learn testtools tonight ¬¬ | 13:47 |
ArxCruz | i mean, last night | 13:47 |
ArxCruz | :) | 13:50 |
*** tkammer has quit IRC | 13:52 | |
*** CaptTofu has quit IRC | 13:55 | |
*** CaptTofu has joined #openstack-infra | 13:55 | |
openstackgerrit | Arx Cruz proposed a change to openstack-infra/devstack-gate: Add Fedora, Red Hat and CentOS compatibility https://review.openstack.org/54386 | 13:55 |
*** guohliu has quit IRC | 13:58 | |
*** CaptTofu has quit IRC | 13:58 | |
*** CaptTofu has joined #openstack-infra | 13:59 | |
openstackgerrit | Arx Cruz proposed a change to openstack-infra/zuul: Add the possibility to pass the zuul url https://review.openstack.org/54196 | 13:59 |
*** Bada has quit IRC | 14:00 | |
*** mrodden has joined #openstack-infra | 14:02 | |
*** dims has quit IRC | 14:02 | |
*** mrodden has quit IRC | 14:03 | |
sdague | comments on this as a diagram to explain grenade - http://ubuntuone.com/4LlFUvmVx89pI62GLF7WtO ? | 14:03 |
*** dims has joined #openstack-infra | 14:03 | |
sdague | fungi: so I'm sure you don't trust me any more... however, I think, for real, I know the fix | 14:04 |
*** CaptTofu has quit IRC | 14:04 | |
*** CaptTofu has joined #openstack-infra | 14:04 | |
sdague | https://review.openstack.org/#/c/54518/ | 14:05 |
sdague | oh, you already +2ed it :) | 14:05 |
openstackgerrit | sebastian marcet proposed a change to openstack-infra/config: Review and create Puppet configuration for dev server https://review.openstack.org/53644 | 14:05 |
fungi | sdague: the diagram looks good for a high-level view | 14:06 |
*** SergeyLukjanov has quit IRC | 14:06 | |
sdague | yeh, this is in my "Jenkins Failed my Patch, what do I do?" talk | 14:06 |
fungi | (answer: panic and take up drinking) | 14:06 |
*** tkammer has joined #openstack-infra | 14:07 | |
sdague | to explain the various systems that we test, so it's easier to explain where things might not break | 14:07 |
fungi | yeah, that's a good idea | 14:08 |
*** CaptTofu has quit IRC | 14:08 | |
fungi | so many of these tests are just black boxes to some devs, no matter how thoroughly we document and prominently publish the source for them | 14:08 |
sdague | yep | 14:08 |
sdague | I'm assuming these tracks will be recorded again, so we'll have a video of this | 14:09 |
*** CaptTofu has joined #openstack-infra | 14:09 | |
*** jergerber has joined #openstack-infra | 14:09 | |
fungi | what is it javelin does? creates a faux production environment within devstack? | 14:09 |
openstackgerrit | sebastian marcet proposed a change to openstack-infra/config: Review and create Puppet configuration for dev server https://review.openstack.org/53644 | 14:10 |
*** pblaho has joined #openstack-infra | 14:10 | |
*** devanand1 is now known as devananda | 14:11 | |
*** lcestari has quit IRC | 14:11 | |
*** guohliu has joined #openstack-infra | 14:13 | |
*** ryanpetrello has quit IRC | 14:18 | |
*** guohliu has quit IRC | 14:20 | |
*** lcestari has joined #openstack-infra | 14:21 | |
*** mrodden has joined #openstack-infra | 14:21 | |
*** matsuhas_ has quit IRC | 14:25 | |
*** matsuhashi has joined #openstack-infra | 14:25 | |
*** markwash has quit IRC | 14:27 | |
sdague | fungi: it creates a guest, with volumes, and a few objects | 14:27 |
sdague | so there is real state that has to survive the upgrade | 14:28 |
sdague | I was just going to talk to that | 14:28 |
sdague | our first test is that that javelin guest is still running when we bring up the n-cpu daemon | 14:28 |
sdague | so show it doesn't kill the guest on startup | 14:28 |
*** datsun180b has joined #openstack-infra | 14:29 | |
*** matsuhashi has quit IRC | 14:30 | |
*** wenlock has joined #openstack-infra | 14:35 | |
*** ryanpetrello has joined #openstack-infra | 14:41 | |
fungi | very cool | 14:41 |
*** sileht has quit IRC | 14:43 | |
*** sileht has joined #openstack-infra | 14:43 | |
*** johnthetubaguy1 is now known as johnthetubaguy | 14:45 | |
portante | Where can I find the coverage and unit test history for accepted commits for a given project? | 14:48 |
*** alcabrera has joined #openstack-infra | 14:49 | |
*** adrian_otto has joined #openstack-infra | 14:51 | |
portante | also, not sure why, but this commit did not start its gate jobs right away, had to "reverify no bug" to get them going: https://review.openstack.org/54153 | 14:51 |
*** sarob has joined #openstack-infra | 14:52 | |
*** Bada has joined #openstack-infra | 14:52 | |
*** pcm_ has quit IRC | 14:53 | |
fungi | portante: i don't think coverage reports are being published and trended anywhere. that would be something for people interested in coverage reports to discuss with us and maybe work out an implementation as to what would be most effective | 14:55 |
fungi | for the moment you have to dig individual reports out of the log server | 14:55 |
portante | fungi: okay, thanks | 14:56 |
fungi | as for 54153, i have seen a few cases in recent months of zuul missing an event on the gerrit event stream, but no indication in either gerrit's or zuul's logs as to why... can't tell if gerrit failed to emit it or zuul failed to notice it | 14:57 |
fungi | i'm leaning toward the former, but i have no actual evidence to back it up | 14:57 |
*** sarob has quit IRC | 15:01 | |
*** sarob has joined #openstack-infra | 15:02 | |
*** tkammer has quit IRC | 15:03 | |
portante | fungi: okay, thanks | 15:03 |
*** sarob has quit IRC | 15:06 | |
*** ljjjustin has joined #openstack-infra | 15:08 | |
*** pcm_ has joined #openstack-infra | 15:10 | |
*** senk has joined #openstack-infra | 15:12 | |
*** senk1 has joined #openstack-infra | 15:13 | |
*** UtahDave has joined #openstack-infra | 15:15 | |
*** senk has quit IRC | 15:16 | |
*** tkammer has joined #openstack-infra | 15:17 | |
ryanpetrello | can I convince somebody to have a look at https://review.openstack.org/#/c/54333/ :)? | 15:17 |
ryanpetrello | I'd love to release a new version for the summit | 15:17 |
*** CaptTofu has quit IRC | 15:17 | |
ryanpetrello | but want to be sure we have gating for important projects like this | 15:17 |
*** CaptTofu has joined #openstack-infra | 15:18 | |
romcheg | Morning folks | 15:20 |
romcheg | I need some help with devstack-gate | 15:20 |
fungi | romcheg: then you're in good company. i think we all need help with devstack-gate | 15:21 |
romcheg | fungi: indeed :) | 15:22 |
romcheg | I recently implemented a patch to devstack that can install Ironic | 15:22 |
wenlock | hey morning, has anyone documented a way or is there a way to automate with cli change to add administrators push permission to refs/meta/config branch? This is the manual step you currently have to do in gerrit setup from the UI. | 15:22 |
fungi | ryanpetrello: looks fine to me on first pass. with all the parameterization though my brain's a little twisted around so i'm not 100% sure i'm thinking it through | 15:22 |
romcheg | However, after I added ironic services to devstack-gate I get "Cloning not allowed in this configuration" | 15:23 |
fungi | romcheg: i think for starters we need to make sure that nodepool is configured to pre-cache the ironic git repo on the slaves | 15:23 |
fungi | wenlock: i think yolanda (robla) was working on that | 15:24 |
wenlock | fungi cool. when is he usually on? | 15:24 |
fungi | wenlock: sounded pretty close to having it working | 15:24 |
wenlock | did he do the whole bit... or just that part? | 15:25 |
*** jcoufal is now known as jcoufal|afk | 15:25 | |
rakhmerov | fungi: FYI, mistral is still empty | 15:25 |
*** chandankumar has joined #openstack-infra | 15:25 | |
fungi | wenlock: i honestly can't remember what times yolanda comes around. was working on 100% automation of gerrit setup from scratch with no manual intervention, including initial administrative accounts | 15:26 |
romcheg | fungi: Ah, thanks. I will try to figure out the problem now | 15:26 |
wenlock | intreasting, i should catch up to him and maybe we can exchange some stuff | 15:27 |
fungi | rakhmerov: i know, i'm continuing to hammer at the script but we're running into near constant github api errors and timeouts from it. i may just resort to manually cloning the repo and doing that bit of the setup | 15:27 |
wenlock | fungi: ty for the lead | 15:27 |
*** wenlock has quit IRC | 15:27 | |
rakhmerov | fungi: ok, got it | 15:27 |
fungi | romcheg: the nodepool prepare scripts are in openstack-infra/config (i believe in modules/openstack_project/files/nodepool directory) | 15:28 |
fungi | one of those, i forget which, embeds a list of projects to clone on the base image | 15:28 |
romcheg | fungi: Found it, thanks | 15:30 |
openstackgerrit | Roman Podoliaka proposed a change to openstack-infra/config: Add release config for tuskar components. https://review.openstack.org/53190 | 15:30 |
openstackgerrit | Roman Podoliaka proposed a change to openstack-infra/config: Add release config for tripleo-heat-templates https://review.openstack.org/52206 | 15:30 |
sdague | mordred or clarkb awake to look at the d-g script? | 15:30 |
*** dcramer_ has joined #openstack-infra | 15:31 | |
*** rpodolyaka has joined #openstack-infra | 15:33 | |
fungi | finally got around to looking into the glut of deleting nodes in the nodepool graphs. most were in rax as suspected. did a nodepool delete on all of them and the graph is back to sane levels of deleting now | 15:33 |
*** sarob has joined #openstack-infra | 15:34 | |
fungi | stepping out to grab an early lunch, but will bbiab | 15:36 |
*** senk has joined #openstack-infra | 15:37 | |
*** adrian_otto has quit IRC | 15:38 | |
openstackgerrit | Arx Cruz proposed a change to openstack-infra/zuul: Add the possibility to pass the zuul url https://review.openstack.org/54196 | 15:38 |
*** adrian_otto has joined #openstack-infra | 15:38 | |
*** senk1 has quit IRC | 15:40 | |
*** Ryan_Lane has joined #openstack-infra | 15:40 | |
*** ^d has joined #openstack-infra | 15:41 | |
*** ^d has joined #openstack-infra | 15:41 | |
*** blamar has joined #openstack-infra | 15:48 | |
openstackgerrit | Roman Prykhodchenko proposed a change to openstack-infra/config: Precache Ironic and Ironic client https://review.openstack.org/54569 | 15:51 |
*** pblaho has quit IRC | 15:55 | |
*** nati_ueno has quit IRC | 15:57 | |
*** wenlock has joined #openstack-infra | 15:59 | |
*** dkliban has quit IRC | 16:00 | |
*** tkammer has quit IRC | 16:00 | |
*** lnxnut has joined #openstack-infra | 16:00 | |
*** romcheg has quit IRC | 16:03 | |
*** honour has joined #openstack-infra | 16:03 | |
*** hdd has joined #openstack-infra | 16:07 | |
*** sdake has joined #openstack-infra | 16:08 | |
*** Bada has quit IRC | 16:10 | |
*** honour has quit IRC | 16:10 | |
*** prodazhi has joined #openstack-infra | 16:10 | |
*** ljjjustin has quit IRC | 16:10 | |
*** jcoufal|afk is now known as jcoufal | 16:13 | |
*** nicedice__ has joined #openstack-infra | 16:14 | |
*** romcheg has joined #openstack-infra | 16:17 | |
*** hashar is now known as hasharConfCall | 16:20 | |
*** markwash has joined #openstack-infra | 16:23 | |
mordred | sdague: what you want me to look at? | 16:26 |
*** markmcclain has quit IRC | 16:32 | |
*** SergeyLukjanov has joined #openstack-infra | 16:34 | |
*** dstanek has quit IRC | 16:37 | |
*** Ryan_Lane has quit IRC | 16:38 | |
*** sandywalsh_ has quit IRC | 16:39 | |
fungi | mordred: he's talking about https://review.openstack.org/54518 | 16:39 |
*** dansmith is now known as damnsmith | 16:40 | |
*** damnsmith is now known as dansmith | 16:41 | |
tsufiev | mordred: hi! could you take a look at https://review.openstack.org/#/c/54370/ ? | 16:42 |
*** hasharConfCall is now known as hashar | 16:42 | |
clarkb | sdague: I can look in about an hour or so | 16:46 |
*** jcoufal has quit IRC | 16:47 | |
clarkb | actuall I took a quick look at it now and +1 from me until I can give it proper time | 16:48 |
*** pcrews has quit IRC | 16:50 | |
rpodolyaka | Hey, guys! Could you please take a look at https://review.openstack.org/#/c/53190 and https://review.openstack.org/#/c/52206/ when you have a minute? We need those to be able to do releases of TripleO projects (tuskar, tuskar-ui, python-tuskarclient, tripleo-heat-templates). Thanks! | 16:52 |
*** yamahata has quit IRC | 16:54 | |
*** sandywalsh_ has joined #openstack-infra | 16:54 | |
*** CaptTofu has quit IRC | 16:58 | |
*** CaptTofu has joined #openstack-infra | 16:58 | |
*** ArxCruz_ has joined #openstack-infra | 16:59 | |
*** sandywalsh_ has quit IRC | 17:00 | |
*** reed has joined #openstack-infra | 17:02 | |
*** aardvark has joined #openstack-infra | 17:02 | |
*** ArxCruz has quit IRC | 17:03 | |
*** WarrenUsui has quit IRC | 17:06 | |
*** guohliu has joined #openstack-infra | 17:09 | |
*** hashar has quit IRC | 17:09 | |
reed | hi all | 17:09 |
*** yassine has quit IRC | 17:10 | |
sdague | clarkb: yeh, given the number of times I've messed this one up, I'm totally cool with it getting proper time | 17:10 |
*** sandywalsh_ has joined #openstack-infra | 17:13 | |
*** davidhadas has joined #openstack-infra | 17:18 | |
*** guohliu has quit IRC | 17:19 | |
*** dstanek has joined #openstack-infra | 17:23 | |
*** nati_ueno has joined #openstack-infra | 17:24 | |
*** notmyname has quit IRC | 17:25 | |
*** notmyname has joined #openstack-infra | 17:26 | |
*** vipul-away is now known as vipul | 17:27 | |
*** harlowja has joined #openstack-infra | 17:29 | |
*** alcabrera is now known as alcabrera|afk | 17:32 | |
*** hub_cap has joined #openstack-infra | 17:35 | |
*** prad has joined #openstack-infra | 17:36 | |
*** guohliu has joined #openstack-infra | 17:36 | |
*** prad has quit IRC | 17:37 | |
*** prad has joined #openstack-infra | 17:37 | |
*** derekh has quit IRC | 17:41 | |
*** starmer has joined #openstack-infra | 17:43 | |
starmer | Hi, we're trying to get Gerrit gating set up, and I've sent an e-mail to the openstack-infra list for a service account. Jus thought I'd ping the network in case someone can help set that up (the email is awaiting moderation). Thanks! | 17:45 |
sdague | fungi: was there an upstream url for setting up your own fake d-g kind of thing? (this is for my slides for summit) | 17:45 |
pleia2 | starmer: I let your email through, it's just a super busy week for us with the summit coming up, I'm sure someone will have a look soon :) | 17:45 |
starmer | pleia2: I completely understand. Thanks for the help! | 17:46 |
fungi | sdague: it's in the d-g readme, if you're thinking of https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/README.rst#n100 | 17:47 |
*** CaptTofu has quit IRC | 17:48 | |
*** CaptTofu has joined #openstack-infra | 17:48 | |
*** mattymo has quit IRC | 17:49 | |
*** cody-somerville has quit IRC | 17:49 | |
*** mattymo has joined #openstack-infra | 17:50 | |
*** pcrews has joined #openstack-infra | 17:53 | |
*** krotscheck has joined #openstack-infra | 17:54 | |
reed | fungi, whatever, just lets close the review 53472 with any decision and move on | 17:55 |
reed | I can't stand leaving so many pending items before a long flight :) | 17:56 |
fungi | reed: mostly i'd like to get input from jeblair once he's around and available to discuss | 17:56 |
reed | ah! it's all the fault of the absents :) | 17:56 |
fungi | since he committed the atc scripts and might have strong feelings about why they ended up in openstack-infra/config (though my suspicion is that it was just a convenient dumping ground at the time) | 17:57 |
*** ArxCruz_ has quit IRC | 17:57 | |
reed | fungi, I think it's a simple decision to make, honestly, I think convenience comes to mind | 17:57 |
reed | yeah, I'm sure he just dropped it wherever it was convenient | 17:58 |
reed | just like I did :) | 17:58 |
*** ArxCruz has joined #openstack-infra | 17:58 | |
reed | I'm happy to take the offer by ttx to host it in odsreg | 17:58 |
fungi | as i said in the review, i'm okay with doing that for now and then discussing further. it's just as easy to move both as one | 17:58 |
fungi | particularly if we stick them in the same place though | 17:58 |
fungi | so openstack-infra/config seems better short term than openstack-infra/odsreg | 17:59 |
reed | approve it and lets move on | 17:59 |
fungi | unless openstack-infra/odsreg is the long-term place for them both to end up | 17:59 |
reed | i give up | 17:59 |
* reed declares decision bankruptcy | 18:00 | |
fungi | agreed, it's bikeshed city and there's no way to make everyone happy. published is better than not published, just also don't want to go approving it when i'm the only infra core weighing in. but it's rotted there for a week, so can't say there wasn't opportunity to discuss. approving | 18:01 |
*** cody-somerville has joined #openstack-infra | 18:04 | |
*** jerryz has joined #openstack-infra | 18:07 | |
openstackgerrit | A change was merged to openstack-infra/config: Adds the script used to send summit invitations to ATC https://review.openstack.org/53472 | 18:08 |
fungi | reed: ^ | 18:11 |
reed | WOOT | 18:11 |
*** starmer has quit IRC | 18:11 | |
lifeless | SlickNik: ah git needed; do. | 18:11 |
* reed likes removing things from a list | 18:11 | |
lifeless | SlickNik: care to also file a bug that that isn't tested for? | 18:11 |
reed | now going back to the Blueprint review process discussion | 18:12 |
reed | fungi, where is mordred? already in HK? | 18:12 |
fungi | reed: mordred is like any good subatomic particle... he's everywhere and nowhere until you end up in the same room with him and collapse the wave function | 18:13 |
*** mattymo has quit IRC | 18:13 | |
SlickNik | lifeless: I'm not sure I completely understand what you mean by "that isn't tested for". Can you explain? | 18:13 |
*** denis_makogon has joined #openstack-infra | 18:13 | |
SlickNik | lifeless: It seems like pbr should continue to work even if git isn't installed. | 18:15 |
SlickNik | lifeless: Ah, perhaps you meant have a test to test that, so that a regression of this won't occur. | 18:15 |
SlickNik | lifeless: And I will gladly open a bug for that. | 18:16 |
*** starmer has joined #openstack-infra | 18:17 | |
SlickNik | And if it seems like I'm talking to myself — I'm still on my first cup of coffee. :) | 18:17 |
lifeless | SlickNik: exactly what I mean. | 18:17 |
lifeless | SlickNik: I suggest not sitting on your coffee; it will make your but hot. | 18:17 |
SlickNik | heh | 18:18 |
*** SergeyLukjanov is now known as _SergeyLukjanov | 18:18 | |
*** starmer has quit IRC | 18:18 | |
*** jpich has quit IRC | 18:19 | |
*** _SergeyLukjanov has quit IRC | 18:19 | |
SlickNik | Small price to pay if it means that it'll stop me from occasionally talking out of it… | 18:19 |
*** nati_ueno has quit IRC | 18:19 | |
*** nati_ueno has joined #openstack-infra | 18:20 | |
*** fungi has quit IRC | 18:21 | |
*** SergeyLukjanov has joined #openstack-infra | 18:23 | |
*** alcabrera|afk is now known as alcabrera | 18:23 | |
SlickNik | But enough of that. I've filed bug https://bugs.launchpad.net/ubuntu/+source/python-pbr/+bug/1246438 for having a test for this case. | 18:24 |
*** nati_ueno has quit IRC | 18:24 | |
*** SergeyLukjanov has quit IRC | 18:25 | |
lifeless | SlickNik: LOL | 18:25 |
lifeless | SlickNik: coffee flavoured arse talk. Hopefully I can purge that memory by HK. | 18:25 |
clarkb | fungi timed out :( | 18:27 |
clarkb | sdague: I am looking at https://review.openstack.org/#/c/54518/1/devstack-vm-gate-wrap.sh agan | 18:28 |
*** nati_ueno has joined #openstack-infra | 18:28 | |
clarkb | sdague: and I am trying to convince myself that this change changes the behavior of d-g | 18:28 |
clarkb | Checkout zuul was 1 before | 18:29 |
*** osanchez has quit IRC | 18:29 | |
clarkb | so it would fall into checking the various conditions, if none of them match then do what we do when checkout zuul is 0 | 18:29 |
clarkb | sdague: won't we fall through the conditions identically to before? | 18:29 |
*** markmc has quit IRC | 18:30 | |
*** SergeyLukjanov has joined #openstack-infra | 18:30 | |
openstackgerrit | A change was merged to openstack-infra/config: Remove docs-*-gate for murano-metadataclient project. https://review.openstack.org/54370 | 18:30 |
clarkb | sdague: or were we hitting the exit in the inner if | 18:30 |
*** romcheg has quit IRC | 18:31 | |
mordred | SlickNik, lifeless: what about it doesn't work if git isn't installed? | 18:32 |
*** ryanpetrello has quit IRC | 18:32 | |
mordred | are you suggesting that it should work in a git clone without the existence of git? or that it should work in a tarball without the existence of git? | 18:32 |
*** dcramer_ has quit IRC | 18:32 | |
* mordred goes to read the bug | 18:32 | |
SlickNik | mordred: it should work as a tarball (or download from pypi) without the existence of git. | 18:32 |
mordred | yes. I agree with that | 18:33 |
mordred | thank you! I can find and fix that | 18:33 |
*** romcheg has joined #openstack-infra | 18:33 | |
SlickNik | mordred: https://review.openstack.org/#/c/54510/ | 18:33 |
mordred | SlickNik: omg. you are a god child | 18:34 |
mordred | wait. that didn't sound good | 18:34 |
mordred | you are something positive | 18:34 |
SlickNik | Heh, no matter what that sounded like, I'll take it as a compliment. | 18:35 |
mordred | clarkb: ^^ can I get you to review that? It's simple, but I think we need to land it and release, as there is a break in the wild | 18:35 |
*** DinaBelova has quit IRC | 18:36 | |
clarkb | mordred: ok | 18:36 |
clarkb | sdague: please ping me when you get my comments above, I need a little extra convincing | 18:36 |
SlickNik | mordred / clarkb: a fix and release would be nice. That way I don't have to come up with a workaround to unblock the trove integration-test gate. Thanks! | 18:37 |
mordred | SlickNik: I will release as soon as that lands | 18:38 |
clarkb | mordred: SlickNik: I am trying to decide if which git is the best way to check | 18:38 |
clarkb | does python os come with something like that? | 18:38 |
clarkb | seems like subprocess ought to be able to tell us maybe? | 18:39 |
mordred | dhellmann-afk, dstufft, lifeless: you know deep weird python arcane questions ^^ | 18:41 |
lifeless | what's the question ? | 18:41 |
mordred | any thoughts on the 'best' way to determine if an executable exists in the current path? | 18:41 |
mordred | in python | 18:41 |
mordred | other than shelling to which? | 18:41 |
clarkb | mordred: I ask because _run_shell_command isn't returning an exit code and relying on output seems a little scary | 18:42 |
lifeless | clarkb: subprocess.check_command | 18:42 |
clarkb | oh we can set throw_on_error | 18:42 |
clarkb | lifeless: danke | 18:42 |
lifeless | or $ yeah, but tats a local library thing | 18:42 |
lifeless | subprocess.check_output if you want to capture the output. | 18:43 |
lifeless | anyhow | 18:43 |
lifeless | I think shelling out is fine | 18:43 |
clarkb | ok | 18:43 |
lifeless | otherwise you need to reimplement 'which' which has already been done, but since pbr is dep shy that means code copying or cleanroom | 18:43 |
lifeless | all of which are fail | 18:43 |
clarkb | SlickNik: mordred: do you think the exit code should be checked instead of the output? | 18:43 |
lifeless | if which isn't installed and git is it will show as git not present | 18:44 |
lifeless | but I think that that is fine | 18:44 |
*** hogepodge has joined #openstack-infra | 18:44 | |
mordred | I agree | 18:44 |
mordred | if which isn't installed, JEEZ | 18:44 |
*** fungi has joined #openstack-infra | 18:45 | |
fungi | yeesh. serious irc connection issues suddenly | 18:45 |
clarkb | mordred: I have approved with a comment about using exit codes in general | 18:46 |
clarkb | fungi: we missed you | 18:46 |
SlickNik | I'm okay with that. The reason I went the which route was because I could re-use _run_shell_command() and not have to write more code wrapping another subprocess call. | 18:46 |
clarkb | SlickNik: ya | 18:46 |
clarkb | fungi: https://review.openstack.org/#/c/54518/1/devstack-vm-gate-wrap.sh when failing before were we hitting the exit on line 143? | 18:47 |
clarkb | fungi: sdague's change only makes sense to me if we were hitting that exit | 18:47 |
*** markwash has quit IRC | 18:47 | |
SlickNik | also clarkb, I agree with your comment. Thanks! | 18:48 |
*** SnowDust has joined #openstack-infra | 18:48 | |
*** markwash has joined #openstack-infra | 18:50 | |
*** CaptTofu has quit IRC | 18:50 | |
sdague | clarkb: so... here is my theory | 18:50 |
fungi | clarkb: yeah, now i'm trying to remember what it was i changed there when we were running into branch setup problems a couple weeks ago. git blame to the rescue | 18:50 |
*** CaptTofu has joined #openstack-infra | 18:51 | |
sdague | clarkb: that if is way more complicated than you realize | 18:51 |
sdague | it's actually 3 ORed sets of ANDed conditionals | 18:51 |
*** jcoufal has joined #openstack-infra | 18:51 | |
sdague | so the problem is I actually put the branch check inside the first one | 18:52 |
*** flaper87 is now known as flaper87|afk | 18:52 | |
sdague | which failed, as expected... but it meant L136 was actually executed | 18:52 |
sdague | { [ "$ZUUL_REF" != "" ] && \134 { [ "$ZUUL_REF" != "" ] && \ | 18:52 |
sdague | 137 git fetch $ZUUL_URL/$PROJECT $ZUUL_REF ; } || \ | 18:52 |
sdague | which means that FETCH_HEAD was modified to the zuul_ref regardless | 18:53 |
fungi | ahh, right. what i added was line 123 there, eventually. after spinning my brane for far too long on that if block | 18:53 |
clarkb | sdague: OOOOHHHHH | 18:53 |
fungi | though we talked about how we really ought to refactor that set of conditionals out into something less dense and easier to follow | 18:54 |
sdague | yeh, that's seriously dense code | 18:54 |
clarkb | sdague: it has suddenly clicked | 18:54 |
sdague | yep | 18:54 |
*** esheffield has quit IRC | 18:54 | |
clarkb | sdague: fungi: should I go ahead and approve now? I should have time to grab lunch between approving and it merging | 18:54 |
sdague | yeh, lets try | 18:54 |
fungi | clarkb: sure, i'll be here | 18:54 |
sdague | it's an easy revert if it goes wrong | 18:54 |
fungi | as long as irc doesn't decide to give me the finger again, anywaty | 18:55 |
fungi | er, anyway | 18:55 |
sdague | heh | 18:55 |
clarkb | done | 18:55 |
sdague | cool | 18:56 |
fungi | clearly whoever wrote that initially decided to chain a couple of return codes to get the desired behavior out of a conditional on what was previously a single command | 18:56 |
fungi | and then it snowballed with another and another | 18:56 |
sdague | yep | 18:56 |
sdague | mordred has the oldest fingerprints on that block from git blame, so I'll just hold him responsible :) | 18:57 |
fungi | we always seem to | 18:57 |
sdague | and I just found an easy nova +2 to put behind it in the gate | 18:58 |
sdague | which will prove whether it works or not | 18:58 |
* fungi fumes... i am ""<-this close to commenting out the github api calls in manage-projects and seeing how much faster it runs (and whether i can get it to succeed) | 18:58 | |
sdague | nice | 18:59 |
sdague | wait, I thought I made those optional | 18:59 |
fungi | i think we're overrunning it now that we're at ~200 projects | 18:59 |
*** johnthetubaguy has quit IRC | 18:59 | |
*** markwash has quit IRC | 19:00 | |
fungi | every time i try to run it i get ['not authorized', 'command failed', 'rate limit exceeded'][random(3)] | 19:00 |
fungi | no, wait, sometimes it's '502: server error' too | 19:01 |
clarkb | we hit the rate limit? awesome | 19:02 |
openstackgerrit | Khai Do proposed a change to openstack-infra/jeepyb: add an export option https://review.openstack.org/54416 | 19:02 |
sdague | fungi: wonder if we can ask the github guys for more rate | 19:02 |
sdague | clarkb: given the resets in the gate, you'll have plenty of time for lunch :) | 19:03 |
fungi | clarkb: add to that the times where i try to run it and eventually cancel it after it runs for an hour without returning anything, presumably timing out the api socket | 19:04 |
clarkb | we could look into having them mirror from their side | 19:05 |
clarkb | and deal with that manually as a nice to have (I don't actually like this idea but would lessen the need for API calls) | 19:05 |
*** ^d has quit IRC | 19:06 | |
fungi | well, in this case it's the calls to check github to see whether each project we host exists there so we know to create it when it doesn't | 19:09 |
clarkb | fungi: right, when you have them mirror for you you do that all through email so we could just get rid of those checks | 19:09 |
clarkb | and deal with the project mirrors on github as more a best effort | 19:09 |
clarkb | (I don't like giving up control though) | 19:09 |
fungi | but i suspect running a separate api call to check each of them in rapid-fire every time we run manage-projects is overstepping some limits somewhere now that we have so many projects | 19:09 |
clarkb | fungi: there is the script that closes pull requests too | 19:10 |
clarkb | fungi: we should maybe make that one poll less often | 19:10 |
clarkb | and possibly use a different account? | 19:10 |
fungi | perhaps | 19:10 |
clarkb | I am going to grab lunch now | 19:10 |
fungi | eat well | 19:10 |
*** odyssey4me has quit IRC | 19:14 | |
*** senk has quit IRC | 19:15 | |
*** danger_fo_away has quit IRC | 19:18 | |
*** danger_fo_away has joined #openstack-infra | 19:19 | |
*** nati_ueno has quit IRC | 19:19 | |
*** ryanpetrello has joined #openstack-infra | 19:22 | |
sdague | I think we're good | 19:23 |
sdague | I'm watching the grenade run on the nova change behind it | 19:23 |
sdague | we at least didn't fail at the old point | 19:23 |
*** vipul is now known as vipul-away | 19:25 | |
*** vipul-away is now known as vipul | 19:25 | |
*** jaypipes has quit IRC | 19:28 | |
*** markmcclain has joined #openstack-infra | 19:29 | |
*** SnowDust has quit IRC | 19:30 | |
*** danger_fo_away has quit IRC | 19:33 | |
sdague | yep, grenade seems happy after that | 19:34 |
sdague | yay! probably figured this out | 19:34 |
pleia2 | hmm, gearman.org/manual is broken | 19:35 |
* fungi calls "not it" | 19:35 | |
*** danger_fo_away has joined #openstack-infra | 19:37 | |
*** nati_ueno has joined #openstack-infra | 19:40 | |
sdague | man, when we get reset heavy in the gate, those py26 nodes end up being an interesting bottleneck | 19:42 |
*** ^d has joined #openstack-infra | 19:42 | |
*** danger_fo_away has quit IRC | 19:42 | |
Shrews | pleia2: i believe brian is experimenting with a new way of publishing. probably related. see https://groups.google.com/forum/#!topic/gearman/LQetfXKXo9c | 19:42 |
pleia2 | Shrews: ah, thanks :) | 19:42 |
pleia2 | found a cached version anyway, so I'm good | 19:43 |
*** danger_fo_away has joined #openstack-infra | 19:45 | |
*** hashar has joined #openstack-infra | 19:47 | |
*** ryanpetrello_ has joined #openstack-infra | 19:50 | |
*** ryanpetrello has quit IRC | 19:51 | |
*** ryanpetrello_ is now known as ryanpetrello | 19:51 | |
*** senk has joined #openstack-infra | 19:55 | |
*** mriedem has quit IRC | 19:55 | |
*** markwash has joined #openstack-infra | 19:56 | |
*** Ryan_Lane has joined #openstack-infra | 19:57 | |
*** dolidze has joined #openstack-infra | 19:58 | |
*** senk has quit IRC | 19:59 | |
*** nati_ueno has quit IRC | 20:02 | |
*** dprince has quit IRC | 20:06 | |
*** adrian_otto has quit IRC | 20:07 | |
bodepd_ | is there a way to track the status of accont requests? | 20:09 |
bodepd_ | is: is there an associated ticket? | 20:09 |
*** nati_ueno has joined #openstack-infra | 20:11 | |
mgagne | bodepd_: yes | 20:12 |
*** dolidze has quit IRC | 20:12 | |
mgagne | looks to be on mailinglist | 20:13 |
mgagne | bodepd_: ^ | 20:13 |
mgagne | bodepd_: http://lists.openstack.org/pipermail/openstack-infra/2013-October/000368.html | 20:13 |
mgagne | bodepd_: best is to open a ticket for better follow-up. list isn't the best place for that. | 20:14 |
fungi | basically if someone asks in here and i'm not busy i'll add it (that's not very often these days). if they open an openstack-ci bug on lp or e-mail the openstack-infra ml i'll follow up wherever they asked | 20:15 |
*** weshay has quit IRC | 20:16 | |
fungi | it's easier for me to not miss it if it's on the ml or in a bug, but on the flip side it takes longer for me to notice it compared to irc | 20:16 |
fungi | but there's no set protocol | 20:17 |
*** guohliu has quit IRC | 20:17 | |
*** senk has joined #openstack-infra | 20:20 | |
*** thedodd has quit IRC | 20:20 | |
lifeless | clarkb: https://review.openstack.org/#/c/52206 | 20:21 |
lifeless | clarkb: and the dependency | 20:21 |
lifeless | fungi: ^ | 20:21 |
* lifeless hunts other sympathetic reviewers. | 20:21 | |
lifeless | mordred: ^ ! | 20:21 |
lifeless | is it possible for me to get a new group? tripleo-releasers ? I'd like to have tripleo-ptl have me + tripleo-releasers, for clarity if folk are poking around in gerrit | 20:23 |
*** alcabrera has quit IRC | 20:23 | |
fungi | the original intent was that foo-ptl was short for "foo ptl and release management delegates" | 20:24 |
*** thedodd has joined #openstack-infra | 20:24 | |
fungi | i'm not sure that we should promote gerrit as the official place for people to look up program technical leads | 20:25 |
lifeless | fungi: I'm not promoting that, but other folk do suggest it from time to time. | 20:25 |
*** CaptTofu has quit IRC | 20:26 | |
openstackgerrit | A change was merged to openstack-dev/pbr: Fixed pbr install to not error out and fail if git is not installed https://review.openstack.org/54510 | 20:26 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Correct test for zuulifying the tree https://review.openstack.org/54518 | 20:26 |
*** CaptTofu has joined #openstack-infra | 20:26 | |
lifeless | fungi: however, since I won the 'get jeremy's attention' game; how about reviews :) | 20:27 |
fungi | yeah, i'm looking at those at the moment | 20:28 |
fungi | i think someone filed a ci bug about the missing tuskar tarballs too? | 20:28 |
lifeless | Ng: or rpodolyaka: ^ probably | 20:29 |
lifeless | -> hairdresser bbiab | 20:29 |
*** lcestari has quit IRC | 20:30 | |
*** hogepodge_ has joined #openstack-infra | 20:32 | |
*** melwitt has joined #openstack-infra | 20:32 | |
clarkb | fungi: yeah I saw that bug earlier today | 20:33 |
clarkb | lunch has been acquired. zaro took us on a long walk to a kebab place | 20:33 |
*** hogepodge has quit IRC | 20:33 | |
*** hogepodge_ is now known as hogepodge | 20:33 | |
clarkb | sdague: woot | 20:34 |
*** dcramer_ has joined #openstack-infra | 20:35 | |
sdague | Dear Jim, Please never go on vacation again. K THX BAI | 20:35 |
clarkb | fungi: re tuskar, have they just not merged any code? | 20:35 |
fungi | Ng: one thing which worries me about 52206 and 53190 is that the acls for those projects allow tags to be pushed by some fairly large groups of developers (20 in fact). are they all actually comfortable with the instructions on doing that safely, or should that permission be scaled back to a very small subset? | 20:35 |
clarkb | fungi: I don't think we negotiate that :) | 20:36 |
fungi | well, it got through that way at some point (might have started out as a very small group) | 20:37 |
clarkb | ah | 20:37 |
*** mrda has joined #openstack-infra | 20:37 | |
fungi | but the acls on those projects allow tripleo-core or groups which have included tripleo-core into them on tag refs | 20:38 |
clarkb | lifeless: Ng: reviewed | 20:38 |
fungi | and it's grown to 20 members at this point | 20:38 |
mordred | sdague: he's out the week after the summit too | 20:38 |
clarkb | fungi: I think we should fix that | 20:39 |
zaro | clarkb: more long walks to come in HK. | 20:39 |
clarkb | mordred: I sort of forgive him for that because I have a feeling most of us will be zombies that week | 20:39 |
sdague | ಠ_ಠ | 20:39 |
* fungi is always a zombie | 20:39 | |
openstackgerrit | Roman Podoliaka proposed a change to openstack-infra/config: Add release config for tuskar components. https://review.openstack.org/53190 | 20:42 |
openstackgerrit | Roman Podoliaka proposed a change to openstack-infra/config: Add release config for tripleo-heat-templates https://review.openstack.org/52206 | 20:42 |
*** alcabrera has joined #openstack-infra | 20:46 | |
clarkb | fungi: any luck with tuskar? | 20:56 |
clarkb | the tarball jobs specifically | 20:56 |
*** afazekas has quit IRC | 20:56 | |
*** marun has joined #openstack-infra | 20:58 | |
clarkb | mordred: are you going to release new pbr? | 20:59 |
mordred | clarkb: I can't - behind firewall | 20:59 |
mordred | clarkb: you want to? | 20:59 |
clarkb | mordred: I can | 20:59 |
mordred | clarkb: 0.5.23 | 21:00 |
clarkb | k | 21:00 |
fungi | clarkb: i hadn't hunted down the bug report yet | 21:00 |
SlickNik | clarkb: <3 | 21:01 |
fungi | last couple bugs about that which were opened, the projects lacked local machinery to create an sdist the way our jobs expect (particularly popular is forgetting testenv:venv section in tox.ini) | 21:01 |
SlickNik | clarkb: ping me when it's done, so I can re-trigger trove-integration tests. | 21:02 |
clarkb | tag pushed | 21:02 |
*** dkranz has quit IRC | 21:02 | |
SlickNik | clarkb: cool, thanks! | 21:02 |
clarkb | SlickNik: I would retrigger in about an hour | 21:02 |
clarkb | to make sure all of the bits trickle down to where they need to go (like the mirror) | 21:03 |
SlickNik | clarkb: Good point. Thanks! | 21:04 |
openstackgerrit | Monty Taylor proposed a change to openstack-dev/pbr: Communicate to user when we skip a requirement https://review.openstack.org/54628 | 21:07 |
openstackgerrit | Monty Taylor proposed a change to openstack-dev/pbr: Base python 2.6 skip list on parsed names https://review.openstack.org/54152 | 21:07 |
*** nati_ueno has quit IRC | 21:07 | |
*** nati_ueno has joined #openstack-infra | 21:08 | |
*** nati_ueno has quit IRC | 21:08 | |
*** nati_ueno has joined #openstack-infra | 21:09 | |
clarkb | fungi: http://git.openstack.org/cgit/openstack/python-tuskarclient/log/ I think tuskarclient hasn't merged any code | 21:10 |
*** dcramer_ has quit IRC | 21:12 | |
fungi | that seems like a good hypothesis. last commit was under the 0.1.1 tag a couple weeks ago | 21:12 |
fungi | er, 0.0.1 | 21:12 |
fungi | nah, it has gerrit notes though... http://git.openstack.org/cgit/openstack/python-tuskarclient/commit/?id=b44af3e | 21:13 |
clarkb | right but if that tarball jobs went in after that merged | 21:13 |
fungi | a.k.a. https://review.openstack.org/52414 | 21:13 |
fungi | oh, i see what you mean. post the config change to add the tarball job | 21:13 |
clarkb | yeah tarball stuff went in a few days ago | 21:14 |
fungi | yep, git log says 5 days after the last commit on that project | 21:17 |
*** nati_ueno has quit IRC | 21:18 | |
*** nati_ueno has joined #openstack-infra | 21:18 | |
*** senk has quit IRC | 21:19 | |
*** pcrews has quit IRC | 21:20 | |
*** senk has joined #openstack-infra | 21:20 | |
*** nati_ueno has quit IRC | 21:22 | |
*** senk has quit IRC | 21:23 | |
*** rakhmerov has quit IRC | 21:24 | |
*** dims has quit IRC | 21:25 | |
*** jergerber has quit IRC | 21:25 | |
*** pcm_ has quit IRC | 21:29 | |
*** senk has joined #openstack-infra | 21:31 | |
*** rcleere has quit IRC | 21:32 | |
*** senk has quit IRC | 21:32 | |
*** dkliban has joined #openstack-infra | 21:35 | |
sdague | clarkb: ... ok, so that last patch isn't exactly perfect yet | 21:35 |
sdague | I did the novaclient revert revert and it passed | 21:36 |
sdague | it looks like it should be $ZUUL_BRANCH == $BRANCH after all | 21:39 |
lifeless | clarkb: fungi: I'm not worried about that at the moment | 21:39 |
sdague | to handle the client case | 21:39 |
lifeless | clarkb: fungi: though I will admit we didn't realise the permissions worked that way | 21:39 |
*** dims has joined #openstack-infra | 21:40 | |
clarkb | sdague: ok | 21:41 |
*** senk has joined #openstack-infra | 21:41 | |
sdague | fortunately, it means we just aren't catching one case, but it's not busted | 21:42 |
clarkb | lifeless: I am worried about it | 21:42 |
clarkb | lifeless: because tags that don't get done correctly usually involve a fair amount of effort from infra to clean up after | 21:43 |
*** senk has quit IRC | 21:43 | |
*** mkirk has quit IRC | 21:43 | |
fungi | a lot of effort, to the point where it's often better not to clean them up and live with the mistake | 21:43 |
*** marun has quit IRC | 21:43 | |
clarkb | it is better to have a small group of people that do it semi often and understand the process which avoids problems | 21:43 |
lifeless | clarkb: there are 20 core reviews for tripleo. | 21:44 |
lifeless | reviewers | 21:44 |
clarkb | right that is about 18-17 too many for tag privileges | 21:44 |
*** CaptTofu has quit IRC | 21:45 | |
clarkb | we really need gerrit to make tags reviewable | 21:45 |
clarkb | then we can avoid this trouble | 21:45 |
lifeless | well | 21:45 |
fungi | for comparison, across all of the integrated server projects, there are something like a total of three people tagging them for releases | 21:45 |
lifeless | Having some automation around 'good tag' rules would be nice | 21:45 |
*** CaptTofu has joined #openstack-infra | 21:45 | |
lifeless | clarkb: fungi: nevertheless, this is orthogonal to being able to do releases at all. | 21:45 |
lifeless | clarkb: fungi: because the set of people is already enabled for e.g. os-apply-config, no ? | 21:46 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Ensure clients get tested correctly in upgrades https://review.openstack.org/54638 | 21:46 |
fungi | yes, which is why i +2'd the patches, just raising a concern about the existing acl | 21:46 |
clarkb | mordred: did you see jd__'s message about requirements proposing updates when stable/havana requirements change? | 21:46 |
lifeless | fungi: cool, but I had the impression clarkb was holding off | 21:46 |
clarkb | lifeless: I am | 21:46 |
fungi | k | 21:46 |
lifeless | and I don't want that, as we have weeks of inventory building up | 21:46 |
openstackgerrit | Khai Do proposed a change to openstack-dev/pbr: show transitive dependencies https://review.openstack.org/54639 | 21:46 |
clarkb | I think we should fix that before adding more projects that can be tagged | 21:46 |
fungi | that's a reasonable concern | 21:47 |
lifeless | clarkb: I think thats holding an unrelated change hostage | 21:47 |
clarkb | it is a quick fix and should be corrected | 21:47 |
lifeless | clarkb: which I am investigating now | 21:47 |
*** thomasem has quit IRC | 21:47 | |
*** CaptTofu has quit IRC | 21:47 | |
*** CaptTofu has joined #openstack-infra | 21:47 | |
clarkb | lifeless: it isn't unrelated, aren't we adding more projects that use the group? | 21:48 |
clarkb | we are multiplying the surface area of breakage | 21:48 |
lifeless | clarkb: we're fixing the broken config on existing projects | 21:48 |
lifeless | clarkb: we've had no incidents of bad tags so far | 21:48 |
clarkb | I must have changes confused then | 21:48 |
lifeless | clarkb: from this group | 21:48 |
clarkb | I thought we were talking about the Ng change with tuskar projects release stuff being added | 21:48 |
lifeless | yes, they are broken at the moment | 21:48 |
lifeless | we can't do releases. | 21:48 |
lifeless | we want | 21:48 |
lifeless | we've wanted to for three weeks | 21:48 |
fungi | well, the changes add jobs to release based on tags to additional projects, but the acl already allows the tags to be pushed by the large set of people | 21:48 |
*** senk has joined #openstack-infra | 21:48 | |
lifeless | the overly large set has existing for 2 months | 21:49 |
lifeless | so bad tags can already be pushed to all of these projects | 21:49 |
fungi | so the risk that someone pushes tags incorrectly to those projects already existst whether or not the jobs get added | 21:49 |
lifeless | they just won't trigger releases for the ones we're fixing. | 21:49 |
clarkb | lifeless: correct, but we are adding new ones now | 21:49 |
*** jcoufal has quit IRC | 21:49 | |
lifeless | no, again, we're not. | 21:49 |
lifeless | they can already push tags. | 21:49 |
clarkb | lifeless: can you point me to the change? | 21:49 |
clarkb | because I am confused | 21:49 |
clarkb | lifeless: they can push tags but without the jobs bad stuff doesn't really happen | 21:49 |
fungi | well, some stuff does. the tags end up in the repository and replicated to all the mirrors | 21:50 |
lifeless | clarkb: https://review.openstack.org/#/c/52206 | 21:50 |
lifeless | clarkb: and https://review.openstack.org/#/c/53190/4 | 21:50 |
lifeless | clarkb: neither of these change the tagging rules | 21:50 |
clarkb | lifeless: https://review.openstack.org/#/c/53190/4/modules/openstack_project/files/zuul/layout.yaml is the one I am concerned about | 21:51 |
lifeless | clarkb: since we have had /zero/ incidents I think your fear of mistakes is out of proportion to the risk | 21:51 |
clarkb | because it is changing what happens on a tag | 21:51 |
lifeless | clarkb: and I've already said I'm looking into the fix you want | 21:51 |
clarkb | lifeless: why can't we just add a chnage to fix it? | 21:51 |
*** mrodden has quit IRC | 21:51 | |
fungi | i will gladly expedite any acl patch to that effect | 21:52 |
lifeless | clarkb: because we're here debating rather than doing that ? | 21:52 |
*** senk has quit IRC | 21:53 | |
openstackgerrit | lifeless proposed a change to openstack-infra/config: Tagging of TripleO jobs should be tripleo-ptl. https://review.openstack.org/54641 | 21:54 |
lifeless | clarkb: fungi: ^ | 21:54 |
*** markmcclain has quit IRC | 21:54 | |
*** rakhmerov has joined #openstack-infra | 21:55 | |
clarkb | thanks | 21:55 |
clarkb | it has my +2 | 21:55 |
clarkb | fungi feel free to approve then I will lookat at the stack on 53190 | 21:55 |
*** romcheg has left #openstack-infra | 21:56 | |
*** adalbas has quit IRC | 21:56 | |
openstackgerrit | A change was merged to openstack-infra/config: Tagging of TripleO jobs should be tripleo-ptl. https://review.openstack.org/54641 | 21:57 |
*** mgagne has quit IRC | 21:58 | |
clarkb | fungi: lifeless: stack approved | 21:59 |
*** mgagne has joined #openstack-infra | 21:59 | |
*** SergeyLukjanov has quit IRC | 21:59 | |
openstackgerrit | A change was merged to openstack-infra/config: Add release config for tuskar components. https://review.openstack.org/53190 | 21:59 |
lifeless | clarkb: thank you | 22:00 |
openstackgerrit | A change was merged to openstack-infra/config: Add release config for tripleo-heat-templates https://review.openstack.org/52206 | 22:01 |
*** adrian_otto has joined #openstack-infra | 22:01 | |
openstackgerrit | DennyZhang proposed a change to openstack-infra/config: defesive coding for docbook-properties.sh https://review.openstack.org/54642 | 22:03 |
*** prad has quit IRC | 22:04 | |
*** thedodd has quit IRC | 22:05 | |
*** dstanek has quit IRC | 22:05 | |
openstackgerrit | DennyZhang proposed a change to openstack-infra/config: defesive coding for docbook-properties.sh https://review.openstack.org/54642 | 22:06 |
*** pcrews has joined #openstack-infra | 22:07 | |
hashar | hey there :-] | 22:10 |
hashar | I got a quick question, if I want to add a sphinx extension, should I put it in requirements.txt or test-requirements.txt ? :D | 22:10 |
*** Ryan_Lane has quit IRC | 22:10 | |
clarkb | sdague: where did that 3A come from in your commit msg for the d-g change? | 22:11 |
sdague | bad keystroke? | 22:11 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: Ensure clients get tested correctly in upgrades https://review.openstack.org/54638 | 22:12 |
clarkb | hashar: runtime requirements should go in requirements.txt. If you only need it during doc build time I would put it in test-requirements.txt | 22:12 |
hashar | so be it. thank you clarkb ! | 22:12 |
clarkb | hashar: I think ceilometer may be using a doc-requirements.txt too maybe | 22:12 |
hashar | ah | 22:12 |
clarkb | hashar: but I would just do test-requirements. it is simple and works today | 22:12 |
hashar | I am wondering whether pbr supports doc-requirements already | 22:12 |
clarkb | hashar: I don't think there is direct support | 22:13 |
sdague | clarkb: I think I might have started in a weird editor for the commit message, and some arrow key went odd | 22:13 |
hashar | clarkb: thank you very much. | 22:13 |
*** alcabrera has quit IRC | 22:15 | |
*** dkliban has quit IRC | 22:16 | |
*** senk has joined #openstack-infra | 22:17 | |
*** senk has quit IRC | 22:18 | |
*** gyee has joined #openstack-infra | 22:22 | |
*** sandywalsh_ has quit IRC | 22:25 | |
*** rakhmerov has quit IRC | 22:26 | |
*** rakhmerov1 has joined #openstack-infra | 22:26 | |
*** dims has quit IRC | 22:26 | |
*** datsun180b has quit IRC | 22:26 | |
*** dims has joined #openstack-infra | 22:27 | |
*** rakhmerov1 has quit IRC | 22:30 | |
anteaya | HK money is large, the mony gets bigger the greater th denomination | 22:37 |
anteaya | like euros but bigger | 22:37 |
openstackgerrit | DennyZhang proposed a change to openstack-infra/config: remove code duplication in run-*.sh https://review.openstack.org/54646 | 22:37 |
*** ryanpetrello_ has joined #openstack-infra | 22:39 | |
*** ryanpetrello has quit IRC | 22:41 | |
*** ryanpetrello_ is now known as ryanpetrello | 22:41 | |
*** sandywalsh_ has joined #openstack-infra | 22:42 | |
pleia2 | anteaya: neat | 22:46 |
*** sdake has quit IRC | 22:49 | |
*** ryanpetrello has quit IRC | 22:49 | |
*** esker has quit IRC | 22:52 | |
clarkb | are they plastic? | 22:52 |
*** shardy is now known as shardy_afk | 22:52 | |
anteaya | just the 10's | 22:53 |
anteaya | for what my bank gave me at least | 22:54 |
openstackgerrit | Antoine Musso proposed a change to openstack-infra/zuul: doc: enhance gating reference with diagrams https://review.openstack.org/54648 | 22:55 |
openstackgerrit | Antoine Musso proposed a change to openstack-infra/zuul: doc: enhance gating reference with diagrams https://review.openstack.org/54648 | 22:55 |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Added globbed parameters to the job specification https://review.openstack.org/48688 | 22:57 |
hashar | clarkb: http://docs-draft.openstack.org/48/54648/2/check/gate-zuul-docs/e145a10/doc/build/html/gating.html :-] | 22:59 |
hashar | clarkb: browse at bottom for some nice graphs made possible with a sphinx plugin \O/ | 22:59 |
clarkb | hashar: nice! | 22:59 |
hashar | http://blockdiag.com/ .. similar to graphviz but way easier, python based and integrates with sphinx | 23:00 |
*** mdenny has quit IRC | 23:00 | |
hashar | i luuuve python | 23:00 |
clarkb | interesting. the doc team is triyng to use graphviz in sphinx stuff, maybe I should point them at that to reduce the system dependencies | 23:01 |
hashar | definitely | 23:01 |
hashar | if that match their need | 23:01 |
hashar | at worth, they could hack it and upstream their change | 23:01 |
hashar | clarkb: another example for them https://commons.wikimedia.org/wiki/File:Integrationwikimediaci-toolsinteractions.png | 23:02 |
hashar | that more or less describe the gerrit/zuul/jenkins workflow | 23:02 |
*** mindjiver has quit IRC | 23:03 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Jenkins Ownership Plugin https://review.openstack.org/51441 | 23:04 |
*** gyee has quit IRC | 23:05 | |
*** mindjiver has joined #openstack-infra | 23:07 | |
*** gyee has joined #openstack-infra | 23:09 | |
*** oubiwann has quit IRC | 23:17 | |
*** fbo has quit IRC | 23:19 | |
*** jhesketh has joined #openstack-infra | 23:20 | |
openstackgerrit | lifeless proposed a change to openstack-infra/reviewday: Add all the TripleO projects. https://review.openstack.org/54649 | 23:21 |
openstackgerrit | lifeless proposed a change to openstack-infra/reviewday: Ignore editor backup files. https://review.openstack.org/54650 | 23:21 |
*** CaptTofu has quit IRC | 23:22 | |
*** dcramer_ has joined #openstack-infra | 23:22 | |
*** CaptTofu has joined #openstack-infra | 23:23 | |
*** denis_makogon has quit IRC | 23:24 | |
morganfainberg | anteaya, i just sent another of my coworkers to your vagrant blog post (and other blog posts) told him to hit you up if he has feedback ... or other cool tricks and such :) | 23:25 |
morganfainberg | anteaya, figured i'd give ya a heads up :) | 23:25 |
*** rakhmerov has joined #openstack-infra | 23:26 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Adding support for the Build Result Trigger plugin https://review.openstack.org/52126 | 23:27 |
*** UtahDave has quit IRC | 23:29 | |
*** hashar has quit IRC | 23:33 | |
*** pcrews has quit IRC | 23:35 | |
*** rwsu has quit IRC | 23:38 | |
*** julim has quit IRC | 23:39 | |
*** wenlock has quit IRC | 23:43 | |
*** gyee has quit IRC | 23:44 | |
*** rwsu has joined #openstack-infra | 23:45 | |
*** julim has joined #openstack-infra | 23:45 | |
anteaya | morganfainberg thank you for both, the complement and the heads up | 23:48 |
*** CaptTofu has quit IRC | 23:48 | |
*** CaptTofu has joined #openstack-infra | 23:49 | |
* morganfainberg goes back to watching ZuulTV | 23:50 | |
*** markwash has quit IRC | 23:57 | |
*** adrian_otto has quit IRC | 23:57 | |
*** rakhmerov has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!