Tuesday, 2014-07-01

*** wenlock has quit IRC00:00
anteayaclarkb: okay00:00
*** msabramo has joined #openstack-infra00:01
pleia2re: ansible support, I think we're stuck on this one: https://review.openstack.org/#/c/87821/17 (and I don't know the answer)00:01
*** nati_uen_ has quit IRC00:01
anteayaclarkb: and I have commented back00:05
*** anode has quit IRC00:11
*** msabramo has quit IRC00:12
*** zz_dimtruck is now known as dimtruck00:12
*** r1chardj0n3s is now known as r1chard_afk00:13
*** annegent_ has joined #openstack-infra00:15
*** zigo has quit IRC00:17
*** zigo has joined #openstack-infra00:21
*** annegent_ has quit IRC00:23
*** msabramo has joined #openstack-infra00:23
*** markmcclain has quit IRC00:24
*** gangil has quit IRC00:24
*** markmcclain has joined #openstack-infra00:26
*** gokrokve has quit IRC00:26
*** markmcclain has quit IRC00:27
*** markmcclain has joined #openstack-infra00:27
*** praneshp has quit IRC00:27
clarkbanteaya: thank you00:28
clarkbanteaya: I don't think jeblair has changed his mind. So that lgtm00:28
clarkbmattoliverau: re comment on https://review.openstack.org/#/c/87821/ does my comment help clear it up?00:31
anteayaclarkb: k thanks00:32
*** zigo has quit IRC00:35
*** pradk has quit IRC00:35
anteayaclarkb: could you walk me through rebasing this? https://review.openstack.org/#/c/101361/00:36
anteayausing teh button on the gui failed00:36
anteayaI have updated the remotes00:36
anteayaand am in the branch00:36
anteayaI have rebased before and blown away my changes00:36
clarkbanteaya: next step is `git rebase -i master` assuming you updated your local master00:36
anteayaI'd like to avoid that this time00:36
anteayaI updated my local master yes00:36
clarkbthis will open an editor with the list of changes we are rebasing. should be just one line with that change00:37
clarkbif it is just that one line, save the file and quit, this starts the rebase00:37
anteayaon master I had run `git remote update` and then `git pull --ff-only origin master`00:37
clarkbyup00:37
anteayaokay I will run git rebase -i master00:37
clarkbanteaya: run that command with your change checked out00:37
jesusaurusclarkb: if both foo and foo.gz exist, then that find command will ignore foo.gz but will try to compress foo to foo.gz (and prompt about overwriting foo.gz) aiui00:37
clarkbthat tells git to rebase the current thing (your change) onto master00:37
clarkbjesusaurus: gotcha, so uh why not delete foo?00:38
anteayayes I have done that, from the change branch I have run git rebase -i master00:38
jesusaurusclarkb: are we sure foo.gz is up-to-date?00:38
anteayaI am in an editor00:38
clarkbjesusaurus: in our case yes00:38
clarkbanteaya: cool, just double check it has one line at the top for your change and no othercommits then save and quit00:39
*** sarob has quit IRC00:39
clarkbjesusaurus: these are logs in job run specific dirs00:39
anteayathis is what I see: http://paste.openstack.org/show/85191/00:39
clarkbjesusaurus: those logs should only ever be written once00:39
jesusaurusah, true00:39
clarkbanteaya: yup that looks correct, go ahead and save and quit00:39
jesusaurusi was missing the context of what was being compressed00:39
anteayaclarkb: k00:39
*** sarob has joined #openstack-infra00:39
*** salv-orlando has quit IRC00:40
jesusaurusclarkb: then yeah, we can just delete foo00:40
clarkbanteaya: you will probably get a message that says blah blah conflicts00:40
*** GheRivero is now known as GheRivero_afk00:40
clarkbanteaya: at this point do `git status` and it will show you the files that have conflicts00:40
anteayaclarkb: I was in vi and did :wq and got: http://paste.openstack.org/show/85192/00:40
clarkbanteaya: yup that is what I expect, next step is `git status` to see which files have conflicts00:41
anteayakk00:41
anteayahttp://paste.openstack.org/show/85193/00:41
jesusaurusclarkb: how long does it take for upstream to run nodepool image-update? does your initial puppet run also take forever and a day?00:42
clarkbanteaya: so for each file in the unmerged paths list. open that file. edit it to resolve conflicts. Basically make the file look like you want it to be in the resulting commit (don't forget to remove the git annotation tags like >>>>>>>>>>> HEAD or whatever)00:42
anteayakk00:42
clarkbanteaya: once you have edited the files so that they look like you want them to. Do git add filename00:42
clarkbdo that for each filename00:42
anteayaokay00:42
clarkbthen once all of that is done do git rebase --continue00:42
clarkbjesusaurus: it takes about an hour iirc00:43
jesusaurusclarkb: thanks00:43
clarkbjesusaurus: initial puppet run for non devstack slaves can be slow00:43
clarkbbecause all the packages and db stuff00:43
anteayaclarkb: alright, I'll be a while00:43
clarkbanteaya: np00:43
*** sarob has quit IRC00:44
*** marcoemorais has quit IRC00:45
*** marcoemorais has joined #openstack-infra00:45
clarkbanteaya: feel free to put up a file if you want an example of how I would resolve one. But once you figure out what git is telling you it should be straightforward00:45
clarkbone chunk is old, one chunk is new and you have to make a decision on what the next version should look like00:45
*** zigo has joined #openstack-infra00:47
anteayaclarkb: I screwed up, because I forgot to bring down the latest patchset of that series (which came from dhelmann) before I began00:47
anteayaright, I am seeing that00:47
anteayaclarkb: how do I abort this rebase?00:47
anteayaor can I at this point?00:47
*** marcoemorais has quit IRC00:49
clarkbyou can00:49
clarkbanteaya: just do `git rebase --abort`00:49
*** marcoemorais has joined #openstack-infra00:49
anteayaclarkb: k00:49
clarkbthat will put you back to where you started00:49
anteayagreat00:49
clarkband you can update your branch and start again00:49
anteayaright00:49
anteayanow I do git review -d patchset,patchnumber00:50
anteayabut will that take me to a new branch?00:50
clarkbit might00:51
clarkbyou can also leave off patchnumber and it will do the latest patch00:51
mattoliverauclarkb: thanks, I've changed my vote, but also gave some future suggestions if we start using ansible for more complicated tasks. Not for that change, but just from what I've learned from my somewhat limited experience with ansible :)00:51
anteayakk00:52
clarkbmattoliverau: thanks00:52
*** msabramo has quit IRC00:52
anteayayeah, on a new branch have the latest patchset00:53
anteayatrying again00:53
*** dprince has joined #openstack-infra00:53
*** marcoemorais has quit IRC00:55
*** r1chard_afk is now known as r1chardj0n3s00:56
*** dkranz has joined #openstack-infra00:56
*** markmcclain has quit IRC00:56
*** HenryG has quit IRC00:57
*** bdpayne has quit IRC00:57
*** HenryG has joined #openstack-infra00:59
*** msabramo has joined #openstack-infra01:00
*** jaypipes has joined #openstack-infra01:05
*** gokrokve has joined #openstack-infra01:05
*** dstanek is now known as dstanek_zzz01:06
*** gokrokve_ has joined #openstack-infra01:07
*** gokrokve has quit IRC01:10
ianwdoes the infra puppet scripts support puppet 3?01:12
*** msabramo has quit IRC01:13
clarkbianw: the slave bits do01:17
clarkbianw: the other bits probably don't01:17
*** yamahata has joined #openstack-infra01:18
*** thingee_ is now known as thingee01:19
*** dimtruck is now known as zz_dimtruck01:19
ianwclarkb: ok, so i should just verify openstack_project::single_use_slave for centos ATM?01:20
clarkbianw: yes01:21
clarkband it should work as far as puppet3 is concerned01:21
ianwclarkb: ok.  one thing i noticed that will bite is https://tickets.puppetlabs.com/browse/PUP-1082 ... puppet3 assumes that if on redhat, pip is called "pip-python" (the pkg name)01:23
openstackgerritAnita Kuno proposed a change to openstack-infra/config: Bring some uniformity to olso acl files  https://review.openstack.org/10136101:23
clarkbianw: I think we handle that alredy01:24
clarkbianw: iirc dprince added a symlink for that in our manifestse01:24
*** nosnos has joined #openstack-infra01:24
ianwclarkb: hmm, ok, i was seeing that issue running site.pp on centos7, i'll dig further01:24
anteayaclarkb: thank you01:26
anteayait was tedious01:26
anteayaI hope I got it right01:26
anteayadhellmann: the last patchset is mine now, so you can review: https://review.openstack.org/#/c/101361/01:26
dprinceianw: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/pip/lib/puppet/provider/package/pip3.rb01:27
dprinceianw: not sure that will help you out (haven't seen your error) but that is what I did a while back01:28
ianwdprince: hmm, ok -- on centos7 i'm seeing puppet use /usr/share/ruby/vendor_ruby/puppet/provider/package/pip.rb01:29
*** chuckC has quit IRC01:29
dprinceianw: with what puppet version?01:30
*** HenryG has quit IRC01:30
ianwdprince: 3.6.2 as installed by https://review.openstack.org/#/c/103458/01:30
*** HenryG has joined #openstack-infra01:30
ianwdprince: oh, hang out, that is pip3?01:32
dprinceianw: yes01:33
ianwdprince: so this is just "pip", but yeah, same-ish problem01:33
dprinceianw: Ah. So yeah. We aren't overwriting that provider so the default puppet 'pip' provider would get used.01:34
dprinceianw: honestly I'm a bit disappointed we haven't gotten an upstream puppet symlink/fix for this yet. But... you may just have to put a symlink in to make things happy01:34
*** bdpayne has joined #openstack-infra01:34
dprinceianw: was that your take on this?01:35
ianwdprince: does osfamily "RedHat" not match fedora? (googling now :)01:36
*** dstanek_zzz is now known as dstanek01:36
dprinceianw: osfamily should. Yes01:37
dprinceianw: distro would not though01:37
dprinceianw: See what we do for Fedora already in install_puppet.sh. We simply create the symlink in bash...01:38
dprinceianw: if you do the same on Centos 7 I think you may be set01:38
ianwdprince: and that answers my question of why this works on f20 and not centos7 :)01:38
ianwdprince: in terms of upstream https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/package/pip.rb#L3801:39
ianwdprince: do you think it's reasonable to propose a change that if 'pip' exists, someone must have gone to the effort of get-pip.py, so use it?01:40
anteayayup, I called it01:41
anteaya4th night in a row for fireworks01:41
dprinceianw: I'm not sure that belongs in Puppet proper (now that I remember how all this works)01:41
dprinceianw: this is certainly a finer point to our setup... I mean ideally we could just use a packaged version of pip and then puppet would work. But we don't... so symlinking it is probably fine01:42
*** moted has quit IRC01:44
*** jyu has joined #openstack-infra01:45
*** liusheng has quit IRC01:45
*** gokrokve_ has quit IRC01:45
ianwdprince: so https://tickets.puppetlabs.com/browse/PUP-1082 is invalid?01:47
*** timrc is now known as timrc-afk01:47
nibalizerclarkb: i tend to agree that the -not -name \*\[.-\]gz should keep already gziped files out01:48
ianwdprince: the symlink works, but it seems that we should either propose a fix or just say !packaged pip is always going to be an error (and probably enhance the err msg upstream to save people trying to figure out what the issue isse)01:48
dprinceianw: I don't think it is invalid. Improving things for the future would be great. Perhaps link that ticket into the install_puppet.sh script so we can loop back and check it again?01:49
dprinceianw: I haven't had the bandwidth to push puppet patches for all the wishlist items I find01:50
dprinceianw: but what you say would certainly be an improvement. The ability to explicitly pass in the location would be cool.01:51
dprinceianw: for that matter, we could inline our own pip.rb provider today and just use it. I don't see that as being as maintainable as having it upstream in puppet but we could certainly have our way with it.01:52
dprinceianw: if we did that probably best to rename it though to something unique.01:52
dprinceianw: our_pip, my_pip, me_pip? You'd have to change all the modules to use that instead of 'pip' too01:53
ianwdprince: i think the symlink is probably better than our own provider, now i understand.  i'll expand the comment a bit.  i doesn't seem like it would be hard to override that from an env variable, i'll see what i can do01:53
ianwi only speak pidgin ruby :)01:54
*** gfa has quit IRC01:54
dprinceianw: :). Agree w/ you on the symlink. Good luck01:54
*** gfa has joined #openstack-infra01:55
ianwdprince: thanks.  i think rhel7/centos7 is in pretty good shape then, i'm just going to revert to fresh and try it all again01:55
dprinceianw: awesome, good work01:56
*** devkulkarni has joined #openstack-infra01:59
*** liusheng has joined #openstack-infra02:01
*** wenlock_ has joined #openstack-infra02:09
*** adalbas has quit IRC02:10
*** yaguang has joined #openstack-infra02:11
*** annegent_ has joined #openstack-infra02:18
*** hdd_ has joined #openstack-infra02:22
*** annegent_ has quit IRC02:23
*** jyu has left #openstack-infra02:23
*** jyu has joined #openstack-infra02:23
*** bcrochet is now known as bcrochet|g0ne02:25
*** devkulkarni has quit IRC02:30
*** mmaglana has joined #openstack-infra02:30
*** nati_ueno has joined #openstack-infra02:33
*** enikanorov has joined #openstack-infra02:33
*** devkulkarni has joined #openstack-infra02:35
*** enikanorov_ has quit IRC02:38
*** zhiyan_ is now known as zhiyan02:38
mordredianw: what is it we did?02:42
mordreddprince: also, no one should EVER use a packaged version of pip02:42
mordreddprince: EVER02:42
mordredthey're all broken02:42
mordredianw: ^^ you too02:43
ianwmordred: what do you mean by "we did"?02:43
dprincemordred: my point was more along the lines of that is what puppet is catering for02:43
*** homeless has quit IRC02:43
mordredwhat is - packaged pip?02:43
dprincemordred: but sure, I understand your sentiment02:43
mordrednod02:43
mordredI mean, believe me - packaged python works 10000000* better than pip python02:44
dprincemordred: packaged pip is -> expload things into my system dirs02:44
mordredbut if you're going to puppet install things using the pip provider, there is no reason to not get pip using pip instead of getting pip using packages02:44
*** homeless has joined #openstack-infra02:44
dprincemordred: given your comments you are making me lean towards us having our own puppet provider for pip (like we do for pip3)02:45
mordreddprince: yup. I started on one of those02:45
mordreddprince: it's in use interally at hp, in fact, because it has to be because the puppet pip provider doesn't do proxies right02:46
mordreddprince: but when I tried to land it in infra it broke and I haven't gone back to figure it out yet02:46
*** dims_ has quit IRC02:46
dprincemordred: ah02:46
mordreddprince: otoh - dstufft assures me that installing stuff system-wide with pip is a bug and should never be done02:46
mordredthat you should either install by packages ...02:46
mordredor you should use a virtualenv02:47
dstufftwell02:47
mordreddstufft: I'm speaking for you now :)02:47
dstufftthat's true for the common case02:47
*** crc32 has quit IRC02:47
dstufftinstalling stuff system wide with pip is one of those, if you break it you get to keep both pieces kind of things02:47
dstufftit can mess your system up, sometimes it painful ways02:48
dprinceYep. Agree for the common case. I wouldn't go so far as to say never though. In a pinch packaged pip might come in handy even if it does ruin your distro02:48
mordreddprince: have you seen: https://github.com/stankevich/puppet-python ? it's pretty good with puppetting virtualenv things02:48
mordreddprince: ++02:48
* mordred agrees with dprince and dstufft02:49
dstufftdprince: yea I do it oftenish, mostly it's "if you don't know how to unbreak it, you probably don't want to do it"02:49
dprincemordred: I had not seen it. looks promising02:49
mordreddprince: we're using it for one thing ...02:49
mordreddprince: I want to enhance it to make it grok multi-pip and also to grok installing pip from get-pip (if requested)02:49
dstufftI wouldn't use a system packages pip, even though I contribute to helping them package it02:50
*** zhiyan is now known as zhiyan_02:50
dstufftmostly because they do things like unbundle stuff which I don't think should be bundled :/02:50
dstufftshould be unbundled*02:50
dprincemordred: cool02:51
*** zhiyan_ is now known as zhiyan02:51
mordreddstufft: yah. well, also because it's invariably old02:51
mordredand I honestly think if you're willing to use pip to install software, you should be willing to use pip to install pip02:51
dstufftmordred: yea there's that too02:52
dstuffttrusty is ok right now02:52
mordredyah02:52
dstufftbut in 2 years it'll be lol'old02:52
mordredkinda like pip on precise02:52
dstufftoh god yes02:52
dstufft1.002:52
dstufftit doesn't even TLS02:53
mordredyah02:53
mordredoh, speaking of ...02:53
mordredfungi: are you the only one who can do SSL certs for us any more?02:53
*** dprince has quit IRC02:53
*** msabramo has joined #openstack-infra02:54
mordredZOMG. we have too many images02:55
mordreddstufft: oh - btw - the new bandersnatch mirror is live02:55
*** marun is now known as marun_afk02:56
ianwmordred: can't we fix upstream rather that write our own provider?  or is there history i don't know about?02:56
mordredianw: well ... we can - but we're running puppet2 still02:56
mordredianw: I'd love to fix upstream02:56
mordredianw: and then migrate to puppet302:56
mordredlargely though - I think we just haven't gotten aroudn to it02:56
ianwmordred: well, not running 3 on the f20 (& centos7 when i get to it) nodes02:57
* mordred hands ianw a goat and a case of beer02:57
ianwmordred: i mean, "are running" ^02:57
mordred++02:57
r1chardj0n3smordred: I was going to comment on the bandersnatch review: I think you want to configure it to not  delete packages02:57
r1chardj0n3smordred: sorry I didn't get around to commenting - I've been mired in devstack and horizon02:58
dstufftmordred: awesome :)02:58
mordredr1chardj0n3s: no worries - I think we need it to respond to the deletes02:58
dstufftdeleting packages is what you want if you want it to match what PyPI has02:58
mroddenbandersnatch == pypi mirror thing?02:58
dstufftmrodden: yes02:58
mordredmrodden: yes02:58
mordredhttp://pypi.openstack.org/simple02:59
mordredis the new mirror02:59
dstufftgotta love how slow /simple/ is02:59
dstuffteven on PyPI itself where it's cached02:59
mroddenwow02:59
r1chardj0n3syeah but it's effectively a private mirror so there's really no reason to delete; and you'll protect yourselves against a package owner deleting something you use02:59
mroddenis that ALL of pypi?02:59
dstufftmrodden: yes02:59
mordredr1chardj0n3s: that's actually ok though02:59
r1chardj0n3syep :)02:59
mroddenwow02:59
r1chardj0n3smordred: ok, just wanted to raise it as a thing :)03:00
dstufftprobably they want to know if someone deletes a thing from PyPIif that's going to break them03:00
mordredr1chardj0n3s: because if that happens, it means everyone who uses opensatck is screwed with that package and we want to know abou tit03:00
mordredyah. what dstufft03:00
mordredr1chardj0n3s: but it's an excellent point03:00
r1chardj0n3sI just know I've had a lot of heat from poeople complaining when packages are deleted :P03:00
mordredmrodden: we're getting out of the business of gating by something not being in the mirror03:00
mordredr1chardj0n3s: well, people shouldn't delete packages03:00
r1chardj0n3smordred: that's a fair call, yeah03:00
mordred:)03:00
*** mriedem has quit IRC03:00
r1chardj0n3smordred: well, that I can't control03:00
*** zz_dimtruck is now known as dimtruck03:00
mordredSO many things I can't control03:00
mordredmrodden: next step is per-cloud-region mirrors03:01
r1chardj0n3smordred: I have to respect someone saying that they just don't want their package out there any longer03:01
mordredmrodden: so that we're never pulling from outside of a cloud region for a given build03:01
mroddenhow many packages does it pull locally?03:01
mroddenerr03:01
mroddenhow much package space/ disk usage03:01
mordredmrodden: the mirror is taking ~95G03:01
mroddenhmm thats not terrible03:01
* mordred afks to store03:02
*** r1chardj0n3s is now known as r1chard_afk03:02
*** harlowja is now known as harlowja_away03:02
*** jyu has quit IRC03:02
*** jyu has joined #openstack-infra03:03
*** jyu has quit IRC03:04
mroddenguess i should probably look into replacing my run_mirror.py with that eventually03:04
*** arnaud has quit IRC03:05
*** arnaud__ has quit IRC03:05
*** msabramo has quit IRC03:09
*** harlowja_away is now known as harlowja03:09
*** praneshp has joined #openstack-infra03:10
*** harlowja is now known as harlowja_away03:10
*** harlowja_away is now known as harlowja03:11
*** AaronGr has quit IRC03:12
*** praneshp_ has joined #openstack-infra03:13
*** AaronGr has joined #openstack-infra03:14
*** ArxCruz has quit IRC03:15
*** praneshp has quit IRC03:16
*** praneshp_ is now known as praneshp03:16
*** jdahlin has quit IRC03:17
*** HenryG has quit IRC03:17
*** annegent_ has joined #openstack-infra03:18
*** HenryG has joined #openstack-infra03:20
*** devkulkarni has quit IRC03:24
*** annegent_ has quit IRC03:24
*** UtahDave has joined #openstack-infra03:27
*** yaguang has quit IRC03:29
*** rlandy has joined #openstack-infra03:30
*** sabeen1 has quit IRC03:33
*** sabeen1 has joined #openstack-infra03:33
*** r1chard_afk is now known as r1chardj0n3s03:37
*** nosnos has quit IRC03:38
*** annegent_ has joined #openstack-infra03:38
*** annegent_ has quit IRC03:41
*** annegentle has joined #openstack-infra03:42
*** yaguang has joined #openstack-infra03:42
*** niska has quit IRC03:54
*** niska has joined #openstack-infra03:59
*** chandan_kumar has joined #openstack-infra03:59
*** gnuoy` has quit IRC04:00
*** jamespage has quit IRC04:00
*** gnuoy has joined #openstack-infra04:00
*** jamespage has joined #openstack-infra04:00
*** jamespage has joined #openstack-infra04:00
*** devkulkarni has joined #openstack-infra04:00
*** homeless_ has joined #openstack-infra04:02
*** TheJulia has quit IRC04:05
*** sileht has quit IRC04:06
*** homeless has quit IRC04:06
*** TheJulia has joined #openstack-infra04:06
*** nati_ueno has quit IRC04:06
*** sileht has joined #openstack-infra04:08
*** craigbr has joined #openstack-infra04:08
*** blamar has quit IRC04:11
*** jogo_awa1 has quit IRC04:11
*** fifieldt has quit IRC04:11
*** bcrochet|g0ne has quit IRC04:11
*** gyee has quit IRC04:11
*** ildikov has quit IRC04:11
*** mfer has quit IRC04:11
*** rfolco has quit IRC04:11
*** pcm__ has quit IRC04:11
*** annegentle has quit IRC04:11
*** gema has quit IRC04:11
*** grantbow has quit IRC04:11
*** jpeeler has quit IRC04:11
*** jamielennox has quit IRC04:11
*** ekarlso has quit IRC04:11
*** rakhmerov has quit IRC04:11
*** bradm has quit IRC04:11
*** wenlock_ has quit IRC04:11
*** hdd has quit IRC04:11
*** chmouel has quit IRC04:11
*** dimtruck has quit IRC04:11
*** gyee_ has quit IRC04:11
*** pfalleno1 has quit IRC04:11
*** proffalken has quit IRC04:11
*** sc68cal has quit IRC04:11
*** clarkb has quit IRC04:11
*** devananda has quit IRC04:11
*** jeblair has quit IRC04:11
*** slagle has quit IRC04:11
*** SlickNik has quit IRC04:11
*** gothicmindfood has quit IRC04:11
*** Kiall has quit IRC04:11
*** jcooley has quit IRC04:11
*** saper has quit IRC04:11
*** jd__ has quit IRC04:11
*** portante has quit IRC04:11
*** briancline has quit IRC04:11
*** tnurlygayanov has quit IRC04:11
*** mrda has quit IRC04:12
*** arnaud has joined #openstack-infra04:13
*** arnaud__ has joined #openstack-infra04:14
*** homeless_ has quit IRC04:17
*** ArxCruz has joined #openstack-infra04:21
*** ArxCruz has quit IRC04:21
*** craigbr has quit IRC04:21
*** mrda has joined #openstack-infra04:22
*** annegentle has joined #openstack-infra04:22
*** wenlock_ has joined #openstack-infra04:22
*** blamar has joined #openstack-infra04:22
*** jogo_awa1 has joined #openstack-infra04:22
*** fifieldt has joined #openstack-infra04:22
*** bcrochet|g0ne has joined #openstack-infra04:22
*** gyee has joined #openstack-infra04:22
*** pcm__ has joined #openstack-infra04:22
*** ildikov has joined #openstack-infra04:22
*** mfer has joined #openstack-infra04:22
*** rfolco has joined #openstack-infra04:22
*** tnurlygayanov has joined #openstack-infra04:22
*** gema has joined #openstack-infra04:22
*** hdd has joined #openstack-infra04:22
*** chmouel has joined #openstack-infra04:22
*** dimtruck has joined #openstack-infra04:22
*** gyee_ has joined #openstack-infra04:22
*** pfalleno1 has joined #openstack-infra04:22
*** proffalken has joined #openstack-infra04:22
*** sc68cal has joined #openstack-infra04:22
*** clarkb has joined #openstack-infra04:22
*** devananda has joined #openstack-infra04:22
*** grantbow has joined #openstack-infra04:22
*** bradm has joined #openstack-infra04:22
*** ekarlso has joined #openstack-infra04:22
*** jamielennox has joined #openstack-infra04:22
*** rakhmerov has joined #openstack-infra04:22
*** jpeeler has joined #openstack-infra04:22
*** jeblair has joined #openstack-infra04:22
*** slagle has joined #openstack-infra04:22
*** SlickNik has joined #openstack-infra04:22
*** gothicmindfood has joined #openstack-infra04:22
*** Kiall has joined #openstack-infra04:22
*** jcooley has joined #openstack-infra04:22
*** saper has joined #openstack-infra04:22
*** jd__ has joined #openstack-infra04:22
*** portante has joined #openstack-infra04:22
*** briancline has joined #openstack-infra04:22
*** Guest37445 is now known as zaro04:23
*** msabramo has joined #openstack-infra04:24
zarojhesketh: ping04:24
jheskethzaro: pong04:25
*** UtahDave has left #openstack-infra04:26
*** willroberts has quit IRC04:26
*** dstufft|laptop has joined #openstack-infra04:26
*** willroberts has joined #openstack-infra04:26
*** gema_ has joined #openstack-infra04:27
zarojhesketh: attempt to remove envinject failed.  https://jenkins07.openstack.org/job/check-gerrit-buck/11/console04:27
*** gema has quit IRC04:27
zarojhesketh: are you able to view the workspace and see if the version.properties file exists?04:28
jheskethzaro: nope, I don't have root sorry :-(04:28
* jhesketh can only merge code that breaks the world.. . he cannot fix the world04:28
jheskethclarkb: if you're around ^04:28
zarojhesketh: any idea why source isn't sourcing the file?04:28
jheskethlooking..04:30
clarkbjhesketh: zaro the workspaces don't exist because we delete the salves04:30
zaroohh right.04:31
*** gyee has quit IRC04:31
*** jamielennox has quit IRC04:31
clarkbthe filename looks tyoped04:31
*** ildikov has quit IRC04:31
*** gyee has joined #openstack-infra04:31
clarkbyup thats it04:31
*** jamielennox has joined #openstack-infra04:32
clarkbproperites04:32
*** dkehn has joined #openstack-infra04:32
*** ildikov has joined #openstack-infra04:32
*** jogo_awa1 has quit IRC04:32
zaroduhh!!!04:32
*** jogo_away has joined #openstack-infra04:32
*** grantbow has quit IRC04:32
zarowill put put a fix.04:32
jhesketh:-( sorry04:32
jheskethzaro: I can if you prefer?04:32
lifelessclarkb: mmm salves04:34
*** dkehnx has quit IRC04:34
*** rlandy has quit IRC04:35
openstackgerritKhai Do proposed a change to openstack-infra/config: Fix filename typo in gerrit jobs.  https://review.openstack.org/10372104:36
*** dimtruck is now known as zz_dimtruck04:36
zarojhesketh, clarkb ^04:36
*** grantbow has joined #openstack-infra04:36
jheskeththanks z04:36
jhesketh*zaro04:36
zaronp.  git grep for same typo but didn't find any other so it's isolated.04:38
jheskethyeah ditto04:39
openstackgerritA change was merged to openstack-infra/config: Fix filename typo in gerrit jobs.  https://review.openstack.org/10372104:42
*** gokrokve has joined #openstack-infra04:45
*** nosnos has joined #openstack-infra04:45
*** doude has quit IRC04:46
*** talluri has joined #openstack-infra04:48
*** ildikov has quit IRC04:49
*** nosnos has quit IRC04:51
*** nosnos has joined #openstack-infra04:51
*** hdd_ has quit IRC04:57
jheskethzaro: that seems to have fixed it : https://jenkins04.openstack.org/job/check-gerrit-buck/11/console05:00
*** adrian_otto has quit IRC05:00
*** devkulkarni has quit IRC05:03
*** dstanek is now known as dstanek_zzz05:05
*** harlowja is now known as harlowja_away05:06
*** amitpp has joined #openstack-infra05:09
*** om has joined #openstack-infra05:12
*** dstanek_zzz is now known as dstanek05:13
*** wenlock_ has quit IRC05:21
*** pcrews has quit IRC05:21
*** Longgeek has joined #openstack-infra05:22
*** ildikov has joined #openstack-infra05:22
*** wenlock_ has joined #openstack-infra05:24
*** Longgeek has quit IRC05:25
wenlock_hi, in your puppet runs for infra for hpcloud have you been seeing puppet exceptions similar to the following? http://paste.openstack.org/show/85195/05:26
wenlock_i know this is a known issue... https://projects.puppetlabs.com/issues/17855 but it didnt seem to come up very often... this week it seems to be coming up alot05:27
*** e0ne has joined #openstack-infra05:28
*** dstanek is now known as dstanek_zzz05:28
*** rcarrillocruz has joined #openstack-infra05:31
*** basha has joined #openstack-infra05:32
*** tkelsey has joined #openstack-infra05:32
*** gokrokve has quit IRC05:33
*** rcarrill` has quit IRC05:34
*** Longgeek has joined #openstack-infra05:35
*** reed has quit IRC05:35
*** chandan_kumar has quit IRC05:37
*** bdpayne has quit IRC05:39
*** bdpayne has joined #openstack-infra05:41
openstackgerritIan Wienand proposed a change to openstack-infra/config: Initial RHEL7/Centos7 support  https://review.openstack.org/10345805:41
openstackgerritIan Wienand proposed a change to openstack-infra/config: Add functions & cleanup install_puppet.sh  https://review.openstack.org/10373505:41
*** wenlock_ has quit IRC05:42
*** e0ne has quit IRC05:44
openstackgerritIan Wienand proposed a change to openstack-infra/config: Add functions & cleanup install_puppet.sh  https://review.openstack.org/10373505:44
openstackgerritIan Wienand proposed a change to openstack-infra/config: Initial RHEL7/Centos7 support  https://review.openstack.org/10345805:44
*** e0ne has joined #openstack-infra05:44
*** e0ne has quit IRC05:45
*** bdpayne has quit IRC05:46
*** gyee has quit IRC05:47
*** otherwiseguy has quit IRC05:47
*** e0ne has joined #openstack-infra05:50
openstackgerritPh. Marek proposed a change to openstack/requirements: Add dbus library, needed for the DRBD cinder driver integration.  https://review.openstack.org/9901305:51
*** markmc has joined #openstack-infra05:52
*** e0ne has quit IRC05:55
*** bdpayne has joined #openstack-infra05:55
*** e0ne has joined #openstack-infra05:57
*** dkehn__ has joined #openstack-infra06:00
*** nosnos has quit IRC06:01
*** harlowja_at_home has joined #openstack-infra06:02
*** dkehn has quit IRC06:03
*** nosnos has joined #openstack-infra06:03
*** bdpayne has quit IRC06:06
*** e0ne has quit IRC06:08
*** unicell1 has joined #openstack-infra06:09
*** e0ne has joined #openstack-infra06:10
*** unicell has quit IRC06:10
*** dkehn has joined #openstack-infra06:12
*** zigo has quit IRC06:13
*** zigo has joined #openstack-infra06:14
*** dkehn__ has quit IRC06:15
*** Longgeek has quit IRC06:16
*** abhishek has joined #openstack-infra06:17
*** e0ne has quit IRC06:18
*** Longgeek has joined #openstack-infra06:19
*** nati_ueno has joined #openstack-infra06:19
abhishekhttps://review.openstack.org/#/c/102215/06:20
abhishekHi all: can any one review this patch and help me to get this merged?06:20
abhishekThank you06:20
openstackgerritMark McLoughlin proposed a change to openstack-infra/elastic-recheck: Add query for rare MySQL-Python install failure  https://review.openstack.org/10374606:20
*** msabramo has quit IRC06:22
*** andreykurilin has joined #openstack-infra06:22
*** e0ne has joined #openstack-infra06:22
*** krtaylor has quit IRC06:23
*** mkoderer has joined #openstack-infra06:24
*** salv-orlando has joined #openstack-infra06:29
*** e0ne has quit IRC06:29
*** dstanek_zzz is now known as dstanek06:29
*** e0ne has joined #openstack-infra06:33
*** andreykurilin has quit IRC06:34
*** krtaylor has joined #openstack-infra06:35
*** e0ne has quit IRC06:38
*** harlowja_at_home has quit IRC06:40
*** unicell1 is now known as unicell06:40
*** unicell has joined #openstack-infra06:40
*** chandan_kumar has joined #openstack-infra06:41
*** e0ne has joined #openstack-infra06:41
*** e0ne has quit IRC06:44
*** e0ne has joined #openstack-infra06:47
*** sballe_ has joined #openstack-infra06:49
*** e0ne has quit IRC06:50
*** pblaho has joined #openstack-infra06:52
*** sballe has quit IRC06:53
*** yolanda has joined #openstack-infra06:56
*** rcarrillocruz has quit IRC06:56
*** camunoz is now known as camunoz_gone06:58
*** rcarrillocruz has joined #openstack-infra07:02
*** arnaud__ has quit IRC07:06
*** arnaud has quit IRC07:06
*** morganfainberg is now known as morganfainberg_Z07:12
*** arnaud has joined #openstack-infra07:14
*** jcoufal has joined #openstack-infra07:14
*** mfer has quit IRC07:15
*** arnaud__ has joined #openstack-infra07:15
*** mrmartin has joined #openstack-infra07:18
*** arnaud__ has quit IRC07:20
*** arnaud has quit IRC07:20
*** arnaud has joined #openstack-infra07:20
*** arnaud__ has joined #openstack-infra07:20
nibalizerarstu/win 1207:22
*** dstanek is now known as dstanek_zzz07:24
*** skolekonov has joined #openstack-infra07:25
*** mmaglana has quit IRC07:26
*** jgallard has joined #openstack-infra07:26
*** praneshp has quit IRC07:27
*** lpaduano_freenod has joined #openstack-infra07:28
*** eglynn-reg-offic is now known as eglynn-office07:31
*** mkerrin has quit IRC07:33
mattoliverauWell I'm calling it a night, night all07:35
*** rdopieralski has joined #openstack-infra07:37
*** hashar has joined #openstack-infra07:38
*** e0ne has joined #openstack-infra07:39
*** ociuhandu has joined #openstack-infra07:39
*** rdopieralski has quit IRC07:40
*** dstufft|laptop has quit IRC07:40
*** dstufft|laptop has joined #openstack-infra07:42
*** rcarrill` has joined #openstack-infra07:42
*** rcarrillocruz has quit IRC07:43
*** Guest9812 has joined #openstack-infra07:43
*** liusheng has quit IRC07:44
*** salv-orlando has quit IRC07:46
*** mrda is now known as mrda-away07:49
openstackgerritA change was merged to openstack-infra/devstack-gate: Make one copy of grenade log files  https://review.openstack.org/10013107:52
*** vponomaryov has joined #openstack-infra07:53
*** flaper87|afk is now known as flaper8707:55
*** jpich has joined #openstack-infra07:55
*** morganfainberg_Z is now known as morganfainberg07:55
*** HACKING-TWITTER has joined #openstack-infra07:57
*** roeyc has joined #openstack-infra08:04
*** e0ne has quit IRC08:05
*** derekh_ has joined #openstack-infra08:05
*** e0ne has joined #openstack-infra08:05
*** nati_ueno has quit IRC08:05
*** r1chardj0n3s is now known as r1chard_afk08:05
*** matrohon has joined #openstack-infra08:06
*** denis_makogon has joined #openstack-infra08:06
*** ominakov has joined #openstack-infra08:06
*** ihrachyshka has joined #openstack-infra08:08
*** rcarrill` has quit IRC08:12
*** ArxCruz has joined #openstack-infra08:15
*** ArxCruz has quit IRC08:15
*** dstanek_zzz is now known as dstanek08:15
*** ihrachyshka has quit IRC08:15
*** ihrachyshka has joined #openstack-infra08:16
*** lcheng has joined #openstack-infra08:17
*** HACKING-TWITTER has quit IRC08:17
*** rcarrillocruz has joined #openstack-infra08:17
*** HACKING-TWITTER has joined #openstack-infra08:19
*** jlibosva has joined #openstack-infra08:20
*** HACKING-TWITTER has joined #openstack-infra08:20
*** HACKING-TWITTER has quit IRC08:20
SergeyLukjanovjeblair, fungi, clarkb, unfortunately I'll be in flight in time of infra meeting, so, most probably /me will be unavailable today08:21
*** HACKING-TWITTER has joined #openstack-infra08:21
*** HACKING-TWITTER has joined #openstack-infra08:23
*** HACKING-TWITTER has joined #openstack-infra08:24
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Remove obsolete #openstack-climate channel  https://review.openstack.org/9491308:25
*** dstanek is now known as dstanek_zzz08:25
*** HACKING-TWITTER has joined #openstack-infra08:26
*** HACKING-TWITTER has joined #openstack-infra08:27
*** HACKING-TWITTER has joined #openstack-infra08:29
*** arnaud__ has quit IRC08:29
*** arnaud has quit IRC08:29
*** liusheng has joined #openstack-infra08:30
*** zz_johnthetubagu is now known as johnthetubaguy08:30
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Add gerrit reviews into patchset approvals  https://review.openstack.org/9739008:34
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Allow a pipeline to specify alternative gerrit acc  https://review.openstack.org/9739108:34
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: WIP Add support for negative requirements  https://review.openstack.org/10272608:34
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Add support for negative requirements  https://review.openstack.org/10272608:35
*** mflobo has joined #openstack-infra08:38
*** mflobo has quit IRC08:38
*** andreaf_ has joined #openstack-infra08:39
*** doude has joined #openstack-infra08:39
hasharjhesketh: :-)08:42
jheskethHey hashar08:42
*** ildikov has quit IRC08:42
hasharjhesketh: my little Zuul cloner is working more or less as expected. James told me he will eventually review it soon â„¢08:42
jheskethexcellent :-)08:42
jheskethI've been excited for it08:42
hasharjhesketh: thank you for all your encouragements and reviews over the last few months08:43
jheskethanytime!08:43
hasharI started using it to build an integration job that test mediawiki/core + all the extensions we have on Wikimedia website08:43
hasharexample output https://integration.wikimedia.org/ci/job/mediawiki-core-extensions-integration/52/consoleFull  :-D08:43
hasharthat spam a lot. I believe that will help turbo-hipster / third party CI and maybe devstack will be interested in it08:43
hasharwe will see08:43
jheskethneat !08:44
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Set docs jobs for Manila project as votable  https://review.openstack.org/10380308:46
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Set docs jobs for Manila project as votable  https://review.openstack.org/10380308:49
*** rcarrillocruz has quit IRC08:52
*** ociuhandu has quit IRC08:54
*** ildikov has joined #openstack-infra08:54
*** rcarrillocruz has joined #openstack-infra08:57
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Add py33 job for python-manilaclient  https://review.openstack.org/10380908:59
*** IvanBerezovskiy1 has joined #openstack-infra09:05
eglynn-officefolks if anyone has any spare review cycles in the next hour or two09:06
eglynn-office... it would be great to have a look at the ceilo-specs backlog09:06
eglynn-office... https://review.openstack.org/#/q/status:open+project:openstack/ceilometer-specs,n,z09:06
eglynn-officethanking y'all in advance :)09:06
* eglynn-office hits the road to Paris09:07
eglynn-office... happy travels to anyone who's heading to the mid-cycle09:07
hashareglynn-office: enjoy Paris09:07
*** eglynn-office is now known as eglynn-office-af09:07
*** eglynn-office-af is now known as eglynn-offic-afk09:08
*** andreaf has quit IRC09:10
*** andreaf_ is now known as andreaf09:11
*** andreaf_ has joined #openstack-infra09:11
*** yamahata has quit IRC09:12
*** flaper87 is now known as flaper87|afk09:13
*** ihrachyshka has quit IRC09:13
*** jgallard has quit IRC09:14
*** dstanek_zzz is now known as dstanek09:16
*** jp_at_hp has joined #openstack-infra09:17
*** jp_at_hp has quit IRC09:17
*** jp_at_hp has joined #openstack-infra09:17
*** ildikov has quit IRC09:19
*** pelix has joined #openstack-infra09:23
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Added devstack job for manila  https://review.openstack.org/9993309:23
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Added devstack job for manila  https://review.openstack.org/9993309:24
*** dstanek is now known as dstanek_zzz09:26
*** mmaglana has joined #openstack-infra09:26
*** oomichi has quit IRC09:30
*** mmaglana has quit IRC09:31
*** basha_ has joined #openstack-infra09:38
*** basha has quit IRC09:39
*** basha_ is now known as basha09:39
openstackgerritThierry Carrez proposed a change to openstack-infra/config: Change ACLs to apply on proposed/* branches  https://review.openstack.org/10282209:47
*** dstanek_zzz is now known as dstanek09:48
*** Longgeek has quit IRC09:49
*** Longgeek has joined #openstack-infra09:50
abhishekHi all: can any one review this patch and help me to get this merged?09:51
abhishekhttps://review.openstack.org/#/c/102215/09:51
abhishekThank you09:51
openstackgerritA change was merged to openstack-infra/jeepyb: Update global requirements  https://review.openstack.org/10333809:58
*** dstanek is now known as dstanek_zzz09:58
*** Longgeek_ has joined #openstack-infra10:00
*** Guest9812 has quit IRC10:01
*** roeyc has quit IRC10:01
*** Guest9812 has joined #openstack-infra10:02
*** Longgeek has quit IRC10:03
*** jgallard has joined #openstack-infra10:04
*** skolekonov has quit IRC10:09
openstackgerritA change was merged to openstack-infra/reviewstats: Added skraynev to heat-core  https://review.openstack.org/10315210:14
*** ihrachyshka has joined #openstack-infra10:16
*** ociuhandu has joined #openstack-infra10:18
yolandamordred, jeblair, i was wondering about the nodepool image-update command. Actually with our approach, it will make more sense to do: nodepool image-update dibname , and then this will rebuild image in dib, and propagate to all providers that have this image in their list, what do you think?10:22
openstackgerritA change was merged to openstack-infra/config: Make py33 job voting in check and gate for Rubick  https://review.openstack.org/10351610:24
*** jcoufal has quit IRC10:26
*** salv-orlando has joined #openstack-infra10:26
*** jcoufal has joined #openstack-infra10:26
*** roeyc has joined #openstack-infra10:28
*** Guest9812 has quit IRC10:28
openstackgerritSergey Skripnick proposed a change to openstack-infra/config: Enable ceilometer in rally-dsvm jobs  https://review.openstack.org/10383610:29
openstackgerritA change was merged to openstack-infra/config: Add openstack-security to accessbot and eavesdrop  https://review.openstack.org/10365010:31
openstackgerritA change was merged to openstack-infra/config: Enable py33 gating for Stackalytics  https://review.openstack.org/10365410:31
openstackgerritSergey Skripnick proposed a change to openstack-infra/config: Enable ceilometer in rally-dsvm jobs  https://review.openstack.org/10383610:34
*** yaguang has quit IRC10:35
*** talluri has quit IRC10:36
openstackgerritChristian Berendt proposed a change to openstack-infra/gerritlib: Changed home-page used in setup.cfg  https://review.openstack.org/10186010:38
openstackgerritA change was merged to openstack-infra/config: Add functional tests for swift3  https://review.openstack.org/10229310:40
*** e0ne has quit IRC10:41
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Enable ceilometer in rally-dsvm jobs  https://review.openstack.org/10383610:41
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Enable ceilometer in rally-dsvm jobs  https://review.openstack.org/10383610:42
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Auth error handling  https://review.openstack.org/10186710:44
bookwarif i have a chain of >10 dependent patchsets in Gerrit, and then commit a change to the first patchset, every dependent patchset gets changed and goes through review process?10:46
*** chandan_kumar has quit IRC10:47
*** dstanek_zzz is now known as dstanek10:49
sdaguebookwar: if the other patches aren't changed, their review votes stay10:49
*** jyuso006 has joined #openstack-infra10:50
bookwarbut gerrit event is triggered anyway?10:50
*** openstack has joined #openstack-infra10:51
*** talluri has joined #openstack-infra10:56
hasharbookwar: if only the first patch is changed and the dependent one are not rebased, that would be only one patch being triggered10:57
hasharbookwar: but I guess most folks rebase the whole chain, so that is new patchets that needs testing indeed10:57
*** dstanek is now known as dstanek_zzz10:59
*** talluri has quit IRC11:00
*** talluri has joined #openstack-infra11:00
*** basha has quit IRC11:01
*** talluri has quit IRC11:05
*** cnesa has quit IRC11:07
*** dims_ has joined #openstack-infra11:07
*** e0ne has joined #openstack-infra11:08
*** Longgeek_ has quit IRC11:08
*** Longgeek has joined #openstack-infra11:09
bookwaryeah, testing is still needed for those patchsets, let's see if i an rearrange or reschedule them on the CI side11:09
bookwarthanks11:10
*** tkelsey has quit IRC11:17
*** chandan_kumar has joined #openstack-infra11:20
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Adds a docs-specs repository  https://review.openstack.org/10311511:21
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Fix logging usage  https://review.openstack.org/10384811:26
*** mmaglana has joined #openstack-infra11:27
*** adalbas has joined #openstack-infra11:28
*** basha has joined #openstack-infra11:28
*** mmaglana has quit IRC11:32
Kialljeblair / clarkb - when you have a chance, could you have a look at my comment on https://review.openstack.org/#/c/99611/ ?11:32
*** basha has quit IRC11:33
*** jyuso006 has quit IRC11:35
*** basha has joined #openstack-infra11:39
*** basha has quit IRC11:39
*** Longgeek_ has joined #openstack-infra11:39
*** weshay has joined #openstack-infra11:43
*** Longgeek has quit IRC11:43
*** weshay has quit IRC11:44
*** weshay has joined #openstack-infra11:44
*** ArxCruz has joined #openstack-infra11:46
*** ArxCruz_ has joined #openstack-infra11:47
*** skolekonov has joined #openstack-infra11:49
*** dstanek_zzz is now known as dstanek11:50
*** ArxCruz has quit IRC11:51
*** dkehn__ has joined #openstack-infra11:51
*** HACKING-TWITTER has quit IRC11:55
*** amitpp has quit IRC11:55
*** dkehn has quit IRC11:55
*** HACKING-TWITTER has joined #openstack-infra11:57
*** rlandy has joined #openstack-infra11:58
*** HACKING-TWITTER has joined #openstack-infra11:59
*** dstanek is now known as dstanek_zzz12:00
*** HACKING-TWITTER has joined #openstack-infra12:00
*** talluri has joined #openstack-infra12:01
*** HACKING-TWITTER has joined #openstack-infra12:02
*** ArxCruz_ has quit IRC12:02
*** ArxCruz_ has joined #openstack-infra12:02
*** mbacchi has joined #openstack-infra12:03
*** HACKING-TWITTER has joined #openstack-infra12:03
*** HACKING-TWITTER has quit IRC12:03
*** HACKING-TWITTER has joined #openstack-infra12:04
*** talluri has quit IRC12:05
*** HACKING-TWITTER has joined #openstack-infra12:06
*** cnesa has joined #openstack-infra12:07
*** kashyap_bbiab has joined #openstack-infra12:08
*** Longgeek_ has quit IRC12:08
*** Longgeek has joined #openstack-infra12:08
*** kashyap_bbiab has quit IRC12:09
*** kashyap_bbiab has joined #openstack-infra12:09
*** kashyap` has joined #openstack-infra12:10
*** doug-fish has joined #openstack-infra12:10
*** kashyap has quit IRC12:11
*** Longgeek has quit IRC12:13
*** kashyap_bbiab has quit IRC12:14
*** kashyap` is now known as kashyap12:16
*** ihrachyshka has quit IRC12:16
*** Longgeek has joined #openstack-infra12:19
*** jcoufal has quit IRC12:19
*** dhellmann is now known as dhellmann_12:19
openstackgerritA change was merged to openstack-infra/config: Add python34 build and job template  https://review.openstack.org/10284712:20
abhishekHi all: can any one review this patch and help me to get this merged?12:21
abhishekhttps://review.openstack.org/#/c/102215/12:21
abhishekThank you12:21
*** rdopieralski has joined #openstack-infra12:21
rdopieralskiI wanted to ask what we would need to do to get openstack/tuskar-ui to auto-update its requirments to match global-requirements?12:22
*** pdmars has joined #openstack-infra12:23
*** jhesketh has quit IRC12:23
chmouelhow come something like that can happen ? http://paste.openstack.org/raw/85221/ in devstack review https://review.openstack.org/#/c/65113/12:24
Kiallrdopieralski: I've just done that for designate - https://review.openstack.org/#/c/103316/12:24
rdopieralskiKiall: awesome, thank you!12:24
*** pdmars has quit IRC12:24
*** pdmars has joined #openstack-infra12:25
*** dprince has joined #openstack-infra12:26
*** bcrochet|g0ne is now known as bcrochet12:27
*** dims_ has quit IRC12:32
*** dims_ has joined #openstack-infra12:32
*** Longgeek has quit IRC12:34
*** Longgeek has joined #openstack-infra12:34
*** ihrachyshka has joined #openstack-infra12:36
openstackgerritChristian Berendt proposed a change to openstack-infra/config: Add pep8 check/gate for stackforge/doc8  https://review.openstack.org/10386312:36
*** mrmartin has quit IRC12:37
openstackgerritKashyap Chamarthy proposed a change to openstack-infra/elastic-recheck: Add query for Neutron functional test failures bug 1336172  https://review.openstack.org/10386412:37
uvirtbotLaunchpad bug 1336172 in neutron "neutron/agent/linux/ovs_lib.py: int(ofport) returns 'NoneType' instead of string or a number" [Undecided,New] https://launchpad.net/bugs/133617212:37
*** Longgeek_ has joined #openstack-infra12:39
*** jcoufal has joined #openstack-infra12:39
*** Longgee__ has joined #openstack-infra12:40
*** Longgee__ has quit IRC12:40
*** Longgee__ has joined #openstack-infra12:41
*** Longgeek_ has quit IRC12:41
*** Longgeek has quit IRC12:42
fungirdopieralski: you would need to add the project to http://git.openstack.org/cgit/openstack/requirements/tree/projects.txt12:43
fungioh, i see Kiall already answered12:43
*** tkelsey has joined #openstack-infra12:43
*** ArxCruz_ has quit IRC12:43
*** Longgeek has joined #openstack-infra12:44
*** Longgee__ has quit IRC12:44
openstackgerritPeter Belanyi proposed a change to openstack/requirements: Add tuskar-ui to projects.txt  https://review.openstack.org/10386812:46
openstackgerritChristian Berendt proposed a change to openstack-dev/cookiecutter: Use spaces instead of tabs for indentation  https://review.openstack.org/10386912:46
*** alexpilotti has joined #openstack-infra12:47
fungichmouel: was that in the neutron functional test job?12:47
*** ArxCruz has joined #openstack-infra12:48
fungiyeah, looks like there was some sort of permissions problem with that job... note that things started falling apart long before that sudo error message12:48
*** lpaduano_freenod has left #openstack-infra12:49
*** basha has joined #openstack-infra12:49
openstackgerritA change was merged to openstack-infra/release-tools: Add spec2bp.py script  https://review.openstack.org/10156612:50
openstackgerritA change was merged to openstack-infra/release-tools: Add basic milestone check feature to ms2version.py  https://review.openstack.org/10307012:50
*** dstanek_zzz is now known as dstanek12:50
*** basha has quit IRC12:51
*** Longgeek has quit IRC12:52
*** Longgeek_ has joined #openstack-infra12:52
*** julim has joined #openstack-infra12:52
fungichmouel: yeah, it persists across runs on the current patchset, so something about that change is breaking the neutron functional tests12:55
*** Longgeek has joined #openstack-infra12:56
*** xianghuihui has joined #openstack-infra12:56
*** lcheng_ has joined #openstack-infra12:57
*** chandan_kumar has quit IRC12:59
openstackgerritThierry Carrez proposed a change to openstack-infra/config: Add proposed/* branches support to merge_tags.sh  https://review.openstack.org/10281212:59
*** Longgeek_ has quit IRC12:59
*** xianghui has quit IRC12:59
ttxfungi: tried to address your concern and mordred's "should not be that complex" remark ^13:00
*** mfer has joined #openstack-infra13:00
*** dstanek is now known as dstanek_zzz13:00
*** lcheng has quit IRC13:00
*** talluri has joined #openstack-infra13:02
fungittx: what is the possibility that this will be run for tags on master branches of projects, and should those short-circuit too if so13:03
fungi?13:03
*** Guest9812 has joined #openstack-infra13:03
*** weshay has quit IRC13:03
*** Longgeek has quit IRC13:03
fungior will it just try to merge a tag from master back into master and be a no-op?13:04
*** Longgeek has joined #openstack-infra13:04
*** yaguang has joined #openstack-infra13:04
ttxfungi: it will certainly run, for example when we push the tag for intermediary swift versions onmaster13:04
ttxbut I think it should noop13:05
*** yaguang has quit IRC13:05
openstackgerritA change was merged to openstack/requirements: Add Designate to projects.txt  https://review.openstack.org/10331613:05
*** yaguang has joined #openstack-infra13:05
ttxfungi: if not, I guess we could also exclude master if it --contains TAG13:06
*** talluri has quit IRC13:06
*** hdd_ has joined #openstack-infra13:07
*** chandan_kumar has joined #openstack-infra13:07
openstackgerritThierry Carrez proposed a change to openstack-infra/infra-specs: Added specification for storyboard story tags  https://review.openstack.org/9721113:09
*** blamar has quit IRC13:09
*** pblaho_ has joined #openstack-infra13:09
*** pblaho has quit IRC13:09
*** Longgeek_ has joined #openstack-infra13:10
*** lcheng__ has joined #openstack-infra13:11
*** blamar has joined #openstack-infra13:11
fungittx: yeah, it runs under errorexit, so if the git merge call exits nonzero the job is going to be marked as failed13:12
fungitesting now13:12
*** lcheng_ has quit IRC13:13
fungittx: aha! git merge responds "Already up-to-date." and exits 0 in that no-op case, so wfm13:14
*** Longgeek has quit IRC13:14
ttxfungi: cool, thx for testing13:14
*** markmcclain has joined #openstack-infra13:15
*** dkehn__ is now known as dkehnx13:15
*** dkliban_afk is now known as dkliban13:16
*** weshay has joined #openstack-infra13:16
*** yaguang has quit IRC13:19
*** yaguang has joined #openstack-infra13:19
*** mriedem has joined #openstack-infra13:22
*** mriedem has quit IRC13:22
*** mriedem has joined #openstack-infra13:22
*** changbl has quit IRC13:23
*** devkulkarni has joined #openstack-infra13:25
*** ominakov has quit IRC13:27
*** mmaglana has joined #openstack-infra13:28
*** chandan_kumar is now known as chandankumar13:30
*** markmcclain has left #openstack-infra13:31
*** prad has joined #openstack-infra13:32
*** ominakov has joined #openstack-infra13:32
*** zz_gondoi is now known as gondoi13:32
*** msabramo has joined #openstack-infra13:33
*** pblaho_ is now known as pblaho13:33
*** mmaglana has quit IRC13:33
*** pblaho_ has joined #openstack-infra13:33
*** amalagon has joined #openstack-infra13:36
fungimordred: check your cronspam... you probably want to configure bandersnatch to not log info/debug-level messages to stdout/stderr13:36
*** jgrimm has joined #openstack-infra13:37
*** pblaho has quit IRC13:37
*** pcrews has joined #openstack-infra13:39
openstackgerritIan Cordasco proposed a change to openstack/requirements: Add rfc3986 for Nova URI validations  https://review.openstack.org/10389213:41
*** mdenny has joined #openstack-infra13:44
annegentledhellmann_: around? sphinx theming question for ya13:47
annegentlemordred: I talked to Lauren and we want to indicate a project is incubating with the docs output. I commented on https://review.openstack.org/#/c/103349/13:48
annegentlemordred: what I want to figure out now is the best way to provide the theme13:48
*** basha has joined #openstack-infra13:49
*** sarob has joined #openstack-infra13:50
*** dstanek_zzz is now known as dstanek13:51
* annegentle drives to work13:52
*** devkulkarni has quit IRC13:57
fungiannegentle: if that sort of distinction is needed, it may be good to dynamically select it based on the programs.yaml in the governance repo (at per-project-per-release granularity presumably?)13:57
*** om has quit IRC13:58
*** homeless has joined #openstack-infra13:58
*** otherwiseguy has joined #openstack-infra13:58
*** chandan_kumar has joined #openstack-infra13:58
*** chandankumar has quit IRC13:59
*** basha has quit IRC13:59
*** chandan_kumar is now known as chandan|afk14:00
*** chandan|afk is now known as ciypro|afk14:00
*** david-lyle has joined #openstack-infra14:01
*** nelsnelson has joined #openstack-infra14:02
*** pblaho_ is now known as pblaho14:02
*** eharney has quit IRC14:02
*** chandan_kumar has joined #openstack-infra14:04
*** roeyc has quit IRC14:05
*** duran has joined #openstack-infra14:05
*** mestery has quit IRC14:05
*** flaper87|afk is now known as flaper8714:06
*** basha has joined #openstack-infra14:06
*** gokrokve has joined #openstack-infra14:06
*** gokrokve_ has joined #openstack-infra14:07
*** basha has quit IRC14:07
*** ominakov has quit IRC14:08
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Adding Search endpoints and sqlalchemy impl  https://review.openstack.org/10147614:08
*** chandan_kumar is now known as chandankumar14:09
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Adding Search endpoints and sqlalchemy impl  https://review.openstack.org/10147614:09
*** gokrokve has quit IRC14:11
*** devkulkarni has joined #openstack-infra14:11
*** nosnos has quit IRC14:13
*** mestery has joined #openstack-infra14:14
*** nosnos has joined #openstack-infra14:14
*** basha has joined #openstack-infra14:15
openstackgerrityolanda.robla proposed a change to openstack-infra/nodepool: Build images using diskimage-builder  https://review.openstack.org/4648214:15
*** ominakov_ has joined #openstack-infra14:15
*** camunoz has joined #openstack-infra14:16
*** jamielennox has quit IRC14:17
*** camunoz_ has joined #openstack-infra14:17
*** camunoz_gone has quit IRC14:17
*** camunoz_ has quit IRC14:18
*** radez_g0` is now known as radez14:18
*** ianw has quit IRC14:18
*** camunoz_ has joined #openstack-infra14:18
*** ianw has joined #openstack-infra14:18
*** nosnos has quit IRC14:18
*** camunoz_ has quit IRC14:18
*** ianw has quit IRC14:18
*** camunoz_ has joined #openstack-infra14:19
*** jamielennox has joined #openstack-infra14:19
*** ianw has joined #openstack-infra14:19
*** msabramo has quit IRC14:19
*** ianw has quit IRC14:19
*** camunoz__ has joined #openstack-infra14:19
*** ianw has joined #openstack-infra14:19
ArxCruzanteaya: hello14:19
ArxCruzWe are facing some network issues in our infra, we are already working on it14:20
*** jamielennox_ has joined #openstack-infra14:21
*** camunoz has quit IRC14:21
*** camunoz has joined #openstack-infra14:22
*** jamielennox_ is now known as jamielennox|away14:22
*** jamielennox has quit IRC14:23
*** annegent_ has joined #openstack-infra14:24
*** ianw has quit IRC14:24
*** camunoz_ has quit IRC14:24
yolandamordred, jeblair, i pushed an approach of building image once and deploying for all providers, but it still needs work, as image update doesn't need the provider to be passed (for dib), and also image-update command shouldn't need the provider in this case. Can i get some feedback?14:24
*** camunoz__ has quit IRC14:24
*** ianw has joined #openstack-infra14:25
*** Longgeek_ has quit IRC14:25
*** Longgeek has joined #openstack-infra14:26
*** dims_ has quit IRC14:28
*** zz_dimtruck is now known as dimtruck14:29
*** salv-orlando has quit IRC14:29
*** reed has joined #openstack-infra14:30
*** vhoward- has joined #openstack-infra14:31
*** gondoi is now known as zz_gondoi14:34
*** nati_ueno has joined #openstack-infra14:34
*** wenlock has joined #openstack-infra14:35
openstackgerritA change was merged to openstack-infra/config: Make ATC lists sort E-mails with preferred first  https://review.openstack.org/8538514:36
openstackgerritA change was merged to openstack-infra/config: Modernize ATCs script to Gerrit 2.8, programs.yaml  https://review.openstack.org/10269814:38
*** nati_ueno has quit IRC14:38
*** eharney has joined #openstack-infra14:39
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for Neutron functional test failures bug 1336172  https://review.openstack.org/10386414:39
uvirtbotLaunchpad bug 1336172 in neutron "neutron/agent/linux/ovs_lib.py: int(ofport) returns 'NoneType' instead of string or a number" [Undecided,New] https://launchpad.net/bugs/133617214:39
*** gokrokve_ has quit IRC14:40
jeblairKiall: so the main thing is that code is used for both long-running servers, like review.o.o, etc, as well as test nodes.  i don't think we want to change anything about our long-running servers, so i think the puppet needs to grow to handle both cases separately.14:40
sdaguefungi: we've got a config revert coming in for something that's causing 100% failure in neutron functional tests14:41
sdaguewhich means blocking tempest / devstack / neutron changes14:41
*** dims_ has joined #openstack-infra14:41
fungisdague: oh, exciting14:41
jeblairyolanda: i'll take a look in a bit, thanks14:42
sdaguealso, is there something to be done to clear out all the stuck check jobs? (like fail them)14:42
openstackgerritHenry Gessau proposed a change to openstack-infra/config: Revert "Proper passing of SUDO flag for neutron functional tests"  https://review.openstack.org/10390914:43
*** dkehn__ has joined #openstack-infra14:43
fungisdague: push new patchsets, or a zuul restart14:43
yolandajeblair, thx, i only updated the part of build and update images, but i think the imageupdate command and checks needs some rewrite in that case14:44
*** zz_gondoi is now known as gondoi14:44
yolandabut now image-update needs provider to be passed14:44
sdaguefungi / jeblair: https://review.openstack.org/103909  would be good to move through fast14:44
sdaguefungi: ok, I suspect most of the authors don't know their patches are stuck because they got no negative feedback14:45
jeblairsdague, fungi: is the cause of the stuck jobs known?14:45
sdaguejenkins03 died yesterday14:46
*** dkehnx has quit IRC14:46
fungijeblair: yes, jenkins03's network connectivity completely dropped yesterday morning14:46
fungijeblair: rebooting it brought it back to being reachable again14:46
jeblairoh, probably the rax network maint14:46
jeblairoh, maybe not then if manual intervention was required14:47
fungijeblair: nothing obvious in its logs indicating why. it was responsive from the virtual console but with no backdoor passworded account to log in with, i couldn't investigate live from that end14:47
*** msabramo has joined #openstack-infra14:48
Kialljeblair: Okay, that sounds reasonable.. I'll take a stab at it14:49
*** hdd_ has quit IRC14:49
fungijeblair: investigation showed we lost contact with nothing else, just jenkins03. figured if a reboot didn't clear it or if it recurred, i'd open a ticket with rackspace about it14:50
*** chuckC has joined #openstack-infra14:50
jeblairfungi, sdague: i'm assuming the problem that was trying to be solved on the neutron functional tests is that the OS_SUDO_TESTING env var wasn't available to tox and the test runner14:50
jeblairfungi, sdague: so probably the command wants -E, or we want to set the env variable in tox.ini14:51
*** Longgeek has quit IRC14:51
*** Longgeek has joined #openstack-infra14:51
sdaguejeblair: possibly, I'd rather revert to working14:51
sdagueand give those folks a chance to regroup with the right change later14:52
*** gokrokve has joined #openstack-infra14:52
jeblairsdague: i'm assuming we'll have to deal with it again at some point.  i'd prefer one change to two.  but i can also just be stingy reviewing a later change.14:54
*** xianghuihui has quit IRC14:54
*** eharney has quit IRC14:54
*** devkulkarni has joined #openstack-infra14:54
*** rdopieralski has quit IRC14:55
*** james_li has joined #openstack-infra14:55
*** mzoeller has joined #openstack-infra14:56
*** flaper87 is now known as flaper87|afk14:57
sdaguejeblair: honestly, no idea. Right now it's blocking a bunch of stuff, including the devstack change that probably helps with the ES backlog. The neutron team is in favor of a revert14:58
HenryGI vote for a revert to give us more time to find the real issue/solution. Right now all of neutron is up in arms.14:58
jeblairfungi: i'm preparing for a zuul restart14:58
openstackgerritA change was merged to openstack-infra/config: Revert "Proper passing of SUDO flag for neutron functional tests"  https://review.openstack.org/10390914:58
fungijeblair: okay14:58
HenryGThanks!14:58
*** PLATOSCAVE has joined #openstack-infra14:59
*** david-lyle has quit IRC14:59
*** HACKING-TWITTER has quit IRC15:00
*** david-lyle has joined #openstack-infra15:00
mordredfungi: nod (re cronspam)15:00
*** marun_afk is now known as marun15:01
*** adrian_otto has joined #openstack-infra15:01
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Comment on current state of neutron functional test  https://review.openstack.org/10391915:01
jeblairsdague, HenryG: sure, but we've gone from obviously-broken to un-obviously broken.15:01
*** dstanek is now known as dstanek_zzz15:02
sdaguejeblair: we've also gone from 100% blocking non neutron projects, to letting them merge code15:02
jeblairsdague, HenryG: i'd prefer the tree either work, or explain why it doesn't actually do what it looks like it says it does15:02
jeblairsdague: if you think my request is unreasonable, feel free to -1 that patch15:03
*** flaper87|afk is now known as flaper8715:03
*** david-lyle has quit IRC15:03
*** Longgeek has quit IRC15:03
*** david-lyle has joined #openstack-infra15:03
*** david-lyle has quit IRC15:03
sdaguejeblair: I'm fine with your comment15:04
*** david-lyle has joined #openstack-infra15:04
*** moted has joined #openstack-infra15:05
jeblairrestarting zuul15:06
*** otherwiseguy has quit IRC15:06
*** ominakov_ has quit IRC15:06
*** otherwiseguy has joined #openstack-infra15:08
*** gokrokve has quit IRC15:08
*** HACKING-TWITTER has joined #openstack-infra15:08
*** HACKING-TWITTER has joined #openstack-infra15:09
*** dstanek_zzz is now known as dstanek15:09
*** PLATOSCAVE has quit IRC15:09
*** HACKING-TWITTER has quit IRC15:09
*** HACKING-TWITTER has joined #openstack-infra15:09
*** sarob has quit IRC15:10
jeblairclarkb, fungi, mordred: the new pipeline requirements prevent zuul enqueue from enqueing changes in gate (they don't pass the clean check req)15:10
*** lttrl has joined #openstack-infra15:10
*** sarob has joined #openstack-infra15:10
*** ominakov_ has joined #openstack-infra15:10
jeblairi've rechecked the 3 things in the gate queue manually15:11
*** david-lyle has quit IRC15:12
*** talluri has joined #openstack-infra15:12
fungijeblair: ohh... good to know. i wonder how we'd want that override to work (obviously there are some requirements that we want to be able to skip in that situation and others we want still enforced as a safety check?)15:12
fungior should we add a zuul vote subcommand?15:13
jeblairfungi: oh, that's in interesting idea -- simulate a gerrit event?15:13
fungiseems like there might be more than one use case for that as a feature15:14
hasharzuul event --trigger gerrit -- --code-review +2 --workflow +1 -m "Oh yeah!"15:14
jeblairhashar: you forgot "--username kool-aid-man"15:14
*** sarob has quit IRC15:15
*** yolanda has quit IRC15:15
*** jpich has quit IRC15:15
fungibwahaha15:15
*** jpich has joined #openstack-infra15:16
*** yolanda has joined #openstack-infra15:16
hasharI should be attending your infra meeting in a few hours. Hoping to quickly introduce my evil Zuul Cloner15:16
jeblairhashar: neato15:17
*** eharney has joined #openstack-infra15:17
*** praneshp has joined #openstack-infra15:20
*** basha has quit IRC15:21
hasharand I found out that 'git fetch' from a Zuul local repo takes up to 15 seconds :/15:23
hasharit apparently send all the local refs/zuul/ to Gerrit and do a diff which takes a while15:23
jeblairhashar: yeah, we need to do something to clean up old refs periodically15:24
*** talluri has quit IRC15:24
*** ihrachyshka has quit IRC15:24
hasharI thought about altering the ref format using year/month to namespace them15:24
hasharwould make time based garbage collection trivial15:24
jeblairhashar: good idea15:25
*** talluri has joined #openstack-infra15:25
hasharI am dreaming about Zuul workflow overnight15:25
jeblairyaay! :)15:25
fungithere might already be a git feature to find refs older than a certain date?15:26
fungiif so, namespacing them might not be necessary to accomplish that15:26
fungigit gc --prune=<date>15:27
jeblairi didn't think refs had that info; i thought there were merely dumb pointers15:27
fungii guess --prune just handles loose objects15:27
jeblairfungi: yeah, and there won't be that many because most of them will have refs pointed at them15:28
fungiyep15:28
fungithough the fs timestamp on the ref file might give it away15:29
*** mmaglana has joined #openstack-infra15:29
*** talluri has quit IRC15:29
*** adrian_otto has quit IRC15:29
jeblairfungi: except i think we pack them?  so most of them will just be in one file15:30
*** changbl has joined #openstack-infra15:32
fungiahh, yeah. poking around in the trees on zm01 to see if there15:32
*** skolekonov has quit IRC15:33
hasharyup git pack-refs is in a cron15:33
hasharI have stolen that cron on my setup :D15:33
*** dimtruck is now known as zz_dimtruck15:33
*** mmaglana has quit IRC15:33
hasharunrelated note for pelix : we have a bit more information for Gerrit hash mismatch bug https://bugzilla.wikimedia.org/show_bug.cgi?id=53895#c22 \O/15:34
hasharout for some fresh air, bbl15:36
mordredfungi, jeblair: we're on updated images everwhere except two images15:36
YorikSarfungi: btw, how about merging my workaround for "hash mismatch" in git-review tests?15:36
*** Shrews_ is now known as Shrews15:36
*** virmitio has joined #openstack-infra15:36
*** timrc-afk is now known as timrc15:36
*** hashar has quit IRC15:36
fungiYorikSar: i'm okay with it, was just hoping another core reviewer would chime in as well15:36
mordredI'm kicking those by hand right now15:36
YorikSarhttps://review.openstack.org/#/c/98354/15:36
*** mmaglana has joined #openstack-infra15:38
pelixhashar: wow that's a nice suble bug :)15:38
*** IvanBerezovskiy1 has left #openstack-infra15:38
YorikSarfungi: Hm... Who of core team might know what that's all about?15:38
sdagueso once this merges - https://review.openstack.org/#/c/103658/ - kicking off a nodepool image build would be good. Right now we're downloading the trove image every time.15:39
fungiYorikSar: clarkb has also been looking at that bug from time to time15:39
*** flaper87 is now known as flaper87|afk15:40
YorikSarfungi: Oh, right. I see him in CC.15:40
fungiYorikSar: though hashar's link is interesting--if we get a gerrit point release with that fixed mina-sshd the problem may just go away15:41
YorikSarclarkb: Can you please take another look at https://review.openstack.org/#/c/98354/ and approve it if you're OK with it?15:41
fungiYorikSar: my only reservation about retrying ssh connections automatically in the tests is that there's a potential for it to mask other bugs15:41
*** msabramo has quit IRC15:42
*** flaper87|afk is now known as flaper8715:42
fungithe commit message is hilarious too... "[SSHD-330] Handshake fails (wrong shared secret) 1 out of 256 times"15:42
YorikSarfungi: I think we should test with Gerrit release that is deployed in community. And I doubt anyone would want to upgrade it soon,15:42
*** lttrl has left #openstack-infra15:42
*** gangil has joined #openstack-infra15:43
*** lttrl has joined #openstack-infra15:43
fungiYorikSar: well, we currently test with 2.8.5, and this would probably end up in 2.8.6 which we'd be likely to upgrade to in production fairly quickly as well15:43
YorikSarfungi: Retry happens only if 'hash mismatch' string is found. So it shouldn't hide other bugs, only this one.15:43
*** gokrokve has joined #openstack-infra15:43
fungiYorikSar: good point15:43
fungi"The shared secret returned by KeyAgreement.generateSecret() is a byte array, which can (by chance, roughly 1 out of 256 times) begin with zero byte (some JCE providers might strip this, though)."15:44
fungiclassic15:44
YorikSarfungi: From what it looks like we've achieved 9 out of 6 ratio in tests.15:44
YorikSar*out of 1015:44
*** adrian_otto has joined #openstack-infra15:45
*** dstanek is now known as dstanek_zzz15:47
mordredI think 9 out of 6 is what I aim for15:47
YorikSarhashar, fungi, jeblair: I think simple 'find .git/refs/zuul -not -ctime 10 -delete' would clean those refs good enough.15:50
fungiYorikSar: as pointed out, packed refs defeat that15:50
fungiYorikSar: since they all end up as lines in the .git/packed-refs file15:50
*** markwash has joined #openstack-infra15:50
YorikSarfungi: But still it would send all refs to Gerrit wouldn't it?15:51
YorikSarfungi: Ah, you mean, this solution sucks. Right.15:51
*** pballand has joined #openstack-infra15:51
*** basha has joined #openstack-infra15:52
fungiYorikSar: yeah, looking at the data stored for packed refs, namespacing them does indeed seem to be about the only sane solution after all15:52
*** dangers_` is now known as dangers15:53
YorikSarfungi: I think you can do the same find but with logs instead refs to find refs that should be deleted...15:53
*** thedodd has joined #openstack-infra15:54
*** markmc has quit IRC15:55
fungimaybe with the reflog... but does expiring unreachable refs from the reflog also remove the corresponding refs from the repo?15:56
*** hashar has joined #openstack-infra15:56
*** basha has quit IRC15:57
*** praneshp has quit IRC15:57
YorikSarfungi: No, it won't. But we can at least find them there.15:57
*** ominakov_ has quit IRC15:58
*** annegent_ has quit IRC15:58
YorikSarfungi: And then sed them out of packed-refs15:59
zaromorning15:59
*** praneshp has joined #openstack-infra15:59
abhishekHi all: can any one review this patch and help me to get this merged?16:00
abhishekhttps://review.openstack.org/#/c/102215/16:01
abhishekThank you16:01
*** hashar is now known as hasharMeeting16:01
*** msabramo has joined #openstack-infra16:01
fungiYorikSar: i'm a little uncomfortable directly manipulating files in .git just because it's not really a stable api for git. we'd probably still want to use actual git calls to delete the refs once we identify them16:01
*** vhoward- has left #openstack-infra16:01
*** chandankumar has quit IRC16:02
*** zz_dimtruck is now known as dimtruck16:02
jeblairfungi: i'm not aware of a way to delete packed refs other than manipulating (or removing) the packed-refs file :/16:02
*** Ng has quit IRC16:02
fungioh, ewwww16:03
*** gondoi is now known as zz_gondoi16:03
*** david-lyle has joined #openstack-infra16:03
YorikSarjeblair: git tag -d zuul/smth should do that16:03
*** Ng has joined #openstack-infra16:03
openstackgerritMonty Taylor proposed a change to openstack-infra/devstack-gate: Stop doing tricky things with mirrors  https://review.openstack.org/10336316:03
jeblairYorikSar: what a strange thing for the tag command to do :)16:03
fungii would have expected git branch -D to be able to do it16:04
fungibut i guess not16:04
*** Ng has quit IRC16:04
pleia2good morning16:04
YorikSarOh, git tag -d refs/zuul/smth16:04
jeblairwell, neat :)16:05
YorikSarfungi: branches are a subset of refs16:05
* mordred learns something new every day16:05
mordredfungi, jeblair ^^ I think that devstack-gate patch is the next thing that wants to land for the bandersnatch mirror16:06
YorikSarOh, wait. Verified that. Not tag. Direct manipulations with refs are handled by update-ref16:06
*** Ng has joined #openstack-infra16:06
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Restrict 1336172 query on build_name  https://review.openstack.org/10393316:08
*** whoops has joined #openstack-infra16:09
*** salv-orlando has joined #openstack-infra16:10
*** unicell has quit IRC16:10
*** sarob has joined #openstack-infra16:11
*** e0ne has quit IRC16:11
*** mzoeller has quit IRC16:11
*** anode has joined #openstack-infra16:11
*** sarob_ has joined #openstack-infra16:13
YorikSarjeblair, fungi: So smth like this: cd .git/logs; find zuul -cname 10 | sed 's|./|delete |' | git update-ref --stdin16:13
mordredjeblair: you know a lot about python logging ... if this is the log module for bandersnatch: http://paste.openstack.org/show/85249/16:14
*** mmaglana has quit IRC16:14
mordredjeblair: is there a way I can control that with a logging config file?16:14
*** unicell has joined #openstack-infra16:14
*** crosebraugh has joined #openstack-infra16:15
*** mmaglana has joined #openstack-infra16:15
*** crosebraugh has left #openstack-infra16:15
*** sarob has quit IRC16:15
*** rcarrillocruz has quit IRC16:15
mordredjeblair: I have to make a patch to bandersnatch to be able to do that don't I?16:16
*** rcarrillocruz has joined #openstack-infra16:16
*** signed8bit has joined #openstack-infra16:16
jeblairianw: in 101110, why did you chose to sort the requests twice rather than change the getPriority function to incorporate a "time waiting" term?16:17
YorikSarmordred: Yes16:17
* mordred cries a little bit16:17
*** sarob_ has quit IRC16:17
YorikSarmordred: But it's just one call that can happen after this method is called.16:18
fungiYorikSar: in your example, did you mean -ctime? and .git/refs instead of .git/logs?16:18
jeblairmordred: yeah, that's nicely hard-coded!16:18
*** zhiyan is now known as zhiyan_16:19
*** zz_gondoi is now known as gondoi16:19
*** mmaglana has quit IRC16:19
YorikSarfungi: "-ctime n" means "file's status has been changed more that n*24 hours ago"16:19
fungiYorikSar: yeah, you said -cname (not a valid find option) which is why i asked16:20
fungii'm familiar with -ctime16:20
*** gyee_ has quit IRC16:20
*** arnaud has joined #openstack-infra16:20
YorikSarfungi: And cd to logs so that we can find those stale refs by their old logs.16:20
fungiYorikSar: yeah, those refs don't seem to have any logs16:21
YorikSarfungi: Oh, right, didn't see my mistake16:21
YorikSarfungi: Oh...16:21
YorikSarfungi: That's sad...16:21
*** cindyo has joined #openstack-infra16:21
YorikSarfungi: How are they created then?16:21
fungiYorikSar: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/merger/merger.py#n13316:22
*** mugsie_ has quit IRC16:23
fungiYorikSar: gitpython's repo.create() method16:23
*** abhishek has quit IRC16:24
fungier, git.reference.create16:25
*** pballand has quit IRC16:25
*** andreaf has quit IRC16:26
fungianyway, the more i dig into this, the more sure i am that zuul is going to need to record something which can be used to identify the age of the ref, so encoding it into the ref's name seems as sane as anything16:26
YorikSarfungi: Well... Then I guess some Python script should remove them...16:27
fungiYorikSar: well, or at least (pre)identify them for us16:27
*** Ryan_Lane has joined #openstack-infra16:28
YorikSarfungi: Yeah. We can't rely on any information left then.16:28
*** gyee has joined #openstack-infra16:28
*** talluri has joined #openstack-infra16:28
*** dstanek_zzz is now known as dstanek16:28
*** pballand has joined #openstack-infra16:29
*** arnaud has quit IRC16:30
YorikSarfungi: Btw, are mergers heavily loaded? I wonder if it'd be better to use libgit2 bindings instead GitPython16:32
*** amitpp has joined #openstack-infra16:32
*** rcarrill` has joined #openstack-infra16:32
*** arnaud has joined #openstack-infra16:33
mordredjeblair, YorikSar: does this look reasonable to you: http://paste.openstack.org/show/8525116:33
jeblairYorikSar: libgit2 (pygit2) looks like it supports creating merges, so we could probably use it.16:34
*** bdpayne has joined #openstack-infra16:34
jeblairi've been starting to think that gitpython may be becomming unmaintained16:34
mordredjeblair: uhoh16:34
*** rcarrillocruz has quit IRC16:34
YorikSarjeblair: Last release is RC in 2011...16:35
fungiyeah.... :/16:35
jeblairwhich, to be fair, is about when we started using it :)16:35
mordredjeblair: pip install pygit2 fails, btw16:36
jeblairyay!16:36
mordredjeblair: it does not seem to declare dependencies16:36
mordredalso, libgit2 does not exist in debian16:36
jeblairmordred: that's because it has none! http://libgit2.github.com/16:36
jeblairmordred: "Zero Dependencies"16:36
mordredthat's libgit216:36
jeblairi'm attempting to make a funny16:37
mordred:(16:37
YorikSarmordred: Looks fine except wording for 'fp' and I'm not sure if extra check for existense worth it.16:37
mordredYorikSar: oh - well, I just copied that part from gerritbot :)16:37
*** jgallard has quit IRC16:38
mordredjeblair: so our choices are start to use a new library which is very hard to install ... or adopt another abandoned library?16:38
jeblairmordred: where was setup_logging called before?16:39
mordredjeblair: it's called at the very top with no args because the author uses logging to report config errors16:39
jeblairmordred: or check back in on https://pypi.python.org/pypi/dulwich16:39
jeblairmordred: which used to be insufficient, perhaps it has changed16:39
mordredjeblair: https://bitbucket.org/mordred/bandersnatch/src/e3176c3426e5d5a81510c870fc486123e91d6d9a/src/bandersnatch/main.py?at=default#cl-2916:40
YorikSarmordred: How did pygit2 fail?16:40
mordredYorikSar: in two ways16:40
mordredYorikSar: one - there is no libgit216:40
mordredoh! wait16:40
YorikSarmordred: It is in Ubuntu packages :)16:40
jeblairmordred: though dulwich is GPLv2 (not lgpl)16:40
*** tkelsey has quit IRC16:40
mordredit's in ubuntu16:40
mordredit's not in debian16:40
mordredbecause *STAB*16:41
mordredzul: ^^ wth man?16:41
jeblairmordred: oO16:41
YorikSarmordred: Oh, you're on Debian?16:41
mordredYorikSar: yes. I am running debian16:41
mordredand I find it very angering when someone has uploaded something to ubuntu without uploading it to debian16:41
*** dims_ has quit IRC16:41
jeblairit's not in precise16:41
mordredjeblair: it's in trusty16:42
*** dims_ has joined #openstack-infra16:42
mordredjeblair: (I have a trusty node held to check on the bandersnatch stuff, so I just checked there)16:42
mordredYorikSar: but also, it requires cffi and pygit2 does not declare that16:42
mordredthat's easy enough to deal with - but it is kinda super lame16:42
fungiseems to be in debian experimental, but not in unstable for some unspecified reason... https://packages.debian.org/libgit216:42
mordredok. at least it's been uploaded16:43
YorikSarmordred: Ok... Missing cffi is strange...16:43
jeblairthey seem to accept pull requests16:43
*** duran has quit IRC16:43
YorikSarmordred: And here it fails even with cffi and libgit2 :)16:43
mordredoh. well,16:44
fungilooks like it was uploaded into debian/experimental by the ubuntu maintainer of libgit216:44
mordredthey're not going to accept that pull request16:44
mordredbecause they use distutils and not setuptools16:44
mordredand they recently accepted a pull request to add to the readme that you should install cffi first16:44
mordredI hate python developers16:44
fungirequest for an upload to unstable: https://bugs.debian.org/74596016:44
*** talluri has quit IRC16:45
fungiwow... "16:45
fungi"The primary reason for not uploading this to Debian Unstable is that the ABI changes with every release."16:45
pleia2yowch16:45
*** basha has joined #openstack-infra16:45
*** talluri has joined #openstack-infra16:45
fungithat may be a good reason for us not to switch to it just yet... ;)16:45
mordredI'm going to, yeah, say - uhm, wow16:46
YorikSarfungi: Python bindings follow libgit2's versions.16:46
YorikSarSo ABI might not be a problem.16:46
*** talluri has quit IRC16:46
*** talluri has joined #openstack-infra16:46
*** basha has quit IRC16:46
fungiYorikSar: well, it's a problem if distro maintainers of the c lib aren't comfortable with its development stability16:46
fungialso links to https://github.com/libgit2/libgit2/issues/239816:46
*** mmaglana has joined #openstack-infra16:47
mordredthat does not seem stable enough for us to use16:47
*** nati_ueno has joined #openstack-infra16:48
*** hasharMeeting has quit IRC16:48
*** yaguang has quit IRC16:48
YorikSarmordred: I wonder how GitHub bears using it :)16:49
fungione up-side to abandoned projects... their development is *very* stable ;)16:49
*** bdpayne has quit IRC16:50
fungiYorikSar: when did github become a free software project and try to support their project running on a variety of different platforms?16:50
reedfungi, yes, like a clockwatch stuck at 11:10 will show the exact time twice in a day and another watch running  fast will almost never16:51
fungii suspect they just pick a version and use that, and upgrade their systems to a new version when they test it16:51
*** talluri has quit IRC16:51
*** pelix has quit IRC16:52
fungiwhich works fine if you're running as proprietary infrastructure, but not so well for a cross-platform free software project16:52
*** ominakov has joined #openstack-infra16:53
YorikSarWell... In context of Python service (that wouldn't ever care about ABI) that's primarily used in our infrastructure, I don't see downsides of using such lib given it had packages in Ubuntu.16:53
dstufftmordred: you can use install_requires with distutils16:54
dstufftmordred: distutils will just ignore it, and pip/easy_install patch shit to make it work16:54
mordreddstufft: yup. but it doesn't do anything16:54
mordreddstufft: well, that does not work with pygit216:54
mordreddstufft: they have it in install_requires16:54
dstufftoutput?16:54
mordredhttps://bitbucket.org/ctheune/bandersnatch/pull-request/5/add-ability-to-configure-logging-with/diff16:54
mordredbtw16:54
*** sarob has joined #openstack-infra16:54
*** jcoufal has quit IRC16:54
*** Ng has quit IRC16:55
fungiYorikSar: the question is whether the pygit2 module is bound to a specific libgit2 version, and how you support a user doing a pip install of zuul onto different distros which may have a variety of libgit2 (and thus pygit2) versions, since the latest pygit2 on pypi probably doesn't support every libgit2 version in use16:55
dstufftuse build_clib and include the source of libgit2 for static linking for maximum distro hate16:56
YorikSarfungi: It is tightly bound to libgit2 version.16:56
YorikSarOverall I get it.16:56
*** pblaho has quit IRC16:56
mordreddstufft: http://paste.openstack.org/show/85256/16:56
*** ihrachyshka has joined #openstack-infra16:56
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Cleanup manual jobs  https://review.openstack.org/10364616:56
YorikSarOf course we could do an enteprise move and bundle .so library with Zuul :)16:56
*** anode has quit IRC16:56
mordredYorikSar: no16:56
mordredYorikSar: I know you're joking - but no16:57
*** Ng has joined #openstack-infra16:57
fungiYorikSar: right, so pip install zuul wouldn't be able to automatically pull in the right pygit2 from pypi to support a particular platform's libgit216:57
*** arnaud has quit IRC16:57
dstufftmordred: oh they are importing cffi in setup.py, they need to do a little more machinations to make that work16:57
mordredsigh16:57
*** SumitNaiksatam has quit IRC16:57
mordredwell, in general, it's not worth futzing with16:57
mordredthe underlying c lib is unstable16:57
dstuffthttps://github.com/pyca/cryptography/blob/master/setup.py16:57
*** anode has joined #openstack-infra16:58
YorikSarmordred: (about logging patch) But htis way you'll have all debug logs from all this initialization.17:00
mordredYorikSar: I made the patch a little simpler in the pull request above17:00
*** devkulkarni has quit IRC17:01
*** devkulkarni has joined #openstack-infra17:01
YorikSarOk, I've disturbed this channel with libgit2 enough. I guess I should take a look at dulwich proposed by jeblair...17:01
*** derekh_ has quit IRC17:01
openstackgerritMonty Taylor proposed a change to openstack-infra/config: Log bandersnatch to file  https://review.openstack.org/10394217:02
mordredfungi: ^^ that should fix cronspam17:02
*** zzelle_ has joined #openstack-infra17:02
fungimordred: does http://logs.openstack.org/63/103363/2/check/check-tempest-dsvm-full/644b2c1/logs/devstacklog.txt.gz#_2014-07-01_16_13_15_066 mean that it went to pypi.python.org for the download instead of using the bandersnatch mirror?17:02
*** dkehn__ is now known as dkehn17:02
mordredfungi: we haven't fixed things for devstack yet ... https://review.openstack.org/#/c/103363/17:02
mordredfungi: but looking17:02
*** dkehn is now known as dkehnx17:03
*** atiwari has joined #openstack-infra17:03
fungimordred: i know--that's a devstack run from the 103363 change17:03
fungii'm evaluating the logs for the most recent patchset (even though it hasn't reported just yet)17:03
fungibefore approving17:03
mordredhrm17:04
*** chandan_kumar has joined #openstack-infra17:04
mordredfungi: it DOES appear to have done that ...17:04
*** basha has joined #openstack-infra17:04
*** adrian_otto has left #openstack-infra17:05
krtayloranteaya, re: qa/infra midcycle meetup - any 3rd party topics? none on agenda17:05
*** jpich has quit IRC17:05
mordredfungi:17:05
mordred2014-07-01 16:08:10.756 | + sudo install -D -m0644 -o root -g root /home/jenkins/.pip/pip.conf '~root/.pip/pip.conf'17:05
mordred2014-07-01 16:08:10.758 | install: cannot stat ‘/home/jenkins/.pip/pip.conf’: No such file or directory17:05
fungimordred: ahh, yep, my eyes glossed right over that error17:05
mordredfungi: it's possible this ran on a host that was pre-dependent patch?17:06
*** harlowja_away is now known as harlowja17:06
* fungi looks17:06
*** flaper87 is now known as flaper87|afk17:06
fungidevstack-trusty in hpcloud-b517:06
mordredfungi: any way to tell how old the node is?17:07
fungimordred: well, we can guess17:07
jeblairmordred: what's the dependent patch?17:07
*** melwitt has joined #openstack-infra17:08
fungimordred: the image for that az completed updating right at 2 hours ago, about an hour before that job started17:08
mordredfungi: the devstack-trust hpcloud-b5 image is 1.96 hours old17:08
mordredjeblair: the one that installs the pip.conf file in ~jenkins17:08
fungimordred: so if we think we would have cycled through all our devstack-trusty nodes in an hour's time, then it should have had the new script17:09
jeblair2014-07-01 16:00:46,505 DEBUG nodepool.NodeLauncher: Launching node id: 67009617:09
jeblairfungi: no need to guess ^17:09
fungioh, right!17:09
fungilogs... duhhh17:09
*** melwitt1 has joined #openstack-infra17:09
fungiso yes, it should have built from the latest image there17:09
jeblairmordred: any chance you have a link to the dependent change?17:09
mordredjeblair: one sec17:10
fungii can hold a node in that az and poke into the slave scripts to see if they're correct17:10
mordredjeblair: https://review.openstack.org/#/c/103354/17:10
mordredthis: https://review.openstack.org/#/c/103354/9/modules/openstack_project/manifests/slave.pp is the important bit17:10
*** SumitNaiksatam has joined #openstack-infra17:11
fungii'll confirm presence of those files17:11
mordredfungi: I've got them in rax-ord, fwiw17:12
*** basha has quit IRC17:12
*** melwitt has quit IRC17:12
*** sarob_ has joined #openstack-infra17:13
*** KurtMartin is now known as kmartin17:13
jeblairmordred: the select mirror script will write them, right?17:13
fungimordred: yeah, they exist with the correct ownership/permissions and contents17:14
jeblairmordred: so if you check a used node, you would expect to find them there, but written by that script17:14
jeblairmordred: only an unused node would tell you for sure, yeah?17:14
*** amcrn has joined #openstack-infra17:14
fungioh, right17:14
* fungi checks the image build logs instead17:15
*** amcrn has quit IRC17:15
clarkbo/17:15
*** bdpayne has joined #openstack-infra17:16
*** basha has joined #openstack-infra17:17
*** sarob_ has quit IRC17:17
jeblairmordred: i don't think that file is used by the nodepool puppet invocation17:17
*** msabramo has quit IRC17:19
jeblairmordred: i think it's openstack_project::single_use_slave -> [jenkins::slave, openstack_project::slave_common]17:19
fungisingle_use_slave17:19
fungiyeah17:19
fungiwas just tracing that, and completely agree17:19
*** Alexei_987 has joined #openstack-infra17:20
jeblairopenstack_project::slave appears to be the entry point for long running slaves17:21
*** whoops has quit IRC17:21
*** james_li has quit IRC17:21
*** rcarrillocruz has joined #openstack-infra17:22
*** whoops has joined #openstack-infra17:22
*** dstanek is now known as dstanek_zzz17:22
*** rcarrill` has quit IRC17:23
*** aaronorosen is now known as arosen17:23
*** melwitt1 is now known as melwitt17:24
*** Guest9812 has quit IRC17:24
*** devkulkarni has quit IRC17:24
*** ociuhandu has quit IRC17:25
*** dstanek_zzz is now known as dstanek17:25
*** mmaglana has quit IRC17:26
mordredgah17:26
mordredso - slave_common then?17:26
*** whoops has quit IRC17:26
fungimordred: probably, if the goal is to avoid duplication17:27
openstackgerritMonty Taylor proposed a change to openstack-infra/config: Put the mirror config into slave_common  https://review.openstack.org/10395417:28
*** mmaglana has joined #openstack-infra17:28
clarkbso it looks like the mirror stuff isn't all happy and that is being worked ^ is part of that17:29
clarkbjeblair: fungi: I did not follow why things are not ending up in the gate17:29
*** markwash_ has joined #openstack-infra17:29
*** lttrl has quit IRC17:29
clarkbwe seemed to go from restarting zuul to nothing can end up in the gate17:29
*** dimtruck is now known as zz_dimtruck17:30
fungiclarkb: when jeblair did a zuul restart and wanted to re-enqueue things into the gate, the new clean check pipeline requirement prevented the zuul enqueue cli subcommand from actually working for enqueuing into the gate pipeline17:30
*** e0ne has joined #openstack-infra17:30
clarkbwe have a new clean check pipeline requirements?17:30
fungiclarkb: because zuul had already removed the previous jenkins +1 verify vote when it enqueued them itself before17:30
*** markwash has quit IRC17:31
*** markwash_ is now known as markwash17:31
clarkbI think that is what I am missing in the middle there17:31
fungiclarkb: they were moved from trigger filters to pipeline requirements17:31
fungiafter the zuul restart last weekend to pick up the new patches17:31
jeblairmade the gate config much simpler17:31
clarkboh I thought we knew that wouldn't work?17:32
clarkbor maybe that was an old iteration17:32
*** whoops has joined #openstack-infra17:32
clarkboh weekend now I know why I am so confused :)17:32
fungiheh17:32
mordredfungi: can you +2 https://review.openstack.org/#/c/103942/1 as well - I'd like to land both so that we can start update-image-ing all of the images17:32
fungimordred: yep, i was digging in to make sure that exec actually runs that under a shell so that the redirects will work as intended17:32
mordredfungi: ooh, good call17:33
fungioh, right... this is not an actual puppet exec, it's a crontab line17:33
fungi+2 from me17:34
*** melwitt has quit IRC17:34
*** melwitt has joined #openstack-infra17:34
*** msabramo has joined #openstack-infra17:34
openstackgerritA change was merged to openstack-infra/nodepool: Make template and node hostnames configurable  https://review.openstack.org/9088317:34
clarkboh and check continues to use trigger requirements. maybe that was the big that wasn't quite expected to work with pipeline requires. in any case I grok now. There were zuul changes over the weekend and they affected reenqueing change today17:35
cindyofungi: My first stackforge review was merged, and package on pypi!!  How do we set the version number in the futuer?17:35
cindyofor the pypi package?17:35
fungicindyo: when you push a signed tag with a version number, it should (if the right jobs and configuration are in place in your project) upload it with that version number17:36
cindyofungi: I used to use git tag '1.0.1' since we have pbr17:36
jeblairmordred, fungi: jenkins::jenkinsuser defines file { '/home/jenkins/.pip':17:36
jeblairmordred, fungi: is that going to be a problem?17:36
clarkbjeblair: it will cause a duplicate namevar error17:37
cindyofungi: COOL!  so same tag process, but then it's automatically built17:37
fungijeblair: oh, maybe... yes17:37
cindyoyes I use a signed tag17:37
*** basha has quit IRC17:37
fungicindyo: right, same process. just make sure you're using pbr for your packaging, and that 'tox -evenv python setup.py sdist' creates a properly-named tarball in dist/17:38
fungicindyo: and the pypi registration for your project needs to have openstackci as a package index maintainer, and you need to have pypi jobs for your project in zuul's layout.yaml (in the openstack-infra/config repo)17:39
*** pblaho has joined #openstack-infra17:39
*** ominakov_ has joined #openstack-infra17:40
*** signed8bit is now known as signed8bit_ZZZzz17:40
*** zz_dimtruck is now known as dimtruck17:41
*** ominakov has quit IRC17:42
mordredjeblair: so really we can just not do file { '/home/jenkins/.pip' and require jenkinsuser ...17:42
fungijeblair: mordred: in fact... http://logs.openstack.org/54/103954/1/check/gate-config-puppet-apply-precise/e0465d8/console.html#_2014-07-01_17_34_35_59017:42
cindyofungi i think we have the registration right, would the pypi package have built automatically without that?17:42
jeblairbtw, what's it take to fix that job and make it vote?17:43
fungicindyo: it would have been built but our systems would have been unable to upload successfully... it wasn't clear from your comment that it had been uploaded automatically or you had uploaded the tarball yourself17:43
fungijeblair: i believe the problem right now is that it tries to apply things like our cgit module on an ubuntu precise node17:44
fungiwhich are only written to work on centos 6 at the moment17:44
clarkbyes it takes everything and applies it to one node type17:44
openstackgerritMonty Taylor proposed a change to openstack-infra/config: Put the mirror config into slave_common  https://review.openstack.org/10395417:44
cindyofungi: the upload worked too, I was able to install it with pip17:44
clarkbthe centos6 job has the inverse problem17:44
fungicindyo: what's the project name, out of curiosity?17:45
mordredfungi, jeblair: nibalizer and jesusaurus had some ideas on using some tool to do a proper version of that job17:45
cindyofungi: python-monascaclient, if you see problems please let me know.  Thnks!17:45
clarkbmordred: beaker17:46
*** melwitt has quit IRC17:46
*** sarob has quit IRC17:47
jeblairoh, that's where that's coming from.  :)17:47
*** melwitt has joined #openstack-infra17:48
jeblairi never knew the motivation for using beaker17:48
jeblairwe had that conversation about how beaker was trying to do what zuul and nodepool do.  then we ran out of time and didn't finish talking about it.17:48
*** arnaud has joined #openstack-infra17:48
clarkbright I think the conversation started on the wrong foot17:48
*** arnaud__ has joined #openstack-infra17:48
clarkbit should have started with we have a test, it only does rudimentary puppet apply. We can use beaker to do this properly17:49
clarkbthen figure out the overlap with existing tools17:49
jeblairclarkb: i didn't think it was a contentious conversation17:49
clarkbit wasn't17:49
clarkbbut it never framed the problem being solved17:49
jeblairwell, anyway, i hope we can continue it at some point17:50
clarkbso there was a bit of confusion around why we would even use beaker17:50
fungicindyo: i don't see the openstackci account as having permission to upload to https://pypi.python.org/pypi/python-monascaclient so it looks like someone must have uploaded it manually. also i don't see any tags in your git repository: http://git.openstack.org/cgit/stackforge/python-monascaclient/17:50
clarkb++17:50
jeblairclarkb: if beaker doesn't work out, i'm sure we can find a way to filter the node types by os and fix up the apply test17:51
cindyofungi: it was the first review (adding a .gitreview) and it was automatically built and uploaded I think. We were told to create the name and version ahead of time.17:51
cindyoon pypi17:52
fungicindyo: right, got it. usually i just put in a version of 0 when i register a pypi project17:52
clarkbjeblair: we can, but I think beaker makes a lot more sense when framed in a "let us test puppet apply without noop on appropriate nodes automagically and do testing with the result" way rather then. There is this puppet tool that overlaps with our tools what do we do17:53
cindyook, makes sense.  In the future we'll use the signed tags17:53
*** dhellmann_ is now known as dhellmann17:53
*** rcarrillocruz has quit IRC17:53
clarkbjeblair: I really like the idea of having beaker just spin up containers and on a nodepool provided node and runnin tests against that. Because it makes it easy for any dev to reproduce on their dev machines17:53
fungicindyo: so anyway, make sure to add openstackci as a package index owner in the pypi registration17:53
*** Guest9812 has joined #openstack-infra17:54
cindyofungi ok will make that change17:54
jeblairclarkb: sure, as long as that's a reasonable test environment; however, i'm sure it's better than most of the puppet testing we have, so it probably is :)17:54
clarkbmordred: safe to approve 103942?17:54
clarkbit looks safe /me approves17:55
cindyofungi: is a package index owner the same as adding a Role of Owner for openstackci?17:56
cindyoto our project17:56
fungicindyo: i believe so, yes17:56
cindyook, tks!17:56
*** Guest9812 is now known as Hal_17:56
clarkbmordred: the common slave move looks good to me as well17:57
fungicindyo: otherwise, it looks like you have the correct upload jobs defined for it in zuul's configuration, and pbr appears to create a proper sdist on a signed tag, so should be good to go. just be aware that only members of https://review.openstack.org/#/admin/groups/382,members can push a signed tag to that project17:57
nibalizerjeblair: sure lets continue that conversation :)17:58
openstackgerritA change was merged to openstack-infra/config: Log bandersnatch to file  https://review.openstack.org/10394217:59
nibalizermy only concern with docker is 'does stuff like iptables work in docker?'17:59
nibalizeri could just investigate that17:59
clarkbnibalizer: it won't but we are already working on that for disk image builder17:59
clarkbnibalizer: so I think items like that may be mostly solved17:59
clarkbactually it may be that iptables does work because network namespaces (that would be cool)18:00
nibalizermk ya i've seen monty's point18:00
nibalizeris there use of the other lxc deal breakers in infra/config i.e. nfs/iscsi?18:00
sdaguehmmm... so do we have any theories about the pip install socket timed out?18:01
sdaguehttp://logs.openstack.org/58/103658/1/check/check-grenade-dsvm/f2661dd/logs/grenade.sh.txt.gz#_2014-07-01_15_45_56_78418:01
*** signed8bit_ZZZzz is now known as signed8bit18:01
sdaguebecause that was on a rax node18:01
clarkbnibalizer: not for infra18:01
*** rcarrillocruz has joined #openstack-infra18:01
sdagueoh, never mind, it wasn't18:01
cindyofungi: without the signed tag pushed, on a merge no package would be built and uploaded - correct?18:01
clarkbsdague: I think that is what 103954 is supposed to address18:01
*** Ryan_Lane has quit IRC18:01
*** Hal_ has quit IRC18:01
clarkbnibalizer: for devstack yes, so the testing of openstack can't happen that way but the testing of infra puppet should be ok18:01
sdagueit's on an hp node that has 3ms ping to mirror.rackspace.com18:02
zulmordred:  hmmm?18:02
nibalizersure18:02
*** timrc is now known as timrc-afk18:02
nibalizeri think tthe folks working on stackforge/puppet-* would probably prefer to spin up real nodes, but infra/config can probably use the dockers18:03
*** jamielennox|away is now known as jamielennox18:04
fungicindyo: it should upload a snapshot tarball at http://tarballs.openstack.org/python-monascaclient/python-monascaclient-master.tar.gz on each new change which is merged18:05
*** dimtruck is now known as zz_dimtruck18:05
clarkbdhellmann: are you around today? looking for a quick review on moving oslo* py27/doc/pep8 jobs to trusty18:05
jeblairnibalizer: so here's the situation: we have a tool that spins up nodes for test jobs -- nodepool18:05
jeblairnibalizer: it can spin up single nodes, or multiple nodes for use in a single test18:05
openstackgerritA change was merged to openstack-infra/config: Allow integer-only release tags  https://review.openstack.org/9525718:06
jeblairnibalizer: the tests we run can do anything they want on those nodes18:06
*** zz_dimtruck is now known as dimtruck18:06
jeblairnibalizer: so it's conceivable that the stackforge/puppet-* folks could run a multi-node puppet openstack deploy test18:06
nibalizeryes18:07
jeblairnibalizer: i'm not sure that we want to do a 30 node infra deployment.  maybe someday, but let's consider that a stretch goal :)18:07
nibalizeri got some push back from you on, i think, wasting a node as the controller18:08
fungithat might be interesting as a check experimental18:08
nibalizerwhich i think is yes wasteful, but kinda necessary18:08
nibalizerjeblair: oh no i never wanted 30 infra nodes18:08
nibalizershould be at most 4 or something, i figure you just need to check the pipeline of things working together18:08
clarkbnibalizer: you need gerrit -> zuul -> jenkins -> slave -> logs -> logstash -> elasticsearch18:08
nibalizerbut yea 30 nodes, duplicate infra in ci, then test the crap out of, that would be pretty awesome18:08
jeblairnibalizer: yes -- i would like to not waste a node on the controller.  though, i would be more likely to waste a node as a controller in a multi-node situation18:09
clarkbnibalizer: I think that is our biggest node graph18:09
nibalizeryea the multinode situation is where you need the controller18:09
*** anode has quit IRC18:09
jeblairnibalizer: but if you take a step back, we don't need to waste a node telling another single node to run a job.  we have the world's biggest system for doing that already.  :)18:09
nibalizerclarkb: hrm. well thats only 818:09
*** harlowja is now known as harlowja_away18:10
nibalizerjeblair: ya, i figure we can tell beaker to use localhost and there might be some weirdness to that but we can probably overcome any of that18:10
jeblairnibalizer: so, pretending that i don't know anything about beaker -- at some point, beaker logs into the second node and runs something18:10
*** thedodd has quit IRC18:11
jeblairnibalizer: what's the difficulty in just making a job that just does the "runs something" part?18:11
*** anode has joined #openstack-infra18:12
nibalizerjeblair: i think the way to make that work is to tell it to ssh into a node called localhost :)18:12
*** arnaud__ has quit IRC18:13
*** arnaud has quit IRC18:13
nibalizerits a little gross, but for the check each server independent of the others I think its a good middle ground18:13
jesusaurusjeblair: i dont know very much about beaker myself, but i think that beaker conflates 'log in to remote system' with 'make server-spec easier to use'18:13
nibalizerya, what we have here is a unixy system with specific roles for different tools, beaker just does everything18:14
jesusaurusjeblair: so we could just use server-spec directly, but we lose some of the ease-of-use that beaker gives us18:14
jeblairjesusaurus: server-spec is the thing beaker runs to actually run a test?18:14
*** W00die has quit IRC18:14
jeblairhttps://github.com/puppetlabs/beaker/wiki/Overview suggests to me that beaker just runs whatever tests you tell it to18:15
jesusaurusjeblair: server-spec is the format the tests are written in aiui18:15
nibalizerhttp://serverspec.org/18:15
jesusaurusif bodepd is around, he can describe it better18:15
nibalizeror anode  or hogepodge18:16
jesusaurushe and i chatted about this for a while at the summit18:16
clarkbAaronGr: if you are around I would like to talk about 101603 to make sure my -1 isn't completely missing the point18:17
*** salv-orlando_ has joined #openstack-infra18:17
mordredjesusaurus, nibalizer: in this case, the "ease of use" may be a misnomer ... in our case, it's not actually 'easier' ... since it conflicts with the system we already have for doing many of those takss18:18
*** salv-orlando has quit IRC18:18
*** salv-orlando_ is now known as salv-orlando18:18
*** johnthetubaguy is now known as zz_johnthetubagu18:19
nibalizerya we could just write serverspec tests and a scrip to do puppet apply18:19
nibalizerand we'd be sorta golden18:19
clarkbmordred: so I disagree on that statement18:19
clarkbmordred: because we want tests to be reproduceable by devs18:19
hogepodgehi nibalizer18:19
*** ociuhandu has joined #openstack-infra18:19
mordredclarkb: yah18:19
mordredI get taht18:19
clarkbmordred: so if this makes it easy for devs then maybe overlap is fine18:19
nibalizeri like the idea of integrating with proper beaker because then the stackforge/puppet-* folks and us can use the same tooling18:19
mordredI'm just saying "if we only use $blah" we lose ease of us18:19
nibalizerand us and the rest of the puppet community can use the same tooling18:19
nibalizerwhich has cool feel good advantages, but might not be really worth it18:20
wenlockhey guys, so have you guys not run into any puppet exceptions today that look like http://paste.openstack.org/show/85260/18:20
mordredso- maybe we should make a patch to beaker to allow us to tell it in infra that we don't need it to do node management18:20
nibalizerand i'm pretty new to all of this18:20
jesusaurusyeah, i really like using the same tooling as the rest of the puppet community, rather than us doing our own thing off to the side18:20
nibalizerhogepodge: ohai18:20
mordredwenlock: nope18:20
clarkbwenlock: yes that is basically the hpcloud is broken go drinking issue18:20
mordredwenlock: ignore me, listen to clarkb18:21
clarkbmordred: that is facter trying to talk to metadata server18:21
clarkbmordred: and timing out because it isn't there18:21
mordredah - yes. that fun18:21
mordreddidn't we patch that out?18:21
clarkbmordred: no, jeblair -1.9'd it iirc18:21
mordredgotcha18:21
mordredjesusaurus, nibalizer: yes to using same tools as rest of puppet community18:22
fungiif it removes a valid reason to go drinking, i'm fully -2 on it18:22
hogepodgeThe fundamental problem of testing multiple systems is not having the sub test-harness pollute your test environment. I would think that a Docker container for Beaker could mitigate that. It has the ability to target providerless nodes.18:22
mordredI think I just want to push back hard on us communicating that it actually makes part of our world HARDER if beaker insists on doing provisioning tasks18:22
clarkbwenlock: hpcloud doesn't always hook up the metadata server. when that happens facter fails18:22
mordredhogepodge: right, but we dont' need a docker container18:22
mordredI think it's a great idea for beaker to know how to do that stuff ...18:22
clarkbwenlock: I had luck submitting a support ticket about it once. But not much luck since then18:23
wenlockclarkb, yes, we knew that tool, we started a helion_public_ipv4 facter in our code as well, but this one is causing puppet to basically role over18:23
nibalizermordred: so i think that the designers of beaker WANT you to use WHATEVER you want to do to provision nodes18:23
mordrednibalizer: awesome18:23
nibalizeryou write a hypervisor plugin to beaker18:23
mordrednononono18:23
hogepodgewouldn’t beaker pollute the test environment with libraries?18:23
mordredI don't want beaker to do that18:23
nibalizermordred: no thats what they want18:23
clarkbwenlock: so I stopped spinning up centos6 nodes in east where it failed 100% of the time18:23
nibalizerthe problem is that, for us, that puts the cart before the horse18:23
wenlockclarkb wanted to hear about approaches you guys took for how to fix it... i was thinking about meta programing in a fix to work around the function thats failing18:23
clarkbwenlock: see above. go drinking :)18:23
mordrednibalizer: I know. what I want is for us to be able to tell beaker to skip the "make me a new node" step18:24
wenlockclarkb hahaha!!! gotcha18:24
clarkbwenlock: if you depend on metadata server and it isn't there you are pretty dead in the water18:24
mordredhogepodge: I think, given what we're testing, "polluting the test env with libraries" is less of an inssue18:24
hogepodgemordered that functionality exists.18:24
nibalizerif we gave the beaker node credentials and a bit of code, it could api call into hp cloud or rax and provision its nodes and have its fun18:24
mordredsweet18:24
mordredhogepodge: if that exists, then I think we're golden18:24
wenlockclarkb but thats the problem we don't want to depend on it but because well its facter , puppet knows better18:24
nibalizerthat is more or less how puppet uses it and intends for you to extend it18:24
jeblairnibalizer: that's not possible18:24
mordrednibalizer: we dont' want it to provision nodes18:24
clarkbwenlock: I don't follow18:24
nibalizerjeblair: i know18:24
jesusaurusmordred: it looks like there is a --no-provision argument we can use18:24
clarkbwenlock: you need metadata server for things like ssh key injection and so on18:24
mordredAWESOME18:24
* mordred is excited now18:24
jeblairnibalizer: our test nodes will never have permission to do that because they run untrusted code submitted over the internet18:24
clarkbwenlock: it breaks a lot more than justfacter18:24
wenlockclarkb even if you don't use the facter values, it still crashes puppet18:24
nibalizeri know its not what we want, but i feel i've not said this to all of you before so i want to get it out18:24
clarkbwenlock: you just happen to see it in facter18:25
mordrednibalizer: :)18:25
nibalizerjeblair: just trying to sync stuff up18:25
clarkbwenlock: correct, so think of this as a canary18:25
wenlockclarkb right it breaks the entire puppet run18:25
clarkbwenlock: if facter can't talk to metadata server nothing else can and you shouldn't use that node18:25
nibalizermordred: so there is a patch i made which instead of provisioing, you tell beaker what ip to log into and it gets it done18:25
clarkbwenlock: you can probably work around it depending on your actual needs by eg deleting the ec2.rb facter file18:26
wenlockclarkb but it's not just a node thing18:26
nibalizerso we could leverage the mulitnode nodepooling jobs18:26
clarkbwenlock: this was mordred's proposal18:26
clarkbwenlock: ?18:26
wenlockclearkb we see it on almost all our nodes18:26
jeblairmordred: i believe nibalizer's suggestion is that we use the no-provision argument and tell beaker to log into localhost to run its test18:26
clarkbwenlock: yes I know18:26
*** UtahDave has joined #openstack-infra18:26
clarkbwenlock: on my account I cannot spin up any centos6 nodes in east18:26
jesusaurusnibalizer: did you test using --no-provision at all? does that just use localhost?18:26
nibalizerhttps://review.openstack.org/#/c/102071/18:26
clarkbwenlock: they fail 100% fo the time with this error18:26
wenlockclarkb for me it's ubuntu on west18:27
nibalizerjesusaurus: i'm not passing command line args, im setting in the nodeset file what to do18:27
jeblairmordred, nibalizer: i believe that functionally, that will work for us, but it's an awkward solution due to the fact that beaker seems to be conflating test environment setup with test running18:27
mordredjeblair: correct. that's what I'm saying.18:27
jesusaurusjeblair: yes, that is exactly the problem we are trying to work around18:27
hogepodgejeblair, right, the beaker host machine just becomes another test node18:27
wenlockclarkb and it looks like an easy fix, just catch the Timeout exception properly in this sucker /usr/lib/ruby/vendor_ruby/facter/util/ec2.rb18:27
clarkbwenlock: except not18:27
clarkbwenlock: because you need the metadata server usually18:28
nibalizerhogepodge: are yall running beaker in jenkins right now?18:28
clarkbwenlock: to eg add the first account's ssh keys18:28
mordredclarkb: do they?18:28
clarkbwenlock: or determine what your external IP is18:28
mordredclarkb: that shold have already happened18:28
mordredbecause they've connected and are running puppet18:28
wenlockclarkb exaclty thus implement helion_ec2_public_ipv418:28
nibalizerjeblair: i agree with your last statement18:28
clarkbwenlock: right but it won't work18:28
wenlockone that works for hpcloud18:28
mordredonce you're to "run puppet" - you should be able to ignore the metadata service18:28
hogepodgepuppet testing is using beaker internally from jenkins18:28
clarkbwenlock: if metadata server isn't accepting connections you lose18:28
clarkbmordred: wenlock explicitly talked about adding a special ec2 fact18:29
clarkbmordred: so I assumed they needed it18:29
mordredah. gotcha18:29
clarkbwenlock: the fix here is to keep pushing hpcloud to fix it18:29
wenlockclarkb right we don't need the fact for sure, but because ec2.rb is bugged, even if your not using it, your gona fail18:29
*** james_li has joined #openstack-infra18:29
clarkbwenlock: in the mean time you may get away with deleting ec2.rb from facter18:29
wenlockclarkb hmmm delete it18:30
clarkbwenlock: yes but anything that needs metadata will fail18:30
mordredclarkb: right. which might be fine if you don't need the metadata18:30
jeblairso if the consensus is the best way to use beaker to run a single node test on an already-allocated node (without wasting an extra control node) is ssh to localhost, i'm okay giving that a try18:30
mordredjeblair: ++18:30
nibalizerawesome18:30
clarkbjeblair: wait why not containers?18:30
nibalizerim happy to post up a review to demo that18:30
mordredclarkb: why add the complexity?18:31
clarkbcontainers seem so much better to me18:31
jeblair(and ftr, i'm okay with a control node for multi-node system)18:31
mordredwhy?18:31
mordredwe have a cloud18:31
clarkbmordred: so that you can have a gerrit talk to a zuul18:31
mordredalreayd18:31
wenlockisn't container spin up like 5 times faster?18:31
clarkbmordred: and I can run it on my laptop18:31
clarkbwenlock: spin up isn't reall an issue18:31
clarkbfor me it is more about how do I test this lcoally?18:31
clarkbcontainers with beaker make that really easy18:31
mordredI'm assuming that beaker can already do that18:31
mroddenclarkb: ++18:31
mordredright18:31
wenlocktrue18:31
nibalizerclarkb: i think jelair is saying lets get single node working first before we tackle multi node, in part because we can get consensus18:31
mordredbut WE dont' have to use a container model in infra with beaker18:32
wenlocksame reason i like vagrant18:32
wenlockso it uses docker for its' containers or something else?18:32
nibalizeryou know what would be really cool is run the multinode tests on nodepool and on docker and see if anything is different18:32
clarkbmordred: except for the go reproduce $failure issue18:32
hogepodgeIf you get single node working, you get the proof of concept of beaker hosted on its own test node.18:32
mroddeni actually use test-kitchen (which i think works with puppet, but sounds like beaker is their official thing) with LXC or an OpenStack API for testing18:32
clarkbanyways ignore me18:32
nibalizerclarkb: no i think running it on your laptop is a good use case18:33
clarkbstarting somewhere is more important18:33
hogepodgeIt will also give you an indication of if a container is required to host beaker. Despite the complexity, it think a contained beaker is the correct solution.18:33
sdaguelooks like python33 is broken across the board on oslo-incubator18:33
sdaguehttp://logs.openstack.org/15/99015/2/check/gate-oslo-incubator-python33/1f4da50/console.html#_2014-07-01_15_22_19_36018:33
*** harlowja_away is now known as harlowja18:33
mroddenthe server_specs are the common verification layer, not sure why that wouldn't be the best option here considering its easy to run them in localhost mode18:33
nibalizeri think having two files in nodesets, one being nodepool.yml and one being docker.yml or vagrant.yml is important18:33
jeblairclarkb: i'm not opposed to containers too, i have a small concern that the test env may not repro production18:33
mordredI am opposed to containers18:33
nibalizerand local developers could use the other nodesets18:33
mordredstrongly18:33
clarkbjeblair: the single node test case doesn't either18:33
clarkbmordred: why?18:33
mordredthey are an optimization. if they work as an optiization, then neat18:34
mordredbut, if this DOES NOT WORK with actual nodes if we ahve them18:34
mordredthen something is broken18:34
jeblairmrodden raises a good point18:34
clarkbso the reason is local reproduceablility18:34
clarkbits the whole reason for eg dib18:34
mordredI'm not _Eventually_ opposed to containers18:34
wenlockclarkb so devstack , local reproducible, why not that?18:35
jeblairclarkb: ideally, if it's broken for real, it's broken in the container too.  and if it isn't, then testing in containers is less testing.18:35
wenlockrun those test locally in devstack?18:35
mordredI'm just opposed to them as "The Answer" - because all they are is an easy way to have access to multiple things that look like machines18:35
*** UtahDave has left #openstack-infra18:35
clarkbwenlock: devstack isn't locally reproduceable if the current situation is evidence18:35
hogepodgeI don’t understand. I don’t want beaker poking around a node that’s being provisioned. It’s a way of creating a wall between the test sub-harness and the machines you’re testing on.18:35
clarkbmordred: thats fine18:35
clarkbmordred: I don't think anyoen is saying they are the answer18:35
*** Hal_ has joined #openstack-infra18:36
clarkbmordred: I am saying it is a good balance between what our test infra does, what beaker does, and what we should attempt to target as a reasonable use case18:36
*** thedodd has joined #openstack-infra18:36
mroddencontainers and real nodes are different, so it would have to be tested in both, but its easy to find 80% of the problems with just LXC and they are faster...18:36
*** ominakov has joined #openstack-infra18:37
clarkbmordred: right now we have almost no testing of this sort18:37
*** ihrachyshka has quit IRC18:37
jeblairmrodden: the node already exists before the test starts; spin up time is not a concern because it is '0'.18:37
nibalizerya if you just set the container test to run first18:37
clarkbmordred: we do puppet apply --noop18:37
mroddenright so simple case now would be run locally on the node18:37
nibalizerthen you'll hit a lot of the problems before you pay for starting all those vms18:37
mordredI believe we've already determined that the container is not needed to move forward18:37
*** ociuhandu has quit IRC18:37
nibalizeroh wait our systme donesn't bail does it?18:37
mordredwe dont' pay for starting all those vms18:37
jesusaurusclarkb: that would change if we start using beaker18:37
clarkbjesusaurus: right18:38
openstackgerritElizabeth K. Joseph proposed a change to openstack-infra/nodepool: Fix Configuration link in docs  https://review.openstack.org/10396118:38
mordredthey're already started18:38
mroddeni think i would just do a puppet-apply and run the server_specs, you'll have to write server_specs for beaker too18:38
clarkbjesusaurus: so I can run puppet apply --noop just about anywhere safely18:38
clarkbjesusaurus: if you go to beaker ssh localhost that changes18:38
jesusaurusoh right18:38
clarkbit makes testing locally harder18:38
nibalizermrodden: you don't write server spec for beaker you write a slightly different syntax called beaker-rspec18:38
mroddenthat seems weird...18:38
*** ominakov_ has quit IRC18:39
mroddencan you plug in server_specs?18:39
wenlockwhy wouldn't you use puppetlabs spec helper?18:39
nibalizerthis by the way is what the beaker-rspec tests for my puppetboard module look like: https://github.com/nibalizer/puppet-module-puppetboard/blob/master/spec/acceptance/class_spec.rb18:39
clarkbbut I think getting something is more important at this point than arguing fine points without any data18:39
nibalizerfor the curious18:39
mordredclarkb: ++18:40
mordredso - to summarize18:40
clarkbso I am fine with the beaker ssh localhost step. But I also think we shouldn't throw out alternatives that make local testing easier18:40
mordredwe have a system that provides large numbers of available nodes that have full root access for things runing on them18:40
clarkbimo one of the biggest problems we have today is that our massive CI system makes it too easy to throw local testing out18:40
mordredany solution we work on that also solves the problem of "getting a node is costly"18:40
mordredis doing something wrong for us18:40
mordredotoh18:41
mordredclarkb's point about local testing is important18:41
mordredso if there were a way to have the tests that we run in infra also be able to be run in a local container for a dev18:41
*** hashar has joined #openstack-infra18:41
clarkbI think sdague is the only person that runs devstack + tempest locally18:41
mordredwithout infra having to run them in a container18:41
mordredthat would be teh awesome18:41
sdagueclarkb: just about18:41
nibalizerso using beaker means we can have two nodeset files (the specifications of the machines that beaker will configure) and we could have one called nodepool.yml and one called docker.yml and one called vagrant.yml18:42
nibalizernodepool.yml runs in our ci, docker.yml and vagrant.yml can be run at home18:42
jeblairnibalizer: yeah, that's kind of along the lines i was thinking18:43
clarkbnibalizer: so the problem with that is that the ones used not in ci will break in about 2 weeks18:43
mordrednibalizer: ++18:43
nibalizerim confident that clark is already typing 'if we dont test vagrant/docker in ci it will break'18:43
nibalizeroh i was write18:43
nibalizerer right18:43
mordredhow about18:43
mordredwe try nibalizer's thing18:43
mordredfor a bit18:43
mordredand then18:43
mordredif we discover that vagrant/docker breaks18:43
mordredwe can test it18:43
jesusaurusi like the multi-file approach to start with18:43
mordrednot because we need docker for our testing - but because at that point we're testing that the dev env works18:44
mordredwhich I can get behind18:44
jeblairmordred: assuming we have at least one vagrant/docker user, sure.  :)18:44
mordredjeblair: yup18:44
*** arnaud has joined #openstack-infra18:44
wenlockif vagrant/docker break and it remains broken, it means they hold no value for developers anyway18:44
openstackgerritElizabeth K. Joseph proposed a change to openstack-infra/nodepool: Fix Configuration link in docs  https://review.openstack.org/10396118:44
mordredI want to see someone caring about that case before we care about in infra18:44
clarkbwenlock: that is not true18:44
nibalizeralso if the docker/vagrant stuff breaks its probably because the ubutun/centos images used are having drifftt from what we provision in nodepool, and so if we're actually buidling those in dib we could publish docker and vagrant images18:44
clarkbwenlock: stuff bitrots18:44
mordredclarkb: I agree with wenlock18:44
nibalizerwhich would be kinda cool18:44
clarkbwenlock: all the time. That doesn't have to do with holding value for devleopers18:45
nibalizerokay /me -> shower -> infra meeting -> worldcup18:45
sdaguemordred: based on bitrot of fedora stuff in devstack, it doesn't mean it doesn't hold value18:45
nibalizerthansk a lot everyone for making this conversation fun and productive!18:45
wenlockso its more than just the local test use case then18:45
jeblairclarkb, mordred, wenlock: i think if they keep breaking and don't get fixed, they hold little value18:45
sdagueit means that devs ragequit because stuff that's supposed to work doesn't18:45
mordredright now the set of devs we're talking about is about 1518:46
jeblairclarkb, mordred, wenlock: in the case of fedora/devstack, people kept fixing fedora and we said "oh, okay, so we should test that then".18:46
wenlockmordred much larger if we're using it18:46
mordredthis is a different scope than devstack18:46
mordredright18:46
mordredit's still not 200018:46
mordredwe don't need to solve this on devstack scale out of the gate18:46
wenlocktrue... <5018:46
mordredwe may need to solve it18:46
mordredjust not first thing18:46
sdaguemordred: so, unrelated to this conversation. Something is not right with wheels and py3318:47
mordredsdague: EXCELLENT18:48
sdagueas far as I can tell *all* olso-incubator jobs are failing18:48
*** anode has quit IRC18:48
clarkboh cool18:48
sdaguehttp://logs.openstack.org/60/103960/1/check/gate-oslo-incubator-python33/444dbb7/console.html#_2014-07-01_18_27_55_63918:48
mordredsdague: you gota  link to a fail? we _Are_ having half-finished mirror issues18:48
clarkbmaybe I can move them to trusty now >_> and then if anything breaks it was already broken18:48
sdague"Routes is in an unsupported or invalid wheel"18:48
*** melwitt has quit IRC18:48
mordredwow18:49
mordreddstufft: ^^18:49
sdagueit's across the board from what I can see18:49
mordredsdague: this might have something to do with the old mirror not really pulling wheels in properly18:49
sdaguethough ES is backed up again, so seeing is tough18:49
clarkboh its py33, that won't be trustied18:50
dstufftuh what18:50
wenlockclarkb can you point me to the change that was -2'd for the ec2 factered thing... i want to see some history there18:50
mordreddstufft: on the new bandersnatch mirror, we're seeing ^^18:50
mordredhttp://logs.openstack.org/60/103960/1/check/gate-oslo-incubator-python33/444dbb7/console.html#_2014-07-01_18_27_55_63918:50
mordreddstufft: which is telling us something about an invalid wheel18:50
clarkbwenlock: if I can find it18:50
dstufftdoes it work if you install it manually outside of a slave18:50
* mordred trying18:50
wenlockclarkb thanks ... crossing fingers18:51
sdagueanyway, I need to run off to csa pickup. Figured I'd raise the issue, as it is blocking all of the incubator18:51
clarkbwenlock: https://review.openstack.org/#/c/87360/ that was easier than I expected18:51
wenlockclarkb awesome sauce ty ty !18:51
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Notification Priority Constant  https://review.openstack.org/10396618:52
mordredclarkb: trusty no have python 3.318:53
clarkbmordred: right18:53
clarkbwe will switch everything tp py3.4 in a bit18:53
*** melwitt has joined #openstack-infra18:53
fungilooks like http://pypi.openstack.org/simple/Routes/ has Routes-2.0-py27-none-any.whl and Routes-2.0-py33-none-any.whl matching the requested spec, as well as a variety of tarballs18:53
wenlockclarkb ok cool... this was the same conclusion i was reaching as well on deleting it outright... if someone runs our stuff on a cloud that doesn't have issues with ec2 , then all is good... btw, we see this on private openstack clouds  too18:54
wenlockclarkb its not just a hpcloud issue18:54
*** UtahDave has joined #openstack-infra18:54
clarkbwenlock: it may be an openstack issue then18:54
*** jlibosva has quit IRC18:54
clarkbwenlock: and someone like hpcloud with reproduceable cases of it should fix it :P18:54
wenlockclarkb to me it looks like there service is falling over to be honest, and its a puppet bug in that  they don't handle the timeout exceptions18:54
clarkbwenlock: yes, puppet definitely has a bug (issue is the exception thrown by the http stuff doesn't inherit from the base exception class)18:55
wenlockclarkb which is why i was thinking a metaprogram that offers an update to the exception handling might be ok18:55
wenlockie; if they have our alternate facter, then we fix with a metaprogram there exception bug18:55
mordreddstufft, clarkb: routes installs in a virtualenv from our mirror on python3.418:55
wenlockplus they depricate that function in newer puppet18:55
clarkbwenlock: well facter and puppet are decoupled ish18:56
clarkbwenlock: also I think I decided  Icouldn't side load a fix18:56
dstufftmordred: has the routes thing happened more than once?18:56
wenlockclarkb yeah, but it cause puppet core to fail18:56
clarkbwenlock: because facter loads facts funny18:56
mordreddstufft: it's happening in all of our oslo-incubator builds apparently18:56
clarkbwenlock: so by the time you sideload the fix you are already dead18:56
fungimordred: clarkb: dstufft: though is python 3.4 going to download the 27 or 33 whl?18:56
* mordred gonna try something else18:56
dstufftfungi: 3318:57
dstufftIIRC18:57
fungiokay18:57
dstuffti'm lik e99% sure18:57
mordredfungi: there Routes-2.0-py33-none-any.whl18:57
wenlockclarkb leme test my idea here.... maybe your right on that part18:57
wenlockclarkb i was thinking facters could re-write that code18:57
clarkbwenlock: I didn't test it I just raed through the code with a (former) facter maintainer and we decided it wouldn't be pretty if it was possible at all18:57
*** ianw has quit IRC18:57
*** alexpilotti has quit IRC18:57
fungimordred: yep, see my earlier comment about what we have in our mirror for that18:58
clarkbwenlock: iirc facter loads all bundled facts first18:58
clarkbwenlock: then evaluates user provided code18:58
fungimordred: clarkb: dstufft: i can successfully install the 33 whl from our mirror by hand into a 3.3 venv too18:58
*** ianw has joined #openstack-infra18:58
clarkbwait why are we using wheels?18:58
mordredI'm now trying to install the oslo-incubator requirements from our mirror18:58
clarkbtox 1.6.1 provided pip shouldn't wheel by default18:58
*** derekh_ has joined #openstack-infra18:59
fungiclarkb: good question. presumably gate-oslo-incubator-python33 is using newer pip than provided in the venv initially?18:59
clarkbfungi: maybe?18:59
dstufftit must be18:59
*** marcoemorais has joined #openstack-infra18:59
mordredyes19:00
mordredinstall_command = pip install -U {opts} {packages}19:00
clarkbmordred: but that should use pip from tox19:00
mordredthe -U, when it hits the pbr requirement, will cause pip to get upgraded19:00
clarkbmordred: but ti won't restart pip19:00
clarkbmordred: so that only hits us if pip is called twice?19:00
mordredh - wait19:01
mordrednope. it didn't19:01
mordredit's fine19:01
dstufftactually i'm real confused now19:01
dstufftI think the error message is only in pip develop branch19:01
*** sarob has joined #openstack-infra19:01
*** olaph has joined #openstack-infra19:02
fungii'm rerunning it outside our mirror to upstream pypi just to confirm this really is mirror-specific19:02
clarkbwelcome to the twilight zone19:02
jeblairmeeting time...19:02
dstufftno I'm wrong, stupid git19:02
dstufftits' 1.5.3+19:03
*** UtahDave has quit IRC19:04
openstackgerritAdrian Otto proposed a change to openstack/requirements: Add python-marconiclient to requirements  https://review.openstack.org/10397019:05
yolandamordred, jeblair, were you able to take a look at nodepool change? is that a good approach to continue?19:09
jeblairyolanda: not yet19:09
dhellmannclarkb: sorry, I was afk yesterday and am still catching up on the resulting backlog; would waiting until tomorrow put you far behind?19:10
clarkbdhellmann: no that should be fine19:10
dhellmannclarkb: is there a patch to review, or do you just mean to do some testing by hand?19:11
clarkbdhellmann: I did testing by hand already for oslo* as reported by gerrit ls-projects. did the py27/docs/pep8 tests19:11
*** ildikov has joined #openstack-infra19:11
dhellmannclarkb: ok, and py26 runs on fedora, right?19:11
clarkbdhellmann: and https://review.openstack.org/#/c/103660/ is up for review19:11
clarkbdhellmann: centos619:11
dhellmannI was close19:12
*** devkulkarni has joined #openstack-infra19:12
fungiso running the py33 tox env myself, i do get Routes 2.0 installed from pypi (according to pip list) and seem to be running pip 1.5.619:12
clarkbfungi: hax19:12
dhellmannclarkb: if the jobs work, it's probably ok. I haven't looked at the review backlog though -- did you need any changes to oslo code or tox configs?19:12
clarkbdhellmann: no changes needed19:13
clarkbdhellmann: I was surprised that oslosphinx has no docs :)19:13
dhellmannclarkb: ironic, no? :-)19:13
openstackgerritJames Polley proposed a change to openstack-infra/nodepool: Track last allocations to ensure forward-progress  https://review.openstack.org/10111019:14
dhellmannclarkb: https://bugs.launchpad.net/oslo/+bug/133649119:15
uvirtbotLaunchpad bug 1336491 in oslo "oslosphinx needs documentation explaining how to use it" [Undecided,New]19:15
clarkbdhellmann: in any case I was just hoping for a sanity check from you. I am fairly confident in the switch as I tested the 3 tests for all the oslo* projects19:16
clarkbdhellmann: I can wait for tomorrow if that helps you19:16
dhellmannclarkb: did you test any of the oslo projects that don't have oslo in the name? (pbr, taskflow, stevedore, cliff, pycadf)19:16
clarkbdhellmann: I did not, but they won't be grabbed by my regex so should be fine for this round19:17
dhellmannclarkb: ok19:17
*** MarkAtwood has joined #openstack-infra19:18
dhellmannclarkb: I'll +1 the patch, and you can go ahead today. There's no need for you to wait if you've already tested and don't need me to test. Thanks for asking!19:18
clarkbgreat thanks19:19
openstackgerritKhai Do proposed a change to openstack-infra/config: Setup a private gerrit instance for security reviews  https://review.openstack.org/4793719:19
*** amcrn has joined #openstack-infra19:19
mordredfungi, dhellmann: I cannot reproduce the oslo-incubator fail locally19:19
fungimordred: nor can i19:20
*** arnaud has quit IRC19:20
*** ildikov has quit IRC19:20
fungii'll need to hold a node and dig deeper19:20
*** UtahDave has joined #openstack-infra19:21
*** UtahDave has left #openstack-infra19:21
*** moted has quit IRC19:22
*** doude has quit IRC19:22
*** prad has quit IRC19:23
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Session states now use Notifications  https://review.openstack.org/10397919:24
mfischIs there a way to get Jenkins to run the test suite on trunk? I have a stackforge module that I'm almost certain is broken in its current state19:24
fungimfisch: run which test suite?19:25
fungimfisch: oh, you mean without running on a change19:25
mfischfungi: yeah, like a null commit19:25
fungimfisch: we do that with periodic bitrot jobs19:25
mfischI susect that the underlying deps changed on puppet-openstack client as the tests appear to be broken through no fault of my own19:26
fungimfisch: though if this is a one-time need i suppose you could just upload a change to something insignificant, like the readme19:26
mfischfungi: simple enough19:26
*** yolanda has quit IRC19:27
*** rcarrill` has joined #openstack-infra19:28
*** lcheng__ has quit IRC19:29
*** rcarrillocruz has quit IRC19:30
*** signed8bit has quit IRC19:30
*** pballand has quit IRC19:30
*** ildikov has joined #openstack-infra19:30
*** prad has joined #openstack-infra19:30
*** signed8bit has joined #openstack-infra19:31
*** chandan_kumar has quit IRC19:31
*** yolanda has joined #openstack-infra19:31
mfischfungi: this project may be abandoned, I will talk to the commuity and maybe it should just be removed from gerrit. Thanks for the idea19:34
*** pballand has joined #openstack-infra19:38
*** jp_at_hp has quit IRC19:38
*** amitpp has quit IRC19:41
krotscheckclarkb: Are there HP things happening in the office today?19:45
clarkbkrotscheck: apparently19:45
clarkbkrotscheck: I decided to wfm to catch infra meeting and game more easily19:46
krotscheckclarkb: I did the same.19:46
*** devkulkarni has quit IRC19:46
krotscheckWell, more because sunshine.19:46
krotscheckBut now I’m sligtly crispy19:46
*** mestery has quit IRC19:46
*** mrmartin has joined #openstack-infra19:46
*** cnesa2 has joined #openstack-infra19:46
openstackgerritDoug Hellmann proposed a change to openstack-infra/infra-specs: Add spec for adding cross-project unit test jobs  https://review.openstack.org/9588519:47
*** jaypipes is now known as jaypipes-matchti19:47
*** jaypipes-matchti is now known as jaypipes-GOOAL19:47
openstackgerritDoug Hellmann proposed a change to openstack-infra/infra-specs: Add spec for adding cross-project unit test jobs  https://review.openstack.org/9588519:47
*** mestery has joined #openstack-infra19:48
*** sarob has quit IRC19:48
*** sarob has joined #openstack-infra19:48
*** sarob has quit IRC19:53
*** devkulkarni has joined #openstack-infra19:54
*** annegent_ has joined #openstack-infra19:55
*** MarkAtwood has quit IRC19:55
*** SumitNaiksatam has quit IRC19:56
*** arnaud has joined #openstack-infra19:58
fungiclarkb: wfb20:00
fungi(work from bar)20:00
clarkbjeblair: fungi: I am going to settle into watching the game, but then my plan is to delete elasticsearch01 and its volume to free up some quota, then add 1TB ssd volume to es07 and pvmove to that vlume20:00
clarkbthat sound good to you guys?20:00
fungiclarkb: cool beans20:01
*** zehicle_at_dell has joined #openstack-infra20:01
clarkbalso lunch. I have lox apparently20:01
fungii'm going to hold a py3k node real quick and see what i can figure out about the oslo-inc job, then disappear for an early dinner before coming back to polish the git-review release tasks20:01
clarkbfungi: have fun20:01
clarkbfungi: also don't forget to watch the game :P20:01
fungiclarkb: i suppose i can find a stream (or a suitable bar). i don't have teevee at the new place20:02
*** krtaylor has quit IRC20:02
fungiseemed not terribly important20:02
*** dhellman_ has joined #openstack-infra20:02
*** HACKING-TWITTER has quit IRC20:03
hasharyour meetings are nice. I hope I haven't overwhelmed you with my copy / pasted sentences20:04
fungihashar: not at all. your input is very important to us20:04
*** gokrokve has quit IRC20:05
hasharI guess I am representing the archetype of an happy "customer" :D20:05
hasharlove the product, love the guys behind it and use it daily.20:06
*** annegent_ is now known as annegentle_20:06
*** anode has joined #openstack-infra20:06
*** HACKING-TWITTER has joined #openstack-infra20:06
*** HACKING-TWITTER has quit IRC20:06
*** annegentle_ is now known as Guest7979520:06
fungihashar: i could say the same about wikipedia and mediawiki20:07
fungiyou all do a great job20:07
*** cnesa2 has quit IRC20:07
hasharthanks!20:07
*** HACKING-TWITTER has joined #openstack-infra20:08
*** HACKING-TWITTER has quit IRC20:08
*** krtaylor has joined #openstack-infra20:08
* krotscheck is trying to come up with a sane API endpoint for subscribing to a resource. Anyone have opinions on /v1/subscription/{resourcetype}/{resourceid}?20:09
*** HACKING-TWITTER has joined #openstack-infra20:09
*** HACKING-TWITTER has quit IRC20:09
fungimordred: dhellmann: sdague: dstufft: so i have an update... i can reproduce the "Routes is in an unsupported or invalid wheel" error straight from pypi.python.org, so it's not our bandersnatch mirror causing it at least20:09
fungiworking now to narrow down the reproducible case20:10
krotscheckConcrete example would be /v1/subscription/project/23220:10
dstufftfungi: what version of routes20:10
fungidstufft: the only one they've uploaded a wheel for... 2.020:11
*** HACKING-TWITTER has joined #openstack-infra20:11
*** moted has joined #openstack-infra20:11
krotscheck….but then they’re user specific. So a subresource maybe under user?20:11
* krotscheck wanders off to throw ideas on a wall and see which one feel best.20:12
dstufftlol20:12
dstufftfungi: I bet20:12
fungidstufft: http://paste.openstack.org/show/85270/20:13
*** adalbas has quit IRC20:13
*** Guest79795 is now known as theannegentle20:13
dstufftfungi: Routes-2.0-py33-none-any.whl has both a Routes-2.0.dist-info and a Routes.egg-info20:13
dstufftI bet pkg_resources is getting confused20:13
fungidstufft: sounds like we should update our version spec to include !2.0 and file a bug with them20:14
*** e0ne has quit IRC20:14
*** mbacchi has quit IRC20:14
sdaguewendar: on the MD server thing, are all the fails that you see in neutron envs?20:14
*** dims__ has joined #openstack-infra20:15
fungidstufft: the traceback from the pip log: http://paste.openstack.org/show/85271/20:15
*** Hal_ has quit IRC20:16
*** SumitNaiksatam has joined #openstack-infra20:16
dstufftfungi: yea, that means that pip couldn't find a Wheel-Version20:16
dstufftWheel-Version exists inside of a dist-info, not inside of an egg-info20:16
fungidstufft: pip==1.5.6, setuptools==3.620:17
fungimordred: ^20:17
fungidhellmann: ^20:17
*** dims_ has quit IRC20:18
mordredfungi: so it'll only show up in our tox-based things because of the pin20:18
mordredit woudl be great if tox would release the next version20:19
*** HACKING-TWITTER has quit IRC20:19
*** sarob has joined #openstack-infra20:19
*** dims__ has quit IRC20:20
fungilooks like their bug tracker is https://github.com/bbangert/routes/issues20:20
fungiaha! https://github.com/bbangert/routes/issues/3120:20
fungi"Incorrect wheel for py3k"20:20
clarkbwhy are we using 1.5.6 with tox?20:20
clarkbshould be 1.4.x iirc20:20
fungiclarkb: virtualenv 1.11.620:21
clarkboh did we update virtualenv? interesting20:21
fungiclarkb: that's what i find on py3k-precise nodes20:21
sdaguefungi: so can we block Routes 2.0 ?20:22
fungimordred: so reqs pin of Routes to exclude 2.0 and reference that upstream bug?20:22
clarkbok thats not bad20:22
sdagueis that the fix?20:22
fungisdague: i suppose?20:22
*** HACKING-TWITTER has joined #openstack-infra20:22
fungiif there's consensus, i'll submit a reqs change asap20:22
mordred++20:22
sdaguefungi: I'm good with that20:23
*** gokrokve has joined #openstack-infra20:23
fungiinbound20:23
*** sarob has quit IRC20:23
openstackgerritJeremy Stanley proposed a change to openstack/requirements: Exclude Routes 2.0  https://review.openstack.org/10399120:25
fungisdague: mordred: ^20:25
*** e0ne has joined #openstack-infra20:25
sdaguefungi: +220:25
sdagueI like the topic as well :)20:26
*** PLATOSCAVE has joined #openstack-infra20:26
fungithat wheel was apparently uploaded to pypi 2013-11-17, so it was just our failure to mirror wheels ourselves that shielded us from it before now20:27
fungiand with that, i'm heading to dinner20:27
*** mbacchi has joined #openstack-infra20:27
sdagueenjoy20:28
sdagueand thanks20:28
*** mbacchi has quit IRC20:28
*** HACKING-TWITTER has quit IRC20:28
*** mbacchi has joined #openstack-infra20:28
*** gokrokve has joined #openstack-infra20:29
*** ArxCruz has quit IRC20:29
*** nati_ueno has quit IRC20:30
*** ogelbukh has quit IRC20:30
*** julim has quit IRC20:32
*** dims__ has joined #openstack-infra20:33
*** devkulkarni has quit IRC20:33
*** julim has joined #openstack-infra20:33
*** gokrokve has quit IRC20:33
*** julim has quit IRC20:33
*** MarkAtwood has joined #openstack-infra20:34
*** sarob has joined #openstack-infra20:35
*** devkulkarni has joined #openstack-infra20:36
*** MarkAtwood has quit IRC20:38
*** amcrn has quit IRC20:38
*** dims__ has quit IRC20:41
*** saper has quit IRC20:41
marunjeblair: Given that we can't seem to replicate the gate failure locally for neutron-dsvm-functional, would it be reasonable to move it back to the experimental queue so it can be run with sudo enabled and not block merges?j20:42
marunjeblair: (non-voting of course)20:42
*** MarkAtwood has joined #openstack-infra20:42
*** gokrokve has joined #openstack-infra20:43
*** e0ne has quit IRC20:43
arosenHi was wondering if someone could review this one. I was hoping to start pushing code here :) https://review.openstack.org/#/c/102697/20:43
*** otherwiseguy has quit IRC20:43
*** andreykurilin_ has joined #openstack-infra20:46
*** MarkAtwood has quit IRC20:47
*** lttrl has joined #openstack-infra20:47
*** mbacchi has quit IRC20:50
*** signed8bit is now known as signed8bit_ZZZzz20:51
*** pdmars has quit IRC20:51
*** prad has quit IRC20:52
*** pblaho has quit IRC20:53
*** lttrl has quit IRC20:53
*** prad has joined #openstack-infra20:55
*** crc32 has joined #openstack-infra20:56
*** dprince has quit IRC20:57
*** ominakov has quit IRC20:58
*** misterpink has joined #openstack-infra21:01
*** rcarrillocruz has joined #openstack-infra21:02
mordredfungi: that's actually great - the new mirror has actually caught a real-world error that we had let through21:02
mordredjeblair: ^^21:02
*** rcarrill` has quit IRC21:04
sdaguezomg the trove image thing landed finally!21:04
sdaguewhen do nodepool images normally rebuild?21:05
clarkbI haev added an ssd volume to es07 as /dev/xvdc. Now I need to format, pvcreate, then pvmove /dev/xvdb1 /dev/xvdc1 ?21:05
sdagueonly took 3 rechecks on 2 different bugs :)21:05
clarkband as final step vgreduce main /dev/xvdb1 ?21:06
marundoes anyone know whether the 'stack' user is supposed to be able to use password-less sudo?21:06
clarkbfungi: ^ you did this before21:06
*** prad has quit IRC21:06
marunthe errors seen in this failure log suggest otherwise: http://logs.openstack.org/90/102490/4/check/check-neutron-dsvm-functional/9db189a/console.html#_2014-07-01_03_31_05_12621:06
marunsudo: no tty present and no askpass program specified21:06
*** lttrl has joined #openstack-infra21:06
clarkbmarun: I think it should doesn't devstack use that user in that way?21:07
clarkbsdague: ^21:07
marunclarkb: I'm not sure what the sudo error would indicate other than a password being required21:08
sdagueno, devstack sets that up, but devstack-gate removes those permissions before tests run21:09
sdaguehttps://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L30621:09
marunsdague: *sigh*21:10
clarkbmarun: you can use the jenkins user maybe?21:10
sdagueclarkb: I thought we took it's permissions away as well?21:10
clarkbsdague: it may, but it needs to sudo during cleanup so I would be a bit surprised if it did21:11
sdagueclarkb: it might be more limitted21:11
marunso if I need to enable sudo testing to make sure the fixes for https://bugs.launchpad.net/neutron/+bug/1336172 are going to work, do I have to disable voting or is there another way?21:12
sdaguemarun: also, I thought these were things that were expected to migrate into tempest, in which case, they need to not be calling out to sudo21:12
uvirtbotLaunchpad bug 1336172 in neutron "neutron/agent/linux/ovs_lib.py: int(ofport) returns 'NoneType' instead of string or a number" [Critical,New]21:12
marunsdague: no, these are not slated for tempest21:12
marunsdague: these are functional tests of neutron's system interaction21:12
clarkbwhich should use rootwrap right?21:12
marunsdague: i.e. low level, and requiring sudo to run21:12
clarkbwhich I assume is what devstack allows to happen21:13
clarkbotherwise how does it work21:13
marunclarkb: hmmm21:13
marunclarkb: I think the initial conception was not to require rootwrap, but there's no reason that can't be changed21:13
jeblairmarun, sdague: jenkins should have full sudo perms, and yeah, after devstack runs, sudo perms are removed21:13
sdagueyeh, rootwrap is the hole that you need21:13
clarkbwell if you are testing system interaction isn't the way to do that rootwrap?21:13
marunclarkb: that would also serve to postitively validate that rootwrap is configured correctly21:13
sdagueclarkb: ++21:13
jeblairagreed21:13
jeblairmarun: and that's actually why we remove stack user sudo perms -- to make sure that rootwrap is tested correctly in devstack21:14
clarkbjeblair: your lvm foo is probably better than mine. Does my list of steps above look correct? I need to partition, pvcreate, then pvmove?21:14
marunok, fair enough.21:14
marunso, to my previous question...21:14
marunhow do I validate the fixes for https://bugs.launchpad.net/neutron/+bug/1336172 (fix the TypeError and replace sudo with rootwrap)?21:15
uvirtbotLaunchpad bug 1336172 in neutron "neutron/agent/linux/ovs_lib.py: int(ofport) returns 'NoneType' instead of string or a number" [Critical,New]21:15
marunDo I need to disable voting or is there another way?21:15
*** otherwiseguy has joined #openstack-infra21:15
marunI don't want to cause any more havoc.21:15
sdaguemarun: so if you just change to doing rootwrap calls, that should be self testing21:16
*** mrmartin has quit IRC21:17
jeblairclarkb: i think so, and the commands in http://ci.openstack.org/sysadmin.html#cinder-volume-management should help for the first part of that21:17
marunsdague: the problem is that the tests don't run without defining OS_SUDO_ENABLED at present, so that developers can run the functional job and skip tests that require privileged commands by default21:17
clarkbthanks21:17
sdaguemarun: the issue is that config changes aren't self tested21:18
sdaguemarun: propose a version that defaults it on21:18
sdaguesee that it runs21:18
sdagueWIP it so it doesn't land21:18
sdaguethen you'll know if it works21:18
marunsdague: how do I run the neutron functional job in response to a config repo patch?21:18
sdaguemarun: you don't21:18
marunsdague: :/21:18
sdaguewhat I'm saying is make your fix patch21:19
sdagueput another patch on top of it which hard codes OS_SUDO_ENABLED=True21:19
sdaguein the neutron tests21:19
marunsdague: ah, gotcha21:19
sdaguedon't land the 2nd one, just use it to make sure it works21:19
marunsdague: my gate-foo is weak :(21:19
sdaguemarun: yeh, no worries, we do these funny tail patches quite often to force things like this21:20
clarkbok pvmove started21:27
clarkbrunning in a screen owned by me21:27
sdagueclarkb: when does nodepool respin images?21:27
clarkbsdague: 140021:28
clarkbUTC21:28
sdagueok, well about an hour ago the trove image fix landed in devstack. Once we get images based on that, we'll stop downloading trove image on every devstack run, which is about 1 minute time, and a ton of log lines21:28
sdagueand part of what I think is impacting the indexer21:29
clarkbok, I can kick them off manually now21:29
sdagueyeh, that would be great21:29
sdaguethanks much!21:29
*** amalagon has quit IRC21:29
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard-webclient: [WIP] Subscription  https://review.openstack.org/10400421:30
*** Ryan_Lane has joined #openstack-infra21:30
clarkbsdague: I am starting with rax-ord and once those images are done will do them en mass21:31
sdaguecool21:32
*** dims__ has joined #openstack-infra21:36
clarkbjeblair: fungi: I called the es07 volume ssd01... hopefully that doesn't create any confusion but it made sense to me to have the cinder side describe that bit21:36
clarkbjeblair: fungi: you can always `pvs` and compare that to cinder show21:36
*** james_li has quit IRC21:37
mordredjeblair, fungi: https://review.openstack.org/#/c/103954/ <-- need this to get our builds using the mirror again - devstack are all going to pypi.p.o21:37
clarkboh maybe I shouldn't rebuild everything so fast and wait for 10395421:38
mordredclarkb: yes please21:39
*** dhellman_ has quit IRC21:40
*** zehicle_at_dell has quit IRC21:42
*** radez is now known as radez_g0n321:43
*** mfer has quit IRC21:44
*** dkliban is now known as dkliban_afk21:45
*** bcrochet is now known as bcrochet|afk21:45
*** lcheng has joined #openstack-infra21:49
clarkb10% on pvmove21:50
*** r1chard_afk is now known as r1chardj0n3s21:50
*** jhesketh has joined #openstack-infra21:52
jheskethMorning21:52
clarkbo/21:53
*** signed8bit_ZZZzz is now known as signed8bit21:55
clarkbjhesketh: any chane you can review 103954 so that we can get that in and build new images?21:56
mordredjhesketh: yes please21:57
mattoliverauMorning all21:58
*** jgrimm has quit IRC21:58
mordredmorning mattoliverau21:59
*** theannegentle has quit IRC22:00
*** mrda-away is now known as mrda22:02
jheskethmordred: Is there a reason the .pip dir isn't ensured in that change?22:03
*** andreykurilin_ has quit IRC22:03
clarkbjhesketh: yup the jenkins::slave manifest does it22:04
*** hashar has quit IRC22:06
jheskethah cool22:06
jheskethclarkb, mordred: approved22:07
clarkbty22:07
openstackgerritA change was merged to openstack-infra/config: Put the mirror config into slave_common  https://review.openstack.org/10395422:10
*** Ryan_Lane has quit IRC22:11
mordredwoot22:13
*** james_li has joined #openstack-infra22:15
*** pcm__ has quit IRC22:15
*** weshay has quit IRC22:16
*** gondoi is now known as zz_gondoi22:17
*** zehicle_at_dell has joined #openstack-infra22:17
*** jp_at_hp has joined #openstack-infra22:18
clarkbpvmove is at 25%22:18
*** chuckC has quit IRC22:26
clarkbmordred: I am rebuilding images now22:26
mordredclarkb: awesome22:26
openstackgerritAaron Rosen proposed a change to openstack-infra/config: Add python-congressclient as new project to stackforge  https://review.openstack.org/10269722:27
*** dstanek is now known as dstanek_zzz22:28
*** sarob has quit IRC22:29
*** r1chardj0n3s is now known as r1chard_afk22:31
*** doug-fish has left #openstack-infra22:32
derekh_ianw: jeblair clarkb added comment to https://review.openstack.org/#/c/101110/822:34
*** lcheng has quit IRC22:35
clarkbderekh_: cool22:35
fungiback from eats22:37
clarkbfungi: I am rebuilding images now to pick up a couple of fixes22:37
clarkbfungi: and the es07 pvmove is in progress22:37
clarkband the US lost22:37
fungione of the many things to appreciate about life on the beach... shrimp boil22:37
clarkbthat is what hapepend while you were eating22:37
clarkbooh shrimp boil22:37
fungiclarkb: and wow, that's a fast pvmove!22:37
ianwderekh_: thanks.  don't think being proportional with sorting the waiting queue would be too big a change; i just thought that is best as a follow-on as it's already a big enough change22:38
*** otherwiseguy has quit IRC22:38
*** devkulkarni has quit IRC22:38
clarkb++22:38
mordredclarkb: do you have a 'good' process to rebuild all fo the images?22:38
*** rlandy has quit IRC22:38
clarkbI really do like the approach taken in ianw's change22:38
clarkbmordred: I fire up screen, start one build per screen window22:38
*** thedodd has quit IRC22:38
fungimordred: i open a ton of separate windows in a screen session on nodepool and yeah22:39
mordredclarkb: ok. so the answer is "no"22:39
fungithat22:39
clarkbmordred: makes it easy to stay on top of where things are for various builds22:39
clarkbmordred: its not bad22:39
derekh_ianw: yup, sounds good to me, once its in action we'll know if more is needed22:39
* mordred considers a "rebuild all" command22:39
fungifor ornery builds which need lots of retries (old hpcloud 1.0 az2 will live in infamy) i have one-liner shell script which loops until it succeeds and then dies22:39
clarkbmordred: well you usually don't want that22:40
clarkbmordred: because you want to start small22:40
clarkbone or two bad images is so much better than 5022:40
fungirebuild this, now rebuild that, now heck with it just rebuild all the others i'm sure they'll work22:40
mordredyah22:41
mordredwell22:41
mordredI mean22:41
mordredso you just described a canary build22:41
*** derekh_ has quit IRC22:41
mordredwhich means it still should be possible to have doing this be a thing that nodepool knows how to do22:41
*** PLATOSCAVE has quit IRC22:41
clarkbmordred: its not even a canary build so much as when clouds are cranky they get cranky22:41
mordredsure. but I guess it depends on why you're rebuilding22:41
*** MarkAtwood has joined #openstack-infra22:42
funginot that the new guard is at all related to the old guard, but wow cool! http://www.datacenterknowledge.com/archives/2014/06/30/openstack-integration-in-the-works-for-cray-supercomputers/22:42
*** HACKING-TWITTER has joined #openstack-infra22:43
*** james_li has quit IRC22:43
clarkbjroll: the containers discussion in the solum meeting has me curious. is your docker image build working well now?22:43
*** dims__ has quit IRC22:44
clarkbfungi: is that how you know we have made it?22:44
clarkbcray is supporting us therefore we win?22:44
jrollclarkb: the post job? I don't think so, but I haven't checked in on it22:44
clarkbjroll: ya that job22:44
clarkbalso :(22:44
fungiclarkb: when wang supercomputers claws its way back out of the scrap heap and becomes a corporate sponsor, *that's* when we'll know we've won ;)22:44
jrollclarkb: oh, JayF added this: https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/full_trusty_build.sh22:44
jrollclarkb: just need to update the job22:45
clarkbjroll: cool22:45
jrollclarkb: I can put that patch up, if you point it out to me. jay just signed off for the day, I believer22:45
jroll-r22:45
jrollpoint out where that patch should be*22:46
*** lttrl has quit IRC22:46
clarkbjroll: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/ironic-python-agent-jobs.yaml that file22:46
jrollthanks22:46
fungi(not that wang labs ever made a supercomputer, but they totally should once they're back from the grave)22:46
clarkbjroll: basically update the shell script in that job to call out to your script22:46
jrollyep, just had know idea where it was :)22:46
clarkb37.9%!22:46
clarkbalso it is warm in seattle today22:46
clarkbok I am going to take the plunge and trusty oslo* and trove* now22:49
jrollclarkb: is my cwd at the root of the project?22:49
*** dstanek_zzz is now known as dstanek22:49
clarkbjroll: yes, you end up in root of your repo22:49
jrollgreat, thanks22:49
mordredclarkb: ++22:50
*** changbl has quit IRC22:50
jesusaurusnibalizer: im looking at https://review.openstack.org/#/c/83678/15/modules/openstack_project/manifests/puppetmaster.pp and im wondering: what is the @{u} doing in that cron command? where does that come from?22:50
SlickNikclarkb: ++ on trusty trove :) (And thanks!)22:52
clarkbjesusaurus: iirc git magic22:52
clarkbjesusaurus: its a special ref alias for upstream main tracking branch or some such22:52
openstackgerritJim Rollenhagen proposed a change to openstack-infra/config: Use build script in IPA repo  https://review.openstack.org/10401222:53
jrollclarkb: ^22:53
jrollbrb22:53
jesusaurusclarkb: oh, fancy. TIL22:53
mordredjroll: your real name is so much longer than your irc nick!22:55
*** r1chard_afk is now known as r1chardj0n3s22:56
*** msabramo has quit IRC22:56
dtroyeris there a magic URL to find a commit in gerrit given the SHA?22:58
*** dims__ has joined #openstack-infra22:58
clarkbdtroyer: if you put the git sha in the search box and search for it you will see what the url looks like22:58
dtroyerI get the usual change url: https://review.openstack.org/#/c/103168/22:59
clarkboh huh I thought it gave you a listing, one sec I will dig up what it is22:59
clarkbdstufft: /#q,$1,n,z where $1 is the sha123:00
clarkber dtroyer ^23:00
fungiyou haz tabcompletefail23:00
dtroyercoolness, thanks23:01
mordredjeblair, fungi: SSL certs...23:01
*** virmitio has quit IRC23:01
fungimordred: oh, right, i saw that in scrollback this morning and then got sidetracked23:01
mordredfungi: pip carps at people now if it's pointed at non ssl23:01
fungii have not actually ordered any x.509 certs for openstack.org subdomains before23:02
clarkbneither have I23:02
mordredfungi: so we should really probably put a cert on pypi.openstack.org23:02
*** rcarrill` has joined #openstack-infra23:02
clarkb++23:02
fungibeing the only one on the team working directly for the foundation now, i'm okay if it falls on me to start taking care of those, but pointers from jeblair on prior process would be helpful23:03
openstackgerritA change was merged to openstack-infra/config: Comment on current state of neutron functional test  https://review.openstack.org/10391923:03
*** rcarrillocruz has quit IRC23:03
*** pballand has quit IRC23:03
fungimordred: however, if it's merely a cosmetic error in this case, there's some counterargument that it will act to discourage people from using our mirror in their production configurations23:04
clarkbmordred: do we need to rebuilds the bare-* nodes too?23:04
clarkbmordred: I started with devstack-*23:04
jrollmordred: soooo long.23:04
clarkbfungi: I am going to completely change the subject23:05
clarkbfungi: how d oyou want to handle all of the new project requests up there that use the old version of the gerrit acl files23:05
clarkbfungi: maybe you want to update them using your handy script?23:05
fungiclarkb: i think landing them as-is is probably fine, and then we can run another cleanup pass of that script in a week or so, and after that the chances of old patterns being cargo-culted are dwindling to nonexistent most likely23:06
clarkbfungi: ok23:06
fungiafter that if we still see people adding new projects with outmoded configuration (for example copied from a previous review) we can politely ask them to revisit the docs23:07
mordredclarkb: yeah. we should really do all of them23:07
clarkbmordred: ok23:07
*** oomichi has joined #openstack-infra23:07
*** sarob has joined #openstack-infra23:07
*** chuckC has joined #openstack-infra23:07
clarkbarosen: I am going to update your congressclient change really quickly with a thing then +2 it23:07
fungithe whole point of that script was so that we could do this in an eventually-consistent fashion with minimal disruption to review process23:08
arosenclarkb: cool thanks23:08
mordredfungi: well, I think a) in the future pip is going to change from warning to error23:08
mordredfungi: and b) we should ssl just because23:08
mordredfungi: also, dstufft wants us to23:08
fungimordred: i believe those are all (three) wholly valid reasons23:08
*** msabramo has joined #openstack-infra23:09
*** whoops has quit IRC23:09
fungiso anyway, i'll try to remember to hash out the details with jeblair if he wants/needs to no longer be doing that himself23:09
*** zzelle_ has quit IRC23:10
clarkbtwo small changes23:10
*** ildikov has quit IRC23:10
*** eharney has quit IRC23:11
openstackgerritClark Boylan proposed a change to openstack-infra/config: Add python-congressclient as new project to stackforge  https://review.openstack.org/10269723:11
*** nati_ueno has joined #openstack-infra23:11
clarkbarosen: ^ we really do like to restrict the group that can push tags as cleanup is painful23:11
clarkbarosen: and stackforge/python-designateclient had snuck back into patchset 423:12
arosenclarkb: cool that's fine with me, we can restrict it to me for now if you want.23:12
openstackgerritA change was merged to openstack-infra/config: Enable ceilometer in rally-dsvm jobs  https://review.openstack.org/10383623:13
clarkbalmost to 50% o nthe pvmove23:13
openstackgerritA change was merged to openstack-infra/config: Add py33 job for python-manilaclient  https://review.openstack.org/10380923:13
fungiclarkb: do you feel it's negatively impacting cluster performance as much as node rolls would?23:13
*** dimtruck is now known as zz_dimtruck23:13
arosenclarkb: cool thanks for merging that for me :)23:14
clarkbfungi: no I think this is mostly sane23:14
fungiawesomesauce23:14
clarkbarosen: well its +2 not +A yet :)23:14
clarkbmaybe fungi or mordred or jhesketh will change that23:14
*** otherwiseguy has joined #openstack-infra23:14
arosenclarkb: whoops k23:14
openstackgerritA change was merged to openstack-infra/config: Set docs jobs for Manila project as votable  https://review.openstack.org/10380323:14
openstackgerritA change was merged to openstack-infra/config: Add functions & cleanup install_puppet.sh  https://review.openstack.org/10373523:15
clarkbfungi: it is definitely having an affect but rebalances are really bad compared to this imo23:15
jhesketharosen: what do you need reviewed?23:15
*** wenlock_ has joined #openstack-infra23:15
mordreddone23:15
jeblairfungi: wfm -- they're $10 a pop so i doubt hp would care if i expensed it, but it seems more correct for the foundation to do so...23:15
mordredarosen: +A23:15
jhesketharosen: also, I need you to take a look at comments I had for you from a while back: https://review.openstack.org/#/c/77347/23:15
arosenthanks mordred23:15
dstufftmordred: pip will likely always allow HTTP, but the other two are totally legit23:15
mordredjeblair: I would approve that expense - but also, yeah. jbryce should really buy it :)23:16
mordreddstufft: ssssh. the misinformation is our friend23:16
dstufftI also din't think not having security is a good incentive anyways, people are amazingly willing to do all sorts of grossly inseure things if it makes their life better in the moment23:16
jeblairfungi: moreover, i try to get new certs around july of every year -- just so i only have to worry about it once a year23:16
fungijeblair: what a coincidence--it's july!23:16
jeblairfungi: so now's probably a good time to check and see what's up :)23:16
*** wenlock has quit IRC23:17
*** wenlock_ is now known as wenlock23:17
fungijeblair: should i just generate new csrs for anything coming up for expiration, and do you happen to have a comprehensive list of which ones i should keep tabs on?23:18
clarkbjenkins.o.o can probably go self signed?23:18
ianwmordred: all oslo-incubator-python33 failing with "Routes is in an unsupported or invalid wheel"; the only thing i can find via google is https://review.openstack.org/#/c/92221/ which you wrote; any tips?23:19
ianwmordred: example failing log http://logs.openstack.org/60/103960/1/check/gate-oslo-incubator-python33/444dbb7/console.html23:19
dstufftRoutes has a borken wheel23:19
clarkbianw: routes 2.0 whl is broken. requirements update pushed by fungi to remove that version23:19
fungiianw: fix is at the top of the gate, about to merge23:19
jeblairfungi: nothing expires this year :)23:19
jeblairfungi: http://paste.openstack.org/show/85284/23:20
openstackgerritA change was merged to openstack-infra/config: Revert "Downgrade puppetlabs-apache to version 0.0.4."  https://review.openstack.org/10325823:20
fungijeblair: convenient--have you been getting multi-year i guess?23:20
openstackgerritA change was merged to openstack-infra/config: Use build script in IPA repo  https://review.openstack.org/10401223:20
jeblairfungi: 2 year, yeah23:20
* dstufft is glad we have a single star cert and we just route everything through TLS termination23:20
dstufftwe = PSF23:20
ianwclarkb / fungi : thanks23:20
* jeblair is very glad that the cert on jenkins.o.o can't be used for any actually important hosts23:20
fungidstufft: yeah, for us (cloud using peeps) that would add an additional point of risk for outages23:21
fungiif we had dedicated servers in a data center together, different story perhaps23:21
jeblairand the tls termination approach would require us to additionally secure the termination<->server path against an untrusted cloud provider23:21
fungiyup23:22
dstufftfungi: pft that's what a globally distributed load balancing CDN is for :D23:22
fungis/untrusted cloud provider/dstufft's employer/ ;)23:22
*** salv-orlando has quit IRC23:22
jeblair(which isn't to say that we don't occasionally trust our cloud providers, just that we grumble about it when we do)23:22
fungiwe like to grumble23:23
*** lcheng has joined #openstack-infra23:23
clarkbargle bargle23:23
fungicurmudgeons the lot of us23:23
fungiglop glyf!23:23
jeblaircoremudgeons even23:23
*** MarkAtwood has quit IRC23:23
fungithose as well, yes23:23
clarkbseriously how hard is dns that doesn't blacklist good ips23:23
dstufftsounds pretty hard23:23
jeblairfungi: so anyway, that's the list -- i have a script on ci-puppetmaster that generates csrs23:23
dstufft(idk)23:23
dstufft(im joking too)23:23
fungiclarkb: apparently about as hard as policing your tenants who run compromised/malicious payloads23:24
gothicmindfoodeverything is hard at HP23:24
gothicmindfood(everything)23:24
mordredgothicmindfood wins23:24
clarkbgothicmindfood: I think that is why we use rax23:24
* clarkb hides23:24
mordredI was just telling wenlock some things about that earlier23:24
fungiexcept it's rax that's blacklisting our servers for someone else's dns queries, so whatevs23:24
jeblairfungi: ~root/certs/gencert.sh; will create csrs with standard info23:24
mordredoh right23:24
fungijeblair: thanks! added to the notes23:25
clarkbfungi: right but hpcloud doesn't even give us consistent servers of dnssec23:25
clarkbfungi: they say "use this it doesn't work and I don't know where it came from or what it will be tomorrow. have fun"23:25
fungijeblair: have you been using a fresh csr for each renewal, or an original csr on file with the vendor?23:25
jeblairfungi: and i use namecheap comodo domain validated certs cause they're $1023:25
jeblairfungi: fresh23:25
wenlocki bet that is the problem with ec2 metadata props too23:25
jeblairsince the command makes everything anyway23:25
fungijeblair: wfm. thanks again!23:26
clarkbwenlock: possibly23:26
wenlockthose http request are being blocked23:26
wenlockfrom some of hp's range23:26
clarkbwenlock: but likely not. since two different clouds23:26
openstackgerritA change was merged to openstack-infra/config: Add proposed/* branches support to merge_tags.sh  https://review.openstack.org/10281223:26
wenlockclarkb not really23:26
jeblairfungi: and then i use "hostmaster@openstack.org" as the approver email; password for the account is in the usual place23:26
wenlockclarkb it's 15.x.x stuff23:26
fungijeblair: yep. perfect23:26
clarkbwenlock: metadata happens over the 169 network iirc23:26
wenlockand it leaks out of the cloud23:26
dstufftfungi: maybe you should stop spinning up servers with lame IP addresses then!23:26
clarkbwenlock: so it shouldn't leak and should be entirely local23:27
mordredfungi, jeblair, clarkb: SO ... I had disabled puppet over the weekend because I was going to apply the ansible chance23:27
clarkbmordred: yes?23:27
fungidstufft: well, we spin up so many servers, statistically many of them will get lame ip addresses. it's the way of the universe23:27
clarkbI think I +2'd the stack should be good to go right?23:27
mordredand I jst remembered - so I went and undisabled it and ran a puppet agent --test ...23:27
mordredand it applied a change that should have been applied back in april23:27
clarkbwoo23:28
dstufftsoftware.23:28
fungii trust some of this software more than my own wetware23:28
dstufftthe only way to really be happy with computer is to keep lowering your expectations until it matches what the computer decides to give you23:28
jeblairmordred: where?23:29
openstackgerritA change was merged to openstack-infra/config: remove non-existing job  https://review.openstack.org/10341223:30
openstackgerritA change was merged to openstack/requirements: Exclude Routes 2.0  https://review.openstack.org/10399123:30
fungiianw: ^23:31
mordredjeblair: on the puppetmaster23:32
openstackgerritA change was merged to openstack-infra/config: Stop running pypi-mirror for old mirror creation  https://review.openstack.org/10336023:32
mordredjeblair: so, I mean, it was a TINY amount of changes23:32
*** devkulkarni has joined #openstack-infra23:32
mordredbut still23:33
openstackgerritA change was merged to openstack-infra/config: Add python-congressclient as new project to stackforge  https://review.openstack.org/10269723:34
fungidhellmann: https://review.openstack.org/10402223:38
*** sarob has quit IRC23:38
clarkbyum is so slow23:39
clarkbmtreinish: we should pacman everyting23:39
ianwfungi: that will auto-merge, right?23:39
fungiianw: huhwhat?23:39
*** lcheng has quit IRC23:40
fungiianw: requirements proposal changes like 104022 still get reviewed and approved like any normal change23:40
fungiianw: though you can see if it's working by watching https://jenkins07.openstack.org/job/gate-oslo-incubator-python33/438/console23:40
fungifwiw, it's been running long enough that i'm pretty sure it's not going to bail on the Routes wheel problem (it'23:41
fungis past the installdeps stage now)23:41
*** MIDENN_ has joined #openstack-infra23:41
ianwfungi: sorry, yeah i meant "will have a change automatically created to fix the issue" i.e. don't do anything :)  https://review.openstack.org/#/c/104022/ i didn't realise it was basically instant23:42
fungiand it succeeded23:42
fungiianw: yeah, https://jenkins.openstack.org/job/propose-requirements-updates/90/console did that23:43
fungiianw: it runs as a post pipeline job on openstack/requirements commit merges23:43
*** mdenny has quit IRC23:44
ianwfungi: cool.  i've fulfilled my learn something about -infra quota for today and it's only 9:30 :)23:45
clarkb9:30 ianw where d oyou sit?23:45
clarkbaustralia?23:45
*** nelsnelson has quit IRC23:46
ianwclarkb: sydney australia.  i about half stand/half sit, after I read that a standing desk was like the equivalent of running 3 marathons a year or something23:46
* fungi stands all day every day, and also walks about 10km every morning before work. totally worth it23:48
fungiianw: get a good commercial grade anti-fatigue mat to stand on. it made all the difference for me. i can stand through a 12+ hour workday nonstop without problems23:49
fungithe kind they use in restaurant kitchens. a little expensive, but totally worth it23:50
clarkbthe problem with workfing from home to watch world cup is I miss out on my walks23:50
*** penguinRaider has joined #openstack-infra23:50
*** Sukhdev has joined #openstack-infra23:52
*** devkulkarni has quit IRC23:53
clarkbargle bargle23:54
clarkbimage builds race puppet creating projects in the project list23:55
clarkbmordred: any chance you can look into why python-congressclient isn't a thing yet?23:55
clarkbimages won't work until that is sorted and I am melting23:56
jheskethclarkb: so.. what do we need to do to remove the envinject module from Jenkins?23:56
clarkbjhesketh: we need to put one in shutdown mode (07 is the usual suspect), wait for jobs to finish on it, shut it down, remove the plugin, start it again23:57
*** unicell has quit IRC23:57
clarkbthen use it as a guinea pig to make sure nothing terrible has happeend23:57
clarkband if not we usually do the remaining master two at a time23:57
*** sabari has joined #openstack-infra23:58
jheskethclarkb: at the moment puppet only has the module removed from jenkins-dev... should we try it out there somehow?23:58
clarkbmordred: looking at logs Tue Jul  1 23:15:05 UTC 2014 was the last time puppet ran on review.o.o?23:59
clarkbjhesketh: we could get nodepool running on jenkins dev as well as zuul-dev23:59
clarkbjhesketh: I know zaro was working on that a while back and don';t know where it got left off23:59

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