Wednesday, 2014-06-18

*** pdmars has quit IRC00:01
*** Sukhdev has quit IRC00:01
sdaguejogo: 115 ftw00:01
jheskethlifeless: nope00:02
*** _nadya_ has joined #openstack-infra00:03
sdagueclarkb: got a moment to reduce ES index load? - https://review.openstack.org/#/c/99333/00:04
clarkbsdague: you are ok with non verbose grenade logs?00:05
sdagueit's in the log file00:05
clarkboh we are writing to both currently?00:05
sdagueyep00:05
clarkbgotcha00:05
sdagueand we're indexing the grenade.sh00:05
sdaguefungi landed that monday00:05
clarkbsdague: what does -oL and -eL do?00:05
sdagueto make sure that we didn't lose anything00:05
clarkbis that stdout and stderr locations?00:06
*** mmaglana has quit IRC00:06
sdagueforces unbuffering00:06
*** gokrokve_ has joined #openstack-infra00:06
sdagueor line buffering to be exact00:06
clarkbthanks, I think its worth a single +2 before approval since we are double accounting without it00:06
clarkband done00:07
*** adalbas has joined #openstack-infra00:07
*** _nadya_ has quit IRC00:07
*** sarob has quit IRC00:08
sdaguehonestly, now that I've got the better outfilter, we can probably get real summary steps from devstack up in the console as well00:08
sdaguelike we're getting from grenade00:08
clarkbnice00:08
*** sarob has joined #openstack-infra00:08
sdaguenotice that in the console/00:08
jesusaurusclarkb: jhesketh: I would really like to get https://review.openstack.org/#/c/97659/ merged but it needs to be rebased against the changes for es0700:08
* fungi +2'd as well after the fact, though he's really not here00:09
jesusaurusif i push up the new patchset are you around to give it a quick review?00:09
*** gokrokve has quit IRC00:09
*** CaptTofu_ has joined #openstack-infra00:09
sdagueclarkb: any thoughts on this - http://logs.openstack.org/33/99333/5/check/check-grenade-dsvm/2df50d6/console.html#_2014-06-17_18_26_01_74600:09
jheskethjesusaurus: yep00:09
clarkbjesusaurus: me too00:10
sdaguebesides the fact that we seem to be abusing -eq all over this script00:10
openstackgerritK Jonathan Harker proposed a change to openstack-infra/config: Move variables in manifests/site.pp to top of file  https://review.openstack.org/9765900:10
jesusaurusjhesketh: clarkb ^^00:10
clarkbsdague: is that a case of variable not being quotes so [ is unbalanced expression?00:11
*** bknudson has quit IRC00:11
*** marun has quit IRC00:11
sdagueclarkb: oh, that could be00:11
*** marun has joined #openstack-infra00:11
*** marun has quit IRC00:12
clarkbjesusaurus: +2 from me I assume jhesketh will approve when he has a sec00:12
sdagueclarkb: if we double up brackets, that protects us as well, right?00:12
jesusaurusclarkb: sdague: yeah, that looks like an unquoted and unset variable error to me00:12
clarkbsdague: maybe?00:13
clarkbsdague: I tend to be more bourne than bash compliant when possible00:13
*** Sukhdev has joined #openstack-infra00:13
*** sarob has quit IRC00:13
jheskethjesusaurus, clarkb: looks good, Approved00:13
*** nati_ueno has quit IRC00:14
clarkbsdague: a quick test indicates it returns true00:14
clarkbsdague: which may be worse00:14
jesusaurusjhesketh: clarkb: thanks!00:14
*** mwagner_lap has joined #openstack-infra00:14
sdagueclarkb: no, the double bracket seems to fix it as well00:17
clarkbsdague: it fixes the error but may let the code fall through00:17
*** Ryan_Lane has quit IRC00:17
clarkb[[ $NOTSET -eq "foo" ]] ; echo $?00:17
sdaguethe vars are still quoted00:17
sdaguethat's not actually the issue00:17
clarkboh ya if you quote then both will work00:17
clarkbits the unquoted behavior that is interesting00:18
sdague[ "$NOTSET" -eq "1" ]00:18
sdagueis actually what we are hitting00:18
sdaguewith $NOTSET not existing00:18
clarkboh yes this is the x thing00:18
sdagueright00:18
clarkb[ "x$NOTSET" -eq "1" ]00:18
sdaguebut in bash you can fix that with double [00:18
clarkber "x1"00:18
sdagueyep00:18
sdaguewhich is much less ugly than the x hack00:19
clarkbya or you can -n "$NOTSET" && do above check'00:19
clarkbbut going bash wfm (most everything isn't bourne compatbile anyways and nibalizer likes to say bash is way more standard than sh because sh comes in so many symlink flavors)00:20
nibalizeromg this ^^00:20
nibalizerash or dash or just /bin/sh and (lol)00:20
nibalizeror like solaris which has a real /bin/sh it just doesn't support read -r00:21
lifelessclarkb: can you please rebiew 96608 - dhellmann specifiaclly wants you and monty to do so00:21
*** Ryan_Lane has joined #openstack-infra00:22
clarkblifeless: I am not sure I grok the collision problem00:23
clarkblifeless: N is always increasing00:23
clarkband your next tag would be eg 1.3.300:23
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety  https://review.openstack.org/10072800:23
clarkblifeless: is the issue when you rewrite proposed history and N colides?00:23
sdagueclarkb: also, we explicitly are bash already :)00:24
sdagueanyway, that patch should fix it00:24
lifelessclarkb: no, its because it floats above the last tag00:24
clarkblifeless: but it increases monotonically00:25
lifelessclarkb: say you have 1.2.0 tagged in trunk, and you have a new feature so semver says next version is 1.3.000:25
sdagueok, I'm out for the evening. See folks in the morning.00:25
lifelessso your dev builds are 1.3.0.devN00:25
lifelesswhere N == distance to 1.2.000:25
lifelessnow, if you tag 1.2.1 in trunk (there's a few ways, but reachable-from-HEAD is the constraint)00:25
lifelessyour next commit will get 1.3.0.dev1 because its now distance to 1.2.100:26
clarkbI see. So this is prevent human screw up type stuff00:26
clarkbwfm00:26
openstackgerritA change was merged to openstack-infra/config: Move variables in manifests/site.pp to top of file  https://review.openstack.org/9765900:26
clarkb(you shouldn't tag a 1.2.1 in that case ever, but that doesn't stop people)00:26
lifelessthe way I expect a human will mess itup is00:26
lifelessthey make a branch00:26
lifelesstag 1.2.1 in that00:27
lifelessand then merge that into trunk as a forward port of the bugfix.00:27
clarkbroger00:27
lifelessthat will put 1.2.1 in 'reachable from HEAD' status00:27
lifelessthis seems reasonable until you look at the detail, which is why I want to call it out.00:27
*** lcheng has quit IRC00:28
clarkblifeless: is line 235 correct? I assume rpm weirdness?00:35
lifelessclarkb: yes, covered higher up00:36
lifelessclarkb: rpm has no sort-before primitive00:37
clarkbugh :/00:37
lifelessee line 9700:37
clarkblifeless: and done00:39
lifelessclarkb: thanks00:39
*** CaptTofu_ has quit IRC00:39
lifelessclarkb: the other thing I wanted to ask for help wit was tchaypo's new job patch00:39
morganfainbergsdague, ping re: token scrubbing (if you have a moment) - can wait till later if you're swamped.00:40
morganfainbergsdague, that is token log scrubbing.00:40
lifelesstchaypo: ^ # ?00:40
openstackgerritA change was merged to openstack/requirements: Update skip pep8 clause to use hacking  https://review.openstack.org/10070200:40
tchaypohttps://review.openstack.org/10007600:41
openstackgerritOpenStack Proposal Bot proposed a change to openstack-dev/hacking: Updated from global requirements  https://review.openstack.org/10068000:41
tchaypohttps://review.openstack.org/100078 is related00:41
*** bhuvan has quit IRC00:41
clarkbis a placeholder valuable?00:41
clarkbor is this more of a WIP?00:41
morganfainbergsdague, but there is compliance issues with SHA1 (can't use hashing), so i'll be proposing a fix to use a unique id that will be embeded in the token unless the unique id doesn't exist, then i'll just redact (i'll fix novaclient as well to match)00:41
lifelessclarkb: we can iterate very fast once the infra job is in place00:41
clarkblifeless: I don't grok how it is a placeholder then00:41
*** yamahata has quit IRC00:41
lifelessneither do I00:42
lifelessI panned the commit message last time00:42
clarkbit looks like copy pasta00:42
lifelessbut I don't think tchaypo and I properly connected. This is the full final thing until we're ready to vote it.00:42
lifelessline 225 is the key thing00:42
clarkbit sets the TRIPLEO_TEST differently00:42
tchaypofixened00:42
*** trinaths has joined #openstack-infra00:42
clarkbwhich will not be available in the gate hook00:42
clarkbunless d-g passes that into localrc00:43
clarkblooks like other jobs use it so I assume it is a thing00:43
lifelessclarkb: hmm? I confused; this is how we differentiate jobs in tripleo-ci currently00:43
lifelessclarkb: see toci_devtest.sh00:43
clarkblifeless: ya I think it must be working00:44
*** [1]trinaths has joined #openstack-infra00:44
clarkbanyways I think I grok now. It isn't really a placeholder as much as allowing you to iterate on your end00:44
tchaypoI've removed WIP from the name00:45
*** melwitt has quit IRC00:46
*** trinaths has quit IRC00:47
*** [1]trinaths is now known as trinaths00:47
clarkbtchaypo: can you update the commit to not say placeholder and maybe describe the intent?00:47
lifelesstchaypo: it adds a vlan test job for tripleo, pure and simple.00:48
lifelesstchaypo: its experimental because all new jobs are.00:48
tchaypolifeless: bug number for your spec?00:48
lifelesstchaypo: this is perhaps a cultural thing about infra / openstack CI stuff you didn't know :)00:48
tchaypowhich i imagine I should link to?00:48
lifelesstchaypo: nope, no need.00:48
lifelesstchaypo: 'VLAN configurations are not yet tested. This new job will exercise them.'00:49
lifelessfin.00:49
tchaypoYeah, I imagined having something much more polished before it was ready to land00:49
lifelesswe'll have much more polish, in the tripleo actual code, before its ready to *vote*.00:49
*** CaptTofu_ has joined #openstack-infra00:50
jogosdague: 116 woot00:51
*** ramashri has quit IRC00:52
*** wenlock_ has joined #openstack-infra00:53
*** otter768 has joined #openstack-infra00:54
tchaypoclarkb: is the commit ready now?00:57
*** rwsu has quit IRC00:58
clarkbtchaypo: this change https://review.openstack.org/#/c/100076/00:58
openstackgerritJames Polley proposed a change to openstack-infra/config: Add zuul definitions for TripleO VLAN check job  https://review.openstack.org/10007600:59
tchaypo*ahem*00:59
*** yaguang has joined #openstack-infra01:01
clarkbtchaypo: thanks01:01
*** trinaths has quit IRC01:03
openstackgerritA change was merged to openstack-infra/devstack-gate: display grenade summary instead of all of it  https://review.openstack.org/9933301:05
*** sweston has quit IRC01:06
*** marcoemorais has quit IRC01:06
*** prad_ has quit IRC01:09
morganfainbergclarkb, if i want to capture an extra log into the logs directory (we're getting one of the logs but not both) it's just adding it to the list in the jenkins_job_builder right?01:09
clarkbmorganfainberg: it depends on the test01:09
morganfainbergclarkb, apache based keystone that is.01:09
morganfainbergclarkb, tempest01:10
clarkbyou need to have d-g copy the log into the logs/* dir then it gets copied automagically01:10
clarkbone sec I will link you to the function01:11
morganfainbergclarkb, ah cool. tyvm01:11
clarkbhttps://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n427 that crazy thing01:11
morganfainbergclarkb, thanks! i'll make sure i get the extra log i need captured. it's rare it's really needed but apache squashes some bugs in ways the error log wont ever see it.01:12
clarkbmorganfainberg: and there should be a section in that function for the horizon logs01:13
clarkbwhich would be similar to what you need I think01:13
morganfainbergaye.01:13
morganfainbergsilly extra question for you thne... we do post-formatting on logs, oslocfg format and apache01:13
morganfainbergwhat if a log could be ... either?01:13
clarkbit shouldn't be01:14
morganfainbergclarkb, the keystone screen log could be apache format or oslo01:14
clarkbthe biggest thing about that is where the timestamp and severity are located and you shouldn't mix that right?01:14
clarkbmorganfainberg: hrm I would do apache-key.txt instead or something along those lines01:14
morganfainbergclarkb, hm..... ok i'll poke around and make it more friendly while i'm mucking around in here01:15
morganfainbergclarkb, thanks again :)01:15
mattoliverauclarkb, mordred, fungi: I was trying to help people last night, and deleting stackforge branches came up. Is that something someone with gerrit root access has to do, or is it an ACL problem. I thought the former because in searching I found people raising bugs to infra for branches to be removed. So am assuming a member of a repo's/project's core team can usually create a branch or tag (dpeending on01:15
mattoliverauthe repo config). But can't delete them. (just want to add it to my own brain database so if asked again while you all sleeping) :)01:16
clarkbmattoliverau: it is partially both01:17
morganfainbergclarkb, it's looking like we're saving ~5+ minutes on a tempest run w/ apache keystone woo :)01:17
fungimattoliverau: i was talking to zaro about it the other day, and i believe they need create permission on refs/* within their project to be able to push an empty ref to a branch (which is how you delete a remote branch via git push) or to light the delete button in the webui01:17
clarkbfungi: not quite01:17
fungiit *used* to be that you needed project owner permission01:17
*** chuckC has quit IRC01:17
clarkbfungi: you need force push access to delete branches that way01:17
clarkbfungi: whcih we don't give to anyone01:17
*** matsuhashi has joined #openstack-infra01:17
fungioh, push --force is needed for that? ick01:18
clarkbyes01:18
fungiso i guess we still have to delete branches for stackforge projects01:18
clarkbso ti may be that the change you descirbe will work for the web ui though01:18
clarkbthe web ui deletion and git push were decoupled in the past01:18
fungiit's possible. i haven't tested it on review-dev01:18
* clarkb rereads docs01:19
* fungi get back to reassembling life necessities and failing at being /away01:19
fungimovers are supposed to be here in the morning, bringing all the things i did not01:20
clarkbfungi: https://review.openstack.org/Documentation/access-control.html#category_owner01:20
clarkbmattoliverau: ^01:20
mattoliveraufungi, clarkb: lol, nothing is ever black or white is it :P So for the moment, gerrit root users will need to delete branches.. sounds like no one will be given the ACL, but there may be a web ui button that works in the future :)01:20
clarkbmattoliverau: well the web ui button exists today but apparently we have to give people the owner permission on repos for it to work01:21
mattoliverauclarkb: thanks, I'll check out the link :)01:21
clarkbmattoliverau: which may or may not be a good thing01:21
fungioh, interesting, you can add an owner for a ref pattern now?01:22
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety  https://review.openstack.org/10072801:22
clarkbfungi: no01:22
clarkbfungi: I think it is project global01:22
clarkbfungi: which is why they get the ability to change acls and stuff01:22
fungiclarkb: the section you linked to disagrees01:23
fungithe last paragraph starting "Ownership over a particular branch subspace may be delegated by entering a branch pattern..."01:23
clarkbit does?01:23
clarkboh hey01:23
clarkbthats nice01:23
clarkbso yeah I think you add owner = group something-ptl under refs/heads/*01:23
fungiclarkb: though zaro had linked to a forum post implying that jut having create on refs/* worked, at least for making branch deletion work in the webui01:24
fungithough i don't have the link handy01:24
clarkbyeah for creation its different01:25
*** salv-orlando has quit IRC01:25
clarkbI think the underlying complication is that a git push :foo is not fast forward safe01:26
clarkbso require force01:26
*** yamahata has joined #openstack-infra01:26
clarkbgerrit tries to work around this with the owner stuff01:26
fungigot it. so maybe we can do it without owner or push --force via the gerrit webui but not using git directly01:26
*** Ryan_Lane has quit IRC01:26
mattoliveraufungi, clarkb: cool thanks for clearing that up guys :)01:27
fungior failing that, branch-pattern-specific owner (which would still grant an awful lot we probably don't want, albeit to a subspace)01:27
clarkbfungi: ya if create works that would be great01:27
fungimattoliverau: the upshot of this is that the docs may be misleading/incomplete, but testing is required to figure that out01:28
fungimattoliverau: if you really want to figure it out for certain, get gerrit 2.8.5 up and running on a vm and test different acl permutations yourself. i'd be curious of the results01:28
wenlock_clarkb when you get a minute can i pick your brain on that problem i was seeing earlier?  I think we have it isolated to one git repo, but i can't seem to figure out how to clear it up... or how i should proceed (ie; backup/restore, etc).  The git clone works locally on the repo but git fsck shows 1 dangling blob...(could that be it?).01:28
clarkbwenlock_: and dangling blob shouldn't hurt anything01:30
clarkbwenlock_: I wonder if you are running into jgit limits01:30
clarkbwenlock_: did you push an iso into your repo?01:30
mattoliveraufungi: I do have the a infra puppet master in my test environment, so maybe I will whip a gerrit server up :)01:30
clarkb(I wish that this had never actually happeend and that I did not know that this can make gerrit unhappy)01:30
clarkbwenlock_: jgit is also more strict about git repo file permissions01:30
wenlock_clarkb, i don't think so, hmmm but then again, i don't know anything on how to check jgit limits either01:31
clarkbwenlock_: for example the noVNC repo cannot be hosted in gerrit without pain01:31
wenlock_so check perms too01:31
clarkbjesusaurus: ^ do you remember the particulars on that?01:31
*** nosnos has joined #openstack-infra01:31
clarkbwenlock_: ya it had to do with a thing that git accepted but shouldn't have and jgit decided to be strict about it01:31
clarkband old old versions of git do it wrong01:32
fungimattoliverau: gerrit was the first thing i installed locally for my own testing purposes when i began to work on openstack infra a couple years ago. i highly recommend getting familiar with it at a sysadmin level anyway--very enlightening01:32
wenlock_clarkb that would explain why git clone file:// is happy... but not gerrit01:32
clarkbwenlock_: yup01:32
wenlock_clarkb is there an easy way to inspect all these object files to see which ones are not text?01:34
clarkbwenlock_: not that I know of01:35
mattoliveraufungi: cool, thanks for the tip, so far I have zuul and nodepool spitting out Turbo hipster instances. Maybe now its time to play withh gerrit :)01:35
clarkbwenlock_: you might be able to do something with filter branch and find01:35
wenlock_clarkb perms look good01:35
wenlock_there arn't that many commits... leme just browse each one01:35
fungimattoliverau: yeah, we didn't write it (at least not significant portions though i and other regular infra committers have some commits in it), but as a project we rely on it as heavily as some of the more significant things we have written01:37
*** Sukhdev has quit IRC01:37
clarkbwenlock_: also it isn't the perms in your working dir01:37
clarkbwenlock_: it is the perms encoded in git itself01:37
*** chuckC has joined #openstack-infra01:38
wenlock_oooh  hrm01:38
*** weshay has joined #openstack-infra01:38
wenlock_yikes01:38
* wenlock_ scared of scm perms01:38
*** yjiang|2 has joined #openstack-infra01:38
clarkbwenlock_: `git fsck --strict` will catch the perms thing01:39
wenlock_cool!01:39
wenlock_leme try that01:39
clarkbI think01:39
clarkbthe man page seems to indicate it does01:39
*** sweston has joined #openstack-infra01:42
*** CaptTofu_ has quit IRC01:42
*** yjiang5 is now known as yjiang5_away01:43
*** yjiang|2 has quit IRC01:43
jesusaurusclarkb: wenlock_: the novnc jgit issue has to do with zero-padded permissions01:46
jesusaurusa missing leading zero somewhere causes a git warning that turns into a jgit error01:47
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for grenade bug 1331274  https://review.openstack.org/10074301:47
uvirtbotLaunchpad bug 1331274 in grenade "Couldn't find log for * at /opt/stack/new/screen-logs/*" [Undecided,New] https://launchpad.net/bugs/133127401:47
clarkbjesusaurus: thanks01:47
wenlock_jesusaurus ic  my issue is that a git clone on a project causes gerrit thread to eat up 100% cpu on the server01:47
clarkbwenlock_: do you have java melody in your gerrit?01:49
clarkbthat may shed light on what is breaking too01:49
clarkbor just do a thread dump with the jvm01:49
wenlock_clarkb no, don't have that, but will look for that now01:49
clarkbit is a plugin in gerrit 2.8 or you get it when a config flag in openstack gerrit 2.4 fork01:50
clarkbs/when/with/01:50
wenlock_clarkb im using 2.4.4-14-gab7f4c101:50
wenlock_from config project01:51
clarkbthen ya you should just need a config item. to enable it01:51
clarkbwenlock_: check really quickly at revew.foo.com/monitoring01:51
clarkbas we turned it on our servers you may have it running01:51
wenlock_checking01:51
wenlock_clarkb, oh wow!01:52
clarkbyou need to be in the gerrit administrator group to see it01:52
clarkbbut ya that may help01:52
wenlock_thats cool01:52
clarkbwenlock_: particularly in the thread list you can sort by cpu time01:52
clarkband look at stacktraces for the worst offenders01:52
*** praneshp has quit IRC01:53
wenlock_def see something there... just not sure what to make of it yet01:55
*** dims has quit IRC01:58
*** weshay has quit IRC02:01
*** adalbas has quit IRC02:02
openstackgerritMorgan Fainberg proposed a change to openstack-infra/devstack-gate: Use the DEVSTACK_GATE_USE_APACHE flag to disable  https://review.openstack.org/10074402:04
*** homeless has quit IRC02:10
*** otherwiseguy has quit IRC02:11
*** sweston has quit IRC02:12
*** lttrl has quit IRC02:18
*** mestery has joined #openstack-infra02:19
*** annegentle has joined #openstack-infra02:21
*** dims has joined #openstack-infra02:24
*** SumitNaiksatam has joined #openstack-infra02:28
*** dims has quit IRC02:28
*** matsuhashi has quit IRC02:29
*** maxbit_ has joined #openstack-infra02:32
*** matsuhashi has joined #openstack-infra02:32
*** maxbit__ has joined #openstack-infra02:32
*** craigbr has joined #openstack-infra02:33
*** norman has quit IRC02:34
*** jhesketh has quit IRC02:34
*** maxbit has quit IRC02:35
*** maxbit_ has quit IRC02:36
*** Sukhdev has joined #openstack-infra02:38
*** gokrokve_ has quit IRC02:43
*** zhiyan_ is now known as zhiyan02:44
*** mmaglana has joined #openstack-infra02:48
*** harlowja is now known as harlowja_away02:48
*** praneshp has joined #openstack-infra02:50
*** renlt has joined #openstack-infra02:54
*** gokrokve has joined #openstack-infra02:54
*** jhesketh_ has joined #openstack-infra02:55
*** sweston has joined #openstack-infra02:57
*** sarob has joined #openstack-infra02:57
bodepdhow do I debug gearman when it's launched by zuul?02:57
*** maxbit has joined #openstack-infra02:57
jhesketh_bodepd: it depends on your logging configuration... by default it'll log into /var/log/zuul/debug.log02:58
jhesketh_you can also telnet into it and run 'status' or 'workers' etc02:58
jhesketh_(it being gearman.. eg: telnet localhost 4730)02:58
*** praneshp_ has joined #openstack-infra02:59
*** gyee has quit IRC02:59
*** matsuhashi has quit IRC03:00
*** maxbit__ has quit IRC03:01
*** praneshp has quit IRC03:02
*** praneshp_ is now known as praneshp03:02
bodepdthat's good to know03:02
*** CaptTofu_ has joined #openstack-infra03:02
bodepdI can see the logs for gearman, but they don't really tell me very much03:03
bodepdI can see logs spinning when an event occurs, but no build is laucnhed in jenkins.03:03
bodepdI'm trying to poke the in-between bits03:03
lifelessclarkb: fixed up, I hope03:04
clarkblifeless: hrm?03:04
lifelessclarkb: the spec03:05
clarkboh let me see03:05
clarkbyup looks good03:06
*** trinaths has joined #openstack-infra03:11
openstackgerritFei Long Wang proposed a change to openstack/requirements: Make sure msgpack-python version >= 0.4.0  https://review.openstack.org/10075103:11
*** changbl has joined #openstack-infra03:19
*** camunoz has quit IRC03:27
*** nosnos has quit IRC03:27
jhesketh_bodepd: is Jenkins registered as a worker with gearman? you can send 'workers' over telnet to list them03:30
*** sweston has quit IRC03:31
*** sunrenjie6 has joined #openstack-infra03:33
*** fanhe has joined #openstack-infra03:34
*** sweston has joined #openstack-infra03:37
*** sunrenjie6 has quit IRC03:39
*** camunoz has joined #openstack-infra03:40
*** sunrenjie6 has joined #openstack-infra03:40
sarobpost job training-guides-tox-doc-publishdocs is not working03:40
clarkbsarob: have a link?03:41
sarobi have dug through comparing openstack-manuals to training-guides projects03:41
sarobhttps://bugs.launchpad.net/openstack-training-guides/+bug/132956703:41
uvirtbotLaunchpad bug 1329567 in openstack-training-guides "since the repo move, new html is not getting on docs.openstack.org" [Critical,Triaged]03:41
sarobprojects.yaml and layout.yaml seem to be correct03:42
clarkbdo you have a link to a failing job?03:42
clarkbif not I can go hunting03:42
sarobah, right i dont see a jenkins failure03:42
*** CaptTofu_ has quit IRC03:42
sarobjust the content is not getting up to http://docs.openstack.org/training-guides/03:43
clarkbhttp://logs.openstack.org/03/035922110c09a9d5c26088192b034dbe2c234b4c/post/training-guides-tox-doc-publishdocs/e4006d1/console.html#_2014-06-17_18_21_21_01003:44
clarkbthat is the issue03:44
*** sunrenjie6 has quit IRC03:44
sarobrighto03:44
clarkbI think it wants you to dump your output into www/* ?03:45
*** arnaud has quit IRC03:46
sarobyeah, im at a loss with that as well03:46
sarobi need to look at a good operations manual job or install guide03:47
sarobpost job03:47
clarkbwell they are all the same03:47
clarkbI think that means the template is looking for output to end up in www/*03:47
sarobcept theirs dont fail ;)03:47
clarkbwell sure, but they will run the same cp03:48
*** arnaud has joined #openstack-infra03:49
*** matsuhashi has joined #openstack-infra03:53
sarobhow do i find the post job console logs?03:53
clarkbsarob: the url is http://logs.openstack.org/firsttwocharsofsha1/sha103:54
clarkban example of that is in the urb I posted above03:54
sarobfound off https://jenkins07.openstack.org/view/Openstack-manuals/builds03:55
clarkbsarob: that works too but the above method is more exact03:55
clarkbyou don't have to go from jenkins to jenkins looking03:55
sarobhttp://logs.openstack.org/firsttwocharsofsha1/sha1 no work03:55
clarkbsarob: it does work, its how I got you the log above03:55
sarobi cant follow03:56
clarkbhttp://logs.openstack.org/00/0014a989912e7f09f6518160102d46871690156a/post/openstack-manuals-tox-doc-publishdocs/5c5f980/ for example03:56
clarkbyou need to replace firsttwocharsofsha1 and sha1 with actual values03:56
sarobright didnt read it03:57
trinathssweston: hi03:58
*** mestery has quit IRC04:00
sarobdiff is /home/jenkins/workspace/openstack-manuals-tox-doc-publishdocs$ /bin/bash -c cp -a www/* publish-docs/04:01
*** maxbit_ has joined #openstack-infra04:01
*** maxbit_ has quit IRC04:01
sarobto /home/jenkins/workspace/training-guides-tox-doc-publishdocs$ /bin/bash -c cp -a www/* publish-docs/04:01
*** maxbit_ has joined #openstack-infra04:02
*** yfried_ has quit IRC04:02
bodepdjhesketh_: I think I'm close to figuring it out...04:02
*** mestery has joined #openstack-infra04:02
*** maxbit_ has quit IRC04:02
clarkbsarob: right because www/ exists and contains the data to publish04:02
bodepdjhesketh_: I think the jobs may not actually be getting launched b/c things are merging cleanly04:02
bodepds/are/are not/04:02
jhesketh_bodepd: ah, that'd make sense04:03
jhesketh_bodepd: you might want to set up an smtp reporter while you test this so you can see the results that zuul would send to gerrit04:03
jhesketh_(such as merge-failures)04:03
*** arnaud has quit IRC04:03
bodepdI think the info was staring me in the face, but I still went off in the wrong direction04:03
*** arnaud has joined #openstack-infra04:03
*** mestery has quit IRC04:04
jhesketh_heh, it happens04:04
*** arnaud has quit IRC04:04
*** arnaud has joined #openstack-infra04:05
*** maxbit has quit IRC04:05
sarobclarkb so the www source is missing04:05
sarobclarkb cp fails04:05
clarkbthat is what the erorr said04:06
*** arnaud has quit IRC04:06
*** wenlock_ has quit IRC04:06
*** arnaud has joined #openstack-infra04:07
*** nosnos has joined #openstack-infra04:07
sarobclarkb the rest of the build seems exactly same as manuals04:09
sarobclarkb no failures04:09
*** mbacchi has quit IRC04:09
clarkbright04:10
*** jhesketh_ has quit IRC04:10
clarkbso I would make sure that your mvn target or whatever does the build spits out to www/*04:10
sarobill double check04:11
sarobclarkb if you figure out more post here https://bugs.launchpad.net/openstack-training-guides/+bug/132956704:11
uvirtbotLaunchpad bug 1329567 in openstack-training-guides "since the repo move, new html is not getting on docs.openstack.org" [Critical,Triaged]04:11
clarkbsarob: I am about to fall asleep04:11
clarkbbut I am 99% sure your mvn target just needs to output to www/* and it will work04:11
sarobclarkb no prob thanks for help04:11
sarobclarkb will focus there thanks04:11
*** sarob has quit IRC04:11
*** matsuhashi has quit IRC04:12
*** amotoki has joined #openstack-infra04:12
*** sarob has joined #openstack-infra04:12
*** matsuhashi has joined #openstack-infra04:12
craigbrIs there something else I need to do for https://review.openstack.org/99767? I resolved the outstanding issues yesterday and nothing has happened with it04:14
*** sarob has quit IRC04:14
craigbrThis is adding python-monascaclient04:14
clarkbcraigbr: patience? we have been understaffed for a week and a half due to vacations and illness04:15
clarkbwe will get around to things but it will probably take some time04:15
*** matsuhashi has quit IRC04:17
craigbrOk. Thanks.04:17
trinathshi all, I need help on improving my CI setup. it takes around 25 min to download the git codebase. Is there any other work around to reduce this time?04:18
clarkbtrinaths: we cache all of the repos on our test slaves04:18
*** annegentle has quit IRC04:18
*** jgrimm has joined #openstack-infra04:19
*** matsuhashi has joined #openstack-infra04:21
trinathsclarkb: can you kindly help me on 'cache' all of the repos. I think I have all the repositories in /opt/git. but every time at "Setting up the workspace" all the repositories are downloaded.04:21
clarkbtrinaths: then you probably just need to copy /opt/git into the workspace or wherever you are dealing with the git repos04:22
trinathsclarkb, okay. thank you. will check this.04:23
*** sarob has joined #openstack-infra04:23
*** Longgeek has joined #openstack-infra04:26
trinathsclarkb: I observed, that the data from /opt/git is being rsyncd to the /opt/stack/new directory and then its updated from git.openstack.org. its taking time for all the projects specified in devstack-gate.04:27
*** dims has joined #openstack-infra04:27
clarkboh then it is using teh cache04:27
clarkbnext thing I would check is that your caches are up to date04:28
clarkbwhen was the last time they were updated? we aim for once a day04:28
bodepdwould someone mind taking a quick peek at some log exerts from zuul/gearman?04:28
*** sarob has quit IRC04:28
*** sarob has joined #openstack-infra04:28
trinathsclarkb, we manually update them or there exists some self defined cron which updates them daily.04:29
clarkbtrinaths: it shouldn't take 25 minutes to updat per test then04:30
clarkbtrinaths: I suppose it could be geographical thing or other traffic shaping? may help to use a local git mirror?04:30
clarkbbodepd: maybe?04:30
bodepdhttps://gist.github.com/bodepd/7fa5084403e5f569b1b904:30
bodepdclarkb: ^^^ I'm pretty hopelessly lost on understanding what is happening in those logs04:31
clarkbbodepd: which file did that come from?04:31
bodepdgearman-server.log04:31
clarkbI find looking at not the gearman logs to be more useful usually04:31
*** dims has quit IRC04:32
bodepdit corresponds to an event from zuul04:32
*** mmaglana has quit IRC04:32
bodepdyeah, I had a look at the code,and saw that it will print out the job name and such04:32
clarkbbut that logs says submit a job, job is then created, some polling happens then something asks for a new job but there are no jobs04:32
clarkbbodepd: the zuul debug log is probably more informative04:33
*** sarob has quit IRC04:33
bodepdclarkb: let me retrigger an event and capture all three logs together04:33
*** jhesketh_ has joined #openstack-infra04:34
clarkbbodepd: but it looks like a merge job is being run then the merger asks for more work04:35
*** denis_makogon has joined #openstack-infra04:35
bodepdhttps://gist.github.com/bodepd/7fa5084403e5f569b1b904:35
*** matsuhashi has quit IRC04:36
bodepdclarkb: does the merge job get triggered through gearman?04:36
bodepdclarkb: and does that mean I need a job setup in jenkins for it (IIRC, zuul used to just handle this)04:36
*** matsuhashi has joined #openstack-infra04:36
clarkbit does get triggered though gearman. It means you need a zuul merger running somewhere04:37
clarkbwe dno't use jenkins for it we just run the little included daemon04:37
*** Sukhdev has quit IRC04:38
*** otter768 has quit IRC04:38
bodepdmaybe that isn't even running. hold on.04:39
clarkbhttps://gist.github.com/bodepd/7fa5084403e5f569b1b9#file-gistfile1-txt-L120 looks like it is working though04:39
*** matsuhashi has quit IRC04:39
clarkbit says it verified +1 there04:39
*** matsuhashi has joined #openstack-infra04:39
clarkbor rather something added it I guess it could be not your zuul04:39
*** yfried_ has joined #openstack-infra04:39
bodepdzuul-merger was not running04:40
clarkbbodepd: http://ci.openstack.org/zuul/merger.html04:40
bodepdlemme try again04:40
bodepdI don't know how I missed the merger. I tried to read all-da-docs today04:40
swestontrinaths: hello there04:45
*** dims has joined #openstack-infra04:46
*** mmaglana has joined #openstack-infra04:47
*** dims has quit IRC04:51
*** sarob has joined #openstack-infra04:51
sarobClarkb looks like https://review.openstack.org/#/c/100056/ should fix the problem with training guide publish04:53
clarkbsarob: pretty sure that is not the fix04:54
clarkbthat just adds a new job which we don't want04:54
clarkbwe want to be using the templates04:54
clarkband that change doesn't run the job that was added...04:55
sarobRluethi are you online?04:56
sarobI'm a bit spent and not able to debug clearly04:57
clarkbsarob: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/projects.yaml#n32 those are your jobs04:58
clarkbthey are templated so you get the same jobs as all the other manuals04:58
sarobRight04:58
*** rluethi has joined #openstack-infra04:59
sarobRluethi projects.yaml has the jobs04:59
clarkbI left a review on 10005605:00
clarkbpretty sure you just need to make sure that what that jobs runs spits data out in www/*05:00
sarobClarkb thx05:00
sarobRluethi I think the templated manuals job is building a www dir and then copying it05:01
*** mmaglana has quit IRC05:01
sarobRluethi our post job is failing on the www copy05:01
*** talluri has joined #openstack-infra05:03
*** sarob has quit IRC05:04
rluethisarob: seems to me our job is failing before that, when it should create whatever is supposed to be in www05:04
clarkbrluethi: yes your job should put content in www/*05:05
*** sarob has joined #openstack-infra05:05
rluethiwhen I looked into it, I blamed our tox.ini05:06
*** unicell has quit IRC05:06
rluethithe buildlang-* stuff is all disabled.05:07
bodepdclarkb: cool, with that turned on, the loggin is much better, I05:08
bodepdam now at a failure that I can understand05:08
clarkbwoot05:08
bodepdwell, it doesn't work, but it's clear why :)05:09
bodepdmy job from jenkins is not in the gearman job list05:09
*** ildikov has quit IRC05:09
*** sarob has quit IRC05:10
*** matsuhashi has quit IRC05:11
bodepdclarkb: "all known Jenkins jobs as functions that Zuul can invoke"05:11
bodepdclarkb: what exactly does that mean? what indicates that a job can be invoked by Zuul?05:11
*** matsuhashi has joined #openstack-infra05:11
clarkbrluethi: http://logs.openstack.org/03/035922110c09a9d5c26088192b034dbe2c234b4c/post/training-guides-tox-doc-publishdocs/e4006d1/console.html#_2014-06-17_18_16_31_736 is what it is running05:12
clarkbrluethi: not sure why that needs buildlang-*05:12
clarkbbodepd: where is that quote coming from? I feel like I am missing a bunch of context05:13
clarkbbut in generla zuul lays a lightweight protocol on top of the gearman protocol05:13
rluethiclarkb: thank you. I'll look into it.05:13
bodepdclarkb: http://ci.openstack.org/zuul/launchers.html#gearman-jenkins-plugin05:13
clarkbso jobs that are registered and intended to be run by zuul need to follow that. The jenkins gearman plugin does this for you05:13
*** annegentle has joined #openstack-infra05:14
bodepdto follow the protocol. lemme look at those docs...05:14
*** annegentle is now known as Guest2592605:14
clarkbright so it is saying that all of your jenkins jobs will be registered with german05:15
clarkb*gearman05:15
*** matsuhashi has quit IRC05:16
bodepdclarkb: but it's not all is it?05:16
clarkbyes all05:16
bodepdclarkb: they need to have soe special build parameters to be registered?05:16
clarkbno05:16
clarkbzuul will just shove them over the wire for you05:16
bodepdclarkb: ok,then registration must have failed :)05:16
clarkband jenkins does the right thing05:16
clarkbbodepd: yes does zuul status show a bunch of jobs registered?05:16
*** matsuhashi has joined #openstack-infra05:17
bodepdclarkb: no, you mean the main page of zuul?05:18
clarkbno sorry. I meant 'status' against the gearman 4730 tcp port05:18
bodepdoh. sorry.05:18
*** Guest25926 has quit IRC05:19
tchaypoclarkb: i thought you were going away hours ago05:19
bodepdno, the job I expect is not there. It shoudl have been installed by the jenins plugin?05:19
clarkbtchaypo: I was but then the disease kept me awake05:19
tchaypoThat's no way to talk about greghaynes05:19
bodepdor me :)05:20
clarkbbodepd: yes, though did you start the jenkins server after installing the plugin? and or maybe was the gearman server started long after jenkins?05:20
clarkbpretty sure plugin installation requires restart05:20
bodepdclarkb: it was definitely started after jenkins05:20
clarkbyou can trigger a reregister by disabling then enabling that job in jenkins05:20
bodepdclarkb: I can restart jenkins05:20
greghaynestchaypo: hahaha, thanks05:21
clarkbbut it may also just be that it gave up when you gearman service never showed up05:21
clarkbalso firewalls05:21
*** ihrachyshka has joined #openstack-infra05:23
*** wenlock has joined #openstack-infra05:23
*** matsuhashi has quit IRC05:24
*** ihrachyshka has quit IRC05:24
*** matsuhashi has joined #openstack-infra05:24
openstackgerritIan Wienand proposed a change to openstack-infra/nodepool: Grant at least one allocation  https://review.openstack.org/10076505:29
*** sarob has joined #openstack-infra05:29
clarkbianw: I am not sure that "fixes" it in the case where I have 1 node to give you but 6 images with at least one ready to go05:31
clarkbit may work in the nearly constrained case05:32
clarkbbut I will need to think about it more when brain is awak05:32
clarkbe05:32
*** sarob has quit IRC05:33
ianwclarkb: i need to think about it more too :)05:33
clarkbianw: I believe it will attempt to boot say 6 nodes but we have only room for one. So the first one to hit the nova api wins and the other 5 will 500 and the allocation starts again05:34
clarkbianw: a test may be provider1=1, label1=reallybignumber, label2=reallybignumber, label3=205:35
clarkbthat should respond with 1 for each label05:35
ianwclarkb: whereas, you mean that should return (1,0,0) or some variant of05:37
clarkbianw: correct05:37
clarkbianw: because 1,1,1 will fail to boot the second two nodes every time05:37
ianwclarkb: hmm, ok.  for some reason it fails gerrit tox but passed on my local test anyway :(05:38
*** reed has quit IRC05:38
ianwclarkb: i wonder if derekh's random choice is better idea then05:38
ianwsomehow we need to guarantee forward progress05:38
clarkbya so my suggestion is to use a PID controller05:39
clarkbbut that requires maths05:39
greghaynesclarkb: ohai05:39
clarkbhey there05:39
clarkbIt should be possible to tune a PID controller that dones't overcompensate for the heavily favored images05:40
clarkband account for change over time and so on05:40
*** rdopieralski has joined #openstack-infra05:40
clarkbafter a few cycles the I really need small number of x should amplify and become more important than the initial large number of x05:41
*** annegentle has joined #openstack-infra05:44
*** annegentle is now known as Guest6658105:45
ianwclarkb: yeah, i'll have to think on that because i don't understand how it would maintain the current error, etc across requests05:45
ianwclarkb: i'm probably missing it, but i don't see where it's saving past requests, etc?05:46
clarkbianw: those are queued up in the task managers05:46
*** dims has joined #openstack-infra05:47
ianwclarkb: but can nodepool access info about those queue lengths, etc?05:47
clarkbit is python so sure?05:47
*** unicell has joined #openstack-infra05:50
ianwclarkb: oh, sorry, i see.  i was thinking there was an rpc layer in between05:50
*** lukego has joined #openstack-infra05:51
*** e0ne has joined #openstack-infra05:51
*** Guest66581 has quit IRC05:51
*** rcarrillocruz has joined #openstack-infra05:51
*** dims has quit IRC05:51
ianwwell, i know more about nodepool than i did this morning, so i guess i'll call that a success :)05:52
clarkb:)05:52
*** rcarrill` has quit IRC05:53
*** afazekas_ has joined #openstack-infra05:54
*** yjiang|2 has joined #openstack-infra05:55
*** harlowja_at_home has joined #openstack-infra05:56
*** e0ne has quit IRC06:01
*** e0ne has joined #openstack-infra06:01
*** wendar has quit IRC06:04
*** wendar_ has joined #openstack-infra06:04
*** e0ne has quit IRC06:05
*** dragondm has quit IRC06:05
*** dragondm has joined #openstack-infra06:08
*** gokrokve has quit IRC06:11
*** basha has joined #openstack-infra06:17
bashaclarkb: hi06:17
*** doude has joined #openstack-infra06:18
*** lukego has quit IRC06:19
*** denis_makogon has quit IRC06:19
*** pcrews has quit IRC06:19
*** doug-fish has quit IRC06:19
*** jamespd has quit IRC06:19
*** YorikSar has quit IRC06:19
*** openstackgerrit has quit IRC06:19
*** mrodden has quit IRC06:19
*** vladan has quit IRC06:19
*** lukego has joined #openstack-infra06:23
*** denis_makogon has joined #openstack-infra06:23
*** pcrews has joined #openstack-infra06:23
*** doug-fish has joined #openstack-infra06:23
*** jamespd has joined #openstack-infra06:23
*** YorikSar has joined #openstack-infra06:23
*** openstackgerrit has joined #openstack-infra06:23
*** mrodden has joined #openstack-infra06:23
*** vladan has joined #openstack-infra06:23
*** _nadya_ has joined #openstack-infra06:29
*** ildikov has joined #openstack-infra06:29
*** sarob has joined #openstack-infra06:29
*** sarob has quit IRC06:34
*** miqui has quit IRC06:36
*** jcoufal has joined #openstack-infra06:37
*** lcheng has joined #openstack-infra06:42
*** arnaud has quit IRC06:45
*** annegentle has joined #openstack-infra06:46
*** annegentle is now known as Guest4419106:46
mattoliveraunight all06:46
*** dims_ has joined #openstack-infra06:47
*** Guest44191 has quit IRC06:50
*** basha has quit IRC06:51
*** camunoz has quit IRC06:51
*** jhesketh_ has quit IRC06:52
*** wenlock has quit IRC06:53
*** dims_ has quit IRC06:53
*** yolanda has joined #openstack-infra06:55
*** basha has joined #openstack-infra06:56
trinathssweston Hi06:56
*** mrmartin has joined #openstack-infra06:57
*** renlt has quit IRC06:58
*** harlowja_at_home has quit IRC07:00
*** sunrenjie6 has joined #openstack-infra07:00
*** sunrenjie6 has quit IRC07:01
*** sunrenjie6 has joined #openstack-infra07:01
*** gokrokve has joined #openstack-infra07:02
openstackgerritVictor Sergeyev proposed a change to openstack/requirements: Add oslo.db library  https://review.openstack.org/9140707:03
*** sunrenjie6_ has joined #openstack-infra07:05
*** sunrenjie6_ has left #openstack-infra07:06
*** gokrokve has quit IRC07:07
*** matsuhashi has quit IRC07:08
*** matsuhas_ has joined #openstack-infra07:08
*** camunoz has joined #openstack-infra07:09
bashaHi Im having some issues with tests failing on python3. Anybody here?07:10
*** jcoufal has quit IRC07:12
*** renlt has joined #openstack-infra07:13
*** jcoufal has joined #openstack-infra07:13
*** jhesketh has joined #openstack-infra07:13
*** chandan_kumar has quit IRC07:17
*** sunrenjie6 has quit IRC07:17
*** sunrenjie6 has joined #openstack-infra07:18
*** salv-orlando has joined #openstack-infra07:21
*** sunrenjie6 has quit IRC07:22
*** jgallard has joined #openstack-infra07:24
*** doude has quit IRC07:25
renltjhesketh: hello, would you please help to check this if this is a good time for you, https://bugs.launchpad.net/openstack-ci/+bug/1330854 , which is about delete remote branch.07:27
uvirtbotLaunchpad bug 1330854 in openstack-ci "Please help to delete stack/icehouse branch of stackforge/powervc-driver" [Undecided,New]07:27
*** flaper87|afk is now known as flaper8707:33
*** freyes has joined #openstack-infra07:36
*** jlibosva has joined #openstack-infra07:38
*** andreykurilin_ has joined #openstack-infra07:40
*** jgallard has quit IRC07:40
*** jgallard has joined #openstack-infra07:41
*** trinaths has quit IRC07:41
*** yjiang|2 has quit IRC07:42
*** annegentle has joined #openstack-infra07:46
*** annegentle is now known as Guest936907:47
*** e0ne has joined #openstack-infra07:48
*** denis_makogon has quit IRC07:51
*** Guest9369 has quit IRC07:51
*** mrda is now known as mrda-away07:53
*** ihrachyshka has joined #openstack-infra07:54
*** jpich has joined #openstack-infra07:57
*** basha has quit IRC07:57
*** dizquierdo has joined #openstack-infra08:01
*** e0ne has quit IRC08:01
*** hashar has joined #openstack-infra08:01
*** gokrokve has joined #openstack-infra08:03
*** ociuhandu has quit IRC08:03
*** e0ne has joined #openstack-infra08:03
*** jistr has joined #openstack-infra08:03
*** roeyc has joined #openstack-infra08:05
*** yamahata has quit IRC08:05
*** yamahata has joined #openstack-infra08:05
*** renlt has quit IRC08:06
*** renlt has joined #openstack-infra08:06
*** gokrokve has quit IRC08:08
*** praneshp has quit IRC08:09
*** renlt has quit IRC08:10
*** andreaf_ has quit IRC08:11
*** renlt has joined #openstack-infra08:11
*** renlt has quit IRC08:11
*** renlt has joined #openstack-infra08:11
*** doude has joined #openstack-infra08:12
*** amotoki has quit IRC08:14
*** basha has joined #openstack-infra08:16
*** arnaud has joined #openstack-infra08:19
*** Hal_ has joined #openstack-infra08:23
*** renlt has quit IRC08:25
*** renlt has joined #openstack-infra08:26
*** renlt has quit IRC08:27
*** viktors|afk is now known as viktors08:28
*** renlt has joined #openstack-infra08:28
*** zz_johnthetubagu is now known as johnthetubaguy08:29
*** sunrenjie6 has joined #openstack-infra08:29
*** renlt has quit IRC08:32
*** renlt has joined #openstack-infra08:32
*** amotoki has joined #openstack-infra08:32
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Sync name validation regex with API  https://review.openstack.org/10045808:33
*** denis_makogon has joined #openstack-infra08:38
*** IvanBerezovskiy has joined #openstack-infra08:41
*** viktors has left #openstack-infra08:42
*** moted has quit IRC08:45
*** Longgeek has quit IRC08:46
*** Longgeek has joined #openstack-infra08:46
*** jp_at_hp has joined #openstack-infra08:48
*** Longgeek has quit IRC08:48
*** Longgeek has joined #openstack-infra08:49
*** pelix has joined #openstack-infra08:50
*** dims_ has joined #openstack-infra08:51
*** arnaud has quit IRC08:52
*** Longgeek has quit IRC08:53
*** Longgeek_ has joined #openstack-infra08:53
*** dims_ has quit IRC08:56
*** andreaf has joined #openstack-infra08:57
*** andreaf_ has joined #openstack-infra08:58
*** matsuhas_ has quit IRC09:01
*** andreaf has quit IRC09:01
openstackgerritAntoine Musso proposed a change to stackforge/python-jenkins: Speed up job existence tests by fetching less info  https://review.openstack.org/8958909:03
*** gokrokve has joined #openstack-infra09:03
*** Hal_ has quit IRC09:03
*** Hal_ has joined #openstack-infra09:04
openstackgerritAntoine Musso proposed a change to stackforge/python-jenkins: Speed up job existence tests by fetching less info  https://review.openstack.org/8958909:07
openstackgerritAntoine Musso proposed a change to stackforge/python-jenkins: Speed up job existence tests by fetching less info  https://review.openstack.org/8958909:08
*** matsuhashi has joined #openstack-infra09:09
*** markmc has joined #openstack-infra09:09
*** gokrokve has quit IRC09:09
openstackgerritDenis M. proposed a change to openstack-infra/config: Added job for trove functional tests with heat support enabled  https://review.openstack.org/10060109:12
*** isviridov|away is now known as isviridov09:13
openstackgerritDenis M. proposed a change to openstack-infra/config: Added job for trove functional tests with heat support enabled  https://review.openstack.org/10060109:19
e0nehi all! i've got a question about how infra works. there is a script to cache openstack repos to slave nodes https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/nodepool/scripts/cache_git_repos.py. And there is one more script to clone repose for devstack gate: https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L149. Why doesn't it uses cashes repos? and for what do we use cached rep09:19
e0neos?09:19
*** mkerrin has quit IRC09:22
*** ilyashakhat has joined #openstack-infra09:27
openstackgerritAntoine Musso proposed a change to stackforge/python-jenkins: Adds assert_node_exists()  https://review.openstack.org/10083509:28
*** talluri has quit IRC09:28
*** talluri has joined #openstack-infra09:30
openstackgerritA change was merged to stackforge/python-jenkins: Fix remaining PEP8 errors in docstrings  https://review.openstack.org/9820409:32
*** sunrenjie6 has quit IRC09:35
vponomaryovSergeyLukjanov: ping09:35
*** sunrenjie6 has joined #openstack-infra09:35
*** matsuhashi has quit IRC09:38
*** sunrenjie6 has quit IRC09:39
hashare0ne: there is not many people from infra present during the european mornings09:40
e0neyep:(. i'll try to ask again in UTC morning09:40
e0nes/UTC/PST09:40
hashare0ne: or you could ask on the openstack-infra mailing list09:42
*** talluri has quit IRC09:42
*** mkoderer has quit IRC09:42
openstackgerritDenis M. proposed a change to openstack-infra/config: Added job for trove functional tests with heat support enabled  https://review.openstack.org/10060109:44
e0nehashar: openstack-infra mailing list is not very active. maybe it will be useful to send it to openstack-dev too.09:44
hasharmaybe :]09:44
*** andreykurilin_ has quit IRC09:47
*** renlt has quit IRC09:48
*** zhiyan is now known as zhiyan_09:51
*** dims_ has joined #openstack-infra09:51
openstackgerritFlorian Preinstorfer proposed a change to openstack-infra/jenkins-job-builder: Fix a broken include in the configuration documentation.  https://review.openstack.org/10083809:55
*** dims_ has quit IRC09:56
*** ihrachyshka has quit IRC09:56
*** jcoufal has quit IRC09:59
*** ociuhandu has joined #openstack-infra10:01
*** matsuhashi has joined #openstack-infra10:01
*** gokrokve has joined #openstack-infra10:04
*** matsuhashi has quit IRC10:06
*** matsuhashi has joined #openstack-infra10:08
*** _nadya_ has quit IRC10:09
*** gokrokve has quit IRC10:10
*** Hal_ has quit IRC10:11
*** fanhe has quit IRC10:12
*** rlandy has joined #openstack-infra10:13
*** talluri has joined #openstack-infra10:13
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: Add fingerprint for grenade bug 1331274  https://review.openstack.org/10074310:14
uvirtbotLaunchpad bug 1331274 in grenade "Couldn't find log for * at /opt/stack/new/screen-logs/*" [Undecided,New] https://launchpad.net/bugs/133127410:14
*** freyes has quit IRC10:15
*** yaguang has quit IRC10:16
*** matsuhashi has quit IRC10:17
*** matsuhashi has joined #openstack-infra10:18
*** talluri has quit IRC10:18
*** talluri has joined #openstack-infra10:19
*** talluri_ has joined #openstack-infra10:25
*** talluri has quit IRC10:26
*** mkerrin has joined #openstack-infra10:27
*** freyes has joined #openstack-infra10:27
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add fingerprint for grenade bug 1331274  https://review.openstack.org/10074310:29
uvirtbotLaunchpad bug 1331274 in grenade "Couldn't find log for * at /opt/stack/new/screen-logs/*" [Undecided,New] https://launchpad.net/bugs/133127410:29
*** yamahata_ has joined #openstack-infra10:31
openstackgerritRadomir Dopieralski proposed a change to openstack/requirements: Add xstatic-packaged JavaScript libraries for Horizon  https://review.openstack.org/9779310:35
*** lcheng has quit IRC10:37
*** hashar has quit IRC10:40
sdaguemorning folks10:41
*** jgallard has quit IRC10:42
sdaguee0ne: we do use the caches10:43
*** jcoufal has joined #openstack-infra10:43
sdaguehttp://logs.openstack.org/76/79776/15/gate/gate-tempest-dsvm-full/700ee7e/logs/devstack-gate-setup-workspace-new.txt.gz#_2014-06-04_19_50_59_57210:43
*** dims_ has joined #openstack-infra10:43
sdaguethey are seeds for the repos10:44
sdaguebut the caches will by definition be out of date10:44
sdaguee0ne: https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L30210:44
SergeyLukjanovvponomaryov, pong10:46
SergeyLukjanovsdague, morning10:46
sdaguemorning10:46
vponomaryovHello, I wanted to ask you review https://review.openstack.org/#/c/97747/ , if you have time.10:47
*** talluri_ has quit IRC10:47
*** talluri has joined #openstack-infra10:48
sdaguevponomaryov: those should really be set with your preset hook10:49
sdaguenot in there10:49
vponomaryovsdague: at the moment there is no keeping of localrc10:50
sdaguevponomaryov: ok, well that's a thing that needs to be adjusted in d-g then10:50
sdagueto allow for it10:51
vponomaryovsdague: also, in pre_test_hook all of these can be redefined10:51
*** jpich has quit IRC10:51
vponomaryovsdague: so, I don't see any harm in adding some export explicitly in job10:52
*** talluri has quit IRC10:52
*** talluri has joined #openstack-infra10:54
*** lcheng has joined #openstack-infra10:56
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: Now that grenade logs are separated, use console  https://review.openstack.org/10085110:56
*** Longgeek_ has quit IRC10:58
e0nesdague: thanks! does devstack-gate use cahces?10:58
*** Longgeek has joined #openstack-infra10:59
*** yamahata has quit IRC10:59
openstackgerritA change was merged to openstack-infra/elastic-recheck: Now that grenade logs are separated, use console  https://review.openstack.org/10085110:59
*** ihrachyshka has joined #openstack-infra11:00
*** ildikov has quit IRC11:01
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: move console.html -> console  https://review.openstack.org/10085411:01
*** ildikov_ has joined #openstack-infra11:01
sdaguevponomaryov: because it's a lot of extra review churn when those need to be changed for whatever reason11:02
sdagueand really should be in your job setup itself11:02
openstackgerritafazekas proposed a change to openstack-infra/elastic-recheck: Make the #1235437 query time limit change resitent  https://review.openstack.org/9407911:02
sdaguee0ne: yes it does11:02
sdaguesee https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L30211:02
*** flaviof_zzz is now known as flaviof11:02
e0nethanks, sdague! looks like something went wrong in my env. i need to investigate it11:03
sdaguee0ne: the cache is only used as a starting point11:04
openstackgerritNikita Konovalov proposed a change to openstack-infra/storyboard: Validate Project and Project Groups names  https://review.openstack.org/9872811:04
sdagueyou'll still need to pull references11:04
sdaguebut it should be a smaller amount of data11:04
*** jpich has joined #openstack-infra11:04
e0neyes, but I cached my fork (see https://review.openstack.org/100547) and devstack-gate uses original repo11:05
openstackgerritA change was merged to openstack-infra/elastic-recheck: move console.html -> console  https://review.openstack.org/10085411:05
*** gokrokve has joined #openstack-infra11:05
*** lcheng has quit IRC11:05
*** bookwar has quit IRC11:07
sdaguee0ne: so devstack gate is going to eventually hit gerrit11:08
vponomaryovsdague: Ok, thanks, will minimize it11:08
sdaguee0ne: what exactly are you trying to do? That might be more helpful to understand11:09
*** gokrokve has quit IRC11:10
*** bookwar has joined #openstack-infra11:11
*** Longgeek has quit IRC11:13
*** Longgeek has joined #openstack-infra11:14
*** paul-- has quit IRC11:15
*** e0ne has quit IRC11:15
*** vponomaryov has quit IRC11:16
*** dims_ has quit IRC11:16
*** bogdando has quit IRC11:16
*** denis_makogon has quit IRC11:17
*** achuprin_ has quit IRC11:17
*** paul-- has joined #openstack-infra11:17
*** Longgeek has quit IRC11:19
*** Longgeek has joined #openstack-infra11:19
*** ihrachyshka has quit IRC11:21
*** ihrachyshka has joined #openstack-infra11:25
*** denis_makogon has joined #openstack-infra11:25
*** e0ne has joined #openstack-infra11:25
*** vponomaryov has joined #openstack-infra11:26
*** bogdando has joined #openstack-infra11:26
e0nesdague: sorry, i've got issues with my internet connection11:28
e0nesdague: i'm trying to setup local infra for testing cinder with this patch-set(https://review.openstack.org/65113) with local devstack-gate. it will be integrated as a 3rd-party CI later11:28
*** achuprin_ has joined #openstack-infra11:29
*** jpich has quit IRC11:29
*** talluri has quit IRC11:32
*** d0ugal has quit IRC11:33
*** d0ugal has joined #openstack-infra11:34
sdagueok, so you are trying to test that patch? or test other things, and also have that patch?11:34
sdaguenow you're at a point where some of the zuul math comes into effect, and probably need someone that understands that piece better11:34
sdaguejhesketh or jeblair11:34
openstackgerritValeriy Ponomaryov proposed a change to openstack-infra/config: Keep devstack's localrc for manila jobs  https://review.openstack.org/9774711:34
sdaguejhesketh might still be up11:34
e0nei'm trying to setup devstack-gate to test cinder with ceph.11:35
e0nei need that patch for it11:36
*** prad_ has joined #openstack-infra11:36
vponomaryovsdague: updated https://review.openstack.org/#/c/97747/11:37
*** jpich has joined #openstack-infra11:42
*** eharney has joined #openstack-infra11:43
*** CaptTofu_ has joined #openstack-infra11:47
*** matsuhashi has quit IRC11:48
*** jpich has quit IRC11:49
*** lcheng has joined #openstack-infra11:50
*** matsuhashi has joined #openstack-infra11:51
openstackgerritA change was merged to openstack-infra/elastic-recheck: Make the #1235437 query time limit change resitent  https://review.openstack.org/9407911:51
openstackgerrityolanda.robla proposed a change to openstack-infra/storyboard-webclient: Display dates in timeago format  https://review.openstack.org/9671311:52
*** hashar has joined #openstack-infra11:54
*** matsuhashi has quit IRC11:56
*** lukego has quit IRC11:56
*** matsuhashi has joined #openstack-infra11:56
*** dprince has joined #openstack-infra11:57
*** Longgeek has quit IRC11:57
*** weshay has joined #openstack-infra11:58
*** Longgeek has joined #openstack-infra11:58
*** mbacchi has joined #openstack-infra11:58
*** sandywalsh_ has joined #openstack-infra11:58
e0nesdague: another usecase: if somebody needs to setup private infra for private openstack repos11:59
sdaguee0ne: right, you'd have to replicate everything then, including the gerrit11:59
sdaguethe downstream fork use case isn't well supported in the tools today12:00
*** talluri has joined #openstack-infra12:00
e0neabsolutely agree with you about forks. i need it only for testing for now12:00
*** matsuhashi has quit IRC12:01
*** lcheng has quit IRC12:01
*** jistr is now known as jistr|english12:02
*** jpich has joined #openstack-infra12:02
*** bogdando has quit IRC12:02
*** achuprin_ has quit IRC12:03
*** bogdando has joined #openstack-infra12:03
*** e0ne_ has joined #openstack-infra12:03
*** achuprin_ has joined #openstack-infra12:03
*** talluri has quit IRC12:04
*** talluri has joined #openstack-infra12:04
*** jpich_ has joined #openstack-infra12:05
*** e0ne has quit IRC12:05
*** gokrokve has joined #openstack-infra12:06
*** jpich has quit IRC12:06
*** gcha has joined #openstack-infra12:07
*** vponomaryov has quit IRC12:08
*** denis_makogon has quit IRC12:08
gchahi guys, I need your help to delete an old gerrit account of mine.12:08
*** achuprin_ has quit IRC12:08
*** talluri has quit IRC12:08
*** e0ne_ has quit IRC12:08
*** bogdando has quit IRC12:09
gchaactually I've two account with my email gchamoul@redhat.com12:09
*** mwagner_lap has quit IRC12:10
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety  https://review.openstack.org/10072812:10
*** gokrokve has quit IRC12:10
*** yamahata has joined #openstack-infra12:11
*** e0ne has joined #openstack-infra12:11
*** denis_makogon has joined #openstack-infra12:12
*** achuprin_ has joined #openstack-infra12:12
*** ArxCruz has joined #openstack-infra12:12
*** bogdando has joined #openstack-infra12:13
*** lcheng has joined #openstack-infra12:13
*** smarcet has joined #openstack-infra12:15
gchafungi: can you please give a hand about this ?12:15
e0nealso, i've created several blueprints to make repos settings configurable. i'm working on patches12:15
*** jpich_ is now known as jpich12:16
anteayaI have to go to an appointment this morning, I don't know when I will be back online12:16
*** adalbas has joined #openstack-infra12:17
BobBallDo we have any jobs that verify devstack's exercise.sh? I don't think we do, but I wanted to check.12:17
sdagueBobBall: not intentionally12:21
BobBallOK - I've explained in -qa when you look over there; but I think it's recently broken and if we want it kept then we should probably add a check job as a minimum12:21
*** vponomaryov has joined #openstack-infra12:23
*** annegent_ has joined #openstack-infra12:23
*** lcheng has quit IRC12:24
*** annegent_ is now known as annegentle_12:24
*** lcostantino has joined #openstack-infra12:27
*** trinaths has joined #openstack-infra12:28
*** katyafervent is now known as katyafervent_awa12:28
*** rfolco has joined #openstack-infra12:29
*** ArxCruz has quit IRC12:30
openstackgerritA change was merged to openstack-infra/config: Add zuul definitions for TripleO VLAN check job  https://review.openstack.org/10007612:31
*** _nadya_ has joined #openstack-infra12:31
*** ArxCruz has joined #openstack-infra12:32
*** trinaths has quit IRC12:34
*** CaptTofu_ has quit IRC12:36
*** andreaf_ has quit IRC12:37
*** andreaf_ has joined #openstack-infra12:37
*** CaptTofu_ has joined #openstack-infra12:38
*** lukego has joined #openstack-infra12:39
*** andreaf_ is now known as andreaf12:40
*** katyafervent_awa is now known as katyafervent12:40
*** lcheng has joined #openstack-infra12:40
*** pdmars has joined #openstack-infra12:48
*** Longgeek has quit IRC12:48
*** pdmars has quit IRC12:50
*** lcheng has quit IRC12:50
*** pdmars has joined #openstack-infra12:50
*** xyang2 has joined #openstack-infra12:50
*** julim has joined #openstack-infra12:51
*** annegentle_ has quit IRC12:53
*** ildikov_ has quit IRC12:53
*** basha has quit IRC12:57
*** Longgeek has joined #openstack-infra12:57
*** mfer has joined #openstack-infra12:58
*** dkranz has joined #openstack-infra12:58
*** ildikov has joined #openstack-infra12:59
*** yfried_ has quit IRC13:01
*** gokrokve has joined #openstack-infra13:01
*** gokrokve_ has joined #openstack-infra13:02
*** gokrokv__ has joined #openstack-infra13:02
*** annegent_ has joined #openstack-infra13:03
*** dims_ has joined #openstack-infra13:04
openstackgerritNikita Konovalov proposed a change to openstack-infra/infra-specs: StoryBoard search spec  https://review.openstack.org/9725413:05
*** gokrokve has quit IRC13:05
*** gokrokve_ has quit IRC13:06
*** yamahata has quit IRC13:07
*** yamahata has joined #openstack-infra13:07
*** flaviof is now known as flaviof_zzz13:08
*** kashyap` has joined #openstack-infra13:13
*** annegent_ has quit IRC13:13
*** dims_ has quit IRC13:13
*** bknudson has joined #openstack-infra13:14
KiallRandom Q - Is there any way with Zuul to dequeue changes?13:14
KiallAs in - Just tell zuul not to bother testing them13:14
*** Hal_ has joined #openstack-infra13:16
sdagueKiall: nope13:17
sdagueonce it's in play, it will get tested13:17
*** dims_ has joined #openstack-infra13:17
*** kashyap has quit IRC13:17
sdagueSergeyLukjanov: https://review.openstack.org/#/c/100728/ - that removes some spurious warnings13:18
Kiallsdague: damn `kill -9` the only option then? (the -9 to avoid zuul persisting it's state)13:18
yolandasdague, even some dirty way? i enqueued a huge list of changes by mistake13:18
sdagueKiall: possibly13:18
sdagueyolanda: no that I know of13:18
Kiallsdague: K .. Thanks, worth a shot asking ;)13:19
*** kashyap`` has joined #openstack-infra13:20
*** unicell has quit IRC13:21
*** unicell has joined #openstack-infra13:21
*** mriedem has joined #openstack-infra13:22
*** kashyap` has quit IRC13:22
openstackgerritA change was merged to openstack-infra/config: Publish osprofiler on pypi and add check for requirements  https://review.openstack.org/10005213:23
Kiallsdague: has nobody ever accidentally queued up hundreds and hundreds of patches by mistake? I'd have thought that would be regular occurrence :)13:23
sdagueKiall: nope13:23
sdaguenot that I've seen13:23
sdaguegit review usually makes that pretty hard13:24
sdaguebecause you have to be presented with a giant screen of 100s of changes, then say yes :)13:24
KiallYea - There are some cases where you will see 100's in the list, but only 1 will actually submit though ;)13:25
sdaguethere may be something in the zuul rpc interface to od that thing13:25
sdaguewell, if I saw a screenful I'd be scared :)13:25
Kiallhah - a really really weird internal process has trained me to think "That's OK, only one of these will actually submit" ;)13:27
Kiall(and apparently, yolanda too ;))13:27
*** unicell has quit IRC13:27
*** jesusaurus has quit IRC13:27
yolandaweird that i didn't see a huge list of commits going13:28
yolandathe review looked fine13:28
sdaguedid it trigger a rebase?13:28
yolandai git a git review -R13:28
*** otherwiseguy has joined #openstack-infra13:28
yolandabecause i didn't have to modify anything, just rebase13:28
*** dims_ has quit IRC13:28
sdagueKiall: maybe worth trying to figure out a way to make git review support a sanity check for that case13:29
sdaguethat's probably why we've never seen it upstream13:29
*** CaptTofu_ has quit IRC13:30
*** unicell has joined #openstack-infra13:30
openstackgerritThierry Carrez proposed a change to openstack-infra/release-tools: Support swift and oslo milestone releases  https://review.openstack.org/9989213:30
KiallYea.. Possibly!13:31
*** jswarren has quit IRC13:31
yolandawell, everything looks fine now, but was a huge mistake from my side13:31
KiallWe do some weird stuff with rebase+merges to allow `git pull --ff-only` to wok after integrating upstream changes, end result is a confused git-review13:31
sdagueyeh, I would think making git-review be less confused there would be goodness13:32
*** mestery has joined #openstack-infra13:32
sdagueit's never fun to use a tool, and have it do an unexpected thing13:33
KiallAnd making Zuul dequeue changes when they get abandoned would be another ++ ;)13:33
sdagueyeh, I'd agree with that13:35
*** annegent_ has joined #openstack-infra13:35
openstackgerritAntoine Musso proposed a change to openstack-infra/zuul: Make swiftclient an optional dependency  https://review.openstack.org/9793313:35
*** kashyap`` is now known as kashyap13:36
*** jgallard has joined #openstack-infra13:36
openstackgerritIvan Kolodyazhny proposed a change to openstack-infra/devstack-gate: Optional Ceph backend for Cinder  https://review.openstack.org/10090513:36
*** annegent_ is now known as annegentle_13:36
*** jesusaurus has joined #openstack-infra13:37
*** jswarren has joined #openstack-infra13:38
*** maxbit has joined #openstack-infra13:38
*** changbl has quit IRC13:38
*** CaptTofu_ has joined #openstack-infra13:39
*** dizquierdo has quit IRC13:39
*** lcheng has joined #openstack-infra13:40
*** annegentle_ has quit IRC13:40
*** annegent_ has joined #openstack-infra13:41
*** yolanda has quit IRC13:42
hasharKiall: sdague: there is a patch pending somewhere to dequeue abandoned changes13:42
*** jistr|english is now known as jistr13:42
hasharKiall: sdague: though it has been abandoned blaaa https://review.openstack.org/#/c/65467/13:43
*** sandywalsh_ has quit IRC13:43
*** thomasbiege has joined #openstack-infra13:43
*** mestery has quit IRC13:44
hasharbah I forgot about it . Let me restore / rebase / test13:44
*** penguinRaider has joined #openstack-infra13:45
*** miqui has joined #openstack-infra13:46
sdaguehashar: cool13:47
openstackgerritThierry Carrez proposed a change to openstack-infra/release-tools: Support swift and oslo milestone releases  https://review.openstack.org/9989213:47
*** thomasbiege has quit IRC13:47
*** mwagner_lap has joined #openstack-infra13:47
*** salv-orlando has quit IRC13:49
*** salv-orlando_ has joined #openstack-infra13:49
Kiallhashar: lol - typical ;)13:53
*** sballe has joined #openstack-infra13:54
openstackgerritAntoine Musso proposed a change to openstack-infra/zuul: dequeue abandoned changes  https://review.openstack.org/6546713:54
hasharKiall: sdague: I have rebased my change https://review.openstack.org/65467  . The tests might even pass!13:54
sdaguenice13:54
hasharthe change itself is very simple13:55
hasharit took me a while to figure out how to write the tests though :-(13:55
*** wenlock has joined #openstack-infra13:55
*** amotoki has quit IRC13:55
*** homeless has joined #openstack-infra13:56
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1331476  https://review.openstack.org/10091213:56
uvirtbotLaunchpad bug 1331476 in nova "ERROR nova.api.metadata.handler [-] Failed to get metadata for ip" [Undecided,New] https://launchpad.net/bugs/133147613:56
*** _nadya_ has quit IRC13:57
*** Longgeek_ has joined #openstack-infra13:57
*** jistr has quit IRC13:59
*** yfried_ has joined #openstack-infra13:59
Kiallhashar: I don't see a test case for what happens when a change that is not in the queue (e.g. already completed) is abandoned, that might be covered elsewhere though :)13:59
*** Longgeek has quit IRC13:59
KiallLooks like it's covered by the removeChange method already :)14:00
*** mrodden has quit IRC14:00
*** jistr has joined #openstack-infra14:01
*** sdague has quit IRC14:02
openstackgerritA change was merged to openstack/requirements: Add oslo.db library  https://review.openstack.org/9140714:03
*** nosnos has quit IRC14:03
openstackgerritA change was merged to openstack/requirements: Make sure pymongo version >= 2.5  https://review.openstack.org/10077714:03
openstackgerritA change was merged to openstack/requirements: Make sure msgpack-python version >= 0.4.0  https://review.openstack.org/10075114:04
*** trinaths has joined #openstack-infra14:07
*** sdague has joined #openstack-infra14:08
*** atiwari has joined #openstack-infra14:09
*** [1]trinaths has joined #openstack-infra14:09
*** annegent_ has quit IRC14:09
*** yolanda has joined #openstack-infra14:10
*** annegent_ has joined #openstack-infra14:10
*** trinaths has quit IRC14:13
*** [1]trinaths is now known as trinaths14:13
hasharKiall: feel free to comment on the patch :]14:13
*** devananda has quit IRC14:14
*** rcarrillocruz has quit IRC14:14
*** devananda has joined #openstack-infra14:15
*** krotscheck has quit IRC14:16
*** esker has joined #openstack-infra14:16
*** mestery has joined #openstack-infra14:16
yolandaclarkb, mordred, are you available to discuss about the ssh keys options for nodepool and dib?14:17
*** markmcclain has joined #openstack-infra14:18
yolandawhat i've seen so far is that nodepool generates a keypair in openstack, then it uses that for connecting into the instance, and injecting nodepool keys there, the /etc/nodepool/id_rsa pairs14:19
yolandawe need to have a way to connect to the instances generated by dib also, what do you think about it?14:20
*** rcarrillocruz has joined #openstack-infra14:20
*** krotscheck has joined #openstack-infra14:22
*** esmute has quit IRC14:22
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: additional review guidelines  https://review.openstack.org/10091614:22
*** esmute has joined #openstack-infra14:25
andreafsdague, clarkb: do you have a moment for a review on https://review.openstack.org/#/c/98365/ ? This is support for an alternate features.yaml14:26
*** dims_ has joined #openstack-infra14:27
*** dangers_away is now known as dangers14:29
*** andreykurilin_ has joined #openstack-infra14:29
*** rwsu has joined #openstack-infra14:29
sdagueandreaf: done14:30
*** trinaths has quit IRC14:30
andreafsdague: thanks - I agree I'll add a section for allowed branches14:31
*** trinaths has joined #openstack-infra14:31
sdagueandreaf: is the normalize_branch code going to work for your use case?14:32
sdagueas not using the standard branches has an impact there as well14:32
andreafsdague: right, perhaps we should have in features.yaml a section to configure the knonw branch prefixes14:33
andreafe.g. stable branches are stable/* and feature branches are feature/*14:34
andreafsdague: would that address your concern?14:35
*** _nadya_ has joined #openstack-infra14:37
sdagueandreaf: well, honestly, I'm also happy if we tell downstream folks that part of this logic is going to remain fixed14:38
*** dizquierdo has joined #openstack-infra14:38
*** rm_work|away is now known as rm_work14:38
andreafsdague: right, so if we do the normalize_branch would work14:39
andreafsdague: so it's enough to add a comment in the code?14:39
sdagueandreaf: honestly, it's mostly a question14:39
sdaguerealistically if yuo make the supported branches explicit in the yaml, I'm +214:39
sdagueI was just curious about the other thing. But I'd actually rather not make the stable/ feature/ thing configurable, because I think it would be complexity for little value14:40
andreafsdague: ok - I tested the normalize_branch with a "trunk" branch called something different from "master" and it worked fine14:40
sdaguecool, works me for me14:41
andreafsdague: yes I agree my aim here was only for the master branch to be called something different14:41
*** sabeen has joined #openstack-infra14:41
*** Longgeek_ has quit IRC14:43
*** eharney has quit IRC14:44
*** jistr has quit IRC14:44
*** Longgeek has joined #openstack-infra14:44
*** reed has joined #openstack-infra14:44
boris-42jhesketh hi there14:45
*** Longgeek has quit IRC14:45
boris-42jhesketh thanks for review14:45
*** eharney has joined #openstack-infra14:47
*** Longgeek has joined #openstack-infra14:47
*** jistr has joined #openstack-infra14:48
*** _nadya_ has quit IRC14:49
*** flaviof_zzz is now known as flaviof14:50
*** mrodden has joined #openstack-infra14:54
*** mrmartin has quit IRC14:55
mordredKiall: there is a patch to git-review we can make that will present that list more properly in cases of merge commits14:56
mordredKiall: I think I even wrote it in the past and then it languished for some reason14:56
*** annegent_ has quit IRC14:57
KiallProbably because no one accepts merge commits ;)14:57
Kiallmordred: hashar's patch also would have been useful :) https://review.openstack.org/#/c/65467/14:57
mordredKiall, sdague also - someone pushing up a bazillion commits by accident has happened exactly once before... but it was pre zuul and in the early days of gerrit14:57
mordredand was the reason we put in the one-patch check to git-review when we wrote it14:57
mordred(it was vishy, fwiw)14:58
mordredKiall: ++14:58
mordredKiall: https://review.openstack.org/#/c/50401/14:59
mordredKiall: it's possible you either have an old git review or we haven't released ina  while14:59
KiallThat's possible - Would need to check :)14:59
KiallYea - That's not included in any tags15:00
mordredyup. WOW - we havea  aLOT of outstanding changes15:00
mordredfungi: we should maybe makea  git-review release15:00
*** denis_makogon_ has joined #openstack-infra15:01
boris-42mordred hi could you approve patch with already 2x 2+ https://review.openstack.org/#/c/100243/ pls=)15:03
*** virmitio has joined #openstack-infra15:03
boris-42mordred they forgot to approve it lol=)15:03
*** jergerber has joined #openstack-infra15:04
*** roeyc has quit IRC15:04
*** dkranz has quit IRC15:04
*** annegent_ has joined #openstack-infra15:04
mordredboris-42: no clever name?15:04
boris-42mordred ahah=)15:05
boris-42mordred I don't know15:05
boris-42mordred that's ok for start15:05
boris-42mordred I think..15:05
boris-42mordred we can make voting when there will be more contributor and rename it15:05
boris-42contributors*15:05
mordredboris-42: :)15:06
*** hdd_ has joined #openstack-infra15:07
mordredboris-42: so - I just made a -1 - could you fix one thing please, then I'll approve it15:07
boris-42mordred sure15:07
boris-42mordred one sec15:07
mordredboris-42: also, the second thing is a question for real - it's ok if the answer is no15:07
*** dims_ has quit IRC15:07
boris-42mordred I mean it's good for start name15:08
boris-42mordred when there will be at least 10-15 devs we can make voting to rename it15:08
boris-42mordred but at least for now it's quite clear what logaas is =)15:08
boris-42from name=)15:08
*** dizquierdo has quit IRC15:08
*** dizquierdo has joined #openstack-infra15:09
*** gothicmindfood has quit IRC15:09
*** andreykurilin_ has quit IRC15:10
*** afazekas_ has quit IRC15:10
*** ihrachyshka has quit IRC15:11
boris-42mordred one more thing can I add coverage job as well on check?15:12
boris-42mordred as I am pushing new change15:12
boris-42mordred as it's quite small project -coverage jobs will be quite fast15:12
*** yamahata has quit IRC15:13
*** adalbas has quit IRC15:14
*** rdopieralski has quit IRC15:14
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Add new project logaas  https://review.openstack.org/10024315:15
boris-42mordred ^15:16
*** dprince has quit IRC15:17
*** moted has joined #openstack-infra15:17
*** basha has joined #openstack-infra15:18
* mordred wags fingers at boris15:18
mordredboris-42: my friend - that was almost all correct ...15:18
boris-42mordred haha=)15:18
*** dkranz has joined #openstack-infra15:19
boris-42mordred I can remove py26/py27 ?)15:20
*** Longgeek has quit IRC15:20
boris-42mordred from check and gate?)15:20
openstackgerritMonty Taylor proposed a change to openstack-infra/config: Add new project logaas  https://review.openstack.org/10024315:20
mordredboris-42: ^^15:20
*** otherwiseguy has quit IRC15:21
mordredboris-42: you can remove pretty much everything because it's all covered by templates you selected already15:21
*** IvanBerezovskiy has left #openstack-infra15:21
*** sweston has quit IRC15:21
boris-42mordred except coverage..15:21
boris-42mordred or coverage is include in python jobs?15:21
mordredboris-42: yes. if you want to add coverage to the check queue, you'd need ot add it15:21
mordredboris-42: if you go to the python-jobs section at the top of the file, you'll see what it does15:22
boris-42mordred yep it has almost all=)15:22
*** jpich has quit IRC15:22
*** shardy is now known as shardy_afk15:23
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Add new project logaas  https://review.openstack.org/10024315:24
boris-42mordred thanks for help =)15:25
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add tox target for building docs  https://review.openstack.org/10094015:25
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Update readme with more guidelines for writing queries  https://review.openstack.org/10094115:25
*** blamar has joined #openstack-infra15:25
boris-42mordred i'm going to refactor a bit layouts of rally/rally python client and osprofiler to use templates=)15:27
boris-42mordred btw what do you think about refactoring whole document?15:27
*** adalbas has joined #openstack-infra15:27
boris-42mordred so new members will just copy paste already working stuff15:27
*** denis_makogon has quit IRC15:28
*** denis_makogon_ is now known as denis_makogon15:28
mordredboris-42: I think we've been trying to do that slowly15:28
*** dmakogon_ has joined #openstack-infra15:28
mordredboris-42: refactoring the whole thing at once is probably hard due to merge conflicts15:29
boris-42mordred yep, but I can take this task and do it in separated patches if you would like15:29
boris-42mordred and keep them up-to-date until they will be merged15:29
*** _nadya_ has joined #openstack-infra15:29
boris-42mordred if it make sense..15:30
mordredboris-42: yah. I'd love to see more things using templates15:31
boris-42mordred okay I'll start with some baby steps15:32
*** _david_ has joined #openstack-infra15:32
boris-42mordred just trying to find first task to start doing something about infra=)15:32
*** dims_ has joined #openstack-infra15:32
*** otherwiseguy has joined #openstack-infra15:33
boris-42mordred thanks for approve15:34
_david_Which storage engine on MySQL database are you using for Gerrit ? I hope Innodb? Wikimedia?15:34
mordredinnodb15:34
mordredalways innodb15:35
openstackgerritRoger Luethi proposed a change to openstack-infra/config: Add manuals-jobs for training-guides  https://review.openstack.org/10005615:35
fabohashar: any chance to reach https://review.openstack.org/98437 soon?15:36
_david_mordred, have you noticed this: https://code.google.com/p/gerrit/wiki/OutstandingTopics#Intermittent_data_loss_and_corruption ?15:36
mordredjeblair: in gertty, for some reason my muscles continually want "u" to get me from the patch screen back to the review screen. I tried to write a patch to make that happen, but it did not work15:36
_david_mordred, what is Wikimedia guys are using?15:36
mordredjeblair: a) would you support such a thing? b) could you point me at where to do that?15:37
clarkbo/15:37
clarkbmordred: 'u'?15:37
mordred_david_: I don't know what they're using - but I do know that domas mitzuas, who is one of the world's leading mysql experts and guys who runs mysql for facebook helps run mysql for wikimedia15:37
mordred_david_: so if they weren't using innodb I'd be shocked15:38
mordredclarkb: yah15:38
mordredI type u on the diff page _every_ single time15:38
_david_mordred, thanks, then i would update the page and you as reference ;-)15:38
clarkbwhere does u come from?15:38
Alex_GaynorCan I bother anyone for a quick r? on https://review.openstack.org/#/c/100703/15:38
*** packet has joined #openstack-infra15:38
mordredclarkb: I'm not 100% sure - but it's clearly a muscle memory from a similar program15:38
mordredclarkb: markmc or markmclain also had the same urge to want to press u there15:39
mordredclarkb: (btw - I believe u is "up" in this context)15:39
*** jpich has joined #openstack-infra15:39
mordredweird. u works in gerrit to do that but I do not use it there myself15:39
*** xyang2 has quit IRC15:40
*** gothicmindfood has joined #openstack-infra15:40
clarkbI find myself more vim moded so esc works well15:40
clarkbbut hjkl not working trips me a lot15:40
openstackgerritA change was merged to openstack-infra/config: Add new project logaas  https://review.openstack.org/10024315:40
hasharfabo: ah thanks for the refactoring. Looks fine at a first glance but will need a bit more time to test it out :]15:41
hasharfabo: I have been sick for most of the week, slowly recovering.  Your patch is definitely in my review queue though :)15:41
markmcmordred, wasn't me :)15:41
fabohashar: thanks :)15:41
_david_mordred, Done, check again ;-)15:41
clarkbso when someone says we will rename it?15:41
*** SumitNaiksatam has quit IRC15:42
clarkber we will rename it. I immediately think we should -2 the change15:42
clarkbI hate renaming things15:42
clarkbit is a waste of time15:42
clarkbmordred: boris-42 ^15:42
*** mkoderer has joined #openstack-infra15:42
boris-42clarkb ?15:42
*** dims_ has quit IRC15:42
clarkberror prone, time consuming by infra rooters, and usually for reasons that could have been prevented15:43
boris-42clarkb I mean I like that name15:43
clarkbboris-42: you told mordred you will rename logaas15:43
*** SumitNaiksatam has joined #openstack-infra15:43
clarkbor at least consider it15:43
boris-42clarkb but if community will dislike it so much that they will like to chose another we will rename it15:43
* mordred thinks logaas will probably stick15:43
boris-42clarkb I will rename if it community will require such sutff15:43
boris-42clarkb for me it's ok15:44
*** dims_ has joined #openstack-infra15:44
boris-42clarkb and for couple of future developers that are going to start working on it15:44
boris-42clarkb mordred  sooo15:44
mordredI was mainly giving boris-42 a hard time about the boring name15:44
clarkbboris-42: I will -2 renaming it because some team votes to do so15:44
mordredboris-42: in general, we probably won't be ok with renaming it just for the heck of it - so if you like logaas, then I think it's going to be logaas15:44
*** changbl has joined #openstack-infra15:44
*** Sukhdev has joined #openstack-infra15:45
boris-42mordred clarkb  for me at this point is most important to get it on stackforge15:45
boris-42mordred clarkb  so everybody will be able to start collaborating15:45
clarkbI understand and the name is fine by me15:45
boris-42clarkb okay good it's fine by me as well=)15:45
clarkbI just don't want there to be an expectation that we will ername on a whim because a group of people decide the name is too boring15:46
*** _david_ has quit IRC15:46
* clarkb spins up gertty and tries to help catch up on reviews15:48
*** ildikov has quit IRC15:48
jesusaurusboris-42: is logaas related to meniscus?15:48
boris-42jesusaurus ?15:49
boris-42jesusaurus what is meniscus?)15:50
*** unicell has quit IRC15:51
jeblairmordred: should 'u' == esc?15:53
*** sballe_ has joined #openstack-infra15:53
mordredjeblair: maybe? I only seem to feel like it's the right key when I'm on the diff page - I can't explain _why_ I think it's right there and why I don't have the urge to use it elsewhere15:54
jesusaurusboris-42: http://developer.rackspace.com/blog/project-meniscus-an-update.html15:54
boris-42jesusaurus lol15:54
boris-42jesusaurus so actually it can be used as backend for logaas15:55
boris-42jesusaurus like logstach15:55
boris-42jesusaurus goal of logaas is to unify how we deal with logs in openstack15:55
*** jpich has quit IRC15:55
boris-42jesusaurus e.g. all services will auto discover where to store logs15:55
openstackgerrityolanda.robla proposed a change to openstack-infra/nodepool: Build images using diskimage-builder  https://review.openstack.org/4648215:56
*** moted has quit IRC15:56
boris-42jesusaurus and there will be OpenStack API for quering logs15:56
*** sballe has quit IRC15:56
*** moted has joined #openstack-infra15:56
yolandaclarkb, jeblair, so i did some approach, just ignoring all the ssh connect and key creation part, if image is created using diskimage15:56
boris-42jesusaurus on top of this we would like to build specific for OpenStack aggregation of logs with black jack and15:56
boris-42jesusaurus so probably you should join openstack-logaas15:56
boris-42jesusaurus #openstack-logaas room*15:57
yolandawe'll need to add jenkins key for connecting into nodepool using some element anyway, and as it's a generic task on nodepool/dib, i was wondering if we could automate that inside nodepool itself15:57
yolandaotherwise nodes get added into jenkins, but they cannot be woke up15:57
clarkbyolanda: I think it is better to continue using a common key15:57
*** hogepodge has joined #openstack-infra15:57
clarkbits simple and when it breaks easy to debug15:57
*** dims_ has quit IRC15:57
yolandaclarkb, the way it worked with snapshot images is that they create a new keypair, and it adds on the create  server stuff, into nova15:58
boris-42jesusaurus have to go but will be later15:58
yolandathen it uses that key to connect and inject the nodepool bits, right?>15:58
clarkbyolanda: but only for the image build step15:58
clarkbyolanda: the image build itself adds the key that jenkins uses to connect to the slaves with15:59
clarkbyolanda: with DIB we essentially remove that first step and can continue with the second15:59
yolandaclarkb, do you suggest creating some element for it?15:59
boris-42mordred clarkb btw could somebody create logaas group?15:59
boris-42mordred clarkb  thanks15:59
clarkbyolanda: puppet does it for us today15:59
yolandaclarkb, can you point me where?15:59
*** gyee has joined #openstack-infra16:00
yolandamaybe i'm missing bits of the configuraiton16:00
mordredyolanda: it's in the jenkins slave puppet manifest16:00
*** dkranz has quit IRC16:00
clarkbyolanda: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/manifests/single_use_slave.pp#n4016:00
*** salv-orlando_ has quit IRC16:00
jeblairkrotscheck: ui feedback: the new red and black in storyboard doesn't give me a happy feeling when using it; to some people, large areas of red mean 'very bad' at subconcious level.  (of course to others, it means let's celebrate)16:01
yolandaclarkb, and there will be an element in dib that will populate these bits?16:01
*** esker has quit IRC16:01
*** sandywalsh_ has joined #openstack-infra16:02
clarkbyolanda: yes, we will run puppet in dib16:02
*** jcoufal has quit IRC16:02
yolandaclarkb, that's done on the config part of nodepool/dib ?16:02
*** amotoki has joined #openstack-infra16:02
*** jlibosva has quit IRC16:03
*** sabeen1 has joined #openstack-infra16:03
yolandaso with the changes i did now, to just ignore all ssh and injection stuff for dib images, we will be able to make it work, could you take a look at my changes to verify if i'm right'16:03
openstackgerritA change was merged to openstack/requirements: Require django-pyscss 1.0.1 for django-compressor 1.4 support  https://review.openstack.org/9776016:03
*** terryw has joined #openstack-infra16:03
clarkbyup, gertty is syncing now so I should haev a good list of changes to work with shortly16:04
*** freyes has quit IRC16:04
clarkblifeless: I don't like the remoev WIP changes from listings because it seems to remove them from the all changes list as well16:04
*** sabeen has quit IRC16:05
*** otherwiseguy has quit IRC16:07
zarohashar: ping16:07
openstackgerrityolanda.robla proposed a change to openstack-infra/nodepool: Build images using diskimage-builder  https://review.openstack.org/4648216:08
*** jgallard has quit IRC16:09
mordredclarkb: oh. hrm. the ssh health check ...16:10
clarkbmordred: that isn't needed because no ssh is involved in a dib image build16:10
mordredclarkb: no, when we spin up a node, we check to see if it's ssh-able before declaring it live16:11
hasharzaro: hi16:11
clarkbmordred: right isn't that independent of image building and uses nova metadata stuff?16:11
*** sandywalsh_ has quit IRC16:11
*** dkranz has joined #openstack-infra16:12
mordredclarkb: oh, does it?16:13
clarkbI thought so16:13
mordredclarkb: yeah - I think you're right16:13
clarkbthe image build istelf doesn't know about that16:13
clarkbnodepool does it all through nova, I could be wrong though as I haven't double checked the code on that16:13
*** terryw has quit IRC16:13
yolandaso i just ignored all work involving on ssh on spinning a node16:13
yolandait spins it ok, and adds to jenkins16:13
zarohashar: i think i am able to release python-jenkins now.  would you be interested in a release?16:13
mordredyolanda: ok. cool16:13
yolandabut as i miss config,slave is down16:13
*** sandywalsh_ has joined #openstack-infra16:14
yolandai will add the puppet bit16:14
*** otherwiseguy has joined #openstack-infra16:14
mordredyolanda: why skip ready script if a disk image was used?16:14
yolandamordred, we cannot connect by ssh16:15
yolandaso we cannot run it16:15
*** sweston has joined #openstack-infra16:15
*** yjiang5_away is now known as yjiang516:15
*** hemna_ is now known as hemna16:15
hasharzaro: current master is fine. It does not add that much new features16:16
mordredclarkb: nope. looks like we rely on key from image build time to do the run time ssh checks16:17
mordredyolanda: I am now following the issue16:17
clarkbmordred: the jenkins key or the special nodepool key?16:17
mordredclarkb: the nodepool key16:17
clarkbhuh16:17
mordredwe do not inject it at boot time16:17
hasharzaro: would really love https://review.openstack.org/#/c/89589/ to land in (that speed up job existences checks)16:17
yolandaoh, yes, i had to skip also the time check16:17
zarohashar: ok. i'm interested in the added feature to query plugin info.16:18
mordredso I think the options would be "inject it at boot time" and "use the local-config element to inject it at build time"16:18
yolandait's just doing a basic ssh check16:18
zarohashar: will take a look.16:18
*** eharney has quit IRC16:18
*** chandan_kumar has joined #openstack-infra16:18
*** eharney has joined #openstack-infra16:18
yolandaone thing about local-config, is that only injects keys into /root16:18
mordredhrm.16:18
yolandaso if we are connecting as nodepool, we wont' have them16:18
mordredwell, we could _write_ an element that adds the key configured in the yaml file16:19
mordredas in, the yaml file contains key config16:19
mordrednodepool could add that value into the dib invocation as a parameter16:19
mordredand we could have an element that does a similar task to local-config but injects the key we've configured in nodepool.yaml16:19
yolandawell, element should be the same process as the one we are doing on writeNodepoolInfo, right?16:20
hasharzaro: you also suggested to add a assert_node_exists() which I did today ( https://review.openstack.org/#/c/100835/ ) :D16:20
mordredyolanda: yeah. I think so16:20
mordredish16:20
clarkbmordred: yolanda I think I prefer what I described above16:20
clarkbbecause that is what I would expect nodepool to do16:20
clarkbalso wow world cup16:21
mordredclarkb: WOAH MY GOD16:21
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add tox target for building docs  https://review.openstack.org/10094016:21
hasharzaro: but feel free to cut a new release if you can't wait. We can always have a new cut next week.16:21
mordredclarkb: so you think we should do nova key injection?16:21
yolandainject nova keys on each instance?16:21
*** dprince has joined #openstack-infra16:22
clarkbmordred: I think so. The only possible problem with that is metadata services don't always work16:22
clarkbso injection may fail and we throwaway a node we could otherwise use16:22
*** amalagon has quit IRC16:22
mordredclarkb: so - funny story ... both of our clouds support config drive now16:22
clarkbisn't config drive one of the bad things?16:22
mordredclarkb: so we could actually start telling our nodes to use config drive - which cloudinit also knows how to use16:23
mordredclarkb: some people hate config drive and love metadata service. some people hate metadata service and love configdrive16:23
clarkbmordred: well bad in that it doesn't work16:23
mordredclarkb: but both of our clouds allow us to use config drive, and that is not true about metadata service16:23
clarkbbut I think it was just libguestfs doing direct injection that was terrible16:23
tteggelerm hpcloud has currently broken config drive, sorry16:23
mordredclarkb: metadata service also doesn't always work16:23
tteggelback next week :)16:23
clarkbtteggel: oh I am not crazy16:24
mordredtteggel: oh. awesome16:24
clarkbmordred: our experience upstream is that metadata server is more reliable than other choices16:24
zarohashar: i'm in no hurry.  the feature was merged a while ago.  would help jjb modules and backwards compatability.16:24
mordredclarkb: so - I think we _weren't_ doing key addition at boot because we had to add one for image build anyway16:24
clarkbmordred: yes it breaks, but it breaks less often16:24
mordredclarkb: so since it was there, we just re-used it at boot time16:24
clarkbmordred: correct16:24
hasharzaro: cut it right now so :]16:24
clarkbI think we should do key injections and be cloudy16:24
zarohashar: wanted to get in sync with you and msambro before doing a release though.16:24
mordredwell..16:24
hasharzaro: the other patches pending can make it later next week and a yet another release made out of them16:24
mordredclarkb: how about this ...16:24
clarkbworst case nodepool doesn't spin up nodes and we yell at our clouds16:24
mordredclarkb: how about we do key injection at dib time for now, so that we have less branching paths in nodepool16:25
hasharzaro: there are some patches pending james review/ I copy pasted some code he wrote in Nodepool and moved it to python-jenkins16:25
mordredand then go back in a second pass and rework the whole thign to do cloud key stuff16:25
zarohashar: in any case release won't be available in precise unless we do something special.16:25
mordredso that it's consistent in both codepaths16:25
clarkbsure16:25
hasharzaro: I can backport the package :-]16:26
zarohashar: +2 for 89589, might want to ping mgagne to shepard it thru.16:26
mordredand/or do the move to cloud keys if we decide to completely remove the script-based snapshot path16:26
yolandaclarkb, mordred, key injection at build time should always work as it doesn't depend on any provider setting16:26
clarkbmordred: well the snapshot path will work with key inejection too16:26
*** arnaud__ has joined #openstack-infra16:26
yolandait's just add a recognized key in the image16:26
hasharzaro: we might need a few more core review on python-jenkins16:26
zarohashar: ohh, good to know  :)16:26
mordredtteggel: so config drive at hp normally works but has a bug this week and should be good next week? or it's been broken for a while?16:26
tteggelmordred: been broken for a while16:27
zarohashar: i was thinking same thing.  maybe jjb as well?16:27
yolandaso... nodepool should be adding a image element ALWAYS to the generate images, independently of the elements included in config16:27
tteggelmordred: deploy next week will fix16:27
mordredtteggel: awesome16:27
hasharzaro: ditto16:27
* mordred has been learning more about nova metadata things and has some ways he'd like to think about using them16:27
hasharzaro: for JJB David Caro / Darragh Bailey / Mark Abramovitz come to mind.16:28
mordredclarkb: that seemed like a handball in the box by the guy from oz16:28
hasharzaro: definitely trust Marc Abramovitz for python-jenkins :]16:28
hasharzaro: I am not sure what is the process to add some more core reviewers.  Should they be suggested to the project cabal? (aka the current core-reviewers?)16:28
zarohashar: marc might be best only because he's most active on IRC. although don't see msabmro around lately.16:29
yolandaso mordred, clarkb,do we agree with that as a first step?16:29
hasharzaro: marc is probably busy / on vacations :D16:29
zarohashar: ahh, that would make sense.16:29
zaromordred: what is the process to nominate core reviewer for jjb and python-jenkins?16:30
clarkbmordred: yup they are useful. when they work16:30
clarkbmordred: I still cannot boot centos ndoes in 1.1 with my personal account16:30
*** bknudson has quit IRC16:30
clarkbmordred: metadata fails 100% of the time16:30
clarkbmailed support nothing changed16:30
clarkbgave up on centos16:30
zarohashar: btw, i really like pelix contributions to jjb.16:30
mgagnezaro: looks ok to me16:31
* pelix blushes16:31
*** wendar_ is now known as wendar16:31
yolandaclarkb, mordred, need to step out for a while, but i'll be reading the backlog, so please send comments if you have somehting16:31
pelixthks zaro :p16:31
*** e0ne_ has joined #openstack-infra16:31
zaroohh our other ** to busy** core :)16:31
pelixif only I could find more time16:31
clarkbyolanda: yes I think what mordred descirbes is a good next step16:31
clarkbyolanda: have dib emulate what the current process does then we can refine in the future16:31
yolandaclarkb, makes sense16:31
hasharzaro: pelix being Darragh Bailey? :D16:32
yolandai didn't like the idea to ignore the regular checks because of this16:32
zarothe one and only16:32
pelixhaha16:32
yolandaand then we can also run the ready script if needed16:32
mordredyolanda: ++16:32
hasharzaro: so I am definitely supporting pelix has a core reviewer for JJB :D16:32
mordredpelix: I would like you to have more time as well16:33
zaropelix: so does that mean you are disqualifing yourself for nominationa as jjb core?16:33
mgagnehaha16:33
mordredpelix: do you want me to go yell at someone?16:33
*** e0ne has quit IRC16:33
clarkbianw: comment on 100131 for you16:34
clarkbianw: I am +2 regardless but wanted to point that out16:34
zaromordred: actually i like pelix reviews as well.  just feel not able to contact as easily :(16:34
pelixI think if I can be more consistently available over the next 2 weeks then, yes I would love to help, but if I can't manage to adjust my schedule then it's no better than the current situation and I'm only taking up a core reviewer slot16:34
hasharzaro: family is back home. I am disconnecting. Will be back later tonight (i.e. in roughly 3 hours)16:34
openstackgerritA change was merged to stackforge/python-jenkins: Speed up job existence tests by fetching less info  https://review.openstack.org/8958916:34
pelixdefinitely recommend Mark Abramovitz as well btw16:35
pelixas core reviewer for jjb16:35
jogostrange the number of test nodes in deleting is back up16:35
zarohashar: enjoy.16:35
*** hashar has quit IRC16:37
zarocool, now we got our guy. just need to go thru process.  BTW: its Marc, not Mark.16:40
*** Hal_ has quit IRC16:41
*** arnaud__ has quit IRC16:44
*** arnaud__ has joined #openstack-infra16:44
*** lcheng has quit IRC16:45
*** ramashri has joined #openstack-infra16:45
*** e0ne_ is now known as e0ne16:46
krotscheckjeblair: I’ll send that feedback back to the UX team.16:47
jeblairkrotscheck: thx.  i left a comment on the subscription spec16:48
*** Ryan_Lane has joined #openstack-infra16:50
*** _nadya_ has quit IRC16:51
*** mugsie has quit IRC16:54
*** graham__ has joined #openstack-infra16:54
clarkbwoot cleared d-g backlog16:54
clarkbon to config16:54
jogosdague: maybe we can beat yesterdays merge record in openstack/openstack16:55
mtreinishclarkb: https://review.openstack.org/99835 :)16:56
mordredjogo: what was the merge record yesterady?16:56
mtreinish117 right?16:56
jogohttps://github.com/openstack/openstack/graphs/commit-activity16:56
*** isviridov is now known as isviridov|away16:56
jogo118, I think the github logic is a little off because it can go up after the end of day16:56
*** lcheng has joined #openstack-infra16:57
*** _nadya_ has joined #openstack-infra16:57
mordredwe did 132 back in march16:57
mordredweek of march 2216:57
*** arnaud__ has quit IRC16:57
clarkb147 in february16:57
jogo147 on feb 10th16:57
jogorecord for a week is 54916:58
mordredgiven the recent woes though, I think 118 yesterday is great16:58
clarkbmtreinish: did the d-g change merge that 99835 depends on?16:58
jogomordred: yeah all quite on the western front16:58
clarkbgertty doesn't make it easy to open that change by change id16:58
*** arnaud has joined #openstack-infra16:58
* clarkb looks in gerrit16:59
mtreinishclarkb: yeah I think it did16:59
jeblairclarkb: ctrl-o16:59
*** e0ne_ has joined #openstack-infra16:59
jeblairclarkb: or do you mean Iabcdef?16:59
jogomordred: we haven't even had any sustained periods of hitting capacity in the past 24 hours16:59
*** dims_ has joined #openstack-infra16:59
clarkbjogo: ya Iabcdef16:59
clarkber jeblair ^16:59
jeblairclarkb: ah, k.  that's next on my list.16:59
clarkbhttps://review.openstack.org/#/c/99833/ it did16:59
*** johnthetubaguy is now known as zz_johnthetubagu17:00
bookwarare there exist certain umbrella domain names for openstack-related oss projects? something like <project_name>.openstackcommunity.org?17:00
bookwarmaybe http://stackforge.org/ can share third-level subdomains?17:01
*** e0ne has quit IRC17:01
*** adalbas has quit IRC17:01
clarkbmtreinish: last question for you. Do you think devstack and d-g should get the experimental job too?17:02
clarkbmtreinish: I suppose we can start with nova and tempest and if you run into something weird we can add others17:02
clarkbbookwar: we have been trying to stop doing third level subdomains like that17:03
clarkbso ones that did exist like nova.openstack.org are now redirects to the consolidated docs location17:03
*** esker has joined #openstack-infra17:03
mtreinishclarkb: yeah I'm not sure they need it. I don't think there will be many commits that will touch that config path.17:04
mtreinishyeah I guess we can always add it if we need it17:04
reedbookwar, what do you mean?17:04
clarkbwhat is this UX team personas stuff? not related to mozilla persona I take it?17:04
clarkbkrotscheck: jeblair ^17:04
reedclarkb, not related indeed17:05
krotscheckclarkb: No, it’s a UX term for describing a user archetype.17:05
reedclarkb, personas is a way to describe 'users'17:05
clarkbwhy can't we just say users?17:05
*** maxbit has quit IRC17:05
mordredclarkb: it's a UX term17:05
* clarkb is clearly not built for resolving hash table collisions17:05
bookwarclarkb: i agree that openstack.org is better to be reserved, but some less official names could be used17:05
*** trinaths has quit IRC17:05
clarkbbookwar: maybe its better to talk about use cases17:06
clarkbbookwar: is there something you would like to accomplish?17:06
reedbookwar, what do you have in mind? not clear to me17:06
krotscheckclarkb: Because users are users. Archetypes are “User who’s an admin”, “User who’s a customer”, “User who’s a unicorn".17:06
*** harlowja_away is now known as harlowja17:06
clarkbkrotscheck: I think you just said why its silly :)17:06
krotscheckclarkb: :-P17:06
clarkbwe have terms for the specific types of users17:06
clarkband users are users17:06
reedclarkb, read the description of what a persona is.. let me get one for you17:06
jeblairclarkb: because jargon is a shibboleth.  ;)17:06
bookwarclarkb, reed: we have a project Fuel which is hosted at stackforge repos17:06
annegent_clarkb: personas are not users17:06
*** sweston has quit IRC17:07
*** amotoki has quit IRC17:07
*** lcheng has quit IRC17:07
bookwarand i'd like to have a domain name like fuel-stackforge.org to be point to our server where we can host description, docs, download links and so on17:07
reedclarkb, https://wiki.openstack.org/wiki/PersonaPotentialRoles17:07
bookwarfuel.stackforge.org*17:07
reedbookwar, now I see what you mean, thanks17:08
reedbookwar, you'd like to have something like a minisite to host specific content for projects managed via stackforge17:09
*** _nadya_ has quit IRC17:09
reedbookwar, I think so far we have wiki pages and project-specific domains/sites (thinking of solum.io or devstack.org as examples)17:10
reedclarkb, https://wiki.openstack.org/wiki/PersonaPotentialRoles and see how more specific the personas can be17:10
reeduh, penalty!17:10
*** e0ne_ has quit IRC17:10
clarkbreed: right I get that. I think what I don't get is why this needs a different term than the generally accepted term for the thing17:10
mgagnebookwar: I never used/tried it but why not readthedocs.org ?17:10
mordreddude17:11
reedfor Austrial... bbl17:11
clarkbmgagne: bookwar right so most stackforge projects use read the docs17:11
clarkband you can host build artifacts on tarballs.o.o17:11
clarkbmordred: just wow17:11
mordredbookwar: we used to have project-specific subdomains and we got rid of them17:11
reedand goal17:11
mordredDUDE17:11
*** salv-orlando has joined #openstack-infra17:11
*** e0ne_ has joined #openstack-infra17:11
bookwarok, thanks to all, i'll look into other options mentioned17:12
reedmordred: am I spoiling it for you? :)17:12
mordredreed: nope. I'm sitting happily in front of the TV17:12
reedlol17:12
mordredthis is not the channel to be in if you don't want to hear things about worldcup matches17:13
*** dizquierdo has quit IRC17:13
*** e0ne__ has joined #openstack-infra17:13
reedbookwar, is your problem finding a good place to host your content or enhancing discoverability of that content?17:14
morganfainbergice hockey and world cup, the 2 reaosns i miss TV this year.17:14
reedbecause hosting it somewhere should not be too hard, as clarkb mentioned readthedocs is available, tarballs too, our wiki, etc...17:14
bookwarreed: i have servers and stuff, so the problem is just with the name, not the hardware17:14
clarkbmordred O_O17:15
mordredWHAT IS THIS GAME???17:15
reedmorganfainberg, strictly a TV is not needed for any17:15
reedmordred: ping pong match :)17:15
mordredreed: dude17:15
clarkbmorganfainberg: get an antenna, plug into tv17:15
*** basha has quit IRC17:15
clarkbproblem sovled17:15
morganfainbergreed, but i don't have enough screen realestate on my computer to really watch atm :P not with all the code windows up.17:15
morganfainbergclarkb, ++ :)17:15
mordredmorganfainberg: you can listen to the games on espnradio17:15
clarkbmordred: but you really want the univision commentaristas17:15
*** adalbas has joined #openstack-infra17:16
morganfainbergmordred, ah! good idea, i forget about those things17:16
reedmorganfainberg, I only listen, in spanish and ALT-TAB when the noise sounds like excitement17:16
clarkbbecause you get GOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL MESSI MESSI MESSI MESSI GOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL17:16
mordredclarkb: hehe17:16
reedenglish commentary is bad17:16
morganfainbergclarkb, LOL17:16
openstackgerritIvan Kolodyazhny proposed a change to openstack-infra/devstack-gate: Optional Ceph backend for Cinder  https://review.openstack.org/10090517:16
morganfainbergreed, yeah always has been ... well US commentary17:16
morganfainbergreed, not sure about british commentary17:16
*** thedodd has joined #openstack-infra17:16
mordredclarkb: but, when you're driving from vegas to death valley and brazil is playing, you may want the english on the radio to know what's going on  :)17:16
reednot just that, clarkb, it's the poetry in describing anything that goes on... the brits on ESPN are ... boring17:16
*** e0ne_ has quit IRC17:16
bookwarlaptop with the game near the main screen - works for me )17:16
*** e0ne__ has quit IRC17:17
*** marcoemorais has joined #openstack-infra17:17
clarkbreed: well I don't really grok spanish :( but yes they are clearly far more into it than any english broadcast I have heard17:17
*** crc32 has joined #openstack-infra17:17
reed;)17:17
*** praneshp has joined #openstack-infra17:17
*** annegent_ has quit IRC17:17
reedI tuned in late, missed the first two goals of the game, apparently they're both spectacular17:18
morganfainbergclarkb, wanted to ask you, once i get the default mod_wsgi (so close, i can taste it) for keystone in devstack, which tempest run makes the most sense to run as eventlet? I don't want to cease gating on eventlet keystone unless we remove that support (not happening, as much as I'd like to)17:19
reedbookwar, probably your idea of having mini-sites for stackforge projects hosted somewhere on openstack infra is not bad, it may even be a simple static page, templated, with links to relevant details hosted somewhere else17:19
*** sballe__ has joined #openstack-infra17:19
morganfainbergclarkb, the only one i definitate don't want to run eventlet is the dsvm-full run (main run)17:19
reedbookwar, I meant to say that it's worth discussing it further via email17:19
*** lcheng has joined #openstack-infra17:19
clarkbmorganfainberg: postgres or neutron or something like probably17:20
*** timrc is now known as timrc-afk17:20
*** sandywalsh_ has quit IRC17:20
clarkbalso we are so bad at choosing names...17:20
*** rm_work is now known as rm_work|away17:20
morganfainbergclarkb,are we merging neutron and postgres? i lost track of that change / end decision17:20
clarkbmorganfainberg: I have decided to stay out of that because there is so much wrong info on that thread you can't reason with people17:21
morganfainbergclarkb, i think my mind was subconciously attempting the same thing17:21
clarkbI almost feel like restarting the whole thread with "No one is suggesting we stop testing db migrations and functionality against postgres. The suggestion was we stop doing integration testing with postgres"17:22
clarkbbut for some reason people have read that to mean no postgres anywhere17:22
morganfainbergclarkb, similarly, does it make sense to nuke the apache_services tempest instead of making it expirimental now https://review.openstack.org/#/c/97638/17:23
*** sballe_ has quit IRC17:23
clarkball of the things people want to do can be done without that integration test...17:23
morganfainbergclarkb, i'm leaning towards mkaing those tests go away since we wont need them.17:23
bodepdI have a question about this line of code: https://github.com/openstack-infra/config/blob/master/modules/jenkins/manifests/plugin.pp#L5517:23
bodepdit seems to mean that you cannot bootstrap a functional jenkins server (with the gearman plugin installed) without a manual restart?17:23
clarkbmorganfainberg: ya we will run it everywhere so having an experimental job isn't necessary17:24
morganfainbergclarkb, cool i'll just make that review destroy the apache_services one17:24
clarkbbodepd: correct17:24
clarkbbodepd: jenkins restarts are tricky17:24
clarkbpuppet doesn't have a big enough world view to do them correctly17:25
openstackgerritA change was merged to openstack-infra/config: Add tempest jobs with nova-v3 enabled  https://review.openstack.org/9983517:25
clarkbmorganfainberg: O_O17:25
clarkber mordred ^17:25
mordredgoooooooooooooooooooooooooooooooooooooooooooooooooooooooool17:25
*** esker has quit IRC17:25
reedhe kicked a soap bar17:26
reedslippery as hell17:26
mordredyah man17:26
* bookwar waits for next one17:27
*** pcrews has quit IRC17:27
*** jp_at_hp has quit IRC17:28
*** pcrews has joined #openstack-infra17:29
*** lcheng has quit IRC17:29
*** hdd_ has quit IRC17:29
sdaguejogo: well actually it double counts, we need a version that only counts merges17:30
sdaguewhich is why it keeps creeping up for yesterday17:30
*** sandywalsh_ has joined #openstack-infra17:31
*** bknudson has joined #openstack-infra17:32
clarkbgraphite has the data17:33
mordredclarkb: does graphite have it for just the integrated release though?17:33
*** jp_at_hp has joined #openstack-infra17:34
clarkbmaybe?17:34
clarkbit may separate by pipeline in gate17:35
*** bknudson has quit IRC17:36
*** rm_work|away is now known as rm_work17:36
*** changbl has quit IRC17:40
zaroanybody tried running gertty on mac?  i can't seem to get it to work.17:40
clarkbmordred: so there are a lot of ansible changes up17:41
clarkbmordred: and I am having a hard time groking whyfor17:41
mordredzaro: I think marckmclain did17:41
*** nati_ueno has joined #openstack-infra17:41
*** esker has joined #openstack-infra17:42
*** ramashri has quit IRC17:42
*** lcheng has joined #openstack-infra17:43
mordredclarkb: mainly that if we use it for run_remote_puppet instead of just a shell-script loop, then as with the follow-on patches, we can start to collect little playbooks for common admin tasks we run - as well as using it for ad-hoc activities that we may need to do17:43
*** ildikov has joined #openstack-infra17:43
mordredclarkb: since it's ssh-based, we don't have to run a master like we do with salt17:44
mordredor do the extra key exchange17:44
mordredso it's a reduction in complexity there17:44
openstackgerritA change was merged to openstack-infra/config: Project configuration for cachemonkey  https://review.openstack.org/9657017:44
clarkbmordred: ok17:45
clarkbmordred: it just seems like a pet project without much of a goal17:45
clarkbcurious to see what we are trying to solve there17:45
*** Sukhdev has quit IRC17:45
clarkbI think I grok it now17:45
mordredyah. the main benefit to us I think is the ability to collect some of our ops tasks into reviewable and re-useable chunks17:46
*** zzelle has joined #openstack-infra17:47
clarkbmordred: so bottom of that stack lifeless has -1'd and suggested a linter. Can we just remove that file?17:48
clarkbbecause double triple account is silly17:48
mordredclarkb: looking17:49
*** bknudson has joined #openstack-infra17:49
mordredclarkb: no - you need that file to be able to handle "!git0*.openstack.org"17:50
clarkbmordred: then maybe the current process is better?17:50
clarkbbecause it is built out of puppet17:50
mordredI'm not particularly worried about needing to add a line to a file when we add a server - we already have to add references in like 5 files when we add projects which happens _way_ more often17:51
mordredand having a file allows us to use that for adhoc things very easily17:51
morganfainbergsomeone smarter than I am when it comes to pip dependency resolution may know this: what happens if 2 pip packages depend on each other?17:51
clarkbexcept when a project breaks the world doesnt break17:51
mordredwhich the current single-purpose script does not give us17:51
clarkbgetting this wrong may literally break everything in some circumstances17:52
morganfainbergcan pip properly resolve that (assuming the versions aren't conflicting)17:52
mordredclarkb: and you think we can't get taht right with code review?17:52
clarkbmordred: no we miss things all the time17:52
dstufftmorganfainberg: depends what you mean by propery17:52
dstufftproperly17:52
mordredalso - we _Could_ write an inventory plugin that would do the puppet list17:52
dstuffttheortically yes, at the moment it's kind of YOLO17:52
morganfainbergdstufft, so we want to split the auth_token middleware out to it's own package.17:53
*** lcheng has quit IRC17:53
*** nati_ueno has quit IRC17:53
morganfainbergdstufft, unfortunately it depends on things in keystoneclient, and we currently provide it at keystoneclient.middleware17:53
*** _nadya_ has joined #openstack-infra17:53
*** dims_ has quit IRC17:53
dstufftdoes keystone depend on auth_token17:53
morganfainbergdstufft, we don't want to break the whole "you can get the middleware at keystoneclient.middleware" (for now)17:53
morganfainbergdstufft, no.17:53
mordredclarkb: but honestly, having the file on disk makes adhoc tasks quick and easy to run (I maintain one on my own laptop for all of infra)17:53
clarkbmordred: quick and easy and likely out of date >_>17:54
morganfainbergdstufft, the middleware is currently inside keystoneclient.17:54
*** nati_ueno has joined #openstack-infra17:54
* reed working hard before 'el partido de la muerte' 17:54
*** jp_at_hp has quit IRC17:54
dstufftI think it'll work, but I don't think it's that usual so it may or may not17:54
morganfainbergdstufft, so we want to split those two out.  maybe the right answer is freeze the middleware in keystoneclient, deprecate it and say "use this other package instead"17:54
*** ramashri has joined #openstack-infra17:55
dstufftnormally if you have a circular dependency it means you're doing something wrong17:55
*** e0ne has joined #openstack-infra17:55
clarkbmordred: concrete example. We convert a proper node to a vhost or consolidate nodes. DNS will be updated in that case, ansible ssh's into consolidated node and breaks everything17:55
morganfainbergdstufft, we did something wrong moving middleware into keystoneclient instead of it's own package :P17:55
dstufftmorganfainberg: that seems like a reasonable thing to do, or just say you have to install this other package to make keystone.middleware17:55
dstufftwork17:55
morganfainbergdstufft, but now i want to fix that :)17:55
clarkbmordred: its not likely but not terribly complex of a thing to get wrong because humans17:55
mordredclarkb: why would it break everything?17:55
dstufftyou can install conditonally import it from there17:55
clarkbmordred: because you would apply the wrong puppet in that case17:55
dstufftstill* not install17:55
mordredclarkb: no you woukdn't17:55
clarkbmordred: you would be apply the puppet from the old thing or default depending on the level of cleanup17:56
mordredclarkb: you'd run "puppet agent --test"17:56
*** e0ne has quit IRC17:56
morganfainbergdstufft, the issue is backwards compat, we could break everyone using... say grizzly if we released a keystoneclient that didn't provide the middleware17:56
clarkbmordred: oh, then what is the list for?17:56
*** e0ne has joined #openstack-infra17:56
mordredclarkb: so, the way it works is that you tell ansible what hosts to ssh in to ...17:56
morganfainbergdstufft, we try so hard not to break people with new releases of the client(s)17:56
mordredso for the puppet runs17:56
morganfainbergdstufft, i think i want to avoid doing that now as well (still)17:57
dstufftmorganfainberg: freezing and deprecating and removing later sounds like a reasonable thing to me17:57
mordredyou tell it "ansible git0*.openstack.org run puppet ; ansible review.o.o run puppet; ansible !git0*.o.o,!review.o.o run puppet"17:57
mordredso for the puppet case, it's fine17:57
clarkbmordred: ok17:58
mordredbut it means you can do things like "ansible *.slave.openstack.org delete workspace $foo please"17:58
mordredas an example of an ad-hoc admin task we can easily script17:58
morganfainbergdstufft, i'll propose that instead. should be easier/cleaner and in some <insert time> we can say 'this will go away'17:58
clarkbmordred: couldn't you add a puppet cert list as the first step?17:58
clarkband dump that into a file?17:58
mordredyou could write an inventory plugin to do puppet cert list17:58
mordredor you could do puppet cert list into the file to generate it from puppet perhaps17:59
mordredthere are certainly ways in which we can manage that inventory more automatically17:59
mordredI have not spent any time on them because "add line to file" didn't seem hard to me17:59
mordred:)17:59
clarkbits not hard its just wrong :)17:59
mordredwell, it didn't seem wrong to me17:59
mordredbut if it'sa  blocker for you, I can certainly work on that18:00
*** salv-orlando has quit IRC18:00
*** esker has quit IRC18:00
clarkbits not a blocker was just trying to understand the utility of why it is set up that way18:00
clarkbwhereas the old system is able to do it automagically18:00
clarkbthis appears to be a regression18:00
clarkbwhich makes me question it18:00
mordredit's more of the usefulness for tasks that aren't run remote puppet18:00
*** e0ne has quit IRC18:01
mordredif we make step one of any ad hoc task run "puppet cert list" - it's going to be annoyingly slow to use18:01
*** dims_ has joined #openstack-infra18:01
*** andreykurilin_ has joined #openstack-infra18:01
clarkbthe list is already out of date too, but that is a race18:02
clarkbdoes ansible run commands in parallel for hosts matching a glob?18:03
clarkbmordred: reviewed18:05
*** packet has quit IRC18:05
clarkbmordred: I am good with it but can't babysit as I do not really know much about ansible other than it uses ssh and fireball18:05
*** lcheng has joined #openstack-infra18:05
yolandajeblair,mordred, taking a look at the writeNodepoolInfo task, but it doesn't do the same that we need18:06
clarkbalso I think I get personas now. It is amarketing thing18:06
clarkbwhich explains the less specific term to our use case18:06
yolandathis code generates an unique keypair for each node, but is not the key used for connecting from nodepoool to the node, that's done using the nova injection18:06
yolandaare we ok to have an unique keypair in nodepool machine, then our script will add the entry into the authorized_keys bit? the location of the keypair could be indicated in the yaml file18:07
*** rcarrillocruz has quit IRC18:08
*** rcarrillocruz has joined #openstack-infra18:08
clarkbmordred: also see comment on the irc notifications. It is sad making18:09
openstackgerritDarragh Bailey proposed a change to openstack-infra/git-review: Ensure username is set for all tests  https://review.openstack.org/9936518:10
openstackgerritDarragh Bailey proposed a change to openstack-infra/git-review: Only set one gitreview.username value  https://review.openstack.org/10097618:10
openstackgerritDarragh Bailey proposed a change to openstack-infra/git-review: Provide nicer user message for missing remote ref  https://review.openstack.org/9775118:11
*** shayneburgess has joined #openstack-infra18:11
*** lcheng has quit IRC18:15
*** changbl has joined #openstack-infra18:16
*** basha has joined #openstack-infra18:18
*** packet has joined #openstack-infra18:19
pelixIs anyone looking into the problem with JJB and jenkins plugin compatibility?18:22
pelixi.e. as the XML output is updated in JJB for newer plugins it sometimes breaks compatibility with older versions, leaving people in the position that they can't upgrade to the newer version of JJB unless they also upgrade the affected plugin on their running instances.18:22
SlickNikclarkb / fungi: Could I please get one of you to look over and bless https://review.openstack.org/#/c/98517/ when you get a chance?18:23
SlickNikIt's the first step towards trying to get away from using third party rdjenkins for trove testing. Thanks much!18:23
*** timrc-afk is now known as timrc18:24
clarkbpelix: there is no good way of solving that aiui because we are dealing with internal "apis" that jenkins and plugins don't advertise18:24
clarkbpelix: JJB cannot know what version to supply18:24
*** e0ne has joined #openstack-infra18:24
clarkbat least that has been the previous determination18:24
clarkbSlickNik: I am working through my backlog now18:24
clarkbshould get to that soonish18:24
pelixthere is a well defined api to retrieve the plugin version from jenkins18:24
pelixclarkb: ^^18:25
clarkbpelix: but not the actual xml desired18:25
clarkbso there is always a lag18:25
pelixno, but you can then at least in the modules say 'if plugin_version < X'18:25
clarkbsure18:25
SlickNikclarkb: Thanks! It's not super-urgent, so whenever you get to it is fine.18:25
clarkbpelix: does python-jenkins support retrieving that data?18:26
clarkbpelix: if not that would be the first step in fixing this18:26
clarkbor fixing it as best as possible given what JJB can know18:26
*** lttrl has joined #openstack-infra18:27
*** lcheng has joined #openstack-infra18:28
openstackgerritBoris Pavlovic proposed a change to openstack-infra/config: Fix name of job in logaas  https://review.openstack.org/10098718:28
pelixclarkb: I don't believe it does directly, but jenkinsapi certainly does18:29
sdaguethere are days I really wish debs were atomic18:29
sdagueanyone have a cantrip for removing a deb without it running scripts?18:30
pelixclarkb: https://github.com/salimfadhley/jenkinsapi18:30
clarkbpelix: that won't work for us if the readme is correct18:31
pelixit's a little out of date18:31
pelixhttps://github.com/salimfadhley/jenkinsapi/blob/master/jenkinsapi/plugins.py18:31
clarkbsdague: pipe the list of files it installs to rm ?18:31
clarkbpelix: so probably best to add that functionality to python-jenkins18:32
sdagueclarkb: I need to recover the db though18:32
sdaguesudo dpkg --purge --force-all seemed to do the trick18:32
*** gfa has quit IRC18:32
*** terryw has joined #openstack-infra18:34
sdaguedhellmann: oslo.db isn't in d-g?18:35
*** otherwiseguy has quit IRC18:35
dhellmannsdague: it's on the list for them to do that, but it's possible it hasn't landed yet18:35
*** dims_ has quit IRC18:36
sdagueoh, the devstack patch didn't actually include installing it18:36
pelixclarkb: any particular reason not to have jenkinsapi as a dependency?18:37
jeblairmordred, clarkb, sdague: can you all please take a look at 97254 when you have a moment?  (storyboard search spec)18:37
*** marcoemorais has quit IRC18:37
sdagueyeh, it looks like victor missed that part18:37
*** jaypipes has joined #openstack-infra18:37
clarkbpelix: because it won't work for us18:37
clarkbpelix: our jenkins are far too new18:37
*** marcoemorais has joined #openstack-infra18:37
clarkbpelix: and we already use python-jenkins18:37
dhellmannsdague: ah, yeah18:37
clarkbwhich supports newer jenkins18:37
*** lukego has quit IRC18:37
sdaguedhellmann: you want to propose?18:38
dhellmannsdague: I can do that, so you can +218:38
sdagueyep18:38
*** lcheng has quit IRC18:38
sdaguedevstack : lib/oslo , it should be pretty clear18:39
dhellmannsdague: yeah, I'm looking to see what order to put it in18:39
*** jasond` is now known as jasond18:39
*** pelix has quit IRC18:39
sdagueI'll let you decide the order18:39
sdaguefor what you think makes sense18:39
jeblairyolanda: did you get an answer to your question?18:39
dhellmannsdague: where are the FOO_REPO variables defined? do I need to set one of those?18:40
clarkbSlickNik: reviewing that change. how new of mongodb do you need?18:40
dhellmannalso the branch variable18:40
sdaguedhellmann: stackrc18:40
sdagueI though they were there already18:40
clarkbSlickNik: if you have similar needs to ceilometer you won't be able to run mongodb until trusty18:40
dhellmannsdague: yeah, I was just checking to make sure18:40
clarkbjeblair: speaking of trusty maybe we can talk about dib soonish?18:40
jeblairclarkb: i talked to mordred and came up with a third way :)18:41
clarkbjeblair: even better :)18:41
*** basha has quit IRC18:41
dhellmannsdague: https://review.openstack.org/10099118:41
anteayaI'm around again and catching up18:42
anteayalet me know if there is anything urgent I can help with18:42
*** gokrokv__ has quit IRC18:42
jeblairclarkb: basically -- i'd like to reduce the node type specialization (we've been working toward that for a while).  having sudo on all nodes was part of that.18:42
jeblairclarkb: so if we focused on caching (like on the devstack nodes) and minimal package installation/configuration during the build, we can avoid dib complications from package installs18:43
jeblairclarkb: then have the unit test jobs install databases before they drop sudo18:43
clarkbjeblair: so I disagree on that front18:44
*** gfa has joined #openstack-infra18:44
jeblairclarkb: this does not avoid something that you wanted to avoid: wasting time installing databases on nodes (in fact, it makes it worse because it will happen _during_ the job run)18:44
*** eharney has quit IRC18:44
clarkbjeblair: in aggregate there is a significant waste of resources when you push things like that into the tests18:44
jeblairclarkb: however, i think the overall system simplification is worth it18:44
anteayapypi was unavailable for me for a moment, via the gui18:44
yolandajeblair, not yet18:45
clarkbjeblair: if we weren't running up to 25k tests per day and hitting quota and other rate issues with our cloud providers I would agree with you18:45
clarkbjeblair: but I think experience has shown that we actually need to care about this stuff18:45
jeblairclarkb: i do care.  i think it's part of test setup.18:46
bodepdthanks clarkb . It looks like the wheel of openstack-infra are starting to turn again :)18:46
*** doude has quit IRC18:46
clarkbjeblair: so that was just one issue, what about unbound, and the chroot vs kernel stuff?18:46
*** eharney has joined #openstack-infra18:46
jeblairclarkb: and having fewer node types means nodepool is better able to deal with high demand18:47
clarkbjeblair: and so I understand correctly you think installing dependencies for unittests is part of the unittest job?18:47
clarkbeverything from packages to configuration?18:47
yolandajeblair, what i wanted to do is to add the private key and public key paths into the diskimages elements, so we can copy the pub key to the authorized keys into the dib generated image, and we could connect to that using priv key in nodepool process18:48
yolandanot sure in using one keypair per image, or just add a setting to indicate a keypair for all18:49
*** markwash has quit IRC18:49
openstackgerritA change was merged to openstack-infra/config: Add puppet-openstack project puppet-openstacklib  https://review.openstack.org/9735718:49
jeblairclarkb: i think we should factor out the current precise/devstack-precise differences; i think that will be a fairly minimal install, yeah.  i'm imagining docs jobs that say "install the standard docs toolset" in their jjb definitions, rather than a huge list of everything every job in the system needs in puppet commented with "this is used by docs..."18:49
jeblairclarkb: i also think we should cache all the packages, so that network and runtime impacts are minimized.18:50
clarkbjeblair: sure we already do that for the devstack nodes18:50
clarkbwhich mostly sort of works18:50
*** annegent_ has joined #openstack-infra18:50
*** lcheng has joined #openstack-infra18:51
jeblairyolanda: i think that's handled by puppet right now.  are you working on the same thing mordred is?18:52
*** sandywalsh_ has quit IRC18:52
clarkbjeblair: so if we ignore runtime performance stuff and put that on the jobs. There are still a few things like unbound which have to be hacked around horribly in dib. While it works it is gross. Do we just eat the grossness?18:53
*** annegent_ has quit IRC18:53
yolandajeblair, more or less, but i'm working into the nodepool bits18:53
clarkbjeblair: and for the hypothetical but really world breaking problem of image types not being able to build due to incompatibility with kernels we deal with taht if we get tehre?18:53
yolandait's not really the same that we do at puppet, because the keys in puppet are used to connect to jenkins18:53
*** annegent_ has joined #openstack-infra18:54
yolandawhat we need to generate is a key to interact between nodepool and images generated by dib18:54
yolandajeblair, in the ones we generated by nova, we inject a key and we use that to connect between nodepool and the instance18:54
*** jistr has quit IRC18:55
yolandabut in the ones generated by dib, we needed to inject the keys at boot time, for each instance, or inject something in build time using dib, we are following the last approach18:55
yolandaso my idea was just to define a fixed keypair path on yaml file, and use that to inject with dib, and to connect18:55
*** BobBall is now known as BobBall_Away18:55
jeblairyolanda: i think i understand.  i don't think what we do now with nova is important in the dib world....18:56
yolandajeblair, but we wanted to preserve the ability to connect, mostly for periodical checks18:56
jeblairyolanda: the key you describe is only used to log into the "template server" when we start configuring it for the snapshot image18:56
yolandaright now, we cannot connect from nodepool to instances, so we cannot do the check for health18:56
yolandathat's mostly the worse issue18:56
*** blamar has quit IRC18:57
jeblairyolanda: now the process to create that image is dib18:57
jeblairyolanda: so dib (via our puppet) will install the jenkins key on it18:57
jeblairyolanda: and in both cases, now, and in the future, nodepool will do health checks using that key18:57
jeblairyolanda: nothing about that needs to change18:57
jeblairyolanda: to put it another way, the nova key you are describing is only used temporarily now; and it's only used for a function that dib is replacing.18:58
*** _nadya_ has quit IRC18:58
*** rlandy has quit IRC18:59
jeblairclarkb: so i think i summarized the approach mordred was working on as follows:18:59
yolandamm, so we discussed before with clarkb and mordred, and we arrived to the conclusion that nodepool needed to inject some key to control the nodes, rather than relying on the puppet jobs18:59
yolandawe were talking about using dib anyway, but have a custom element injected directly from nodepool19:00
*** ociuhandu has quit IRC19:00
jeblairclarkb: dib disables services, installs stuff, then re-enables services; then mordred has a second install phase with services enabled where ones that are safe to start in a chroot (and must be started in order for install to complete) are done19:00
yolandaso clarkb, mordred, what do you think¿19:00
*** dims_ has joined #openstack-infra19:00
jeblairclarkb: that seems like a reasonably generalized approach to handling the grossness.19:01
*** lcheng has quit IRC19:01
jeblairclarkb: and of course could be done instead of / regardless of the node de-specialization19:01
jeblairclarkb: he may still work on that, i'm not sure19:02
jeblairclarkb: i think being unable to build image types would be very bad and we should probably do early testing on that.  before we go all in, we ought to at least make sure we can build ubuntu and fedora nodes.19:03
yolandajeblair, so i still can continue running the readyscript bits, and the periodical checks for dib, trusting that the jenkins key will be there? if not, just consider that is not a valid one and destroy the node?19:04
jeblairyolanda: yes, that is the idea.  if nodepool can't log in with the jenkins ssh key, then neither can jenkins.19:04
clarkbjeblair: I agree on the early testing for the current situation. The hypothetical breakage comes from systemd's stance on kernel support19:04
clarkbjeblair: for example next fedora may not work on precise or trusty19:05
clarkbjeblair: which is why it is more hypothetical than actual.19:05
jeblairhopefully we will have some people with a crossover interest in fedora and dib to help dig into that if it happens19:06
yolandaok jeblair, i'll do a try with that19:06
yolandathx for the clarification19:06
*** openstackgerrit has quit IRC19:06
clarkbjeblair: the other side of the whole dib thing is do we continue to hold off trusty for dib?19:07
yolandajeblair, we should make clear on docs that if using DIB, an element that copies keys properly needs to be used, otherwise it won't work19:07
jeblairyolanda: i think you need to specify the username (jenkins) and private key location to nodepool for the check to work; see our production nodepool.yaml for an example19:07
jeblairyolanda: that's true for the current situation too -- it's not a dib thing.19:07
yolandajeblair, yes, i have it, and i had it working with nova19:07
yolandabut i was doing some basic testing with dib, just using ubuntu and vm, so it didn't have anything to copy the keys19:08
*** openstackgerrit_ has joined #openstack-infra19:08
jeblairclarkb: what is the alternative?19:08
*** openstackgerrit_ is now known as openstackgerrit19:08
clarkbjeblair: use the current process19:08
*** jaypipes has quit IRC19:09
*** dims_ has quit IRC19:09
clarkbjeblair: we can have nodes in ~an hour if we do that19:09
clarkband I think dib has been enough of a headache that we should consider it19:09
jeblairclarkb: our puppet is ready for it?19:09
clarkbjeblair: yes19:10
clarkbat least for devstack nodes19:10
clarkbI may not have tested the thick nodes19:10
jeblairclarkb: and all our providers have trusty images?19:10
jeblairclarkb: and our puppet "corrects" all of our provider images?19:10
clarkbrax has normal trusty images and hpcloud has partner trusty images but apparently partner images is all we are getting19:10
clarkbjeblair: what do you mean by "corrects"19:10
*** e0ne has quit IRC19:11
jeblairclarkb: the images we get have random variations; all the permutations need to work: hp-trusty, hp-devstack-trusty, rax-trusty, rax-devstack-trusty, tripleo-trusty...19:12
clarkbI have not tested all of the permutations19:13
*** terryw has quit IRC19:13
clarkbbut its teh same puppet as on precise so assuming they make the same variations (probably a bad assumption) then we should be fine19:13
*** annegent_ has quit IRC19:13
jeblairright, so is it worth someone spending time on that, or actually moving things forward with dib?19:13
clarkbjeblair: considering it is a few line change in nodepool.yaml I don't see why not19:14
*** lcheng has joined #openstack-infra19:14
clarkbwhereas dib has been and probably will be more weeks of work19:14
sdaguethere is also another consideration to put into the mix19:14
clarkbSlickNik: any answer on mongodb?19:15
sdagueis the set of bits created by dib close enough to what other folks are running, that we won't be debugging issues with the base images?19:15
jeblairclarkb: that's a shame.  i'm starting to think the time estimates for nodepool dib may not have been accurate.19:15
clarkbjeblair: they were not accurate at all and this is why I am frustrated by it19:15
*** gokrokve has joined #openstack-infra19:15
clarkband why I ahve made the statement that dib is a tool with a use case and ours isn't that19:15
jeblairsdague: i believe dib still starts with a base image from the os vendor; plus, one of the benefits of this is that we can supply the image itself to anyone who wants to use it directly.19:16
clarkbjeblair: we can still do that with our current image build process19:16
jeblairi am not sure how they will log into it.  but details.19:16
*** jasond has left #openstack-infra19:16
sdaguejeblair: ok, just checking19:16
*** e0ne has joined #openstack-infra19:16
jeblairclarkb: export the snapshot from glance?19:17
clarkbjeblair: that was another thing I realized while digging into dib. Is that we seem to have a set of issues with teh current process and we think dib fixes them. But really dib isn't a requirment for that19:17
clarkbjeblair: yes19:17
clarkbthe one thing dib would help is the build your own story19:17
jeblairclarkb: well, it's a neutral ground.  i _really_ want the same image on all our providers19:17
clarkbjeblair: we can have that with nodepool too19:17
*** markmc has quit IRC19:17
jeblairclarkb: and i don't want to have to use file injection on one, and dhcp on another...19:17
clarkbjeblair: build image in one rax region (or wherever), download, upload to remaining clouds19:17
*** julim has quit IRC19:18
jeblairclarkb: how do we stop using rax file injection?19:18
clarkbjeblair: I don't think you can19:18
clarkbbut if you enable dhcp and never get a lease you are fine19:18
clarkband file injection should just work right?19:18
jeblairclarkb: i don't see how it is possible to upload your own image and have that work with file injection19:18
jeblairclarkb: you need to know what files to inject19:19
clarkbjeblair: now I am confused. You are asking about disabling file injection but don't see how to enable it?19:19
reedGooooooooooooooooooooooooooooooooooooooooooooooooool19:20
clarkbjeblair: or if we back up. How is this different than with dib19:20
*** gokrokve has quit IRC19:20
jeblairclarkb: mordred suggested that configdrive is supported everywhere, so i'm presuming that specifying configdrive would disable file injection...19:20
clarkbjeblair: it is not supported everywhere19:20
reedspain's defense is awful19:20
clarkbit will be supported everywhere supposedly19:20
clarkbbut is not working on hpcloud right now19:21
jeblairclarkb: where is it not supported?19:21
clarkbsee scrollback from tteggel today19:21
jeblairclarkb: and yes, this is not a dib question, this is a user-supplied-glance-image question19:21
jeblairclarkb: you're referring to it being broken this week but back next?19:22
clarkbyes19:22
clarkbbut it was broken for "a long time" eg not supported until next week19:22
jeblairclarkb: that seems rather irrelevant to the conversation19:22
SlickNikclarkb: mongo 2.0.4 should work for us. Frankly though, I have mongodb there mostly as an example to make sure I've templated the job correctly and to test experimental support.19:23
SlickNikThere's still a couple of kinks we need to iron out with the mongodb tests themselves, and we're still working on them, so we're not looking at gating on it anytime in the next couple of weeks.19:23
*** ociuhandu has joined #openstack-infra19:23
bodepdzuul might be broken, it looks like it installs a new release of apscheduler19:23
bodepdI19:23
bodepdam trying to track down more info about how the install occurs19:23
clarkbSlickNik: ok your change lgtm I will approve19:23
clarkbjeblair: it is relevant if you intend on relying on configdrive19:23
bodepds/it/the zuul module/19:23
SlickNikclarkb: I'll definitely check-in with you guys before we plan on making mongo gating changes.19:23
clarkbjeblair: as it pushes our time to deployment out until at least next week19:23
jeblairclarkb: i promise i won't rely on it until next week19:23
SlickNikclarkb: Thanks!19:23
clarkbjeblair: assuming it actually gets fixed19:24
*** e0ne has quit IRC19:24
jeblairclarkb: i don't even know what we're talking about anymore19:24
clarkbjeblair: whether or not we should stop requiring dib nodepool to deploy trusty19:24
clarkbjeblair: we have wasted far too much t ime on this and I think we should just add trusty nodes19:24
*** jaypipes has joined #openstack-infra19:24
*** lcheng has quit IRC19:25
lifelessclarkb: oh, you want l to be 'totally unfiltered'? Thats easy enough to do.19:25
clarkblifeless: yes19:26
*** otherwiseguy has joined #openstack-infra19:26
jeblairclarkb: i think managing provider image differences is a huge burden, and even if some of the work is already done for trusty, i doubt we have done all of it.19:26
jeblairclarkb: doing that work may be less than dib at this point.  do you want to do it?19:27
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: double up [ for safety  https://review.openstack.org/10072819:27
clarkbjeblair: I would be happy to help debug issues. I have a hunch that most of them will be related to the trusty switch and not provider differences19:27
*** terryw has joined #openstack-infra19:28
lifelessclarkb: say so in the review as a note to me, I will get to it19:28
clarkbjeblair: we can also upload our own base image to all the clouds and boot off of that to create the snapshots as another alternative19:28
jeblairclarkb: i don't think part of our process should be "manually upload an image"19:29
*** _nadya_ has joined #openstack-infra19:29
clarkbjeblair: it doesn't have to be manual nodepool could do it19:29
clarkbjeblair: it doesn't support doing that today. Just pointing out it is an optiopn19:29
sdagueclarkb: removed the errant xtrace, I missed that one19:29
jeblairthat's an interesting idea19:29
*** hogepodge_ has joined #openstack-infra19:29
*** hogepodge has quit IRC19:29
*** hogepodge_ is now known as hogepodge19:29
jeblairclarkb: it seems very wrong though, to upload a stock image to 10 providers, then build an identical custom image on all 10 in order to snapshot it.19:31
clarkbjeblair: right which is why I also think we can build the image in one provider and copy it to the others19:31
jeblairclarkb: don't you think that we should be trying to have a single image across all of them?19:31
clarkbyup19:31
jeblairclarkb: why treat one specially though?  and which one?  that seems weird19:31
*** otherwiseguy has quit IRC19:32
clarkbjeblair: it wouldn't be special, it is just the tool used to make the image19:32
clarkbjeblair: this is how packer works. we could even use packer instead19:32
clarkband give people a bit more local reproduceability19:32
sdagueclarkb: or make it easy to run against a local devstack19:32
sdagueyou just need "a" openstack, right?19:33
clarkbsdague: yes19:33
clarkbjeblair: but I find myself in the use Vms for image builds not chroots camp19:33
clarkbjeblair: this is not the camp of lifeless and mordred19:33
openstackgerritA change was merged to openstack/requirements: Add nodeenv for horizon  https://review.openstack.org/9778819:33
*** mriedem has quit IRC19:33
jeblairclarkb, sdague: we don't need openstack/devstack to run a vm, it could just use libvirt...19:34
clarkbya packer will libvirt too19:34
sdaguejeblair: true, if it was nodepool it's already talking openstack19:34
clarkbwhich would be bad in nodepool VM because nested virt is bad19:34
clarkbbut could be good for people at home19:35
lifelessproblem with running libvirt19:35
mordredclarkb: or, and i'm sorry I'm slow - we could let me finish the patch I'm working on19:35
clarkbmordred: sure I am not working on this19:35
lifelessis that when you're in a cloud, qemu == slow, but we did time that and it was feasble19:35
clarkbmordred: I just awnt everyone to undersdtand there are alternatives and they are not crazy19:35
*** e0ne has joined #openstack-infra19:35
bodepdthe problem with zuul fresh installation via puppet appears to be this: https://gist.github.com/bodepd/8965088a2b3e3bc8f24519:35
clarkbbodepd: you need newer pip19:36
*** mrmartin has joined #openstack-infra19:36
openstackgerritcraigtracey proposed a change to openstack-infra/config: Add Octavia to Stackforge  https://review.openstack.org/10057619:36
clarkbbodepd: newer pip won't install a beta release unless you explicitly tell it to19:36
mordredbodepd: never install pip from distro packages btw - ALWAYS install pip from get-pip19:36
*** e0ne has quit IRC19:36
mordredbodepd: there is too much churn currently and the distro packages will hamstring you righ tnow19:37
openstackgerritA change was merged to openstack-infra/config: Added new experimental job for trove functional tests  https://review.openstack.org/9851719:37
*** e0ne has joined #openstack-infra19:37
openstackgerritA change was merged to openstack-infra/config: Use job-template for gate-trove-buildimage jobs  https://review.openstack.org/9968019:37
*** lcheng has joined #openstack-infra19:37
jeblairmordred: clarkb is impatient and would like to use trusty with our current process (since trusty images are on both providers now); when do you think a working dib system is feasible?19:38
*** mriedem has joined #openstack-infra19:38
jeblairmordred: do you think we should go ahead and try trusty the "old way" while continuning to work on the new way?19:39
bodepdthanks mordred clarkb : darn , I had added a pip package install to the pip module. I found the real install process in install_puppet.sh19:39
mordredsoon. however, I'm personally not opposed to manually uploading a base trusty image so that work can start on debugging what in devstack might be broken19:39
jeblairmordred: i understand that no manually uploading is necessary19:39
jeblairmordred: as there are trusty images in both clouds19:39
mordredoh. neat. then yeah, I think there is no reason to not start working with those while we work on finishing the other things for sure19:39
*** e0ne has quit IRC19:40
jeblairclarkb: okiedokie then :)19:40
clarkbcool19:40
clarkbI will whip up a change to add some nodes after lunch19:41
*** gokrokve has joined #openstack-infra19:42
*** blamar has joined #openstack-infra19:43
sdaguewell, for what it's worth I'm basically only using trusty for devstack and grenade locally, so it's working for my normal through path. I'm sure we'll find some edge conditions, and smoking those out soon would be good.19:43
reedSpain, unrecognizable19:44
clarkbreed: this is great19:44
clarkbjeblair: the other thing was zuul restart should happen at some point. Do you have a preference for when that happens?19:44
reedclarkb, their defense is asleep, badly positioned19:44
jeblairclarkb: nope19:45
*** ramashri has quit IRC19:45
devanandajeblair: hi! what do I need to do to be added to stable-maint gerrit group, eg. so that I, too, can +A patches to the stable branch of ironic19:46
*** mmaglana has joined #openstack-infra19:46
sdagueso the ubuntu package mirror thing is interesting19:46
sdaguehttp://logstash.openstack.org/#eyJmaWVsZHMiOltdLCJzZWFyY2giOiJtZXNzYWdlOlwiRTogVW5hYmxlIHRvIGZldGNoIHNvbWUgYXJjaGl2ZXMsIG1heWJlIHJ1biBhcHQtZ2V0IHVwZGF0ZSBvciB0cnkgd2l0aCAtLWZpeC1taXNzaW5nP1wiIEFORCBOT1QgKGJ1aWxkX3F1ZXVlOmNoZWNrLXRyaXBsZW8gT1IgYnVpbGRfcXVldWU6ZXhwZXJpbWVudGFsLXRyaXBsZW8pIEFORCB0YWdzOlwiY29uc29sZVwiXG4iLCJ0aW1lZnJhbWUiOiI2MDQ4MDAiLCJncmFwaG1vZGUiOiJjb3VudCIsIm9mZnNldCI6MCwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwibW9kZSI6InNjb3JlIiwi19:47
sdagueYW5hbHl6ZV9maWVsZCI6ImJ1aWxkX25vZGUifQ==19:47
jeblairdevananda: the stable maintainers are responsible for ironic yet, right?19:47
jeblairdevananda: the stable maintainers aren't responsible for ironic yet, right?19:47
jeblair(typo)19:47
clarkbjeblair: ok I may take a stab at that this evening19:47
devanandajeblair: i dont know who is responsible for it. I do not have +A powers on that branch.19:47
sdaguehmmm... short url - http://goo.gl/Te5AR119:47
devanandajeblair: I would be doing it if I could ...19:47
*** lcheng has quit IRC19:48
mrmartinre19:48
reedguys, can we get a couple of eyes review this please? https://review.openstack.org/#/c/99481/19:48
sdaguein that it's mostly rax ord failing19:48
jeblairdevananda: i don't know off hand if they only manage releases for integrated projects, or if they do incubated too.19:49
jeblairdevananda: i would ask ttx or apevic19:49
mordredjeblair: I think only integrated - we maybe should put ironic-core as the people owning stable* for ironic19:50
*** pdmars has quit IRC19:50
jeblairdevananda: and for the branches of ironic they do not manage, then we can do an acl change for those ironic branches19:50
*** e0ne has joined #openstack-infra19:50
*** e0ne has quit IRC19:50
mordreddevananda, jeblair: also, adam_g is stable-maint I think, no?19:50
devanandaI'll rephrase: there is a stable/icehouse branch of ironic in gerrit. I would like the ability to +A patches on that branch. stable-maint currently has the ability to do that, which i'm fine with.19:50
jeblairmordred: i think we should enumerate the branches, but yes19:50
devanandayes, adam_g is19:50
mordredjeblair: ++19:50
*** e0ne has joined #openstack-infra19:51
*** rm_work is now known as rm_work|away19:51
*** julim has joined #openstack-infra19:51
clarkbeglynn-regus: are you around?19:52
* clarkb just responds to ml and hopeflly not too many people get mad19:52
sdaguedevananda: I think the point being made is that stable branches aren't project owned19:53
sdaguethey are owned by stable-maint team19:53
devanandasdague: oh, my misunderstanding then. i thought they were largely ptl managed.19:53
sdaguedevananda: nope19:53
devanandai'm totally fine if other folks want to (and will stay on top of) managing ironic's stable/icehouse branch19:54
sdaguebecause the stability of openstack as a whole passes to that second team19:54
jeblairdevananda: it's an exclusive OR.  Either stable-maint has jurisdiction on the branch or ironic-core does.  i think ironic-core gets all pre-incubated, and i think all pre-graduated branches.19:54
mordredbodepd: I honestly think we should update upstream puppet anywhere that it ensure's pip is installed to always install from get-pip19:54
mordredbodepd: but I might get shouted down ...19:54
devanandasdague: ack. that makes sense.19:54
jeblairdevananda: it's that last bit i don't recall right now -- whether they want the incubated branches or not.19:54
devanandaso more complete context is taht pecan has been trying to gate thir stable branch on ours, since, well, we co-gated during icehouse19:55
reedmordred: ping (pm)19:55
*** e0ne has quit IRC19:55
*** pdmars has joined #openstack-infra19:55
*** timrc is now known as timrc-afk19:56
devanandajeblair: there are no pre-incubated stable branches of ironic.19:56
devanandajeblair: so the question is whether stable-maint wants to take on stable/icehouse for ironic, or wants us to do that.19:56
jeblairdevananda: sounds like it.  let's try to get an answer from them on that.19:57
*** shayneburgess has quit IRC19:57
*** ryanpetrello has joined #openstack-infra19:57
devanandajeblair: ++19:57
*** shayneburgess has joined #openstack-infra19:59
jeblairit's way past my lunchtime, gotta run19:59
reedthat reminds me, i'm hungry too19:59
*** timrc-afk is now known as timrc20:00
*** lcheng has joined #openstack-infra20:00
*** e0ne has joined #openstack-infra20:01
*** sarob has joined #openstack-infra20:03
*** terryw has quit IRC20:04
dougwighi infra team.  i sent a request for a gerrit service account last week, but didn't hear back.  was there something wrong with the request, or anything else that i need to do?20:05
anteayadougwig: link to the request?20:05
anteayaSergeyLukjanov: addressed some of them today20:05
*** blamar has quit IRC20:05
dougwighttp://lists.openstack.org/pipermail/openstack-infra/2014-June/001312.html20:06
*** e0ne has quit IRC20:06
dougwigif i messed up the naming policy, anything can be renamed and i'll adjust my end.20:06
JayFHmm. If I want a gerrit service account for a project on Friday, should I request it today?20:06
openstackgerritSean Dague proposed a change to openstack-infra/config: add devstack-vagrant project to openstack-dev  https://review.openstack.org/9683520:07
sdagueclarkb: thanks for review, respun20:07
anteayadougwig: no, you have given us enough to work with, he must have missed it on his latest pass20:07
dougwigno problem, thanks.  it's for a neutron CI system, if that helps anything.20:07
anteayaI don't have gerrit permissions to set up new accounts, else I'd do it for you20:07
clarkbJayF: you should have requested ti last week :)20:08
anteayathanks, yeah the name looks like it fits with our current naming policy, which I have to submit a patch to publish20:08
dougwigshould i resend, or is it in the queue, so to speak?  :)20:08
pleia2they'll get to it eventually20:08
anteayadougwig: it is in the queue, don't resend20:08
pleia2looks like it was just sent on friday, sometimes it takes a few days :)20:08
JayFclarkb: :( If I request it now, and say pretty please, is there any way it could be fast-tracked? I'm intending on setting up some third-party CI for ironic-python-agent20:08
anteayahey pleia220:08
pleia2o/20:09
anteayadougwig: you are welcome to check in with me though, I'll keep an eye out for you20:09
clarkbJayF: I was mostkly referring to dougwig's question. Its definitely possibly but we have been swamped and sick and stuff20:09
dougwiganteaya: thanks.  :)20:09
anteayaJayF: everybody wants it fast tracked20:09
JayFI know it's not been a good couple of weeks for you guys20:09
anteayaJayF: the best thing you can do is name it properly, I'll get you the format20:09
JayFHonestly, I'm just not used to user account creations like this being a manual process. I can do most things via an API :P20:10
clarkbJayF: it has to do with gerrit being silly20:10
anteayaJayF: lines 11-16: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format20:10
*** sarob has quit IRC20:10
anteayaJayF: follow those and it should only take one email to get it created20:10
clarkbJayF: also I think we have decided there needs to be oversight20:10
anteayano kidding20:10
clarkbotherwise the whole thing melts down quickly :/20:10
pleia2JayF: these aren't regular user accounts, they need to be reviewed manually and added to specific groups (and it's a pretty new process, so it has some bumps)20:10
anteayaI think we are above 60 accounts now20:11
*** lcheng has quit IRC20:11
JayF-bot vs -ci matter much? In this case it'll initially be making and providing public builds, but I could easily see it moving to being true tempest testing and voting in the future20:11
anteayaJayF: yes20:11
anteayaif it will ever be voting it is a -ci20:11
anteayaif it will never ever be voting it is a -bot20:11
* krotscheck _almost_ pities spain.20:12
dougwigkrotscheck: lol20:12
anteayaman, when I win the etherpad colour lottery, I never want to power down my laptop again20:12
krotscheck_almost_20:12
JayFanteaya: so rackspace-ipa-ci is good? Or would you rather rackspace-ironic-python-agent-ci20:13
anteayaI can never get the good colours back20:13
JayFanteaya: or just rackspace-ironic-ci?20:13
sdagueanteaya: use gcolor2 to figure out what color it is20:13
anteayaJayF: rax-ipa-ci?20:13
anteayasdague: I can get the hex number for it20:13
JayFanteaya: we're... discouraged from using 'rax' externally as it's our stock ticker20:13
anteayaJayF: rackspace-ipa-ci20:13
anteayaand understood20:14
anteayabut I dont' know what to do with the hex number20:14
*** blamar has joined #openstack-infra20:14
anteayais there a spot I can input the hex value in etherpad?20:14
*** rcarrill` has joined #openstack-infra20:14
anteayaJayF: we discourage using the program name in the ci name20:14
anteayasince you might branch out and test other things20:15
JayFso 'thing it is testing' in that etherpad wouldn't be ironic then? interesting.20:15
anteayano20:15
JayFI'm very OK with using -ipa- though20:15
anteayaipa20:15
anteayayou are testing your ipa thing20:15
anteayaif you were testing ironic the test would be in the gate20:15
*** jaypipes has quit IRC20:15
mordredkrotscheck: I mean, pity is ... not quite the right word20:15
JayFWhat we're doing for now only, is building agent images for every agent merge req20:15
anteayawhat is ipa?20:16
*** rcarrillocruz has quit IRC20:16
anteayait is some super secret rackspace thing?20:16
krotscheckmordred: Yeaaaah…. I dunno. Weren’t they supposed to be defending champions or something?20:16
anteayaJayF: because if you aren't testing something that is a secret or specific hardware, the test should be in the gate20:17
JayFanteaya: ironic-python-agent20:17
anteayathe name (optimally) should say why you are not testing in the gate20:18
anteayaso why are you not testing in the gate?20:18
JayFanteaya: in this case, it's not testing (yet), it's building agent images.20:18
anteayawhat is stopping you from putting this in the gate?20:18
JayFanteaya: our preferred ironic-python-agent image uses CoreOS+containers, and requires dependencies that we didn't believe we could get in devstack20:18
mordredkrotscheck: they ... do not look good20:18
JayFanteaya: so in this case, we're going to do it as a third-party CI and for devstack-testing will add support for DIB images20:19
clarkbJayF: uhm out of curiousity does that even resebmle how it would be used in the real world?20:19
krotscheckmordred: But hey, Netherlands!20:19
clarkbJayF: I don't imagine you would deploy coreos containers to bare metal20:19
JayFclarkb: it's /exactly/ how we run it in our enviornment20:19
JayFclarkb: this is the deployment ramdisk20:19
clarkbJayF: o_O20:19
clarkbJayF: right so why containers20:19
clarkbCoreOS I can kind of see20:19
JayFhttps://github.com/openstack/ironic-python-agent/tree/master/imagebuild/coreos20:20
anteayaso SergeyLukjanov when you read backscroll, it looks like this one got missed: http://lists.openstack.org/pipermail/openstack-infra/2014-June/001343.html20:20
clarkband fwiw you can do containers all day on our test nodes...20:20
clarkbI do not know why people keep acting like this is impossible20:20
anteayaSergeyLukjanov: thanks for doing the others today though :D20:20
JayFclarkb: so Docker is a-OK to use as a build dependency in your nodes?20:20
clarkbno docker is not20:20
clarkbcontainers are20:20
*** lttrl has quit IRC20:20
*** e0ne has joined #openstack-infra20:20
clarkbdocker <> containers20:20
clarkbJayF: also I am going to spin up trusty nodes today and start working on that20:21
clarkbthis will give you docker too20:21
JayFclarkb: we use docker to build the container. You should look at the code, it's actually pretty interesting :). I will be blogging about exactly how we're using those images right now.20:21
*** elitenudel2500 has joined #openstack-infra20:21
*** annegent_ has joined #openstack-infra20:21
*** timrc is now known as timrc-afk20:21
JayFclarkb: we're willing to set it up, today (well, Friday), via third-party CI, in order to help Ironic devs working on the agent get working agent images for 'free'20:21
*** elitenudel2500 has quit IRC20:23
*** lukego has joined #openstack-infra20:24
*** lcheng has joined #openstack-infra20:24
clarkbJayF: sure I don't have any issues with third party stuff. It is a bit frustating sometimes when we could collaborate and get this done upstream but instead we end up with lots of people in corners20:24
jrollclarkb: rackspace has driven the IPA work. today we do not have a diskimage-builder for this, because our team will never use it. there will be one eventually, which will be used in devstack/tempest testing.20:25
JayFclarkb: If you get to a point where I can use docker in devstack as a build requirement, I'll very very happily add support for building the agent image directly into devstack20:25
JayF(well I might not be happy while I do it, but I'll be happy to do it anyway :D)20:25
clarkbJayF: right I am saying you will haev that today20:25
jrollclarkb: in the meantime, we would like to provide "gold master" coreos images for each build, that others may use for working on / testing IPA.20:25
*** annegent_ has quit IRC20:26
jrollclarkb: our efforts to do this as third-party CI are driven by: 1) docker does not exist in gate nodes today; and 2) our team has capacity we're willing to use for this.20:26
JayFclarkb: that's pretty awesome then. If that's the case, then I'll integrate with devstack instead. Although I thought in some ways you guys might also appreciate us taking on the load of building them as well.20:26
clarkbjroll: right so for item 1 you could help make it available because it brings lots of other good stuff20:27
clarkblike I said I am happy for third party stuff20:27
clarkbit enables a lot of flexibility and so on20:27
JayFclarkb: My only other question is would you guys be OK running, in 'check' testing (IDK the proper name) a job that would build a docker image, and provide it for download for Ironic devs?20:27
clarkbbut it also seems to have become a default and promote less collaboration20:27
JayFs/docker/ipa/20:28
clarkbJayF: yes I think the trove folks are doing something similar20:28
jrollclarkb: sure - what can I do to help? :)20:28
*** blamar has quit IRC20:28
*** melwitt has joined #openstack-infra20:28
JayFclarkb: to be blunt; I'm very, very surprised you guys are OK with having docker in devstack. Hearing that makes my heart grow two sizes, and makes me slightly less of a grinch :)20:28
clarkbwhere they build trove images and upload them to tarballs.o.o iirc20:28
clarkbJayF: well I have nothing to say about devstack20:28
clarkbJayF: but you can run docker on the test nodes20:28
JayFclarkb: I'm confused then, I thought all test nodes ran Devstack?20:29
clarkbJayF: they don't20:29
clarkbJayF: we basically give you root and say have fun20:29
clarkbmost people choose to use devstack because the work is done for them20:29
JayFclarkb: can you tell me a place I can go look at an example for arbitrary testing being done?20:29
clarkband you can extend devsatck with plugin like things20:29
clarkbsure20:29
JayFDevstack only supported precise last time I looked, which is not compatible with Docker without a kernel upgrade20:29
JayFsounds like that's changed though20:30
*** ramashri has joined #openstack-infra20:30
clarkbJayF: sdague reports devstack works on trusty just fine20:30
mordredJayF: we're working _today_ on adding trusty to infra20:30
*** markwash has joined #openstack-infra20:30
JayFawesome, that was not the case in April :)20:30
*** mrmartin has quit IRC20:30
sdagueJayF: devstack supports a ton of things :)20:30
clarkbwell it wasn't released until the end of aprikl...20:30
*** melwitt1 has joined #openstack-infra20:30
*** maxbit has joined #openstack-infra20:30
clarkbJayF: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/requirements.yaml#n120:31
sdaguehttps://github.com/openstack-dev/devstack/blob/master/stack.sh#L14520:31
clarkbJayF: that is a mostly arbitrary thing run in the context of devstack20:31
clarkbJayF: the runs in the context of devstack as a cheap dependency resolver iirc20:31
*** eharney has quit IRC20:31
mordredhttps://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/trove-images.yaml too20:32
clarkbbut that shell: can do whatever it wants20:32
mordredJayF: ^^20:32
clarkbmordred: it just moved to trove.yaml20:32
clarkbhttps://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/trove.yaml20:32
mordredoh. bother20:32
clarkbmordred: sorry :)20:32
* mordred stabs clarkb with a wet cat20:32
SlickNikyeah that was me :)20:33
*** eharney has joined #openstack-infra20:33
* mordred stabs SlickNik with a wet cat20:33
*** melwitt has quit IRC20:33
*** annegent_ has joined #openstack-infra20:33
clarkbjroll: as far as what to do to help, I am going to add images to nodepool asap. Assuming those build fine (they should this was tested like last week), next step is to start running jobs on them20:34
*** julim has quit IRC20:34
clarkbjroll: JayF so having some trusty tests would help20:34
* SlickNik lobs a water filled klien bottle at mordred20:34
SlickNik*klein even20:34
*** amalagon has joined #openstack-infra20:34
JayFclarkb: the downside of you guys supporting all this properly is that instead of doing the semi-fun thing I was going to do Friday in setting up the 3rd party CI, I'll be hacking inside devstack instead :P20:35
*** lcheng has quit IRC20:35
JayFnot a bad tradeoff for being less of a rogue20:35
mordredwe're evil like that20:35
*** ramashri has quit IRC20:35
*** shayneburgess_ has joined #openstack-infra20:36
*** ramashri has joined #openstack-infra20:36
boris-42mordred Hi could you please create groups logaas-ptl and logaas-core, thanks20:36
*** shayneburgess has quit IRC20:36
*** shayneburgess_ is now known as shayneburgess20:36
jrollclarkb: what sort of trusty tests? things that are known to work on trusty, or that only work on trusty, or?20:36
*** ramashri has quit IRC20:37
clarkbjroll: things that only work on trusty may be a good start simple because we don't have to migrate anything20:37
*** ramashri has joined #openstack-infra20:37
clarkbjroll: most of the work once we have nodes will be in migrating juno off of precise20:37
*** lcheng has joined #openstack-infra20:37
jrollclarkb: cool, I'll see if there's anything I can come up with besides JayF's work20:38
*** lukego has quit IRC20:38
*** melwitt1 has quit IRC20:39
*** crc32 has quit IRC20:39
*** melwitt has joined #openstack-infra20:39
openstackgerritAnita Kuno proposed a change to openstack-infra/config: Adds Third Party CI account name format directions  https://review.openstack.org/10101320:39
JayFjroll: you should help me with it20:41
JayFjroll: then it becomes JayF and jroll's work20:41
JayF:D20:41
mordreddude. he just fell over20:42
* jroll runs20:42
* jroll rolls away, rather20:42
openstackgerritAnita Kuno proposed a change to openstack-infra/config: Adds Third Party CI account name format directions  https://review.openstack.org/10101320:43
*** elitenudel2500 has joined #openstack-infra20:44
*** otherwiseguy has joined #openstack-infra20:49
*** jp_at_hp has joined #openstack-infra20:52
zarogertty don't like /me20:53
openstackgerritClark Boylan proposed a change to openstack-infra/config: Add trusty to nodepool image list  https://review.openstack.org/10101820:53
openstackgerritAnita Kuno proposed a change to openstack-infra/config: Adds Third Party CI account name format directions  https://review.openstack.org/10101320:53
clarkbmordred: jeblair ^ that should add it to nodepool20:53
*** doude has joined #openstack-infra20:53
clarkbif you guys want to merge that I can pay attention to image builds and do manual runs of unittests and stuff20:53
clarkbthen as a follow up change we can soemthing like experimental jobs that run on trusty20:54
*** miqui has quit IRC20:54
mordredclarkb: +220:54
mordredwell, THAT wasn't what I was expecting20:55
clarkbmordred: aroo?20:56
*** blamar has joined #openstack-infra20:56
*** mwagner_lap has quit IRC21:00
*** yamahata has joined #openstack-infra21:00
sdagueclarkb: can we stack an experimental devstack and devstack-gate change on top of that?21:01
clarkbsdague: sure21:01
* clarkb does tis21:01
sdaguethanks sir21:01
e0nehi all! could anybody explain me why we reset git remote origin here https://github.com/openstack-infra/devstack-gate/blob/master/functions.sh#L235, please? It overrides current repo remote origin. I'm trying to configure git_base during repo cache creation(using https://review.openstack.org/#/c/100547/) and don't want to reset origin remote later.21:01
*** _nadya_ has quit IRC21:01
*** _nadya_ has joined #openstack-infra21:02
e0nei'm not good known with infra, but i'm not sure that configuring git_base in two different places is the best solution21:02
*** rfolco has quit IRC21:04
morganfainbergoooh expirimental on trusty21:04
jeblairclarkb: have you downed any jenkins nodes for the plugin upgrade?21:05
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Update readme with more guidelines for writing queries  https://review.openstack.org/10094121:05
*** dims_ has joined #openstack-infra21:06
*** denis_makogon has quit IRC21:06
jeblairclarkb: i take that as a no; i'll put jenkins07 in shutdown21:07
openstackgerritClark Boylan proposed a change to openstack-infra/config: Add experimental tempest and grenade trusty jobs  https://review.openstack.org/10102321:07
clarkbjeblair: have not yet21:07
clarkbjeblair: and roger21:07
*** _nadya_ has quit IRC21:07
jeblairjenkins07 is in shutdown mode21:08
*** jamielennox is now known as jamielennox|away21:08
clarkbsdague: 101023 similar to what you had in mind?21:08
sdagueclarkb: yep21:09
krotscheckWell, that settles group B21:09
*** virmitio has quit IRC21:11
*** virmitio has joined #openstack-infra21:12
*** sweston has joined #openstack-infra21:12
*** sweston has quit IRC21:14
*** sweston has joined #openstack-infra21:14
mroddenso i have a fun project rename for bash8 that I was wondering how much work it would be to do (https://bugs.launchpad.net/bash8/+bug/1331548)21:16
uvirtbotLaunchpad bug 1331548 in bash8 "rename bash8" [Medium,Triaged]21:16
mroddeni dont actually own the pypi namespace, since that was created after the fact21:16
clarkbmrodden: I think on principle of what I said this mornign I should -2 that :)21:16
clarkbbut I defer to mordred and jeblair because they appreciate debians concerns more than I do21:16
mroddenclarkb: must have missed that?21:17
clarkbmrodden: I do not want to rename projects for sillyreasons21:17
mroddeni agree...21:17
openstackgerritA change was merged to openstack-infra/config: Add trusty to nodepool image list  https://review.openstack.org/10101821:17
clarkbimo debian saying the name you chose is bad is a silly reason21:17
mroddeni think its weird they actually would build a .deb for it TBH21:17
mroddenbut to each their own21:18
*** yamahata has quit IRC21:18
jeblairclarkb: i tend to think the concerns raised are legitimate; i'm not sure how we'd feel about someone writing "nova8".21:18
*** dkranz has quit IRC21:18
sdagueyeh, the namespace issue I found to be a reasonable enough arguement21:18
lifelessthey're building a deb because they want to make it available for people21:18
*** yamahata has joined #openstack-infra21:18
lifelessand yeah the concerns are legitimate IMNSHO21:18
*** mmaglana has quit IRC21:18
jeblairclarkb: so "because debian said so" may be silly, but "because the points raised by people who happened to be debian mantainers are good" is less silly21:18
clarkbjeblair: I must've mised the good points then21:19
sdaguejeblair: well most of their points were silly, the namespace one was the only valid one :)21:19
clarkbare yo uguys talking about how debian unfortunately puts versions in names?21:19
jeblairclarkb: you are in rare form today :)21:19
clarkbjeblair: well its not my fault debain does things like ruby1.9.1 is actually ruby 1.9.321:19
*** dprince has quit IRC21:19
jeblairclarkb: no, the "bash8" is to "bash" what "python3" is to "python"21:19
sdagueclarkb: in that bash8 as a binary might be confusing to people being bash is also on their system21:19
*** elitenudel2500 has quit IRC21:19
*** dprince has joined #openstack-infra21:20
dtroyer_zzpackaging it "because it's there" seems silly to me…where is devstack.deb?21:20
jeblairoh, we lost the project renames section... one sec21:20
*** dims_ has quit IRC21:20
*** eharney has quit IRC21:20
clarkbI see.  Ithought it was more of the package name space21:20
clarkband had ruby on my mind21:20
*** dims_ has joined #openstack-infra21:21
jeblairmrodden: put it on https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting21:21
clarkbso on that note do we make harlowja rename doc8?21:21
jeblairmrodden: and when we get a few project renames batched up (or if it sits there for a while), we'll schedule it21:21
clarkbor is that ok because `doc` isn't bash21:21
jeblairclarkb: no one here is making anyone rename anything21:22
mroddenjeblair: ok sounds good, thanks21:22
harlowjaclarkb if we must :-P21:22
harlowjadocate?21:22
anteayadtroyer_zz: do let me know when you are ready to have the 'what do we need to get the devstack docs into infra' discussion21:22
jeblairmrodden: put your name next to it and we'll ping you for the scheduling21:22
sabeen1Hi All - I'm trying to change the port for the keystone endpoint in devstack, but have not been successful so far.. i've tried changing the port irc://chat.freenode.net:6667/# in openrc and in keystone.conf but it still remains at 5000 .. does anyone know how to do that?21:22
anteayadtroyer_zz: because I don't know if you are really asleep or not21:22
jeblairmrodden: or at least try to :)21:22
harlowjaclarkb rst8, rstate21:22
clarkbharlowja: I don't really care :)21:22
dtroyer_zzanteaya: I'm here…znc needs some TLC it would appear.    tomorrow would be fine21:23
zarowhat is the deal with gertty?  i can't get any projects to show up on my install21:23
clarkbbut maybe in general this is a good rule even when there are no known suffixes to prevent future collisions21:23
anteayadtroyer_zz: okay I will try to find you tomorrow21:23
clarkbzaro: did you subscribe to any?21:23
harlowjaclarkb i don't really care either, haha21:23
anteayaI hope I remember21:23
zaroclarkb: none to subscribe to21:23
clarkbzaro: even if you hit 'l' to list them/21:23
zaroclarkb: yep21:23
zaroonly one i see is 'all-projects'21:24
clarkbsabeen1: last time I looked at it the devstack scripts didn't actually properly change the port21:24
sdaguejeblair: so, question, that's just for the git tree rename right?21:24
clarkbsabeen1: morganfainberg may have insight because he has been poking at keystone in devstack recently21:24
sdaguewe could restructure inside the repo for the new name well in advance of the gerrit change right21:24
sabeen1clarkb: ok, thanks!21:24
jeblairsdague: yep21:25
sdaguethe other question I think was pypi21:25
sdaguebecause this is an artifact out there already21:25
clarkbfor pypi I think you create the new name and leave the old one21:25
sdagueok21:25
clarkbwith maybe a pointer on the old one to say this is purely for backward compat please see bashate21:25
*** weshay has quit IRC21:25
clarkbdstufft: ^ does pypi have an officia lway to handle this?21:25
jeblaircould also delete after a certain amount of time21:25
clarkblike automagic redirects?21:26
dtroyer_zzsabeen1: you want to change the 'public' listen port?   put KEYSTONE_SERVICE_PORT=1234 in local.conf21:26
bodepdmordred: that would be hard to enforce21:26
*** dims_ has quit IRC21:26
*** jergerber has quit IRC21:26
sabeen1dtroyer_zz: yes, ok i'll try that - thank you!21:26
harlowjaclarkb any good name suggestions?21:27
clarkbharlowja: no I am terrible at naming things21:27
*** fifieldt has quit IRC21:27
*** mfer has quit IRC21:27
harlowja:)21:27
harlowjaya, me too, haha21:27
clarkbe0ne: the reason we reset the origin there is after the clone on the previous line the origin is local21:27
clarkbe0ne: but we want the origin to be remote21:28
*** srenatus has quit IRC21:28
clarkbarg new project creation isn't working21:30
clarkbhttps://review.openstack.org/#/admin/projects/stackforge/cachemonkey,access should haev stuff in it. debugging now21:30
e0neclarkb: i don't see it:(. i'll try to debug21:30
ryanpetrelloanybody aware of any ongoing issues w/ SCP connecting to static.openstack.org?21:31
ryanpetrelloe.g., https://jenkins05.openstack.org/job/gate-pecan-tox-storyboard-tip/1/console21:31
ryanpetrellolooks like the pip dependency install failed too21:32
clarkbryanpetrello: I think scp is fine21:32
jeblairryanpetrello: i don't see a connection problem, i see that the expected output is not there21:32
clarkbit just couldn't find those files to copy them21:32
mroddenclarkb: jeblair: i dont own the pypi namespace for bash8, since it was registered under the OpenStack creds, so i'm not sure how to handle that21:33
*** dkehn_ has joined #openstack-infra21:33
jeblairmrodden: don't worry about it for now; do you know the new name you want?21:33
clarkbFetchConfigException in jeepyb for cachemonkey21:34
mroddenjeblair: pretty sure we had an agreement on bashate21:34
ryanpetrelloclarkb, jeblair: what’s curious is that this same test just passed an hour ago: http://logs.openstack.org/82/97282/1/gate/gate-pecan-tox-storyboard-tip/61d2fd2/console.html21:34
*** dkehnx has quit IRC21:34
mroddenseems to be the most popular anyways21:34
ryanpetrellos/an hour ago/20 minutes ago :)21:34
openstackgerritA change was merged to openstack-infra/gear: Use Job repr instead of handle in Client debug log  https://review.openstack.org/8659621:34
*** dkehn has quit IRC21:34
mroddenpeople didn't like my idea of megawaffles, not sure why21:34
clarkbmrodden: I would vote for megawaffles21:35
clarkbbut I haven't contributed to bash8 so don't have a vote21:35
jeblairmrodden: okay, i registered bashate21:35
mroddenjeblair: cool thanks21:35
*** dkehn has joined #openstack-infra21:36
*** mmaglana has joined #openstack-infra21:36
openstackgerritA change was merged to openstack-infra/gear: doc update: move the server usage info  https://review.openstack.org/8840121:36
*** mmaglana has quit IRC21:36
*** mmaglana has joined #openstack-infra21:37
openstackgerritA change was merged to openstack-infra/gear: Remove print statement  https://review.openstack.org/9305721:37
sdagueclarkb: I do like mordred's idea that we should rename elastic recheck - sunspots21:37
sdagueonce we get to a thing that might be installable21:37
*** packet has quit IRC21:38
anteayaha ha ha21:38
*** smarcet has quit IRC21:38
*** changbl has quit IRC21:38
*** mrda-away is now known as mrda21:38
ryanpetrelloclarkb, jblair: I ran the check again w/ `recheck no bug` and it passed this time21:39
*** hogepodge has quit IRC21:40
clarkbryanpetrello: you may have intertest conflicts21:40
clarkbor some orther racy thing causing trouble21:40
*** wenlock has quit IRC21:40
*** fifieldt has joined #openstack-infra21:40
ryanpetrellothe tests being run are storyboard tests ;)21:40
morganfainbergsabeen1, dtroyer_zz changing the keystone port to what?21:41
*** wenlock has joined #openstack-infra21:41
sdagueryanpetrello: that's the pypi mirror connectivity bug21:41
morganfainbergsabeen1, dtroyer_zz, ah those variable / conf should work to change the ports21:42
*** Abhishek has joined #openstack-infra21:42
morganfainbergbut not sure how many other things could be affected when doing that21:42
ryanpetrellosdague: bug #?21:42
*** Abhishek has quit IRC21:42
sabeen1morganfainberg: ok, i'm trying that right now21:43
*** Abhishek has joined #openstack-infra21:43
sdagueryanpetrello: Bug 1268725 is close, though I don't know if we have another one for this21:43
uvirtbotLaunchpad bug 1268725 in openstack-ci "ConnectionError: HTTPConnectionPool(host='pypi.openstack.org', port=80): Max retries exceeded" [Medium,Triaged] https://launchpad.net/bugs/126872521:43
*** annegent_ has quit IRC21:44
ryanpetrellookay thanks, sdague21:44
mordredsdague, clarkb: ^^ do we need to start running a load-balacned mirror _farm_ ?21:44
sdaguethis is the connectivity issue between hp cloud and rax21:44
mordredoh21:44
mordredso this is the "run per-region mirrors" solution21:44
sdaguemordred: I think so21:44
sdaguehow big is the mirror?21:45
sdaguecould we just stick it in the nodepool image?21:45
devanandalifeless: so i'm slightly concerned about how stable branches of ironic will be testable in devstack, if dib/tie/etc are installed from master (rather than from versioned packages)21:45
devanandalifeless: as is suggested by this patch https://review.openstack.org/#/c/92256/3/stackrc21:45
openstackgerritClark Boylan proposed a change to openstack-infra/jeepyb: Fix remote updating  https://review.openstack.org/10103021:45
clarkbmordred: jeblair ^ I think that will fix jeepyb21:46
mordredsdague: updates are then an issue-sh21:46
sabeen1morganfainberg: yeah, it didnt work.. i get an error saying that keystone did not start21:46
sdaguemordred: sure, but most of the content would come local21:46
morganfainbergsabeen1, what port are you trying to change to?21:46
sdagueat least it's less random sockets to die21:46
mordredsdague: 2.3G21:46
devanandalifeless: i think we will either need dib/tie to have stable branches, or be installed via pip in ironic's venv (from the requirements.txt file)21:46
*** arnaud has quit IRC21:46
openstackgerritA change was merged to openstack-infra/gear: Use except x as y instead of except x, y  https://review.openstack.org/9661321:46
mordredsdague: but in the past there was an issue with file:// pypi mirror locations21:47
mordredsdague: I _think_ that's been fixed now though21:47
sdaguemordred: ok, fair, is there some native way to get pip to do retries?21:47
*** jp_at_hp has quit IRC21:47
sdaguebecause the pip connections actually look like blips21:47
sdagueunlike the apt ones21:47
adam_gdevananda, im still trying to find it, but there is some magic that ensures versions specified in projects requirements.txt are satisfied over git clones of master, ie client libraries21:47
mordreddstufft: ^^ ?21:47
*** andreaf has quit IRC21:48
devanandaadam_g: that's what i suspected21:48
clarkbmordred: its not fixed in our version of pip iirc21:48
mordredclarkb: kk.21:48
clarkbmordred: once tox releases we can get around that21:48
mordredwell, honestly, I think per-region mirrors gets us the furthest. but also, now that we have no externals, we can just do per-region bandersnatch mirrors21:49
clarkbyup21:49
mordredand concede that clarkb was right a year ago and that we shouldn't use the mirror as an enforcemnt21:49
clarkbor hrm21:49
* clarkb goes a bit crazy21:49
clarkbput the mirrors in swift21:49
clarkbbut that probably requires code21:49
mordrednah. I wan to do it with AFS21:50
clarkbwfm21:50
mordredbecause we can do atomic publish21:50
lifelesscrazy man21:50
mordredbut that's about 3 tasks down on the list21:50
clarkbcan't you do cross cloud swift atomicity?21:51
clarkbclearly that should be a feature21:51
notmynamewho can push a patch to stable/icehouse? https://review.openstack.org/#/c/101032/21:51
*** arnaud has joined #openstack-infra21:51
notmynameclarkb: ya, I think I can make cross-cloud swift atomic operations work. no problem. should just take a couple of days. also suspending the rules of physics21:52
clarkbnotmyname: awesome21:52
clarkbnotmyname: I think the stable maintainers group21:52
clarkbttx, adam_g apevec et al21:53
lifelessdevananda: hmm21:53
notmynamettx: adam_g: et al: please push https://review.openstack.org/#/c/101032/. the corresponding patch to master is https://review.openstack.org/#/c/101031/21:53
mordredclarkb: when I say "atomic with AFS, I don't mean that all clients will see the next version at the exact same time21:54
mordredclarkb: I just mean that the publish of the volume to live will be a single atomic flip per storage node21:54
mordredclarkb: so you dont' have half-repo states21:54
* notmyname again thinks it's weird that a PTL can approve patches for a backport21:54
clarkbnotmyname: me too21:54
mordredthe same notmyname physics apply to the actual cross-cloud activations21:54
clarkbmordred: that is achievable with swift too fwiw, just swap out containers21:55
clarkbmordred: it may be gross compared to afs though21:55
notmynamemordred: wait. isnt' the right word here "federation"? I think if we use that we can get lots of funding. maybe "federated docker" or something like that21:55
mordrednotmyname: federated ansible docker21:55
* jeblair cuts mordred a check for $3.00.21:56
mordrednotmyname: remember the good old days when all we had to say we cloud?21:56
* jeblair takes the check back from mordred.21:56
lyxusadding a public key to the profile. Does it take a long time before being able to do a git review ?21:56
clarkblyxus: no it should be immediate21:57
mroddensoftware defined federated ansible docker environments?21:57
mordredmrodden: NFV!21:57
notmynameadam_g: thanks. I see your +2 on it21:57
mordredmrodden: software defined ansible docker NFV21:57
mordredfederated21:57
mordreddamn21:57
mroddenwow21:57
mordredsoftware defined federated ansible docker NFV21:57
adam_gnotmyname, np.21:57
notmynameya, the "software defined" is the part I though21:58
jeblairadam_g: does stable-maint review stable branches of incubated projects, eg, ironic stable/icehouse?21:58
notmynameI predict we'll see someone use "software defined federation" non-ironically in the near future21:58
jeblairadam_g: or do you wait for the 1st post-graduation release21:58
notmynameadam_g: also, why can't PTLs approve backport patches?21:58
lyxushttp://pastebin.com/69Ws5s1F21:59
*** mriedem has quit IRC21:59
lyxusclarkb, i am confused then21:59
*** hogepodge has joined #openstack-infra21:59
clarkbjeblair: mordred adam_g would it be crazy to add the ptl groups to the stable maint group?21:59
adam_gjeblair, in the past we've helped on a case by case basis. we did some stable releases for ceilometer and heat prior to graduation. the branches were mostly maintained by the project tho. ironic is looking to something similar now21:59
clarkb(I don't want to step ontoes)21:59
jeblairclarkb: you're stepping on toes :)22:00
anteayalyxus: what is the username for the account?22:00
adam_gclarkb, notmyname for some reason i thought they already were.22:00
lyxusanteaya, fyelles22:00
jeblairclarkb: the stable maint team has a pretty strict policy on what gets approved; i think they welcome new members, but it's not presumed that a ptl automatically has the disposition22:00
clarkbjeblair: gotcha22:00
clarkbI kenw that was the separation but I thought it was more directed at the larger core groups22:01
clarkbbut I suppose it applies to the ptl too22:01
jeblairnotmyname: it's because you're too reckless ;)22:01
notmynamejeblair: gotcha22:01
clarkblyxus: so the username checks out. do you have an ssh agent with the old key loaded?22:01
anteayalyxus: can you paste the git review command and use the -v flag, and this time paste the command as well please22:01
clarkblyxus: or maybe there was a copy pasta failure of the pubkey into gerrit22:01
adam_galso-swift has never really been part of the stable release process. is that changing? (i was not at the summit)22:01
*** dprince has quit IRC22:02
sdaguenotmyname: we do have some pretty good gate throughput at the moment though. if storage policies are close, you probably don't need to do the merge commit trickery22:02
notmynameadam_g: have we not? we've backported security patches (like this one) the few times it's come up.22:02
*** mkoderer has quit IRC22:02
clarkblyxus: you may want to use ssh -i to be specific about the key to use22:02
lyxusclarkb, anteaya , sure22:02
notmynamesdague: merge commits are good. it gives a single commit that introduces the feature. that is good for later code archeology22:03
adam_gnotmyname, i mean the scheduled point releases22:03
*** andreykurilin_ has quit IRC22:03
sdaguenotmyname: sure.22:03
*** timrc-afk is now known as timrc22:03
*** chianingwang has joined #openstack-infra22:04
lyxusanteaya, http://pastebin.com/WS40c0zH22:04
*** blamar has quit IRC22:05
*** Sukhdev has joined #openstack-infra22:06
notmynamesdague: current gate pass rate is about 85%. with 29 patches, that gives us still a pretty long window to wait for the gate to merge stuff. I'd expect with the current gate health to take through the weekend if we submitted it now22:06
sdaguenotmyname: we merged 107 today22:06
sdagueso I would not expect that22:07
notmynamesdague: but those weren't a chain of 107 that had to land in a specific order22:07
anteayalyxus: can you paste the output of ssh -i <path to ssh public key> -p 29418 fyelles@review.openstack.org gerrit ls-projects22:07
sdaguenotmyname: nope, I expect if you put 29 into the queue you'd bounce 3 times22:07
sdagueso if you had someone on it, you'd be merged by midnight22:07
dstufftmordred: I was at the dentist, what'd you need?22:09
sdagueok, got to run. Catch folks in the morning.22:09
lyxusanteaya, Permission denied (publickey).22:09
anteayalook at the key you pasted22:09
anteayaare there newlines in it?22:09
notmynameI'm led to believe by mordred that getting a feature branch set up and then doing one merge commit is fairly simple to do. beyond the gate, the merge-commit also gives good code history and helps with finding issues later. that is the direction I want to go.22:10
anteayaif there are newlines, the newline counts as a charater in the key and it won't match22:10
anteayasdague: night sean22:10
*** shayneburgess has quit IRC22:10
lyxusanteaya, i copy/pasted the one from the launchpad and it's in one line without any \n22:12
mordreddstufft: was a question about retries in pip - but I looked22:12
clarkbmordred: speaking back to tox releasing, how crazy would it be to install tox from source22:13
clarkbjeblair: mordred: I am not happy with the idea but it is an option I suppose22:14
dstufftmordred: ok22:14
anteayalyxus: hmmmm22:14
anteayalyxus: and you used the -i flag with the path to your ssh public key?22:14
lyxusanteaya, yes, I am a bit stomped (http://pastebin.com/34KRRS4G)22:15
*** ramashri has quit IRC22:15
*** ramashri has joined #openstack-infra22:15
*** dims_ has joined #openstack-infra22:16
anteayaclarkb: feel like splunking the gerrit db?22:16
anteayaand seeing what gerrit thinks lynx has as a public key?22:16
clarkbanteaya: I suppose I can give me a couple minutes so that I can kick off some trusty tests22:16
anteayalyxus22:16
anteayaclarkb: /me nods22:16
anteayathanks22:16
clarkbthey will take a bit so want to get them going22:16
anteayacause I'm out of guesses22:16
anteayayeah22:17
lyxusanteaya, clarkb , Thanks !22:17
*** dkehn_ is now known as dkehnx22:17
clarkblyxus: what is your account number?22:18
clarkbit shows up i nthe main gerrit settings page22:18
lyxusclarkb, 993222:19
*** dims_ has quit IRC22:19
*** dims_ has joined #openstack-infra22:20
lyxusclarkb, i might have found the issue... git me a asecond22:20
clarkblyxus: I have key ending in qJ816N in gerrit22:21
clarkbit looks correct22:21
lyxusclarkb, anteaya , Too much redbull.. I had a different keys on my launchpad id... and  the gerrit server22:21
*** chianingwang_ has joined #openstack-infra22:21
lyxusssh -i worked...22:21
lyxusclarkb, anteaya , thanks for the help and sorry for the trouble !22:22
clarkbnp22:22
*** chianingwang_ has quit IRC22:22
anteayak22:23
*** rm_work|away is now known as rm_work22:24
*** e0ne has quit IRC22:25
*** doude has quit IRC22:26
*** chianingwang_ has joined #openstack-infra22:26
*** e0ne has joined #openstack-infra22:26
*** chuckC has quit IRC22:26
lifelessdevananda: I don't see the issue22:27
*** virmitio has quit IRC22:28
devanandalifeless: ironic pxe driver depends on the deploy element. if that ever changes, stable/icehouse of ironic will cease to function with master of dib22:28
*** yamahata has quit IRC22:28
lifelessdevananda: ironic depends on neutronclient, same issue.22:28
devanandalifeless: except we version pin neutronclient in requirements.txt22:29
*** thedodd has quit IRC22:29
devanandalifeless: so we are now going to version pin dib the same way22:29
devanandalifeless: so problem solved :22:29
devananda:)22:29
*** e0ne has quit IRC22:29
*** e0ne has joined #openstack-infra22:30
lifelessI seem to recall the test infra overriding those pings22:30
lifelesspins22:30
mordredyup22:30
devanandalifeless: it overrides branches22:30
lifelessdevananda: anyhow, dib == neutronclient for this discussion22:30
mordredwe override client lib pins22:30
devanandamordred: oh22:31
devanandamordred: so how does a stable/icehouse test run? with the pinned (older) pips, or current requirements' pips?22:31
mordredyou really shouldn't pin an upper bound on any openstack client lib22:31
mordredwith tip of current lib master branch22:31
devanandaexcept dib isn't a client lib22:31
* mordred goes to look to see what's going on with dib22:32
lifelessdevananda: we explicitly want to make sure trunk of semver projects like clientlibs work with stable branches of non-server projects like the API servers22:32
lifelessdevananda: It isn't but its in the same space in the ecosystem, no ?22:32
devanandamordred: nothing yet22:32
devanandamordred: i'm looking ahead as we just actally opened stable/icehouse (there were some bugs... it now passes our tempest suite)22:33
mordreddevananda: dib is installed from source22:33
mordredwhich means we'll be installing tip of dib master22:33
devanandamordred: right22:33
*** shayneburgess has joined #openstack-infra22:33
devanandamordred: it's ALSO in global requirements22:33
devanandamordred: and has a lower version bound22:33
*** e0ne has quit IRC22:33
devananda(well, there's a review up to add taht)22:33
mordredright. but that won't be honored22:33
devanandagah22:34
mordredbecause dib will have already been installed from source22:34
mordredthis is by design22:34
devanandaso we can not change the deploy-ironic element in dib22:34
mordredyou'll want to add this to your hate list of reasons that asymmetric gating is terrible for you22:34
devanandabecase it wold break stable/icehouse22:34
devanandamordred: it's mroe than just asymmetric22:34
mordreddevananda: well, you can only change it in a way that does not break stable/icehouse22:34
devanandawell, right22:35
devanandawhich is bonkers. it should be versioned.22:35
mordreddevananda: or, you can convince lifeless that stable branches of dib are important22:35
devanandathe element at that time worked with ironic at that time22:35
devanandawe may want to evolve them both22:35
devanandaactually, i will want to deprecate that element eventually -- once IPA is up to speed, which hopefully will be next cycle22:36
lifelessdevananda: you can change it, you just need to do it compatibly22:36
lifelessdevananda: which btw you *have to do anyway* or you can't get past the gate.22:36
lifelessdevananda: because its in two different trees.22:36
devanandamordred: also, this means we need a stable branch test of ironic, which i dont think we have22:36
devanandamordred: or i know nothing about22:36
mordredit should run all the normal jobs22:36
lifelessdevananda: tempest is moving to one-tree-tests-all-versions22:37
mordreds/is moving/has moved/22:37
clarkbone tree to rule them all22:37
devanandalifeless: yes22:37
*** e0ne has joined #openstack-infra22:37
clarkbtempest needs tshirts that say something like that22:37
clarkbmtreinish: ^22:37
mordreddevananda: you do probably want to start working on grenade jobs now that you have a stable branch22:37
devanandamordred: indeed we do22:37
*** wenlock has quit IRC22:38
lifelessdevananda: so I still don't see how there is an issue here22:38
mordreddevananda: but you have tests for your stable branch already22:38
devanandamordred: though i'm not sure taht anyone on our team has the bandwidth right now22:38
lifelessdevananda: we just don't delete the backwards compat code in any evolution of ironic-deploy22:38
mtreinishclarkb: heh I like that22:39
devanandamordred: i mean, things we depend on (like dib) are only tested with current master of ironic. not stable/icehouse22:39
devanandamordred: should they be?22:39
devanandaI dunno22:39
lifelessdevananda: same question applies to neutronclient :)22:39
lifelessdevananda: I'm sorry to sound like a broken record22:39
clarkblifeless: we do test neutronclient against icehouse now22:40
clarkblifeless: I don't think it is voting22:40
clarkbjogo: ^22:40
lifelessdevananda: but there seems to be to me a long history here, and I'm not sure why dib is making you concerned specifically22:40
jogoclarkb: that sounds correct22:40
lifelessclarkb: I'd love to have such jobs for tripleo components, we're just discussing where we will and wont break compat now22:40
devanandalifeless: right. you're saying "we dont need to have a stable/icehouse branch of DIB because we can't make backwards incompatible changes anyway, ever"22:40
jogoclarkb: although we could make it gating now I think22:41
lifelessdevananda: we can't do it atomically anyhow; we can incrementally if we really wanted to22:41
SpamapSOk, interesting question. If I wanted to make tempest depend on os-collect-config .. would that be o-k?22:41
lifelessSpamapS: openstack-qa is the channel for that q22:41
*** flaper87 is now known as flaper87|afk22:41
SpamapSoh22:41
SpamapSduh22:41
devanandalifeless: i'm trying to understand how this should be organized as we move forward with three things: stable branch maint for ironic, dependency management (eg, with dib and IPA), and testing image builds and dependencies betwen ironic and the (pxe and IPA) agent code22:41
SpamapSI just think of in here as "the qa people" :)22:41
anteayaSpamapS: always nice to see you though22:41
* mordred throws a happy wet cat at SpamapS22:42
devanandalifeless: no, we can't do it incrementally today, because it would break stable/icehouse tests of ironic22:42
devanandalifeless: that's my point22:42
devanandalifeless: well, my earlier point. one of them22:42
lifelessdevananda: I meant technically, not policy. Its *possible* to do it incrementally. I don't think we should.22:42
clarkbjroll: JayF: so devstack-trusty and bare-trusty labelled nodes are now booting in nodepool22:42
lifelessdevananda: and I think we should have tests that we don't.22:42
clarkbjroll: JayF I am manually running some uniitests on bare-trusty but if you want to propose jobs for devstack-trusty (don't let the name fool you devstack not required) feel free22:43
lifelessmordred: btw if you have a timeslice 96608 should be ready for a +222:43
*** e0ne has quit IRC22:43
devanandalifeless: right. I'm not saynig we should. i'm saying, AFAIUI, it's technically impossible now (without breaking ironic stable/icehouse tests) to move forward, in an incremental fasion, in a way that introduces changes to dib which are not compatible with stable/icehouse version of ironic's PXE deploy driver22:43
devanandaI also don't think we should :)22:43
mordredclarkb: jeblair and I yesterday had a conversation about ways to merge devstack-$foo and bare-$foo to just $foo22:43
*** e0ne has joined #openstack-infra22:44
clarkbmordred: ya we were discussing tat above. I don't completely agree with it for the reasons I told lifeless22:44
clarkbmordred: but if that is a compromise to simplify the overall thing then ok22:44
openstackgerritSpencer Krum proposed a change to openstack-infra/config: Fixing deprecation warnings  https://review.openstack.org/10104522:44
lifelessbare-foo? I'm lost22:44
mordredclarkb: how far above?22:44
clarkbmordred: not too far, it was PDT afternoon22:44
jrollclarkb: cool, thanks22:44
*** otherwiseguy has quit IRC22:45
mordredlifeless: foo=(precise|trusty)22:45
*** mwagner_lap has joined #openstack-infra22:47
clarkbmordred: jeblair any thoughts on the addition of experimental jobs for trusty?22:47
clarkbmordred: jeblair nodepool is ready to handle those now22:47
mordredclarkb: seems fine to me22:47
clarkbwoot nova unittests just passed on trusty22:48
*** e0ne has quit IRC22:48
clarkbfor py2722:48
clarkbnow neutron22:48
*** chuckC has joined #openstack-infra22:49
openstackgerritAnita Kuno proposed a change to openstack-infra/config: Adds Third Party CI account name format directions  https://review.openstack.org/10101322:49
jeblairclarkb: i want to reduce node types so we only have one per os on its own merits -- that it makes nodepool dib cleaner is a possible bonus22:49
clarkbjeblair: yup and in theory it makes it easier for individuals to run the tests too22:50
jeblairclarkb: no objection to experimental; i'm buried in paperwork at the moment, sorry.22:50
*** rm_work is now known as rm_work|away22:50
clarkbjeblair: understood I will push it through then22:50
clarkbmordred: did you want to review 101023 before I push it through?22:51
JayFanteaya: thanks for that, I left a comment on it :)22:51
*** mestery has quit IRC22:51
jeblairclarkb: i think jenkins07 is ready for the upgrade but can't context switch back to that just yet; if you have time and inclination, feel free to do the upgrade, otherwise i should be able to later22:52
clarkbjeblair: my concern is more to do with time spent on those steps but if we do work to cache all the things we can minimise it and I can deal with it22:52
clarkbjeblair: ok22:52
jeblairclarkb: yeah, i'm hoping the in-test hit is like 5-10 seconds22:52
clarkbI think I have time let me context switch over to that once experimental jobs are approved22:52
anteayaJayF: great, thanks for the review22:52
*** sballe__ has quit IRC22:52
JayFanteaya: np, glad to see the docs updated. I didn't think I had seen any of the stuff about naming when I read the third-party ci docs :)22:53
anteayaJayF: yeah, I was working on them last week and we started to use them and just got a pointer for where they should be published yesterday22:55
anteayaso fairly fresh22:55
anteayaI'll wait until jenkins comes back since I want to see tests pass22:56
clarkbjeblair: actually I need to step away for ~20 minutes but can do jenkins07 then22:56
*** markmcclain has quit IRC22:56
clarkbok back shortly22:57
openstackgerritSpencer Krum proposed a change to openstack-infra/infra-specs: Multiple Data Dirs proposal  https://review.openstack.org/10036322:58
openstackgerritA change was merged to openstack-infra/config: Add experimental tempest and grenade trusty jobs  https://review.openstack.org/10102323:00
openstackgerritAnita Kuno proposed a change to openstack-infra/config: Adds Third Party CI account name format directions  https://review.openstack.org/10101323:03
mordredgoooooooooooooooooooooooooooooooooooool23:04
devanandamordred: so. i think i found the source of my confusion23:05
mordreddevananda: woot23:05
devanandamordred: ironic now has a stable/icehouce branch, whcih passes the current tempest suite23:05
devanandamordred: i think that's grand. and i'd like it to continue being stable23:05
mordredyes!23:05
devanandamordred: but ironic is not tested in any of the stable/icehouse tests that run against tempest (and even if it was, it wouldn't vote on them)23:06
devanandamordred: ironic doesn' thave a separat stable/icehouse test job. and it's not enabled in the tempest stable branch tests23:06
devanandaso. should we even have a stable branch, when ironic can't vote on it?23:07
devanandadoes that make sense?23:07
*** mrodden has quit IRC23:07
devanandabeing not-integrated but having a stable branch is weird, isn't it?23:07
devananda... or do we just need to make a tempest-dsvm-virtual-ironic-icehouse-nv job?23:08
*** chianingwang has quit IRC23:09
*** chianingwang_ has quit IRC23:10
*** mordred_ has joined #openstack-infra23:11
jheskethMorning23:11
anteayamorning jhesketh23:11
*** sabeen1 has quit IRC23:11
*** pdmars has quit IRC23:12
boris-42jhesketh morning and thanks for reviews=)23:13
clarkbjeblair: zaro ok back. I am just upgrading the timeout plugin to 1.14?23:13
clarkbhrm looks like the gearman plugin should be upgraded too23:14
clarkbif that looks correct I will upgrade those two and restart23:14
*** lcheng has quit IRC23:16
*** mordred_ has quit IRC23:16
*** mordred_ has joined #openstack-infra23:17
*** dims_ has quit IRC23:17
*** jamielennox|away is now known as jamielennox23:19
*** prad_ has quit IRC23:20
*** CaptTofu_ has quit IRC23:21
*** CaptTofu_ has joined #openstack-infra23:22
*** adrian_otto has joined #openstack-infra23:23
clarkbneutron passes py27 on trusty too23:23
adrian_ottoSolum core should have +2 -2 rights on the stackforge/solum-specs repo, but we don't. See: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/gerrit/acls/stackforge/solum-specs.config#L2 and https://review.openstack.org/#/admin/groups/229,members23:24
adrian_ottois something messed up?23:24
*** mordred has quit IRC23:24
*** mordred_ is now known as mordred23:24
clarkbadrian_otto: yes23:24
clarkbthough tis may be different let me look really quick23:25
adrian_ottoclarkb: tx23:25
clarkbadrian_otto: ya empty ACLs in gerrit https://review.openstack.org/#/admin/projects/stackforge/solum-specs,access23:25
clarkbhttps://review.openstack.org/#/c/101030/ should fix that23:25
*** CaptTofu_ has quit IRC23:26
adrian_ottoclarkb: thanks!23:27
anteayaclarkb: glory be23:27
mordredok. my irc setup is now officially insane23:28
anteayamordred: great23:28
anteayayou can't beat markmcclains though23:29
*** CaptTofu_ has joined #openstack-infra23:29
anteayasometimes I talk to three of him23:29
clarkbmordred: is it full of turtles?23:29
anteayaclarkb: all the way down23:29
anteayaclarkb: so does --prune work, this is what you do to fix the acls now?23:29
*** mordred_phone has quit IRC23:30
*** mordred has quit IRC23:30
*** mordred has joined #openstack-infra23:30
anteayaha ha ha23:30
mordredclarkb: yup23:30
mordredclarkb: I now have a znc bouncer23:30
clarkbnice23:30
mordredwhich weechat is connected to23:30
mordredand which my web irc account is connected to23:31
mordredso mordred_phone is no more23:31
clarkbyay23:31
clarkbit confuses the tab completion23:31
anteayayeah23:31
mordredyah23:31
anteayaone mordred to rule them all23:31
anteayaso --prune, clarkb23:32
anteayayou are using this now to keep things moving?23:32
clarkbanteaya: you know I maybe should have left that out23:32
clarkbanteaya: but all of the other remote updates use it23:33
clarkbso I figured consistency may be good and help avoid mixups in the future23:33
anteayahave we tested your proposed command anywhere?23:33
anteayaI'm fine with consistency23:33
anteayajust wondering if this has had any airtime yet23:33
anteayain the manual mode23:33
clarkbno I have not run that at all. I can give it a go, but am trying to focus on trusty and jenkins23:33
anteayaright okay23:34
anteayawell I will hold off until I hear from you23:34
clarkbzaro: jeblair please do let me know if build-timeout and gearman plugin are the only two that need upgrading23:34
anteayaand yeah focus on trusty and jenkins23:34
*** otherwiseguy has joined #openstack-infra23:35
mordredmmm. trusty23:35
*** adrian_otto has quit IRC23:36
clarkbflake8 works on trusty23:36
clarkbchecking doc build23:36
clarkbbut things look reasonably well. I am doing my best to test both rax and hp too23:37
anteayacool23:37
clarkbooh we may have our first error23:40
clarkblibvirt:  error : internal error: could not initialize domain event timer23:40
clarkbbut I didn't get a test traceback so we'll see23:40
clarkbit says tests passs so maybe that is an intentional thing23:41
mikalsdague: I wonder if we see this one in the gate? I guess its unlikely unless we're launching many instances at once -- https://bugs.launchpad.net/nova/+bug/131662123:42
uvirtbotLaunchpad bug 1316621 in nova "ebtables calls can race with libvirt" [Medium,Confirmed]23:42
*** dangers is now known as dangers_away23:42
*** shayneburgess has quit IRC23:44
mordredphone test23:46
mordredsweet23:46
clarkbcan you hear me now?23:47
mordredclarkb: you're breaking up a little23:47
clarkbdarn23:47
anteayamorning mikal23:47
clarkbhttps://jenkins04.openstack.org/job/check-tempest-dsvm-full-trusty/1/console for the excitement23:48
*** mmaglana has quit IRC23:50
*** chianingwang has joined #openstack-infra23:52
anteayaohhhhh23:53
* anteaya watches the little thing turn in circles23:53
*** otherwiseguy has quit IRC23:53
anteayalike watching clothes in the washer23:53
*** chianingwang_ has joined #openstack-infra23:54
*** adrian_otto has joined #openstack-infra23:55
*** dims_ has joined #openstack-infra23:56
*** hogepodge has quit IRC23:56
*** zzelle has quit IRC23:56
mordredclarkb: is that going to make me happy?23:56
clarkbmordred: I hope so23:56
clarkbwe should know in about ~45 minutes if stuff works23:57
*** adrian_otto has quit IRC23:57

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