Tuesday, 2014-04-01

clarkbnodepool graceful shutdown can be slow... going to give it a bit longer before I take a bigger hammer to it00:00
kiallclarkb, no worries - openstack-infra downtime is the OpenStack equivalent of http://xkcd.com/303/ ;)00:01
sdaguein going back through - code I pushed for review earlier today - https://review.openstack.org/#/c/84213/00:02
*** Hefeweiz1n has quit IRC00:02
*** andreaf has quit IRC00:02
sdaguelooks like stable havana is broken on ceilometer vs. alembic00:03
clarkbsdague: wow it really loves us00:03
*** gokrokve has quit IRC00:04
sdagueanyway, I need to call it a night00:06
*** miguelzuniga has quit IRC00:08
*** mfer has joined #openstack-infra00:08
fungiokay, back and reading scrollback00:10
clarkbfungi: tl;dr is /opt/nodepool was updated with broken code (proposed fix at https://review.openstack.org/#/c/84278/) so I have stopped puppet on nodepool, checked out ad7b9a849b1b2c7e6c9a7d28700e3952bfa18324 in/opt/nodepool, installed nodepoolm and applied lifeless' nodepool.yaml fix by hand and am waiting for graceful nodepool restart00:11
*** dstanek has joined #openstack-infra00:14
lifelessclarkb: how long does a graceful shutdown take?00:14
clarkblifeless: it waits for all pending operations to complete00:14
openstackgerritRyan Petrello proposed a change to openstack-infra/config: Report pecan changesets, merges, and +2s to #pecanpy  https://review.openstack.org/8356400:14
*** khyati_ has joined #openstack-infra00:15
clarkblifeless: right now we are waiting on ~23 more nodes to finish building and whatever deletes are running to complete00:15
clarkblifeless: I think we are about 5-10 minutes away at athis point00:15
*** jswarren has quit IRC00:16
asselinhi all, anyone have tips for nodepool to setup the http proxy configuration? via script? or predefine it in the image?00:17
*** mgagne has joined #openstack-infra00:17
asselinfor the provider node00:17
clarkbasselin: simplest thing is probably to have the image build script set it for you00:20
asselinclarkb, yes, I did that and it works until it gets to here, when the env vars seem to get lost in the new bash session. wget https://git.openstack.org/cgit/openstack-infra/config/plain/install_puppet.sh00:21
asselinsudo bash -xe install_puppet.sh00:21
clarkbasselin: how are you setting the env vars?00:21
asselinexport http_proxy="..."00:22
asselinjust a few lines above in the script00:22
asselinand export https_proxy00:22
clarkbasselin: is that in the script that calls install_puppet.sh?00:23
mordredI have dealt with proxy in all parts of infra _except_ in nodepool00:23
clarkbdown to 15 building nodes now00:23
mordredclarkb, asselin: sudo blowing away the environment is going to be the thing killing you there00:23
clarkbI guess my 10 minute guestimate was bad00:23
clarkbmordred: oh right00:23
asselinyes. this is in prepare_node.sh00:23
fungiclarkb: oh, so much fun... 84278 lgtm00:23
mordredyou can fix it by adding -E to the sudo command - but then you're forking prepare_node.sh00:23
clarkbfungi: note, I haven't actually tested 84278 so I am just going back to the old stuff for now00:24
asselinmordred, yes...good point sudo might be clearing it out too00:24
fungiclarkb: "going back to the old stuff" means reverting all of the subnode-related patches, or what"00:24
fungi"00:24
fungi?00:24
clarkbfungi: not reverting, I just checked out the code we were running on friday and stopped puppet00:25
clarkband installed that code00:25
*** ildikov_ has quit IRC00:25
fungiclarkb: that will work with the migrated db and new-style config with labels (or are you not going back past the labels change)?00:25
clarkbI am not going past the labels change00:26
*** gokrokve has joined #openstack-infra00:26
fungiokay, fair enough00:26
clarkbfungi: ad7b9a849b1b2c7e6c9a7d28700e3952bfa18324 is the commit I am using00:26
clarkbdown to 12 building00:26
clarkb1000:27
*** matsuhashi has joined #openstack-infra00:27
clarkbI expect it to be happy nyan cats after the graceful restart00:27
fungihaving nodepool abort and delete building nodes on sigusr1 is probably in order to speed this up next00:28
clarkb++00:29
clarkbsince they will all be deleted on start anyways00:29
lifelesswhy does nodepool delete everything on startup ?00:30
*** qs201 has joined #openstack-infra00:31
*** bknudson has joined #openstack-infra00:31
clarkblifeless: it has to do with jenkins not cleaning them up otherwise00:31
clarkbiirc the issue is nodepool can miss jenkins events while it is off and thus must assume nodes hae been used00:31
*** mfer has quit IRC00:32
clarkbfungi: is there a better way to do the restart?00:32
lifelessclarkb: hmm00:32
*** mfer has joined #openstack-infra00:33
lifelessif only there was some way to find out whether a job has been run on a node00:33
fungiclarkb: i've ended up killing nodepoold via normal initscript stop (not graceful stop) and then deleting any building state nodes when it comes back up, and checking back to delete stale nodes a few hours later00:33
clarkbdown to 5 building nodes now00:34
clarkbmaybe we can let it do the restart on its own00:34
clarkbnow 300:36
clarkb200:37
clarkbits the final countdown!00:37
lifelessnah-na-naaah-nah00:37
clarkb1!00:39
*** JayF has quit IRC00:40
*** JayF has joined #openstack-infra00:41
clarkbhrm building is down to 0, must be waiting on something else now00:41
clarkba loop through the deletes?00:41
clarkbbut at this point I Think it is relatively safe to kill with more fire00:41
clarkbfungi: ^ thoughts?00:41
*** mfer has quit IRC00:43
lifelessclarkb: since its going to delete it all anyhow...00:43
lifelessclarkb: there doesn't seem any benefit in waiting00:44
openstackgerritRyan Petrello proposed a change to openstack-infra/config: Add the correct pecan homepage for Github mirroring.  https://review.openstack.org/8430200:44
clarkbright, though I really expected it to stop at this point00:44
clarkbsince it should have only been waiting on the nodes to launch to finish launching00:44
*** markmcclain has quit IRC00:45
*** dstanek has quit IRC00:45
fungiclarkb: moar fire00:46
clarkbthread dump indicates it is waiting on those deletions though00:46
clarkbfungi: ok proceeding with more fire00:47
fungisome of those deletes may never complete, i think00:47
fungithough not sure if it merely waits for them to time out00:47
clarkbok it is doing stuff now00:48
*** changbl has joined #openstack-infra00:48
clarkbbuilding 778 nodes...00:49
*** amcrn has quit IRC00:49
*** CaptTofu has joined #openstack-infra00:49
mordredwow00:52
mordredthat's a lot of nodes00:52
clarkbwoot we have new ready nodes finally00:53
clarkband a bunch of nodes transitioning to used00:53
*** markmcclain has joined #openstack-infra00:53
*** gothicmindfood has left #openstack-infra00:54
*** CaptTofu has quit IRC00:54
*** UtahDave has quit IRC00:54
clarkbmordred: https://review.openstack.org/#/c/84278/ should fix this for us, but I haven't been able to test it00:54
clarkbwas too busy making nodepool go again00:54
mordredclarkb: looking00:55
mordredclarkb: that patch looks completely reasonable00:55
*** amcrn has joined #openstack-infra00:55
clarkbnothing like finding a bunch of ENOENT errors after you restart nodepool00:56
*** zhiyan_ is now known as zhiyan00:57
mordredclarkb: +2 - I defer to you on thoughts of landing00:57
clarkbtripleo images are building00:58
clarkbapparently those got deleted?00:58
openstackgerritA change was merged to openstack-infra/config: Fixup tripleo nodepool config.  https://review.openstack.org/8426300:58
*** Sukhdev has quit IRC00:58
clarkbI am going to take this as an opportunity to walk home00:59
* clarkb &00:59
*** zigo has quit IRC00:59
*** jepoy has joined #openstack-infra00:59
*** zigo has joined #openstack-infra01:01
lifelessyay building images01:03
*** alff_ has quit IRC01:06
*** alff has quit IRC01:06
*** mrodden has joined #openstack-infra01:13
asselinmordred, clarkb thanks for the tips for the sudo proxy issues. I got past them. Next is pip proxy...will do that tomorrow.01:15
*** ameade has joined #openstack-infra01:17
*** CaptTofu has joined #openstack-infra01:17
*** dstanek has joined #openstack-infra01:17
*** khyati_ has quit IRC01:17
*** thomasem has quit IRC01:20
*** ameade has quit IRC01:21
*** SumitNaiksatam has joined #openstack-infra01:23
*** nosnos has joined #openstack-infra01:26
*** yaguang has joined #openstack-infra01:32
*** mbacchi has quit IRC01:34
tchaypo clarkb: why weren't you at home already?01:39
clarkbI am one of those weirdos that sits in an office01:39
tchaypoIf it's walking distance from home, that's probably not terrible01:40
tchaypoyou get a comfy chair and the mindset-altering benefit of a dedicated "work" space, plus some exercise01:40
clarkbthe exercise is quite important01:41
persia(short) commutes are good for thinking.  If you aren't lucky enough to have one, faking it by taking a 20 minute walk a couple times a day tends to increase productivity.01:41
tchaypoI've been breaking my day into two stints - the morning at home for review/triage/updating existing changesets, then I head into the city for lunch and do the new work in the afternoon from a library01:41
tchaypoI find the break does me good, and the different environments and times-of-day work well for the different kinds of work01:42
persiaRequires a nice library, but that sounds like a lovely model.01:43
clarkbas the weathwr gets nicer I intend on doing waterfront hacking01:44
tchaypoI've got a passel of libraries within range of me - two different universities, the city council has a few nice branches (which even have free wifi), and the state library (although I haven't been there yet)01:45
tchaypoIf they get stale there's a few cow-orking spaces I'm looking at as well01:46
clarkbcan sit in the park by mohai or on one of the many piers and tether01:46
fungionce i settle on a place with a dock, i plan to do *plenty* of bayside hacking01:47
*** kiall has quit IRC01:47
fungiwell, technically it's a sound, not a bay, but close enough01:47
fungicalm enough for a dock regardless01:48
persiaI used to go to a park by the river, but I found it hard to find a place that provided a good upright seat, a nice view, and didn't flood my laptop with sunlight.  You may need to arrange strategically placed trees or a large umbrella, or something.01:49
fungieh, i'll erect a sunshade or or something by the boat lift01:50
*** CaptTofu has quit IRC01:51
*** _Alexandra_ has joined #openstack-infra01:54
*** jepoy has quit IRC01:54
lifelesspersia: o/01:56
*** CaptTofu has joined #openstack-infra01:58
persialifeless: Good afternoon.01:59
*** wenlock has joined #openstack-infra02:01
*** dstanek has quit IRC02:01
*** zehicle_at_dell has quit IRC02:03
*** adalbas has quit IRC02:03
*** Ryan_Lane has quit IRC02:04
*** zhiyan has left #openstack-infra02:04
*** zhiyan has joined #openstack-infra02:05
tchaypolifeless merely hacks from his eyrie on a clifftop overlooking the peons below02:05
tchaypoor so I've heard02:05
clarkbI dunno skype made it look like a closet last we used video02:06
*** weshay has quit IRC02:06
*** maxbit has joined #openstack-infra02:07
*** jepoy has joined #openstack-infra02:10
*** zehicle has joined #openstack-infra02:15
clarkbjogo: large ops is broken02:18
clarkbconnecting to blah blah blah via ssh timed out02:19
clarkbwhich is funny because those aren't real nodes02:19
clarkbdid tempest somehow make a chagne to what the test lists are?02:19
*** dims has quit IRC02:23
*** dims has joined #openstack-infra02:24
clarkbjogo: looks like it is running smoke tests against the largeops job02:24
clarkbguessing d-g d61f684619763a42a849bb907934929a2cd711f6 is at faulkt02:25
clarkbhuh maybe not that change passed large ops in the gate02:27
*** dstanek has joined #openstack-infra02:27
clarkbunless that is somehow not self gating02:28
clarkbhttp://logs.openstack.org/43/80643/5/gate/gate-tempest-dsvm-large-ops/ff376c2/logs/localrc.txt.gz sets it to 50 so I don't think that is the problem02:29
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Add in train tracks to status page  https://review.openstack.org/8431402:30
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Ensure bootstrap's fonts are installed  https://review.openstack.org/8431502:30
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Tweak job layout css in status page  https://review.openstack.org/8431602:30
openstackgerritJoshua Hesketh proposed a change to openstack-infra/zuul: Adjust change headings on status page  https://review.openstack.org/8431702:30
*** amotoki_ has quit IRC02:30
clarkbI see the problem, patch incoming02:31
lifelesstchaypo: if by peons you mean my cats02:34
openstackgerritClark Boylan proposed a change to openstack-infra/devstack-gate: Enable large ops if flag is >= 1.  https://review.openstack.org/8431902:34
clarkbjogo: fungi jeblair mordred SergeyLukjanov ^02:34
clarkbI will probably self approve that once the largeops test runs and confirms that fixes the problem02:34
StevenKclarkb: "Maybe the comparison flag -ge 1 to catch non default cases." ? :-)02:35
clarkbugh did I tyop it?02:36
StevenKclarkb: Yeah, in the commit message02:36
StevenKBut y and b are no where near k :-P02:36
openstackgerritClark Boylan proposed a change to openstack-infra/devstack-gate: Enable large ops if flag is >= 1.  https://review.openstack.org/8431902:37
clarkbStevenK: there ^02:37
clarkbStevenK: I have a faulty typing driver02:37
clarkbit caches movements and sometimes the cache eviction algorithm fails and it spits out random crap02:37
*** timrc is now known as timrc-afk02:38
lifelessclarkb: / fungi: is nodepool happy w/tripleo cloud ?02:41
*** dstanek has quit IRC02:42
lifelessI see some error VMs being deleted and 3 new ones spawning02:42
*** amcrn has quit IRC02:42
*** david-lyle has joined #openstack-infra02:44
clarkblifeless: I don't currently have access to those logs, looks like 3 nodes started tests in the last ~5 minutes though02:45
openstackgerritTom Fifield proposed a change to openstack-infra/config: Add --force to transifex pull  https://review.openstack.org/8432302:56
clarkbI have self approved the d-g change that fixes large ops, it just passed its own large ops test run02:57
*** markmcclain has quit IRC02:59
*** harlowja is now known as harlowja_away02:59
openstackgerritJoshua Hesketh proposed a change to openstack-infra/config: Add a slave-script to Jenkins for pushing to swift  https://review.openstack.org/7118903:01
*** lcheng has joined #openstack-infra03:06
nibalizeri just changed my launchpad username from krum-spencer to nibalizer, will I have any trouble with openstack services?03:08
clarkbnibalizer: you might, depends on whether or not launchpad gievs you a new openid endpoint03:09
clarkbI never remember what circumstances that happens under03:09
nibalizeri was able to review03:09
*** harlowja_away is now known as harlowja03:09
nibalizerso hopefully we're good03:09
clarkbnibalizer: the CLA stuff is usually what breaks03:09
*** khyati_ has joined #openstack-infra03:10
*** chandan_kumar has joined #openstack-infra03:11
nibalizerhrm i'll try and push a change03:12
nibalizerya thats what i'm worried about03:12
*** yamahata has joined #openstack-infra03:13
clarkbdstufft: are you around? http://logs.openstack.org/78/84278/1/check/gate-nodepool-docs/3acec62/console.html has me confused, I can reproduce that using tox, but not pip 1.5 directly03:14
clarkbdstufft: basically test-requirements says sphinx>=1.1.2,<1.2 and 1.2b3 is being installed which breaks us03:14
clarkbdstufft: with my manual testing I have installed every dep individually looking for the ones that install sphinx and it doesn't appear to install sphinx until it is explicitly called out in test-requirements.txt so I don't think this is transitive weirdness03:15
clarkbdstufft: any idea why Sphinx==1.2b3 is being installed?03:15
*** timrc-afk is now known as timrc03:16
tchaypolifeless: no, cats would be overlords03:17
lifelessclarkb: 1.2b3 is < 1.203:18
clarkblifeless: yes, but pip shouldn't install beta releases without explicitly being told to03:18
clarkboh I know03:18
clarkbtox tells pip to do that03:18
lifelessclarkb: as long as its properly detecting they are betas, sure.03:18
clarkbbecause tox03:18
lifelessclarkb: or that ;)03:19
*** matsuhashi has quit IRC03:22
openstackgerritClark Boylan proposed a change to openstack-infra/nodepool: Fix tox's insane pip install command.  https://review.openstack.org/8432703:23
*** timrc is now known as timrc-afk03:28
*** mgagne has quit IRC03:32
*** Ryan_Lane has joined #openstack-infra03:33
*** chandan_kumar has quit IRC03:35
*** nosnos has quit IRC03:37
*** alff_ has joined #openstack-infra03:38
*** alff has joined #openstack-infra03:38
*** alexpilotti has joined #openstack-infra03:40
*** CaptTofu has quit IRC03:41
*** markwash has quit IRC03:47
*** markwash has joined #openstack-infra03:50
*** alexpilotti has quit IRC03:52
*** _Alexandra_ has quit IRC03:56
*** chandan_kumar has joined #openstack-infra03:59
*** chandan_kumar has quit IRC04:01
*** lcheng has quit IRC04:03
*** chandan_kumar has joined #openstack-infra04:06
*** _Alexandra_ has joined #openstack-infra04:07
clarkblifeless: so I just realized, when I applied your change to nodepool.yaml I did it by hand and may not have bumped the max servers which would explain the behavior we are seeing04:09
clarkblifeless: I can fix that tomorrow04:09
*** _Alexandra_ has quit IRC04:12
*** thuc has joined #openstack-infra04:13
*** thuc_ has joined #openstack-infra04:13
*** matsuhashi has joined #openstack-infra04:14
*** thuc has quit IRC04:17
*** thuc has joined #openstack-infra04:21
*** nosnos has joined #openstack-infra04:23
*** thuc_ has quit IRC04:24
*** dstanek has joined #openstack-infra04:28
*** dstanek has quit IRC04:33
*** ArxCruz has quit IRC04:33
*** ArxCruz has joined #openstack-infra04:35
*** sandywalsh_ has quit IRC04:44
*** sandywalsh has joined #openstack-infra04:46
*** gokrokve_ has joined #openstack-infra04:50
*** alff has quit IRC04:52
*** alff_ has quit IRC04:52
*** gokrokve has quit IRC04:53
*** mihgen has joined #openstack-infra04:55
*** zul has quit IRC05:00
lifelessclarkb: ah yes that would explain it05:00
*** cody-somerville has quit IRC05:01
*** matsuhas_ has joined #openstack-infra05:02
*** matsuhas_ has quit IRC05:02
*** cody-somerville has joined #openstack-infra05:02
*** mrmartin has joined #openstack-infra05:02
*** zul has joined #openstack-infra05:04
*** matsuhashi has quit IRC05:05
*** matsuhashi has joined #openstack-infra05:05
*** harlowja is now known as harlowja_away05:11
*** thuc has quit IRC05:13
*** thuc has joined #openstack-infra05:14
*** thuc has quit IRC05:18
*** ildikov_ has joined #openstack-infra05:20
*** gyee has quit IRC05:20
*** qs201 has quit IRC05:21
*** khyati_ has quit IRC05:22
*** mrmartin has quit IRC05:27
*** CaptTofu has joined #openstack-infra05:30
*** CaptTofu has quit IRC05:35
*** zhiyan is now known as zhiyan_05:42
*** peddamat has joined #openstack-infra05:46
openstackgerritSpencer Krum proposed a change to openstack-infra/config: Nothing to see here. Move along  https://review.openstack.org/8435605:47
*** _nadya_ has joined #openstack-infra05:48
openstackgerritClark Boylan proposed a change to openstack-infra/devstack-gate: Enable large ops if flag is >= 1  https://review.openstack.org/8431905:54
clarkbfor anyone that wakes up before me in the morning, ^ will fix the devstack-gate trouble. I pushed a new patchset and re self approved because it failed in the gate and this was the quickest way to turn it around and get it back into the gate without simply merging it05:55
*** pcrews_ has quit IRC06:00
*** Ryan_Lane1 has joined #openstack-infra06:04
*** _nadya_ has quit IRC06:07
*** basha has joined #openstack-infra06:08
*** saju_m has joined #openstack-infra06:21
*** e0ne has joined #openstack-infra06:23
*** thuc has joined #openstack-infra06:24
*** dstanek has joined #openstack-infra06:27
*** _nadya_ has joined #openstack-infra06:27
*** thuc has quit IRC06:29
*** imcsk8_lap has quit IRC06:30
*** imcsk8_lap has joined #openstack-infra06:30
*** _nadya_ has quit IRC06:31
*** dstanek has quit IRC06:31
*** _nadya_ has joined #openstack-infra06:33
openstackgerritNikhil Manchanda proposed a change to openstack-infra/config: Remove experimental trove devstack-gate job  https://review.openstack.org/8436906:36
*** e0ne has quit IRC06:37
*** AJaeger has quit IRC06:38
*** nosnos has quit IRC06:45
*** nosnos has joined #openstack-infra06:46
*** mihgen has quit IRC06:48
*** flaper87|afk is now known as flaper8706:50
*** mkoderer has joined #openstack-infra06:51
*** jhesketh has quit IRC06:55
*** matsuhashi has quit IRC06:58
*** matsuhashi has joined #openstack-infra06:58
*** _nadya_ has joined #openstack-infra07:01
*** matsuhashi has quit IRC07:03
*** _nadya_ has quit IRC07:03
*** Ryan_Lane1 has quit IRC07:04
*** yolanda has joined #openstack-infra07:05
*** mrmartin has joined #openstack-infra07:05
*** matsuhashi has joined #openstack-infra07:06
*** Ryan_Lane has quit IRC07:07
*** jhesketh has joined #openstack-infra07:08
openstackgerritAndreas Jaeger proposed a change to openstack-infra/config: Add new job to build misc Japanese Guides  https://review.openstack.org/8397907:09
openstackgerritJamie Lennox proposed a change to openstack-infra/config: Remove py3 gate job from kite  https://review.openstack.org/8437207:13
*** jcoufal has joined #openstack-infra07:16
*** CaptTofu has joined #openstack-infra07:19
*** sandywalsh_ has joined #openstack-infra07:22
*** CaptTofu has quit IRC07:23
*** sandywalsh has quit IRC07:25
*** jamielennox is now known as jamielennox|away07:27
*** mrda is now known as mrda_away07:29
*** basha has quit IRC07:29
*** salv-orlando has quit IRC07:30
*** salv-orlando has joined #openstack-infra07:30
*** alff_ has joined #openstack-infra07:32
*** alff has joined #openstack-infra07:32
*** mihgen has joined #openstack-infra07:35
*** marun is now known as maru_afk07:36
*** jgallard has joined #openstack-infra07:37
*** gokrokve_ has quit IRC07:40
*** gokrokve has joined #openstack-infra07:41
*** hashar has joined #openstack-infra07:44
*** gokrokve has quit IRC07:45
*** basha has joined #openstack-infra07:47
*** jpich has joined #openstack-infra08:02
*** matsuhashi has quit IRC08:03
openstackgerritA change was merged to openstack-infra/devstack-gate: Enable large ops if flag is >= 1  https://review.openstack.org/8431908:04
*** matsuhas_ has joined #openstack-infra08:05
*** dizquierdo has joined #openstack-infra08:14
*** liam__ has joined #openstack-infra08:18
*** liam__ has quit IRC08:19
*** CaptTofu has joined #openstack-infra08:20
*** derekh has joined #openstack-infra08:24
*** CaptTofu has quit IRC08:24
*** matsuhas_ has quit IRC08:33
*** jlibosva has joined #openstack-infra08:35
*** andreaf has joined #openstack-infra08:37
*** zhiyan_ is now known as zhiyan08:38
*** matsuhashi has joined #openstack-infra08:38
*** alff_ has quit IRC08:40
*** alff has quit IRC08:40
*** saju_m has quit IRC08:43
*** peddamat has quit IRC08:45
*** jp_at_hp has joined #openstack-infra08:53
*** jpich has quit IRC08:55
*** saju_m has joined #openstack-infra08:56
*** afazekas has joined #openstack-infra08:57
*** yonglihe_ has quit IRC09:00
openstackgerritBob Ball proposed a change to openstack-infra/devstack-gate: Only install libvirt if the driver is libvirt(!)  https://review.openstack.org/8438109:01
*** jpich has joined #openstack-infra09:07
*** jooools has joined #openstack-infra09:10
openstackgerritBob Ball proposed a change to openstack-infra/devstack-gate: Only install libvirt if the driver is libvirt(!)  https://review.openstack.org/8438109:14
*** gokrokve has joined #openstack-infra09:16
*** jcoufal has quit IRC09:18
*** jooools has quit IRC09:20
*** gokrokve has quit IRC09:21
*** jooools has joined #openstack-infra09:22
*** chandankumar_ has joined #openstack-infra09:25
*** dstanek has joined #openstack-infra09:30
*** dstanek has quit IRC09:34
*** dizquierdo is now known as dizquierdo_afk09:42
*** morganfainberg is now known as morganfainberg_Z09:46
*** gokrokve has joined #openstack-infra09:47
*** jcoufal has joined #openstack-infra09:50
*** gokrokve has quit IRC09:52
*** chandankumar_ has quit IRC09:54
*** david-lyle has quit IRC09:55
*** jepoy has quit IRC09:56
*** jepoy has joined #openstack-infra09:56
openstackgerritSergey Skripnick proposed a change to openstack-infra/config: Enable sudo for rally-install job  https://review.openstack.org/8439409:58
*** ihrachys is now known as ihrachys|lunch09:59
*** jepoy has quit IRC10:01
*** e0ne has joined #openstack-infra10:03
*** CaptTofu has joined #openstack-infra10:09
*** mrmartin has quit IRC10:11
*** e0ne has quit IRC10:13
*** wenlock has quit IRC10:13
*** CaptTofu has quit IRC10:13
*** matsuhashi has quit IRC10:14
*** jgallard has quit IRC10:20
*** yamahata has quit IRC10:24
*** e0ne has joined #openstack-infra10:24
*** e0ne_ has joined #openstack-infra10:30
*** DuncanT- has quit IRC10:31
*** e0ne has quit IRC10:33
*** hashar has quit IRC10:33
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: normalize the services for a post grizzly world  https://review.openstack.org/8428710:35
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: turn back on log enforcement  https://review.openstack.org/8121610:35
*** nosnos has quit IRC10:40
*** yaguang has quit IRC10:43
*** markmc has joined #openstack-infra10:45
*** CaptTofu has joined #openstack-infra10:46
*** pblaho has joined #openstack-infra10:48
*** gokrokve has joined #openstack-infra10:48
*** dizquierdo_afk is now known as dizquierdo10:49
*** mrmartin_ has joined #openstack-infra10:52
*** gokrokve has quit IRC10:53
*** CaptTofu has quit IRC10:54
*** yaguang has joined #openstack-infra10:55
*** roeyc has joined #openstack-infra11:06
amotokihi, i noticed OpenStack Style Guidelines still has vim modelines http://docs.openstack.org/developer/hacking/11:08
amotokithough it is removed in  the latest hacking repo but it is not released yet.11:08
amotokiit seems it is better to be updated.11:08
*** mbacchi has joined #openstack-infra11:10
*** bookwar has joined #openstack-infra11:16
*** sandywalsh_ is now known as sandywalsh11:18
*** e0ne_ has quit IRC11:22
*** ihrachys|lunch is now known as ihrachys11:24
*** jgallard has joined #openstack-infra11:24
*** hashar has joined #openstack-infra11:28
*** maxbit has quit IRC11:30
*** ominakov has joined #openstack-infra11:30
*** ociuhandu has joined #openstack-infra11:33
*** alexpilotti has joined #openstack-infra11:34
*** kiall_ is now known as Kiall11:36
*** ams0 has joined #openstack-infra11:40
*** moted has quit IRC11:40
*** weshay has joined #openstack-infra11:41
*** ameade has joined #openstack-infra11:42
*** yolanda has quit IRC11:42
*** moted has joined #openstack-infra11:44
*** yaguang has quit IRC11:44
*** saju_m has quit IRC11:44
*** maxbit has joined #openstack-infra11:44
*** ams0 has quit IRC11:46
openstackgerritA change was merged to openstack/requirements: Using happybase >=0.5,!=0.7  https://review.openstack.org/8243811:47
*** gokrokve has joined #openstack-infra11:47
*** gokrokve has quit IRC11:52
*** jgallard has quit IRC11:52
*** jgallard has joined #openstack-infra11:52
*** yolanda has joined #openstack-infra11:54
*** saju_m has joined #openstack-infra11:56
*** mflobo has joined #openstack-infra11:57
*** maxbit has quit IRC12:00
*** pdmars has joined #openstack-infra12:01
*** saju_m has quit IRC12:02
*** e0ne has joined #openstack-infra12:02
*** peddamat has joined #openstack-infra12:02
*** peddamat has quit IRC12:02
*** rfolco has joined #openstack-infra12:03
*** rfolco has quit IRC12:03
*** e0ne has quit IRC12:04
*** e0ne has joined #openstack-infra12:04
*** afazekas has quit IRC12:04
*** rfolco has joined #openstack-infra12:04
*** che-arne has joined #openstack-infra12:08
*** CaptTofu has joined #openstack-infra12:08
*** dkranz has joined #openstack-infra12:09
*** e0ne has quit IRC12:09
*** amotoki has quit IRC12:11
*** CaptTofu has quit IRC12:16
*** jepoy has joined #openstack-infra12:16
*** jhesketh has quit IRC12:16
*** eharney has joined #openstack-infra12:16
*** dcramer_ has quit IRC12:17
*** afazekas has joined #openstack-infra12:18
*** e0ne has joined #openstack-infra12:19
*** saju_m has joined #openstack-infra12:21
*** dims has quit IRC12:22
*** saju_m has quit IRC12:23
*** thomasem has joined #openstack-infra12:26
*** thomasem_ has joined #openstack-infra12:27
*** markmcclain has joined #openstack-infra12:28
*** maxbit has joined #openstack-infra12:29
*** aysyd has joined #openstack-infra12:30
*** thomasem has quit IRC12:31
*** dstanek has joined #openstack-infra12:32
fungiand so it begins12:33
*** basha has quit IRC12:33
fifieldtduh duh duuuuuuuhhh12:34
*** dims_ has joined #openstack-infra12:34
*** rlandy has joined #openstack-infra12:34
*** dstanek has quit IRC12:36
*** CaptTofu has joined #openstack-infra12:37
*** timrc-afk is now known as timrc12:38
*** saju_m has joined #openstack-infra12:39
*** jgrimm has joined #openstack-infra12:43
*** signed8bit has joined #openstack-infra12:47
*** gokrokve has joined #openstack-infra12:47
*** signed8bit has quit IRC12:48
*** thuc has joined #openstack-infra12:50
*** dkliban has quit IRC12:51
*** gokrokve has quit IRC12:52
*** thuc_ has joined #openstack-infra12:52
*** thuc has quit IRC12:55
*** dprince has joined #openstack-infra12:57
*** heyongli has joined #openstack-infra13:01
*** signed8bit has joined #openstack-infra13:03
*** mrmartin_ has quit IRC13:03
*** zehicle has quit IRC13:07
*** dstanek has joined #openstack-infra13:07
*** pblaho has quit IRC13:08
*** dkranz has quit IRC13:10
*** jeckersb_gone is now known as jeckersb13:11
*** afazekas has quit IRC13:13
*** saju_m has quit IRC13:15
*** mriedem has joined #openstack-infra13:15
*** mflobo has quit IRC13:16
*** dcramer_ has joined #openstack-infra13:18
*** signed8bit has quit IRC13:18
*** eharney has quit IRC13:21
*** CaptTofu has quit IRC13:23
*** dkranz has joined #openstack-infra13:24
*** saju_m has joined #openstack-infra13:28
*** esker has joined #openstack-infra13:31
*** dkliban has joined #openstack-infra13:31
*** signed8bit has joined #openstack-infra13:34
*** zns has joined #openstack-infra13:34
*** jgrimm has quit IRC13:34
*** changbl has quit IRC13:35
*** julim has joined #openstack-infra13:38
*** mwagner_zzz has quit IRC13:40
*** nkinder has quit IRC13:40
*** thuc_ has quit IRC13:40
*** thuc has joined #openstack-infra13:41
*** prad has joined #openstack-infra13:41
*** thuc has quit IRC13:45
*** markmcclain has quit IRC13:45
*** thuc has joined #openstack-infra13:45
*** vhoward- has left #openstack-infra13:46
*** signed8bit has quit IRC13:46
*** afazekas has joined #openstack-infra13:46
*** vhoward- has joined #openstack-infra13:46
*** signed8bit has joined #openstack-infra13:47
*** gokrokve has joined #openstack-infra13:47
*** lcostantino has joined #openstack-infra13:49
*** gokrokve_ has joined #openstack-infra13:49
*** pcrews_ has joined #openstack-infra13:49
*** ams0 has joined #openstack-infra13:50
*** gokrokve has quit IRC13:52
*** gokrokve_ has quit IRC13:54
*** ams0 has quit IRC13:56
*** afazekas has quit IRC13:57
*** zns has quit IRC14:01
*** heyongli has quit IRC14:05
sdaguefungi: is the ES cluster ok?14:07
sdagueI'm getting more proxy timeouts than I expect14:07
fungisdague: i'll check14:07
*** yassine has joined #openstack-infra14:09
*** zns has joined #openstack-infra14:11
fungisdague: i don't see any obvious signs of trouble... cluster health is green, no missing members, no missing workers, no shards being shuffled or anything14:11
sdaguehmmm14:11
sdagueok14:11
fungiwe're not backed up on tasks in the queue14:11
sdagueyep14:11
sdaguewas there a kibana update recently?14:12
Kiallfungi: could I trouble you for a quick review on a gerrit ACL change? https://review.openstack.org/#/c/84273/ :)14:12
sdaguethe moment I try to search the 24hr bucket, I basically time out14:13
sdagueand the browser gets killed with whatever gets handed back14:13
*** saju_m has quit IRC14:14
fungisdague: ohh... i saw someone mention in here the other day that something might be wrong with multi-line handling and that some log files were getting treated as one line or something like that14:15
fungiyesterday maybe14:15
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for nova ec2 bug 1280532  https://review.openstack.org/8444914:15
uvirtbotLaunchpad bug 1280532 in nova "Detach volume fails with "Unexpected KeyError" in EC2 interface." [High,In progress] https://launchpad.net/bugs/128053214:15
*** rcleere has joined #openstack-infra14:16
fungiperhaps a change in some log format confused the multi-line filter, or could we have recently changed the filter?14:16
sdagueoh, right14:16
*** roeyc has quit IRC14:18
*** alff_ has joined #openstack-infra14:18
*** alff has joined #openstack-infra14:18
*** basha has joined #openstack-infra14:19
fungiKiall: are you actually expecting to have milestone-proposed branches in python-designateclient? none of the official openstack python-fooclient projects do that as far as i know14:19
*** jgrimm has joined #openstack-infra14:20
fungiKiall: and milestone-proposed makes very little sense on a project unless you're going to use it as a staging area for future stable branches (which aren't reflected in the python-designateclient acl)14:20
*** markmcclain has joined #openstack-infra14:21
*** adalbas has joined #openstack-infra14:21
Kiallfungi: your're probably right about the client, but that side was just keeping things synced up14:21
Kiall(e.g. All-Projects does that for everything, and just overriding the Release Managers group here for designate-milesone)14:22
*** eharney has joined #openstack-infra14:23
fungiKiall: well, all-projects does a lot of things which only apply if you have a particular branch on your project. you chose not to do a stable/* ref section (which is also something all-projects has) so why milestone-proposed?14:23
*** basha has quit IRC14:23
KiallHeh - I missed that obviously -_- Either way, I think your right about just removing that section.14:24
fungiKiall: if you want to respin it without that section now, i'll go ahead and approve it for you14:24
sdaguefungi: so, how do you feel about a ninja merge of this then? - https://review.openstack.org/#/c/84213/14:24
Kiallalready on it, thanks :)14:24
sdaguebecause that's the fix for the kibana issue14:25
sdaguehowever it's deadlocked on ceilometer breaking stable14:25
fungiKiall: it makes perfect sense on designate since that's a server project so you presumably expect to follow a release workflow similar to the other integrated servers, just not really relevant on python-designateclient14:25
fungisdague: looking, but probably yes14:25
fungisdague: on a related note, do you have any details on what ceilo needs to be doing to fix their migrations in havana?14:26
openstackgerritKiall Mac Innes proposed a change to openstack-infra/config: Sync up designate repo ACLs  https://review.openstack.org/8427314:27
*** shardy is now known as shardy_afk14:28
sdaguefungi: so actually jd__ has a fix in the gate now14:28
anteayaKiall: how many people in the designate-milestone group?14:28
KiallOkay .. that should be the minimal set of items in the 2x designate ACLs without needlessly overriding anything we're already inheriting from All-Projects14:29
sdaguealembic now requires code in sqla >= 914:29
sdagueso it's a requirements fix14:29
anteayajust wondering why designate needs more that -ptl and -core14:29
sdaguewhich actually means we should be able to fix this the real way14:29
Kiallanteaya: right now it includes all the designate-core people, but may change over time14:29
*** skraynev is now known as skraynev_afk14:30
fungianteaya: looks like if you merge it with the -core group (which is listed as an included group) and then de-dupe the results it's four humans14:30
fungiso not a ton14:30
*** nkinder has joined #openstack-infra14:30
KiallAlso - the group is pre-exiting14:30
fungiKiall: but anteaya does have a point, that you may want to consider the -milestone group roughly equivalent to the ptl and list of delegates performing release management duties, and drop the -core inclusion on it14:30
KiallYea, it just so happens that today those are the same group of people14:31
fungithough that's something you can do entirely through gerrit's group management interface14:31
fungiKiall: note that changes to acls aren't going to take effect until this friday when we run manage-projects manually14:32
Kiallfungi: ah.. it's disabled at the moment?14:32
anteayaI'm comfy with an individual doing tagging and other release management things, not really a group14:33
anteayaan individual being the -ptl14:33
fungiKiall: yes, mordred and others are working on fixing some remaining race conditions on new project creation, so we don't run manage-projects automatically at the moment14:33
KiallIn the meantime, could I ask you to cut a branch on stackforge/designate using 2445db42520f828cd19255a68538fa094c6cad2b - at least reviews can pile up there until Friday14:33
fungiKiall: gladly14:33
Kiallexcellent :)14:34
fungisdague: https://review.openstack.org/84433 i guess14:34
*** david-lyle has joined #openstack-infra14:35
*** basha has joined #openstack-infra14:35
fungisdague: i can enqueue 84213 into the gate bypassing the check pipeline requirement if it's urgent to get it in before 84433 merges14:35
*** CaptTofu has joined #openstack-infra14:36
fungiwhich should be in ~45 minutes if we're lucky14:36
openstackgerritA change was merged to openstack-infra/config: Sync up designate repo ACLs  https://review.openstack.org/8427314:36
sdaguefungi: no, it can merge after14:36
*** afazekas has joined #openstack-infra14:37
fungiKiall: you want 2445db4 as the base of milestone-proposed or some other branch name?14:37
sdagueactually, given that it was a requirements fix, this should probably work now. Let me redo results14:38
Kiallfungi: Yes, 2445db4 as the base of milestone-proposed is right14:38
fungiKiall: done14:38
KiallThanks :)14:39
fungiKiall: you're welcome, and your patience is appreciated as always!14:39
*** CaptTofu has quit IRC14:40
fungias luck would have it, when there's one zone/region where we're having trouble getting image updates to complete, the job i want to re-run always seems to end up grabbed by a node from that one stale provider14:41
anteayadoes that zone/region have a name?14:41
anteayais it az-2 still?14:41
fungianteaya: it's your old friend hpcloud-az214:41
fungiyes14:41
anteayak14:42
*** thuc_ has joined #openstack-infra14:42
fungithere were some other rax and hpcloud zones where the overnight image updates haven't succeeded for a day or two, but i was able to get images built in those after a few manual retries14:42
anteayago you14:42
anteayabut az-2 still stonewalls14:42
anteayasigh14:42
fungii'm just going to start a loop there retrying to get a new image until it sticks14:43
fungilast time this happened, it had to cycle continuously for about 36 hours to get one to complete14:43
Kiallfungi: any idea why it's failing?14:45
anteayawow14:45
anteayalike testr --until-failure14:45
fungiKiall: timeouts from facter trying to query the metadata server in that az14:45
anteayaonly az-2 --until-image14:45
*** thuc has quit IRC14:45
*** wenlock has joined #openstack-infra14:45
Kiallfungi: ah yes, my good old friend the metadata service -_-14:46
*** thuc_ has quit IRC14:46
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for nova ec2 bug 1280532  https://review.openstack.org/8444914:47
*** gokrokve has joined #openstack-infra14:47
uvirtbotLaunchpad bug 1280532 in nova "Detach volume fails with "Unexpected KeyError" in EC2 interface." [High,In progress] https://launchpad.net/bugs/128053214:47
KiallI'm actually surprised nodepool does it that way, rather than using DIB and uploading the same image everywhere14:47
fungiKiall: we haven't had glance available in our providers14:47
fungiKiall: so we had to design it launching a template instance off a base image in the provider's existing catalog, modify that, shut it down and snapshot it14:48
KiallOh.. Well, yea - HP Cloud's new regions give you glance access if you ask for it14:49
KiallNo clue if RAX does the same though14:50
fungiKiall: yep, and we have (and have tested it now successfully), but the new regions were also having other issues we've been trying to work around first (dns configuration strangeness, not enough cpu:ram ratio, high error rate on nova boot calls)14:50
fungiand rax also just started offering glance as well, though you have to do "special things" apparently to be able to make it work there at the moment14:51
*** basha has quit IRC14:51
KiallYea that - dns config one annoys me too, but it's a neutron issue :'(14:51
fungiso we're getting closer to being able to start looking at switching how nodepool does image management for the bette14:51
fungir14:51
*** gokrokve has quit IRC14:51
*** gokrokve has joined #openstack-infra14:53
BobBallI've got a strange issue in RAX with the introduction of trove... http://paste.openstack.org/show/74753/ - I thought that localhost is being resolved to the v6 address yet everything is configured to use v4 - but preferring v4 didn't solve it.14:56
BobBallActual error is when accessing localhost: "Caused by <class 'socket.gaierror'>: [Errno -9] Address family for hostname not supported"14:56
*** thedodd has joined #openstack-infra14:56
*** thuc has joined #openstack-infra15:00
fifieldtlol@ freenode+15:00
*** thuc has quit IRC15:01
ttxfifieldt: that would make the DefCore meetings much more integrated.15:01
fifieldthaha15:01
fifieldttouche!15:01
*** thuc has joined #openstack-infra15:02
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: normalize the services for a post grizzly world  https://review.openstack.org/8428715:05
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: turn back on log enforcement  https://review.openstack.org/8121615:05
sdagueBobBall: how about that https://review.openstack.org/81216 for 3rd party?15:05
*** zehicle_at_dell has joined #openstack-infra15:06
BobBallPerfect sdague - thanks.15:08
openstackgerritSean Dague proposed a change to openstack-infra/config: turn off clean log enforcement in stress tests  https://review.openstack.org/8446915:08
jeblairfungi: good morning15:09
fungijeblair: good morning to you15:09
fungijeblair: not sure if you caught the scrollback, but while i was at dinner last night clarkb ended up restarting nodepool and discovered that the subnode changes were failing to write into the nonexistent /etc/nodepool dir on new nodes15:10
sdaguejeblair / fungi: because this will change how a few things run in d-g jobs, https://review.openstack.org/#/c/84146/ would be good to make sure infra folks are on board with15:11
sdaguedhellmann: actually your thoughts on https://review.openstack.org/#/c/84146/ would be welcome as well, I think it gets us out of the olso vs. pip issue once and for all15:11
jeblairfungi: ah yeah, that's created by the image build step; so i think we need to kick off image builds when we start with that version15:11
fungihe proposed a change to solve that, but in the interim rolled back the local clone on nodepool.o.o to the point where labels were introduced and then temporarily stopped puppet so we could consult with you15:11
mordredsdague: it looks good to me - do you forsee any particular effects in the gate?15:12
*** chandankumar_ has joined #openstack-infra15:12
mordredsdague: (other than less breakage)15:12
mordredsdague: it LOOKS like it will be a no-op for us15:12
fungiokay, so we roll those back forward, restart nodepoold and start new image builds immediately?15:13
sdaguemordred: I don't15:13
jeblairfungi: yeah, that change won't work -- the user doesn't (necessarily) have permission to create that directory (it gets created world-writable)15:13
sdaguejust the old instincts on the breaks will change15:13
sdaguebecause installing things from /opt into the system path will be a different behaviro15:14
fungijeblair: how would you want to handle, e.g., being unable to get new images reliably in hpcloud-az2 for a while when we do that?15:14
jeblairfungi: so yes, that's probably the safest thing to do, though it'll slow new node creation for a bit.  we could merge a change that avoids writing to that directory and make this a two-parter15:14
sdagueI'm hoping it just makes all things better15:14
*** erw has joined #openstack-infra15:14
*** dkliban is now known as dkliban_afk15:15
fungijeblair: oh, good point, if we can get in the image build change first and then the subnode change which uses it, we have the luxury of getting image updates in some of those problem providers on longer time scales15:15
BobBallWhy in stack.sh is RABBIT_HOST defaulted to localhost rather than SERVICE_HOST? http://git.openstack.org/cgit/openstack-dev/devstack/tree/stack.sh#n42815:18
*** jcoufal has quit IRC15:18
sdagueBobBall: I expect to get around some failure we found previously15:19
*** hashar_ has joined #openstack-infra15:19
BobBallhmmmm... the use of "localhost" is the eventual cause of my trove ipv4/ipv6 issues... of course I can just specify RABBIT_HOST in my localrc but if there is a more general fix I wanted to go that route15:20
sdagueactually this seems to go away back15:22
*** hashar has quit IRC15:22
*** ildikov_ has quit IRC15:22
*** CaptTofu has joined #openstack-infra15:22
*** zns has quit IRC15:22
*** alexpilotti has quit IRC15:22
sdagueBobBall: if you want to propose a change, it's probably fair to15:23
*** alff_ has quit IRC15:23
*** alff has quit IRC15:23
fungijeblair: http://git.openstack.org/cgit/openstack-infra/zuul/tree/tools/trigger-job.py#n46 is no longer really a sane url... any ideas on how we should be identifying which merger to use when triggering?15:25
*** markwash has quit IRC15:25
*** markwash has joined #openstack-infra15:26
*** hashar_ has quit IRC15:26
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add a multinode label to nodepool  https://review.openstack.org/8447315:27
jeblairfungi: if retriggering, then use the value in ZUUL_URL that was previously passed15:28
fungijeblair: ahh, right-o15:28
jeblairfungi: it should show up as a param15:28
fungiit does... how log will it take those to get gc'd? (or will they?)15:28
fungis/log/long/15:28
*** zehicle_at_dell has quit IRC15:30
*** dstanek has quit IRC15:30
*** zehicle_at_dell has joined #openstack-infra15:31
BobBallsdague: https://review.openstack.org/#/c/84475/ FYI15:31
*** hashar has joined #openstack-infra15:33
jeblairfungi: they should be there until we rm-rf the underlying git repos15:34
fungijeblair: perfect15:34
mordredzaro: so, if you're going to start hacking on gerrit ...15:35
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add a multinode label to nodepool  https://review.openstack.org/8447315:35
mordredzaro: I'd love it if I could request that gerrit attach patches or inline diffs into emails it sends me :)15:35
*** mrodden has quit IRC15:35
jeblairmordred: i think the new change screen is terrible from a usability and design pov.  i don't think we can in good concience upgrade to a version that requires it.  should addressing that be a priority?15:37
mordredjeblair: yah. prolly so15:38
sdaguejeblair: ++15:38
mordredzaro: as usual, NEVER listen to me when I drop suggestions for things to work on into irc15:38
sdagueheh15:38
mordredjeblair: do we know a timeline for 2.9 yet?15:39
jeblairmordred: i don't know15:39
sdaguemordred: honestly, I was sort of thinking once we have a version of gerrit that has a REST api, to do the notification piece completely out of band15:39
mordredjeblair: and/or is making a plan for addressing the new screen a thing we dan do at the summit?15:39
sdaguebecause the filters available for notification are kind of limitted anyway15:40
mordredsdague: I think there are _many_ opportunities that open up once we have a REST api15:40
sdagueagreed15:40
mordredsdague: but for notifications, if we wanted non-internal notifies...15:40
*** yassine has quit IRC15:40
*** mgagne has joined #openstack-infra15:40
*** dizquierdo has quit IRC15:40
mordredI think the ssh event-stream is probably a better place to get info to do that15:40
sdaguepossibly15:40
mordredlike, we could do that now - the biggest problem would be how users change their notification settings15:41
mordredbecause I don't think code-review-per-dev-per-setting is a good idea :)15:41
*** mfer has joined #openstack-infra15:41
jeblairmordred: i'm shocked to hear you say that!15:41
mordredjeblair: I'm a shocking guy15:41
jeblairsdague: can you give me a really short summary of why non-editable makes a difference for the oslo libs?15:42
sdaguebecause if you pip install -e /opt/stack/oslo.config15:42
mordredjeblair: aparently python namespace packages and pip and editable break because $reasons15:42
sdaguethen pip install olso.vmware15:42
sdagueoslo.config is now gone15:42
mordredsdague: oh - you know what - that's the most helpful description I've heard of the problem15:43
mordredthat actually makes sense to me why it breaks15:43
fungijeblair: mordred: zaro: are we also stuck not being able to use the wip workflow feature effectively without using the new change screen, or was i misunderstanding that discussion about the pop-up pane for workflow only working in the new change screen?15:43
fungiis that a 2.9-only concern for now?15:43
sdaguemordred: well, we only hit that issue yesterday :)15:43
sdagueand I agree, it clarifies15:43
mordredsdague: but in general - develop/pip -e works by making symlinks to places /python .pth files - namespace pacakges use similar magic hacks15:44
mordredthe two together ... oy15:44
sdaguemordred: yep15:44
*** zns has joined #openstack-infra15:45
mordredsdague: have we considered just finishing the job and getting rid of our use of namespace packages? I'm not sure they REALLY actually get us anything...15:45
sdaguemordred: I keep asking that one :)15:45
mordredI mean, 'it looks nicer' is all well and good - but there isn't an actual technical benefit15:46
sdaguehowever, I think the use case for pip install -e for libraries in devstack isn't sound anyway15:46
sdaguepip install -e /opt/stack/nova makes a ton of sense15:46
mordredsdague: sure15:46
jeblairsdague, mordred: thank you both very much for that.  a lot of things make sense now (at least, i have a working knowledge)15:46
sdaguejeblair: no problem15:46
*** dprince has quit IRC15:46
mordredjeblair: it was helpful for me too15:46
*** thuc_ has joined #openstack-infra15:47
fungii also found it elucidating15:48
sdaguejeblair: want to +A it - https://review.openstack.org/#/c/84146/15:48
fungii had never really thought much about how setup.py develop worked before15:48
*** jcoufal has joined #openstack-infra15:48
*** UtahDave has joined #openstack-infra15:48
sdagueI think all parties have weighed in now on it15:48
jeblairsdague: will do15:49
sdaguethanks15:49
*** gyee has joined #openstack-infra15:49
pleia2good morning15:49
sdaguecompletely unrelated, is infra using tagmail anywhere?15:49
sdaguefor puppet things15:49
*** thuc has quit IRC15:50
anteayamorning pleia215:50
jeblairsdague: does that cells-full failure look okay?15:51
*** thuc_ has quit IRC15:51
sdaguethe non voting one?15:51
*** mrodden has joined #openstack-infra15:51
jeblairsdague: yep15:51
sdagueyeh, that's not supposed to pass yet15:51
jeblaircool15:51
dstufftmordred: ~magic hacks~15:51
dstufftmordred: FWIW +100 on killing namespace packages15:52
dstufftmordred: oslo_foo isn't inheriently worse than oslo.config, but it'll work more consistently :)15:52
mordreddstufft: ++15:53
mordredsdague: we do not15:54
mordredsdague: we try to avoid email notification for things if we can :)15:54
cody-somervilleIf one changes the min-ram value in nodepool, do you have to do anything special to deploy that change?15:54
sdaguemordred: no worries, just curious. I hacked together a gpg version of it this weekend for personal use, was trying to figure out if it was of interest15:55
*** julienvey has quit IRC15:55
mordredsdague: I think too many of our puppet configs are too chatty (stuff that erroneously thinks it's changed every time) for it to be of use15:56
fungiyeah, idempotent exec blocks are a problem there15:57
fungithey run on every update and that shows as a change15:57
jeblaircody-somerville: theoretically, no, but if there's a bug it may need to be restarted.  also, depending on in what manner you changed it and the behavior of your cloud, launching new nodes from an image of a different size may or may not work.15:57
sdaguefungi: sure, you definitely need to make the idempotent exec blocks have a creates or unless clause15:57
*** jlibosva has quit IRC15:57
fungisdague: right, we should totally do that. we haven't (yet)15:58
sdaguemy config is only about 6 - 8 nodes15:58
sdagueand pretty clean on runs, so it's handy to see when things update15:58
cody-somervillejeblair: Will it delete and recreate nodes that no longer meet min-ram requirement?15:58
jeblaircody-somerville: no15:59
*** changbl has joined #openstack-infra15:59
*** jgallard has quit IRC16:00
*** hashar has quit IRC16:00
*** lcheng has joined #openstack-infra16:01
fungizaro: i was finally able to get my https://review.openstack.org/82583 fix for ggp onto enough nodepool images that retriggering each of the tag jobs for your presentation hit a node with the updated script and worked, so it's showing up in the index now16:01
*** hashar has joined #openstack-infra16:01
*** moted has quit IRC16:01
*** hashar has quit IRC16:02
*** moted has joined #openstack-infra16:02
jeblairnibalizer: regarding https://review.openstack.org/#/c/84356/1 may i suggest the openstack-dev/sandbox repo? :)16:03
*** alff has joined #openstack-infra16:03
*** zehicle_at_dell has quit IRC16:03
*** BadCub01 has quit IRC16:05
*** jlibosva has joined #openstack-infra16:05
*** jgallard has joined #openstack-infra16:08
*** e0ne has quit IRC16:09
*** amotoki has joined #openstack-infra16:10
*** Sukhdev has joined #openstack-infra16:10
*** jepoy has quit IRC16:12
*** mwagner_zzz has joined #openstack-infra16:16
*** jgallard has joined #openstack-infra16:17
*** alff_ has joined #openstack-infra16:17
*** thuc has joined #openstack-infra16:18
*** thuc has quit IRC16:18
*** thuc has joined #openstack-infra16:19
zaromordred:16:19
zaromorning16:19
zarofungi: thanks16:20
*** mfer has quit IRC16:20
openstackgerritEric Windisch proposed a change to openstack-infra/config: Skip requirements check for Docker  https://review.openstack.org/8448816:22
jogoclarkb: ahh that explains the spike on http://jogo.github.io/gate/16:23
*** jgallard has quit IRC16:24
sdaguejogo: what is the explanation?16:24
*** CaptTofu has quit IRC16:24
jogosdague: https://review.openstack.org/#/c/84319/16:25
sdagueoh, interesting16:25
jogographite shows things are fixed again16:25
sdagueok, so that was the big spike16:26
jogoheh yeah, that would result in a 100% failure16:26
jogothe graph hides that in its moving averaging16:26
sdagueso how come that triggered the network bugs?16:26
sdaguehttp://status.openstack.org/elastic-recheck/16:27
jogosdague: well large-ops doesn't  create VMs16:27
sdagueI didn't think they ran real ssh16:27
sdagueok, so how are the ssh tests failing at all?16:27
sdaguebecause they shouldn't be run, right?16:27
fungii believe the problem is that they weren't supposed to, and that's where it was breaking16:27
jogosdague: it ran ssh by accident or something16:27
fungiyeah, because that comparison was missed, tests tried to ssh to not-real virtual machines16:28
sdagueok16:28
jogosdague: it hit the else statement16:28
jogowhich triggered smoke tests which then failed16:28
sdagueand d-g doesn't gate on large-ops jobs?16:28
fungithe end result, of course, looks a lot like the bug where the instance doesn't come up and the ssh test legitimately fails16:29
sdagueyep16:29
*** dtroyer has quit IRC16:29
sdaguejust trying to sort out if this is a hole we need to fill16:29
zarofungi, mordred : I believe WIP will work on old change screen.  from my discussion with _david_ what doesn't work is that when you set/unset WIP the popup to provide a message does not work.16:29
jogosdague: hmm good question16:29
nibalizerjeblair: okay, next time i'll use that one16:29
fungizaro: ahh, okay. so the instructions which go along with the workflow label aren't displayed?16:30
jogosdague: devstack gate does gate on large-ops16:30
zarofungi, mordred : the change does get set into the correct state, but user just doen't get to enter a message for the state change.16:30
*** dtroyer has joined #openstack-infra16:30
sdaguejogo: that seems like something we probably want to change16:31
*** zns has quit IRC16:31
fungizaro: oh! the comment box. okay16:31
zarofungi, mordred : yes, comment box.  i assume if you wanted to add a comment you would have to do it in two steps.16:31
jogosdague:  ohh I think I see it16:32
*** maru_afk is now known as marun16:32
jogosdague: when https://review.openstack.org/#/c/80643/ merged the large-ops value was 116:32
fungizaro: that seems like something we could live with i suppose16:32
*** UtahDave has quit IRC16:32
*** dteselkin_ has joined #openstack-infra16:32
zarojeblair: your thoughts? ^16:33
jogosdague: then it got switched to 5016:33
jogoas we want to turn that from a bool (0,1) to the number of instances to boot16:33
fungijogo: sdague: so the hole where we didn't gate on the actual bug trigger was the change to the jjb definition16:33
*** Sukhdev has quit IRC16:33
sdagueah, so it was layout change16:33
sdaguewhich isn't currently gateable16:34
jogosdague: correct I98baec6e0890f6b10afbd9858ded933f59db972516:34
jogoin config16:34
sdagueyou know, that would be a kind of awesome thing to do, actually run the results of layout job changes16:34
fungithat's something else which might be more solvable in a post-jenkins world... testing the implications of alterations to the definitions of existing jobs16:34
sdaguebut it would be complicated16:34
*** zns has joined #openstack-infra16:34
sdaguefungi: yep16:35
jogomy limited understanding of this, is this would be a side effect of gozer. as they will need to produce code to automatically spin up an infra16:35
dhellmannsdague: we talked about that the other day -- I'm ok with it, and we'll see if it causes issues or confusions for devs working on those libs.16:35
dhellmannsdague: were you still seeing ns package issues?16:36
zaromordred: i've been exposed! i'll keep your gerrit request on the peripheral.  i think i saw an openstack-ci bug on that already.16:36
zaromordred: been assigning gerrit bugs to myself, but really just wanted to star them but didn't see a way to do that in LP.16:37
openstackgerritJames E. Blair proposed a change to openstack-infra/nodepool: Protect against /etc/nodepool not existing  https://review.openstack.org/8449216:40
jeblairfungi: ^ (testing that took a while)16:41
fungijeblair: oh! awesome16:41
*** dprince has joined #openstack-infra16:41
jeblairfungi: (i chose the provider that is slow to spin up nodes as opposed to the one that has trouble building images)16:41
fungijeblair: yep, that looks like just the ticket16:41
fungijeblair: hah16:42
fungijeblair: it's awesome that we have such a choice16:42
*** gothicmindfood has joined #openstack-infra16:43
dansmithclarkb: doh, sorry about that largeops omission16:43
*** derekh has quit IRC16:44
*** dstanek has joined #openstack-infra16:50
fungijeblair: are the 'No handlers could be found for logger "nodepool.NodePool"' errors during test_subnodes on your radar?16:53
*** mihgen has quit IRC16:54
*** mriedem has quit IRC16:56
fungioh, actually that was on a python26 jobm but now i;m looking at a python27 job hitting something else...16:56
*** mihgen has joined #openstack-infra16:57
*** mihgen has quit IRC16:57
*** BadCub01 has joined #openstack-infra16:57
*** khyati has joined #openstack-infra16:58
fungi"AttributeError: 'NoneType' object has no attribute 'db'" (not sure whether that's a cause or an effect yet)16:58
fungiself.config is ending up as None16:58
*** mgagne has quit IRC16:59
fungiahh, yep they're the same... just the python26 job ends up with the traceback in the subunit log rather than the console17:00
*** zehicle_at_dell has joined #openstack-infra17:00
*** SumitNaiksatam has quit IRC17:00
*** arosen has quit IRC17:02
*** jcoufal has quit IRC17:04
*** imcsk8_lap has quit IRC17:04
*** imcsk8_lap has joined #openstack-infra17:04
*** afazekas is now known as afazekas_noteboo17:05
*** amotoki has quit IRC17:05
*** Sukhdev has joined #openstack-infra17:05
*** CaptTofu has joined #openstack-infra17:05
*** Ryan_Lane has joined #openstack-infra17:08
*** afazekas has joined #openstack-infra17:08
*** Sukhdev has quit IRC17:09
*** UtahDave has joined #openstack-infra17:10
clarkbmorning17:11
*** sarob has joined #openstack-infra17:12
*** harlowja_away is now known as harlowja17:14
anteayamorning clarkb17:14
fungiclarkb: see the immediate above, i think we may have an issue with the unit tests for nodepool (in addition to the tox/sphinx issue you spotted last night). also getting 84492 in so we could restart puppet on nodepool.o.o (i can arrange a nodepool restart later in the evening so we're sure it's sane) would be stellar17:14
*** mwagner_zzz is now known as mwagner_17:14
fungii'm poking at the nodepool unit test failures now17:14
clarkbfungi: yup, I didn't have time to look into the unit test failures last night17:14
clarkbfungi: I am guessing some small minor detail issue difference between jeblair's local test setup and the slaves17:15
clarkbfungi: it looked like jobs were timing out at the partial subunit log was being captured though17:15
fungiclarkb: yeah, i think it's not getting a config object17:15
*** SumitNaiksatam has joined #openstack-infra17:15
fungiself.config ends up being None for the pool17:16
*** mgagne has joined #openstack-infra17:17
*** david-lyle is now known as david-lyle_afk17:17
*** jpich has quit IRC17:18
clarkbinteresting17:19
*** mriedem has joined #openstack-infra17:19
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard-webclient: ngDelete directive  https://review.openstack.org/8450417:20
*** jlibosva has quit IRC17:20
fungii wonder if we need to call pool.loadConfig() before pool.getDB17:21
fungior maybe self.updateConfig()17:23
*** alexpilotti has joined #openstack-infra17:25
*** afazekas_noteboo is now known as _afazekas17:25
jeblairfungi, clarkb: oh, do you have a link to a completed test run?17:25
fungijeblair: http://logs.openstack.org/27/84327/1/check/gate-nodepool-python27/41c96ba/console.html17:26
fungii've got a change on my way to test the theory17:26
fungibut if you know off the top of your head what's missing, all's the better17:26
jeblairfungi: not off the top17:26
*** zhiyan is now known as zhiyan_17:27
jeblairfungi: maybe a race17:27
openstackgerritJeremy Stanley proposed a change to openstack-infra/nodepool: Update config before accessing DB in tests  https://review.openstack.org/8450517:27
fungimay not have the method order correct on that either17:27
jeblairfungi: wow, you'd think a 2 second sleep would be enough.  yay python thread scheduling?17:28
jeblairfungi: tripleo looks :(17:29
*** thedodd has quit IRC17:29
jeblairis there a known issue or should i investigate?17:29
fungijeblair: issue is not known to me, if it's known17:30
jeblairfungi, clarkb: so the nodepool docs error looks like a sphinx version issue we've seen before: https://jenkins03.openstack.org/job/gate-nodepool-docs/2/console17:30
*** morganfainberg_Z is now known as morganfainberg17:31
jeblairi don't understand why we would hit it though, i think we're using the same version as the other infra tools17:31
fungijeblair: looking at test_node and test_subnodes, should we be calling pool.startup() instead of pool.start() maybe?17:31
*** CaptTofu has quit IRC17:32
fungijeblair: the docs error clarkb has a change to the tox.ini which fixes it17:32
*** CaptTofu has joined #openstack-infra17:32
jeblairfungi: no -- .start() starts the thread (which calls the run method which calls startup)17:32
fungijeblair: https://review.openstack.org/8432717:33
jeblairfungi, clarkb: oooooh.  thanks.  :)17:33
fungijeblair: oh, okay. i was having trouble tracking down where start() was defined17:33
jeblairfungi: threading.Thread.start()17:33
fungiyep. yay inheritance17:34
jogowhat is the status of python-openstackclient?17:34
jogoits not part of a program17:34
mordredjogo: that is its status17:34
mordredjogo: dtroyer may apply for a program for it17:34
jogomordred: then it belongs in stackforge17:34
jogo?17:34
mordredjogo: we had that discussion, which led to the idea that dtroyer needed to apply for a program potentially17:35
jogoand we shouldn't be deprecating other CLIs and say use openstackcl;ient17:35
fungijogo: except we'd rather not go to the trouble of moving it twice in a short period of time17:35
jogofungi: ahh17:35
jogoFWIW I am all for it17:35
*** sarob has quit IRC17:35
mordredjogo: I think the idae is that, as a project, we want this openstackclient thing - but we haven't gotten our ducks in a row about it yet17:35
*** ildikov_ has joined #openstack-infra17:35
mordredso I think we're trying to figure out how to properly indicate that everytone actually does want the thing :)17:35
*** sarob has joined #openstack-infra17:36
jogomordred: right, so the question is: keystoneclient doesn't have a v3 CLI instead they say use openstackclient17:36
jogomordred: which is really awesome, but IMHO premature17:36
jeblairfungi: topic in tripleo says 'firedrill ci overcloud down' so i'm assuming it's known17:36
*** mfer has joined #openstack-infra17:36
jeblairfungi: we're getting ClientException: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-b74e7625-1bb2-43e1-a456-6159760a2c52)17:36
fungijeblair: okay, thanks!17:36
dtroyerfwiw, I should have the program application shortly…still working out a few details WRT other projects that may be a part of it17:36
*** che-arne has quit IRC17:36
*** dkliban_afk is now known as dkliban17:36
*** CaptTofu has quit IRC17:37
jogodtroyer: awesome, so I have a hunch that with the other CLI related efforts that program application will have to cover a few stackforge repos as welll17:37
jogodtroyer: thoughts on if it makes sense for 'If you mean the keystone CLI that is currently bundled as part of the17:37
*** mfer has quit IRC17:37
jogopython-keystoneclient then yes that is deprecated in favour of17:37
jogopython-openstackclient.17:37
jogo'17:37
mordredjogo: which stackforge projects?17:38
* jogo checks17:38
dtroyerjogo: I've been supportive of that since dolphm started saying it a while back…the Identity stuff is the most completely implemented set of commands in OSC of any of the APIs17:39
jogostackforge/python-openstacksdk17:39
* mordred is in favor of our new dtroyer overlord17:39
mordredjogo: I think that may or may not be a related thing, depending on what dtroyer pitches the scope of openstackclient to be17:39
jogomordred: perhaps17:40
dtroyerThe SDK projects are likely not going to be part of the initial program application based on what I know ATM, they will follow when they are ready...17:40
mordredjogo: the sdk is intended to be like openstackclient - except to be a single lib for devs to use17:40
jeblairyaaaaaaaaaaaaaaaaaaaaaaay17:40
mordredyah. I think it would be great to have both things - but I think the sdk is a bit younger17:40
dtroyerthe python-openstackclinet will eventually be the bottom half of OSC...17:40
jogodtroyer: I guess you can always ammend the charter for the program17:40
dtroyerrats….the python-openstacksdk will ...17:40
mordreddtroyer: ++17:40
*** e0ne has joined #openstack-infra17:40
jogoI think a client focused prgram would be neat17:40
*** sarob has quit IRC17:40
notmynamewhat version of pep8 are we using now?17:40
mordrednotmyname: 1.4.517:41
jogonotmyname: http://git.openstack.org/cgit/openstack-dev/hacking/tree/requirements.txt?h=0.8.x17:41
mordredjogo's answer is better17:41
jogodtroyer: so I think we have a chicken and egg issue with you answer17:41
jogore: use OSC17:41
dtroyerno, today OSC uses the existing client libs…when its ready, it'll be using the SDK17:42
jogodtroyer: well from a governance point of view I mean.17:42
anteayadtroyer: while we are talking programs and such, do you think devstack can rustle up a ptl candidate?17:42
dtroyerI expect that'll be some time based on how fast things seem to be going in this area17:42
dtroyerah, well, we don't use them today17:43
jogodtroyer: a program shouldn't be able to depreacte something in favor of something that *should* be in stackforge17:43
dtroyerand I expect that python-openstacksdk will be part of the program before we start shipping an OSc that uses it17:43
* anteaya notes that 8 programs still need at least one ptl candidate17:43
dtroyeranteaya: noted…*its in the mail*17:43
anteayadtroyer: thank you17:44
jogoanteaya: whoa what programs?17:44
anteayahttps://wiki.openstack.org/wiki/PTL_Elections_March/April_201417:44
mtreinishjogo: you going to step up?17:44
jogodtroyer: one solution to this is to sort out the status of OSC soon17:44
anteayathe ones with no candidates17:44
dtroyerthat lazy devstack ptl for one hasn't sent anything yet17:44
dtroyerjogo: right…hence the pending program app for it17:44
jogomtreinish: oh I should do keystone ;)17:44
jogodtroyer: oh its pending?17:45
anteayaat least one commit in the last two releases gets your hat in the ring17:45
*** erw has quit IRC17:45
dtroyerwell, pending in the sense that I'm going to send it in a day or three…I'm still getting everything put together17:45
anteayafair enough17:45
jogodtroyer: ahh cool17:46
jogoanteaya: I do have a commit in keystone heh17:46
anteayaI'm going to send out a nagging email tonight17:46
sdaguedtroyer: have you given devstack a run on 13.10 recently?17:46
anteayajogo: send an email17:46
jogohaha17:46
*** chandankumar_ has quit IRC17:46
dtroyersdague: no, I'm stuck in precise-land for now17:46
mtreinishjogo: you've got my vote (except I don't think I have any keystone commits :) )17:46
anteayamtreinish: there's still time17:46
sdagueso, it definitely doesn't work from my recent experiences17:47
mtreinishanteaya: that was my intent with: https://review.openstack.org/#/c/84258/ :)17:47
sdaguewhich concerns me about 14.0417:47
anteayamtreinish: why not, pressure some cores, get yourself a vote17:48
mtreinishanteaya: actually would python-keystoneclient count so I could vote for jogo?17:49
openstackgerritMichael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Tag Input Control  https://review.openstack.org/8394517:50
jogomtreinish: woot!17:50
*** amcrn has joined #openstack-infra17:51
*** jooools has quit IRC17:51
sdagueanteaya: is voting based on contributions in icehouse? or i & h?17:53
anteayamtreinish: sure17:53
anteayai & h17:53
anteayaand eligibiligy is based on time17:53
anteayacommitters for one of the program projects over the Havana-Icehouse timeframe (April 4, 2013 06:00 UTC to April 4, 2014 05:59 UTC)17:54
anteayaso merges prior to April 4 05:50 utc count17:54
anteayafire up those doc string patches17:54
clarkbsorry actually at computer now17:55
clarkbtripleo is sad because my by hand fix to the nodepool.yaml didn't bump the max servers. Looking at that now17:56
clarkbfixed, max is now 5517:57
openstackgerritKhai Do proposed a change to openstack-infra/config: update jenkins build-timeout plugin  https://review.openstack.org/8451218:00
*** mtreinish has quit IRC18:00
fungiclarkb: that doesn't seem like it would cause "ClientException: The server has either erred or is incapable of performing the requested operation. (HTTP 500)"18:00
*** mwagner_ is now known as mwagner_afk18:00
clarkboh, is that the actual issue?18:00
clarkbI guess thats something we punt to #tripleo?18:00
fungilooks that way, and they have a firedrill topic up in their channel so we assume they're working to solve it18:01
clarkbok18:01
*** zns has quit IRC18:01
fungibut can't hurt to check with them if you want to18:01
openstackgerritA change was merged to openstack-infra/config: Enable sudo for rally-install job  https://review.openstack.org/8439418:01
*** mtreinish has joined #openstack-infra18:01
*** jamielennox|away is now known as jamielennox18:02
clarkbfungi: jeblair: anything else I can do to help debug the nodepool unit test trouble?18:03
jeblairhrm, i ran fungi's test locally and it works but it looks like the tests are still heading for a timeout18:04
jeblairi'll hold a node18:04
fungiyay, my loop of image build attempts for hpcloud-az2 bare-precise finally paid off!18:04
*** julim has quit IRC18:04
fungiyeah, i think that didn't seem to help matters18:04
clarkbmy change will need to be squashed into whatever the unittest fix is, feel free to do that and abandon my change when we get there18:05
fungii'm going to try shuffling the updateConfig() call to come after the start() call18:05
sdaguehttps://review.openstack.org/#/c/81216/5 could use a +A before another service manages to break itself :)18:05
fungisdague: that's safe now?18:05
sdagueI believe so18:05
jeblairfungi: not sure i like that; could end up with concurrent modification in multiple threads18:05
openstackgerritKhai Do proposed a change to openstack-infra/config: fix installation of nodepool on jenkins-dev  https://review.openstack.org/6132118:05
openstackgerritKhai Do proposed a change to openstack-infra/config: update jenkins-dev.o.o with same plugins as jenkins.o.o  https://review.openstack.org/7704018:05
clarkbsdague: looking18:06
sdaguefungi: it's green across the board (nv included)18:06
fungijeblair: you're probably right. i'm short-sighted when it comes to concurrency stuff18:06
*** julim has joined #openstack-infra18:06
*** derekh has joined #openstack-infra18:07
fungisdague: last time we discovered that there were some errors which were existing already and cropped up infrequently but frequently enough to introduce some heavy gate reset frequency. should we recheck this a few times to be sure?18:07
sdaguefungi: realistically, we've massively decreased the logs here18:07
fungisdague: okay, sounds pretty safe in that case18:08
sdagueso I'm feeling pretty good18:08
sdaguewe also were previously whitelisting lines in a log18:08
*** LinuxJedi has joined #openstack-infra18:08
fungii'm +2 on it but i'll let clarkb approve18:08
sdaguenow we are whitelisting whole log files18:08
fungithat gives you a bigger hammer, agreed18:08
clarkbapproved18:09
sdaguewoot18:09
sdagueok, now time for a bike ride18:09
fungiget a log file clean and then remove it from the whitelist, lather, rinse, repeat18:09
sdagueas we've finally got our first sunny 60 degree day here18:09
LinuxJedihey guys.  So I'm trying to do "git log master..branch-X" on libra but it is showing stuff that has been cherry-picked from one to the other.  Is there some magic to this I'm missing?18:09
*** zns has joined #openstack-infra18:09
sdaguefungi: exactly18:09
anteayasdague: woot enjoy your bike ride18:09
jeblairhttp://closedstack.org/18:10
fungiLinuxJedi: i don't think there's an easy way to filter out cherry-picks other than maybe commit message matching... a c-p is just another commit from a git standpoint18:10
LinuxJedishit...  thanks anyway18:10
*** Ajaeger has joined #openstack-infra18:10
fungiit doesn't share the original's commit sha18:10
clarkbjeblair: nice18:11
fungigets a new one of its own18:11
fungiteehee18:11
fungi"9:00 Doors Locked - Nobody may leave from this point forward."18:11
*** jerryz has joined #openstack-infra18:11
*** sarob has joined #openstack-infra18:12
anteayanice logo too18:12
*** dkranz has quit IRC18:12
*** sarob has quit IRC18:12
LinuxJediI'm not sure whether it will be faster to manually go through these 126 commits or script it.... Actually I don't work for cloud any more, I'll get the team that run it now to do it...18:12
openstackgerritKhai Do proposed a change to openstack-infra/config: fix installation of nodepool on jenkins-dev  https://review.openstack.org/6132118:12
clarkbLinuxJedi: protip, use the merge18:12
fungiLinuxJedi: :( we miss you!18:12
*** dstufft has quit IRC18:13
fungii also very much like "address the needs of antsy, threatened IT departments worldwide"18:13
*** ominakov has quit IRC18:13
*** _nadya_ has joined #openstack-infra18:13
LinuxJediclarkb: I'll take that in advisement if I ever run a project on Gerrit again :)18:13
jogoanteaya: ping18:13
LinuxJedifungi: lol!  You guys are doing fine without me :)18:14
anteayahere I am18:14
*** dstufft has joined #openstack-infra18:14
*** Sukhdev has joined #openstack-infra18:14
jogohow do I find the contact info for hpyer-v CI?18:14
LinuxJediand I'll be at the Openstack Summit this year18:14
fungiLinuxJedi: find us. there will be beer18:14
mordredjogo: primeministerp should know18:14
anteayajogo: https://review.openstack.org/#/admin/groups/91,members18:14
LinuxJediI shall, hopefully I'll be able to talk about the things I'm working on at that point18:14
jogoprimeministerp:  its broken18:14
anteayathe email is listed18:14
anteayajogo: do you have a link to the brokeness?18:15
jogoanteaya: perfect18:15
jogoanteaya: https://review.openstack.org/#/c/83752/18:15
jogohttp://64.119.130.115/83752/2/18:15
*** talluri has joined #openstack-infra18:15
anteayajogo: help me see the brokeness, so far I am not spotting it18:17
clarkbjeblair: for https://review.openstack.org/#/c/76057/4 I had a brilliant idea this morning. Would it be ok if I make TestScheduler's setUp() more modular, then introduce a second class that inherits from TestScheduler and sets gerrit.fetch_url and gerrit.poll? Then we would continue to test existing behavior and make sure that all of those tests work with the new behavior too. It will double the test18:18
clarkbcount though18:18
jogoanteaya: that is the review to open up juno18:18
clarkbjeblair: its a bit of a hack to make it work, but is straightforward18:18
jogoand I went on the comments the systems added saying passed or failed18:18
openstackgerritJamie Lennox proposed a change to openstack-infra/config: Remove py3 gate job from kite  https://review.openstack.org/8437218:18
*** Ajaeger has quit IRC18:19
anteayahmmm, not sure what behaviour a ci system should have in that situation18:19
anteayabut build failed probably isn't it18:20
jogoanteaya: this is more useful http://www.rcbops.com/gerrit/reports/nova-cireport.html18:20
fungijogo: its tempest log shows some failing tests, for sure18:20
fungijogo: are you saying hyperv should work, or that it shouldn't report results which differ from upstream ci, or what?18:21
*** thuc has quit IRC18:22
anteayayeah 100% failing probably needs some attention18:22
jogofungi: that it should be passing18:22
jogoanteaya: exactyl18:22
jogoexactly18:22
*** CaptTofu has joined #openstack-infra18:23
fungijogo: got it. is there a chance someone approved a change not long ago which breaks the hyperv backend and ignored a report from it?18:24
*** thuc has joined #openstack-infra18:24
*** thuc has quit IRC18:24
fungiat some point more than a week ago18:24
*** thuc has joined #openstack-infra18:24
jogoperhaps18:24
*** vhoward- has left #openstack-infra18:25
jogoasking the maintainers18:25
*** dkranz has joined #openstack-infra18:26
openstackgerritJames E. Blair proposed a change to openstack-infra/nodepool: Fix tox's insane pip install command.  https://review.openstack.org/8432718:27
jeblairclarkb, fungi: ^ that's an attempt to fix all current nodepool test problems18:27
fungiaha!18:27
jeblair(i had a gearman server running locally)18:27
fungithat explains the connection failures on the gearman socket in the logs. i glossed over them because i figured they were a symptom18:28
fungiand my eyes jumped to the python traceback about the db connection18:28
openstackgerritKhai Do proposed a change to openstack-infra/config: fix installation of nodepool on jenkins-dev  https://review.openstack.org/6132118:32
jeblairclarkb, fungi: https://review.openstack.org/#/c/84327/ is ready18:32
openstackgerritJames E. Blair proposed a change to openstack-infra/nodepool: Protect against /etc/nodepool not existing  https://review.openstack.org/8449218:33
jeblairand ^ is rebased18:33
*** erw has joined #openstack-infra18:34
clarkbjeblair: is making the dir not the right solution?18:36
clarkbalso I think it throws an IOError excepton if you want ot be more specific in the ignored error18:36
jeblairclarkb: the directory is created in the image build step; it didn't work yesterday because it was using old images18:37
jeblairclarkb: also, it raises "Exception" because it's the ssh client that does that (it raises Exception on any non-zero exit code of anything it runs)18:38
openstackgerritKhai Do proposed a change to openstack-infra/config: update jenkins-dev.o.o with same plugins as jenkins.o.o  https://review.openstack.org/7704018:39
clarkbjeblair: right, but nodepool can create the dir18:39
*** e0ne has quit IRC18:39
clarkbjeblair: seems like nodepool should be responsible for that if it needs to write to that directory18:39
jeblairclarkb: nodepool _does_ create the dir -- in the image build18:39
*** e0ne has joined #openstack-infra18:40
*** e0ne has quit IRC18:40
jeblairclarkb: that's the only time it can create it because it might lose sudo after running the setup script18:40
fungiclarkb: this ties in with lifeless's comment yesterday about not assuming nodepool has root privs on the launched servers... it does on the template instance and sets /etc/nodepool 0x0777 at that point18:40
*** e0ne has joined #openstack-infra18:40
jeblairfungi: yup18:41
clarkbfungi: no lifeless' comment was about not assuming nodepool had root on the nodepool daemon server18:41
fungioh, then i entirely misunderstood what he was talking about in that case18:41
clarkbfungi: he misunderstood what I was saying too :)18:42
fungifree misunderstandings for everyone!18:42
jeblairi was about to go read the logs, but it sounds like it might not be the best use of time.  :)18:42
clarkbjeblair: is nodepool hardcoding the /etc/nodepool creation in code? or are users expected to know to create that dir?18:43
clarkbI think my biggest concern is whether or not users need to write special bash to make this work (and I don't remeber from reviews if nodepool does this regardless)18:43
jeblairclarkb: http://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/nodepool.py#n77218:44
clarkbdanke18:44
jeblairclarkb: it's documented too: http://docs-draft.openstack.org/27/84327/2/check/gate-nodepool-docs/e96a56e/doc/build/html/configuration.html#labels18:45
clarkbroger thanks18:46
* clarkb reviews the change that deals with the exception18:46
jeblairclarkb: and if you want to see the expected use: https://review.openstack.org/#/c/84473/18:46
clarkbjeblair: comment on 8449218:48
openstackgerritA change was merged to openstack-infra/nodepool: Fix tox's insane pip install command.  https://review.openstack.org/8432718:48
*** thedodd has joined #openstack-infra18:48
jeblairclarkb: that change is intended to be temporary18:49
jeblairclarkb: i'll revert it in a day or so18:49
jeblairclarkb: so basically, i don't expect to actually care whether the code in that block runs during the time the try/except is there18:50
openstackgerritArnaud Legendre proposed a change to openstack/requirements: Update oslo.vmware to be >= 0.3  https://review.openstack.org/8012018:54
*** dstanek has quit IRC18:55
openstackgerritBK Box proposed a change to openstack-infra/config: Join gerritbot to the satori IRC channel  https://review.openstack.org/7905818:56
*** jhesketh has joined #openstack-infra18:59
*** jcoufal has joined #openstack-infra18:59
*** jerryz has quit IRC18:59
jheskethMorning19:00
SergeyLukjanovmeeting time>19:00
anteayamorning jhesketh19:01
*** alff has quit IRC19:02
*** alff_ has quit IRC19:02
*** jcoufal has quit IRC19:03
*** jcoufal has joined #openstack-infra19:03
*** alff has joined #openstack-infra19:06
*** alff_ has joined #openstack-infra19:06
*** _nadya_ has quit IRC19:06
*** e0ne has quit IRC19:06
*** e0ne has joined #openstack-infra19:07
*** rlandy has quit IRC19:07
*** rlandy has joined #openstack-infra19:07
*** blamar has joined #openstack-infra19:08
*** markmcclain has quit IRC19:08
*** gokrokve_ has joined #openstack-infra19:10
*** dkranz has quit IRC19:11
*** thuc has quit IRC19:12
rcarrillocruzhey guys, i need another +2 from a core reviewer for https://review.openstack.org/#/c/79205/ , could anyone please have a look?19:12
*** thuc has joined #openstack-infra19:12
rcarrillocruzclarkb: ^ , i put your suggestions, jeblair already gave the lgtm19:12
*** ameade has quit IRC19:13
*** gokrokve has quit IRC19:14
*** gyee has quit IRC19:14
*** denis_makogon has joined #openstack-infra19:14
*** thuc has quit IRC19:17
*** talluri has quit IRC19:19
thervefungi, If you can have a look at https://review.openstack.org/#/c/83761/ that'd be great19:19
thervejeblair, I think I handled your comments on that one too ^^19:19
kevinbentonis there a known bug for this?19:20
kevinbentonhttp://logs.openstack.org/77/84377/2/gate/gate-neutron-python27/55e30bd/console.html19:20
*** gokrokve_ has quit IRC19:20
kevinbentonthe parent patch has nothing to do with the components that are failing there19:21
*** dkliban has quit IRC19:22
clarkbkevinbenton: have you checked the rechecks page? http://status.openstack.org/rechecks/19:22
kevinbentonclarkb: yeah, nothing obvious. that's why i came here19:22
clarkbkevinbenton: I would ask in the neutron channel19:23
kevinbentonclarkb: i will probably need to file a bug for it. so that's my second question :-)19:23
clarkbwe don't really track bugs for individual projects unless they are causing the gate to crumble19:23
*** mrmartin has joined #openstack-infra19:23
kevinbentonclarkb: where should a bug be filed against something in the openstack common directory?19:23
clarkbkevinbenton: no that is a neutron bug19:23
clarkblooks like neutron isn't properly mocking out a dir19:24
clarkb(/var/lib/neutron)19:24
*** dkranz has joined #openstack-infra19:25
kevinbentonclarkb: oh, great. another race condition in mocking19:25
kevinbenton:-(19:25
anteayakevinbenton: testr run --parallel --concurrency N --until-failure19:26
anteayato flush out race conditions in unit tests19:27
kevinbentonanteaya, clarkb: thanks19:29
*** dstanek has joined #openstack-infra19:30
*** jp_at_hp has quit IRC19:33
kevinbentonanteaya, clarkb: what is the concurrency level of the jenkins nodes?19:36
clarkbkevinbenton: depends. but either 4 or 8 depending on provider/AZ19:36
fungikevinbenton: in what sense. the jenkins executor count?19:36
fungiahh, you mean in terms of testr19:37
kevinbentonfungi: no, parallel threads for testr. trying to figure out race problem19:37
*** mrmartin has quit IRC19:37
clarkbthe test that failed used 4 http://logs.openstack.org/77/84377/2/gate/gate-neutron-python27/55e30bd/console.html#_2014-04-01_18_23_45_66919:37
kevinbentonclarkb: thanks19:37
*** zns has quit IRC19:38
*** jp_at_hp has joined #openstack-infra19:40
openstackgerritSpencer Krum proposed a change to openstack-infra/config: Puppet nodes are unresponsive after 30 minutes  https://review.openstack.org/8453219:40
nibalizerjeblair: ^19:40
funginibalizer: awesome--thanks again19:44
nibalizer:)19:44
*** gokrokve has joined #openstack-infra19:46
*** hashar has joined #openstack-infra19:46
*** maxbit has quit IRC19:47
*** e0ne_ has joined #openstack-infra19:51
*** e0ne has quit IRC19:51
*** e0ne_ has quit IRC19:53
*** e0ne has joined #openstack-infra19:54
*** sdake_ is now known as sdake__19:54
*** yamahata has joined #openstack-infra19:55
*** CaptTofu has quit IRC19:55
*** CaptTofu has joined #openstack-infra19:56
*** ociuhandu has quit IRC19:56
*** thuc has joined #openstack-infra19:56
*** CaptTofu has quit IRC20:00
clarkbpretty sure that RBAC stuff is not openstack though20:00
*** dkliban has joined #openstack-infra20:01
fungiclarkb: so what we're passing into openstackwatch.ini is called "swift_store_key" in hiera, but appears to actually be the rax special api key for that tenant20:02
mordrednotmyname: ping20:02
fungii guess it wasn't a dedicated api key after all20:02
notmynameyo20:02
clarkbfungi: ah ok20:02
*** zns has joined #openstack-infra20:02
clarkbfungi: so I think we have to use !openstack to make this work20:02
mordredso - we're all asking each other swift questions - I figure you know the answer20:02
clarkbfungi: doable with that key or RBAC20:02
clarkbbut less than ideal20:02
mordrednotmyname: if we want to give a program the ability to write intoa  swift container20:02
notmynamemordred: yes swift has container acls ;-)20:02
clarkbnotmyname: right I don't think anyone is contesting that20:02
*** markmcclain has joined #openstack-infra20:03
clarkbits the application of them in particular clouds that is problematic20:03
notmynamemordred: that's going to be somewhat dependent on the auth system. eg RAX auth just recently supported it20:03
clarkbthis ^20:03
mordredah. gotcha20:03
*** mwagner_afk has quit IRC20:03
clarkbanyways I need to grab lunch20:03
openstackgerritClark Boylan proposed a change to openstack-infra/zuul: Add a remote url override location  https://review.openstack.org/7605720:03
mordredcool. that helps. thanks notmyname20:03
*** zns has quit IRC20:03
clarkbjhesketh: jeblair ^ reviews on what I have done with the tests in 76057 would be greaty20:03
notmynamemordred: clarkb: in general, swift has an opaque bucket in which to store "acls", but they are interpreted by the particular auth system running20:04
clarkbjhesketh: jeblair I think that opens the door to doing more testing with non standard config20:04
clarkbso this will end up being another dns like situation. Not terrible but not great20:04
clarkbwe can probably live iwth it20:04
mordrednotmyname: well, we're REALLY close to starting to store stuff in swift - so there may be further follow up questions :)20:04
notmynamemordred: great!20:05
*** derekh has quit IRC20:05
fungipoking in the rax web dashboard, looks like each sub-user in a tenant gets assigned a dedicated api key. suspecting this means having to use the pyrax extension in the client?20:05
jheskethclarkb: yeah, not sure how to bring that into tests for cloud providers besides just trying and seeing what works20:05
fungioh, apparently they get a username and password too, so maybe normal client operations would support this20:07
*** CaptTofu has joined #openstack-infra20:08
fungiahh, the access controls in the rax dashboard, at least, are limited to either granting the account access to all containers, or none. there doesn't seem to be per-container granularity there20:09
fungiso we could limit the account to only be able to read and write (or only be able to read) all storage containers on the tenant and have no access to create servers, cinder volumes, et cetera if we wanted20:10
jeblairwe could also go with public access20:11
openstackgerritA change was merged to openstack-infra/nodepool: Protect against /etc/nodepool not existing  https://review.openstack.org/8449220:11
jeblairi'm not sure the cdn issues we had are actually blockers (i think it mostly had to do with making new dns cnames for each container)20:11
*** lcostantino has quit IRC20:11
jeblairbut if we don't need to cname them at all, then it's not a big deal20:12
fungimore just wondering how we effectively limit write access per container20:12
jeblairfungi: yeah, that would still be good20:13
jeblairit's just possible we might want a container not controlled by zuul.  :)20:13
*** yolanda has quit IRC20:13
fungiif there's support in the api, it's at least not reflected in their webui20:13
*** Sukhdev has quit IRC20:14
*** CaptTofu has quit IRC20:14
*** CaptTofu has joined #openstack-infra20:15
*** e0ne has quit IRC20:16
jeblairfungi: i guess we may need to put it in openstackjenkins then20:17
*** e0ne has joined #openstack-infra20:17
fungijeblair: for the reasons clarkb (who is apparently smarter than me) outlined earlier, i agree ;)20:18
jeblairfungi: yeah, it just never occured to me that with all the talk of container acls, etc, that they would not actually exist20:18
mordredfungi: I think one of these days we're going to discover taht clarkb has actually stolen all of our brains20:18
fungimordred: i asked him to at least give me a receipt for mine20:18
mordredfungi: good luck with that20:19
clarkb:P20:19
*** CaptTofu has quit IRC20:19
clarkbfungi: one brain, slightly mushy received April first20:19
fungidanke20:19
* fungi vaporizes for a few minutes to cook dinner before his next meeting20:20
jeblairnotmyname: do you know how to use container acls with rackspace?20:20
notmynamejeblair: I read the blog from last week.20:21
jeblairnotmyname: ah, i see it.  does swiftclient support setting those things that the blog post describes using curl for?20:23
jeblairhttp://www.rackspace.com/blog/create-cloud-files-container-level-access-control-policies/20:23
jeblairi assume that's the blog post ^20:23
*** gyee has joined #openstack-infra20:24
*** julim has quit IRC20:25
notmynamejeblair: I think it's in swiftclient. checking20:26
*** mrda_away is now known as mrda20:27
*** branen has joined #openstack-infra20:28
notmynamejeblair: -r for read ACLs and -w for write ACLs. (`swift post --help`)20:29
openstackgerritAdam Gandelman proposed a change to openstack-infra/config: Limit Tempest tests in virutal Ironic devstack job  https://review.openstack.org/8454320:29
notmynamejeblair: and yes, it looks like RAX is using the normal ACL headers, so everything should be fine20:29
notmynamejeblair: FWIW, I haven't actually tried it20:29
jeblairnotmyname: cool, thanks much!  i think we might soon.  :)20:30
jeblairfungi, jhesketh, clarkb: ^20:30
*** zns has joined #openstack-infra20:30
*** jcoufal has quit IRC20:37
openstackgerritA change was merged to openstack-infra/devstack-gate: turn back on log enforcement  https://review.openstack.org/8121620:39
*** ociuhandu has joined #openstack-infra20:40
jeblairdkranz, mtreinish, sdague: ^20:40
sdaguewoot20:40
dkranzjeblair: yes!20:40
*** dizquierdo has joined #openstack-infra20:42
*** dims_ has quit IRC20:44
sdagueclarkb / fungi: https://review.openstack.org/#/c/84469/ should land that as well, as it was somewhat coupled with 8121620:46
*** CaptTofu has joined #openstack-infra20:47
*** mfer has joined #openstack-infra20:47
*** mspreitz has joined #openstack-infra20:47
mspreitzI have a procedural question. If two developers want to collaborate on a patch before submitting it for review, what is the right way to do that?20:47
*** CaptTofu has quit IRC20:48
*** CaptTofu_ has joined #openstack-infra20:48
clarkbback from lunch20:48
*** dkliban has quit IRC20:49
clarkbmspreitz: you are free to do it however you like20:49
anteayamspreitz: well one of them has to submit it to gerrit, so one will be considered the owner20:49
clarkbmspreitz: there is no right or wrong way, we have no enforcement on the space20:49
*** CaptTofu_ has quit IRC20:49
anteayathe owner can set wip and abandonment and reinstatement20:50
mspreitzSo we can do it via Gerrit.20:50
clarkbmspreitz: that said there shouldn't be anything that prevents you using gerrit immediately and iterating on a change using WIP20:50
mspreitzThis patch is for a blueprint that has not yet been submitted... so what do we use for a branch name in Gerrit?20:50
anteayamspreitz: either of you can pull down the change and submit20:50
anteayayou can change the branch name later20:51
*** thomasem_ has quit IRC20:51
mtreinishjeblair: awesome20:51
anteayafuture-bp or something20:51
clarkbbranch names are arbitrary20:51
mspreitzthanks20:51
*** CaptTofu_ has joined #openstack-infra20:53
clarkbjeblair: jhesketh woot zuul was happy with https://review.openstack.org/#/c/76057 which makes me happy20:54
*** blamar has quit IRC20:58
*** blamar has joined #openstack-infra21:00
*** david-lyle_afk is now known as david-lyle21:01
*** aysyd has quit IRC21:01
*** rfolco has quit IRC21:02
*** cyeoh has quit IRC21:04
*** blamar has quit IRC21:04
*** jeckersb is now known as jeckersb_gone21:04
*** cyeoh has joined #openstack-infra21:05
*** blamar has joined #openstack-infra21:06
*** CaptTofu_ has quit IRC21:07
clarkbfungi: jeblair: are you guys ready for me to put nodepool back into puppeting?21:08
*** CaptTofu has joined #openstack-infra21:08
clarkbfungi: jeblair: my plan for that is checkout master in /opt/nodepool, start puppet, then at some point we should restart nodepool21:08
*** mfer has quit IRC21:08
clarkb(puppet should install nodepool using the fix for /etc/nodepool not existing)21:08
jeblairclarkb: ++21:11
*** CaptTofu has quit IRC21:12
mattoliverauMorning21:12
*** blamar has quit IRC21:13
*** blamar has joined #openstack-infra21:14
*** dprince has quit IRC21:14
*** pdmars has quit IRC21:14
openstackgerritA change was merged to openstack-infra/config: turn off clean log enforcement in stress tests  https://review.openstack.org/8446921:15
*** pcrews_ has quit IRC21:15
*** alff_ has quit IRC21:17
*** alff has quit IRC21:17
*** blamar has quit IRC21:18
*** blamar has joined #openstack-infra21:18
*** pcrews_ has joined #openstack-infra21:18
*** dkliban has joined #openstack-infra21:18
*** e0ne has quit IRC21:18
*** jp_at_hp has quit IRC21:19
jogohttp://bit.ly/1lAFeUb21:20
jogolooks like new neutron unit test failure21:20
clarkbjogo: yup21:20
jogomy guess is out of memory or something21:20
clarkbkevinbenton was looing at it before21:20
jogoerr disk  space21:20
*** blamar_ has joined #openstack-infra21:20
clarkbjogo: no, that looks like tests that should be using tmpdir fixtures that don't21:20
clarkbjogo: jenkins user can't write to /var/lib/neutron21:21
*** mspreitz has quit IRC21:21
kevinbentonclarkb: still looking at it. it doesn't make any sense21:21
jogosalv-orlando: ^  ahh21:21
fungiclarkb: sounds safe enough21:21
jogoclarkb: thanks, salv-orlando is preping an e-r patch for this bug21:21
salv-orlandokevinbenton: do you have a bug number?21:22
kevinbentonsalv-orlando: no because i haven't determined the root cause yet. if i clone master and then checkout an old commit. it still fails21:22
kevinbentonsalv-orlando: might be a requirements issue21:23
clarkbkevinbenton: salv-orlando: I would try running testr run --analyze isolation against it21:23
clarkbwhich should point out conflicting tests if that is causing it21:23
*** blamar has quit IRC21:23
openstackgerritJoe Gordon proposed a change to openstack-infra/config: Test stable branch on client changes  https://review.openstack.org/8069821:24
kevinbentonclarkb: not necessary. just run tox -v -epy27 neutron.tests.unit.test_l3_agent.TestL3AgentEventHandler.test_spawn_metadata_proxy21:25
kevinbentonclarkb: that's one of the failing tests. just fails on it's own21:25
*** blamar has joined #openstack-infra21:25
kevinbentonclarkb: that's why it seems like something externally changesd21:25
*** blamar_ has quit IRC21:25
kevinbentonchanged*21:25
salv-orlandoI'll then file a generic bug in neutron just for tracking the issue with an e-r query21:25
kevinbentonsalv-orlando: ok21:26
clarkbkevinbenton: oh interesting21:26
salv-orlandokevinbenton: if you're looking at requirements, markmcclain pointed this patch21:26
salv-orlandohttps://review.openstack.org/#/c/83474/21:26
kevinbentonsalv-orlando: so the strange part is that it's still broken if i do a fresh clone and checkout a commit before that21:29
*** adalbas has quit IRC21:29
kevinbentonsalv-orlando: e.g. 3cc5b4d0a312995d3bf5e4314c8a80df73ebd3e121:29
clarkbfungi: jeblair: proceding with nodepool puppet corrective action now21:29
jogosalv-orlando: link to e-r patch?21:29
salv-orlandocool, do you run tox -r21:29
clarkbI am not going to restart nodepool unless we have more than one person around to help out :) learned my lesson yesterday21:29
jogosalv-orlando: is this a 100% failure?21:29
fungiclarkb: thanks21:30
jeblairclarkb: am around21:30
kevinbentonsalv-orlando: no, what does that do?21:30
jogoif so we can just try doign a revert and see what happens21:30
* fungi is around again too for the next half hour at least21:30
salv-orlandotox -r will download again all the dependencies21:30
*** wenlock_ has joined #openstack-infra21:30
salv-orlandojogo: does not seem 100%. Finishing up e-r query and then I'll fetch exact numbers21:30
kevinbentonsalv-orlando: oh, yeah it's still broken in that case21:30
kevinbentonsalv-orlando: just tried21:31
kevinbentonsalv-orlando: i was barbarically deleting the .tox and .testrepository directories before :-)21:31
*** dizquierdo has quit IRC21:31
kevinbentonso i suspect a version of something bumped upwards and broke backwards compatibility21:31
kevinbentonone of the >= requirements21:32
*** markmc has quit IRC21:32
clarkbjeblair: fungi: in that case I am triggering a service restart in a root screen on nodepool.o.o shortly21:32
clarkbso that it will gracefully restart21:33
kevinbentonsomeone must have released an april fools package21:33
openstackgerritSalvatore Orlando proposed a change to openstack-infra/elastic-recheck: Add a fingerprint for bug 1301027  https://review.openstack.org/8455821:34
uvirtbotLaunchpad bug 1301027 in neutron "Permission denied error in unit tests" [Undecided,New] https://launchpad.net/bugs/130102721:34
*** dkranz has quit IRC21:34
clarkbjeblair: fungi: restart is being triggered now21:34
*** wenlock_ has quit IRC21:35
clarkbwow that was quick21:35
clarkbit is running the new stuff now21:35
salv-orlandojogo0: failure rate is at 100%21:35
fungii have, on occasion, had surprisingly but pleasantly quick graceful restarts from nodepool21:35
jogosalv-orlando: so 100% is good at least its easy to reproduce ;)21:36
salv-orlandofwiw e-r query is here: https://review.openstack.org/#/c/84558/21:36
salv-orlandobut with a 100% failure rate we don't need it21:36
jogosalv-orlando: yeah will holdoff on +2ing that until we have a fix .. so we can get better unclassified failure numbers21:37
clarkbseeing OverLimit: Quota exceeded for ram: Requested 16384, but already used 952320 of 960000 ram (HTTP 413) (Request-ID: req-f6655aec-2e77-4e52-81b4-1db50ede65ee) from tripleo again21:38
fungiclarkb: so i guess next we might want to test triggering an image update. i can take that on21:38
clarkbfungi: go for it and ++21:38
fungishould i pick something relatively unused like new kernel or py3k?21:39
clarkbfungi: sure?21:39
fungithough i guess if it succeeds in building an image, we're most of the way there anyway21:39
jeblairfungi: if something fails (like mkdir /etc/nodepool), it should abort, so i wouldn't worry too much21:40
salv-orlandokevinbenton: if you're tracking this please assign bug 1301027 to yourself. I won't be around for long.21:40
uvirtbotLaunchpad bug 1301027 in neutron "Permission denied error in unit tests" [Undecided,New] https://launchpad.net/bugs/130102721:40
jeblairclarkb: there are no aliens and tripleo has < 55 nodes21:41
*** thuc has quit IRC21:41
jeblairclarkb: so the math doesn't add up21:41
kevinbentonsalv-orlando: will do21:41
jeblairclarkb: taking to tripleo21:41
clarkbjeblair: yup, I think that either the bug wasn't fixed or the explosions broke it in new ways21:41
salv-orlandokevinbenton: thanks21:41
*** thuc has joined #openstack-infra21:41
jogoso the timing for oslo.config fits21:42
kevinbentonversion bump in olso21:43
kevinbentonjogo: yep21:43
lifelessmikal: around ?21:43
salv-orlandojogo: oslo.config change might also explain how we can have 100% failure rate in the gate without having seen failures in check queue first21:43
kevinbentonjogo: testing pinned to a lower version now21:43
*** mriedem has quit IRC21:44
clarkbI am seeing ready nodes that look very new, I think nodepool is happy21:45
clarkbwill continue to monitor it21:45
*** thuc has quit IRC21:46
*** hashar has quit IRC21:46
kevinbentonsalv-orlando, clarkb, jogo: lower version fixed it. //review.openstack.org/8456221:47
kevinbentonhttp://review.openstack.org/8456221:47
jogodhellmann: ^21:47
clarkbkevinbenton: will that pass tests?21:47
clarkbhttps://git.openstack.org/cgit/openstack/requirements/tree/global-requirements.txt#n48 may need to be updated first21:48
kevinbentonclarkb: i didn't run the full suite. just the broken ones21:48
clarkbor, is it possible neutron is just using oslo.config in a weird way21:48
fungimmm, my image-update died in a paramiko eoferror... so much fun. restarting21:48
kevinbentoni think it could be something related to the evaluation of '$state' in the unit tests21:49
jogohopefully dhellmann is still around to help on that side21:49
clarkbjogo: I think he mentioned in another channel that his battery was dying21:49
*** salv-orlando_ has joined #openstack-infra21:49
*** enikanorov has joined #openstack-infra21:49
kevinbentonlike now it's being evaluated on definition rather than reference21:49
jogoclarkb: oh yeah21:49
*** e0ne has joined #openstack-infra21:49
fungii didn't even realize dhellmann was a robot, but that makes sense now21:50
jogowhats the oslo room name, lets go over there and ask em21:50
*** pcrews_ has quit IRC21:50
*** erw_ has joined #openstack-infra21:51
jogokevinbenton: confirmed your patch works wonder why ...21:51
*** CaptTofu has joined #openstack-infra21:51
*** e0ne_ has joined #openstack-infra21:51
*** e0ne has quit IRC21:51
salv-orlando_oslo.config works in mysterious ways21:51
kevinbentonjogo: i was suspecting something changed in the cfg parsing of that $state var21:52
*** dstanek has quit IRC21:52
jogo* Fix for parsing error with Dollar Sign ($) in values21:52
kevinbentonthat sounds relevant21:52
kevinbentonjogo: https://github.com/bigswitch/neutron/blob/master/neutron/agent/linux/external_process.py#L3121:52
jogoahhh21:53
* jogo pulls up the oslo patch21:53
*** dguerri_ has joined #openstack-infra21:53
*** sdake_ has joined #openstack-infra21:53
*** sdake_ has quit IRC21:53
*** sdake_ has joined #openstack-infra21:53
*** mtaylor has joined #openstack-infra21:53
*** jpeeler1 has joined #openstack-infra21:53
*** sdake_ is now known as sdake_121:53
clarkbso state_path should probably be a fixtures.TmpDir() or whatever that class is21:53
jogokevinbenton: I4324bda6f3de3cdf05f8eb3cf16052c04018dec821:53
jogoclarkb: yeah21:54
clarkband be set for every test rather than using a global value21:54
jogoand this needs to be applied to the icehouse branch too21:54
*** signed8b_ has joined #openstack-infra21:54
*** DinaBelova2 has joined #openstack-infra21:54
*** signed8bit has quit IRC21:54
*** gaelL_ has quit IRC21:54
*** dguerri has quit IRC21:54
*** jpeeler has quit IRC21:54
*** ruhe has quit IRC21:54
*** openstackgerrit has quit IRC21:54
*** salv-orlando has quit IRC21:54
*** DinaBelova has quit IRC21:54
*** mordred has quit IRC21:54
*** erw has quit IRC21:54
*** sdake has quit IRC21:54
*** Alexandra_ has quit IRC21:54
*** enikanorov_ has quit IRC21:54
*** signed8b_ has quit IRC21:54
*** signed8b_ has joined #openstack-infra21:54
*** salv-orlando_ is now known as salv-orlando21:54
*** gaelL has joined #openstack-infra21:54
*** erw_ is now known as erw21:54
clarkbjogo: possibly, yes21:54
*** DinaBelova2 is now known as DinaBelova21:54
*** ruhe has joined #openstack-infra21:55
*** Alexandra_ has joined #openstack-infra21:55
mikallifeless: wassup, was in meetings21:55
*** e0ne_ has quit IRC21:56
lifelessmikal: hi; you asked for some data on the quota skew thing21:56
lifelessmikal: I'm not sure what data you wanted21:56
lifelessmikal: I don't want to leave the quota wrong, but I don't want to destroy evidence21:56
kevinbentonclarkb: that will be changes in several places. is it necessary for oslo to immediately substitute?21:57
mikallifeless: Hmmm, I don't recall the bug number. Have you got it handy?21:57
*** mfer has joined #openstack-infra21:57
*** mfer has quit IRC21:57
lifeless128442421:58
lifelessmikal: ^22:00
mikalTa22:00
*** dstanek has joined #openstack-infra22:01
*** zehicle has joined #openstack-infra22:02
*** weshay has quit IRC22:04
*** gothicmindfood has left #openstack-infra22:05
mikallifeless: so let me see if I am getting this straight...22:05
*** zehicle_at_dell has quit IRC22:05
mikallifeless: the quotas are wrong because some instances are being double counted?22:05
clarkbkevinbenton: I can't speak for oslo but that is a test bug imo22:05
*** dims_ has joined #openstack-infra22:06
clarkbkevinbenton: even if you weren't using oslo you would want your throwaway data dirs to be managed by a proper fixture22:06
lifelessmikal: we don't know the cause22:06
lifelessmikal: but we know it gets very very out22:06
lifelessmikal: e.g. jeblair says its out right now22:07
*** pcrews has joined #openstack-infra22:07
mikallifeless: but out in the direction of over counting, yes?22:07
mikallifeless: what sort of outage caused the problem this time? Last time it was a control plane outage, yes?22:07
lifelessmikal: the user has more 'used' than the sum of their vms.22:07
anteayafour programs now in need of at least one candidate: https://wiki.openstack.org/wiki/PTL_Elections_March/April_2014#Candidates22:08
lifelessmikal: yes, this time the control plane has been robust but the hypervisors have been falling over like dead flies22:08
anteayahint hint hint22:08
*** denis_makogon has quit IRC22:08
*** dcramer_ has quit IRC22:08
mikallifeless: hmmm, I need to think on this22:08
mikallifeless: I agree its a bug, I just need to work out where to start poking22:09
*** gothicmindfood has joined #openstack-infra22:09
lifelessmikal: is there anything we can do to make poking possible before we destroy the evidence?22:09
*** gothicmindfood has quit IRC22:09
lifelessmikal: since running less slaves than the hardware can is undesirable22:09
mikallifeless: I'd say do a dump of the quota table, and email it to me22:09
mikalThen destroy with vigor22:09
kevinbentonclarkb: right22:10
lifelessSpamapS: ^ thats something you can do to help22:10
* mikal is thinking that perhaps punching devstack nova-compute in the face over and over might duplicate this problem22:10
lifelessSpamapS: get that dump, send to mikal, reset the quota22:10
*** gothicmindfood has joined #openstack-infra22:10
fungiINFO:nodepool.ImageUpdater:Image dsvm-precise-krnl-1396388920.template.openstack.org in rax-dfw is ready22:10
funginow i'll delete all ready nodes for that type22:10
*** hashar has joined #openstack-infra22:11
SpamapSok22:11
SpamapSmikal: just quota_usages ?22:11
mikalSpamapS: and the instances table as well perhaps?22:12
mikalNot sure how big that is in your world22:12
lifeless24K rows22:13
mikalSmall22:13
*** signed8b_ has quit IRC22:13
mikalDump it as well then pleace22:13
lifelessnew cloud22:13
lifelessonly been running 48 hours22:13
mikalOr if you're super keen, mysqldump the world22:13
mikalThat would end the discussion of what tables22:13
*** dstanek has quit IRC22:14
lifelessand boom things are connected22:14
SpamapSugh22:14
*** alff has joined #openstack-infra22:14
*** alff_ has joined #openstack-infra22:14
SpamapSour new percona static binary overlords do not install mysqldump in the path22:14
*** alexpilotti has quit IRC22:14
fungiSpamapS: you know you love them anyway22:15
lifeless47 slaves running / building22:15
lifelessthat seems good22:15
lifelessjeblair: ^22:15
lifelessbroker is back up22:15
lifelesswe should see things happening any second now22:15
fungidsvm-precise-krnl-rax-dfw-3380665.slave.openstack.org is building22:15
fungionce it's up, i'll make sure it has the bits we expect in /etc/nodepool22:16
fungior whether they ended up in /etc/nodepolo instead22:16
clarkbfungi: marco!22:16
fungi(...ni)22:17
fungiwhich apparently still needs a ptl22:17
*** gothicmindfood has quit IRC22:17
SpamapSlifeless: dump made, usages reset22:17
fungiclarkb: you should run for marconi ptl and spearhead the rename22:17
lifelessunused floating-ips deleting now22:18
*** dims_ has quit IRC22:18
clarkbfungi: ha22:18
*** signed8bit has joined #openstack-infra22:19
fungirun on an all-pasta platform. it would capture 100% of the electorate, no doubt22:19
*** bhuvan has joined #openstack-infra22:20
jogofungi: not devanand22:20
jogodevananda:22:20
clarkbyou can build a computer with devanand gates22:21
*** gothicmindfood has joined #openstack-infra22:21
fungidevanxor gates wouldn't work nearly as well22:21
lifelessfungi: the great flying spaghetti monster for ever?22:21
jeblairclarkb: sNORt!22:21
fungilifeless: may his noodly appendage touch you inappropriately22:21
clarkbok back to code review I go :)22:22
lifelessok this https://jenkins04.openstack.org/job/check-tripleo-ironic-seed-precise/108/ is on a new slave post-broker bring-up22:22
lifelessif its all good it will get a testenv and be away pronto22:22
*** jgrimm has quit IRC22:23
*** bhuvan has quit IRC22:24
*** jaypipes has joined #openstack-infra22:24
*** bhuvan has joined #openstack-infra22:24
*** thuc has joined #openstack-infra22:25
SpamapSmikal: dump sent22:25
*** thuc has quit IRC22:25
mikalTa22:26
*** thuc has joined #openstack-infra22:26
*** signed8bit has quit IRC22:28
*** signed8bit has joined #openstack-infra22:30
*** changbl has quit IRC22:32
lifeless37 test environments registered22:32
lifelesshmm, we probably need to sacrifice a couple hypervisors for more TE hosts22:33
*** mspreitz has joined #openstack-infra22:33
lifelessthough, I suspect the outage may have taken a couple tes down as 37 is a strange number to get from 10 hosts22:33
lifelessor perhaps that was the neutron quota limit being hit22:33
*** esker has quit IRC22:34
*** CaptTofu has quit IRC22:34
*** CaptTofu has joined #openstack-infra22:35
*** eharney has quit IRC22:35
*** dims_ has joined #openstack-infra22:36
clarkbjeblair: https://review.openstack.org/#/c/81320/ lgtm22:36
*** khyati has quit IRC22:36
*** BobBall_ has joined #openstack-infra22:38
*** BobBall has quit IRC22:38
*** prad has quit IRC22:38
*** UtahDave has quit IRC22:39
*** CaptTofu has quit IRC22:39
*** nkinder has quit IRC22:40
*** dstanek has joined #openstack-infra22:40
*** BobBall_ has quit IRC22:41
*** thuc has quit IRC22:42
*** thuc has joined #openstack-infra22:42
*** khyati has joined #openstack-infra22:42
*** thedodd has quit IRC22:43
*** BobBall has joined #openstack-infra22:43
*** BobBall is now known as Guest8679822:44
lifelesshmm none of these jobs seem to be acquiring a testenv succesfully22:46
*** thuc has quit IRC22:46
lifelessI might tear down the te's and bring them back up22:47
*** jepoy has joined #openstack-infra22:48
*** rcleere has quit IRC22:49
asselinanyone seen this error when using nodepool on ubuntu 12.04 image? http://paste.openstack.org/show/74781/22:50
*** markmcclain has quit IRC22:52
*** e0ne has joined #openstack-infra22:52
clarkbzigo: are less css files a problem for debian packaging?22:54
clarkbzigo: I seem to remember that was a thing22:54
*** khyati has quit IRC22:56
clarkbzigo: nevermind found your response to the most recent less css thread22:57
*** e0ne has quit IRC22:57
*** thuc has joined #openstack-infra22:57
*** thuc has quit IRC22:58
*** thuc has joined #openstack-infra22:59
*** saschpe has quit IRC23:00
fungiclarkb: jeblair: lgtm... http://paste.openstack.org/show/74784/23:02
fungiasselin: "you have held broken packages"23:03
fungiasselin: any idea why some packages on your base image might have a dpkg held status?23:03
asselinfungi, yes...but I can't figure out which ones. I'm using the devstack config scripts23:04
fungier, hold status23:04
*** michchap has quit IRC23:05
fungiasselin: dpkg --get-selections|grep hold23:05
fungiasselin: that should list any held packages23:06
asselinnothing at all. I did see this: puppet-common : Depends: facter (< 2.0) but 2.0.1-1puppetlabs1 is to be installed23:06
asselin                 Recommends: debconf-utils but it is not going to be installed23:06
asselinperhaps something with a dependency. puppet depends on puppet-common23:06
asselinwhich ubuntu image does infra use?23:07
fungiasselin: 12.04 lts (precise pangolin)23:07
*** yamahata has quit IRC23:08
asselinfungi, that's what I'm using23:08
asselinprecise-server-cloudimg-amd64-disk1.img23:08
*** wenlock has quit IRC23:09
*** mriedem has joined #openstack-infra23:09
*** saschpe has joined #openstack-infra23:10
fungiasselin: are you using something like http://git.openstack.org/cgit/openstack-infra/config/tree/install_puppet.sh to get puppet installed? (we call that script from prepare_node.sh)23:10
asselinfungi, this is suspicious no?  puppet-common : Depends: facter (< 2.0) but 2.0.1-1puppetlabs1 is to be installed23:10
asselinyes...exactly that script23:10
clarkbfungi: jeblair: jhesketh I wasn't too harsh to https://review.openstack.org/#/c/82307/ was I23:11
clarkbno mordred here right now...23:12
*** gokrokve has quit IRC23:12
jheskethclarkb: It's fair, but I've tidied it up a little down the path: https://review.openstack.org/#/c/84069/23:13
jheskethclarkb: I'd prefer to see it merged and then address the packaging issue so it doesn't block the rest of my changes, but clearly I'm biased23:14
*** aarongr_ has joined #openstack-infra23:14
*** mspreitz has quit IRC23:14
asselinfungi, FYI full log is here: http://paste.openstack.org/show/74785/23:14
clarkbjhesketh: for the packaging thing I don't see any reason to switch to less23:16
clarkbthe current css is dead simple (even I understand it)23:16
clarkbfungi: re Bug 1300843 maybe we can convince people to use real editors that put newlines at the end of files23:18
uvirtbotLaunchpad bug 1300843 in openstack-ci "project-requirements-change gates master" [Medium,Triaged] https://launchpad.net/bugs/130084323:18
clarkb>_>23:18
mattoliverauHey all, while building up Turbo-hipster nodes via nodepool, in my own environment, I use config/install_puppet.sh. As of this morning it is failing due to the pinned version of puppet-common needing to install a facter version of < 2.0.. so I think we will need to add anther pin. If I pin factor to version 1.* it workd s for me.23:18
clarkbmattoliverau: you may have solved asselin's problem23:19
clarkbmattoliverau: if that is the fix that works for me23:19
*** marktraceur has quit IRC23:19
*** david-lyle has quit IRC23:20
fungiasselin: all i can figure is that it's a difference in the base image. we run this (so far) on base images provided by rackspace and hpcloud... maybe wherever your base image came from, it had puppet components pre-installed?23:20
mattoliverauclarkb: well your very welcome :P23:20
clarkbfungi: see mattoliverau's comment23:20
jheskethclarkb: so there's no strong reason to move to less, but with grunt building the system it's pretty easy.. it'll also let us retheme for infra using variables rather than overriding css23:21
*** michchap has joined #openstack-infra23:21
*** _afazekas has quit IRC23:21
asselinfungi, I got my image from ubuntu web-site.23:22
clarkbjhesketh: right its easy until someone tries to package it23:22
clarkb:)23:22
asselinclarkb, mattoliverau how did you result it?23:22
mattoliveraufungi: the puppetlabs repo I asusme have been updated, I started getting the same problem as asselin. I modified add a new pin and trying to install puppet again and it worked.23:23
*** derekh has joined #openstack-infra23:23
jheskethclarkb: sure, I don't really know enough about it, maybe we should get some other opinions23:23
jheskethnevertheless, should we merge it and if we end up not using less we can do that in a future patch?23:23
clarkbjhesketh: many people have chimed in re less and horizon in the past http://lists.openstack.org/pipermail/openstack-dev/2013-December/021584.html23:25
clarkbjhesketh: mostly relying on what zigo has said re debian packaging23:25
fungiclarkb: asselin: mattoliverau: ahh, yes! i'm multi-tasking too many discussions again23:25
mattoliveraulol23:25
clarkbjhesketh: http://lists.openstack.org/pipermail/openstack-dev/2013-December/021791.html is the specific response23:26
fungimattoliverau: i wonder why the nodepool image-update i just did (using a precise base image in rackspace) didn't run into that issue23:26
jeblairjhesketh: i like using css for its intended purpose..23:27
fungiunless this has cropped up just in the last 1.5 hours23:27
jheskethI don't feel strongly about this (except that I wasted a lot of time yesterday making the build stuff work)23:28
*** mspreitz has joined #openstack-infra23:29
jeblairjhesketh: i haven't reviewed that patch, and i think it's going to be difficult for me to review because i don't understand any of those tools.23:29
mattoliveraufungi: I think it has. Cause earlier this morning it was working23:29
clarkbjeblair: that was partially my problem23:29
mattoliverauI've been playing with nodepool images for the last 2.5 hours.23:29
fungimattoliverau: okay, then that explains my confusion nicely23:29
clarkbjeblair: there are literally 10 new files that are config or apply config23:29
lifelessok we're back23:30
clarkbjeblair: to run 2 tests23:30
lifelesshttps://jenkins07.openstack.org/job/check-tripleo-undercloud-precise/373/console23:30
lifelessis running23:30
*** thuc has quit IRC23:30
jeblairjhesketh: my opinion on the matter is i think it should be a simple script that uses some css to say things like "this is a change; this is a job"23:30
jheskethjeblair, clarkb: that's fair... perhaps you could pull in the tail of my changes and see that it works for you23:30
jeblairi think the last time we talked, we were all on board with that much at least23:30
*** mrodden has quit IRC23:30
jeblairi get the desire not to have that limit us in the future23:31
jheskethjeblair: as in you don't want to use all of these build utils?23:31
jeblaira possible future where this is the start of a real app23:31
jeblairclearly that's not going to cut it at that point23:31
jeblairbut i'm confused as to why it's necessary now -- does it help get us to the point where we want to be right now?23:32
*** mwagner_afk has joined #openstack-infra23:32
jeblair(which is that we stop having two versions of the status page?)23:32
jheskethit helps with jshint and ensuring the code is clean23:32
jheskethbut you're right, it's not necessary23:32
asselinmattoliverau, clarkb how did you change the pin factor? I'd like to try it out23:33
clarkbjhesketh: I would be on board with the build tools if there was more documentation23:34
mattoliverauasselin: take a look in /etc/apt/preferences.d/23:34
*** marktraceur has joined #openstack-infra23:35
mgagneWith pbr, is there a way to derive the version from unsigned tags instead of GPG-signed tag?23:35
clarkbjhesketh: as is there is very little of what is going on there explained and several scripts are not used at all23:35
clarkbmgagne: I think pbr requires signed tags23:35
mattoliverauasselin: if you used the install_puppet.sh script there will be a 00-puppet.pref file23:35
asselinmattoliverau, thanks I see the file23:35
mattoliverauI added a new pin and the end of that file:23:35
clarkbmattoliverau: want to propose the change to gerrit?23:36
mattoliverauPackage: facter23:36
mattoliverauPin: version 1.*23:36
mattoliverauPin-Priority: 50123:36
mattoliverauclarkb: I sure can, I was just waiting on fungi to see if he can confirm it.. but I'll do it now anyway :)23:37
*** nkinder has joined #openstack-infra23:37
mgagneclarkb: bummer, signed tags are an esoteric concept to most devs here, I was hoping to have a way to smoothly introduce them to pbr without shocking them23:37
jheskethjeblair: what's your feel re clarkb's request for more docs?23:37
*** lcheng has quit IRC23:37
mattoliverauasselin: ^^23:37
clarkbmgagne: let me double check23:37
fungimattoliverau: yeah, that seems like a good solution23:38
fungimattoliverau: based on what i'm hearing, i expect if i try another image-update i'll see the same error you and asselin are23:39
asselinmattoliverau, that worked for me.23:39
fungimgagne: that's too bad... cryptographically attesting to a release is... important23:39
fungimgagne: especially to packagers and many end users23:40
jeblairjhesketh: i'm trying to be really honest here... i worry that the problem is that this change is about adding ~5 (i'm sure i'm not counting right) new tools that i have no experience with or understanding of their purpose or use23:41
fungimgagne: anyway, i expect it's a baked-in assumption for pbr which would require a patch to change23:41
mgagnefungi: yha, I know. We can argue that it is only used for internal consumptions. Others might add that it doesn't matter and attesting releases of internal tools is as important as public ones.23:41
jeblairjhesketh: and i worry that writing docs about that in order to teach me about those probably isn't the right thing to do23:42
jeblairjhesketh: and also hypothetically if i understood all of them, would i think we got a benefit from them worth the cost of using them23:43
clarkbmgagne: ya, git describe --always doesn't seem to grok the unsigned tags23:43
clarkbmgagne: and that is what pbr is doing23:43
mattoliverauclarkb: http://is.gd/NMrFhP23:43
clarkbmattoliverau: awesome thnaks23:43
mattoliveraufungi: ^^23:43
jeblairjhesketh: i was more inclined to say "let's just let the javascript experts put whatever they want there" a while back, but that's how we ended up with two status pages23:43
asselinmattoliverau, fungi out of curiosity, any reason install_puppet.sh is taken from opnstack-infra where the other scripts are scp to the instance?23:44
mattoliveraujeblair: don't worry, now jhesketh is the javascript expert :P23:44
fungimattoliverau: are you our new openstackgerrit bot? ;) (i'll go find out where it wandered off to)23:44
jheskethmattoliverau: oh goodness no!23:45
mgagneclarkb: As you might already know from previous conversations, we are using Jenkins to tag and perform most tasks which would normally be done by a human release manager. It partially explains why our tags aren't signed yet.23:45
mattoliveraufungi: I could be.... but I don't think I'm as efficient :P23:45
jeblairmattoliverau: until he realizes he doesn't actually like javascript.  then the status page is maintanerless again...23:45
fungimattoliverau: that script is also used in other contexts, mainly23:45
*** thuc has joined #openstack-infra23:45
jheskethjeblair: this was meant to be an afternoon of adding zuul results to gerrit.. .it's now taken weeks23:45
jheskethI already know I don't like javascript23:45
clarkbjhesketh: :/23:45
clarkbjhesketh: sorry, I did like the results of what I saw last I fetched it locally23:46
clarkbfwiw23:46
mattoliveraujhesketh: well thanks for taking one for the team :)23:46
jheskethwell it's not terrible, I just miss python23:46
jheskethtanks clarkb :-)23:46
jhesketh*thanks23:46
fungijhesketh: i'm sure it misses you too23:46
*** dstanek has quit IRC23:46
jheskethfungi: it's a sad time for our relationship, but we'll make it23:46
jeblairjhesketh: yeah, so i worry that if we get to the point where the people hacking on zuul like working on the ui even less than we do now, we'll eventually end up in a worse place than we are now23:47
jheskethyeah those are fair concerns23:47
fungimattoliverau: once that merges, i'll try spinning up another image to double-check23:47
mattoliveraufungi: me too in my test environment :)23:47
jheskethjeblair: so I can rearrange my changes to before all of this refactoring, but I don't really want to do so if we'll eventually refactor23:48
jheskethwho's likely to know enough about the issues etc?23:48
jheskethor somebody that might care23:48
jeblairjhesketh: i hate to impede your work, but all things being equal, i'd probably ask mordred to make a case for us all learning 5 new tools in a language we hate and reading a 1300 line patch23:48
jeblairjhesketh: we probably should get mordred and krotscheck to help us evaluate it23:49
jheskethjeblair: no worries, I'd rather slow down than do the wrong thing for the project/long term23:49
jeblairkrotscheck is the expert, mordred is the proponent, and we know the roadmap23:49
*** mspreitz has quit IRC23:49
krotscheckI’m not seeing a patch in the scrollback, is there something I should be looking at?23:50
jeblairkrotscheck: we're talking about https://review.openstack.org/#/c/82307/423:50
fungiis this a potential inflection point for spinning the zuul status ui into a separate project and growing more api separation between it and zuul?23:50
krotscheckOh, that one.23:50
*** hashar has quit IRC23:50
jeblairfungi: i think zuul's api needs to be something other than read-only before that's truly useful23:51
krotscheckjeblair: A language you hate? ;)23:51
jeblairkrotscheck: hate may be strong.  i'll own to dislike though.  :)23:51
fungijeblair: granted23:51
*** e0ne has joined #openstack-infra23:52
jeblairkrotscheck, jhesketh: anyway, i don't want to burn cycles on this now, i think we need mordred around to figure this out23:53
jheskethfungi: that is something I'd like to see long term.. I don't like zuul having a wsgi app23:53
krotscheckWow, that patch got a lot bigger.23:53
jheskethyep +123:53
jheskethkrotscheck: then I've gone and moved it around and fixed it : https://review.openstack.org/#/c/84069/23:53
fungijeblair: just basically wondering whether that's where the ui work should first focus prior to something like mordred's proposed patch, and then once the api is robust enough to support such separation that separate suite of tools is less onerous to the backend server project23:54
jeblairfungi: everything about what you just said makes me thing more and more that the right answer is to ignore everything until we just fix the thing we set out to fix.  :)23:55
*** e0ne has quit IRC23:55
*** mrodden has joined #openstack-infra23:56
fungijeblair: yeah, i basically feel the same23:56
*** flaper87 is now known as flaper87|afk23:57
krotscheckfungi, jeblair: And maybe by then the node people will have their shit in order and have a recent set of packages available.23:58
jeblairkrotscheck: anything's possible! :)23:59
jeblairi have to run for a bit23:59
*** aarongr_ has quit IRC23:59
*** mspreitz has joined #openstack-infra23:59

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