Tuesday, 2013-12-24

*** dcramer_ has quit IRC00:00
*** SumitNaiksatam has joined #openstack-infra00:13
*** sarob has quit IRC00:19
*** matsuhashi has joined #openstack-infra00:19
*** sarob has joined #openstack-infra00:20
*** MarkAtwood has quit IRC00:26
*** pcrews has joined #openstack-infra00:33
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Zuul layout: start using templates  https://review.openstack.org/6385400:36
*** pcrews has quit IRC00:38
cody-somervilleAre projects not allowed to have a stricter requirements file than openstack-infra/requirements?00:44
cody-somervilleex. require a higher version of a python client library than global-requirements00:45
*** cyeoh has joined #openstack-infra00:47
*** nosnos has joined #openstack-infra00:51
*** hogepodge has quit IRC00:56
*** praneshp has joined #openstack-infra01:00
clarkbno01:04
clarkbthey must match01:04
*** harlowja_ has quit IRC01:09
*** krotscheck has joined #openstack-infra01:10
*** ljjjustin has joined #openstack-infra01:16
*** dkliban has quit IRC01:21
*** yamahata has joined #openstack-infra01:24
*** sarob has quit IRC01:27
*** senk has joined #openstack-infra01:29
* StevenK peers at the error in http://logs.openstack.org/99/59699/5/check/gate-glance-pep8/5d6edaa/console.html01:31
StevenKWhy would it try and reach pypi? I thought we had a mirror.01:31
cody-somervilleclarkb: So if a core project needs a bug fix in a client available in a newer version, they can't require it?01:33
*** jhesketh has joined #openstack-infra01:38
clarkbthey can but must bump the requirement first01:38
clarkbStevenK: it shouldn't, according to the log it seems to clearly have updated the settings at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/jenkins/files/slave_scripts/select-mirror.sh#n3601:41
clarkbStevenK: pretty sure that is mysql-python being "smart"01:42
*** ljjjustin has quit IRC01:42
cody-somervilleclarkb: won't that bump it for everyone else?01:43
clarkbcody-somerville: yes01:43
clarkbcody-somerville: that is the idea. we are keeping everyone in sync01:43
cody-somervilleclarkb: so if I update requirements, won't that break everyone else until they update their projects to match?01:45
clarkbonly if they need to propose a change01:45
dstufftThis is mostly a problem because pip doesn't have a real version dependency solver01:46
clarkbnew requirements are used in all integration testing01:46
dstufftso sorry about that01:46
dstufft:[01:46
clarkband we automatically propose changes to update individual repos01:46
clarkbso things don't break, they just get out of sync until someone bothers to review those changes01:46
*** UtahDave has quit IRC01:51
*** matsuhashi has quit IRC01:57
cody-somervilleOk. Thanks for the info. :)01:59
*** matsuhashi has joined #openstack-infra02:00
*** yaguang has joined #openstack-infra02:01
*** guohliu has joined #openstack-infra02:05
*** ladquin has joined #openstack-infra02:13
*** syerrapragada1 has joined #openstack-infra02:13
openstackgerritKen'ichi Ohmichi proposed a change to openstack-infra/devstack-gate: Copy libvirt log file after tempest run  https://review.openstack.org/6189202:15
*** ljjjustin has joined #openstack-infra02:33
*** w__ is now known as olaph02:40
*** ljjjustin has quit IRC02:46
*** alexpilotti has quit IRC02:46
*** harlowja has joined #openstack-infra02:48
*** nati_ueno has joined #openstack-infra02:50
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Collect list of installed packages at end of run  https://review.openstack.org/6355102:54
*** fifieldt has joined #openstack-infra02:56
*** nati_ueno has quit IRC02:56
*** dcramer_ has joined #openstack-infra02:59
*** harlowja has quit IRC03:03
*** harlowja has joined #openstack-infra03:04
*** nati_ueno has joined #openstack-infra03:06
*** rwsu has quit IRC03:06
*** vogxn has joined #openstack-infra03:07
openstackgerritJoe Gordon proposed a change to openstack-infra/config: Explicitly document requirements for 3rd party testing  https://review.openstack.org/6347803:11
*** nati_ueno has quit IRC03:11
*** krotscheck has quit IRC03:12
*** krotscheck has joined #openstack-infra03:14
*** senk has quit IRC03:15
*** dkliban has joined #openstack-infra03:18
*** vkozhukalov has joined #openstack-infra03:27
*** mgagne has joined #openstack-infra03:27
*** coolsvap has joined #openstack-infra03:33
*** dstanek has quit IRC03:40
*** mgagne has quit IRC03:46
*** vogxn has quit IRC03:46
jog0so something is wrong with grenade03:48
jog0http://status.openstack.org/elastic-recheck/03:48
jog0sdague clarkb: assume your both away until new years but ... ^03:48
*** mgagne has joined #openstack-infra03:49
fungii was about to say the same--i see it impacting multiple changes03:51
fungiseems to be impacting devstack exercises prior to upgrade03:52
jog0fungi: yup03:53
jog0just filed a bug https://bugs.launchpad.net/nova/+bug/126382403:53
uvirtbotLaunchpad bug 1263824 in nova "grenade is failing on euca and bundle tests" [Undecided,New]03:53
jog0my guess is either we have an asymetric hole in the gate or this is a dependency issue03:54
fungiwell, we have a huge asymmetric hole in grenade until we go back to gating on exercises or until forward grenade is voting03:56
jog0hmm boto 2.18.0 vs boto 2.21.103:56
jog0http://pypi.openstack.org/openstack/boto/?C=M;O=D03:57
jog0boto 2.2.1 is brand spaking new03:57
jog0boto 2.21.1 is brand spaking new03:57
jog0and euca2ools use boto if I remember coorectly03:58
jog0yup they do03:58
fungiwell, requirements gates on grenade right? i wonder whether we could rule this out by proposing a wip change to cap boto and watching check results03:59
jog0lets do it04:00
jog0it does04:00
jog0I'll push a patch04:00
jog0so we have boto 2.21.0 but there is a 2.21.1 out04:02
*** jerryz has quit IRC04:02
*** dstanek has joined #openstack-infra04:02
fungioh, i wonder if .0 was flat broken04:03
*** krotscheck has quit IRC04:03
jog0http://docs.pythonboto.org/en/latest/releasenotes/v2.21.1.html04:03
clarkbwe just changed project selection for grenade04:03
jog0clarkb: ?04:03
*** dkliban has quit IRC04:03
clarkbbut I don't expect that to cause trouble (it removes stuff doesn't add it)04:03
fungithat crossed my mind04:03
openstackgerritJoe Gordon proposed a change to openstack/requirements: Blacklist boto 2.21.0  https://review.openstack.org/6386304:04
*** krotscheck has joined #openstack-infra04:04
fungihttp://docs.pythonboto.org/en/latest/releasenotes/v2.21.1.html04:05
fungi.0 is 4 days old04:05
fungioh, jog0 beat me to the release notes link04:06
jog0checking logstash to see when the issues started04:07
jog0started at midnight04:07
jog0message:"Failure bundling image bundle.img"04:07
jog0brb relo04:07
jog0hmm the times don't match up .. this may be a red herring04:08
fungiyeah, it doesn't look like 2.21.0 would be at fault... it's been in our mirror since the 20th04:09
fungimost recent success i see was at 2300 utc04:10
StevenKCan someone give me a hint about the failure in http://logs.openstack.org/99/59699/5/check/gate-glance-pep8/5d6edaa/console.html ?04:11
fungiearliest failure i see exhibiting the grenade havana devstack exercises bundle/euca problems started just prior to 2330 utc04:12
fungihttps://jenkins01.openstack.org/job/check-grenade-dsvm/3353/consoleFull04:13
fungiStevenK: looks like mysql-python's setup.py for some hairbrained reason wanted to retrieve a package of distribute from pypi.python.org instead of our pypi.openstack.org mirror04:16
fungiand then encountered some connection issue to it04:16
fungiStevenK: looks like it calls use_setuptools() which wants to run some sort of distribute downloader, which may ignore the easy_install/pip override configs04:18
StevenKfungi: Ah, so they are able to talk to pypi, rather than being firewalled off?04:21
*** mgagne has quit IRC04:22
fungiStevenK: we don't prevent slaves from trying to reach the internet, but even if we did that wouldn't have prevented it from trying and failing to reach upstream pypi there04:22
fungijog0: clarkb: i think we can rule out a dependency change since the window for when it started was 4.5-5 hours ago and the most recent mirror updates were 17 hours ago04:23
* fungi checks devstack image updates04:24
StevenKfungi: Is there a bug for this behavior, or shall I be naughty and recheck no bug?04:24
fungiStevenK: a bug about this would be nice--just a copy of that traceback and a link to your log fileshould be enough to go on, i think04:25
jog0fungi: this didn't start 5 hours ago04:26
jog0err yes it did04:26
jog0UTC math04:26
fungiclarkb: jog0: i don't think nodepool image updates are at fault for the grenade thing either... images were rebuilt a couple hours after it started04:27
StevenKfungi: Under https://launchpad.net/openstack-ci ?04:27
fungiStevenK: yep04:27
* fungi starts looking at timeframes from suspect things merging04:27
fungiwould have been something landing between 2300-2330 utc04:28
StevenKfungi: Bug reported, thanks. I'll comment on the review to recheck against it04:29
fungithanks StevenK!04:31
fungiclarkb: jog0: most recent changes to devstack landed a couple days ago, and devstack-gate last merged a change 2+ hours prior04:32
jog0https://review.openstack.org/#/q/status:merged+branch:stable/havana,n,z04:32
jog0all quite on stable merges04:32
fungii take back what i said about devstack and d-g changes. git logs were showing me the original commit times, not when they merged04:35
fungihttps://review.openstack.org/6364104:35
funginevermind, that merged right after the danger zone04:35
jog0and thats master04:35
clarkbwhat are the times we are talking about?04:35
jog0clarkb: search logstash for "Failure bundling image bundle.img"04:36
clarkbI hvae heard 5 hours ago which is 3pm PST which is not midnight UTC04:36
jog0clarkb: I see midnight lostash time04:36
jog0logstash04:36
clarkbthat is utc04:36
fungiclarkb: last success was 2300, first fail i find was 2329 (utc)04:37
fungithose being job *start* times according to jenkins01 and 0204:37
clarkbso that was well after the d-g change merged04:37
fungiyeah04:37
clarkband reasonably after the devstack change04:37
fungithe last d-g change was 21:0804:38
fungimost recent mirror updates were a dozen hours before it started, ruling out new dependency stuff04:38
funginodepool image updates were well after it started, ruling that out04:39
clarkbcould it be the d-g change? it isn't failing 100% of the time right?04:40
fungi100% of the time, yes04:40
clarkbI am looking at openstack/openstack and nothing looks likely04:40
fungithere was one more d-g change to merge after the project selection one, but it didn't merge until shortly after this04:40
*** dcramer_ has quit IRC04:41
clarkbfungi: and it affected the bits that should avoid grenade04:42
clarkbit is in the if not grenade section04:42
jog0it is boto 2.2104:42
*** krotscheck has quit IRC04:43
jog0on trunk devstack I can reproduce thei ssue04:43
*** nosnos_ has joined #openstack-infra04:44
jog0clarkb fungi: ^04:44
clarkbis this when we all take a shot?04:44
jog0yes04:44
*** krotscheck has joined #openstack-infra04:44
clarkbjog0: I am happy to review a requirements change if you just want to pin it, but am also happy to review a fix to whatever it broke04:44
*** dripton has joined #openstack-infra04:45
jog0clarkb: euca2ools borke04:45
jog0broke04:45
jog0I am not touching that04:45
jog0IMHO we shouldn't gate on it04:45
fungihow is that happening now though? the mirror has had boto 2.21.0 since several days and doesn't have 2.21.1 yet04:47
*** nosnos has quit IRC04:48
jog0fungi: weekend + ?04:48
jog0oh and boto 2.21.x is broken I think04:48
jog0not just 2.21.104:48
fungiso do <2.21 i guess04:48
clarkbfungi: it is getting it from pypi.python.org04:48
fungigah04:49
clarkbfungi: Storing download in cache at /var/cache/pip/https%3A%2F%2Fpypi.python.org%2Fpackages%2Fsource%2Fb%2Fboto%2Fboto-2.21.1.tar.gz04:49
clarkbI have no idea why04:49
fungiwhy are jobs skipping our mirror now?04:49
clarkbno idea (fwiw StevenK's job did not)04:49
jog0https://github.com/eucalyptus/euca2ools hasn't been touched in 17 days04:49
jog0sigh04:50
openstackgerritJoe Gordon proposed a change to openstack/requirements: Blacklist boto 2.21.0  https://review.openstack.org/6386304:50
fungiclarkb: well, StevenK's problem job showed mysql-python calling use_setuptools() in its setup.py which seemed to spawn some hardcoded distribute downloader04:50
fungiwhich very well may ignore pip/easy_install overrides04:51
clarkbya04:51
clarkbfor grenade04:51
clarkbhttp://logs.openstack.org/03/63003/1/gate/gate-grenade-dsvm/95f1aab/logs/devstack-gate-setup-host.txt.gz seems to do the correct thing04:51
clarkbwait04:52
clarkbsudo is derping04:52
clarkbwhat is going on there?04:52
jog0merry Christmas everyone!04:52
jog0https://review.openstack.org/6386504:54
openstackgerritJoe Gordon proposed a change to openstack/requirements: Blacklist boto 2.21.0 and 2.21.1  https://review.openstack.org/6386304:54
jog0now to file a bug with  euca2ools hmm04:55
*** krotscheck has quit IRC04:55
*** jhesketh has quit IRC04:55
*** elasticio has joined #openstack-infra04:57
fungiso what i'm still wondering is what happened between 2300 and 2330 to suddenly start exposing this issue with euca2ools... ubuntu package update?04:57
*** krotscheck has joined #openstack-infra04:57
clarkbboto released today?04:58
clarkb2.21.1 looks like ti is frim today04:58
clarkbfor fixing the mirror stuff how do you edit /etc/hosts if sudo isn't working04:58
jog0maybe another dep was updated today04:58
jog0boto or euca04:58
clarkboh wait, looking in logs it looks like sudo may work afterall? /me tries a thing to clean it up04:59
fungijog0: but the mirrors updated 12 hours prior to this issue starting and hadn't updated since05:00
jog0fungi: I have no idea05:00
*** harlowja is now known as harlowja_away05:00
jog0well lets wwait the requisite hour to see if this actually  fixes it05:00
jog0and then scratch our heads05:01
fungiat least from what i could see (most recent posts were ~17 hours ago, periodics even longer, for all three interpreter versions)05:01
fungibut it's also entirely possible something was lying to me05:01
*** matsuhashi has quit IRC05:02
*** matsuhashi has joined #openstack-infra05:03
jog0if I am right this will pass https://jenkins01.openstack.org/job/check-grenade-dsvm/3389/console05:04
clarkbdoes devstack create a stack group along with the user?05:05
StevenKHm, why does that pull from github?05:05
jog0clarkb: I think so yeah05:07
*** elasticio has quit IRC05:08
fungijog0: https://jenkins02.openstack.org/job/check-grenade-dsvm/3535/console seems to not have worked05:08
jog0fungi: that was a patch to trunk05:08
jog0fungi: it shouldn't work05:08
fungiahh, right. needs to be havana05:08
jog0yeah I forgot that too05:09
openstackgerritClark Boylan proposed a change to openstack-infra/devstack-gate: Fix copying of select-mirror outputs.  https://review.openstack.org/6386905:09
clarkbfungi: jog0 ^ that *should* use the mirror05:09
clarkbwe can confirm when logs are copied off05:09
fungioh, good catch!05:11
jog0fungi: https://jenkins01.openstack.org/job/check-grenade-dsvm/3389/console worked woo05:11
fungiyep, so it did05:12
jog0what launchpad project do I file this against?05:13
jog0opesntack-ci?05:13
fungiclarkb: if we want to review/approve jog0's 63865 asap i'll promote it to the head of the gate05:13
fungijog0: sure, i guess. we don't have one for requirements so wfm05:14
clarkbthe requirements change?05:14
jog0clarkb: yup05:14
clarkbjog0: does 2.21.0 not work?05:14
clarkb2.21.1 is the new one05:14
jog0clarkb: 2.2.1.1 doesn't work05:14
jog0let me confirm 2.21.0 doesn't either05:14
jog0oh wait that is what pip freeze on gate said we ran05:15
fungiahh, right, the havana backport needs a stable branch manager to approve, which i am not05:15
jog02.21.005:15
* fungi could temporarily make himself a stable branch manager to cram this in, if needed05:15
jog0so 2.21.1 doesn't work  and neither does 2.21.005:15
clarkbjog0: ? 2.21.0 has been around for days according to fungi05:16
funginot something i normally feel good doing, but with this being an urgent fix05:16
clarkb2.21.1 came out today05:16
fungiclarkb: the last modified time on .0 in the mirror was the 20th05:16
fungihttp://pypi.openstack.org/openstack/boto/?C=M;O=A05:17
jog0oh wait05:17
jog02.21.0 does work05:17
*** nicedice has quit IRC05:18
fungi*could* be preserved from the download and delayed by days, but even then it would have been added to the mirror at a minimum 12 hours before this problem cropped up (most recent mirror updates)05:18
*** __amotoki__ is now known as amotoki05:18
fungiokay, so we just want to blacklist 2.21.1 then, and presume it was being retrieved from pypi because of what clarkb's https://review.openstack.org/63869 is fixing05:19
openstackgerritJoe Gordon proposed a change to openstack/requirements: Blacklist boto 2.21.1  https://review.openstack.org/6386305:19
clarkbfungi: yup05:20
jog0fungi: ahh now it all makes sense05:20
fungijog0: on havana?05:21
jog0fungi: pushing ow05:21
jog0now05:21
jog0we should blacklist on trunk too I figure05:21
fungiagreed05:22
jog0anyway both patches are up05:22
jog0its time to take that shot and call it a day05:22
fungijog0: awesome05:23
jog0o/05:23
*** coolsvap has quit IRC05:28
fungi#status alert gating is currently wedged by consistent grenade job failures--proposed fix is being confirmed now--eta 06:00 utc05:31
openstackstatusNOTICE: gating is currently wedged by consistent grenade job failures--proposed fix is being confirmed now--eta 06:00 utc05:31
*** ChanServ changes topic to "gating is currently wedged by consistent grenade job failures--proposed fix is being confirmed now--eta 06:00 utc"05:31
*** yfried has quit IRC05:35
fungiokay, so https://jenkins01.openstack.org/job/check-grenade-dsvm/3395/console passed euca exercises but failed bundle exercises? wtf?05:36
clarkbexisting problem?05:36
fungitrying to figure that out now05:39
fungi[ERROR] /opt/stack/old/devstack/exercises/bundle.sh:61 Image ami-00000001 not available within 15 seconds05:39
fungiseems to maybe be bug 117713405:40
uvirtbotLaunchpad bug 1177134 in devstack "[ERROR] /opt/stack/new/devstack/exercises/bundle.sh:61 Image ami-00000001 not available within 15 seconds" [Medium,Triaged] https://launchpad.net/bugs/117713405:40
*** coolsvap has joined #openstack-infra05:41
fungiseems to be known to e-r as of yesterday, but been around much longer05:41
clarkbfungi: http://logs.openstack.org/69/63869/1/check/check-tempest-dsvm-full/d122a6f/logs/devstack-gate-setup-host.txt.gz much cleaner05:53
clarkband grepping for pypi in the console log shows it using our mirror05:53
fungiclarkb: agreed. also that permissions mistake seems to have existed since at least september, possibly longer05:55
clarkbya05:55
fungithe reexec job fail is one known to e-r05:56
fungiand other d-g jobs passed, so approving05:56
clarkbthanks05:56
clarkbis the grenade fix in?05:56
clarkbI am assuming yes because it passed check tests05:57
clarkbor, does using the mirror fix the problem?05:57
fungiusing the mirror fixes it for the moment05:57
clarkbgotcha05:57
fungibut the problem package is about to land on the mirror05:58
fungii think the mirror update periodics start shortlu05:58
StevenKEverything that grenade well, exploded on will be rechecked?05:59
fungiStevenK: not automatically, no. it might get manually rechecked by someone who wants to comb through gerrit results, but that probably won't be me since it's already 2am here06:00
fungii will mention the bug number in the status update here shortly06:01
*** ljjjustin has joined #openstack-infra06:01
clarkbfungi: neither requirements change seems to be in the gate06:02
fungiclarkb: i hadn't approved them06:02
clarkboh I see we are waiting to see if it fixes? wfm06:02
clarkbseems like it failed bundle and euca in grenade testing06:03
fungiwell the havana one seems to not break euca. failed on bundle for what seems to be a separate known bug06:03
clarkbis this a case where we have to fix havana first?06:03
fungihowever, the havana requirements change is testing grizzly->havana and we want to have it fix havana requirements to fix havana->master which isn't tested06:04
fungiso i think it just needs to be approved regardless06:05
*** SumitNaiksatam has quit IRC06:05
fungiit's got probably another half hour on the requirements integration job in check though, so i'm toying with approving without seeing that succeed06:06
fungiopinion?06:06
clarkbfungi: at this point it can't really hurt06:07
clarkbeverything is dying in the gate06:07
fungik06:07
zaroclarkb: hey finally got it up.  https://github.com/jenkinsci/scp-plugin/pulls06:08
fungiclarkb: the havana requirements change is in the gate behind the d-g change06:09
openstackgerritRenat Akhmerov proposed a change to openstack-infra/config: Adding configuration for python-mistralclient project  https://review.openstack.org/6387606:11
fungi#status ok fix for grenade euca/bundle failures is in the gate. changes failing on those issues in the past 7 hours should be rechecked or reverified against bug 126382406:11
openstackstatusNOTICE: fix for grenade euca/bundle failures is in the gate. changes failing on those issues in the past 7 hours should be rechecked or reverified against bug 126382406:11
*** ChanServ changes topic to "Discussion of OpenStack Project Infrastructure | Docs http://ci.openstack.org/ | Bugs https://launchpad.net/openstack-ci | Code https://git.openstack.org/cgit/openstack-infra/"06:11
uvirtbotLaunchpad bug 1263824 in openstack-ci "grenade is failing on euca and bundle tests" [Undecided,New] https://launchpad.net/bugs/126382406:11
*** ladquin has quit IRC06:13
fungii'll try to keep an eye on the two fixes in the gate until they merge, but may not keep my eyes open that long06:14
clarkbzaro: I have left comments06:15
zaroclarkb: looking..06:15
fungibtw, we have http://pypi.openstack.org/openstack/boto/boto-2.21.1.tar.gz as of a few minutes ago (first of the mirror periodic updates just finished)06:17
*** krotscheck has quit IRC06:18
*** yfried has joined #openstack-infra06:21
*** matsuhashi has quit IRC06:35
*** matsuhashi has joined #openstack-infra06:36
fungithe neutronclient grenade failure in the gate right now suggests we may need requirements for havana updated for all projects individually?06:37
*** matsuhashi has quit IRC06:41
zaroclarkb: hey, trying to understand your comment about if else entries06:41
*** dstanek has quit IRC06:41
zaroclarkb: i thought the problem with the copy was due to using the same session.06:42
*** reed has quit IRC06:45
*** matsuhashi has joined #openstack-infra06:45
clarkbzaro: it is06:47
clarkbzaro: the other comment is just pointing out the change is inefficient as is06:47
openstackgerritA change was merged to openstack-infra/devstack-gate: Fix copying of select-mirror outputs.  https://review.openstack.org/6386906:48
clarkbfungi: shouldn't we use the global reqs in devstack, hmm but grenade isn't devstack so maybe?06:48
clarkbfungi: oh I bet those are master changes I bet we now need to update master06:52
zaroclarkb: sorry, your comment is confusing.  so are you just pointing out something or suggesting that i should update the patch?06:53
zaroclarkb: it sounds like the latter.06:53
*** vkozhukalov has quit IRC06:53
clarkbyes you need to update the patch06:53
fungii think maybe the zuul ref for havana reqs isn't being used setting up master grenade jobs06:53
clarkbzaro: if the only file being copied is the console log then two sessions are started06:53
clarkbzaro: it should only start one in that case06:53
* StevenK hands fungi some matchsticks06:53
StevenKBah, I had this scrollbacked06:54
*** simonmcc has quit IRC06:54
zaroclarkb: ohh yeah. i see.  You mean the session in the parent thread is _not_ necessary if only console log is copied?06:55
clarkbyes06:56
zarogotcha.  thanks.06:56
*** bnemec has quit IRC07:02
*** bnemec has joined #openstack-infra07:02
*** simonmcc_ has joined #openstack-infra07:04
*** shardy_afk is now known as shardy07:09
*** cody-somerville has quit IRC07:19
*** sarob has joined #openstack-infra07:28
*** boris-42 has quit IRC07:37
*** coolsvap has quit IRC07:37
openstackgerritRenat Akhmerov proposed a change to openstack-infra/config: Adding configuration for python-mistralclient project  https://review.openstack.org/6387607:38
*** cody-somerville has joined #openstack-infra07:39
*** matsuhashi has quit IRC07:40
*** SergeyLukjanov has joined #openstack-infra07:41
*** matsuhashi has joined #openstack-infra07:41
*** matsuhashi has quit IRC07:47
*** akscram1 is now known as akscram07:48
*** matsuhashi has joined #openstack-infra07:49
enikanorov__dims: hi. apparently lazy umount didn't solve the problem07:50
*** coolsvap has joined #openstack-infra07:54
*** sarob has quit IRC08:12
*** matsuhashi has quit IRC08:16
*** matsuhashi has joined #openstack-infra08:17
*** bogdando has joined #openstack-infra08:20
*** coolsvap has quit IRC08:22
*** vkozhukalov has joined #openstack-infra08:30
*** nati_ueno has joined #openstack-infra08:38
*** sarob has joined #openstack-infra08:43
openstackgerritYair Fried proposed a change to openstack-infra/config: Adds neutron-isolated to Tempest gate as experimental  https://review.openstack.org/6388908:46
*** yassine has joined #openstack-infra08:47
*** dizquierdo has joined #openstack-infra08:49
*** nati_ueno has quit IRC08:50
*** nati_ueno has joined #openstack-infra08:50
*** guohliu has quit IRC09:00
*** yongli is now known as yongli_away09:06
*** nosnos has joined #openstack-infra09:08
*** nosnos_ has quit IRC09:09
*** nati_ueno has quit IRC09:10
openstackgerritRenat Akhmerov proposed a change to openstack-infra/config: Adding configuration for python-mistralclient project  https://review.openstack.org/6387609:11
*** sarob has quit IRC09:15
*** avishayb has joined #openstack-infra09:21
*** yaguang has quit IRC09:29
*** yamahata has quit IRC09:30
*** simonmcc_ has quit IRC09:30
*** simonmcc_ has joined #openstack-infra09:31
*** matsuhashi has quit IRC09:40
*** ruhe has joined #openstack-infra09:52
*** matsuhashi has joined #openstack-infra09:58
*** ljjjustin has quit IRC10:05
*** sarob has joined #openstack-infra10:13
*** avishayb has quit IRC10:16
*** nosnos has quit IRC10:31
*** gilliard has joined #openstack-infra10:39
*** ruhe has quit IRC10:39
openstackgerritVladimir Kozhukalov proposed a change to openstack-infra/devstack-gate: Tiny improvement in paths  https://review.openstack.org/6390210:42
*** sarob_ has joined #openstack-infra10:44
*** sarob has quit IRC10:46
*** yamahata has joined #openstack-infra10:47
*** dpyzhov has joined #openstack-infra10:59
dpyzhovHello. I'm trying to create stable branches in our stackforge/fuel-* repos. All repos are fine. Except fuel-web and fuel-ostf. For some reason I see "Application Error, Server Error, LOCK_FAILURE" message11:02
*** ruhe has joined #openstack-infra11:09
*** boris-42 has joined #openstack-infra11:11
*** avishayb has joined #openstack-infra11:14
*** matsuhashi has quit IRC11:15
*** sarob_ has quit IRC11:16
dpyzhovI've created a bug about our issue: https://bugs.launchpad.net/openstack-ci/+bug/126389911:18
uvirtbotLaunchpad bug 1263899 in openstack-ci "Branch creation on review.openstack.org is broken for some repos" [Undecided,New]11:18
*** michchap_ has quit IRC11:21
*** dpyzhov_ has joined #openstack-infra11:21
*** michchap has joined #openstack-infra11:21
*** dpyzhov has quit IRC11:22
*** dpyzhov_ is now known as dpyzhov11:22
*** heyongli has joined #openstack-infra11:22
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: don't use ec2 devstack exercises for validation  https://review.openstack.org/6390711:24
*** ruhe has quit IRC11:34
*** avishayb has quit IRC11:34
*** fifieldt has quit IRC11:34
*** ruhe has joined #openstack-infra11:36
*** sarob has joined #openstack-infra11:43
*** sarob has quit IRC11:48
*** ruhe has quit IRC12:02
*** ruhe has joined #openstack-infra12:04
chmouelI am getting some reccurent errors with the grenade test12:06
chmouelon different patches (for swift)12:06
sdaguechmouel: a bad boto was released12:06
sdaguehttps://review.openstack.org/63907 is the fix12:07
chmouelsdague: ah yeah that's what i was going to say it had to be with ec2 :)12:07
sdaguehowever it's a d-g fix, and no +2ers on that are awake yet12:07
sdaguethe scrollback looks like they were working on it about 5hrs ago, so unclear when anyone will be awake to handle it12:07
chmouelhumm shame, should we sent an email at least?12:08
dpyzhovCould anyone delete branches from our stackforge repos?12:08
sdaguechmouel: if you like. I'm officially on vacation. :)12:08
chmouelsdague: hey, will do12:08
sdagueI just figured I'd work up that fix12:08
chmouelsdague: thanks12:10
sdagueman, freenode is no fun. I want utf8 nicks12:11
chmouelsdague: by the way you still have your old work email in that review https://review.openstack.org/#/c/63907/12:16
sdagueno, that's my new work email12:16
chmouelsdague: oh okay :) well congrats then12:17
sdagueyeh, I left IBM for Samsung in Nov12:17
sdaguethe week after summit12:17
chmouelcool12:17
*** ruhe has quit IRC12:19
*** ruhe has joined #openstack-infra12:22
*** avishayb has joined #openstack-infra12:27
*** ruhe has quit IRC12:28
*** ruhe has joined #openstack-infra12:30
*** dpyzhov has quit IRC12:31
*** dpyzhov has joined #openstack-infra12:32
*** ruhe has quit IRC12:33
*** dpyzhov has quit IRC12:36
*** ruhe has joined #openstack-infra12:42
*** sarob has joined #openstack-infra12:43
*** sarob has quit IRC12:48
*** dpyzhov has joined #openstack-infra12:53
*** flaper87|afk is now known as flaper8712:55
*** vkozhukalov has quit IRC12:59
*** CaptTofu has joined #openstack-infra12:59
*** rahmu has left #openstack-infra13:00
*** rahmu has joined #openstack-infra13:02
*** dstanek has joined #openstack-infra13:03
yfrieddkranz: ping13:05
*** dstanek has quit IRC13:08
*** tma996 has joined #openstack-infra13:13
openstackgerritVladimir Kozhukalov proposed a change to openstack-infra/devstack-gate: Tiny  improvement in paths  https://review.openstack.org/6390213:15
openstackgerritSergey Lukjanov proposed a change to openstack-infra/zuul: Move tests to the zuul dir  https://review.openstack.org/6392013:15
openstackgerritSergey Lukjanov proposed a change to openstack-infra/zuul: Enable hacking, ignore only failing checks  https://review.openstack.org/6392113:15
openstackgerritSergey Lukjanov proposed a change to openstack-infra/zuul: Fix and enable H2XX (py3k compatibility issues)  https://review.openstack.org/6392213:16
openstackgerritSergey Lukjanov proposed a change to openstack-infra/zuul: Fix and enable H4XX (docstrings) and H902 ('not in')  https://review.openstack.org/6392313:16
*** jishaomei has joined #openstack-infra13:17
ArxCruzfungi: jeblair hey, can you help me and jishaomei in one question regarding zuul yaml configuration?13:19
ArxCruzWe would like to know how to configure zuul to not vote13:20
ArxCruzusing this pipeline works? http://paste.openstack.org/show/56650/13:20
ruheArxCruz: would this example help you? https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n12413:27
ArxCruzruhe: so, just add a voting: false in the job ?13:27
SergeyLukjanovArxCruz, it looks correct13:27
SergeyLukjanovArxCruz, which kind of non-voting do you want?13:28
ArxCruzSergeyLukjanov: we just don't want to report back voting +1 or -113:28
SergeyLukjanovArxCruz, so, to post only comment13:29
ArxCruzSergeyLukjanov: here's my entire layout.yaml http://paste.openstack.org/show/56651/13:29
SergeyLukjanovArxCruz, experimental pipeline only posts comments - https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n8313:29
jishaomeiSergeyLukjanov, we want to configure non-voting pipeline to community and only just reporting message to community.13:30
ArxCruzSergeyLukjanov: but is that correct? Use the pipeline experimental ?13:30
SergeyLukjanovArxCruz, I mean that you can configure pipeline in your zuul using experimental pipeline as example13:31
ArxCruzSergeyLukjanov: can you review the yaml i post? http://paste.openstack.org/show/56651/13:33
SergeyLukjanovArxCruz, sure13:33
ArxCruzthanks13:33
ruheSergeyLukjanov, ArxCruz, you also can set voting=false for the job13:34
ArxCruzruhe: got it13:34
SergeyLukjanovruhe, nope, it'll make zuul always comment +113:34
ruheSergeyLukjanov, hmm, i wouldn't expect that. but you know it better :)13:36
*** dizquierdo has quit IRC13:37
ArxCruzSergeyLukjanov: ruhe so, add voting = false make zuul comment +1 always ?13:38
jishaomeiSergeyLukjanov, do we need to configure  gerrit: verified: 0 in zull configure for non-voting pipeline?13:38
SergeyLukjanovruhe, I'm not really sure now, but it seems like if all jobs are non-voting in pipeline than nothing can change mark13:40
SergeyLukjanovruhe, probably it'll be 0 as the default value, but using vote: false doesn't seems good for me13:40
SergeyLukjanovArxCruz, so, you'd like to run several jobs only for comment 'validate db2' for nova and migrate13:40
SergeyLukjanovArxCruz, you have tons of unneeded lines that are sets some jobs to be non-voting13:40
SergeyLukjanovjishaomei, I think yes13:40
ArxCruzSergeyLukjanov: we want to run several jobs and report that in review.openstack.org without voting +1 or -1 just post the result13:41
ArxCruzbtw would be nice have a yaml sample in documentation13:42
gsamfirahey guys. Just out of curiosity. If you remove the "success" and "failure" fields from the pipeline, would that not cause the pipeline to be non voting? (https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/zuul/layout-dev.yaml)13:42
*** sarob has joined #openstack-infra13:43
SergeyLukjanovgsamfira, I think that in this case nothing will be published to gerrit13:44
gsamfiragot it. Thanks :)13:44
SergeyLukjanovbtw I'd like to note that I'm not a zuul guru ;)13:45
ArxCruzSergeyLukjanov: you don't? :)13:45
ArxCruzsorry, you're not? :)13:46
ArxCruzbad english here :D13:46
gsamfirano worries. Neither are we :D.13:46
*** sarob has quit IRC13:47
*** dpyzhov has quit IRC13:47
*** dstanek has joined #openstack-infra13:48
gsamfirabut your input was enough to put me on the right track. I don't want to generate any more bogus reports :|. So thanks loads :)13:48
*** samuelbercovici has joined #openstack-infra13:48
SergeyLukjanov:)13:48
*** mancdaz has quit IRC13:49
samuelbercovicimaybe unrelated to this channel. who can assist with the login credentails for https://www.openstack.org/profile/?13:49
ruheSergeyLukjanov, btw unit test for non-voting jobs has this comment "Test that non-voting jobs don't vote."13:49
SergeyLukjanovruhe, it only checks that non-voting failed job doesn't breaks merge13:52
ArxCruzSergeyLukjanov: so, my yaml is wrong? can you point me where?13:52
*** b3nt_pin has joined #openstack-infra13:54
*** salv-orlando_ has joined #openstack-infra13:54
*** simonmc| has joined #openstack-infra13:55
*** yfried has quit IRC13:56
*** mestery_ has joined #openstack-infra13:56
SergeyLukjanovArxCruz, I think it should looks like - http://paste.openstack.org/show/56659/13:57
*** simonmcc_ has quit IRC13:58
*** yhe_ has joined #openstack-infra13:58
*** jpeeler1 has joined #openstack-infra13:58
*** yhe_ is now known as yongli13:59
*** yassine has quit IRC13:59
*** bknudson has quit IRC13:59
*** jd__` has joined #openstack-infra13:59
SergeyLukjanovArxCruz, I've removed non needed lines and set success/failure conf like in the experimental pipeline14:00
*** yassine has joined #openstack-infra14:01
ArxCruzSergeyLukjanov: I'm your fan! Thanks a lot!!!14:01
fungiwoke up to discover the boto requirements caps didn't merge, so have reinserted them in the gate. sdague: i see your d-g exercises patch now, which also lgtm14:01
SergeyLukjanovArxCruz, np, I just hope that it'll work ok for you14:02
sdaguefungi: honestly, the d-g change is probably the better one14:02
sdaguebecause we really shouldn't be doing ec2 compat in the sanity check14:02
ArxCruzSergeyLukjanov: so, the trigger is only comment-added ?14:02
fungisdague: okay, i'll kill the requirements changes and approve/promote the d-g ceanup14:02
fungicleanup14:02
*** bknudson has joined #openstack-infra14:02
sdaguecool14:03
ArxCruzSergeyLukjanov: I though it should have patchset-created too14:03
SergeyLukjanovfungi, morning!14:03
SergeyLukjanovArxCruz, if you'd like to trigger it for each change14:03
sdagueI'm puttering on my computer, so if something goes wrong, ping me. Though I'm mostly trying to ignore irc :)14:03
ArxCruzSergeyLukjanov: got it14:04
ArxCruzjishaomei: did you get that?14:04
*** milki_ has joined #openstack-infra14:05
*** samuelbercovici has quit IRC14:06
*** heyongli has quit IRC14:06
*** mestery has quit IRC14:06
*** gsamfira has quit IRC14:06
*** jpeeler has quit IRC14:06
*** ilyashakhat_ has quit IRC14:06
*** beagles has quit IRC14:06
*** salv-orlando has quit IRC14:06
*** gema has quit IRC14:06
*** ekarlso has quit IRC14:06
*** StevenK has quit IRC14:06
*** pabelanger has quit IRC14:06
*** jd__ has quit IRC14:06
*** Grishkin has quit IRC14:06
*** milki has quit IRC14:06
jishaomeiArxCruz,SergeyLukjanov: I got it. thanks for your help.14:06
*** jd__` is now known as jd__14:06
*** salv-orlando_ is now known as salv-orlando14:06
jishaomeiSergeyLukjanov. I am your fans too.14:06
fungiSergeyLukjanov: hi!14:10
*** gema has joined #openstack-infra14:10
*** mancdaz has joined #openstack-infra14:10
*** gsamfira has joined #openstack-infra14:10
*** ilyashakhat_ has joined #openstack-infra14:10
*** ekarlso has joined #openstack-infra14:10
*** StevenK has joined #openstack-infra14:10
*** pabelanger has joined #openstack-infra14:10
*** Grishkin has joined #openstack-infra14:10
* fungi is still trying to wake up and caffeinate14:11
*** SergeyLukjanov has quit IRC14:11
*** SergeyLukjanov has joined #openstack-infra14:14
*** dstanek has quit IRC14:14
SergeyLukjanovfungi, o/14:14
SergeyLukjanovfungi, fungi, first catch for check-projects-yaml-alphabetized :) https://review.openstack.org/#/c/63876/114:14
* SergeyLukjanov can't understand why internet connection is too bad :(14:15
fungiSergeyLukjanov: mine too. but it's because i'm visiting family in a very rural place with no cell phone coverage, extremely old copper lines and very limited options for internet service providers14:16
SergeyLukjanovfungi, :)14:17
fungialso, awesome alphabetization catch. seems to have properly identified the culprit entry14:18
fungiif samuelbercovici shows back up asking about foundation profile authentication problems again, the contact info for the site admins is https://wiki.openstack.org/wiki/Website14:20
* SergeyLukjanov adds https://wiki.openstack.org/wiki/Website to notes14:21
*** rossella_s has joined #openstack-infra14:23
*** dstanek has joined #openstack-infra14:23
*** ruhe is now known as ruhe_14:23
*** ruhe_ has quit IRC14:23
fungiArxCruz: jishaomei: did you still have zuul configuration questions, or are you up to speed now?14:23
ArxCruzfungi: thanks to SergeyLukjanov we think we have everything covered :)14:24
fungigreat!14:24
ArxCruzfungi: is possible you restore jishaomei account so he can test? You can review his zuul yaml to ensure everything is correct14:24
*** ruhe has joined #openstack-infra14:26
fungiArxCruz: jishaomei: of course. i've put that ibm db2 test account back in the external testing tools group so it should be able to add -1/+1 vrfy votes again14:26
*** yamahata has quit IRC14:26
ArxCruzjishaomei: please when you have you yaml done, send to fungi so he can review and restore your account :)14:27
*** yamahata has joined #openstack-infra14:27
fungioh, i already did14:27
*** annegentle_ has quit IRC14:27
fungi(sorry, i probably wasn't clear enough when i said it the first time)14:28
fungiwe didn't completely block that db2 account anyway, we had only removed it from the group which has verify voting privileges, but it's back in there now (as of a couple minutes ago)14:29
*** rossella_s has quit IRC14:30
jishaomeifungi, please set our account for non-voting privileges, I am not sure everything is right and I will check it more carefully.14:32
*** russellb is now known as russellb_see_you14:33
*** russellb_see_you is now known as russellb_gonetil14:33
*** russellb_gonetil is now known as russellb_vacatio14:33
*** russellb_vacatio is now known as russellb_gone14:34
fungijishaomei: oh, okay. well it had the ability to leave comments as long as it didn't also try to leave -1/+1 vrfy votes. i'll put it back that way for now14:35
*** ruhe is now known as ruhe_14:35
*** ruhe_ has quit IRC14:36
fungiibm db2 test has been removed from the external testing tools group again, so it won't be able to -1/+1 vrfy until you're ready to try that feature14:36
fungibut it can still leave comments on changes as long as it does not try to vote14:36
*** dstanek has quit IRC14:38
*** dims has quit IRC14:43
*** sarob has joined #openstack-infra14:43
*** ruhe has joined #openstack-infra14:45
*** dizquierdo has joined #openstack-infra14:45
jishaomeifungi, could you help to review the zuul yaml configuration http://paste.openstack.org/show/56659/, if I configure our zuul like that, it would implement non-voting to community but just report message to commutity?14:46
*** sarob has quit IRC14:48
openstackgerritA change was merged to openstack-infra/devstack-gate: don't use ec2 devstack exercises for validation  https://review.openstack.org/6390714:50
fungiif dpyzhov comes back, i've done the branch removals he requested in bug 126389914:50
uvirtbotLaunchpad bug 1263899 in openstack-ci "Branch creation on review.openstack.org is broken for some repos" [Wishlist,Incomplete] https://launchpad.net/bugs/126389914:50
fungijishaomei: i'm taking a look now14:51
sdaguegrenade should be back to normal now14:52
*** SergeyLukjanov has quit IRC14:53
fungisdague: thanks! i'll keep an eye on the gate for a bit and mark that bug resolved if i don't see it continue to be broken14:54
*** dims has joined #openstack-infra14:55
*** SergeyLukjanov has joined #openstack-infra14:56
fungijishaomei: ArxCruz: you have a job name pattern of ^.*devstack-vm.*$ set to apply single-use node behavior, but you have no job names in your layout matching that pattern14:58
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: comparison to stable/grizzly is not numeric  https://review.openstack.org/6393414:59
sdaguefungi: also, just noticed that while watching that log14:59
fungisdague: yep, saw that last night and bookmarked that line in the script so i'd remember to fix it. thanks!15:04
*** dstanek has joined #openstack-infra15:05
*** yfried has joined #openstack-infra15:06
*** yongli has quit IRC15:06
jishaomeifungi, thx.I fix it in http://paste.openstack.org/show/56671/15:06
*** che-arne has quit IRC15:07
*** boris-42 has quit IRC15:10
*** senk has joined #openstack-infra15:12
jishaomei@fungi,Could you please review the http://paste.openstack.org/show/56671/, Thanks so much15:15
*** senk has quit IRC15:15
fungijishaomei: ArxCruz: the only other mistake i see there is the "^[\s\S]*validate db2[\s\S]*$" pattern, since "[\s\S]" (literally, any space or non-space character) is the same as "."15:16
ArxCruzfungi: and you compile the regex in your head... ;)15:17
*** coolsvap has joined #openstack-infra15:20
fungijishaomei: ArxCruz: i'd recommend using "(?i)^\s*check db2\s*$" like we do with the experimental pipeline15:22
fungihttp://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n9015:23
*** yassine has quit IRC15:24
*** yassine has joined #openstack-infra15:24
jishaomei@fungi, thanks for your review, I'll revise comment_filter as "(?i)^\s*check db2\s*$"15:26
*** dstanek has quit IRC15:29
*** senk has joined #openstack-infra15:32
*** dstanek has joined #openstack-infra15:35
*** avishayb has quit IRC15:36
jeblairfungi: agreed about the "check foo" syntax (if it's needed)15:37
jeblairjishaomei: also, note that will _only_ run when you leave a comment of "check db2".  that's probably a good idea while you're testing; i assume when it's working you'll want to run it on all new patchsets.15:37
*** harlowja_away is now known as harlowja15:38
*** samuelbercovici has joined #openstack-infra15:40
jeblairfungi: gracefully restarting zuul (to upgrade to version with the latest template changes).  eta 40 mins.15:41
*** ruhe is now known as ruhe_15:42
*** sarob has joined #openstack-infra15:43
*** samuelbercovici has quit IRC15:44
*** rahmu has quit IRC15:46
*** sarob has quit IRC15:48
*** dstanek has quit IRC15:48
*** rahmu has joined #openstack-infra15:49
*** flaper87 is now known as flaper87|afk15:50
fungijeblair: okay, great--thanks!15:58
*** krotscheck has joined #openstack-infra16:04
*** ruhe has joined #openstack-infra16:07
krotscheckSo, most of the build boxes are running either centos or ubuntu raring, correct?16:07
jeblairkrotscheck: centos (for python2.6 jobs) or precise16:07
krotscheckPrecise. Check.16:07
krotscheckI'm trying to figure out a way to get the javascript toolchain onto a box in a way that doesn't make our eyes bleed.16:09
krotscheckAny suggestions? The big elephant in the room is the lack of offical recent nodejs package.16:09
fungithe primary reasons for centos and ubuntu lts is long-term security support. usually shiny new versions of things run counter to stable, carefully backported fixes16:10
jeblairkrotscheck: i guess you're concerned about unit tests, right?  not so much a production deployment?16:11
krotscheckjeblair: Correct. I don't need a server, I need the commandline script execution to build and test a binary.16:11
krotschecks/binary/distribution/16:11
krotscheckI have a reference project on github, am finishing the readme now.16:11
jeblairkrotscheck: for the production deployment of etherpad, we install it from github;  however i'm not sure anything being _tested_ targets a later version than what's in precise16:12
jeblairon purpose16:12
*** jishaomei has quit IRC16:13
*** dstanek has joined #openstack-infra16:15
krotscheckWell, thankfully I don't need a production server.16:15
krotscheckFor that, apache will do.16:15
krotscheckAlso... I dunno- installing from github directly feels a little bit "Oh hey your history just got rewritten"16:15
jeblairit's not something we like to do.16:16
krotscheckI don't blame you16:16
krotscheckSo the options are "Install from source", "roll our own package", ... or... have a build box spin up a vm?16:16
krotscheckAny other ideas?16:16
jeblairkrotscheck: what project is this for?16:17
ruhewould it be possible to install needed version of nodejs from ppa and restrict tests to ubuntu nodes?16:17
krotscheckruhe: yes, assuming I can get xvfb and firefox.16:18
krotscheckHere's the repo for now- doc updates and bootstrapper on the way: https://github.com/krotscheck/storyboard-webclient16:19
*** ruhe has quit IRC16:20
*** chandankumar has quit IRC16:20
*** dizquierdo has quit IRC16:21
krotscheckUgh- precise doesn't have a recent version of compass- I guess we'll just use LESS instead16:21
krotscheckFewer dependencies++ though16:22
jeblairkrotscheck: install from source (in the test workspace) is probably the best answer for now.  the primary purpose of the unit test nodes is to test openstack, so we can't install a node.js there that isn't in precise.16:24
krotscheckSeems fair. You're referring to the jenkins workspace, yes?16:26
jeblairyeah16:26
krotscheckGot it. Alright, I'll see if I can get a bootstrap script up and running that'll build it and add it to the runtime PATH16:27
krotscheckDo we blow away those workspaces after every build?16:28
jeblairkrotscheck: before every build, but yes16:28
krotscheckkk. Clean working dir.16:28
*** dstanek has quit IRC16:32
*** boris-42 has joined #openstack-infra16:34
*** reed has joined #openstack-infra16:39
*** sarob has joined #openstack-infra16:43
*** harlowja is now known as harlowja_away16:44
*** sarob has quit IRC16:48
*** dstanek has joined #openstack-infra16:51
*** prad has joined #openstack-infra16:53
jeblairwow.  the devstack-gate checkout logic for branches has been broken since late october.16:54
jeblairso most of those 21 tests that we run on devstack-gate changes aren't actually testing anything.  :(16:55
*** dstanek has quit IRC16:55
*** coolsvap has quit IRC16:55
fungithey were working (could tell by the way they were failing) when i first added them :/16:56
jeblairfungi: yeah, but i think that was a long time ago16:57
jeblairi'm going to try to straighten this out, but it's going to take a while16:57
jeblairfungi: note this change is not self-testing: https://review.openstack.org/#/c/63934/  and it's pretty much exactly the sort of change that we have all those tests there for16:58
fungiargh17:00
jeblairfungi: i'm going to start by attempting to write functional tests for the checkout code.17:01
*** alexpilotti has joined #openstack-infra17:06
fungithis may explain why grenade tests on master changes last night were failing even though a stable/havana requirements change ahead of them should have fixed that failure17:08
jeblair:(17:08
*** nicedice has joined #openstack-infra17:09
fungiit seemed like the zuul ref for the havana change wasn't being incorporated into the tests on master, but it was also very late and i was having trouble seeing straight at that point, so i thought i may have just been misunderstanding the checkout code there17:09
*** prad has quit IRC17:09
*** prad has joined #openstack-infra17:10
*** senk has quit IRC17:15
SergeyLukjanovwill we have an infra meeting today?17:20
jeblairSergeyLukjanov: no, next one is next year17:20
SergeyLukjanovjeblair, k thx17:21
*** reed has quit IRC17:21
*** senk has joined #openstack-infra17:22
*** senk has quit IRC17:27
*** nati_ueno has joined #openstack-infra17:28
*** senk has joined #openstack-infra17:28
*** sarob has joined #openstack-infra17:31
*** senk1 has joined #openstack-infra17:31
*** senk has quit IRC17:33
*** sarob_ has joined #openstack-infra17:35
*** sarob has quit IRC17:35
*** yassine has quit IRC17:38
*** dcramer_ has joined #openstack-infra17:42
*** sarob_ has quit IRC17:43
*** krotscheck has quit IRC17:52
*** gsamfira has quit IRC17:59
*** tma996 has quit IRC18:00
*** senk1 has quit IRC18:01
*** ruhe has joined #openstack-infra18:02
*** reed has joined #openstack-infra18:07
*** ruhe has quit IRC18:15
*** nicedice has quit IRC18:16
*** nicedice has joined #openstack-infra18:17
*** nicedice_ has joined #openstack-infra18:19
*** nicedice has quit IRC18:22
*** salv-orlando has quit IRC18:25
clarkbmorning18:28
clarkbalso :( d-g checkout logic18:29
jeblairclarkb: yeah.  i have failing tests now though.18:29
clarkbjeblair: is CRM114 Revealed the book you referred to earlier about crm114 stuff?18:31
jeblairclarkb: yes18:32
*** senk has joined #openstack-infra18:32
clarkbthanks, I am doing my best to understand the crm program today18:33
jeblairclarkb: k.  lemme know if you have questions.18:33
*** senk has quit IRC18:36
*** sarob has joined #openstack-infra18:48
*** dcramer_ has quit IRC18:50
*** sarob has quit IRC18:52
*** senk has joined #openstack-infra19:02
*** gothicmindfood has joined #openstack-infra19:04
*** gothicmindfood has quit IRC19:05
clarkbjeblair: "CRM114 does NOT have a perfect solution for the Halting Problem!" this book is amusing19:06
jeblairclarkb: yeah, i had a lot of fun hacking in it.  :)19:07
*** senk has quit IRC19:07
clarkbthe language is pretty neat too19:07
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Move workspace setup to functions.sh  https://review.openstack.org/6395619:18
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Move workspace setup to functions.sh  https://review.openstack.org/6395619:20
jeblairclarkb, fungi: when you have a minute, i'd love feedback on whether https://review.openstack.org/#/c/63956/2/test-functions.sh hits all the test scenarios we need it to (and that the assertions about final state are correct)19:21
jeblairit's safe to run in a local checkout of devstack-gate.  it mocks out all of the git actions so it's fully simulated.19:21
jeblair(i mostly did that for speed; early versions actually did all the git actions that were possible, but it's really slow)19:22
fungii just spotted it on the zuul status page and started looking at it. i'll let you know19:23
clarkbsure taking a look now. for the crm stuff http://git.openstack.org/cgit/openstack-infra/config/tree/modules/log_processor/files/classify-log.crm#n53 seems to copy a variable that we never use again and should all of the matches use the <nomultiline> option?19:23
clarkbjeblair: ^19:23
jeblairclarkb: yeah, i think that was left over from an earlier version of the script that echoed the line back, but it turns out we didn't need it (the way i ended up writing the worker-filter just expected a number and associated it with the line which it already knew)19:25
jeblairclarkb: probably some of the other vars (:prob: :pr: :result:, maybe even :stats:) don't need to be isolated either19:26
clarkbya, I don't expect a major performance improvement by not copying those things, but I suppose every little bit counts19:26
clarkbjeblair: also, apparently crm114 can run as a daemon to preserve JIT knowledge. Maybe we should look into doing that as well?19:27
jeblairoh, good idea19:27
*** nicedice_ has quit IRC19:32
*** nicedice_ has joined #openstack-infra19:33
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Move workspace setup to functions.sh  https://review.openstack.org/6395619:35
fungijeblair: the list of scenarios on 63956 looks sufficient to me. i'm still looking deeper into the rest of the change19:35
jeblairfungi: cool.  hoping ^ will make it actually run for real.  :)19:35
fungithough i guess i'll switch to the next patchset to continue ;)19:35
jeblairfungi, clarkb: that patchset was a change to the wrap script, not to the testing infrastructure19:36
fungiseems to run clean when i try it myself19:39
clarkbjeblair: https://review.openstack.org/#/c/63956/2/test-functions.sh line 213 why is that wrong? ZUUL_BRANCH is set to master shouldn't it checkout master/ZC instead of stable/havana/ZC?19:40
jeblairokay, that's running.  there shouldn't be any logic changes, so after it's done in an hour, we'll just want to check that it still does what we expect19:40
clarkboh I see because setup project sets up on the old version first19:41
jeblairclarkb: yep; since the intent is to set up the predicted future state of havana branches, it should grab change #2 (the havana change to glance)19:41
clarkblgtm19:42
fungii guess the idea is that we peel back the commented asserts one or more at a time with accompanying fixes which make them pass?19:44
*** ryanpetrello has joined #openstack-infra19:44
jeblairfungi: yep19:45
jeblairoh, it looks like you can += without needing to assign first; so i think i'll do that real quick for all those array assignments19:48
fungijeblair: on 244 (and maybe i'm just asking the inverse of clarkb's question but) shouldn't that be ZUUL_REF='refs/zuul/stable/havana/ZC' there?19:48
*** mancdaz is now known as mancdaz_away19:49
jeblairfungi: i think you're right about that; it should be the info for the change at the bottom of the queue19:49
*** sarob has joined #openstack-infra19:49
fungiokay, that's what it seemed to me as well. good ;)19:49
jeblairthat fixes the 2nd test failure in there (but not the first)19:51
jeblair(nor the 3rd or 4th)19:52
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Move workspace setup to functions.sh  https://review.openstack.org/6395619:53
jeblairfungi, clarkb: ^ changes to test-functions.sh19:53
*** sarob has quit IRC19:53
*** ryanpetrello has quit IRC19:57
*** alexpilotti has quit IRC20:07
*** krtaylor has quit IRC20:07
*** krtaylor has joined #openstack-infra20:08
*** markmcclain has quit IRC20:19
*** alexpilotti has joined #openstack-infra20:34
fungilooks like it's passing its last test now20:41
*** sarob has joined #openstack-infra20:43
clarkbjeblair: can http://git.openstack.org/cgit/openstack-infra/config/tree/modules/log_processor/files/classify-log.crm#n46 use bychunk instead of bychar to potentially go faster20:47
*** sarob has quit IRC20:48
akscramhello, fungi and clarkb21:00
akscramfungi, clarkb: how can I receive an approval on this https://review.openstack.org/#/c/61215/?21:02
clarkbakscram: we need to merge https://review.openstack.org/#/c/63490/ first21:10
clarkbakscram: jeblair hasn't reviewed that change yet21:10
clarkbjeblair: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/log_processor/files/classify-log.crm#n75 why is pR always positive in that case?21:20
jeblairclarkb: the worker is line-oriented (that is, it sends one line, then waits for a response to that line before sending the next)21:36
jeblairclarkb: i tried writing the whole file then reading all the data in, and that was such a marginal improvement (i don't recall the numbers) that i thought this was easier and good enough21:38
clarkbjeblair: gotcha wfm21:38
clarkbjeblair: for the pR always being positive statement. Is that because you give two stats files and pR will be negative for one and positive for the other21:38
jeblairclarkb: so i that's why it's bychar; it may not be the best, but i don't think there's a big gain to be had with a simple change21:38
jeblairclarkb: if you look at the example stats output (in the comments starting on line 66)21:40
jeblairclarkb: each file gets a pR indicating the probability that file matches or doesn't match21:40
jeblairclarkb: in practice the winning pR is always positive (i'm not certain that's absolutely true, but it's what seems to happen)21:41
jeblairclarkb: so we're basically encoding an extra bit of information in the sign field: which file matched21:42
jeblairclarkb: so if it matched the failure class, we return the negative of the pR for the winning (failure) class21:43
*** sarob has joined #openstack-infra21:43
clarkbI see so it may happen that both pRs are negative with FAILURE being the best match which would be negated turning positive flipping the result however that didn't happen in practice21:43
jeblairclarkb: and if it matched the success class, we return the pR as is21:43
jeblairclarkb: right, that seems like a theoretical possibility, but i didn't explore deeply enough to confirm.21:44
clarkbjeblair: I think that is the only case where the sign flipping breaks21:44
jeblairclarkb: oh, i did leave a (0) comment on https://review.openstack.org/#/c/63490/21:45
clarkbjeblair: I see it now thanks21:45
jeblairclarkb: i think i got the name of the column in elasticsearch backwards; it's not really error_pr, it's non_error_pr.21:46
clarkbI feel much more up to speed on the CRM stuff now. Last question, do we think train everything is working because jenkins is preclassifying for us?21:47
*** sarob_ has joined #openstack-infra21:47
clarkbthat gives the corpus quit a bit more info21:47
*** sarob has quit IRC21:47
jeblairclarkb: yeah, there's a huge opportunity to try other systems (and you'll note the book describes a lot of training methods).  but yeah...21:48
jeblairclarkb: the pre-classification as well as the fact that most lines show up in good and bad runs and we're looking for lines that more stongly correlate to one or the other...21:49
clarkbjeblair: for https://review.openstack.org/#/c/63490/1/jeepyb/cmd/manage_projects.py basically move line 177 out of the loop?21:50
jeblairclarkb: that is to say, that we're really interested in the frequency, seems to make this system work well enough -- at least, slightly better than a TOE system on my very small data set.21:50
jeblairclarkb: yes, i think so (line 177)21:51
clarkbI will test that in a bit on review-dev21:51
jeblairclarkb: i think the reason i didn't leave a vote is because i knew at least your current solution was tested; if you have time to give it another pass, that would be cool, or if you just want to merge to fix, i'm okay with that.21:52
*** sarob_ has quit IRC21:52
*** nicedice_ has quit IRC21:52
*** nicedice_ has joined #openstack-infra21:52
clarkbI should have time a little later, I need to make lunch now21:55
fungiclarkb: if you want to review/approve https://review.openstack.org/63204 then you can use puppet agent again and not need to apply by hand21:55
fungimmmlunch21:55
clarkbfungi: I have actually got a small test script that just does group creation and uuid retrieval because using manage-projects directly was too much extra work :)21:56
clarkbbut I will review that change21:56
*** dims has quit IRC21:58
*** sarob has joined #openstack-infra21:59
clarkbfungi: approved22:00
openstackgerritA change was merged to openstack-infra/config: Revert "upgrade review-dev from gerrit ver 2.4.x to ver 2.8"  https://review.openstack.org/6320422:00
*** sarob has quit IRC22:03
jeblairclarkb, fungi: mordred added OVERRIDE_ZUUL_BRANCH to devstack-gate; but I don't see it used anywhere; do you recall it every being set?22:03
jeblair(note, this is not the same thing as clarkb's BRANCH_OVERRIDE jjb var; that just sets ZUUL_BRANCH)22:03
*** nicedice_ has quit IRC22:07
*** nicedice_ has joined #openstack-infra22:07
*** jaypipes has quit IRC22:07
clarkbyes it is for the client testing that hasn't happened yet22:11
*** senk has joined #openstack-infra22:11
*** dims has joined #openstack-infra22:12
jeblairclarkb: i think that may be the same logic we want to use with the devstack-gate branch check jobs22:14
clarkbgood point22:15
*** senk has quit IRC22:15
*** prad has quit IRC22:21
*** reed has quit IRC22:27
*** nicedice_ has quit IRC22:33
*** nicedice_ has joined #openstack-infra22:33
openstackgerritDmitry Pyzhov proposed a change to openstack-infra/config: Allow fuel-core team manage stable branches  https://review.openstack.org/6397022:33
openstackgerritClark Boylan proposed a change to openstack-infra/jeepyb: Use MySQLDB transactions properly.  https://review.openstack.org/6349022:37
clarkbthat was tested on review-dev it seems to work22:37
*** sarob has joined #openstack-infra22:43
*** reed has joined #openstack-infra22:47
SergeyLukjanovguys, is it theoretically possible to add a comment in gerrit to the approved/merged change?22:48
*** sarob has quit IRC22:48
jeblairSergeyLukjanov: yes; if you use the ssh api, you have to give --force-message (or something like that)22:49
SergeyLukjanovjeblair, interesting, I just thinking about how to make 'post' jobs results easily accessible22:50
SergeyLukjanovlike coverage reports and etc.22:50
jeblairSergeyLukjanov: the unfortunate thing is that the post jobs don't know what change they are for.22:50
SergeyLukjanovjeblair, oh, that's the thing I've missed22:51
jeblairSergeyLukjanov: they run based on the ref-updated event; if gerrit creates a merge commit, that information isn't supplied in the change-merged event22:51
SergeyLukjanovjeblair, yup ,got it, thanks22:51
jeblairSergeyLukjanov: so if "change foo" merges, but gerrit makes a merge commit "Merge 'change foo'", "Merge 'change foo'" is what we want to actually use, but we don't know that.22:52
jeblairsorry, i had typed most of that already.   :)22:52
jeblairi think it's possible to patch gerrit to include that information though.22:53
jeblairthen we could switch22:53
SergeyLukjanovjeblair, interesting, I have never look before at gerrit's code22:54
SergeyLukjanovshould try to do it after completing my patches for zuul22:54
SergeyLukjanovjeblair, are you guys planning to take day offs for xmas holidays?22:55
jeblairSergeyLukjanov: i'm planning on taking tomorrow off22:56
SergeyLukjanovjeblair, heh, it seems like it'll OpenStack shutdown if all infra folks will take a week off ;)22:58
openstackgerritJames E. Blair proposed a change to openstack-infra/devstack-gate: Fix workspace setup  https://review.openstack.org/6397222:59
SergeyLukjanovit'll be*22:59
jeblairclarkb, fungi: ^ a fix!22:59
clarkblooking22:59
clarkbSergeyLukjanov: I will take tomorrow and probably friday off22:59
clarkbI need to write me LCA talk which is why I dove into CRM today. I think I grok enough of it to get through much of the talk on thursday \o/23:00
jeblairclarkb: cool!23:00
clarkbjeblair: BRANCH_OVERRIDE is used in a lot of jobs23:02
clarkbjeblair: is that a different override or will they need updating?23:02
clarkber ZUUL_BRANCH23:04
jeblairclarkb: i _think_ that any time zuul_branch is set, we should set override_zuul_branch instead23:04
clarkbok23:04
jeblairclarkb: for non-periodic jobs, at least.23:05
jeblairclarkb: i _think_ the current set of jobs that use a branch override that are not periodic is exactly equal to the set of devstack-gate alternate branch check jobs...23:06
jeblairclarkb: and i'm fairly certain that's the right thing to do for them.23:06
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Use OVERRIDE_ZUUL_BRANCH in alternate-branch jobs  https://review.openstack.org/6397323:09
jeblairthere's that ^23:09
clarkbjeblair: waiting to see how check tests do, but my eyeballs don't see anything wrong23:20
jeblairclarkb: i don't think the alternate-branch jobs will quite do the right thing until the other change lands; once it does we can recheck the change i -2'd this morning to see.23:24
clarkbjeblair: order should be d-g change, config change, then rechecks?23:24
jeblairyep23:24
jeblairclarkb: i'm going to copy the css files from logstash-worker16 to 01-08, then make it so that worker01-04 processes A-D are the only ones running23:28
jeblairclarkb: (that's the same count as now, but 4 per host instead of 2, and on hosts 1-8 since, well, they are more likely to not be deleted than the higher numbers which are currently running)23:29
jeblair(sorry, 1-4 in that last ^)23:29
clarkbok23:30
*** SergeyLukjanov has quit IRC23:31
jeblairclarkb, fungi: okay, puppet is running on all the logstash workers and is up to date; workers 01-04 are running 4 processes each (A-D).23:39
jeblairclarkb, fungi: if the individual hosts become overloaded, stop one or two of those processes on them.  and if the system gets behind overall, start processes on more hosts23:39
clarkbok23:40
*** sarob has joined #openstack-infra23:43
*** boris-42 has quit IRC23:44
clarkbjeblair: I left a comment on the config change23:45
jeblairclarkb: the first of the d-g changes has failed tempest twice with the same error (but in different jobs)23:45
jeblairlooks like there's a bug for it; bug 126301923:46
uvirtbotLaunchpad bug 1263019 in nova "tempest.api.volume.test_volume_transfers.VolumesTransfersTest.test_create_list_delete_volume_transfer failed" [Undecided,New] https://launchpad.net/bugs/126301923:46
*** sarob has quit IRC23:48
*** xchu has joined #openstack-infra23:55
*** nicedice_ has quit IRC23:59

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