*** sarob has joined #openstack-infra | 00:03 | |
*** sarob has quit IRC | 00:08 | |
*** sarob has joined #openstack-infra | 00:09 | |
*** CaptTofu has joined #openstack-infra | 00:09 | |
*** sarob has quit IRC | 00:13 | |
*** DennyZhang has joined #openstack-infra | 00:19 | |
*** sarob has joined #openstack-infra | 00:28 | |
*** nati_ueno_2 has quit IRC | 00:33 | |
*** sarob has quit IRC | 00:34 | |
*** nati_ueno has joined #openstack-infra | 00:35 | |
clarkb | jeblair: the current doc reord change (45490) lgtm. Did you want another pass at reviewing it? | 00:38 |
---|---|---|
*** michchap has joined #openstack-infra | 00:40 | |
*** michchap_ has joined #openstack-infra | 00:41 | |
*** michchap has quit IRC | 00:44 | |
*** CaptTofu has quit IRC | 00:49 | |
*** CaptTofu has joined #openstack-infra | 00:50 | |
*** sarob has joined #openstack-infra | 00:55 | |
*** DennyZhang has quit IRC | 00:57 | |
*** zhiyan has joined #openstack-infra | 01:09 | |
zhiyan | fungi: ping | 01:10 |
*** sarob has quit IRC | 01:11 | |
*** sarob has joined #openstack-infra | 01:11 | |
zhiyan | anteaya: ping | 01:11 |
anteaya | zhiyan: hello | 01:11 |
zhiyan | anteaya: Hello. sorry to interrupt you, could you pls help look a Jenkins failure? | 01:12 |
zhiyan | http://logs.openstack.org/71/45171/2/check/gate-python-cinderclient-python26/89ba852/ | 01:13 |
* anteaya looks | 01:13 | |
zhiyan | anteaya: i'm confused what's the reason cause that... | 01:13 |
zhiyan | anteaya: thank you | 01:13 |
*** sarob has quit IRC | 01:16 | |
anteaya | well the test failure seems to be consistent across all four times jenkins has tested the patch | 01:18 |
clarkb | zhiyan: anteaya: one of the test runners exited with a non zero return value | 01:18 |
anteaya | so I have to assume that there is something in your code causing the failure | 01:18 |
clarkb | I would check that there isn't a sys.exit somewhere | 01:18 |
zhiyan | anteaya: ok, actually i'm confused on this err msg: | 01:20 |
zhiyan | ft1.37: process-returncode_StringException: Binary content: traceback (test/plain; charset="utf8") | 01:20 |
zhiyan | anteaya: those two at here: http://logs.openstack.org/71/45171/2/check/gate-python-cinderclient-python26/89ba852/testr_results.html.gz | 01:20 |
anteaya | zhiyan: clarkb gave you a good suggestion | 01:20 |
anteaya | is there a sys.exit() call in the code? | 01:21 |
zhiyan | anteaya: hmm, iirc the answer is NO | 01:21 |
*** michchap has joined #openstack-infra | 01:21 | |
zhiyan | i have not added 'sys.exec()' in the patch code | 01:22 |
*** nosnos has joined #openstack-infra | 01:22 | |
clarkb | zhiyan: sys.exit | 01:22 |
clarkb | or anything else that would cause the tes runner to prematurely die | 01:23 |
zhiyan | clarkb: ops, sorry, i want to say 'sys.exit' | 01:23 |
zhiyan | clarkb: actually i can pass all ut test cases in local | 01:24 |
zhiyan | clarkb: i did it everytime before i push it to review | 01:24 |
*** michchap_ has quit IRC | 01:24 | |
clarkb | zhiyan: if you look in http://logs.openstack.org/71/45171/2/check/gate-python-cinderclient-python26/89ba852/subunit_log.txt.gz you will see the sys.exit | 01:25 |
clarkb | it is happening in argparse | 01:25 |
zhiyan | let me check it | 01:25 |
morganfainberg | zhiyan, look to see if you've added an argument somewhere that hasn't been registered. | 01:29 |
morganfainberg | zhiyan, and used that "new" argument. | 01:30 |
zhiyan | morganfainberg: nod, probably. checking | 01:30 |
clarkb | zhiyan: I think you are not passing the volume id in the test | 01:35 |
clarkb | zhiyan: also shouldn't the volume arg be volume_id for consistency? | 01:35 |
zhiyan | morganfainberg: do you think this code is enough for 'register' new agrument? and i'm still confused why i can pass all test on my local dev env... | 01:35 |
jeblair | clarkb: lgtm; do you know whether annegentle_ is ready for approval? | 01:36 |
clarkb | jeblair: I don't know if she is ready | 01:36 |
*** pcrews has joined #openstack-infra | 01:36 | |
morganfainberg | zhiyan, that much i don't know. look at what clarkb recommended, might have something to do with that. | 01:36 |
jeblair | annegentle_: 45490 is ready when you are | 01:37 |
zhiyan | clarkb: which test cause you meaning? | 01:37 |
zhiyan | s/cause/case | 01:38 |
clarkb | zhiyan: the ones in the stacktraces in the subunit log | 01:38 |
zhiyan | clarkb: hmm, i think i have passed the volume in (https://review.openstack.org/#/c/45171/2/cinderclient/tests/v1/test_shell.py L267 and L271). | 01:40 |
zhiyan | clarkb: yes i think they should be consistent. | 01:41 |
clarkb | zhiyan: 267 and 271 are assertions | 01:41 |
clarkb | run_command on 265 and 269 is where you need the argument | 01:41 |
*** Ryan_Lane has quit IRC | 01:43 | |
zhiyan | clarkb: correct. fine, let me check. your suggestion is useful to me, thank you clarkb. | 01:43 |
*** yaguang has joined #openstack-infra | 01:44 | |
*** fifieldt has joined #openstack-infra | 01:46 | |
fifieldt | hi infra :) | 01:47 |
fifieldt | greetings from the pre-doc-camp dinner | 01:47 |
clarkb | fifieldt: hello there. areyou guys ready for 45490 to merge? | 01:47 |
clarkb | fifieldt: both jeblair and I are happy with it but didn't want to hit the go button until we got the go ahead from annegentle_ | 01:47 |
fifieldt | just talked to her :) | 01:48 |
fifieldt | us an hours time or so ok? | 01:48 |
fifieldt | we just want to be in front of screens for the docpocalypse | 01:49 |
clarkb | sure | 01:49 |
clarkb | I am sitting at home watching the football game, I should still be around in an hour or so | 01:49 |
fifieldt | nyg vs dal? | 01:49 |
clarkb | yup | 01:49 |
fifieldt | cool, its on here too | 01:50 |
anteaya | fifieldt: how's the dinner? | 01:51 |
anteaya | got a full house? | 01:51 |
*** CaptTofu has quit IRC | 01:51 | |
*** CaptTofu has joined #openstack-infra | 01:51 | |
fifieldt | about 10 of us here on the Tied House Brewery | 01:51 |
anteaya | nice | 01:52 |
fifieldt | everyone says hi and thanks :) | 01:55 |
*** matsuhashi has joined #openstack-infra | 01:56 | |
anteaya | :D | 01:56 |
*** Ryan_Lane has joined #openstack-infra | 01:58 | |
morganfainberg | hm. docpocalypse eh? | 02:03 |
clarkb | morganfainberg: reorging the structure of the docs to make them more consistent | 02:06 |
clarkb | morganfainberg: this requires updating all of the jenkins jobs | 02:07 |
morganfainberg | clarkb, fun times. | 02:07 |
anteaya | morganfainberg: I am operating under the belief that this is the patch in question: https://review.openstack.org/#/c/45490/12 | 02:09 |
morganfainberg | anteaya, ahh | 02:09 |
anteaya | I'm hands off but watching | 02:09 |
morganfainberg | hehe. | 02:09 |
morganfainberg | i need to stop surfing the web and chatting in IRC so i can write some code i've been procrastinating on :P | 02:10 |
anteaya | ha ha ha | 02:11 |
anteaya | good luck | 02:11 |
morganfainberg | maybe i'll go wander down the street here and get a beer, food, and some coffee first though | 02:11 |
anteaya | I've been fighting for two days to stand something up, I think I have it breathing - I have to wait for pleia2 tomorrow so I can configure it more | 02:11 |
anteaya | happy wandering | 02:12 |
anteaya | may you run into beer | 02:12 |
morganfainberg | oh, plenty of options for that here in pasadena (ca) | 02:12 |
clarkb | morganfainberg: you know it is sunday right? | 02:12 |
clarkb | morganfainberg: and football is back | 02:12 |
morganfainberg | clarkb, and i have a 6hr meeting tomorrow (potentially) and a fix i want to get rolling for keystone for RC1 | 02:12 |
morganfainberg | clarkb, i can watch football on my second monitor if i want :P | 02:13 |
anteaya | woohoo 6hr meeting | 02:13 |
morganfainberg | clarkb, and beer, i am doing this with beer! :P | 02:13 |
anteaya | yay lots to look forward to | 02:14 |
morganfainberg | anteaya, maybe i get to write docs too tomorrow. which reminds me… i need to figure out how to best document configuration of caching in keystone. | 02:14 |
*** sarob has joined #openstack-infra | 02:15 | |
morganfainberg | anyways. good luck with the docpocalypse! | 02:15 |
anteaya | I am confident that after this week's doc boot camp there will be many more folks with knowledge to share about docs | 02:16 |
morganfainberg | yay! | 02:16 |
anteaya | yay! | 02:16 |
morganfainberg | more people who get docs are good. | 02:16 |
*** dguitarbite has joined #openstack-infra | 02:16 | |
anteaya | yes | 02:17 |
anteaya | 'cause I don't really | 02:17 |
*** fifieldt has quit IRC | 02:19 | |
morganfainberg | anteaya, i am better at writing code than doc writing | 02:19 |
morganfainberg | (at least that what i keep telling myself) | 02:19 |
anteaya | ahhh, you beat me at both | 02:19 |
morganfainberg | anteaya, maybe. | 02:19 |
anteaya | I can't claim ability in any area | 02:19 |
morganfainberg | anteaya, I read your blog post… you're probably better at docs ;) | 02:19 |
anteaya | last 2 weeks I just felt like I was spinning in circles | 02:19 |
anteaya | ha ha ha | 02:20 |
anteaya | I'm better at talking | 02:20 |
*** xchu has joined #openstack-infra | 02:20 | |
morganfainberg | anteaya, gotta have those people too | 02:20 |
anteaya | oh good | 02:20 |
anteaya | so I can stay? | 02:20 |
anteaya | at least until tomorrow? | 02:21 |
morganfainberg | anteaya, lol | 02:21 |
anteaya | :D | 02:21 |
*** dims has quit IRC | 02:29 | |
*** zhiyan has quit IRC | 02:30 | |
annegentle_ | I like it! Docpocalypse! | 02:35 |
clarkb | I am really beginning to question if this game is worth watching | 02:39 |
*** fifieldt has joined #openstack-infra | 02:43 | |
*** UtahDave has joined #openstack-infra | 02:43 | |
* anteaya goes to tv to find nyg dallas game | 02:44 | |
*** dkliban has joined #openstack-infra | 02:44 | |
* anteaya admires a nice running play | 02:46 | |
*** michchap has quit IRC | 02:48 | |
*** michchap has joined #openstack-infra | 02:49 | |
anteaya | time for me to sign off | 02:49 |
anteaya | have a good <appropriate time of day> everyone | 02:49 |
annegentle_ | clarkb: jeblair: so is it a good idea to merge in the changes to the jenkins jobs, see if we get a +1 on the openstack-manuals and operations-guide patches of interest, and declare victory if it gets a +1? Game plan? | 02:50 |
fifieldt | thanks anteaya | 02:52 |
clarkb | annegentle_: sure why not | 02:53 |
annegentle_ | clarkb: because, docpocalypse? I dunno :) | 02:53 |
* annegentle_ bites nails | 02:53 | |
annegentle_ | not really | 02:53 |
annegentle_ | lets' try it! | 02:53 |
clarkb | ok I will approve 45490 now | 02:53 |
clarkb | done, now we have to wait ~20 minutes | 02:54 |
annegentle_ | clarkb: okie doke. Nice. | 02:54 |
*** anteaya has quit IRC | 02:54 | |
openstackgerrit | A change was merged to openstack-infra/config: Update manual publishing to new directory layout https://review.openstack.org/45490 | 02:55 |
*** dkliban has quit IRC | 03:02 | |
*** sarob has quit IRC | 03:08 | |
*** sarob has joined #openstack-infra | 03:08 | |
*** sarob has quit IRC | 03:13 | |
clarkb | annegentle_: recheck time? | 03:18 |
annegentle_ | clarkb: 53 to 18... yeah | 03:19 |
annegentle_ | clarkb: oh I did recheck no bug on a patch with a linked bug, is that ok? | 03:20 |
*** nati_ueno has quit IRC | 03:20 | |
*** nati_ueno has joined #openstack-infra | 03:21 | |
*** dguitarbite has quit IRC | 03:21 | |
fifieldt | fail :D | 03:21 |
clarkb | annegentle_: yes | 03:21 |
clarkb | annegentle_: the bug number in recheck is the reason for the recheck, not the bug you are fixing | 03:21 |
fifieldt | : A type incompatibility occured .. hmm | 03:21 |
clarkb | annegentle_: eg you should typically only need to recheck if the test env blew up or ran into a flaky test. THis particular case is why we have recheck no bug | 03:22 |
clarkb | annegentle_: one test failed on test was successful | 03:22 |
fifieldt | oh, it's the fop error | 03:22 |
clarkb | is that a known error? anything I can do to help? | 03:24 |
clarkb | the success of the other jobs indicates all is not broken | 03:24 |
fifieldt | it is clarkb, we think it's load related | 03:25 |
*** nati_ueno has quit IRC | 03:25 | |
clarkb | like slave load related? I don't expect those slaves to be heavily loaded when running maven | 03:25 |
fifieldt | it was a theory, since it started happening more after we parallelised mvn jobs | 03:27 |
fifieldt | but actually, this error is from ops guide repo | 03:27 |
fifieldt | which has just one job | 03:27 |
fifieldt | so, not sure | 03:27 |
fifieldt | https://bugs.launchpad.net/openstack-manuals/+bug/1221721 fyi | 03:27 |
uvirtbot | Launchpad bug 1221721 in openstack-manuals "FontTriplet errors when building on Jenkins when heavy load" [Undecided,Incomplete] | 03:27 |
clarkb | if builds are parallized could it be an inter thread conflict? | 03:29 |
clarkb | we run into those fairly often with parallelized tests | 03:29 |
clarkb | would possibly explain why local builds work (race condition that isn't exposed in the local env) | 03:29 |
*** HenryG has quit IRC | 03:30 | |
fifieldt | maybe! | 03:30 |
fifieldt | cool, ops guide checks fine now too annegentle_ | 03:30 |
annegentle_ | clarkb: inter thread conflict. Huh. I never knew. | 03:31 |
fifieldt | though, the ops guide has only one job ... right? | 03:31 |
annegentle_ | fifieldt: right | 03:32 |
fifieldt | we should just rewrite mvn in python :) | 03:33 |
fifieldt | that'll fix it | 03:33 |
clarkb | fifieldt: you don't have to, just use sphinx problem solved :) | 03:33 |
fifieldt | heh :) | 03:33 |
* fifieldt is only pretending to be a docs person anyway | 03:34 | |
annegentle_ | bwah | 03:34 |
clarkb | also with sphinx you get rst which you can edit in a real editor without tearing your eyes out | 03:34 |
fifieldt | I edit in vim clarkb :) | 03:35 |
fifieldt | like I said, pretending :D | 03:35 |
fifieldt | so what's next | 03:35 |
fifieldt | annegentle_, are you approving those flattenpatches? | 03:35 |
annegentle_ | fifieldt: clark so if the recheck is good I approve, right? | 03:35 |
clarkb | I say go for it, git makes reverting easy if you ahve to | 03:36 |
annegentle_ | Very happy to see gate-openstack-manuals-operations-guide SUCCESS in 2m 16s | 03:36 |
fifieldt | looks like dallas has this | 03:37 |
* fifieldt commences the prayer-to-zuul rituals | 03:38 | |
annegentle_ | heh | 03:38 |
fifieldt | fail | 03:41 |
clarkb | are we approving https://review.openstack.org/#/c/45517/ ? | 03:41 |
fifieldt | just fop again | 03:41 |
clarkb | fop affected a different change | 03:42 |
*** nicedice_ has joined #openstack-infra | 03:42 | |
fifieldt | sorry for the ambiguiity clarkb, I was referring to the gate fail on the one that was previously foping | 03:43 |
fifieldt | I've given it another kick | 03:43 |
fifieldt | 45517 is good to go as far as I'm aware | 03:43 |
fifieldt | ops guide has merged | 03:44 |
fifieldt | annegentle_, are you doing manuals? | 03:44 |
annegentle_ | fonttriplet strikes again | 03:44 |
*** UtahDave has quit IRC | 03:46 | |
annegentle_ | fifieldt: approved it again | 03:47 |
fifieldt | they both merged | 03:49 |
fifieldt | well done :) | 03:49 |
fifieldt | time for the website patch now? | 03:50 |
fifieldt | and the validate.py patch? | 03:50 |
fifieldt | rebasing 45543 | 03:52 |
annegentle_ | fifieldt: yep and yes | 03:52 |
*** dhellmann has quit IRC | 04:01 | |
*** amotoki has joined #openstack-infra | 04:04 | |
annegentle_ | clarkb: jeblair: ok so how do we get a patch to our gate through if the gate tests weren't merged first? https://review.openstack.org/#/c/45597/ | 04:10 |
clarkb | annegentle_: the tests were put in place first | 04:13 |
annegentle_ | clarkb: but validate.py is one of the tests? | 04:13 |
fifieldt | hmm | 04:14 |
* fifieldt looks at logs again | 04:14 | |
*** mtreinish has quit IRC | 04:14 | |
fifieldt | Embedded error: /home/jenkins/workspace/gate-openstack-manuals-validate/doc/common/glossary not found. | 04:14 |
fifieldt | maybe not a problem with merge order, but specific to glossary ...? | 04:15 |
annegentle_ | fifieldt: ooo | 04:15 |
annegentle_ | fifieldt: yeah glossary is now in doc/glossary not common/glossary | 04:15 |
annegentle_ | fifieldt: I mean doc/src/docbkx/common/glossary | 04:15 |
*** mtreinish has joined #openstack-infra | 04:15 | |
fifieldt | so it's a gate error then | 04:16 |
fifieldt | oh | 04:16 |
fifieldt | or | 04:16 |
fifieldt | just an include error | 04:16 |
fifieldt | "Build of book security-guide failed " ... glossary | 04:16 |
annegentle_ | fifieldt: ah yep | 04:17 |
annegentle_ | fifieldt: I can patch that | 04:17 |
*** SergeyLukjanov has joined #openstack-infra | 04:17 | |
fifieldt | by 'gate error' I meant 'job error' | 04:17 |
fifieldt | cool, thanks annegentle_ | 04:17 |
*** sarob has joined #openstack-infra | 04:19 | |
*** dkliban has joined #openstack-infra | 04:23 | |
*** mtreinish has quit IRC | 04:24 | |
*** sarob has quit IRC | 04:25 | |
*** mtreinish has joined #openstack-infra | 04:26 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 04:31 | |
*** SergeyLukjanov_ has quit IRC | 04:33 | |
*** dkliban has quit IRC | 04:40 | |
*** SergeyLukjanov has joined #openstack-infra | 04:41 | |
*** fifieldt has quit IRC | 04:45 | |
*** matsuhas_ has joined #openstack-infra | 05:00 | |
*** matsuhashi has quit IRC | 05:00 | |
*** SergeyLukjanov has quit IRC | 05:04 | |
*** nicedice_ has quit IRC | 05:11 | |
*** matsuhas_ has quit IRC | 05:26 | |
*** matsuhashi has joined #openstack-infra | 05:29 | |
*** nosnos has quit IRC | 05:32 | |
*** nosnos has joined #openstack-infra | 05:32 | |
*** matsuhashi has quit IRC | 05:34 | |
*** dkliban has joined #openstack-infra | 05:35 | |
*** boris-42 has joined #openstack-infra | 05:39 | |
*** matsuhashi has joined #openstack-infra | 05:39 | |
*** sergmelikyan has joined #openstack-infra | 05:41 | |
sergmelikyan | Can anyone help me with pypi jobs issue? | 05:41 |
*** dkliban has quit IRC | 05:41 | |
clarkb | sure, whats up? | 05:42 |
*** salv-orlando_ has joined #openstack-infra | 05:48 | |
*** salv-orlando has quit IRC | 05:52 | |
*** salv-orlando_ is now known as salv-orlando | 05:52 | |
sergmelikyan | This job https://jenkins.openstack.org/job/python-muranoclient-pypi-upload/2/console is successfull | 06:00 |
sergmelikyan | but no package on pypi | 06:00 |
sergmelikyan | clarkb, i could not find reason why it is happened. Any thoughts? | 06:01 |
*** sergmelikyan has quit IRC | 06:01 | |
*** SergeyLukjanov has joined #openstack-infra | 06:03 | |
clarkb | there doesn't appear to be a python-muranoclient registered on pypi | 06:04 |
clarkb | the pypi job can't publish to an unregistered package | 06:04 |
*** SergeyLukjanov has quit IRC | 06:06 | |
*** sergmelikyan has joined #openstack-infra | 06:07 | |
clarkb | sergmelikyan: there isnt a python-muranoclient registered on pypi | 06:07 |
clarkb | sergmelikyan: you will need to register the project and give openstackci package ownership for the pypi jobs to publish on pypi | 06:09 |
sergmelikyan | Thanks, i did not know that! | 06:10 |
sergmelikyan | clarkb, is maintainer right enough? | 06:23 |
sergmelikyan | Or exactly owner right should be assigned to openstackci? | 06:24 |
clarkb | maybe I was looking at https://pypi.python.org/pypi/gerritlib/0.2.5 which has both | 06:24 |
clarkb | heatclient has both too https://pypi.python.org/pypi/python-heatclient | 06:25 |
clarkb | it is possible maintainer is sufficient | 06:26 |
sergmelikyan | but we could not check this without publishing new release, i guess. I also couldn't find a package with only maintainer right for openstackci | 06:27 |
sergmelikyan | Thx, for your help! | 06:31 |
*** thomasbiege has joined #openstack-infra | 06:32 | |
*** thomasbiege has quit IRC | 06:35 | |
*** jeremyb_ has quit IRC | 06:53 | |
*** jeremyb_ has joined #openstack-infra | 06:53 | |
*** jeremyb_ is now known as jeremyb | 06:54 | |
*** odyssey4me has joined #openstack-infra | 06:57 | |
*** matsuhashi has quit IRC | 06:57 | |
*** xchu has quit IRC | 06:58 | |
*** nosnos_ has joined #openstack-infra | 07:00 | |
*** nosnos has quit IRC | 07:01 | |
*** matsuhashi has joined #openstack-infra | 07:01 | |
*** xchu has joined #openstack-infra | 07:11 | |
*** boris-42 has quit IRC | 07:11 | |
*** flaper87|afk is now known as flaper87 | 07:13 | |
*** SergeyLukjanov has joined #openstack-infra | 07:16 | |
*** Bada has joined #openstack-infra | 07:19 | |
*** dizquierdo has joined #openstack-infra | 07:29 | |
*** Bada has quit IRC | 07:39 | |
*** michchap has quit IRC | 07:40 | |
*** shardy is now known as shardy_afk | 07:41 | |
*** dkehn_ has joined #openstack-infra | 07:46 | |
*** dkehn has quit IRC | 07:46 | |
*** Bada has joined #openstack-infra | 07:50 | |
*** SergeyLukjanov has quit IRC | 07:57 | |
*** Bada has quit IRC | 07:59 | |
*** SergeyLukjanov has joined #openstack-infra | 08:01 | |
*** dizquierdo has quit IRC | 08:03 | |
*** SergeyLukjanov has quit IRC | 08:12 | |
*** SergeyLukjanov has joined #openstack-infra | 08:17 | |
*** dripton__ has joined #openstack-infra | 08:18 | |
*** derekh has joined #openstack-infra | 08:19 | |
*** dripton_ has quit IRC | 08:19 | |
*** mkerrin has joined #openstack-infra | 08:20 | |
*** boris-42 has joined #openstack-infra | 08:31 | |
*** Bada has joined #openstack-infra | 08:37 | |
*** afazekas has joined #openstack-infra | 08:40 | |
*** jpich has joined #openstack-infra | 08:40 | |
*** SergeyLukjanov has quit IRC | 08:53 | |
*** dizquierdo has joined #openstack-infra | 08:57 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Remove extra cloud-admin in URL of End User Guide https://review.openstack.org/45640 | 08:59 |
*** dkehn_ has quit IRC | 09:03 | |
openstackgerrit | Mate Lakat proposed a change to openstack-dev/hacking: hacking should conform with OS requirements https://review.openstack.org/45641 | 09:04 |
*** xchu has quit IRC | 09:14 | |
*** Ryan_Lane has quit IRC | 09:15 | |
*** dkehn has joined #openstack-infra | 09:18 | |
*** matsuhashi has quit IRC | 09:26 | |
*** matsuhashi has joined #openstack-infra | 09:26 | |
*** xchu has joined #openstack-infra | 09:27 | |
*** SergeyLukjanov has joined #openstack-infra | 09:31 | |
*** pblaho has joined #openstack-infra | 09:33 | |
*** dizquierdo has quit IRC | 09:38 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 09:39 | |
*** xchu has quit IRC | 09:41 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 09:41 | |
*** dizquierdo has joined #openstack-infra | 09:42 | |
*** Ryan_Lane has joined #openstack-infra | 09:46 | |
*** pblaho has quit IRC | 09:47 | |
*** Ryan_Lane has quit IRC | 09:54 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 09:59 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 10:00 | |
*** alexpilotti has joined #openstack-infra | 10:02 | |
*** Ryan_Lane has joined #openstack-infra | 10:21 | |
*** Ryan_Lane has quit IRC | 10:26 | |
*** pcm_ has joined #openstack-infra | 10:28 | |
*** pcm_ has joined #openstack-infra | 10:29 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 10:29 | |
*** matsuhashi has quit IRC | 10:30 | |
*** SergeyLukjanov_ has quit IRC | 10:31 | |
*** nosnos_ has quit IRC | 10:39 | |
*** lcestari has joined #openstack-infra | 10:42 | |
*** ruhe has joined #openstack-infra | 10:42 | |
*** HenryG has joined #openstack-infra | 10:51 | |
*** Ryan_Lane has joined #openstack-infra | 10:51 | |
*** HenryG_ has joined #openstack-infra | 10:52 | |
*** HenryG_ has quit IRC | 10:54 | |
*** HenryG has quit IRC | 10:55 | |
*** Ryan_Lane has quit IRC | 10:56 | |
*** HenryG has joined #openstack-infra | 10:58 | |
*** ruhe has quit IRC | 10:59 | |
*** yaguang has quit IRC | 11:00 | |
*** afazekas has quit IRC | 11:01 | |
*** SergeyLukjanov has joined #openstack-infra | 11:01 | |
*** michchap has joined #openstack-infra | 11:01 | |
*** michchap has quit IRC | 11:06 | |
*** salv-orlando has quit IRC | 11:08 | |
*** DennyZhang has joined #openstack-infra | 11:11 | |
*** plomakin_ has quit IRC | 11:15 | |
*** cthulhup has joined #openstack-infra | 11:16 | |
*** Ryan_Lane has joined #openstack-infra | 11:22 | |
*** plomakin has joined #openstack-infra | 11:25 | |
*** Ryan_Lane has quit IRC | 11:26 | |
*** afazekas has joined #openstack-infra | 11:27 | |
*** afazekas_ has joined #openstack-infra | 11:27 | |
*** DennyZhang has quit IRC | 11:30 | |
*** ruhe has joined #openstack-infra | 11:30 | |
*** DennyZhang has joined #openstack-infra | 11:31 | |
*** afazekas has quit IRC | 11:32 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 11:32 | |
*** SergeyLukjanov_ has quit IRC | 11:41 | |
*** DennyZhang has quit IRC | 11:47 | |
*** pblaho has joined #openstack-infra | 11:47 | |
*** DennyZhang has joined #openstack-infra | 11:48 | |
*** dims has joined #openstack-infra | 11:48 | |
*** anteaya has joined #openstack-infra | 11:53 | |
*** DennyZhang has quit IRC | 11:58 | |
*** ruhe has quit IRC | 11:59 | |
*** cthulhup has quit IRC | 12:00 | |
*** DennyZhang has joined #openstack-infra | 12:00 | |
*** SergeyLukjanov has joined #openstack-infra | 12:02 | |
*** ArxCruz has joined #openstack-infra | 12:03 | |
*** dhellmann has joined #openstack-infra | 12:06 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 12:06 | |
dhellmann | here's a fun new requirements management issue to think about when the rest of you come online: https://review.openstack.org/#/c/38274/ | 12:08 |
anteaya | morning dhellmann | 12:08 |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 12:08 | |
*** weshay has joined #openstack-infra | 12:08 | |
dhellmann | good morning, anteaya, you're online early (aren't you in western CA?) | 12:09 |
anteaya | Eastern Time Zone (Canada) | 12:09 |
dhellmann | ah, my mistake | 12:09 |
anteaya | so not so early | 12:09 |
*** dims has quit IRC | 12:09 | |
anteaya | that's okay | 12:09 |
*** rfolco has joined #openstack-infra | 12:09 | |
*** dims has joined #openstack-infra | 12:09 | |
dhellmann | same time as me :-) | 12:09 |
anteaya | :D | 12:09 |
anteaya | I just haven't been as early as you much before | 12:10 |
anteaya | so far I haven't been caught in the vortex of requirements issues | 12:10 |
anteaya | so I don't have much to offer | 12:10 |
anteaya | except an ear | 12:10 |
dhellmann | this is early for me, I found that issue while reviewing email over breakfast and wanted to make sure everyone was aware | 12:11 |
anteaya | good idea | 12:12 |
dhellmann | it seems like the solution is to allow different python 3 requirements via a second global list, but that may pose challenges in the test running jobs that I'm not aware of | 12:12 |
* anteaya nods | 12:12 | |
anteaya | I am finding this is the only time of day I can get a sense of what is going on | 12:12 |
anteaya | by the time everyone else arrives I feel like I am running to catch a train | 12:13 |
anteaya | which I never end up catching | 12:13 |
*** weshay has quit IRC | 12:14 | |
dhellmann | I know the feeling | 12:14 |
dhellmann | breakfast is ready, bbiab | 12:15 |
anteaya | happy breakfast | 12:16 |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 12:19 | |
mordred | dhellmann: morning | 12:19 |
anteaya | hey mordred | 12:20 |
anteaya | how was the show? | 12:20 |
mordred | anteaya: it's coming along nicely | 12:20 |
anteaya | dhellmann: you also suggested documenting the need for cliff 1.4.3 for python26 users, that seems like the fastest/easiest to me | 12:21 |
mordred | dhellmann: why is pyparsing broken? :) | 12:21 |
anteaya | mordred: yay | 12:21 |
*** derekh is now known as derekh_afk | 12:21 | |
*** thomasm has joined #openstack-infra | 12:22 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 12:24 | |
*** weshay has joined #openstack-infra | 12:29 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 12:35 | |
*** SergeyLukjanov_ has quit IRC | 12:43 | |
ttx | mordred: see my comment in https://review.openstack.org/#/c/44489/ -- would "resolutions" be a better directory name ? | 12:44 |
*** flaper87 has quit IRC | 12:44 | |
*** flaper87 has joined #openstack-infra | 12:44 | |
anteaya | afternoon ttx | 12:45 |
mordred | ttx: good point. I like resolutions and positions both | 12:45 |
ttx | mordred: other topic, how shall we handle the deletion of h3 milestone-proposed branches and their potential merging back to master ? ISTR you have a script or something ? | 12:45 |
ttx | mordred: ok, let's try resolutions. | 12:45 |
ttx | will update the patch | 12:46 |
mordred | ttx: https://review.openstack.org/#/c/41927/ <-- could use review by clarkb, fungi or jeblair when then get in | 12:46 |
*** ruhe has joined #openstack-infra | 12:46 | |
dhellmann | mordred: pyparsing made the choice to go python 3 only for the 2.x release series | 12:46 |
ttx | mordred: that would be a manually-triggered job ? | 12:47 |
mordred | dhellmann: so it's not possible to just say "pip install pyparsing" and get the right thing? | 12:47 |
mordred | ttx: it would trigger on tags | 12:47 |
dhellmann | anteaya: documenting the requirement means we don't have to change the version of cliff used by the client, but it doesn't fix the problem that our 3.3 gate job can't install the dependencies from our mirror :-( | 12:47 |
mordred | dhellmann: yuck. that just makes me angry | 12:47 |
dhellmann | mordred: no, you have to specify the version based on your python version | 12:47 |
mordred | that's so broken I can't even begin to formulate words to describe it | 12:48 |
* dhellmann misses static linking and a.out | 12:48 | |
* mordred misses engineers caring about software engineering | 12:48 | |
ttx | mordred: ah, ok. Thinking about it for a bit. I ussually delayed branch-deletion-after-tagging to be able to catch an hypothetic weird fail but I'm not sure there is actually any value in waiting | 12:48 |
dhellmann | mordred: checking the python version is not yet a feature of the metadata for installing packages (although it is mentioned in the relevant pep) | 12:48 |
mordred | ttx: you can delay the branch deletion just fine | 12:48 |
mordred | dhellmann: yeah, I'm just saying, none of the tooling supports python version metadata, so maybe do what the rest of us do and make software that works with both | 12:49 |
* dhellmann nods | 12:49 | |
mordred | dhellmann: we can add some special case logic - we're already special casing something else because it | 12:49 |
mordred | because it's a broken turd that doesn't work properly | 12:49 |
mordred | but I'm going to be angry about it | 12:50 |
mordred | dhellmann: how about if we fork pyparsing instead? | 12:50 |
ttx | mordred: OH. You merge back when the tag is done, but that doesn't delete the branch. I see. | 12:50 |
dhellmann | mordred: I could try emailing paul, and see what the back story is first :-) | 12:50 |
mordred | dhellmann: 20 July 2013 - NOTE - Pyparsing 2.0.1 was just released, and supports Python versions 2.6 and later. If you are using Python 2.5 or older, you must specifcally install version 1.5.7. | 12:51 |
dhellmann | ah | 12:51 |
dhellmann | so perhaps that was meant to be a temporary situation | 12:51 |
dhellmann | let me test cliff with 2.0.1 | 12:52 |
mordred | yay! | 12:52 |
ttx | mordred: so... for the H3 case we would still trigger it manually (since the tag already happened) ? | 12:52 |
*** SergeyLukjanov has joined #openstack-infra | 12:52 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 12:52 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 12:52 | |
dhellmann | mordred: we'll need to talk to catherine about a new cmd release, too | 12:52 |
mordred | ttx: yes. and it's easy enough to do | 12:53 |
ttx | mordred: ok, let's wait for the review, no hurry | 12:53 |
*** Ryan_Lane has joined #openstack-infra | 12:53 | |
*** beagles has quit IRC | 12:55 | |
*** jhesketh_ has quit IRC | 12:55 | |
*** Ryan_Lane has quit IRC | 12:58 | |
*** beagles has joined #openstack-infra | 13:01 | |
*** beagles has quit IRC | 13:01 | |
openstackgerrit | A change was merged to openstack-infra/config: Add git.o.o to list of ssl certificates we monitor https://review.openstack.org/45461 | 13:02 |
*** Guest14685 has joined #openstack-infra | 13:02 | |
openstackgerrit | A change was merged to openstack-infra/config: update thirdy party testing how to doc https://review.openstack.org/45495 | 13:02 |
openstackgerrit | A change was merged to openstack-infra/config: Add backup restore docs. https://review.openstack.org/45312 | 13:02 |
*** dprince has joined #openstack-infra | 13:04 | |
openstackgerrit | A change was merged to openstack-infra/nodepool: Add a nodepool command https://review.openstack.org/44553 | 13:09 |
*** dkliban has joined #openstack-infra | 13:11 | |
*** changbl has quit IRC | 13:12 | |
*** rustlebee is now known as russellb | 13:14 | |
*** mriedem has joined #openstack-infra | 13:15 | |
*** Ryan_Lane has joined #openstack-infra | 13:16 | |
*** jpeeler has joined #openstack-infra | 13:17 | |
*** jpeeler has quit IRC | 13:17 | |
*** jpeeler has joined #openstack-infra | 13:17 | |
anteaya | dhellmann: yes having the python3.3 gate jobs able to download the dependencies would be nice | 13:20 |
*** Ryan_Lane has quit IRC | 13:20 | |
*** dripton__ is now known as dripton | 13:30 | |
dhellmann | mordred: https://bitbucket.org/catherinedevlin/cmd2/pull-request/2/use-the-latest-version-of-pyparsing-when/diff | 13:32 |
*** tstevenson has joined #openstack-infra | 13:46 | |
mordred | dhellmann: woot | 13:48 |
dhellmann | mordred: updated at https://bitbucket.org/catherinedevlin/cmd2/pull-request/3/use-the-latest-version-of-pyparsing-when/diff | 13:48 |
mordred | dhellmann: I will be quite pleased when that lands | 13:49 |
openstackgerrit | A change was merged to openstack/requirements: Add falcon to global-requirements https://review.openstack.org/45070 | 13:54 |
*** thedodd has joined #openstack-infra | 13:54 | |
*** afazekas_ is now known as afazekas | 13:55 | |
*** marun has quit IRC | 13:56 | |
*** alaski-vacation is now known as alaski | 13:57 | |
*** prad_ has joined #openstack-infra | 13:58 | |
*** dkranz has quit IRC | 14:00 | |
*** burt has joined #openstack-infra | 14:04 | |
*** mrmartin has joined #openstack-infra | 14:06 | |
*** rnirmal has joined #openstack-infra | 14:13 | |
*** KennethWilke has joined #openstack-infra | 14:15 | |
*** changbl has joined #openstack-infra | 14:18 | |
*** pentameter has joined #openstack-infra | 14:19 | |
*** Guest14685 is now known as beagles | 14:20 | |
*** beagles is now known as b3nt_pin | 14:20 | |
*** b3nt_pin is now known as beagles | 14:20 | |
*** blamar has joined #openstack-infra | 14:20 | |
*** adalbas has quit IRC | 14:22 | |
*** salv-orlando has joined #openstack-infra | 14:29 | |
*** dklyle is now known as david-lyle | 14:37 | |
*** adalbas has joined #openstack-infra | 14:38 | |
*** salv-orlando has quit IRC | 14:40 | |
*** SergeyLu_ has joined #openstack-infra | 14:42 | |
*** ruhe has quit IRC | 14:43 | |
*** DennyZhang has quit IRC | 14:44 | |
*** SergeyLu_ has quit IRC | 14:46 | |
Shrews | fungi, olaph: you guys going to the OS meetup @ redhat hq tomorrow? | 14:46 |
fungi | Shrews: i was planning to | 14:46 |
fungi | not so much interested in the rdo demo they have planned, but i at least want to be around to provide some community support | 14:47 |
anteaya | zhiyan: congratulations getting your patch to pass check tests: https://review.openstack.org/#/c/45171/2 | 14:50 |
*** salv-orlando has joined #openstack-infra | 14:51 | |
*** mrodden has quit IRC | 14:53 | |
*** ruhe has joined #openstack-infra | 14:54 | |
*** nati_ueno has joined #openstack-infra | 14:55 | |
*** nati_ueno has quit IRC | 14:57 | |
*** nati_ueno has joined #openstack-infra | 14:58 | |
mordred | fungi: I can has +2? https://review.openstack.org/#/c/42530/ | 14:59 |
*** mrmartin has quit IRC | 15:01 | |
*** salv-orlando has quit IRC | 15:02 | |
fungi | mordred: lgtm | 15:02 |
mordred | fungi: thanks | 15:02 |
*** krtaylor has quit IRC | 15:02 | |
*** nati_ueno has quit IRC | 15:02 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 15:04 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 15:05 | |
*** mrodden has joined #openstack-infra | 15:05 | |
*** wenlock has joined #openstack-infra | 15:12 | |
*** erfanian has quit IRC | 15:13 | |
*** UtahDave has joined #openstack-infra | 15:14 | |
*** dkranz has joined #openstack-infra | 15:16 | |
mordred | fungi: it's so quiet in here this morning | 15:17 |
mordred | fungi: don't we normally need to freak out about something on a monday morning? | 15:17 |
*** afazekas has quit IRC | 15:17 | |
fungi | mordred: i have to assume everyone is still hung over from the post-h3 revelry | 15:18 |
mordred | fungi: good call | 15:18 |
fungi | and so nobody's noticed it all broke | 15:18 |
fungi | i'm taking a bit of time to try and pound my current e-mail dogpile into something semblance of autofiltered organization in hopes it will improve my responsiveness to it. been a while since i've properly updated all my filters | 15:20 |
anteaya | bet the cronspam category is huge | 15:21 |
mordred | fungi: that's a good idea | 15:24 |
fungi | anteaya: it is pretty crazy, yes. i'm coming up with some exim regular expressions to catch most of that so i can filter cronish automated stuff into one maildir. also wrangling bug e-mail is taking a bit of work since i'm on the bug team for every single project and want to be able to separate out private/security bugs from the rest of the noise | 15:26 |
SpamapS | lol | 15:27 |
SpamapS | exim for mail filtering | 15:27 |
*** dkehn has quit IRC | 15:27 | |
SpamapS | you guys are true heroes | 15:27 |
anteaya | fungi sounds like you have a fun time planned | 15:27 |
SpamapS | fungi: they have this procmail thing... | 15:27 |
anteaya | hopefully it stays quiet enough you can make some progress | 15:27 |
anteaya | filering out the security bugs from the rest will be very interesting | 15:28 |
fungi | SpamapS: i use the exim .forward file filter. the sieve language is much nicer and more maintainable than traditional procmail | 15:28 |
*** pabelanger_ has joined #openstack-infra | 15:29 | |
SpamapS | fungi: oohhh that exim filtering | 15:29 |
*** Bada has quit IRC | 15:30 | |
*** pabelanger_ has quit IRC | 15:30 | |
*** pabelanger_ has joined #openstack-infra | 15:30 | |
SpamapS | I actually refuse to do specific server side filtering... I tend to unsubscribe from things worse than about 10:1 noise:signal | 15:30 |
*** pabelanger has quit IRC | 15:30 | |
SpamapS | Though I do shove all mailing lists into a lists folder, and all bugs into a bugs folder. :) | 15:30 |
*** pabelanger has joined #openstack-infra | 15:31 | |
*** pabelanger has quit IRC | 15:31 | |
*** pabelanger_ is now known as pabelanger | 15:31 | |
SpamapS | just so that my inbox is usable for immediate communication. | 15:31 |
*** pabelanger_ has joined #openstack-infra | 15:31 | |
*** gyee has joined #openstack-infra | 15:32 | |
*** dkehn has joined #openstack-infra | 15:33 | |
*** pabelanger has quit IRC | 15:35 | |
fungi | yeah, that's the real issue. i've added lots of things i need to receive, but didn't get around to shuffling them out of my inbox so they're drowning out more infrequent/important messages | 15:35 |
*** pabelanger has joined #openstack-infra | 15:36 | |
*** dkehn has quit IRC | 15:38 | |
*** sdake has joined #openstack-infra | 15:38 | |
*** jergerber has joined #openstack-infra | 15:41 | |
*** dkehn has joined #openstack-infra | 15:42 | |
*** boris-42 has quit IRC | 15:43 | |
*** mkerrin has quit IRC | 15:47 | |
*** zehicle has quit IRC | 15:47 | |
pleia2 | good morning | 15:53 |
anteaya | morning pleia2 | 15:55 |
openstackgerrit | A change was merged to openstack/requirements: Add TripleO for-heat modules. https://review.openstack.org/45564 | 15:58 |
mordred | morning pleia2 | 15:59 |
*** ruhe has quit IRC | 16:01 | |
*** reed has joined #openstack-infra | 16:04 | |
mordred | dhellmann: wow. pycon cfp wants a description, a detailed abstract and an OUTLINE | 16:05 |
mordred | dhellmann: it's september. the conference isn't until april | 16:05 |
mordred | dhellmann: I promise I have never had an outline that early | 16:05 |
*** nicedice has joined #openstack-infra | 16:13 | |
*** pabelanger has quit IRC | 16:13 | |
*** svarnau has joined #openstack-infra | 16:14 | |
mordred | dstufft, Alex_Gaynor: I've just submitted a pbr talk for pycon - if it gets accepted, please be sure to come heckle | 16:15 |
Alex_Gaynor | mordred: A+, awesome! | 16:15 |
dstufft | assuming i'm able to go :[ | 16:15 |
mordred | dstufft: do you want me to call chris and yell at him until he lets you? | 16:15 |
*** pabelanger has joined #openstack-infra | 16:16 | |
dstufft | mordred: heh, Nebula wouldn't be the problem. My daughter has a chronic condition so leaving the country is kinda hard | 16:19 |
mordred | dstufft: ah. that's fair - and eek | 16:19 |
*** jpich has quit IRC | 16:23 | |
*** ruhe has joined #openstack-infra | 16:23 | |
*** ruhe has quit IRC | 16:24 | |
*** ruhe has joined #openstack-infra | 16:27 | |
clarkb | morning | 16:28 |
anteaya | morning clarkb | 16:29 |
anteaya | I saw what you meant about the game | 16:29 |
clarkb | scrollback isn't insane today, I like this | 16:29 |
anteaya | :D | 16:29 |
*** flashgordon is now known as jog0 | 16:30 | |
openstackgerrit | Dan Prince proposed a change to openstack-infra/reviewday: Add support for latest SS result formats. https://review.openstack.org/45709 | 16:30 |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 16:32 | |
openstackgerrit | Tobias Stevenson proposed a change to openstack-infra/gerritbot: Fix comment event reporting in Gerritbot https://review.openstack.org/44913 | 16:32 |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 16:32 | |
mordred | morning clarkb | 16:32 |
mordred | clarkb: it's really quiet in here this morning | 16:33 |
mordred | clarkb: https://review.openstack.org/#/c/41927/ and https://review.openstack.org/#/c/42530/ are simple and could use some love | 16:33 |
clarkb | mordred: ok | 16:33 |
openstackgerrit | Dan Prince proposed a change to openstack-infra/reviewday: Add support for latest SS result formats. https://review.openstack.org/45709 | 16:33 |
clarkb | I have resolved to catch up on reviews this morning, then hopefully finish up backups stuff this afternoon | 16:33 |
openstackgerrit | Tobias Stevenson proposed a change to openstack-infra/gerritbot: Fix comment event reporting in Gerritbot https://review.openstack.org/44913 | 16:35 |
openstackgerrit | A change was merged to openstack-infra/config: Fix missing requirements list https://review.openstack.org/45283 | 16:35 |
mordred | clarkb: w00t! | 16:35 |
mordred | clarkb: I love it when you catch up on reviews - except, of course, when I have to fix things :) | 16:36 |
clarkb | mordred: speaking of manage_projects https://review.openstack.org/#/c/44496/3 you +2'd that when I think jeblair was hoping you would take ownership of the change | 16:36 |
*** SergeyLukjanov has quit IRC | 16:36 | |
clarkb | mordred: there are comments from patchset 1 that need addressing. Is that something you are interested in? | 16:36 |
mordred | clarkb: looking | 16:37 |
*** pblaho has quit IRC | 16:37 | |
openstackgerrit | A change was merged to openstack-infra/reviewday: Add support for latest SS result formats. https://review.openstack.org/45709 | 16:38 |
*** nati_ueno has joined #openstack-infra | 16:38 | |
mordred | clarkb: most of patch 1 seem to be addressed - the print->log comments seem to be something that should happen in a different patch | 16:38 |
*** ruhe has quit IRC | 16:39 | |
* jeblair is at doc bootcamp | 16:39 | |
clarkb | jeblair: have fun | 16:40 |
jeblair | annegentle_ very much likes git.o.o | 16:40 |
jeblair | pleia2, fungi, clarkb ^ | 16:40 |
clarkb | mordred: and looking at cookiecutter what is the difference between `python setup.py test` and `python setup.py testr`? Were we switching fomr one to the other? | 16:40 |
fungi | jeblair: annegentle_: yay! | 16:40 |
fungi | so do i! | 16:40 |
clarkb | I like that I can remember what the dir structure is like so I can type out urls by hand | 16:41 |
clarkb | jeblair: is https://review.openstack.org/#/c/41927/ something you want to review before approval happens? it adds the job to merge milestone proposed back into master for the tag | 16:42 |
*** mrmartin has joined #openstack-infra | 16:42 | |
mrmartin | hi | 16:42 |
clarkb | mrmartin: hello | 16:42 |
mrmartin | clarkb: if you have some free minutes, may I ask you to review https://review.openstack.org/#/c/42608/ ? | 16:43 |
jeblair | clarkb: doubt i'll be able to do any real work, so if you're in a rush you may have to go without me | 16:43 |
clarkb | mrmartin: sure, I can review that when I am done with 41927 | 16:43 |
openstackgerrit | Khai Do proposed a change to openstack-infra/config: link git sha in commit comments to the gerrit review https://review.openstack.org/45464 | 16:43 |
clarkb | mordred: ^ | 16:43 |
mrmartin | clarkb: the sitedeploy part was rewritten for drupal deployment, thanks | 16:43 |
*** mgagne has joined #openstack-infra | 16:43 | |
mordred | clarkb: python setup.py test should be identical to python setup.py testr - EXCEPT - it should also work for projects that use nose | 16:44 |
*** dizquierdo has left #openstack-infra | 16:44 | |
mordred | clarkb: so it's a potential consistent way for our scripts to run unittests no mater which engine they're using | 16:45 |
mordred | clarkb: we haven't migrated migrated yet because we haven't exhaustively tested it yet | 16:45 |
mordred | and FF was coming up | 16:45 |
openstackgerrit | A change was merged to openstack-infra/config: Add cookiecutter templates repo https://review.openstack.org/42530 | 16:45 |
mordred | clarkb: it works in simple testing - but you know, late in cycle EEK | 16:45 |
clarkb | mordred: gotcha | 16:45 |
clarkb | mordred: before I -1 https://review.openstack.org/#/c/41927/4 :) if the merge-release-tags job is going in the openstack publish job group should we put that job in the same yaml file? | 16:46 |
clarkb | is there some reason for putting it in a new file? | 16:46 |
pleia2 | annegentle_: yay! glad you like git.o.o :) | 16:46 |
*** pabelanger has quit IRC | 16:47 | |
mordred | clarkb: no reason at all - I just didn't think of it as something that was grouped with other jobs | 16:47 |
*** Bada has joined #openstack-infra | 16:47 | |
mordred | clarkb: since it's pretty specific to _only_ things that have a milestone-proposed | 16:47 |
clarkb | mordred: but you put it in that job group :) | 16:47 |
mordred | I did? | 16:47 |
mordred | dammit | 16:47 |
clarkb | mordred: https://review.openstack.org/#/c/41927/4/modules/openstack_project/files/jenkins_job_builder/config/openstack-publish-jobs.yaml | 16:48 |
clarkb | mordred: I will leave a comment | 16:48 |
mordred | clarkb: yeah. then no, I do not see any reason to not co-locate | 16:48 |
*** ruhe has joined #openstack-infra | 16:48 | |
*** amotoki has quit IRC | 16:48 | |
clarkb | mordred: though I am leaning towards removing it from the group instead | 16:48 |
clarkb | mordred: since as you say it is only for things with a milestone-proposed which is a subset of the things with openstack publish jobs | 16:49 |
mordred | clarkb: I think I'd prefer that | 16:49 |
clarkb | mordred: also the job is not a job template so it doesn't need to be in any group | 16:49 |
jeblair | er | 16:51 |
jeblair | i think it needs to be a template | 16:51 |
clarkb | jeblair: it might need to be | 16:51 |
clarkb | jeblair: if cacheing the git repo is important | 16:51 |
jeblair | other wise if it's used for multiple projects it will share a git repo | 16:51 |
jeblair | and the first time it runs for a different project it will be confused | 16:52 |
anteaya | UtahDave is spinning up some vms for some salt testing | 16:52 |
clarkb | jeblair: updating my review | 16:52 |
anteaya | he wants to know do folks want to be involved in each step or should he just go ahead and ask for directions/modifications once it is sorted? | 16:53 |
jeblair | clarkb, mordred: ok i left a -1 review after all. :) | 16:53 |
anteaya | I'm all for learning myself, but I might slow hime down | 16:53 |
clarkb | anteaya: I would like to be more involved, but don't have much time today | 16:53 |
anteaya | s/hime/him | 16:53 |
anteaya | clarkb: okay | 16:53 |
anteaya | so how about he goes ahead and we will ask questions as we can | 16:54 |
anteaya | that okay with you, UtahDave? | 16:54 |
*** hashar has joined #openstack-infra | 16:56 | |
UtahDave | Sounds great! | 16:56 |
*** hashar is now known as hasharSF | 16:57 | |
UtahDave | I think I have understand the basic workflow desired. I'll just need some info at the very end regarding the exact commands to execute | 16:57 |
*** derekh_afk has quit IRC | 16:57 | |
anteaya | sounds good | 16:57 |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Add job to propose tag merges https://review.openstack.org/41927 | 16:57 |
hasharSF | good morning:) | 16:57 |
mordred | clarkb, jeblair ^^ thanks. great catch | 16:57 |
anteaya | good morning hasharSF | 16:58 |
hasharSF | I am in SF for the whole week :-] | 16:58 |
anteaya | wonderful | 16:59 |
pleia2 | hasharSF: we're having an Ubuntu Hour wednesday evening down at the Roastery if you or others wish to join us: http://loco.ubuntu.com/events/ubuntu-california/2501-ubuntu-hour-san-francisco/ | 17:01 |
*** mrmartin has quit IRC | 17:01 | |
*** odyssey4me has quit IRC | 17:02 | |
jeblair | say hi to the docs bootcamp, everyone! | 17:03 |
pleia2 | hello docs bootcamp! :) | 17:03 |
*** fifieldt has joined #openstack-infra | 17:03 | |
pleia2 | sorry about the weather, it's not usually this hot ;) | 17:04 |
clarkb | ohai | 17:04 |
fifieldt | hi docs boot camp! | 17:04 |
*** boris-42 has joined #openstack-infra | 17:04 | |
hasharSF | pleia2: I am not sure what is my exact agenda is going to be. But will definitely show up at that Ubuntu hour if possible :-] | 17:04 |
clarkb | pleia2: I was watching the football game in SF yesterday and the boat race and the change in weather over such a short distance was pretty drastic | 17:04 |
pleia2 | clarkb: yeah, the fog makes things chilly as you get closer to the ocean | 17:05 |
* pleia2 spent yesterday near the ocean | 17:05 | |
*** openstackgerrit has quit IRC | 17:05 | |
*** openstackgerrit has joined #openstack-infra | 17:05 | |
anteaya | hello docs folks | 17:05 |
*** dcramer has joined #openstack-infra | 17:08 | |
*** KLevenstein_away has joined #openstack-infra | 17:08 | |
*** rev has joined #openstack-infra | 17:08 | |
annegentle_ | hai! You guys crack me up! | 17:08 |
* fungi expresses his deepest respect for anyone who can stomach writing docs all the time | 17:08 | |
*** fandikurnia02 has joined #openstack-infra | 17:09 | |
clarkb | pleia2: were you watching the boats? | 17:10 |
jog0 | clarkb: want to approve https://review.openstack.org/#/c/45501/ now that it's not the weekend | 17:10 |
clarkb | jog0: looking | 17:10 |
openstackgerrit | Dan Prince proposed a change to openstack-infra/reviewday: Add SS results support for ceilo/heat. https://review.openstack.org/45714 | 17:11 |
clarkb | jog0: done | 17:11 |
jog0 | clarkb: thanks | 17:11 |
pleia2 | clarkb: nah, they've been in the bay all summer, getting boring :) (LV cup prior to america's cup) | 17:12 |
openstackgerrit | A change was merged to openstack-infra/config: Move gate-tempest-devstack-vm-large-ops to silent https://review.openstack.org/45501 | 17:13 |
clarkb | mordred: new patchset lgtm | 17:15 |
*** pabelanger has joined #openstack-infra | 17:16 | |
*** fifieldt has quit IRC | 17:19 | |
koolhead17 | mordred: Seems like all infra folks are working with you. Just got to know jog0 is with you as well now | 17:19 |
koolhead17 | :P | 17:19 |
clarkb | precise14 had a stuck puppet, I am kicking it. (noticed this because it was sending email to my gmail account intstead of the new address) | 17:22 |
*** tjones has joined #openstack-infra | 17:23 | |
clarkb | mordred: SpamapS: lifeless: safe to approve https://review.openstack.org/#/c/42513/2 ? it adds new gate jobs to tripleo projects | 17:25 |
anteaya | koolhead17: jeblair and fungi are with the foundation | 17:25 |
koolhead17 | jeblair: ok | 17:26 |
UtahDave | Should the Jenkins server have to ability to decide which servers should execute a puppet run? | 17:27 |
UtahDave | Or should the Jenkins server just notify the Salt Master that builds have passed and the let the Salt Master decide which servers should execute puppet? | 17:27 |
clarkb | that is a good question, I think salt master (or at least something not in Jenkins) should make the determination | 17:28 |
fungi | UtahDave: i think the latter, for the moment at least | 17:29 |
clarkb | Jenkins is doing gating not orchestration | 17:29 |
fungi | agree with clarkb | 17:29 |
*** Ryan_Lane has joined #openstack-infra | 17:29 | |
fungi | the less decision-making we leave in jenkins's hands now, the less we might have to rework later too | 17:29 |
fungi | anteaya: though jeblair and i did also work at hp with mordred before moving to the foundation | 17:29 |
UtahDave | ok, I'll head in that direction, then. Thanks! | 17:30 |
anteaya | yes | 17:30 |
*** dkehn has quit IRC | 17:30 | |
anteaya | that is true | 17:30 |
*** dkehn has joined #openstack-infra | 17:30 | |
*** SergeyLukjanov has joined #openstack-infra | 17:30 | |
openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation https://review.openstack.org/45715 | 17:32 |
*** SergeyLukjanov has quit IRC | 17:33 | |
*** pabelanger has quit IRC | 17:34 | |
*** nijaba has quit IRC | 17:35 | |
openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation https://review.openstack.org/45715 | 17:36 |
*** pabelanger has joined #openstack-infra | 17:38 | |
*** SergeyLukjanov has joined #openstack-infra | 17:39 | |
*** cppcabrera has joined #openstack-infra | 17:40 | |
openstackgerrit | A change was merged to openstack-infra/config: Add requirements gating jobs to tripleo https://review.openstack.org/42513 | 17:41 |
*** sarob has joined #openstack-infra | 17:42 | |
*** cppcabrera has left #openstack-infra | 17:42 | |
openstackgerrit | A change was merged to openstack-infra/config: New project: python-climateclient. https://review.openstack.org/35700 | 17:42 |
*** melwitt has joined #openstack-infra | 17:44 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Job-specific subst. in a job group's job list https://review.openstack.org/43013 | 17:50 |
*** ruhe has quit IRC | 17:52 | |
*** moted has quit IRC | 17:53 | |
*** moted has joined #openstack-infra | 17:53 | |
*** jergerber has quit IRC | 17:56 | |
*** HenryG_ has joined #openstack-infra | 18:01 | |
*** nati_ueno has quit IRC | 18:04 | |
*** nati_ueno has joined #openstack-infra | 18:05 | |
*** HenryG has quit IRC | 18:05 | |
*** Ajaeger1 has joined #openstack-infra | 18:08 | |
*** danger_fo_away is now known as danger_fo | 18:08 | |
*** nati_ueno has quit IRC | 18:09 | |
*** zehicle has joined #openstack-infra | 18:10 | |
* anteaya heads out for a walk | 18:11 | |
*** mriedem has quit IRC | 18:18 | |
clarkb | mgagne: https://review.openstack.org/#/c/44685/ | 18:19 |
mordred | koolhead17: that's right | 18:19 |
mgagne | clarkb: checking | 18:19 |
mordred | clarkb: I may have just eaten way too much taco for lunch | 18:20 |
koolhead17 | mordred: am not complaining sir. :D | 18:20 |
clarkb | mordred: I had a chimichanga on Saturday that I am still recovering from | 18:20 |
*** mriedem has joined #openstack-infra | 18:20 | |
fungi | grease fried in more grease | 18:21 |
mordred | clarkb: that sounds like a good chimichanga then | 18:21 |
mgagne | clarkb: want me to approve once reviewed? | 18:21 |
clarkb | mgagne: yes please | 18:21 |
clarkb | mgagne: assuming you are happy with the current patchset | 18:21 |
mgagne | clarkb: my comments were addressed | 18:21 |
mgagne | clarkb: fun, this one was next in my list of reviews =) | 18:22 |
clarkb | mordred: will you be in town next week? | 18:22 |
clarkb | for the tripleo thing | 18:22 |
*** jhesketh has quit IRC | 18:22 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Added SBT builder support https://review.openstack.org/44685 | 18:22 |
*** jhesketh has joined #openstack-infra | 18:23 | |
*** marun has joined #openstack-infra | 18:26 | |
*** mrmartin has joined #openstack-infra | 18:26 | |
*** changbl has quit IRC | 18:27 | |
*** mrmartin has left #openstack-infra | 18:28 | |
*** mrmartin has joined #openstack-infra | 18:28 | |
mrmartin | clarkb: are u here? | 18:29 |
clarkb | mrmartin: yes | 18:29 |
mrmartin | clarkb: nobody:www-data could be a good owner for drupal files? | 18:29 |
mrmartin | clarkb: I was thinking on this ubuntu user ownership going home from the office today, and I can agree with you | 18:30 |
*** changbl has joined #openstack-infra | 18:30 | |
mrmartin | I see two options: nobody:www-data or I can create a separate web user. What you think? | 18:30 |
clarkb | tstevenson: I think the issue is gerritbot wants to daemonize itself, in main() you can comment out lines 304-306 and de-indent the call to _main() one level to run it in the foreground | 18:30 |
clarkb | mrmartin: nobody:www-data works for me | 18:31 |
*** dkehn_ has joined #openstack-infra | 18:31 | |
clarkb | mrmartin: or www-data:www-data? | 18:31 |
clarkb | fungi: ^ any opinion on drupal file ownership? | 18:31 |
*** pabelanger has quit IRC | 18:32 | |
pleia2 | I tend to only give www-data ownership of files to www-data when absolutely required, it's not required all that often (directories+files that need to be writeable mostly, which are few) | 18:32 |
pleia2 | I'd vote for separate user owning them except where www-data needs to write | 18:32 |
mrmartin | www-data have the group of www-data | 18:32 |
pleia2 | suppose it depends on what permissions the group has | 18:32 |
pleia2 | I seem to recall drupal being pretty good about telling you what it needs write access to | 18:33 |
mrmartin | and I don't advise it because, it can the regular files have 0640, directories 0750, and writeable files/dirs are 0660 and 0770 | 18:33 |
mrmartin | it can / n/a | 18:33 |
fungi | keep in mind that making files owned by "nobody" is no better than making them owned by root | 18:33 |
mrmartin | so, maybe a separate user created just for this ownership, with www-data as a group? | 18:34 |
fungi | arguably worse, since any process running as "nobody" then has write access to those files | 18:34 |
*** dkehn has quit IRC | 18:34 | |
UtahDave | Should the Salt Master cause ALL servers to execute their puppet runs? Or are there a specific subset or class of servers that should be targeted? | 18:34 |
fungi | UtahDave: right now, we don't have any means to differentiate which servers should update based on changes to particular subsets of the project or projects | 18:35 |
mrmartin | ok, so could we agree on a separate user? | 18:35 |
clarkb | mrmartin: separate user works for me as well | 18:35 |
UtahDave | fungi: so for now just tell everyone to do a puppet run? | 18:35 |
fungi | UtahDave: yep | 18:35 |
UtahDave | ok, cool. It won't be hard to narrow that down in the future, if needed. | 18:36 |
fungi | mrmartin: what user account does drupal run as? root, or www-data, or something else entirely? | 18:36 |
fungi | and which drupal files specifically? | 18:36 |
mrmartin | drupal just a standard web site, nothing fancy | 18:36 |
mrmartin | it is php | 18:36 |
clarkb | UtahDave: fungi: puppet can build a graph of stuff for your nodes, we could build that graph then find where graphs intersect files that were changed to determine a minimal set of minions that need to kick puppet | 18:36 |
mrmartin | the only requirement is that sites/default/files must be writeable and settings.php must r/o | 18:37 |
clarkb | UtahDave: fungi: I think that cna be added later :) | 18:37 |
fungi | mrmartin: okay, so files which need to be writable by the php scripts for the site? | 18:37 |
mrmartin | just everything under sites/default/files. they have different permissions 0660 / 0770 for user:www-data | 18:37 |
mrmartin | and 0640 / 0750 for everything else | 18:38 |
fungi | clarkb: UtahDave: agreed. for now tell all servers to check for puppet updates whenever we merge any relevant change. i think that's still far better than the current situation where they run every 10-15 minutes, since our change rate is nowhere near that high | 18:38 |
openstackgerrit | A change was merged to openstack-infra/reviewday: Add SS results support for ceilo/heat. https://review.openstack.org/45714 | 18:39 |
mrmartin | jeblair: may I ask you to review https://review.openstack.org/#/c/42608/ the patch, if you find something I want to put into next revision, and avoid additional rounds. | 18:39 |
fungi | mrmartin: so, for files which only need to be readable by www-data and contain potentially sensitive information which should be hidden from other processes, using root:www-data 0640 is warranted. for writeable files, root:www-data 0660 or www-data:www-data 0640 ought to be fine. for directories which rely on added files being group-readable, make sure to add setgid on them | 18:41 |
fungi | and for non-sensitive files www-data needs to read, root:root 0644 is usually good | 18:42 |
mrmartin | ok, it is easier to set everything else to 0640 instead of 0644 | 18:43 |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/pypi-mirror: Rename jeepyb to pypi-mirror https://review.openstack.org/45726 | 18:44 |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/pypi-mirror: Start building wheels in the mirror https://review.openstack.org/45727 | 18:44 |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/pypi-mirror: Strip out the things we don't need from jeepyb https://review.openstack.org/45728 | 18:44 |
mordred | clarkb: yes. will be here next week | 18:44 |
mordred | or there | 18:44 |
mrmartin | so, instead of creating a separate user, do you suggest to use root? If the others agree with that, i can do it, it is easier from script side. | 18:44 |
*** tjones has quit IRC | 18:46 | |
fungi | mrmartin: sure, the files and directories can be root-owned with group www-data and just vary the group write permission if the goal is to prevent the scripts from writing to some of the files while allowing that for others | 18:47 |
clarkb | I am happy with root too I guess. ubuntu just felt weird and pidgeon holed us into deploying on images with that user | 18:47 |
mrmartin | clarkb: it was my mistake, left the testing user there ;) | 18:47 |
clarkb | lifeless: for your config docs any chance we can break down some of the dependencies there to make it easier to merge individual changes? | 18:47 |
clarkb | lifeless: git review -d makes it easy to deal with | 18:48 |
mordred | clarkb: https://review.openstack.org/#/c/45726/ - weird error | 18:48 |
mordred | clarkb: I _just_ cloned from gerrit | 18:48 |
fungi | the main things to watch out for with root-owned files is don't let processes create arbitrary symlinks owned by root and don't make them setuid exec | 18:49 |
clarkb | mordred: zuul is using a new merge strategy | 18:49 |
clarkb | mordred: possibly related to that? I would check the zuul debug log to see if there is any more verbose info there | 18:50 |
mordred | clarkb: but those should be direct patches on top of the current tip that I just cloned | 18:50 |
mordred | I will look there | 18:50 |
mordred | just saying | 18:50 |
clarkb | mordred: right, I wonder if you are tickling a bug in zuul | 18:51 |
clarkb | mordred: I would expect the debug log to show you a stacktrace or similar if so | 18:51 |
mordred | 2013-09-09 18:44:09,867 ERROR zuul.IndependentPipelineManager: Unable to find change queue for project openstack-infra/pypi-mirror | 18:52 |
*** vipul is now known as vipul-away | 18:52 | |
*** vipul-away is now known as vipul | 18:52 | |
clarkb | mordred: that is one of those errors that isn't really an error. Anything else? | 18:52 |
openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: Add documentation for XDG_CACHE_HOME https://review.openstack.org/45729 | 18:52 |
clarkb | mordred: not having a job in one of the IndependentPipelines causes that error (eg silent) | 18:52 |
mordred | clarkb: well, there does not seem to be reference to pypi-mirror in layout.yaml | 18:53 |
clarkb | mordred: it is there, line 369 | 18:54 |
clarkb | er line 369 in this checkout of config | 18:54 |
mordred | do. was on wrong branch :) | 18:54 |
lifeless | clarkb: I can certainly shuffle all the noncontentious ones to the bottom; there are multiple edits to the same files though, so splitting it all out doesn't make sense. | 18:54 |
lifeless | clarkb: I'm waiting on feedback from jeblair on several patches to move forwards | 18:54 |
clarkb | lifeless: ok | 18:55 |
lifeless | clarkb: (he may have given that overnight, haven't looked yet) | 18:55 |
clarkb | lifeless: I updated one of them that jeblair hadn't gotten to with what I thought jeblair was talking about | 18:55 |
*** gyee has quit IRC | 18:56 | |
mordred | clarkb: no tracebacks | 18:57 |
clarkb | mordred: I wonder if the lazy clone of projects isn't working, is pypi-mirror cloned in zuuls repo dirs? | 18:58 |
*** ericw has joined #openstack-infra | 18:59 | |
*** dkehn_ is now known as dkehn | 19:00 | |
*** mriedem1 has joined #openstack-infra | 19:04 | |
openstackgerrit | Stefano Maffulli proposed a change to openstack-infra/config: adding staging server for activity board https://review.openstack.org/42998 | 19:04 |
reed | clarkb, that commit went in a new branch, not sure why :( | 19:05 |
*** mriedem has quit IRC | 19:07 | |
*** Ajaeger1 has quit IRC | 19:07 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Use NNFI scheduler algorithm https://review.openstack.org/45334 | 19:07 |
*** krtaylor has joined #openstack-infra | 19:08 | |
reed | and failed | 19:09 |
*** tjones has joined #openstack-infra | 19:09 | |
*** nicedice has quit IRC | 19:11 | |
*** nicedice has joined #openstack-infra | 19:11 | |
openstackgerrit | Khai Do proposed a change to openstack-infra/config: link to jjb cache documentation in the handbook . https://review.openstack.org/45738 | 19:13 |
openstackgerrit | A change was merged to openstack-infra/config: Add py26 and py33 tests and mirror uploads. https://review.openstack.org/45323 | 19:13 |
clarkb | fungi: for https://review.openstack.org/#/c/45324/2 there was a different change to split that out into a separate importable python module. I think tha tdied because we want to update the manage_projects yaml | 19:13 |
*** tjones has quit IRC | 19:14 | |
clarkb | fungi: any opinion on how we should handle these sorts of things until the new yaml format happens? | 19:14 |
fungi | clarkb: i punted on the new format, suggesting first that we break yaml parsing out into a separate module and use it in the other scripts, which would later make changing file formats easier to do in stages | 19:16 |
*** ruhe has joined #openstack-infra | 19:16 | |
openstackgerrit | A change was merged to openstack-dev/hacking: Redirect stderr for git rev-parse https://review.openstack.org/45150 | 19:17 |
fungi | basically it was "add yaml parsing module, make scripts use it, add new data to existing yaml file format, remove hard-coded data from scripts" | 19:17 |
fungi | that way if the scripts override the local copies of data with what they find in the yaml if present, it makes it easy to transition | 19:18 |
clarkb | fungi: so perfectly safe to modify the scripts directly for now? | 19:18 |
fungi | clarkb: yes, though i need to see what became of the patches for that. someone in the community proposed them (i think one of the savanna folks?) but i don't recall if they ever merged | 19:19 |
fungi | we had a huge uptick of work around then (might have been around oscon) and then i ost track | 19:20 |
fungi | lost track | 19:20 |
SergeyLukjanov | hi, I've started working on it, here is CR | 19:20 |
SergeyLukjanov | https://review.openstack.org/#/c/37620/ | 19:20 |
fungi | SergeyLukjanov: right! that was you. i feel really bad it's been dragging out due to lack of review cycles | 19:20 |
SergeyLukjanov | but it depends on nonexistent projects yaml file format #2 | 19:20 |
SergeyLukjanov | it's not completed yet I think, my bad, fully occupied by savanna activities now( | 19:21 |
*** sarob has quit IRC | 19:21 | |
fungi | i don't think we need a new yaml format right now. just adding the necessary data to the current format and making the parser override what's hard-coded in the scripts with what it finds in the yaml should be enough for us to support a transition | 19:22 |
*** sarob has joined #openstack-infra | 19:22 | |
SergeyLukjanov | ok, I'll try to update it soon | 19:22 |
fungi | a better layout would be great (and i even have a somewhat moldy spec for what it could look like) but i don't want to hold up pragmatic changes for it | 19:22 |
*** hasharSF has quit IRC | 19:23 | |
tstevenson | clarkb: Thanks for the help. I was able to get gerritbot going. I am now getting a PasswordRequiredException: Private key file is encrypted | 19:23 |
tstevenson | clarkb: decrypt with openssl and same error | 19:23 |
fungi | adding the new data to (implied) format v1 will get us benefits now, and then properly versioning the yaml, breaking it up and making the parser module understand both versions for some transitional period is a bit more work, and doesn't have to happen first | 19:24 |
SergeyLukjanov | fungi, yep, agreed | 19:24 |
SergeyLukjanov | oh, I have a question about stackforge/savanna-image-elements, it looks like it's broken | 19:25 |
fungi | #define "broken" | 19:25 |
SergeyLukjanov | https://review.openstack.org/#/c/45575/ | 19:25 |
* fungi looks | 19:25 | |
SergeyLukjanov | jenkins couldn't rebase changes | 19:25 |
openstackgerrit | A change was merged to openstack-infra/config: Add test jobs for RAX DFW and ORD regions https://review.openstack.org/45358 | 19:25 |
* fungi tests it | 19:25 | |
SergeyLukjanov | I think it could be backed by re pushing initial repo state | 19:26 |
clarkb | THis may be the same thing that hit mordred | 19:27 |
*** sarob has quit IRC | 19:27 | |
clarkb | we may need to dig into the zuul logs and see what the repos look like in zuuls working dirs | 19:27 |
fungi | yeah, i don't get merge conflicts between that change and the tip of the target branch, so it does indeed sound like it could be a bad cached repo on zuul | 19:28 |
clarkb | fungi: or the difference in merge resolution | 19:28 |
fungi | been happening since at least the 8th. do we know how far back? | 19:29 |
SergeyLukjanov | this repo has been created several days ago | 19:30 |
SergeyLukjanov | and asked mordred to push initial state to it twice | 19:30 |
clarkb | SergeyLukjanov: I think the repo has an initial state, but zuul doesn't | 19:30 |
SergeyLukjanov | so, looks like prev state has been cached | 19:30 |
SergeyLukjanov | yep | 19:30 |
clarkb | I am going to grab lunch can, help dig in mroe after | 19:32 |
fungi | clarkb: oddly, i copied zuul:/var/lib/zuul/git/stackforge/savanna-image-elements and then reset it to origin/master and was able to successfully cherry-pick 45575 into it | 19:33 |
SergeyLukjanov | guys, what are you thinking about the following lp-related options format for projects yaml http://paste.openstack.org/show/46357/ | 19:36 |
fungi | also, the tip of master in /var/lib/zuul/git/stackforge/savanna-image-elements has the same sha as what's replicated from gerrit to git.o.o and github | 19:36 |
SergeyLukjanov | ? | 19:36 |
fungi | SergeyLukjanov: wrong projects.yaml. what you have there is what zuul uses to decide what tests to run. you need the one which tells jeepyb about the projects themselves... http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/templates/review.projects.yaml.erb | 19:38 |
SergeyLukjanov | oops :) | 19:38 |
SergeyLukjanov | forgot about the fact that we have two project.yaml :) | 19:39 |
fungi | don't sweat it. we really shouldn't have two files named projects.yaml but that's a change for another day | 19:39 |
fungi | it has already confused far too many people | 19:39 |
fungi | clarkb: think i see it in the log... http://paste.openstack.org/show/46361/ | 19:42 |
fungi | No item found with id 'origin/HEAD' | 19:42 |
*** tjones has joined #openstack-infra | 19:42 | |
fungi | and it's right | 19:43 |
fungi | http://paste.openstack.org/show/46362/ | 19:44 |
fungi | mordred: ^ | 19:47 |
fungi | so that at least tells us *how* it's broken. now do brainstorm theories on *why* | 19:48 |
fungi | i wonder if shallow clones fail to create origin/HEAD | 19:49 |
fungi | er, not shallow clones (zuul doesn't anyway). you said lazy clones | 19:52 |
openstackgerrit | A change was merged to openstack-dev/hacking: hacking should conform with OS requirements https://review.openstack.org/45641 | 19:53 |
fungi | and zuul.merger.Repo._ensure_cloned() just seems to rely on git.Repo.clone_from() so nothing terribly exciting there | 19:55 |
*** dprince has quit IRC | 19:55 | |
*** ruhe has quit IRC | 19:58 | |
*** gyee has joined #openstack-infra | 19:58 | |
*** pcrews has quit IRC | 20:01 | |
openstackgerrit | Marton Kiss proposed a change to openstack-infra/config: Add groups-dev.openstack.org node https://review.openstack.org/42608 | 20:01 |
*** boris-42 has quit IRC | 20:11 | |
*** boris-42 has joined #openstack-infra | 20:13 | |
*** tjones has quit IRC | 20:14 | |
*** salv-orlando has joined #openstack-infra | 20:16 | |
*** marun has quit IRC | 20:16 | |
*** boris-42 has quit IRC | 20:21 | |
*** pabelanger_ has quit IRC | 20:22 | |
*** pabelanger has joined #openstack-infra | 20:22 | |
sdague | ttx: your cloud open slides somewhere for me to see so I make sure I don't dupe things you are doing? | 20:24 |
*** HenryG_ has quit IRC | 20:25 | |
*** HenryG has joined #openstack-infra | 20:26 | |
*** hashar has joined #openstack-infra | 20:31 | |
openstackgerrit | Pierre Rognant proposed a change to openstack-infra/jenkins-job-builder: Fix plot plugin support https://review.openstack.org/45754 | 20:40 |
*** thomasm has quit IRC | 20:41 | |
clarkb | fungi: weird | 20:41 |
clarkb | fungi: any more info? | 20:41 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/jeepyb: Improve git2lp and is_direct_release behaviour https://review.openstack.org/37620 | 20:42 |
SergeyLukjanov | it looks like it should now work ok with current review.project.yaml file format | 20:44 |
fungi | clarkb: got sidetracked for a bit. doing a quick comparison now to see how many of the cached repos on zuul are in a similar state. could give us some idea of timing and conditions which led to it | 20:44 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/jeepyb: Map all savanna repos to savanna at launchpad https://review.openstack.org/37621 | 20:44 |
clarkb | fungi: sounds good | 20:45 |
clarkb | fungi: I am going to keep poking at code review, ping me if I need to look at zuul | 20:46 |
openstackgerrit | Pierre Rognant proposed a change to openstack-infra/jenkins-job-builder: Fix plot plugin support https://review.openstack.org/45754 | 20:46 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/jeepyb: Map all savanna repos to savanna at launchpad https://review.openstack.org/37621 | 20:46 |
fungi | clarkb: three repos affected... http://paste.openstack.org/show/46370/ | 20:46 |
UtahDave | On the Jenkins server, does the jenkins user have full sudo rights? | 20:47 |
fungi | there are definitely other projects we've added before, between and after those which are not affected | 20:47 |
mrmartin | clarkb: the updated groups-dev patch landed in gerrit. please review it if you have a few minutes. I just applied the changes discussed today. | 20:47 |
clarkb | UtahDave: I don't think so | 20:47 |
clarkb | mrmartin: will do | 20:48 |
mrmartin | thanks | 20:48 |
openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: Add documentation for XDG_CACHE_HOME https://review.openstack.org/45729 | 20:48 |
UtahDave | clarkb: would it be possible to give Jenkins sudo rights to /usr/bin/salt-call ? | 20:48 |
fungi | UtahDave: is root access required to trigger salt notifications? | 20:48 |
UtahDave | Yeah, the user needs to be able to read the minion's private key in order to send messages on the event bus | 20:49 |
fungi | how about a dedicated jenkins slave where the jenkins user was granted read access to its minion private key? | 20:50 |
UtahDave | fungi: I can go about this another way that would be easier to control from within Salt, but if we can give Jenkins that permission this way is really nice | 20:50 |
UtahDave | fungi: that should work. Let me try that out on my test system here. | 20:50 |
fungi | UtahDave: at least on the jenkins master, we'd want to avoid granting the jenkins account any elevated privileges. if we did it on a slave, we'd still be granting other things run by jenkins on that slave free run of the machine | 20:51 |
UtahDave | Right. Ok, I'll see what I can do here. | 20:52 |
fungi | if giving the jenkins user on that machine read access to its minion key also allows it to make salt run things as root which could allow it to escalate itself, then there's probably not a whole lot of point to dancing around root access on that slave | 20:52 |
UtahDave | good point. | 20:52 |
fungi | but worth exploring how it can best be secured | 20:53 |
fungi | for example, we might hardcode one and only one salt-call command which the jenkins user can run on the slave, parameters and all | 20:53 |
fungi | (via a custom entry to sudoers) | 20:53 |
fungi | that might be a simple workaround | 20:53 |
UtahDave | yeah, that's what I'm investigating right now. I think that may be our best bet. | 20:54 |
*** Bada has quit IRC | 20:54 | |
*** sarob has joined #openstack-infra | 20:54 | |
*** svarnau_ has joined #openstack-infra | 20:56 | |
fungi | and it may be that we decide we can safely assign that job to an existing slave/node-label if we're comfortable doing so. would keep us from spinning up an entire separate machine which just idled waiting to run the occasional salt-call to have everyone update their puppet configs | 20:56 |
*** svarnau has quit IRC | 20:58 | |
*** mrmartin has quit IRC | 20:58 | |
*** dcramer has quit IRC | 20:59 | |
jog0 | jeblair: ping | 21:00 |
jog0 | https://review.openstack.org/#/admin/groups/204,members doesn't look right | 21:00 |
* jog0 is trying to create a 0.7.x branch | 21:01 | |
jog0 | of hacking | 21:01 |
clarkb | jog0: you need to be a member of that group? | 21:01 |
clarkb | jog0: I have added you to the group | 21:02 |
jog0 | clarkb: thanks | 21:02 |
jog0 | now to see if it works | 21:02 |
*** marun has joined #openstack-infra | 21:03 | |
*** amotoki has joined #openstack-infra | 21:03 | |
jog0 | that was easy | 21:03 |
*** emagana_ has joined #openstack-infra | 21:04 | |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/config: Update doc about JJB cache location https://review.openstack.org/45758 | 21:04 |
fungi | clarkb: i think i'm going to have to hunt down the zuul log entries from when one of the affected projects was initially cloned, but i need to cook dinner first before resuming | 21:05 |
fungi | so biab | 21:05 |
clarkb | fungi: ok | 21:05 |
pleia2 | mgagne: did you seek zaro's change https://review.openstack.org/#/c/45738/ | 21:07 |
mgagne | pleia2: unfortunately not | 21:07 |
mgagne | pleia2: will abandon mine | 21:07 |
*** ericw has quit IRC | 21:07 | |
mgagne | pleia2: thanks for pointing that out | 21:07 |
pleia2 | sure | 21:07 |
UtahDave | fungi: I did some testing with this sudoers entry: jenkins ALL=(ALL) NOPASSWD: /usr/bin/salt-call event.fire_master* | 21:09 |
UtahDave | results: http://pastebin.com/ijVfBuDh | 21:09 |
mgagne | zaro: sorry, I "stole" your bug | 21:12 |
UtahDave | That sudoers entry only allows the minion to send a message to the master. Nothing else. | 21:12 |
jog0 | getting ready to release a 0.7.1 branch of hacking https://review.openstack.org/#/q/status:open+project:openstack-dev/hacking+branch:0.7.x,n,z | 21:15 |
*** dcramer has joined #openstack-infra | 21:15 | |
zaro | mgagne: no problem. i'll just assign it back to me. | 21:16 |
mgagne | zaro: thanks for the follow-up on that bug, I lost the submitter's reply. | 21:17 |
*** dizquierdo has joined #openstack-infra | 21:18 | |
pleia2 | so, anteaya has been working on getting a test instance of the new sodabrew dashboard up, and we thought we could quickly toss up a puppetmaster too (on another server) in order to feed it some stuff | 21:25 |
pleia2 | now debugging an error she's getting when running the install_modules.sh script | 21:26 |
pleia2 | anyone have thoughts as to why we'd bump into this dependency issue? http://paste.openstack.org/show/46374/ | 21:26 |
anteaya | Puppet v2.7.23 | 21:26 |
*** vipul is now known as vipul-away | 21:26 | |
pleia2 | it occurs to me that we didn't include puppetlabs-haproxy when I set mine up | 21:26 |
pleia2 | so it could just be a dependency issue I didn't run into | 21:27 |
clarkb | looking | 21:27 |
anteaya | is there a flag I can use when I run install_modules.sh that says don't bother with these modules? | 21:27 |
pleia2 | anteaya: I'm going to log in to the server and poke around a little too | 21:27 |
anteaya | please do | 21:27 |
jog0 | clarkb: as we talked abou the other day I took a stab at recheckwatch https://review.openstack.org/#/c/45511/ | 21:27 |
anteaya | I'll be hands off until you give the word | 21:27 |
anteaya | you are in the puppet-master server, yes? | 21:28 |
pleia2 | anteaya: yeah | 21:28 |
anteaya | k | 21:28 |
clarkb | anteaya: pleia2: I think this is a case of poor dependency resolution that was not a problem when our master installed because things already existed | 21:28 |
clarkb | anteaya: pleia2: but is an issue when starting from scratch | 21:29 |
anteaya | ah | 21:29 |
pleia2 | yeah, that's the conclusion I'm starting to come to as well | 21:29 |
anteaya | so your suggestion is install the dependencies? | 21:29 |
clarkb | though I just installed the modules on my new test etherpad server and that worked | 21:29 |
anteaya | hmmmm | 21:29 |
clarkb | I think ripeinaar concat became puppetlabs-concat | 21:29 |
clarkb | if you remove puppetlabs-concat, install the two moduels that complain about it then rerun install modules I would expect that to work | 21:30 |
*** nati_ueno has joined #openstack-infra | 21:30 | |
anteaya | do you want to do that pleia2 since you are in there? | 21:31 |
anteaya | or shall I? | 21:31 |
jeblair | clarkb: https://review.openstack.org/#/c/44246/ | 21:31 |
pleia2 | anteaya: you can do it, just wondering how puppetlabs-concat was pulled in | 21:31 |
anteaya | pleia2: okay, should I wait for you or go ahead? | 21:32 |
pleia2 | anteaya: go ahead | 21:32 |
anteaya | k | 21:32 |
*** vipul-away is now known as vipul | 21:34 | |
jeblair | clarkb: also https://review.openstack.org/#/c/44496/ | 21:34 |
clarkb | jeblair: reviewing, thanks | 21:35 |
*** nati_uen_ has joined #openstack-infra | 21:36 | |
jeblair | clarkb: no rush; just pointing out things that are ready for your attention that you may not have seen as i'm going through | 21:36 |
clarkb | jeblair: yup its helpful. I think I can approve both of them | 21:36 |
clarkb | (and have approved the first) | 21:36 |
anteaya | so the way forward is `rm -rf /etc/puppet/modules/concat` ? | 21:37 |
*** danger_fo is now known as danger_fo_away | 21:37 | |
pleia2 | anteaya: no, you'll want to use the puppet way, hang on | 21:37 |
anteaya | k | 21:37 |
pleia2 | puppet module uninstall or remove, let me check... | 21:37 |
anteaya | ah okay | 21:37 |
pleia2 | sudo puppet module list | 21:38 |
clarkb | anteaya: pleia2: you might also get around it by changing the order of array elements in install_modules.sh | 21:38 |
pleia2 | will show you them all | 21:38 |
pleia2 | then: puppet module unistall puppetlabs-concat | 21:38 |
clarkb | I am not sure if bash arrays return a deterministic key order when iterating over the keys | 21:38 |
anteaya | uniinstall | 21:38 |
pleia2 | uninstall | 21:38 |
pleia2 | typing :) | 21:38 |
anteaya | :D | 21:38 |
*** mriedem1 has quit IRC | 21:38 | |
*** ArxCruz has quit IRC | 21:39 | |
openstackgerrit | A change was merged to openstack-infra/config: Run tests against pypy for pbr and hacking https://review.openstack.org/44246 | 21:39 |
*** MarkAtwood has joined #openstack-infra | 21:39 | |
anteaya | I'll try your first suggestion first clarkb, it didn't take me long to get here so if I screw it up, pleia2 can refresh me easy enough | 21:39 |
pleia2 | second suggestion was more for a long term solution | 21:39 |
anteaya | ah okay | 21:40 |
*** hashar has quit IRC | 21:40 | |
openstackgerrit | A change was merged to openstack-infra/jeepyb: Add options to manage-projects https://review.openstack.org/44496 | 21:41 |
*** ArxCruz has joined #openstack-infra | 21:41 | |
clarkb | jeblair: fungi: I am pulling a backup restore on wiki.o.o to double check the mysqldump backups (of which we have 5 now) are present as expected | 21:41 |
*** dkranz has quit IRC | 21:42 | |
clarkb | jeblair: fungi: it is running in a root screen if you are curious | 21:42 |
clarkb | figured I could let that run while I reviewed stuff | 21:42 |
jeblair | clarkb: cool! | 21:43 |
pleia2 | anteaya: I think I officially hit the peak of heat in my condo for the day, going to go get some fresh air and turn on the fan for real now, hopefully it'll cool off :) I'll be back in a few (or from a coffee shop that has AC) | 21:43 |
anteaya | pleia2: yes | 21:44 |
anteaya | good call | 21:44 |
anteaya | I will see what happens next thanks | 21:44 |
anteaya | it ran quiet | 21:45 |
mrodden | does anyone have a good grasp of how requirements.txt between oslo and the other projects that copy code from it works? | 21:45 |
mrodden | is the update.py supposed to sync the requirements.txt or does that need to be manually updated if a project get a new requirement that is in oslo but not in the project? | 21:46 |
*** emagana_ has quit IRC | 21:48 | |
*** vipul is now known as vipul-away | 21:49 | |
clarkb | mrodden: I think you need to update requirements.txt manually | 21:49 |
clarkb | I believe I had to do this last time I did an oslo sync | 21:50 |
mrodden | clarkb: yeah looking through the update.py... doesn't look like it does anything special with requirements | 21:50 |
mrodden | clarkb: thanks | 21:50 |
*** hashar has joined #openstack-infra | 21:52 | |
*** svarnau has joined #openstack-infra | 21:52 | |
anteaya | while pleia2 is out I am going to take the opportunity for another quick lap - hey look at that Sun! | 21:53 |
*** pcm_ has quit IRC | 21:54 | |
*** svarnau_ has quit IRC | 21:55 | |
mgagne | anteaya: which sun? all grey and cold here =( | 21:55 |
*** sdake has quit IRC | 22:00 | |
*** sdake has joined #openstack-infra | 22:00 | |
*** SergeyLukjanov has quit IRC | 22:01 | |
*** hemnafk is now known as hemna | 22:02 | |
*** vipul-away is now known as vipul | 22:03 | |
clarkb | jeblair: fungi: https://review.openstack.org/#/c/45464/ was that not approved for somereason? | 22:04 |
clarkb | the number of +2s without approval makes me afraid to hit the go button | 22:04 |
mrodden | thats not a git sha | 22:06 |
mrodden | thats a change id | 22:06 |
mrodden | they are different? | 22:06 |
*** svarnau has quit IRC | 22:07 | |
clarkb | mrodden: change ids and git shas are different | 22:07 |
*** fifieldt has joined #openstack-infra | 22:07 | |
mrodden | but apparently they are queried teh same way in the URLs | 22:07 |
mrodden | interesting | 22:07 |
annegentle_ | why doesn't this build? https://jenkins.openstack.org/view/Openstack-manuals/job/openstack-admin-guide-cloud/ | 22:07 |
clarkb | mrodden: change IDs are prefixed with an I followed by a sha1 whereas git shas are just a sha11 | 22:08 |
*** alexpilotti has quit IRC | 22:08 | |
*** burt has quit IRC | 22:08 | |
mrodden | clarkb: yeah, was getting worried, but apparently the URL it throws them into results in a query on either | 22:08 |
openstackgerrit | A change was merged to openstack-infra/config: Poke a hole through the proxy for cluster health https://review.openstack.org/45500 | 22:09 |
clarkb | annegentle_: we don't run that job on that jenkins master anymore | 22:09 |
clarkb | annegentle_: you will want to check jenkins01 and jenkins02 | 22:09 |
jeblair | clarkb: no aprv from me due to lack of babysitting time | 22:10 |
clarkb | jeblair: ok, I am in the same boat, tomorrow will probably be better | 22:11 |
fifieldt | jeblair, that one looks like the other translation jobs which means according to my understanding of the scripts it should "just work" | 22:11 |
fifieldt | that was 43991 | 22:11 |
*** weshay has quit IRC | 22:11 | |
jeblair | fifieldt: ah thanks; i was confused by receiving your reply to my message via a different irc transport protocol! | 22:12 |
openstackgerrit | A change was merged to openstack-infra/config: Publish hacking docs to docs.openstack.org https://review.openstack.org/45498 | 22:13 |
*** tjones has joined #openstack-infra | 22:13 | |
*** tjones has left #openstack-infra | 22:13 | |
openstackgerrit | Antoine Musso proposed a change to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation https://review.openstack.org/45715 | 22:15 |
*** pentameter has quit IRC | 22:16 | |
*** sarob has quit IRC | 22:16 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation https://review.openstack.org/45715 | 22:16 |
jog0 | if anyone has afew minutes lookign for some help with https://review.openstack.org/#/c/45511/ | 22:16 |
*** sarob has joined #openstack-infra | 22:16 | |
hashar | doh | 22:17 |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Add documentation for XDG_CACHE_HOME https://review.openstack.org/45729 | 22:17 |
clarkb | hashar: was something wrong with 45511? or was that dupe? | 22:18 |
clarkb | er 45715 | 22:18 |
openstackgerrit | A change was merged to openstack-infra/config: Grant tripleo-ptl tags on tripleo-image-elements https://review.openstack.org/45518 | 22:18 |
clarkb | I am confused did I approve a new patchset after reviewing the old? | 22:18 |
clarkb | oh nevermind I approved a different change | 22:19 |
*** sarob has quit IRC | 22:19 | |
openstackgerrit | A change was merged to openstack-infra/config: Remove extra cloud-admin in URL of End User Guide https://review.openstack.org/45640 | 22:19 |
hashar | clarkb: https://review.openstack.org/#/c/45715/ had a very tiny issue (missing a newline at the end of test_requires) so I just amended and reap proved the change. | 22:19 |
*** sarob has joined #openstack-infra | 22:19 | |
hashar | clarkb: you are probably talking about something completely unrelated :-D | 22:19 |
hashar | but apparently that jjb change is breaking the compare-xml job :( | 22:20 |
clarkb | hashar: yup I was talking about something else, I confused myself, sorry | 22:20 |
lifeless | hi | 22:21 |
hashar | does anyone know from where the python dependencies are downloaded in Jenkins ? | 22:21 |
marun | who do I talk to about jenkins-sudo-grep.sh? | 22:21 |
hashar | it can't download the pip module sphinxcontrib-programoutput :/ | 22:21 |
lifeless | clarkb: see #tripleo; something very weird on tie checks | 22:21 |
clarkb | jog0: for testing 45511, I think you will want to create two bogus bugs in launchpad, point a local instance of recheckwatch at review-dev, then recheck a change or two on review-dev with the two bogus bugs, then mark duplicate, recheck another thing (to regenerate the list) | 22:21 |
lifeless | clarkb: 2 days ago we had no errors | 22:23 |
clarkb | lifeless: oh, this is the thing I was told was safe to do by spamaps | 22:23 |
clarkb | lifeless: mordred proposed pep8 run against that repo, I checked with SpamapS this morning and he said go for i | 22:23 |
clarkb | t | 22:23 |
lifeless | hell no | 22:24 |
lifeless | it's not a python repo. | 22:24 |
lifeless | don't run pep8 against it | 22:24 |
jog0 | lifeless: haha | 22:24 |
lifeless | SpamapS: mordred: ^ | 22:24 |
jog0 | clarkb: yeah I was having problems getting recheck running locally | 22:25 |
jog0 | was wondering if there is a trick there I can use | 22:25 |
lifeless | clarkb: it would be nice if 'jobs to run' changes did a check against trunk as their pre-merge gate. | 22:25 |
lifeless | clarkb: should I file a wishlist bug to that effect ? | 22:26 |
clarkb | jog0: dropped what I said here in the change review so that we have it recorded. | 22:26 |
clarkb | jog0: what in particular isn't working? | 22:26 |
*** changbl has quit IRC | 22:26 | |
lifeless | clarkb: and whats the fastest way to unbreak tie? | 22:26 |
clarkb | lifeless: you can, it will end up being semi tricky to orchestrate that | 22:26 |
clarkb | lifeless: also sometimes that is the desired effect (that stuff fails until fixed) | 22:26 |
lifeless | clarkb: we can debate the merit of that another time :) | 22:27 |
clarkb | lifeless: fastest way to unbreak it is to revert just the part of that change that is failing | 22:27 |
clarkb | lifeless: we did it with pbr earlier when it was having so much trouble | 22:27 |
lifeless | clarkb: it's an openstack-config change, right ? | 22:27 |
clarkb | lifeless: we forced pbr to pass a test suite before any other new potentially bad stuff could merge | 22:27 |
clarkb | lifeless: I will propose the partial revert | 22:27 |
lifeless | clarkb: thank you | 22:27 |
*** mrodden has quit IRC | 22:27 | |
*** nati_ueno has quit IRC | 22:27 | |
*** prad_ has quit IRC | 22:28 | |
clarkb | lifeless: should it run the -requirements job? that was the other part of mordred's change | 22:29 |
*** portante is now known as portante|afk | 22:29 | |
*** jhesketh_ has joined #openstack-infra | 22:29 | |
*** nati_uen_ has quit IRC | 22:30 | |
lifeless | clarkb: no | 22:30 |
*** nati_ueno has joined #openstack-infra | 22:30 | |
*** thedodd has quit IRC | 22:30 | |
lifeless | clarkb: it has no requirements, test-requirements | 22:30 |
*** sarob has quit IRC | 22:31 | |
clarkb | lifeless: thanks | 22:31 |
lifeless | clarkb: it's only 'python' in the sense that it has a setup.{py,cfg} used to install a pile of shell. | 22:31 |
*** sarob has joined #openstack-infra | 22:31 | |
jog0 | clarkb: when i run it nothing happens | 22:31 |
lifeless | clarkb: any non-trivial python in it would be a bug, and should live in a separate dedicated real-python-repo | 22:31 |
jog0 | it just exits | 22:32 |
anteaya | mgagne: :( | 22:33 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Partial revert of 131c5bf474dd25121832ad2e470525894c004227 https://review.openstack.org/45787 | 22:33 |
clarkb | lifeless: ^ | 22:33 |
clarkb | jog0: even when run in the foreground? | 22:33 |
*** amotoki has quit IRC | 22:34 | |
clarkb | jog0: I expect in non foreground mode that it would do that as it is probably trying to write to /var/* somewhere where it doesn't have write pems | 22:34 |
clarkb | but foreground should work | 22:34 |
anteaya | saw 2 pileated woodpeckers | 22:34 |
lifeless | clarkb: did mordred add pep8 and requirements to a bunch of tripleo things? | 22:34 |
clarkb | lifeless: yes, well pep8 was already on a bunch of tripleo things. He added requirements where there was already pep8. tie was the exception whwere he added in place of the noop job | 22:35 |
*** sarob has quit IRC | 22:36 | |
jog0 | with -n | 22:36 |
jog0 | brb | 22:36 |
lifeless | tripleo-heat-templates is also not python | 22:37 |
lifeless | clarkb: did he add to it as well? | 22:37 |
lifeless | and the incubator | 22:38 |
clarkb | lifeless: heat-templates was left alone | 22:38 |
clarkb | incubator as well | 22:38 |
lifeless | cool | 22:38 |
lifeless | everything else should be fine | 22:38 |
clarkb | ok | 22:38 |
*** dizquierdo has left #openstack-infra | 22:40 | |
fungi | clarkb: on 45464 i suspect i just didn't realize mordred was leaving a +2 as i was typing the comment on mine, and jeblair was hesitant to approve anything while busy with the docs boot camp | 22:40 |
fungi | er, i guess he just said as much (that'll teach me to respond while still reading scrollback) | 22:41 |
jeblair | fungi: you say that every time. :P | 22:41 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Use bup to backup review-dev to an offsite server https://review.openstack.org/45788 | 22:42 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Use bup to backup review.o.o to an offsite server https://review.openstack.org/45789 | 22:42 |
*** KennethWilke has quit IRC | 22:42 | |
clarkb | jeblair: fungi ^ those are WIP becuase I haven't done the manual steps yet, but that will happen after I confirm wiki.o.o is good | 22:43 |
fungi | jeblair: good point. clearly i do not learn from this experience | 22:43 |
clarkb | still pulling the backup tar 14G and counting | 22:43 |
*** krtaylor has quit IRC | 22:45 | |
anteaya | so I have run `puppet apply -l /tmp/manifest.log --modulepath=modules:/etc/puppet/modules manifests/local.pp` on this dummy server with puppet-master, `ps aux | grep puppet-master` gives me ubuntu 6507 0.0 0.0 8104 924 pts/0 S+ 22:40 0:00 grep --color=auto puppet-master which is me running ps aux grepping for it | 22:45 |
uvirtbot | Launchpad bug 6507 in gnome-system-tools "network-admin can't define several locations" [Medium,Fix released] https://launchpad.net/bugs/6507 | 22:46 |
anteaya | so... how do I tell if puppet-master is up? | 22:46 |
lifeless | anteaya: the ssl port for it is open to anyone | 22:46 |
clarkb | anteaya: uh, puppet apply won't start a mater for you. Is local.pp a hacked up puppetmaster.pp? | 22:46 |
clarkb | anteaya: and it runs in apache as a passenger app | 22:46 |
lifeless | anteaya: but you need a cert to get it to do stuff | 22:47 |
*** lcestari has quit IRC | 22:47 | |
lifeless | fungi: oh hi | 22:47 |
lifeless | fungi: wonder if you could counter-sign clarkb's partial revert to fix tie? | 22:47 |
anteaya | local.pp http://paste.openstack.org/show/46381/ | 22:48 |
clarkb | anteaya: check if apache is running, and you can try talking to it if so | 22:48 |
fungi | UtahDave: thanks, that sudoers entry looks safe enough | 22:48 |
anteaya | it isn't | 22:48 |
anteaya | I will get the apache stick | 22:48 |
UtahDave | ok, thanks, fungi | 22:49 |
fungi | lifeless: sure, i read the discussion. will check and approve if it looks good | 22:49 |
anteaya | clarkb: is it possible for install_puppet.sh and install_modules.sh to run and apache not be installed? | 22:50 |
clarkb | anteaya: yes, but your local.pp should install apache | 22:51 |
anteaya | meaning I need to edit my local.pp so that installs apache? | 22:51 |
*** flaper87 is now known as flaper87|afk | 22:51 | |
anteaya | because there is no apache installed on my system right now, not one I can find | 22:52 |
*** reed has quit IRC | 22:52 | |
clarkb | anteaya: no, I think the existing manifest should do it for you | 22:52 |
anteaya | really? | 22:52 |
*** reed has joined #openstack-infra | 22:52 | |
anteaya | then apache is hiding somewhere that I can't find | 22:52 |
openstackgerrit | A change was merged to openstack-infra/config: Partial revert of 131c5bf474dd25121832ad2e470525894c004227 https://review.openstack.org/45787 | 22:52 |
anteaya | apt-cache policy apache2 says it isn't installed | 22:52 |
anteaya | lifeless: thanks your cert suggestion is still hovering in my mind | 22:53 |
clarkb | anteaya: is puppet-master installed? it is possible that the puppetmaster.pp doesn't do everything | 22:54 |
anteaya | ah okay | 22:55 |
clarkb | jeblair: fungi: I have successfully pulled one of the sql backups out of bup backup tar on wiki.o.o | 22:55 |
*** reed has quit IRC | 22:55 | |
clarkb | I haven't restored the DB in mysql but glancing at the file it looks goo | 22:55 |
fungi | awesome | 22:56 |
clarkb | I am going to do the manual setup for review-dev now | 22:56 |
fungi | assuming it's good and not merely goo | 22:56 |
anteaya | puppet-master isn't installed: http://paste.openstack.org/show/46382/ | 22:56 |
jeblair | i'm okay with goo | 22:56 |
lifeless | grey goo! thermal end to the world! | 22:56 |
anteaya | huh, I was working from the belief that install_puppet.sh and install_modules.sh installed puppet-master, looks like it doesn't | 22:56 |
lifeless | anteaya: install_puppet installs the puppet client agent; install_modules makes the referenced modules usable AIUI | 22:57 |
anteaya | it installs puppetlabs-apache | 22:57 |
clarkb | jeblair: do I need to do this on both servers now? I think you had mentioned spinning up a new server in HP land | 22:57 |
anteaya | okay | 22:57 |
anteaya | I don't know as I even need a puppet-master, I just need enough for this puppet-dashboard to do something | 22:58 |
anteaya | lifeless: okay | 22:58 |
*** MarkAtwood has left #openstack-infra | 22:59 | |
anteaya | lifeless: hmmmm, okay so since I have run install_modules.sh those modules theoretically should be useable | 22:59 |
anteaya | once i figure out how to use them | 22:59 |
openstackgerrit | A change was merged to openstack-infra/jeepyb: Adds <service>-api to map to openstack-api-site in Launchpad https://review.openstack.org/45324 | 23:00 |
clarkb | jeblair: for the `ssh $BUPUSER@ci-backup-rs-ord.openstack.org` step of setting up backups it runs the bup server command on the remote end. Do I need to do anything special to close that connection? | 23:04 |
clarkb | jeblair: or is ~. | 23:04 |
clarkb | sufficient? | 23:04 |
clarkb | will ^D work? | 23:06 |
jeblair | clarkb: not sure? try ^D, then ^C, then ~. in that order? | 23:07 |
*** vipul is now known as vipul-away | 23:08 | |
jeblair | clarkb: i think the old hp server is dead; if it's alive we should kill it; but i have no idea what account it was on or how to access it | 23:08 |
jeblair | clarkb: i have not spun up a replacement | 23:08 |
*** mestery_ has joined #openstack-infra | 23:09 | |
anteaya | I updated http://paste.openstack.org/show/46385/ with the contents of 00-puppet.pref | 23:09 |
clarkb | jeblair: ok, I will ignore it for now | 23:10 |
clarkb | ^D seems to have worked | 23:10 |
uvirtbot | clarkb: Error: "D" is not a valid command. | 23:10 |
clarkb | https://review.openstack.org/#/c/45788/ is ready for review | 23:11 |
*** rnirmal has quit IRC | 23:12 | |
*** mestery has quit IRC | 23:13 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Clarify bup server setup directions. https://review.openstack.org/45791 | 23:13 |
clarkb | review-dev should be ready | 23:13 |
jeblair | clarkb: cool, change lgtm | 23:14 |
clarkb | I need to step out for a bit, will probably get to review.o.o tomorrow after bug day | 23:14 |
fungi | clarkb: no luck dredging the zuul debug log... the earliest cloned project which is lacking origin/HEAD in zuul's cache is savanna-image-elements and i can see where zuul initially cloned it at 2013-09-03 21:52:49,548 in debug.log.2013-09-03 but no obvious errors then or after | 23:14 |
clarkb | jeblair: fungi: maybe we should just try deleting the repos in zuuls work dir? zuul should recreate them | 23:15 |
clarkb | at least as a possible workaround? | 23:15 |
jeblair | clarkb, fungi: wha? | 23:15 |
jeblair | it sounds like you're talking about a possible zuul bug? what's up? | 23:15 |
fungi | i can manually add the ref for that matter (all it does is symbolically point at the default branch on a bare remote) | 23:16 |
clarkb | jeblair: yes possible zuul bug | 23:16 |
fungi | jeblair: several projects were recently cloned in zuul's repo cache missing an origin/HEAD ref | 23:16 |
*** KLevenstein_away has quit IRC | 23:16 | |
fungi | causing the Repo.reset() to fail | 23:16 |
clarkb | ok stepping out biab | 23:16 |
fungi | jeblair: the offending three are openstack-infra/pypi-mirror, stackforge/fuel-main and stackforge/savanna-image-elements | 23:17 |
fungi | all seem to have been on or since september 3rd | 23:17 |
anteaya | I question what install_puppet.sh did then because my list of installed puppet modules seems to come straight from install_modules.sh - what is the purpose of the contents of /etc/apt/preferences.d/00-puppet.pref *she asked herself* | 23:18 |
jeblair | fungi: do you have error messages handy? | 23:18 |
fungi | jeblair: pasted earlier. just a sec and i can grep them back out of my log | 23:18 |
fungi | jeblair: the error when it tries to merge candidate changes is http://paste.openstack.org/show/46361/ | 23:19 |
*** fifieldt has quit IRC | 23:20 | |
*** vipul-away is now known as vipul | 23:20 | |
jeblair | fungi: hrm. i have to leave now. | 23:21 |
*** sarob has joined #openstack-infra | 23:21 | |
fungi | jeblair: no worries. i'll keep poking at it | 23:21 |
jeblair | thanks. good luck | 23:21 |
fungi | thanks back! | 23:21 |
*** dcramer has quit IRC | 23:24 | |
*** rev has quit IRC | 23:25 | |
*** sarob has quit IRC | 23:25 | |
fungi | as zuul i did `echo 'ref: refs/remotes/origin/master' > /var/lib/zuul/git/stackforge/savanna-image-elements/.git/refs/remotes/origin/HEAD` and then rechecked https://review.openstack.org/45575 which seems to have worked | 23:26 |
jog0 | getting further with the recheck test | 23:27 |
fungi | did the same to stackforge/fuel-main now so they won't be surprised | 23:28 |
fungi | and openstack-infra/pypi-mirror since it's easy enough to put back to the same broken state if we want to test (just remove .git/refs/remotes/origin/HEAD) | 23:29 |
*** afazekas has joined #openstack-infra | 23:40 | |
*** afazekas is now known as afazekas_zz | 23:43 | |
*** erfanian has joined #openstack-infra | 23:48 | |
*** dims has quit IRC | 23:49 | |
*** gyee has quit IRC | 23:50 | |
*** vipul is now known as vipul-away | 23:55 | |
*** vipul-away is now known as vipul | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!