Monday, 2013-09-09

*** sarob has joined #openstack-infra00:03
*** sarob has quit IRC00:08
*** sarob has joined #openstack-infra00:09
*** CaptTofu has joined #openstack-infra00:09
*** sarob has quit IRC00:13
*** DennyZhang has joined #openstack-infra00:19
*** sarob has joined #openstack-infra00:28
*** nati_ueno_2 has quit IRC00:33
*** sarob has quit IRC00:34
*** nati_ueno has joined #openstack-infra00:35
clarkbjeblair: the current doc reord change (45490) lgtm. Did you want another pass at reviewing it?00:38
*** michchap has joined #openstack-infra00:40
*** michchap_ has joined #openstack-infra00:41
*** michchap has quit IRC00:44
*** CaptTofu has quit IRC00:49
*** CaptTofu has joined #openstack-infra00:50
*** sarob has joined #openstack-infra00:55
*** DennyZhang has quit IRC00:57
*** zhiyan has joined #openstack-infra01:09
zhiyanfungi: ping01:10
*** sarob has quit IRC01:11
*** sarob has joined #openstack-infra01:11
zhiyananteaya: ping01:11
anteayazhiyan: hello01:11
zhiyananteaya: Hello. sorry to interrupt you, could you pls help look a Jenkins failure?01:12
zhiyanhttp://logs.openstack.org/71/45171/2/check/gate-python-cinderclient-python26/89ba852/01:13
* anteaya looks01:13
zhiyananteaya: i'm confused what's the reason cause that...01:13
zhiyananteaya: thank you01:13
*** sarob has quit IRC01:16
anteayawell the test failure seems to be consistent across all four times jenkins has tested the patch01:18
clarkbzhiyan: anteaya: one of the test runners exited with a non zero return value01:18
anteayaso I have to assume that there is something in your code causing the failure01:18
clarkbI would check that there isn't a sys.exit somewhere01:18
zhiyananteaya: ok, actually i'm confused on this err msg:01:20
zhiyanft1.37: process-returncode_StringException: Binary content:   traceback (test/plain; charset="utf8")01:20
zhiyananteaya: those two at here: http://logs.openstack.org/71/45171/2/check/gate-python-cinderclient-python26/89ba852/testr_results.html.gz01:20
anteayazhiyan: clarkb gave you a good suggestion01:20
anteayais there a sys.exit() call in the code?01:21
zhiyananteaya: hmm, iirc the answer is NO01:21
*** michchap has joined #openstack-infra01:21
zhiyani have not added 'sys.exec()' in the patch code01:22
*** nosnos has joined #openstack-infra01:22
clarkbzhiyan: sys.exit01:22
clarkbor anything else that would cause the tes runner to prematurely die01:23
zhiyanclarkb: ops, sorry, i want to say 'sys.exit'01:23
zhiyanclarkb: actually i can pass all ut test cases in local01:24
zhiyanclarkb: i did it everytime before i push it to review01:24
*** michchap_ has quit IRC01:24
clarkbzhiyan: 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.exit01:25
clarkbit is happening in argparse01:25
zhiyanlet me check it01:25
morganfainbergzhiyan, look to see if you've added an argument somewhere that hasn't been registered.01:29
morganfainbergzhiyan, and used that "new" argument.01:30
zhiyanmorganfainberg: nod, probably. checking01:30
clarkbzhiyan: I think you are not passing the volume id in the test01:35
clarkbzhiyan: also shouldn't the volume arg be volume_id for consistency?01:35
zhiyanmorganfainberg: 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
jeblairclarkb: lgtm; do you know whether annegentle_ is ready for approval?01:36
clarkbjeblair: I don't know if she is ready01:36
*** pcrews has joined #openstack-infra01:36
morganfainbergzhiyan, that much i don't know.  look at what clarkb recommended, might have something to do with that.01:36
jeblairannegentle_: 45490 is ready when you are01:37
zhiyanclarkb: which test cause you meaning?01:37
zhiyans/cause/case01:38
clarkbzhiyan: the ones in the stacktraces in the subunit log01:38
zhiyanclarkb: 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
zhiyanclarkb: yes i think they should be consistent.01:41
clarkbzhiyan: 267 and 271 are assertions01:41
clarkbrun_command on 265 and 269 is where you need the argument01:41
*** Ryan_Lane has quit IRC01:43
zhiyanclarkb: correct. fine, let me check. your suggestion is useful to me, thank you clarkb.01:43
*** yaguang has joined #openstack-infra01:44
*** fifieldt has joined #openstack-infra01:46
fifieldthi infra :)01:47
fifieldtgreetings from the pre-doc-camp dinner01:47
clarkbfifieldt: hello there. areyou guys ready for 45490 to merge?01:47
clarkbfifieldt: 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
fifieldtjust talked to her :)01:48
fifieldtus an hours time or so ok?01:48
fifieldtwe just want to be in front of screens for the docpocalypse01:49
clarkbsure01:49
clarkbI am sitting at home watching the football game, I should still be around in an hour or so01:49
fifieldtnyg vs dal?01:49
clarkbyup01:49
fifieldtcool, its on here too01:50
anteayafifieldt: how's the dinner?01:51
anteayagot a full house?01:51
*** CaptTofu has quit IRC01:51
*** CaptTofu has joined #openstack-infra01:51
fifieldtabout 10 of us here on the Tied House Brewery01:51
anteayanice01:52
fifieldteveryone says hi and thanks :)01:55
*** matsuhashi has joined #openstack-infra01:56
anteaya:D01:56
*** Ryan_Lane has joined #openstack-infra01:58
morganfainberghm. docpocalypse eh?02:03
clarkbmorganfainberg: reorging the structure of the docs to make them more consistent02:06
clarkbmorganfainberg: this requires updating all of the jenkins jobs02:07
morganfainbergclarkb, fun times.02:07
anteayamorganfainberg: I am operating under the belief that this is the patch in question: https://review.openstack.org/#/c/45490/1202:09
morganfainberganteaya, ahh02:09
anteayaI'm hands off but watching02:09
morganfainberghehe.02:09
morganfainbergi need to stop surfing the web and chatting in IRC so i can write some code i've been procrastinating on :P02:10
anteayaha ha ha02:11
anteayagood luck02:11
morganfainbergmaybe i'll go wander down the street here and get a beer, food, and some coffee first though02:11
anteayaI'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 more02:11
anteayahappy wandering02:12
anteayamay you run into beer02:12
morganfainbergoh, plenty of options for that here in pasadena (ca)02:12
clarkbmorganfainberg: you know it is sunday right?02:12
clarkbmorganfainberg: and football is back02:12
morganfainbergclarkb, and i have a 6hr meeting tomorrow (potentially) and a fix i want to get rolling for keystone for RC102:12
morganfainbergclarkb, i can watch football on my second monitor if i want :P02:13
anteayawoohoo 6hr meeting02:13
morganfainbergclarkb, and beer, i am doing this with beer! :P02:13
anteayayay lots to look forward to02:14
morganfainberganteaya, 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-infra02:15
morganfainberganyways.  good luck with the docpocalypse!02:15
anteayaI am confident that after this week's doc boot camp there will be many more folks with knowledge to share about docs02:16
morganfainbergyay!02:16
anteayayay!02:16
morganfainbergmore people who get docs are good.02:16
*** dguitarbite has joined #openstack-infra02:16
anteayayes02:17
anteaya'cause I don't really02:17
*** fifieldt has quit IRC02:19
morganfainberganteaya, i am better at writing code than doc writing02:19
morganfainberg(at least that what i keep telling myself)02:19
anteayaahhh, you beat me at both02:19
morganfainberganteaya, maybe.02:19
anteayaI can't claim ability in any area02:19
morganfainberganteaya, I read your blog post… you're probably better at docs ;)02:19
anteayalast 2 weeks I just felt like I was spinning in circles02:19
anteayaha ha ha02:20
anteayaI'm better at talking02:20
*** xchu has joined #openstack-infra02:20
morganfainberganteaya, gotta have those people too02:20
anteayaoh good02:20
anteayaso I can stay?02:20
anteayaat least until tomorrow?02:21
morganfainberganteaya, lol02:21
anteaya:D02:21
*** dims has quit IRC02:29
*** zhiyan has quit IRC02:30
annegentle_I like it! Docpocalypse!02:35
clarkbI am really beginning to question if this game is worth watching02:39
*** fifieldt has joined #openstack-infra02:43
*** UtahDave has joined #openstack-infra02:43
* anteaya goes to tv to find nyg dallas game02:44
*** dkliban has joined #openstack-infra02:44
* anteaya admires a nice running play02:46
*** michchap has quit IRC02:48
*** michchap has joined #openstack-infra02:49
anteayatime for me to sign off02:49
anteayahave a good <appropriate time of day> everyone02: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
fifieldtthanks anteaya02:52
clarkbannegentle_: sure why not02:53
annegentle_clarkb: because, docpocalypse? I dunno :)02:53
* annegentle_ bites nails02:53
annegentle_not really02:53
annegentle_lets' try it!02:53
clarkbok I will approve 45490 now02:53
clarkbdone, now we have to wait ~20 minutes02:54
annegentle_clarkb: okie doke. Nice.02:54
*** anteaya has quit IRC02:54
openstackgerritA change was merged to openstack-infra/config: Update manual publishing  to new directory layout  https://review.openstack.org/4549002:55
*** dkliban has quit IRC03:02
*** sarob has quit IRC03:08
*** sarob has joined #openstack-infra03:08
*** sarob has quit IRC03:13
clarkbannegentle_: recheck time?03:18
annegentle_clarkb: 53 to 18... yeah03: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 IRC03:20
*** nati_ueno has joined #openstack-infra03:21
*** dguitarbite has quit IRC03:21
fifieldtfail :D03:21
clarkbannegentle_: yes03:21
clarkbannegentle_: the bug number in recheck is the reason for the recheck, not the bug you are fixing03:21
fifieldt: A type incompatibility occured .. hmm03:21
clarkbannegentle_: 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 bug03:22
clarkbannegentle_: one test failed on test was successful03:22
fifieldtoh, it's the fop error03:22
clarkbis that a known error? anything I can do to help?03:24
clarkbthe success of the other jobs indicates all is not broken03:24
fifieldtit is clarkb, we think it's load related03:25
*** nati_ueno has quit IRC03:25
clarkblike slave load related? I don't expect those slaves to be heavily loaded when running maven03:25
fifieldtit was a theory, since it started happening more after we parallelised mvn jobs03:27
fifieldtbut actually, this error is from ops guide repo03:27
fifieldtwhich has just one job03:27
fifieldtso, not sure03:27
fifieldthttps://bugs.launchpad.net/openstack-manuals/+bug/1221721 fyi03:27
uvirtbotLaunchpad bug 1221721 in openstack-manuals "FontTriplet errors when building on Jenkins when heavy load" [Undecided,Incomplete]03:27
clarkbif builds are parallized could it be an inter thread conflict?03:29
clarkbwe run into those fairly often with parallelized tests03:29
clarkbwould possibly explain why local builds work (race condition that isn't exposed in the local env)03:29
*** HenryG has quit IRC03:30
fifieldtmaybe!03:30
fifieldtcool, ops guide checks fine now too annegentle_03:30
annegentle_clarkb: inter thread conflict. Huh. I never knew.03:31
fifieldtthough, the ops guide has only one job ... right?03:31
annegentle_fifieldt: right03:32
fifieldtwe should just rewrite mvn in python :)03:33
fifieldtthat'll fix it03:33
clarkbfifieldt: you don't have to, just use sphinx problem solved :)03:33
fifieldtheh :)03:33
* fifieldt is only pretending to be a docs person anyway03:34
annegentle_bwah03:34
clarkbalso with sphinx you get rst which you can edit in a real editor without tearing your eyes out03:34
fifieldtI edit in vim clarkb :)03:35
fifieldtlike I said, pretending :D03:35
fifieldtso what's next03:35
fifieldtannegentle_, are you approving those flattenpatches?03:35
annegentle_fifieldt: clark so if the recheck is good I approve, right?03:35
clarkbI say go for it, git makes reverting easy if you ahve to03:36
annegentle_Very happy to see gate-openstack-manuals-operations-guide SUCCESS in 2m 16s03:36
fifieldtlooks like dallas has this03:37
* fifieldt commences the prayer-to-zuul rituals03:38
annegentle_heh03:38
fifieldtfail03:41
clarkbare we approving https://review.openstack.org/#/c/45517/ ?03:41
fifieldtjust fop again03:41
clarkbfop affected a different change03:42
*** nicedice_ has joined #openstack-infra03:42
fifieldtsorry for the ambiguiity clarkb, I was referring to the gate fail on the one that was previously foping03:43
fifieldtI've given it another kick03:43
fifieldt45517 is good to go as far as I'm aware03:43
fifieldtops guide has merged03:44
fifieldtannegentle_, are you doing manuals?03:44
annegentle_fonttriplet strikes again03:44
*** UtahDave has quit IRC03:46
annegentle_fifieldt: approved it again03:47
fifieldtthey both merged03:49
fifieldtwell done :)03:49
fifieldttime for the website patch now?03:50
fifieldtand the validate.py patch?03:50
fifieldtrebasing 4554303:52
annegentle_fifieldt: yep and yes03:52
*** dhellmann has quit IRC04:01
*** amotoki has joined #openstack-infra04: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
clarkbannegentle_: the tests were put in place first04:13
annegentle_clarkb: but validate.py is one of the tests?04:13
fifieldthmm04:14
* fifieldt looks at logs again04:14
*** mtreinish has quit IRC04:14
fifieldt Embedded error: /home/jenkins/workspace/gate-openstack-manuals-validate/doc/common/glossary not found.04:14
fifieldtmaybe not a problem with merge order, but specific to glossary ...?04:15
annegentle_fifieldt: ooo04:15
annegentle_fifieldt: yeah glossary is now in doc/glossary not common/glossary04:15
annegentle_fifieldt: I mean doc/src/docbkx/common/glossary04:15
*** mtreinish has joined #openstack-infra04:15
fifieldtso it's a gate error then04:16
fifieldtoh04:16
fifieldtor04:16
fifieldtjust an include error04:16
fifieldt"Build of book security-guide failed " ... glossary04:16
annegentle_fifieldt: ah yep04:17
annegentle_fifieldt: I can patch that04:17
*** SergeyLukjanov has joined #openstack-infra04:17
fifieldtby 'gate error' I meant 'job error'04:17
fifieldtcool, thanks annegentle_04:17
*** sarob has joined #openstack-infra04:19
*** dkliban has joined #openstack-infra04:23
*** mtreinish has quit IRC04:24
*** sarob has quit IRC04:25
*** mtreinish has joined #openstack-infra04:26
*** SergeyLukjanov is now known as SergeyLukjanov_04:31
*** SergeyLukjanov_ has quit IRC04:33
*** dkliban has quit IRC04:40
*** SergeyLukjanov has joined #openstack-infra04:41
*** fifieldt has quit IRC04:45
*** matsuhas_ has joined #openstack-infra05:00
*** matsuhashi has quit IRC05:00
*** SergeyLukjanov has quit IRC05:04
*** nicedice_ has quit IRC05:11
*** matsuhas_ has quit IRC05:26
*** matsuhashi has joined #openstack-infra05:29
*** nosnos has quit IRC05:32
*** nosnos has joined #openstack-infra05:32
*** matsuhashi has quit IRC05:34
*** dkliban has joined #openstack-infra05:35
*** boris-42 has joined #openstack-infra05:39
*** matsuhashi has joined #openstack-infra05:39
*** sergmelikyan has joined #openstack-infra05:41
sergmelikyanCan anyone help me with pypi jobs issue?05:41
*** dkliban has quit IRC05:41
clarkbsure, whats up?05:42
*** salv-orlando_ has joined #openstack-infra05:48
*** salv-orlando has quit IRC05:52
*** salv-orlando_ is now known as salv-orlando05:52
sergmelikyanThis job https://jenkins.openstack.org/job/python-muranoclient-pypi-upload/2/console is successfull06:00
sergmelikyan but no package on pypi06:00
sergmelikyanclarkb, i could not find reason why it is happened. Any thoughts?06:01
*** sergmelikyan has quit IRC06:01
*** SergeyLukjanov has joined #openstack-infra06:03
clarkbthere doesn't appear to be a python-muranoclient registered on pypi06:04
clarkbthe pypi job can't publish to an unregistered package06:04
*** SergeyLukjanov has quit IRC06:06
*** sergmelikyan has joined #openstack-infra06:07
clarkbsergmelikyan: there isnt a python-muranoclient registered on pypi06:07
clarkbsergmelikyan: you will need to register the project and give openstackci package ownership for the pypi jobs to publish on pypi06:09
sergmelikyanThanks, i did not know that!06:10
sergmelikyanclarkb, is maintainer right enough?06:23
sergmelikyanOr exactly owner right should be assigned to openstackci?06:24
clarkbmaybe I was looking at https://pypi.python.org/pypi/gerritlib/0.2.5 which has both06:24
clarkbheatclient has both too https://pypi.python.org/pypi/python-heatclient06:25
clarkbit is possible maintainer is sufficient06:26
sergmelikyanbut we could not check this without publishing new release, i guess. I also couldn't find a package with only maintainer right for openstackci06:27
sergmelikyanThx, for your help!06:31
*** thomasbiege has joined #openstack-infra06:32
*** thomasbiege has quit IRC06:35
*** jeremyb_ has quit IRC06:53
*** jeremyb_ has joined #openstack-infra06:53
*** jeremyb_ is now known as jeremyb06:54
*** odyssey4me has joined #openstack-infra06:57
*** matsuhashi has quit IRC06:57
*** xchu has quit IRC06:58
*** nosnos_ has joined #openstack-infra07:00
*** nosnos has quit IRC07:01
*** matsuhashi has joined #openstack-infra07:01
*** xchu has joined #openstack-infra07:11
*** boris-42 has quit IRC07:11
*** flaper87|afk is now known as flaper8707:13
*** SergeyLukjanov has joined #openstack-infra07:16
*** Bada has joined #openstack-infra07:19
*** dizquierdo has joined #openstack-infra07:29
*** Bada has quit IRC07:39
*** michchap has quit IRC07:40
*** shardy is now known as shardy_afk07:41
*** dkehn_ has joined #openstack-infra07:46
*** dkehn has quit IRC07:46
*** Bada has joined #openstack-infra07:50
*** SergeyLukjanov has quit IRC07:57
*** Bada has quit IRC07:59
*** SergeyLukjanov has joined #openstack-infra08:01
*** dizquierdo has quit IRC08:03
*** SergeyLukjanov has quit IRC08:12
*** SergeyLukjanov has joined #openstack-infra08:17
*** dripton__ has joined #openstack-infra08:18
*** derekh has joined #openstack-infra08:19
*** dripton_ has quit IRC08:19
*** mkerrin has joined #openstack-infra08:20
*** boris-42 has joined #openstack-infra08:31
*** Bada has joined #openstack-infra08:37
*** afazekas has joined #openstack-infra08:40
*** jpich has joined #openstack-infra08:40
*** SergeyLukjanov has quit IRC08:53
*** dizquierdo has joined #openstack-infra08:57
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Remove extra cloud-admin in URL of End User Guide  https://review.openstack.org/4564008:59
*** dkehn_ has quit IRC09:03
openstackgerritMate Lakat proposed a change to openstack-dev/hacking: hacking should conform with OS requirements  https://review.openstack.org/4564109:04
*** xchu has quit IRC09:14
*** Ryan_Lane has quit IRC09:15
*** dkehn has joined #openstack-infra09:18
*** matsuhashi has quit IRC09:26
*** matsuhashi has joined #openstack-infra09:26
*** xchu has joined #openstack-infra09:27
*** SergeyLukjanov has joined #openstack-infra09:31
*** pblaho has joined #openstack-infra09:33
*** dizquierdo has quit IRC09:38
*** SergeyLukjanov is now known as SergeyLukjanov_09:39
*** xchu has quit IRC09:41
*** SergeyLukjanov_ is now known as SergeyLukjanov09:41
*** dizquierdo has joined #openstack-infra09:42
*** Ryan_Lane has joined #openstack-infra09:46
*** pblaho has quit IRC09:47
*** Ryan_Lane has quit IRC09:54
*** SergeyLukjanov is now known as SergeyLukjanov_09:59
*** SergeyLukjanov_ is now known as SergeyLukjanov10:00
*** alexpilotti has joined #openstack-infra10:02
*** Ryan_Lane has joined #openstack-infra10:21
*** Ryan_Lane has quit IRC10:26
*** pcm_ has joined #openstack-infra10:28
*** pcm_ has joined #openstack-infra10:29
*** SergeyLukjanov is now known as SergeyLukjanov_10:29
*** matsuhashi has quit IRC10:30
*** SergeyLukjanov_ has quit IRC10:31
*** nosnos_ has quit IRC10:39
*** lcestari has joined #openstack-infra10:42
*** ruhe has joined #openstack-infra10:42
*** HenryG has joined #openstack-infra10:51
*** Ryan_Lane has joined #openstack-infra10:51
*** HenryG_ has joined #openstack-infra10:52
*** HenryG_ has quit IRC10:54
*** HenryG has quit IRC10:55
*** Ryan_Lane has quit IRC10:56
*** HenryG has joined #openstack-infra10:58
*** ruhe has quit IRC10:59
*** yaguang has quit IRC11:00
*** afazekas has quit IRC11:01
*** SergeyLukjanov has joined #openstack-infra11:01
*** michchap has joined #openstack-infra11:01
*** michchap has quit IRC11:06
*** salv-orlando has quit IRC11:08
*** DennyZhang has joined #openstack-infra11:11
*** plomakin_ has quit IRC11:15
*** cthulhup has joined #openstack-infra11:16
*** Ryan_Lane has joined #openstack-infra11:22
*** plomakin has joined #openstack-infra11:25
*** Ryan_Lane has quit IRC11:26
*** afazekas has joined #openstack-infra11:27
*** afazekas_ has joined #openstack-infra11:27
*** DennyZhang has quit IRC11:30
*** ruhe has joined #openstack-infra11:30
*** DennyZhang has joined #openstack-infra11:31
*** afazekas has quit IRC11:32
*** SergeyLukjanov is now known as SergeyLukjanov_11:32
*** SergeyLukjanov_ has quit IRC11:41
*** DennyZhang has quit IRC11:47
*** pblaho has joined #openstack-infra11:47
*** DennyZhang has joined #openstack-infra11:48
*** dims has joined #openstack-infra11:48
*** anteaya has joined #openstack-infra11:53
*** DennyZhang has quit IRC11:58
*** ruhe has quit IRC11:59
*** cthulhup has quit IRC12:00
*** DennyZhang has joined #openstack-infra12:00
*** SergeyLukjanov has joined #openstack-infra12:02
*** ArxCruz has joined #openstack-infra12:03
*** dhellmann has joined #openstack-infra12:06
*** SergeyLukjanov is now known as SergeyLukjanov_12:06
dhellmannhere'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
anteayamorning dhellmann12:08
*** SergeyLukjanov_ is now known as SergeyLukjanov12:08
*** weshay has joined #openstack-infra12:08
dhellmanngood morning, anteaya, you're online early (aren't you in western CA?)12:09
anteayaEastern Time Zone (Canada)12:09
dhellmannah, my mistake12:09
anteayaso not so early12:09
*** dims has quit IRC12:09
anteayathat's okay12:09
*** rfolco has joined #openstack-infra12:09
*** dims has joined #openstack-infra12:09
dhellmannsame time as me :-)12:09
anteaya:D12:09
anteayaI just haven't been as early as you much before12:10
anteayaso far I haven't been caught in the vortex of requirements issues12:10
anteayaso I don't have much to offer12:10
anteayaexcept an ear12:10
dhellmannthis is early for me, I found that issue while reviewing email over breakfast and wanted to make sure everyone was aware12:11
anteayagood idea12:12
dhellmannit 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 of12:12
* anteaya nods12:12
anteayaI am finding this is the only time of day I can get a sense of what is going on12:12
anteayaby the time everyone else arrives I feel like I am running to catch a train12:13
anteayawhich I never end up catching12:13
*** weshay has quit IRC12:14
dhellmannI know the feeling12:14
dhellmannbreakfast is ready, bbiab12:15
anteayahappy breakfast12:16
*** SergeyLukjanov is now known as SergeyLukjanov_12:19
mordreddhellmann: morning12:19
anteayahey mordred12:20
anteayahow was the show?12:20
mordredanteaya: it's coming along nicely12:20
anteayadhellmann: you also suggested documenting the need for cliff 1.4.3 for python26 users, that seems like the fastest/easiest to me12:21
mordreddhellmann: why is pyparsing broken? :)12:21
anteayamordred: yay12:21
*** derekh is now known as derekh_afk12:21
*** thomasm has joined #openstack-infra12:22
*** SergeyLukjanov_ is now known as SergeyLukjanov12:24
*** weshay has joined #openstack-infra12:29
*** SergeyLukjanov is now known as SergeyLukjanov_12:35
*** SergeyLukjanov_ has quit IRC12:43
ttxmordred: see my comment in https://review.openstack.org/#/c/44489/ -- would "resolutions" be a better directory name ?12:44
*** flaper87 has quit IRC12:44
*** flaper87 has joined #openstack-infra12:44
anteayaafternoon ttx12:45
mordredttx: good point. I like resolutions and positions both12:45
ttxmordred: 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
ttxmordred: ok, let's try resolutions.12:45
ttxwill update the patch12:46
mordredttx: https://review.openstack.org/#/c/41927/ <-- could use review by clarkb, fungi  or jeblair when then get in12:46
*** ruhe has joined #openstack-infra12:46
dhellmannmordred: pyparsing made the choice to go python 3 only for the 2.x release series12:46
ttxmordred: that would be a manually-triggered job ?12:47
mordreddhellmann: so it's not possible to just say "pip install pyparsing" and get the right thing?12:47
mordredttx: it would trigger on tags12:47
dhellmannanteaya: 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
mordreddhellmann: yuck. that just makes me angry12:47
dhellmannmordred: no, you have to specify the version based on your python version12:47
mordredthat's so broken I can't even begin to formulate words to describe it12:48
* dhellmann misses static linking and a.out12:48
* mordred misses engineers caring about software engineering12:48
ttxmordred: 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 waiting12:48
dhellmannmordred: 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
mordredttx: you can delay the branch deletion just fine12:48
mordreddhellmann: 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 both12:49
* dhellmann nods12:49
mordreddhellmann: we can add some special case logic - we're already special casing something else because it12:49
mordredbecause it's a broken turd that doesn't work properly12:49
mordredbut I'm going to be angry about it12:50
mordreddhellmann: how about if we fork pyparsing instead?12:50
ttxmordred: OH. You merge back when the tag is done, but that doesn't delete the branch. I see.12:50
dhellmannmordred: I could try emailing paul, and see what the back story is first :-)12:50
mordreddhellmann: 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
dhellmannah12:51
dhellmannso perhaps that was meant to be a temporary situation12:51
dhellmannlet me test cliff with 2.0.112:52
mordredyay!12:52
ttxmordred: so... for the H3 case we would still trigger it manually (since the tag already happened) ?12:52
*** SergeyLukjanov has joined #openstack-infra12:52
*** SergeyLukjanov is now known as SergeyLukjanov_12:52
*** SergeyLukjanov_ is now known as SergeyLukjanov12:52
dhellmannmordred: we'll need to talk to catherine about a new cmd release, too12:52
mordredttx: yes. and it's easy enough to do12:53
ttxmordred: ok, let's wait for the review, no hurry12:53
*** Ryan_Lane has joined #openstack-infra12:53
*** beagles has quit IRC12:55
*** jhesketh_ has quit IRC12:55
*** Ryan_Lane has quit IRC12:58
*** beagles has joined #openstack-infra13:01
*** beagles has quit IRC13:01
openstackgerritA change was merged to openstack-infra/config: Add git.o.o to list of ssl certificates we monitor  https://review.openstack.org/4546113:02
*** Guest14685 has joined #openstack-infra13:02
openstackgerritA change was merged to openstack-infra/config: update thirdy party testing how to doc  https://review.openstack.org/4549513:02
openstackgerritA change was merged to openstack-infra/config: Add backup restore docs.  https://review.openstack.org/4531213:02
*** dprince has joined #openstack-infra13:04
openstackgerritA change was merged to openstack-infra/nodepool: Add a nodepool command  https://review.openstack.org/4455313:09
*** dkliban has joined #openstack-infra13:11
*** changbl has quit IRC13:12
*** rustlebee is now known as russellb13:14
*** mriedem has joined #openstack-infra13:15
*** Ryan_Lane has joined #openstack-infra13:16
*** jpeeler has joined #openstack-infra13:17
*** jpeeler has quit IRC13:17
*** jpeeler has joined #openstack-infra13:17
anteayadhellmann: yes having the python3.3 gate jobs able to download the dependencies would be nice13:20
*** Ryan_Lane has quit IRC13:20
*** dripton__ is now known as dripton13:30
dhellmannmordred: https://bitbucket.org/catherinedevlin/cmd2/pull-request/2/use-the-latest-version-of-pyparsing-when/diff13:32
*** tstevenson has joined #openstack-infra13:46
mordreddhellmann: woot13:48
dhellmannmordred: updated at https://bitbucket.org/catherinedevlin/cmd2/pull-request/3/use-the-latest-version-of-pyparsing-when/diff13:48
mordreddhellmann: I will be quite pleased when that lands13:49
openstackgerritA change was merged to openstack/requirements: Add falcon to global-requirements  https://review.openstack.org/4507013:54
*** thedodd has joined #openstack-infra13:54
*** afazekas_ is now known as afazekas13:55
*** marun has quit IRC13:56
*** alaski-vacation is now known as alaski13:57
*** prad_ has joined #openstack-infra13:58
*** dkranz has quit IRC14:00
*** burt has joined #openstack-infra14:04
*** mrmartin has joined #openstack-infra14:06
*** rnirmal has joined #openstack-infra14:13
*** KennethWilke has joined #openstack-infra14:15
*** changbl has joined #openstack-infra14:18
*** pentameter has joined #openstack-infra14:19
*** Guest14685 is now known as beagles14:20
*** beagles is now known as b3nt_pin14:20
*** b3nt_pin is now known as beagles14:20
*** blamar has joined #openstack-infra14:20
*** adalbas has quit IRC14:22
*** salv-orlando has joined #openstack-infra14:29
*** dklyle is now known as david-lyle14:37
*** adalbas has joined #openstack-infra14:38
*** salv-orlando has quit IRC14:40
*** SergeyLu_ has joined #openstack-infra14:42
*** ruhe has quit IRC14:43
*** DennyZhang has quit IRC14:44
*** SergeyLu_ has quit IRC14:46
Shrewsfungi, olaph: you guys going to the OS meetup @ redhat hq tomorrow?14:46
fungiShrews: i was planning to14:46
funginot so much interested in the rdo demo they have planned, but i at least want to be around to provide some community support14:47
anteayazhiyan: congratulations getting your patch to pass check tests: https://review.openstack.org/#/c/45171/214:50
*** salv-orlando has joined #openstack-infra14:51
*** mrodden has quit IRC14:53
*** ruhe has joined #openstack-infra14:54
*** nati_ueno has joined #openstack-infra14:55
*** nati_ueno has quit IRC14:57
*** nati_ueno has joined #openstack-infra14:58
mordredfungi: I can has +2? https://review.openstack.org/#/c/42530/14:59
*** mrmartin has quit IRC15:01
*** salv-orlando has quit IRC15:02
fungimordred: lgtm15:02
mordredfungi: thanks15:02
*** krtaylor has quit IRC15:02
*** nati_ueno has quit IRC15:02
*** SergeyLukjanov is now known as SergeyLukjanov_15:04
*** SergeyLukjanov_ is now known as SergeyLukjanov15:05
*** mrodden has joined #openstack-infra15:05
*** wenlock has joined #openstack-infra15:12
*** erfanian has quit IRC15:13
*** UtahDave has joined #openstack-infra15:14
*** dkranz has joined #openstack-infra15:16
mordredfungi: it's so quiet in here this morning15:17
mordredfungi: don't we normally need to freak out about something on a monday morning?15:17
*** afazekas has quit IRC15:17
fungimordred: i have to assume everyone is still hung over from the post-h3 revelry15:18
mordredfungi: good call15:18
fungiand so nobody's noticed it all broke15:18
fungii'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 filters15:20
anteayabet the cronspam category is huge15:21
mordredfungi: that's a good idea15:24
fungianteaya: 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 noise15:26
SpamapSlol15:27
SpamapSexim for mail filtering15:27
*** dkehn has quit IRC15:27
SpamapSyou guys are true heroes15:27
anteayafungi sounds like you have a fun time planned15:27
SpamapSfungi: they have this procmail thing...15:27
anteayahopefully it stays quiet enough you can make some progress15:27
anteayafilering out the security bugs from the rest will be very interesting15:28
fungiSpamapS: i use the exim .forward file filter. the sieve language is much nicer and more maintainable than traditional procmail15:28
*** pabelanger_ has joined #openstack-infra15:29
SpamapSfungi: oohhh that exim filtering15:29
*** Bada has quit IRC15:30
*** pabelanger_ has quit IRC15:30
*** pabelanger_ has joined #openstack-infra15:30
SpamapSI actually refuse to do specific server side filtering... I tend to unsubscribe from things worse than about 10:1 noise:signal15:30
*** pabelanger has quit IRC15:30
SpamapSThough I do shove all mailing lists into a lists folder, and all bugs into a bugs folder. :)15:30
*** pabelanger has joined #openstack-infra15:31
*** pabelanger has quit IRC15:31
*** pabelanger_ is now known as pabelanger15:31
SpamapSjust so that my inbox is usable for immediate communication.15:31
*** pabelanger_ has joined #openstack-infra15:31
*** gyee has joined #openstack-infra15:32
*** dkehn has joined #openstack-infra15:33
*** pabelanger has quit IRC15:35
fungiyeah, 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 messages15:35
*** pabelanger has joined #openstack-infra15:36
*** dkehn has quit IRC15:38
*** sdake has joined #openstack-infra15:38
*** jergerber has joined #openstack-infra15:41
*** dkehn has joined #openstack-infra15:42
*** boris-42 has quit IRC15:43
*** mkerrin has quit IRC15:47
*** zehicle has quit IRC15:47
pleia2good morning15:53
anteayamorning pleia215:55
openstackgerritA change was merged to openstack/requirements: Add TripleO for-heat modules.  https://review.openstack.org/4556415:58
mordredmorning pleia215:59
*** ruhe has quit IRC16:01
*** reed has joined #openstack-infra16:04
mordreddhellmann: wow. pycon cfp wants a description, a detailed abstract and an OUTLINE16:05
mordreddhellmann: it's september. the conference isn't until april16:05
mordreddhellmann: I promise I have never had an outline that early16:05
*** nicedice has joined #openstack-infra16:13
*** pabelanger has quit IRC16:13
*** svarnau has joined #openstack-infra16:14
mordreddstufft, Alex_Gaynor: I've just submitted a pbr talk for pycon - if it gets accepted, please be sure to come heckle16:15
Alex_Gaynormordred: A+, awesome!16:15
dstufftassuming i'm able to go :[16:15
mordreddstufft: do you want me to call chris and yell at him until he lets you?16:15
*** pabelanger has joined #openstack-infra16:16
dstufftmordred: heh, Nebula wouldn't be the problem. My daughter has a chronic condition so leaving the country is kinda hard16:19
mordreddstufft: ah. that's fair - and eek16:19
*** jpich has quit IRC16:23
*** ruhe has joined #openstack-infra16:23
*** ruhe has quit IRC16:24
*** ruhe has joined #openstack-infra16:27
clarkbmorning16:28
anteayamorning clarkb16:29
anteayaI saw what you meant about the game16:29
clarkbscrollback isn't insane today, I like this16:29
anteaya:D16:29
*** flashgordon is now known as jog016:30
openstackgerritDan Prince proposed a change to openstack-infra/reviewday: Add support for latest SS result formats.  https://review.openstack.org/4570916:30
*** SergeyLukjanov is now known as SergeyLukjanov_16:32
openstackgerritTobias Stevenson proposed a change to openstack-infra/gerritbot: Fix comment event reporting in Gerritbot  https://review.openstack.org/4491316:32
*** SergeyLukjanov_ is now known as SergeyLukjanov16:32
mordredmorning clarkb16:32
mordredclarkb: it's really quiet in here this morning16:33
mordredclarkb: https://review.openstack.org/#/c/41927/ and https://review.openstack.org/#/c/42530/ are simple and could use some love16:33
clarkbmordred: ok16:33
openstackgerritDan Prince proposed a change to openstack-infra/reviewday: Add support for latest SS result formats.  https://review.openstack.org/4570916:33
clarkbI have resolved to catch up on reviews this morning, then hopefully finish up backups stuff this afternoon16:33
openstackgerritTobias Stevenson proposed a change to openstack-infra/gerritbot: Fix comment event reporting in Gerritbot  https://review.openstack.org/4491316:35
openstackgerritA change was merged to openstack-infra/config: Fix missing requirements list  https://review.openstack.org/4528316:35
mordredclarkb: w00t!16:35
mordredclarkb: I love it when you catch up on reviews - except, of course, when I have to fix things :)16:36
clarkbmordred: 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 change16:36
*** SergeyLukjanov has quit IRC16:36
clarkbmordred: there are comments from patchset 1 that need addressing. Is that something you are interested in?16:36
mordredclarkb: looking16:37
*** pblaho has quit IRC16:37
openstackgerritA change was merged to openstack-infra/reviewday: Add support for latest SS result formats.  https://review.openstack.org/4570916:38
*** nati_ueno has joined #openstack-infra16:38
mordredclarkb: most of patch 1 seem to be addressed - the print->log comments seem to be something that should happen in a different patch16:38
*** ruhe has quit IRC16:39
* jeblair is at doc bootcamp16:39
clarkbjeblair: have fun16:40
jeblairannegentle_ very much likes git.o.o16:40
jeblairpleia2, fungi, clarkb ^16:40
clarkbmordred: 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
fungijeblair: annegentle_: yay!16:40
fungiso do i!16:40
clarkbI like that I can remember what the dir structure is like so I can type out urls by hand16:41
clarkbjeblair:  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 tag16:42
*** mrmartin has joined #openstack-infra16:42
mrmartinhi16:42
clarkbmrmartin: hello16:42
mrmartinclarkb: if you have some free minutes, may I ask you to review https://review.openstack.org/#/c/42608/ ?16:43
jeblairclarkb: doubt i'll be able to do any real work,  so if you're in a rush you may have to go without me16:43
clarkbmrmartin: sure, I can review that when I am done with 4192716:43
openstackgerritKhai Do proposed a change to openstack-infra/config: link git sha in commit comments to the gerrit review  https://review.openstack.org/4546416:43
clarkbmordred: ^16:43
mrmartinclarkb: the sitedeploy part was rewritten for drupal deployment, thanks16:43
*** mgagne has joined #openstack-infra16:43
mordredclarkb: python setup.py test should be identical to python setup.py testr - EXCEPT - it should also work for projects that use nose16:44
*** dizquierdo has left #openstack-infra16:44
mordredclarkb: so it's a potential consistent way for our scripts to run unittests no mater which engine they're using16:45
mordredclarkb: we haven't migrated migrated yet because we haven't exhaustively tested it yet16:45
mordredand FF was coming up16:45
openstackgerritA change was merged to openstack-infra/config: Add cookiecutter templates repo  https://review.openstack.org/4253016:45
mordredclarkb: it works in simple testing - but you know, late in cycle EEK16:45
clarkbmordred: gotcha16:45
clarkbmordred: 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
clarkbis there some reason for putting it in a new file?16:46
pleia2annegentle_: yay! glad you like git.o.o :)16:46
*** pabelanger has quit IRC16:47
mordredclarkb: no reason at all - I just didn't think of it as something that was grouped with other jobs16:47
*** Bada has joined #openstack-infra16:47
mordredclarkb: since it's pretty specific to _only_ things that have a milestone-proposed16:47
clarkbmordred: but you put it in that job group :)16:47
mordredI did?16:47
mordreddammit16:47
clarkbmordred: https://review.openstack.org/#/c/41927/4/modules/openstack_project/files/jenkins_job_builder/config/openstack-publish-jobs.yaml16:48
clarkbmordred: I will leave a comment16:48
mordredclarkb: yeah. then no, I do not see any reason to not co-locate16:48
*** ruhe has joined #openstack-infra16:48
*** amotoki has quit IRC16:48
clarkbmordred: though I am leaning towards removing it from the group instead16:48
clarkbmordred: since as you say it is only for things with a milestone-proposed which is a subset of the things with openstack publish jobs16:49
mordredclarkb: I think I'd prefer that16:49
clarkbmordred: also the job is not a job template so it doesn't need to be in any group16:49
jeblairer16:51
jeblairi think it needs to be a template16:51
clarkbjeblair: it might need to be16:51
clarkbjeblair: if cacheing the git repo is important16:51
jeblairother wise if it's used for multiple projects it will share a git repo16:51
jeblairand the first time it runs for a different project it will be confused16:52
anteayaUtahDave is spinning up some vms for some salt testing16:52
clarkbjeblair: updating my review16:52
anteayahe 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
jeblairclarkb, mordred: ok i left a -1 review after all.  :)16:53
anteayaI'm all for learning myself, but I might slow hime down16:53
clarkbanteaya: I would like to be more involved, but don't have much time today16:53
anteayas/hime/him16:53
anteayaclarkb: okay16:53
anteayaso how about he goes ahead and we will ask questions as we can16:54
anteayathat okay with you, UtahDave?16:54
*** hashar has joined #openstack-infra16:56
UtahDaveSounds great!16:56
*** hashar is now known as hasharSF16:57
UtahDaveI think I have understand the basic workflow desired. I'll just need some info at the very end regarding the exact commands to execute16:57
*** derekh_afk has quit IRC16:57
anteayasounds good16:57
openstackgerritMonty Taylor proposed a change to openstack-infra/config: Add job to propose tag merges  https://review.openstack.org/4192716:57
hasharSFgood morning:)16:57
mordredclarkb, jeblair ^^ thanks. great catch16:57
anteayagood morning hasharSF16:58
hasharSFI am in SF for the whole week :-]16:58
anteayawonderful16:59
pleia2hasharSF: 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 IRC17:01
*** odyssey4me has quit IRC17:02
jeblairsay hi to the docs bootcamp, everyone!17:03
pleia2hello docs bootcamp! :)17:03
*** fifieldt has joined #openstack-infra17:03
pleia2sorry about the weather, it's not usually this hot ;)17:04
clarkbohai17:04
fifieldthi docs boot camp!17:04
*** boris-42 has joined #openstack-infra17:04
hasharSFpleia2: 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
clarkbpleia2: 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 drastic17:04
pleia2clarkb: yeah, the fog makes things chilly as you get closer to the ocean17:05
* pleia2 spent yesterday near the ocean17:05
*** openstackgerrit has quit IRC17:05
*** openstackgerrit has joined #openstack-infra17:05
anteayahello docs folks17:05
*** dcramer has joined #openstack-infra17:08
*** KLevenstein_away has joined #openstack-infra17:08
*** rev has joined #openstack-infra17:08
annegentle_hai! You guys crack me up!17:08
* fungi expresses his deepest respect for anyone who can stomach writing docs all the time17:08
*** fandikurnia02 has joined #openstack-infra17:09
clarkbpleia2: were you watching the boats?17:10
jog0clarkb: want to approve  https://review.openstack.org/#/c/45501/ now that it's not the weekend17:10
clarkbjog0: looking17:10
openstackgerritDan Prince proposed a change to openstack-infra/reviewday: Add SS results support for ceilo/heat.  https://review.openstack.org/4571417:11
clarkbjog0: done17:11
jog0clarkb: thanks17:11
pleia2clarkb: nah, they've been in the bay all summer, getting boring :) (LV cup prior to america's cup)17:12
openstackgerritA change was merged to openstack-infra/config: Move gate-tempest-devstack-vm-large-ops to silent  https://review.openstack.org/4550117:13
clarkbmordred: new patchset lgtm17:15
*** pabelanger has joined #openstack-infra17:16
*** fifieldt has quit IRC17:19
koolhead17mordred: Seems like all infra folks are working with you. Just got to know jog0 is with you as well now17:19
koolhead17:P17:19
clarkbprecise14 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-infra17:23
clarkbmordred: SpamapS: lifeless: safe to approve https://review.openstack.org/#/c/42513/2 ? it adds new gate jobs to tripleo projects17:25
anteayakoolhead17: jeblair and fungi are with the foundation17:25
koolhead17jeblair: ok17:26
UtahDaveShould 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
clarkbthat is a good question, I think salt master (or at least something not in Jenkins) should make the determination17:28
fungiUtahDave: i think the latter, for the moment at least17:29
clarkbJenkins is doing gating not orchestration17:29
fungiagree with clarkb17:29
*** Ryan_Lane has joined #openstack-infra17:29
fungithe less decision-making we leave in jenkins's hands now, the less we might have to rework later too17:29
fungianteaya: though jeblair and i did also work at hp with mordred before moving to the foundation17:29
UtahDaveok, I'll head in that direction, then.  Thanks!17:30
anteayayes17:30
*** dkehn has quit IRC17:30
anteayathat is true17:30
*** dkehn has joined #openstack-infra17:30
*** SergeyLukjanov has joined #openstack-infra17:30
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation  https://review.openstack.org/4571517:32
*** SergeyLukjanov has quit IRC17:33
*** pabelanger has quit IRC17:34
*** nijaba has quit IRC17:35
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation  https://review.openstack.org/4571517:36
*** pabelanger has joined #openstack-infra17:38
*** SergeyLukjanov has joined #openstack-infra17:39
*** cppcabrera has joined #openstack-infra17:40
openstackgerritA change was merged to openstack-infra/config: Add requirements gating jobs to tripleo  https://review.openstack.org/4251317:41
*** sarob has joined #openstack-infra17:42
*** cppcabrera has left #openstack-infra17:42
openstackgerritA change was merged to openstack-infra/config: New project: python-climateclient.  https://review.openstack.org/3570017:42
*** melwitt has joined #openstack-infra17:44
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Job-specific subst. in a job group's job list  https://review.openstack.org/4301317:50
*** ruhe has quit IRC17:52
*** moted has quit IRC17:53
*** moted has joined #openstack-infra17:53
*** jergerber has quit IRC17:56
*** HenryG_ has joined #openstack-infra18:01
*** nati_ueno has quit IRC18:04
*** nati_ueno has joined #openstack-infra18:05
*** HenryG has quit IRC18:05
*** Ajaeger1 has joined #openstack-infra18:08
*** danger_fo_away is now known as danger_fo18:08
*** nati_ueno has quit IRC18:09
*** zehicle has joined #openstack-infra18:10
* anteaya heads out for a walk18:11
*** mriedem has quit IRC18:18
clarkbmgagne: https://review.openstack.org/#/c/44685/18:19
mordredkoolhead17: that's right18:19
mgagneclarkb: checking18:19
mordredclarkb: I may have just eaten way too much taco for lunch18:20
koolhead17mordred: am not complaining sir. :D18:20
clarkbmordred: I had a chimichanga on Saturday that I am still recovering from18:20
*** mriedem has joined #openstack-infra18:20
fungigrease fried in more grease18:21
mordredclarkb: that sounds like a good chimichanga then18:21
mgagneclarkb: want me to approve once reviewed?18:21
clarkbmgagne: yes please18:21
clarkbmgagne: assuming you are happy with the current patchset18:21
mgagneclarkb: my comments were addressed18:21
mgagneclarkb: fun, this one was next in my list of reviews =)18:22
clarkbmordred: will you be in town next week?18:22
clarkbfor the tripleo thing18:22
*** jhesketh has quit IRC18:22
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Added SBT builder support  https://review.openstack.org/4468518:22
*** jhesketh has joined #openstack-infra18:23
*** marun has joined #openstack-infra18:26
*** mrmartin has joined #openstack-infra18:26
*** changbl has quit IRC18:27
*** mrmartin has left #openstack-infra18:28
*** mrmartin has joined #openstack-infra18:28
mrmartinclarkb: are u here?18:29
clarkbmrmartin: yes18:29
mrmartinclarkb: nobody:www-data could be a good owner for drupal files?18:29
mrmartinclarkb: I was thinking on this ubuntu user ownership going home from the office today, and I can agree with you18:30
*** changbl has joined #openstack-infra18:30
mrmartinI see two options: nobody:www-data or I can create a separate web user. What you think?18:30
clarkbtstevenson: 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 foreground18:30
clarkbmrmartin: nobody:www-data works for me18:31
*** dkehn_ has joined #openstack-infra18:31
clarkbmrmartin: or www-data:www-data?18:31
clarkbfungi: ^ any opinion on drupal file ownership?18:31
*** pabelanger has quit IRC18:32
pleia2I 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
pleia2I'd vote for separate user owning them except where www-data needs to write18:32
mrmartinwww-data have the group of www-data18:32
pleia2suppose it depends on what permissions the group has18:32
pleia2I seem to recall drupal being pretty good about telling you what it needs write access to18:33
mrmartinand I don't advise it because, it can the regular files have 0640, directories 0750, and writeable files/dirs  are 0660 and 077018:33
mrmartinit can / n/a18:33
fungikeep in mind that making files owned by "nobody" is no better than making them owned by root18:33
mrmartinso, maybe a separate user created just for this ownership, with www-data as a group?18:34
fungiarguably worse, since any process running as "nobody" then has write access to those files18:34
*** dkehn has quit IRC18:34
UtahDaveShould 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
fungiUtahDave: right now, we don't have any means to differentiate which servers should update based on changes to particular subsets of the project or projects18:35
mrmartinok, so could we agree on a separate user?18:35
clarkbmrmartin: separate user works for me as well18:35
UtahDavefungi: so for now just tell everyone to do a puppet run?18:35
fungiUtahDave: yep18:35
UtahDaveok, cool. It won't be hard to narrow that down in the future, if needed.18:36
fungimrmartin: what user account does drupal run as? root, or www-data, or something else entirely?18:36
fungiand which drupal files specifically?18:36
mrmartindrupal just a standard web site, nothing fancy18:36
mrmartinit is php18:36
clarkbUtahDave: 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 puppet18:36
mrmartinthe only requirement is that sites/default/files must be writeable and settings.php must r/o18:37
clarkbUtahDave: fungi: I think that cna be added later :)18:37
fungimrmartin: okay, so files which need to be writable by the php scripts for the site?18:37
mrmartinjust everything under sites/default/files. they have different permissions 0660 / 0770 for user:www-data18:37
mrmartinand 0640 / 0750 for everything else18:38
fungiclarkb: 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 high18:38
openstackgerritA change was merged to openstack-infra/reviewday: Add SS results support for ceilo/heat.  https://review.openstack.org/4571418:39
mrmartinjeblair: 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
fungimrmartin: 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 them18:41
fungiand for non-sensitive files www-data needs to read, root:root 0644 is usually good18:42
mrmartinok, it is easier to set everything else to 0640 instead of 064418:43
openstackgerritMonty Taylor proposed a change to openstack-infra/pypi-mirror: Rename jeepyb to pypi-mirror  https://review.openstack.org/4572618:44
openstackgerritMonty Taylor proposed a change to openstack-infra/pypi-mirror: Start building wheels in the mirror  https://review.openstack.org/4572718:44
openstackgerritMonty Taylor proposed a change to openstack-infra/pypi-mirror: Strip out the things we don't need from jeepyb  https://review.openstack.org/4572818:44
mordredclarkb: yes. will be here next week18:44
mordredor there18:44
mrmartinso, 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 IRC18:46
fungimrmartin: 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 others18:47
clarkbI am happy with root too I guess. ubuntu just felt weird and pidgeon holed us into deploying on images with that user18:47
mrmartinclarkb: it was my mistake, left the testing user there ;)18:47
clarkblifeless: 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
clarkblifeless: git review -d makes it easy to deal with18:48
mordredclarkb: https://review.openstack.org/#/c/45726/ - weird error18:48
mordredclarkb: I _just_ cloned from gerrit18:48
fungithe 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 exec18:49
clarkbmordred: zuul is using a new merge strategy18:49
clarkbmordred: possibly related to that? I would check the zuul debug log to see if there is any more verbose info there18:50
mordredclarkb: but those should be direct patches on top of the current tip that I just cloned18:50
mordredI will look there18:50
mordredjust saying18:50
clarkbmordred: right, I wonder if you are tickling a bug in zuul18:51
clarkbmordred: I would expect the debug log to show you a stacktrace or similar if so18:51
mordred2013-09-09 18:44:09,867 ERROR zuul.IndependentPipelineManager: Unable to find change queue for project openstack-infra/pypi-mirror18:52
*** vipul is now known as vipul-away18:52
*** vipul-away is now known as vipul18:52
clarkbmordred: that is one of those errors that isn't really an error. Anything else?18:52
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: Add documentation for XDG_CACHE_HOME  https://review.openstack.org/4572918:52
clarkbmordred: not having a job in one of the IndependentPipelines causes that error (eg silent)18:52
mordredclarkb: well, there does not seem to be reference to pypi-mirror in layout.yaml18:53
clarkbmordred: it is there, line 36918:54
clarkber line 369 in this checkout of config18:54
mordreddo. was on wrong branch :)18:54
lifelessclarkb: 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
lifelessclarkb: I'm waiting on feedback from jeblair on several patches to move forwards18:54
clarkblifeless: ok18:55
lifelessclarkb: (he may have given that overnight, haven't looked yet)18:55
clarkblifeless: I updated one of them that jeblair hadn't gotten to with what I thought jeblair was talking about18:55
*** gyee has quit IRC18:56
mordredclarkb: no tracebacks18:57
clarkbmordred: 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-infra18:59
*** dkehn_ is now known as dkehn19:00
*** mriedem1 has joined #openstack-infra19:04
openstackgerritStefano Maffulli proposed a change to openstack-infra/config: adding staging server for activity board  https://review.openstack.org/4299819:04
reedclarkb, that commit went in a new branch, not sure why :(19:05
*** mriedem has quit IRC19:07
*** Ajaeger1 has quit IRC19:07
openstackgerritJames E. Blair proposed a change to openstack-infra/zuul: Use NNFI scheduler algorithm  https://review.openstack.org/4533419:07
*** krtaylor has joined #openstack-infra19:08
reedand failed19:09
*** tjones has joined #openstack-infra19:09
*** nicedice has quit IRC19:11
*** nicedice has joined #openstack-infra19:11
openstackgerritKhai Do proposed a change to openstack-infra/config: link to jjb cache documentation in the handbook .  https://review.openstack.org/4573819:13
openstackgerritA change was merged to openstack-infra/config: Add py26 and py33 tests and mirror uploads.  https://review.openstack.org/4532319:13
clarkbfungi: 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 yaml19:13
*** tjones has quit IRC19:14
clarkbfungi: any opinion on how we should handle these sorts of things until the new yaml format happens?19:14
fungiclarkb: 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 stages19:16
*** ruhe has joined #openstack-infra19:16
openstackgerritA change was merged to openstack-dev/hacking: Redirect stderr for git rev-parse  https://review.openstack.org/4515019:17
fungibasically 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
fungithat way if the scripts override the local copies of data with what they find in the yaml if present, it makes it easy to transition19:18
clarkbfungi: so perfectly safe to modify the scripts directly for now?19:18
fungiclarkb: 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 merged19:19
fungiwe had a huge uptick of work around then (might have been around oscon) and then i ost track19:20
fungilost track19:20
SergeyLukjanovhi, I've started working on it, here is CR19:20
SergeyLukjanovhttps://review.openstack.org/#/c/37620/19:20
fungiSergeyLukjanov: right! that was you. i feel really bad it's been dragging out due to lack of review cycles19:20
SergeyLukjanovbut it depends on nonexistent projects yaml file format #219:20
SergeyLukjanovit's not completed yet I think, my bad, fully occupied by savanna activities now(19:21
*** sarob has quit IRC19:21
fungii 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 transition19:22
*** sarob has joined #openstack-infra19:22
SergeyLukjanovok, I'll try to update it soon19:22
fungia 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 it19:22
*** hasharSF has quit IRC19:23
tstevensonclarkb: Thanks for the help. I was able to get gerritbot going. I am now getting a PasswordRequiredException: Private key file is encrypted19:23
tstevensonclarkb: decrypt with openssl and same error19:23
fungiadding 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 first19:24
SergeyLukjanovfungi, yep, agreed19:24
SergeyLukjanovoh, I have a question about stackforge/savanna-image-elements, it looks like it's broken19:25
fungi#define "broken"19:25
SergeyLukjanovhttps://review.openstack.org/#/c/45575/19:25
* fungi looks19:25
SergeyLukjanovjenkins couldn't rebase changes19:25
openstackgerritA change was merged to openstack-infra/config: Add test jobs for RAX DFW and ORD regions  https://review.openstack.org/4535819:25
* fungi tests it19:25
SergeyLukjanovI think it could be backed by re pushing initial repo state19:26
clarkbTHis may be the same thing that hit mordred19:27
*** sarob has quit IRC19:27
clarkbwe may need to dig into the zuul logs and see what the repos look like in zuuls working dirs19:27
fungiyeah, 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 zuul19:28
clarkbfungi: or the difference in merge resolution19:28
fungibeen happening since at least the 8th. do we know how far back?19:29
SergeyLukjanovthis repo has been created several days ago19:30
SergeyLukjanovand asked mordred to push initial state to it twice19:30
clarkbSergeyLukjanov: I think the repo has an initial state, but zuul doesn't19:30
SergeyLukjanovso, looks like prev state has been cached19:30
SergeyLukjanovyep19:30
clarkbI am going to grab lunch can, help dig in mroe after19:32
fungiclarkb: 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 it19:33
SergeyLukjanovguys, what are you thinking about the following lp-related options format for projects yaml http://paste.openstack.org/show/46357/19:36
fungialso, 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 github19:36
SergeyLukjanov?19:36
fungiSergeyLukjanov: 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.erb19:38
SergeyLukjanovoops :)19:38
SergeyLukjanovforgot about the fact that we have two project.yaml :)19:39
fungidon't sweat it. we really shouldn't have two files named projects.yaml but that's a change for another day19:39
fungiit has already confused far too many people19:39
fungiclarkb: think i see it in the log... http://paste.openstack.org/show/46361/19:42
fungiNo item found with id 'origin/HEAD'19:42
*** tjones has joined #openstack-infra19:42
fungiand it's right19:43
fungihttp://paste.openstack.org/show/46362/19:44
fungimordred: ^19:47
fungiso that at least tells us *how* it's broken. now do brainstorm theories on *why*19:48
fungii wonder if shallow clones fail to create origin/HEAD19:49
fungier, not shallow clones (zuul doesn't anyway). you said lazy clones19:52
openstackgerritA change was merged to openstack-dev/hacking: hacking should conform with OS requirements  https://review.openstack.org/4564119:53
fungiand zuul.merger.Repo._ensure_cloned() just seems to rely on git.Repo.clone_from() so nothing terribly exciting there19:55
*** dprince has quit IRC19:55
*** ruhe has quit IRC19:58
*** gyee has joined #openstack-infra19:58
*** pcrews has quit IRC20:01
openstackgerritMarton Kiss proposed a change to openstack-infra/config: Add groups-dev.openstack.org node  https://review.openstack.org/4260820:01
*** boris-42 has quit IRC20:11
*** boris-42 has joined #openstack-infra20:13
*** tjones has quit IRC20:14
*** salv-orlando has joined #openstack-infra20:16
*** marun has quit IRC20:16
*** boris-42 has quit IRC20:21
*** pabelanger_ has quit IRC20:22
*** pabelanger has joined #openstack-infra20:22
sdaguettx: 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 IRC20:25
*** HenryG has joined #openstack-infra20:26
*** hashar has joined #openstack-infra20:31
openstackgerritPierre Rognant proposed a change to openstack-infra/jenkins-job-builder: Fix plot plugin support  https://review.openstack.org/4575420:40
*** thomasm has quit IRC20:41
clarkbfungi: weird20:41
clarkbfungi: any more info?20:41
openstackgerritSergey Lukjanov proposed a change to openstack-infra/jeepyb: Improve git2lp and is_direct_release behaviour  https://review.openstack.org/3762020:42
SergeyLukjanovit looks like it should now work ok with current review.project.yaml file format20:44
fungiclarkb: 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 it20:44
openstackgerritSergey Lukjanov proposed a change to openstack-infra/jeepyb: Map all savanna repos to savanna at launchpad  https://review.openstack.org/3762120:44
clarkbfungi: sounds good20:45
clarkbfungi: I am going to keep poking at code review, ping me if I need to look at zuul20:46
openstackgerritPierre Rognant proposed a change to openstack-infra/jenkins-job-builder: Fix plot plugin support  https://review.openstack.org/4575420:46
openstackgerritSergey Lukjanov proposed a change to openstack-infra/jeepyb: Map all savanna repos to savanna at launchpad  https://review.openstack.org/3762120:46
fungiclarkb: three repos affected... http://paste.openstack.org/show/46370/20:46
UtahDaveOn the Jenkins server, does the jenkins user have full sudo rights?20:47
fungithere are definitely other projects we've added before, between and after those which are not affected20:47
mrmartinclarkb: 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
clarkbUtahDave: I don't think so20:47
clarkbmrmartin: will do20:48
mrmartinthanks20:48
openstackgerritKhai Do proposed a change to openstack-infra/jenkins-job-builder: Add documentation for XDG_CACHE_HOME  https://review.openstack.org/4572920:48
UtahDaveclarkb: would it be possible to give Jenkins sudo rights to /usr/bin/salt-call  ?20:48
fungiUtahDave: is root access required to trigger salt notifications?20:48
UtahDaveYeah, the user needs to be able to read the minion's private key in order to send messages on the event bus20:49
fungihow about a dedicated jenkins slave where the jenkins user was granted read access to its minion private key?20:50
UtahDavefungi: 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 nice20:50
UtahDavefungi: that should work. Let me try that out on my test system here.20:50
fungiUtahDave: 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 machine20:51
UtahDaveRight. Ok, I'll see what I can do here.20:52
fungiif 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 slave20:52
UtahDavegood point.20:52
fungibut worth exploring how it can best be secured20:53
fungifor example, we might hardcode one and only one salt-call command which the jenkins user can run on the slave, parameters and all20:53
fungi(via a custom entry to sudoers)20:53
fungithat might be a simple workaround20:53
UtahDaveyeah, that's what I'm investigating right now. I think that may be our best bet.20:54
*** Bada has quit IRC20:54
*** sarob has joined #openstack-infra20:54
*** svarnau_ has joined #openstack-infra20:56
fungiand 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 configs20:56
*** svarnau has quit IRC20:58
*** mrmartin has quit IRC20:58
*** dcramer has quit IRC20:59
jog0jeblair: ping21:00
jog0https://review.openstack.org/#/admin/groups/204,members doesn't look right21:00
* jog0 is trying to create a 0.7.x branch 21:01
jog0of hacking21:01
clarkbjog0: you need to be a member of that group?21:01
clarkbjog0: I have added you to the group21:02
jog0clarkb: thanks21:02
jog0now to see if it works21:02
*** marun has joined #openstack-infra21:03
*** amotoki has joined #openstack-infra21:03
jog0that was easy21:03
*** emagana_ has joined #openstack-infra21:04
openstackgerritMathieu Gagné proposed a change to openstack-infra/config: Update doc about JJB cache location  https://review.openstack.org/4575821:04
fungiclarkb: 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 resuming21:05
fungiso biab21:05
clarkbfungi: ok21:05
pleia2mgagne: did you seek zaro's change https://review.openstack.org/#/c/45738/21:07
mgagnepleia2: unfortunately not21:07
mgagnepleia2: will abandon mine21:07
*** ericw has quit IRC21:07
mgagnepleia2: thanks for pointing that out21:07
pleia2sure21:07
UtahDavefungi: I did some testing with this sudoers entry: jenkins  ALL=(ALL) NOPASSWD: /usr/bin/salt-call event.fire_master*21:09
UtahDaveresults: http://pastebin.com/ijVfBuDh21:09
mgagnezaro: sorry, I "stole" your bug21:12
UtahDaveThat sudoers entry only allows the minion to send a message to the master. Nothing else.21:12
jog0getting 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,z21:15
*** dcramer has joined #openstack-infra21:15
zaromgagne: no problem.  i'll just assign it back to me.21:16
mgagnezaro: thanks for the follow-up on that bug, I lost the submitter's reply.21:17
*** dizquierdo has joined #openstack-infra21:18
pleia2so, 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 stuff21:25
pleia2now debugging an error she's getting when running the install_modules.sh script21:26
pleia2anyone have thoughts as to why we'd bump into this dependency issue? http://paste.openstack.org/show/46374/21:26
anteaya Puppet v2.7.2321:26
*** vipul is now known as vipul-away21:26
pleia2it occurs to me that we didn't include puppetlabs-haproxy when I set mine up21:26
pleia2so it could just be a dependency issue I didn't run into21:27
clarkblooking21:27
anteayais there a flag I can use when I run install_modules.sh that says don't bother with these modules?21:27
pleia2anteaya: I'm going to log in to the server and poke around a little too21:27
anteayaplease do21:27
jog0clarkb: as we talked abou the other day I took a stab at recheckwatch https://review.openstack.org/#/c/45511/21:27
anteayaI'll be hands off until you give the word21:27
anteayayou are in the puppet-master server, yes?21:28
pleia2anteaya: yeah21:28
anteayak21:28
clarkbanteaya: pleia2: I think this is a case of poor dependency resolution that was not a problem when our master installed because things already existed21:28
clarkbanteaya: pleia2: but is an issue when starting from scratch21:29
anteayaah21:29
pleia2yeah, that's the conclusion I'm starting to come to as well21:29
anteayaso your suggestion is install the dependencies?21:29
clarkbthough I just installed the modules on my new test etherpad server and that worked21:29
anteayahmmmm21:29
clarkbI think ripeinaar concat became puppetlabs-concat21:29
clarkbif you remove puppetlabs-concat, install the two moduels that complain about it then rerun install modules I would expect that to work21:30
*** nati_ueno has joined #openstack-infra21:30
anteayado you want to do that pleia2 since you are in there?21:31
anteayaor shall I?21:31
jeblairclarkb: https://review.openstack.org/#/c/44246/21:31
pleia2anteaya: you can do it, just wondering how puppetlabs-concat was pulled in21:31
anteayapleia2: okay, should I wait for you or go ahead?21:32
pleia2anteaya: go ahead21:32
anteayak21:32
*** vipul-away is now known as vipul21:34
jeblairclarkb: also https://review.openstack.org/#/c/44496/21:34
clarkbjeblair: reviewing, thanks21:35
*** nati_uen_ has joined #openstack-infra21:36
jeblairclarkb: no rush; just pointing out things that are ready for your attention that you may not have seen as i'm going through21:36
clarkbjeblair: yup its helpful. I think I can approve both of them21:36
clarkb(and have approved the first)21:36
anteayaso the way forward is `rm -rf /etc/puppet/modules/concat` ?21:37
*** danger_fo is now known as danger_fo_away21:37
pleia2anteaya: no, you'll want to use the puppet way, hang on21:37
anteayak21:37
pleia2puppet module uninstall or remove, let me check...21:37
anteayaah okay21:37
pleia2sudo puppet module list21:38
clarkbanteaya: pleia2: you might also get around it by changing the order of array elements in install_modules.sh21:38
pleia2will show you them all21:38
pleia2then: puppet module unistall puppetlabs-concat21:38
clarkbI am not sure if bash arrays return a deterministic key order when iterating over the keys21:38
anteayauniinstall21:38
pleia2uninstall21:38
pleia2typing :)21:38
anteaya:D21:38
*** mriedem1 has quit IRC21:38
*** ArxCruz has quit IRC21:39
openstackgerritA change was merged to openstack-infra/config: Run tests against pypy for pbr and hacking  https://review.openstack.org/4424621:39
*** MarkAtwood has joined #openstack-infra21:39
anteayaI'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 enough21:39
pleia2second suggestion was more for a long term solution21:39
anteayaah okay21:40
*** hashar has quit IRC21:40
openstackgerritA change was merged to openstack-infra/jeepyb: Add options to manage-projects  https://review.openstack.org/4449621:41
*** ArxCruz has joined #openstack-infra21:41
clarkbjeblair: 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 expected21:41
*** dkranz has quit IRC21:42
clarkbjeblair: fungi: it is running in a root screen if you are curious21:42
clarkbfigured I could let that run while I reviewed stuff21:42
jeblairclarkb: cool!21:43
pleia2anteaya: 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
anteayapleia2: yes21:44
anteayagood call21:44
anteayaI will see what happens next thanks21:44
anteayait ran quiet21:45
mroddendoes anyone have a good grasp of how requirements.txt between oslo and the other projects that copy code from it works?21:45
mroddenis 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 IRC21:48
*** vipul is now known as vipul-away21:49
clarkbmrodden: I think you need to update requirements.txt manually21:49
clarkbI believe I had to do this last time I did an oslo sync21:50
mroddenclarkb: yeah looking through the update.py... doesn't look like it does anything special with requirements21:50
mroddenclarkb: thanks21:50
*** hashar has joined #openstack-infra21:52
*** svarnau has joined #openstack-infra21:52
anteayawhile pleia2 is out I am going to take the opportunity for another quick lap - hey look at that Sun!21:53
*** pcm_ has quit IRC21:54
*** svarnau_ has quit IRC21:55
mgagneanteaya: which sun? all grey and cold here =(21:55
*** sdake has quit IRC22:00
*** sdake has joined #openstack-infra22:00
*** SergeyLukjanov has quit IRC22:01
*** hemnafk is now known as hemna22:02
*** vipul-away is now known as vipul22:03
clarkbjeblair: fungi: https://review.openstack.org/#/c/45464/ was that not approved for somereason?22:04
clarkbthe number of +2s without approval makes me afraid to hit the go button22:04
mroddenthats not a git sha22:06
mroddenthats a change id22:06
mroddenthey are different?22:06
*** svarnau has quit IRC22:07
clarkbmrodden: change ids and git shas are different22:07
*** fifieldt has joined #openstack-infra22:07
mroddenbut apparently they are queried teh same way in the URLs22:07
mroddeninteresting22:07
annegentle_why doesn't this build? https://jenkins.openstack.org/view/Openstack-manuals/job/openstack-admin-guide-cloud/22:07
clarkbmrodden: change IDs are prefixed with an I followed by a sha1 whereas git shas are just a sha1122:08
*** alexpilotti has quit IRC22:08
*** burt has quit IRC22:08
mroddenclarkb: yeah, was getting worried, but apparently the URL it throws them into results in a query on either22:08
openstackgerritA change was merged to openstack-infra/config: Poke a hole through the proxy for cluster health  https://review.openstack.org/4550022:09
clarkbannegentle_: we don't run that job on that jenkins master anymore22:09
clarkbannegentle_: you will want to check jenkins01 and jenkins0222:09
jeblairclarkb: no aprv from me due to lack of babysitting time22:10
clarkbjeblair: ok, I am in the same boat, tomorrow will probably be better22:11
fifieldtjeblair, that one looks like the other translation jobs which means according to my understanding of the scripts it should "just work"22:11
fifieldtthat was 4399122:11
*** weshay has quit IRC22:11
jeblairfifieldt: ah thanks; i was confused by receiving your reply to my message via a different irc transport protocol!22:12
openstackgerritA change was merged to openstack-infra/config: Publish hacking docs to docs.openstack.org  https://review.openstack.org/4549822:13
*** tjones has joined #openstack-infra22:13
*** tjones has left #openstack-infra22:13
openstackgerritAntoine Musso proposed a change to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation  https://review.openstack.org/4571522:15
*** pentameter has quit IRC22:16
*** sarob has quit IRC22:16
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Add usage info to JJB documentation  https://review.openstack.org/4571522:16
jog0if anyone has afew minutes lookign for some help with https://review.openstack.org/#/c/45511/22:16
*** sarob has joined #openstack-infra22:16
hashardoh22:17
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Add documentation for XDG_CACHE_HOME  https://review.openstack.org/4572922:17
clarkbhashar: was something wrong with 45511? or was that dupe?22:18
clarkber 4571522:18
openstackgerritA change was merged to openstack-infra/config: Grant tripleo-ptl tags on tripleo-image-elements  https://review.openstack.org/4551822:18
clarkbI am confused did I approve a new patchset after reviewing the old?22:18
clarkboh nevermind I approved a different change22:19
*** sarob has quit IRC22:19
openstackgerritA change was merged to openstack-infra/config: Remove extra cloud-admin in URL of End User Guide  https://review.openstack.org/4564022:19
hasharclarkb: 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-infra22:19
hasharclarkb: you are probably talking about something completely unrelated :-D22:19
hasharbut apparently that jjb change is breaking the compare-xml job :(22:20
clarkbhashar: yup I was talking about something else, I confused myself, sorry22:20
lifelesshi22:21
hashardoes anyone know from where the python dependencies are downloaded in Jenkins ?22:21
marunwho do I talk to about jenkins-sudo-grep.sh?22:21
hasharit can't download the pip module  sphinxcontrib-programoutput :/22:21
lifelessclarkb: see #tripleo; something very weird on tie checks22:21
clarkbjog0: 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
lifelessclarkb: 2 days ago we had no errors22:23
clarkblifeless: oh, this is the thing I was told was safe to do by spamaps22:23
clarkblifeless: mordred proposed pep8 run against that repo, I checked with SpamapS this morning and he said go for i22:23
clarkbt22:23
lifelesshell no22:24
lifelessit's not a python repo.22:24
lifelessdon't run pep8 against it22:24
jog0lifeless: haha22:24
lifelessSpamapS: mordred: ^22:24
jog0clarkb: yeah I was having problems getting recheck running locally22:25
jog0was wondering if there is a trick there  I can use22:25
lifelessclarkb: it would be nice if 'jobs to run' changes did a check against trunk as their pre-merge gate.22:25
lifelessclarkb: should I file a wishlist bug to that effect ?22:26
clarkbjog0: dropped what I said here in the change review so that we have it recorded.22:26
clarkbjog0: what in particular isn't working?22:26
*** changbl has quit IRC22:26
lifelessclarkb: and whats the fastest way to unbreak tie?22:26
clarkblifeless: you can, it will end up being semi tricky to orchestrate that22:26
clarkblifeless: also sometimes that is the desired effect (that stuff fails until fixed)22:26
lifelessclarkb: we can debate the merit of that another time :)22:27
clarkblifeless: fastest way to unbreak it is to revert just the part of that change that is failing22:27
clarkblifeless: we did it with pbr earlier when it was having so much trouble22:27
lifelessclarkb: it's an openstack-config change, right ?22:27
clarkblifeless: we forced pbr to pass a test suite before any other new potentially bad stuff could merge22:27
clarkblifeless: I will propose the partial revert22:27
lifelessclarkb: thank you22:27
*** mrodden has quit IRC22:27
*** nati_ueno has quit IRC22:27
*** prad_ has quit IRC22:28
clarkblifeless: should it run the -requirements job? that was the other part of mordred's change22:29
*** portante is now known as portante|afk22:29
*** jhesketh_ has joined #openstack-infra22:29
*** nati_uen_ has quit IRC22:30
lifelessclarkb: no22:30
*** nati_ueno has joined #openstack-infra22:30
*** thedodd has quit IRC22:30
lifelessclarkb: it has no requirements, test-requirements22:30
*** sarob has quit IRC22:31
clarkblifeless: thanks22:31
lifelessclarkb: 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-infra22:31
jog0clarkb: when i run it nothing happens22:31
lifelessclarkb: any non-trivial python in it would be a bug, and should live in a separate dedicated real-python-repo22:31
jog0it just exits22:32
anteayamgagne: :(22:33
openstackgerritClark Boylan proposed a change to openstack-infra/config: Partial revert of 131c5bf474dd25121832ad2e470525894c004227  https://review.openstack.org/4578722:33
clarkblifeless: ^22:33
clarkbjog0: even when run in the foreground?22:33
*** amotoki has quit IRC22:34
clarkbjog0: 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 pems22:34
clarkbbut foreground should work22:34
anteayasaw 2 pileated woodpeckers22:34
lifelessclarkb: did mordred add pep8 and requirements to a bunch of tripleo things?22:34
clarkblifeless: 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 job22:35
*** sarob has quit IRC22:36
jog0with -n22:36
jog0brb22:36
lifelesstripleo-heat-templates is also not python22:37
lifelessclarkb: did he add to it as well?22:37
lifelessand the incubator22:38
clarkblifeless: heat-templates was left alone22:38
clarkbincubator as well22:38
lifelesscool22:38
lifelesseverything else should be fine22:38
clarkbok22:38
*** dizquierdo has left #openstack-infra22:40
fungiclarkb: 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 camp22:40
fungier, i guess he just said as much (that'll teach me to respond while still reading scrollback)22:41
jeblairfungi: you say that every time. :P22:41
openstackgerritClark Boylan proposed a change to openstack-infra/config: Use bup to backup review-dev to an offsite server  https://review.openstack.org/4578822:42
openstackgerritClark Boylan proposed a change to openstack-infra/config: Use bup to backup review.o.o to an offsite server  https://review.openstack.org/4578922:42
*** KennethWilke has quit IRC22:42
clarkbjeblair: fungi ^ those are WIP becuase I haven't done the manual steps yet, but that will happen after I confirm wiki.o.o is good22:43
fungijeblair: good point. clearly i do not learn from this experience22:43
clarkbstill pulling the backup tar 14G and counting22:43
*** krtaylor has quit IRC22:45
anteayaso 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 it22:45
uvirtbotLaunchpad bug 6507 in gnome-system-tools "network-admin can't define several locations" [Medium,Fix released] https://launchpad.net/bugs/650722:46
anteayaso... how do I tell if puppet-master is up?22:46
lifelessanteaya: the ssl port for it is open to anyone22:46
clarkbanteaya: uh, puppet apply won't start a mater for you. Is local.pp a hacked up puppetmaster.pp?22:46
clarkbanteaya: and it runs in apache as a passenger app22:46
lifelessanteaya: but you need a cert to get it to do stuff22:47
*** lcestari has quit IRC22:47
lifelessfungi: oh hi22:47
lifelessfungi: wonder if you could counter-sign clarkb's partial revert to fix tie?22:47
anteayalocal.pp http://paste.openstack.org/show/46381/22:48
clarkbanteaya: check if apache is running, and you can try talking to it if so22:48
fungiUtahDave: thanks, that sudoers entry looks safe enough22:48
anteayait isn't22:48
anteayaI will get the apache stick22:48
UtahDaveok, thanks, fungi22:49
fungilifeless: sure, i read the discussion. will check and approve if it looks good22:49
anteayaclarkb: is it possible for install_puppet.sh and install_modules.sh to run and apache not be installed?22:50
clarkbanteaya: yes, but your local.pp should install apache22:51
anteayameaning I need to edit my local.pp so that installs apache?22:51
*** flaper87 is now known as flaper87|afk22:51
anteayabecause there is no apache installed on my system right now, not one I can find22:52
*** reed has quit IRC22:52
clarkbanteaya: no, I think the existing manifest should do it for you22:52
anteayareally?22:52
*** reed has joined #openstack-infra22:52
anteayathen apache is hiding somewhere that I can't find22:52
openstackgerritA change was merged to openstack-infra/config: Partial revert of 131c5bf474dd25121832ad2e470525894c004227  https://review.openstack.org/4578722:52
anteayaapt-cache policy apache2 says it isn't installed22:52
anteayalifeless: thanks your cert suggestion is still hovering in my mind22:53
clarkbanteaya: is puppet-master installed? it is possible that the puppetmaster.pp doesn't do everything22:54
anteayaah okay22:55
clarkbjeblair: fungi: I have successfully pulled one of the sql backups out of bup backup tar on wiki.o.o22:55
*** reed has quit IRC22:55
clarkbI haven't restored the DB in mysql but glancing at the file it looks goo22:55
fungiawesome22:56
clarkbI am going to do the manual setup for review-dev now22:56
fungiassuming it's good and not merely goo22:56
anteayapuppet-master isn't installed: http://paste.openstack.org/show/46382/22:56
jeblairi'm okay with goo22:56
lifelessgrey goo! thermal end to the world!22:56
anteayahuh, I was working from the belief that install_puppet.sh and install_modules.sh installed puppet-master, looks like it doesn't22:56
lifelessanteaya: install_puppet installs the puppet client agent; install_modules makes the referenced modules usable AIUI22:57
anteayait installs puppetlabs-apache22:57
clarkbjeblair: do I need to do this on both servers now? I think you had mentioned spinning up a new server in HP land22:57
anteayaokay22:57
anteayaI don't know as I even need a puppet-master, I just need enough for this puppet-dashboard to do something22:58
anteayalifeless: okay22:58
*** MarkAtwood has left #openstack-infra22:59
anteayalifeless: hmmmm, okay so since I have run install_modules.sh those modules theoretically should be useable22:59
anteayaonce i figure out how to use them22:59
openstackgerritA change was merged to openstack-infra/jeepyb: Adds <service>-api to map to openstack-api-site in Launchpad  https://review.openstack.org/4532423:00
clarkbjeblair: 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
clarkbjeblair: or is ~.23:04
clarkbsufficient?23:04
clarkbwill ^D work?23:06
jeblairclarkb: not sure?  try ^D, then ^C, then ~. in that order?23:07
*** vipul is now known as vipul-away23:08
jeblairclarkb: 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 it23:08
jeblairclarkb: i have not spun up a replacement23:08
*** mestery_ has joined #openstack-infra23:09
anteayaI updated http://paste.openstack.org/show/46385/ with the contents of 00-puppet.pref23:09
clarkbjeblair: ok, I will ignore it for now23:10
clarkb^D seems to have worked23:10
uvirtbotclarkb: Error: "D" is not a valid command.23:10
clarkbhttps://review.openstack.org/#/c/45788/ is ready for review23:11
*** rnirmal has quit IRC23:12
*** mestery has quit IRC23:13
openstackgerritClark Boylan proposed a change to openstack-infra/config: Clarify bup server setup directions.  https://review.openstack.org/4579123:13
clarkbreview-dev should be ready23:13
jeblairclarkb: cool, change lgtm23:14
clarkbI need to step out for a bit, will probably get to review.o.o tomorrow after bug day23:14
fungiclarkb: 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 after23:14
clarkbjeblair: fungi: maybe we should just try deleting the repos in zuuls work dir? zuul should recreate them23:15
clarkbat least as a possible workaround?23:15
jeblairclarkb, fungi: wha?23:15
jeblairit sounds like you're talking about a possible zuul bug?  what's up?23:15
fungii can manually add the ref for that matter (all it does is symbolically point at the default branch on a bare remote)23:16
clarkbjeblair: yes possible zuul bug23:16
fungijeblair: several projects were recently cloned in zuul's repo cache missing an origin/HEAD ref23:16
*** KLevenstein_away has quit IRC23:16
fungicausing the Repo.reset() to fail23:16
clarkbok stepping out biab23:16
fungijeblair: the offending three are openstack-infra/pypi-mirror, stackforge/fuel-main and stackforge/savanna-image-elements23:17
fungiall seem to have been on or since september 3rd23:17
anteayaI 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
jeblairfungi: do you have error messages handy?23:18
fungijeblair: pasted earlier. just a sec and i can grep them back out of my log23:18
fungijeblair: the error when it tries to merge candidate changes is http://paste.openstack.org/show/46361/23:19
*** fifieldt has quit IRC23:20
*** vipul-away is now known as vipul23:20
jeblairfungi: hrm.  i have to leave now.23:21
*** sarob has joined #openstack-infra23:21
fungijeblair: no worries. i'll keep poking at it23:21
jeblairthanks.  good luck23:21
fungithanks back!23:21
*** dcramer has quit IRC23:24
*** rev has quit IRC23:25
*** sarob has quit IRC23:25
fungias 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 worked23:26
jog0getting further with the recheck test23:27
fungidid the same to stackforge/fuel-main now so they won't be surprised23:28
fungiand 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-infra23:40
*** afazekas is now known as afazekas_zz23:43
*** erfanian has joined #openstack-infra23:48
*** dims has quit IRC23:49
*** gyee has quit IRC23:50
*** vipul is now known as vipul-away23:55
*** vipul-away is now known as vipul23:55

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