Tuesday, 2015-03-31

clarkbjeblair: I think what I am tripping up on is what I said earlier about qemu-img-options00:00
clarkbjeblair: because it is entirely possible that those will be different between clouds00:00
jeblairclarkb: yeah, so i'm suggesting they ride with image types00:00
clarkbjeblair: so you end up with same format, same image, multiple times00:00
jeblairclarkb: only if that's necessary...00:01
jeblairclarkb: let's forget about the tripleo cloud for a moment so i can stop saying "except for the tripleo cloud" and we'll get back to that in a minute...00:01
clarkbI'm not sure how tripleo cloud got involved fwiw00:01
jeblairclarkb: okay, then i've been really unclear :)00:01
jeblairclarkb: continue to forget about it for the moment then, and i'll try splainin again00:02
greghaynesclarkb: we can also make dib accept multipe --image-options00:02
greghaynesclarkb: since its simlar to -t00:02
jeblairclarkb: we won't build any more images than necessary.  we need to build 'devstack-trusty' for rackspace and hp, because they use different image formats00:02
jeblairclarkb: so regardless, we will end up with 2 devstack-trusty files on our filesystem, yeah?  one in vhd and one in qcow2.00:02
*** annegentle has quit IRC00:02
clarkbjeblair: yup00:03
jeblairclarkb: so specifically, we will _not_ end up with 16, or even 8 devstack-trusty files like we currently have devstack-trusty snapshot images.00:03
*** achanda_ has joined #openstack-infra00:03
*** patrickeast has quit IRC00:04
jeblairclarkb: because i'm not saying build one for each pseudo-provider, i'm saying, found out what types we need to build and build those types automatically.  but still per-diskimage.00:04
*** yamamoto has joined #openstack-infra00:04
jeblairs/found/find/00:04
jeblairclarkb: so for us, we _need_ both of those files, but nothing else.  and with either of my proposals, we will get exactly only those two files.00:05
jeblairclarkb: make sense so far?00:06
*** baoli_ has quit IRC00:06
*** achanda has quit IRC00:07
clarkbjeblair: mostly but only if you consider image-tpe00:07
clarkbjeblair: I think that breaks down when you add in the other knobs that are semi related00:07
jeblairclarkb: yes, that's important.  i'm considering image-type and image options as a unit.00:07
*** dboik_ has quit IRC00:07
jeblairclarkb: so if provider A uses qcow 0.10 and provider B uses qcow 0.20, and provider C uses vhd, you will get 3 files for each image00:08
*** achanda_ has quit IRC00:08
*** ddieterly has joined #openstack-infra00:08
jeblairclarkb: because an image type is really an image-type-with-options.  so qcow0.10 is not the same as qcow0.2000:08
*** tonytan4ever has quit IRC00:08
clarkbright ok00:08
greghaynesWhy dont we just make it a boolean use_old_qcow and if any providor wants that for a qcow we just make the qcow in 0.10 format00:08
*** yamamoto has quit IRC00:09
jeblairgreghaynes: it's functionally the same i think, but allowing options makes it more future-proof00:09
jeblairclarkb: k, so if we add tripleo back in, we see the difference between my first and second proposals00:09
*** sigmavirus24 is now known as sigmavirus24_awa00:09
greghaynesThe thinking there is that youd get two images in that case, not 300:09
jeblairgreghaynes: no, we need 3 because one provider uses old and one new, and one something else00:09
greghaynesoh, the one who uses the new doesnt work with the old?00:10
jeblairgreghaynes: beats me.  this stuff is all insane and i'd rather just pass it through (like we do now).  i agree though if we think that's feasible to support, we could enhance the proposal to cover it.00:11
greghaynesAIUI you could use 0.10 everywhere and be fine, but 0.20 did not00:11
jeblairanyway, back to tripleo00:11
greghaynesheh, ok00:11
clarkbjeblair: sorry still don't follow on tripleo00:11
clarkbjeblair: we shouldn't need to do anything special there right?00:11
clarkbjeblair: oh wait I see00:11
jeblairclarkb: my first was super simple, it was just "iterate over all providers, and collect image-type-plus-options into a set, and build that set of types for all images"00:11
clarkbwe would only qcow2 that image because rax doesn't use it00:11
jeblairclarkb: right.  so my first proposal would have us building vhd tripleo images which we don't need.00:12
*** carl_baldwin has quit IRC00:12
jeblairclarkb: the second actually says "examine each provider _and_ which images it uses, and collect the union of all file types for diskimages used by that provider"00:12
*** weshay has joined #openstack-infra00:13
jeblairclarkb: so with the first, we build a few extra image files.  i don't think it will kill us because it shouldn't be that many, but yeah, it is inefficient.00:13
openstackgerritClark Boylan proposed openstack-infra/nodepool: Support per provider image type for image uploads  https://review.openstack.org/12674700:14
jeblairclarkb: however, it does mean that you can add an image to hp first, then add an image to rax later and already have the file built00:14
jeblairclarkb: if we went with the second thing, nodepool would have to say "oh, i need to rebuild or convert this qcow image to vhd because it's now being used by a provider that requires vhd"00:14
nibalizerasselin_: wow okay, thats not at all what I expected00:15
*** baoli has joined #openstack-infra00:15
jeblairclarkb: i don't know off the top of my head how complicated that logic would be.  if it's not terrible, then that's the better way to go.00:15
nibalizerbut ill open my mind a bit :)00:15
clarkb jeblair we dont want the first thing because disk already hurts us00:15
clarkbjeblair and we need something like 16gb ti build a vhd image00:16
asselin_nibalizer, good or bad? anyway....looking for better ways to test....00:16
clarkbbut new patchset is incomplete start at doing the second thing you describe00:16
jeblairnibalizer, asselin_: what's the topic of conversation?00:16
jeblairclarkb: to build or store?00:16
clarkbI have to go now but will pick that back up later00:16
asselin_jeblair, https://review.openstack.org/16911700:16
jeblairclarkb: ok, np.  anyway, no argument there, the second way is better.  just a tradeoff we can evaluate.  :)00:17
clarkbjeblair to build because of the vhd utils multi step thing you basically end up with 4 copies of the image during build00:17
nibalizerI thouht we were gonna write a puppet module. I didn't expect to write shell scripts :P00:17
clarkbso 4*4GB == 16GB00:17
asselin_nibalizer, I'm not a puppet expert by any means....learning as I go00:17
nibalizeras for the testing... its gonna be in shell or serversspec or beaker or envassert, and they're all pretty similar so :shrug:00:18
jeblairclarkb: yeah, i think storage space is the main thing here since diskimage builds are single threaded.00:18
clarkbjeblair its raw, qcow2, intermediate, vhd00:18
*** sputnik13 has quit IRC00:18
greghaynesclarkb: I dont think you end up with 4 copies in vhd00:18
*** ibiris_away is now known as ibiris00:18
greghaynesI think you get 200:18
clarkbgreghaynes no its 4 total see above00:19
*** unicell has quit IRC00:19
*** unicell1 has joined #openstack-infra00:19
greghaynesyes, IIRC the vhd-util does an in-place modify00:19
greghaynesand then an mv00:19
*** hdd has quit IRC00:19
greghaynesI should re-test though00:19
clarkbyup I think our maths line up00:19
*** Somay has quit IRC00:19
jeblairclarkb: anyway, you need that space to build regardless; the advantage of option 2 over option 1 is storage space, not in build workspace00:20
clarkbya I am just cranky we dont have 16gb currently00:20
*** rossella_s has quit IRC00:20
jeblairclarkb: we should add a cinder volume00:20
clarkbso will need new disk volime sooner than later00:20
clarkbyup00:20
*** tsg has quit IRC00:20
*** rossella_s has joined #openstack-infra00:20
greghaynesclarkb: ah, so I think the raw shouldnt be included00:22
jesusaurusjeblair: can i pick your brain for a minute about gertty?00:22
greghaynesclarkb: its destructive to the raw image which is why I have the sorting logic to run vhd last00:22
jeblairjesusaurus: yep00:22
jeblairjesusaurus: (also, i left a comment on 168495)00:22
greghayneshrm, did that get removed00:22
clarkbgreghaynes uh it shouldnt do that since we need raw00:22
clarkband yes its not in dib00:22
greghaynesclarkb: yea, disreguard that, the sorting logic was for the raw patch00:23
jesusaurusjeblair: yeah, i saw the comment, thanks00:23
jesusaurusjeblair: im confused about how parser.py works and what p[0] represents00:23
jesusaurusjeblair: how is p_is_term() getting called?00:23
jeblairjesusaurus: so the first thing to know is that i didn't make that up.  :)  that's more or less the standard lex/yacc way of doing things; see: http://www.dabeaz.com/ply/00:24
jeblairjesusaurus: but the way i think about it is that when the parser matches grammar described at the top of the function...00:25
*** asettle has joined #openstack-infra00:25
jeblairjesusaurus: (the _comments_ at the top of the function are actually the grammar -- they aren't just comments, the thing actually works by parsing the comments and building the grammar directly from them!)00:25
jesusaurusjeblair: oh wow, that's a little bit of magic there00:25
jeblairjesusaurus: it invokes that function.  the arguments p[2]...p[n] are the parts after the ':'00:26
jeblairjesusaurus: p[1] is the part before the ':' (so the literal "is:" in this case)00:26
*** dims has joined #openstack-infra00:27
jeblairjesusaurus: and p[0] is the result00:27
jeblairjesusaurus: (or the 'production')00:27
*** mtanino_ has joined #openstack-infra00:27
*** tiswanso has joined #openstack-infra00:28
*** ibiris is now known as ibiris_away00:28
jeblairjesusaurus: so if you have "status:open AND is:watched" -- the "is:watched" produces a sql query for whether something is watched00:28
jesusaurusjeblair: ah, so putting the return value in p[0] is just part of how ply works?00:28
jeblairjesusaurus: same thing for status:open00:28
jeblairjesusaurus: and then it runs the function for "AND", which does a sql query operation that combines its two arguments and returns _that_ as p[0]00:29
jeblairjesusaurus: so that's how they get built up into one big sql query at the end00:29
*** mtanino has quit IRC00:29
jeblairjesusaurus: and yep, the whole p[0] thing is just the way ply works00:29
*** mwagner_lap has joined #openstack-infra00:29
*** signed8b_ has joined #openstack-infra00:30
jesusaurusjeblair: ok, im starting to see how the pieces fit together00:30
jeblairjesusaurus: yeah, take a look at some of the boolean things (and note that there's special handling in there since 'and' isn't necessary -- any two expressions next to each other are 'anded').  i think what they do with arguments are illustrative.00:32
*** signed8bit has quit IRC00:32
jeblairjesusaurus: and also, note the 'main' funciton in __init__.py, which is good for playing with things and testing out the result00:33
jeblairjesusaurus: i have to run now, but let me know how you get on00:33
jesusaurusjeblair: awesome, thanks00:33
clarkbjeblair oh we also need another dib change to support this fwiw00:36
clarkbbut should be fine we already have a stack of those00:36
greghaynesclarkb: oh?00:37
greghaynesI am dib changes as a service lately00:37
clarkbthe qemu ing options to avoid rebuilding inages for that00:37
clarkbnot sure how to model that yet00:38
greghaynesah, well that gets at my question of why not just make all images in 0.10 format00:38
greghaynessince IIRC the issue was that some things didnt recognize 0.2000:38
*** tiswanso has quit IRC00:38
clarkbnot 1.0 byt there are other options too00:38
greghaynesah00:38
*** tiswanso has joined #openstack-infra00:39
clarkbI mean I am mostly trying to expect the unexpected here00:39
greghaynesSo weve been talking about making raw output format caching00:39
clarkbsince thats been the tldr on this stuff00:39
greghaynesso if you want another format, and theres a raw image cached, wed just do a convert from that00:39
*** yamamoto_ has joined #openstack-infra00:43
*** otter768 has joined #openstack-infra00:43
SpamapSHm, we should check that we don't rely on anything that pulls from code.google.com00:52
SpamapShttp://google-opensource.blogspot.com/2015/03/farewell-to-google-code.html00:52
greghaynesyeep00:53
clarkbSpamapS we have a year and yes we have thibgs, gerrit sort of, yappi just off the top of my head00:53
ianwclarkb / fungi : see also https://github.com/eliben/pycparser/issues/72 ; that seems to isolate the memory usage to pycparser ... if there's a bug there is to be determined00:54
*** ghostpl_ has joined #openstack-infra00:54
*** Longgeek has joined #openstack-infra00:54
*** YorikSar has quit IRC00:55
fungiianw: that's an excellent reduction00:56
*** annegentle has joined #openstack-infra00:58
*** emagana has quit IRC01:01
*** rfolco has joined #openstack-infra01:02
*** ghostpl_ has quit IRC01:04
openstackgerritK Jonathan Harker proposed stackforge/gertty: Add is:watched to p_is_term()  https://review.openstack.org/16849501:06
*** ChuckC has quit IRC01:08
*** imcsk8 has quit IRC01:12
*** baoli has quit IRC01:12
*** amotoki has joined #openstack-infra01:12
*** imcsk8 has joined #openstack-infra01:12
*** annegentle has quit IRC01:12
*** annegentle has joined #openstack-infra01:13
*** amotoki has quit IRC01:13
* ianw takes a lunch break and starts reducing some leeks for dinner ... much tastier type of reduction :)01:14
*** Longgeek has quit IRC01:17
*** Longgeek has joined #openstack-infra01:17
*** yamamoto_ has quit IRC01:17
anteayaianw: sounds yummy01:19
*** asselin has joined #openstack-infra01:28
*** thingee has quit IRC01:30
*** aduarte has quit IRC01:30
*** unicell1 has quit IRC01:31
*** unicell has joined #openstack-infra01:31
*** kaisers has joined #openstack-infra01:33
*** ChuckC has joined #openstack-infra01:34
*** xyang1 has quit IRC01:35
*** kaisers1 has quit IRC01:35
openstackgerritThai Tran proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913201:38
*** weshay has quit IRC01:38
*** stevemar has joined #openstack-infra01:39
*** jerryz_ has joined #openstack-infra01:39
*** baoli has joined #openstack-infra01:40
*** amotoki has joined #openstack-infra01:42
*** annegentle has quit IRC01:42
*** tqtran has quit IRC01:45
*** baoli has quit IRC01:47
*** amotoki has quit IRC01:47
*** yamahata has joined #openstack-infra01:47
*** amotoki has joined #openstack-infra01:47
*** carl_baldwin has joined #openstack-infra01:47
*** yamamoto_ has joined #openstack-infra01:49
*** asettle has quit IRC01:49
*** tsg has joined #openstack-infra01:49
*** unicell has quit IRC01:49
*** dboik has joined #openstack-infra01:50
*** amotoki has quit IRC01:52
*** sputnik13 has joined #openstack-infra02:04
*** sputnik13 has quit IRC02:06
*** sputnik13 has joined #openstack-infra02:06
*** dims has quit IRC02:07
*** dims has joined #openstack-infra02:08
*** Longgeek has quit IRC02:09
*** Longgeek_ has joined #openstack-infra02:09
*** sputnik13 has quit IRC02:11
*** sputnik13 has joined #openstack-infra02:12
*** mfink has left #openstack-infra02:12
*** dims has quit IRC02:14
*** carl_baldwin has quit IRC02:14
*** SumitNaiksatam has quit IRC02:14
*** annegentle has joined #openstack-infra02:15
*** marun has quit IRC02:16
*** baoli has joined #openstack-infra02:18
*** Longgeek_ has quit IRC02:20
*** ghostpl_ has joined #openstack-infra02:21
*** carl_baldwin has joined #openstack-infra02:21
*** Longgeek_ has joined #openstack-infra02:23
*** signed8b_ has quit IRC02:26
*** amotoki has joined #openstack-infra02:28
*** ghostpl_ has quit IRC02:28
*** otter768 has quit IRC02:30
*** carl_baldwin has quit IRC02:31
*** achanda has joined #openstack-infra02:34
*** sputnik13 has quit IRC02:35
*** garyh has quit IRC02:38
*** otter768 has joined #openstack-infra02:39
*** carl_baldwin has joined #openstack-infra02:41
*** amotoki has quit IRC02:43
*** crc32 has quit IRC02:44
*** achanda has quit IRC02:44
*** yamahata has quit IRC02:44
jgriffithanybody looking at teh XStatic-angular-irdragndrop issue?02:44
jgriffithie it appeasr the version referenced in setup has been removed :(02:45
*** achanda has joined #openstack-infra02:45
jgriffithOh.. yep, it's here http://goo.gl/NZaSzc02:46
*** miqui has joined #openstack-infra02:47
*** mtanino_ has quit IRC02:50
*** carl_baldwin has quit IRC02:51
openstackgerritJohn Griffith proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913202:51
jgriffithWTH.. is that an L or an I LOL02:53
*** dannywilson has joined #openstack-infra02:54
clarkbl02:57
*** Somay has joined #openstack-infra02:58
jgriffithclarkb: so it's correct on pypi... but not our mirror02:58
clarkbmay just need an update? I cant cgeck it now02:58
*** hichihara has quit IRC02:59
*** dannywilson has quit IRC02:59
*** hichihara has joined #openstack-infra02:59
*** hichihara has quit IRC02:59
jgriffithCool, if that's it devstack should work again, and the global update will go in (although that's not so critical)03:00
*** erlon has quit IRC03:01
*** Somay has quit IRC03:02
jgriffithclarkb: well now I'm confused... Thai is the author of the package on PyPi but he submitted a patch with the wrong letter03:03
*** marcusvrn1 has quit IRC03:06
*** marcusvrn has joined #openstack-infra03:06
*** ddieterly has quit IRC03:07
*** harlowja_ is now known as harlowja_away03:07
*** bhunter71 has quit IRC03:08
*** amotoki has joined #openstack-infra03:10
*** tobiash_ has joined #openstack-infra03:13
*** tobiash has quit IRC03:13
jgriffithclarkb: well.. it's fixed now03:14
jgriffithclarkb: if you fixed it with super powers... thanks :)03:14
jgriffithclarkb: if it fixed itself..... thanks just the same :)03:14
*** otter768 has quit IRC03:15
*** bknudson has quit IRC03:16
*** amotoki has quit IRC03:16
*** tsg has quit IRC03:19
*** unicell has joined #openstack-infra03:20
*** spzala has quit IRC03:22
*** pal has joined #openstack-infra03:25
*** esker has joined #openstack-infra03:25
*** unicell has quit IRC03:27
*** unicell has joined #openstack-infra03:28
*** hdd has joined #openstack-infra03:32
*** carl_baldwin has joined #openstack-infra03:32
*** sputnik13 has joined #openstack-infra03:33
*** ajmiller__ has joined #openstack-infra03:34
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Refine the query for #1326813 to pick up more hits  https://review.openstack.org/16915503:35
*** baoli has quit IRC03:36
*** ajmiller_ has quit IRC03:37
*** SumitNaiksatam has joined #openstack-infra03:38
*** esker has quit IRC03:42
*** hdd has quit IRC03:42
*** amotoki has joined #openstack-infra03:43
*** rfolco has quit IRC03:46
*** ayoung has quit IRC03:48
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Add fingerprint for XStatic-Angular-Irdragndrop dependency failure  https://review.openstack.org/16915603:49
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Add fingerprint for XStatic-Angular-Irdragndrop dependency failure  https://review.openstack.org/16915603:51
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Add fingerprint for XStatic-Angular-Irdragndrop dependency failure  https://review.openstack.org/16915603:52
*** asettle has joined #openstack-infra03:56
*** rfolco has joined #openstack-infra03:58
*** armax has quit IRC04:00
*** sputnik13 has quit IRC04:02
*** sputnik13 has joined #openstack-infra04:03
*** carl_baldwin has quit IRC04:05
*** ddieterly has joined #openstack-infra04:08
openstackgerritSpencer Krum proposed openstack-infra/system-config: Fully qualify puppetdb classes  https://review.openstack.org/16915704:09
*** krtaylor has quit IRC04:11
*** ddieterly has quit IRC04:12
*** achanda has quit IRC04:12
*** ghostpl_ has joined #openstack-infra04:13
*** xnox has quit IRC04:13
*** krtaylor has joined #openstack-infra04:13
*** ghostpl_ has quit IRC04:17
*** achanda has joined #openstack-infra04:19
*** hdd has joined #openstack-infra04:19
*** sputnik13 has quit IRC04:20
*** sputnik13 has joined #openstack-infra04:21
*** achanda has quit IRC04:27
*** sputnik13 has quit IRC04:30
*** achanda has joined #openstack-infra04:30
*** aduarte has joined #openstack-infra04:31
*** sputnik13 has joined #openstack-infra04:31
*** oomichi_ has joined #openstack-infra04:33
*** Longgeek_ has quit IRC04:35
*** achanda has quit IRC04:40
*** annegentle has quit IRC04:40
*** achanda has joined #openstack-infra04:40
*** btully has quit IRC04:41
*** carl_baldwin has joined #openstack-infra04:41
*** Somay has joined #openstack-infra04:42
*** Longgeek has joined #openstack-infra04:42
*** annegentle has joined #openstack-infra04:46
*** annegentle has quit IRC04:49
*** achanda has quit IRC04:51
*** thingee has joined #openstack-infra04:54
*** ildikov has quit IRC04:55
*** sputnik13 has quit IRC04:55
*** mrmartin has joined #openstack-infra04:56
*** ibiris_away is now known as ibiris04:56
*** yfried has quit IRC05:07
*** ddieterly has joined #openstack-infra05:09
stevemarsomething broke everything :(05:09
stevemarangular something or other05:09
*** btully has joined #openstack-infra05:10
greghaynesstevemar: XStatic-Angular-Irdragndrop?05:11
stevemargreghaynes, thats the one, is everything resolved now?05:11
greghaynesI dont believe so, I just had a failing test05:12
greghaynesbut theres some patches up to fix - seems like the pacakge might have been yanked from pypi for some reason05:12
greghaynesor possibly a mirroring issue05:12
stevemargreat timing, just before RC time :)05:13
*** ddieterly has quit IRC05:13
greghaynesyea, the package seems to be missing on pypi05:14
*** carl_baldwin has quit IRC05:15
*** hdd has quit IRC05:15
*** otter768 has joined #openstack-infra05:16
*** ociuhandu has quit IRC05:16
dougwigyou guys aware of the 4 hour horizon requirements breakage?05:18
dougwigahh yes, i see you are.05:19
*** harlowja_at_home has joined #openstack-infra05:19
*** otter768 has quit IRC05:20
greghayneshrm, I notice the package name doesnt match the name we have in requirements05:22
greghayneshttp://git.openstack.org/cgit/stackforge/xstatic-angular-lrdragndrop/tree/xstatic/pkg/angular_irdragndrop/__init__.py05:22
*** achanda has joined #openstack-infra05:22
greghaynesXStatic-IrDragNDrop vs XStatic-Angular-Irdragndrop05:22
*** harlowja_at_home has quit IRC05:25
*** prmtl has joined #openstack-infra05:25
*** dulek has joined #openstack-infra05:29
*** oomichi_ has quit IRC05:29
greghaynesoh my, it is actually uploaded as lrdragndrop05:35
*** pal has quit IRC05:37
StevenKHaha05:37
greghaynesah, it appears that is the real name and we just added a req with a totally wrong name?05:40
openstackgerritgreghaynes proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913205:42
*** aduarte has quit IRC05:42
*** vigneshvar has joined #openstack-infra05:43
*** vigneshvar_ has joined #openstack-infra05:43
*** ajo has joined #openstack-infra05:44
*** yfried has joined #openstack-infra05:45
greghayneshttp://logs.openstack.org/66/147366/3/gate/gate-tempest-dsvm-full/08d97e7/logs/devstacklog.txt.gz#_2015-02-24_23_33_56_65805:45
greghaynesso looks like that package existed once upon a time05:45
*** reed has quit IRC05:47
*** ildikov has joined #openstack-infra05:47
*** markvoelker has quit IRC05:48
*** yfried_ has joined #openstack-infra05:49
*** yfried has quit IRC05:49
greghaynesand https://pypi.python.org/pypi/XStatic-Angular-lrdragndrop/1.0.2.2 was just uploded today05:51
*** annegentle has joined #openstack-infra05:58
*** pcrews_ has joined #openstack-infra05:59
*** pcrews has quit IRC05:59
greghaynesand openstackci is not an owner on pypi05:59
*** pcrews_ has quit IRC05:59
*** ghostpl_ has joined #openstack-infra06:01
openstackgerritOpenStack Proposal Bot proposed openstack-infra/project-config: Normalize projects.yaml  https://review.openstack.org/16917406:01
*** annegentle has quit IRC06:02
*** mmedvede has quit IRC06:04
*** yfried_ is now known as yfried|afk06:06
*** pcrews has joined #openstack-infra06:06
*** achanda has quit IRC06:08
*** achanda has joined #openstack-infra06:08
*** thingee has quit IRC06:08
*** ghostpl_ has quit IRC06:09
*** ddieterly has joined #openstack-infra06:10
AJaegergreghaynes, it got renamed last week06:11
AJaegerseems we missed the wrong pypi setup during review ;(06:11
greghaynesah!06:12
AJaegerflaper87, could you merge the translations for glance, please? https://review.openstack.org/#/c/158570/06:12
*** pal has joined #openstack-infra06:12
AJaegergreghaynes, change I38385017423de00fc91ec4fe4293c29c746dda0706:12
AJaegerlet me email the author...06:12
*** ddieterly has quit IRC06:14
greghaynesAJaeger: So, I wonder how that works with the actual module - its named -irdragndrop06:15
greghaynesare we going to uplaod a pacakge lrdragndrop with a modulre irdragndrop06:15
greghaynesactually, I guess thats whats already uploaded06:15
AJaegergreghaynes, that looks wrong and needs to be changed IMHO.06:15
greghaynesyep, its just going to break horizon06:15
greghayneswe might have to do an import guard in horizon, rename the module, remove import guard?06:16
AJaegergreghaynes, can you fix it?06:16
AJaegergreghaynes, sorry, I can't give good advise here...06:16
greghaynesheh, its np, just thinking out loud :)06:16
*** markvoelker has joined #openstack-infra06:19
flaper87AJaeger: yup06:22
AJaegerthanks, flaper87 !06:23
*** yfried|afk is now known as yfried_06:24
*** BharatK has joined #openstack-infra06:24
*** markvoelker has quit IRC06:26
*** tnovacik has joined #openstack-infra06:27
*** amotoki has quit IRC06:27
*** mrda is now known as mrda-away06:27
*** Hal has joined #openstack-infra06:29
*** scheuran has joined #openstack-infra06:29
*** Hal is now known as Guest8133406:29
mikalHey, is there a simple way to see all the recent votes for a given gerrit user?06:31
mikalI'm trying to determine if the hyper-v ci for nova is as broken as I think it is?06:31
*** nmagnezi has joined #openstack-infra06:31
AJaegermikal, let me dig out something...06:34
mikalSure, just writing an email now and it would be nice to say if its passed at all in say the last 24 hours06:34
AJaegerhttps://review.openstack.org/#/q/reviewer:%22Oracle+ZFSSA+CI%22,n,z06:34
mikalAJaeger: can I search by the vote for that user?06:35
mikalhttps://review.openstack.org/#/q/project:openstack/nova+status:open+reviewer:%22Microsoft+Hyper-V+CI%22,n,z is what I have so far06:35
AJaegerhttp://stackalytics.com/report/driverlog contains hyper v06:36
mikalAhhh, nice. Thanks.06:37
AJaegermika, your URL should works and is the best I know for nova06:38
mikalAJaeger: yeah, it just doesn't filter for positive votes06:39
mikalBut the driverlog thing is close enough, so that keeps me rolling06:39
openstackgerritMerged openstack-infra/project-config: Normalize projects.yaml  https://review.openstack.org/16917406:39
AJaegerAh, you look for positive - should be possible...06:39
*** dboik has quit IRC06:40
AJaegermikal, btw. could you import translations for nova, please? https://review.openstack.org/#/c/162030/06:40
* mikal looks06:40
mikalDine06:40
mikalDone even06:40
AJaegerthanks!06:41
mikalNP06:41
*** Guest81334 has quit IRC06:43
AJaegermikal, do you know the login name of the hyper v ci?06:43
AJaegerYou should be able to add "label:Code-Review=1,aname"06:43
mikalI have the username, its "Microsoft Hyper-V CI" IIRC06:44
AJaegerbut I can't get that one working with name06:44
mikalOh, I see06:44
mikalWe need the username, not the long name06:44
AJaegermikal,exactly06:44
mikalThat should be on the wiki IIRC06:44
AJaegerThat'S my guess after the long name does not work.06:45
mikalhyper-v-ci it seems06:45
mikalhttps://wiki.openstack.org/wiki/ThirdPartySystems/Hyper-V_CI06:45
mikalOh, that wiki page says "offline for maintenance"06:45
*** deepakcs has joined #openstack-infra06:45
mikalOh, ignore me06:45
mikalThat was at the start of the month06:45
mikalIts nearly April now apparenlty06:45
AJaegerUse: reviewer:"Microsoft Hyper-V CI" label:Code-Review=1,hyper-v_ci@microsoft.com06:46
AJaegerbut that looks wrong -1 does not get anything either ;(06:46
mikalIts ok, the driverlog thing is enough I think06:47
*** YorikSar has joined #openstack-infra06:47
*** Somay has quit IRC06:47
*** jamielennox is now known as jamielennox|away06:49
AJaegermikal, ok - otherwise, you need to ask for more help during US time, hope there'll be somebody around with better gerrit search skills06:49
*** Somay has joined #openstack-infra06:50
AJaegermikal, this works: label:Code-Review=0,user:hyper-v_ci@microsoft.com06:50
AJaegerChange 0 to 1..06:50
*** marun has joined #openstack-infra06:51
AJaegerNote this matches only most recent review06:51
AJaegermikal, ignore me - it should give a result but does not - I'm giving up.06:51
mikalGood plan06:52
*** amotoki has joined #openstack-infra06:53
*** sergsh has joined #openstack-infra06:53
AJaegermikal, I saw your email - asking is a good idea ;)06:54
*** marun has quit IRC06:55
*** sdake has quit IRC06:56
*** tkelsey has joined #openstack-infra06:58
*** annegentle has joined #openstack-infra06:58
openstackgerritJan Hruban proposed openstack-infra/jenkins-job-builder: Add support for Display Upstream Changes Plugin  https://review.openstack.org/16919306:59
*** asettle has quit IRC07:01
flaper87https://review.openstack.org/#/c/169132/ :(07:01
*** amotoki has quit IRC07:01
*** amotoki has joined #openstack-infra07:02
*** markus_z has joined #openstack-infra07:02
*** jyuso has quit IRC07:03
AJaegerflaper87, see also change I38385017423de00fc91ec4fe4293c29c746dda0707:04
*** annegentle has quit IRC07:04
*** jyuso1 has joined #openstack-infra07:04
*** yamamoto_ has quit IRC07:04
*** yfried__ has joined #openstack-infra07:04
flaper87AJaeger: thank you!07:04
*** liusheng has joined #openstack-infra07:04
*** achanda has quit IRC07:05
*** tkelsey has quit IRC07:05
flaper87AJaeger: what's the next step there? Do we just have to wait until the new version is uploaded?07:05
AJaegerflaper87, I don't know ;(, let's tell the author. I reached out and told him already about the missing openstackci but not more. greghaynes was bringing this up one hour ago as well...07:06
*** yfried has joined #openstack-infra07:06
AJaegerflaper87, sorry, can't help further with this07:07
*** ajo_ has joined #openstack-infra07:07
flaper87AJaeger: :( ok, I guess I'll just have to wait07:07
flaper87AJaeger: no worries07:07
*** yfried_ has quit IRC07:08
*** ZZelle has quit IRC07:08
*** yfried__ has quit IRC07:09
*** ajo has quit IRC07:10
*** ddieterly has joined #openstack-infra07:10
AJaegerflaper87, now I see also the failures ;(07:13
AJaegerflaper87, greghaynes: Might be best to send an email to openstack-dev and copy Thai Tran07:14
*** teran has joined #openstack-infra07:14
*** ddieterly has quit IRC07:15
*** stevemar has quit IRC07:16
*** otter768 has joined #openstack-infra07:16
*** vigneshvar has quit IRC07:17
*** vigneshvar_ has quit IRC07:17
*** ala_ has joined #openstack-infra07:19
*** mpavone has joined #openstack-infra07:21
*** otter768 has quit IRC07:21
*** markvoelker has joined #openstack-infra07:22
*** rfolco has quit IRC07:24
*** MaxV has joined #openstack-infra07:25
*** arxcruz has joined #openstack-infra07:26
*** markvoelker has quit IRC07:27
AJaegerflaper87, greghaynes: For all the xstatic packages: They are not uploaded via CI ;(07:28
AJaegerThat's why reviewing did not notice it - there were no publish-to-pypi jobs07:29
*** nilasae has joined #openstack-infra07:29
AJaegerStill, the commit message for the rename should have raised some questions but didn't for me ;(07:29
*** MaxV has quit IRC07:30
*** MaxV has joined #openstack-infra07:30
greghaynesAJaeger: The old named package was uploaded, see http://logs.openstack.org/66/147366/3/gate/gate-tempest-dsvm-full/08d97e7/logs/devstacklog.txt.gz#_2015-02-24_23_33_56_65807:30
greghaynesI think the author must have deleted it?07:30
AJaegergreghaynes, might have been deleted - yes. It was uploaded manually, not via OpenStack CI.07:31
greghaynesyep07:31
greghaynesand same with the new one07:31
*** mpaolino has joined #openstack-infra07:32
*** liusheng has quit IRC07:33
* greghaynes needs to sleep, way past EOD07:33
*** liusheng has joined #openstack-infra07:34
AJaegergreghaynes, good night! Here the new day is already over 9hours old ;)07:34
*** MaxV has quit IRC07:35
StevenKAJaeger: Heh, not far enough in the future. The day is over 18 hours old here. :-)07:35
AJaegerStevenK :)07:36
*** rfolco has joined #openstack-infra07:36
*** jlanoux has joined #openstack-infra07:37
*** woodster_ has quit IRC07:40
*** jistr has joined #openstack-infra07:43
*** Somay has quit IRC07:52
*** ghostpl_ has joined #openstack-infra07:53
*** Hal has joined #openstack-infra07:55
*** btully has quit IRC07:55
*** Hal is now known as Guest2261407:55
*** YorikSar has quit IRC07:56
*** yamamoto_ has joined #openstack-infra07:57
*** AJaeger has quit IRC07:57
*** ghostpl_ has quit IRC07:58
*** yamamoto_ has quit IRC07:59
*** annegentle has joined #openstack-infra07:59
*** Guest22614 has quit IRC08:00
*** MaxV has joined #openstack-infra08:00
*** Murad has quit IRC08:01
*** notnownikki has joined #openstack-infra08:01
*** hashar has joined #openstack-infra08:03
*** liusheng has quit IRC08:03
*** rdopiera has joined #openstack-infra08:04
rdopierahi guys, we are having some problem with the global-requirements pip cache here, it seems: http://logs.openstack.org/23/167323/10/check/gate-horizon-pep8/97e8b53/console.html08:04
*** zz_johnthetubagu is now known as johnthetubaguy08:04
*** liusheng has joined #openstack-infra08:04
rdopieraCould not install requirement XStatic-Angular-Irdragndrop>=1.0.2.1 (from -r /home/jenkins/workspace/gate-horizon-pep8/requirements.txt (line 45)) because of error 404 Client Error: Not Found08:04
*** annegentle has quit IRC08:05
*** AJaeger has joined #openstack-infra08:09
*** AJaeger has quit IRC08:09
*** AJaeger has joined #openstack-infra08:09
*** ddieterly has joined #openstack-infra08:11
*** dizquierdo has joined #openstack-infra08:11
*** yfried_ has joined #openstack-infra08:11
*** prmtl has quit IRC08:13
*** yfried has quit IRC08:15
*** ddieterly has quit IRC08:15
*** prmtl has joined #openstack-infra08:15
*** yfried__ has joined #openstack-infra08:16
*** yfried_ has quit IRC08:16
*** markvoelker has joined #openstack-infra08:23
openstackgerritJulien Danjou proposed openstack-infra/project-config: Gnocchi: do not check requirements  https://review.openstack.org/16921008:25
*** markvoelker has quit IRC08:27
cschwederdopiera: i saw the same error on two jobs, added a comment to https://bugs.launchpad.net/openstack-ci/+bug/133455008:29
openstackLaunchpad bug 1334550 in OpenStack-Gate "Could not find any downloads that satisfy the requirement X" [Low,Fix released]08:29
*** derekh has joined #openstack-infra08:29
*** e0ne has joined #openstack-infra08:29
*** YorikSar has joined #openstack-infra08:30
*** ssam2 has joined #openstack-infra08:33
ttxAJaeger: should we push an alert ?08:37
AJaegerttx, might be a nice idea. flaper87, what do you think?08:38
ttxLooks like it's completely stuck08:38
flaper87ttx: AJaeger +2A for the alert08:38
AJaegerttx, is really everything failing? Then let's raise an alert and sent an email to openstack-dev08:38
flaper87it's *completely* stuck08:38
AJaegerflaper87 and ttx, will either of you sent an email to openstack-dev, please?08:39
AJaegerflaper87, we can still merge manuals and infra ;)08:39
ttxflaper87: if you have a good understanding of the fail, I'll let you write the email08:39
ttxI can send the alert based on the language you put in the email08:40
flaper87ok, I can send it08:40
flaper87lemme gather the required info08:41
AJaegerflaper87, ttx, there was some discussion on #openstack-horizon as well, see http://eavesdrop.openstack.org/irclogs/%23openstack-horizon/%23openstack-horizon.2015-03-31.log08:42
ttxIfear that fixing it will require some direct monkeying that is not available for us mortals08:43
*** Hal has joined #openstack-infra08:43
AJaegerttx, yeah08:43
*** Hal is now known as Guest6170308:44
ttxSergeyLukjanov could help but not sure he knows what lever to play with08:44
*** dtantsur|afk is now known as dtantsur08:44
*** e0ne has quit IRC08:45
*** ibiris is now known as ibiris_away08:45
AJaegerAgreed, we need a plan on how to fix this and I fear that needs some expert knowledge from our US fire fighter team ;)08:45
*** pelix has joined #openstack-infra08:46
flaper87ttx: AJaeger http://paste.openstack.org/show/bnrQTSuJ4H82e8AgCMq4/08:48
ttxflaper87: looks good08:50
flaper87sending08:50
ttxI propose alert "CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment."08:51
ttxdoes that sound good?08:51
flaper87ttx: it does08:51
* flaper87 will use it as subject08:51
flaper87:P08:51
ttx#status alert CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment.08:51
openstackstatusttx: sending alert08:51
-openstackstatus- NOTICE: CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment.08:53
*** ChanServ changes topic to "CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment."08:53
*** ghostpl_ has joined #openstack-infra08:54
*** ibiris_away is now known as ibiris08:54
anteayattx thanks08:54
ttxnow we can redirect questions to /topic :)08:55
openstackstatusttx: finished sending alert08:56
openstackgerritFabien Boucher proposed openstack-infra/system-config: Move some hardcoded values as parameters for zuul manifests  https://review.openstack.org/16833008:56
*** ociuhandu has joined #openstack-infra08:59
*** ghostpl_ has quit IRC08:59
anteayattx: ha ha ha08:59
* anteaya goes back to bed09:00
*** tkelsey has joined #openstack-infra09:00
*** annegentle has joined #openstack-infra09:00
*** ZZelle has joined #openstack-infra09:04
*** ghostpl_ has joined #openstack-infra09:04
*** yfried__ is now known as yfried|lunch09:05
*** annegentle has quit IRC09:06
*** ildikov has quit IRC09:06
*** ghostpl_ has quit IRC09:09
*** afazekas has joined #openstack-infra09:11
*** yamamoto has joined #openstack-infra09:12
*** ddieterly has joined #openstack-infra09:12
*** yfried|lunch is now known as yfried|afk09:15
*** Longgeek has quit IRC09:16
*** ddieterly has quit IRC09:16
*** nmagnezi_ has joined #openstack-infra09:17
*** nmagnezi has quit IRC09:17
*** otter768 has joined #openstack-infra09:17
*** mpaolino has quit IRC09:18
*** ildikov has joined #openstack-infra09:19
*** amotoki_ has joined #openstack-infra09:20
*** otter768 has quit IRC09:22
*** ajmiller_ has joined #openstack-infra09:23
*** markvoelker has joined #openstack-infra09:23
*** ajmiller__ has quit IRC09:26
openstackgerrityolanda.robla proposed openstack-infra/nodepool: Independent provider manager for polling tasks  https://review.openstack.org/16923509:27
*** markvoelker has quit IRC09:28
openstackgerritRob Cresswell proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913209:28
*** robcresswell has joined #openstack-infra09:29
*** yfried|afk is now known as yfried|lunch09:36
odyssey4meAny chance we can get https://review.openstack.org/#/c/168531/ reviewed so that we can fix the stackforge packaging configuration for angular lrdragndrop ?09:36
odyssey4meI have the patch ready, but it's being rejected.09:36
*** punith has joined #openstack-infra09:38
odyssey4meAJaeger ^09:38
*** e0ne has joined #openstack-infra09:39
*** ghostpl_ has joined #openstack-infra09:41
*** jamielennox|away is now known as jamielennox09:41
robcresswellJust been chasing this up on the Horizon side, but I'm not terribly familiar with the packaging workflow: https://bugs.launchpad.net/horizon/+bug/143854309:41
openstackLaunchpad bug 1438543 in OpenStack Dashboard (Horizon) "wrong package name 'XStatic-Angular-Irdragndrop' in horizon/requirements.txt" [Critical,Confirmed] - Assigned to Rob Cresswell (robcresswell)09:41
*** ajo_ is now known as ajo09:42
*** jlibosva has joined #openstack-infra09:46
*** yfried|lunch is now known as yfried|afk09:46
*** jamielennox is now known as jamielennox|away09:47
odyssey4merobcresswell yeah, I see so - it would seem that the packaging was originally setup in https://github.com/stackforge/xstatic-angular-lrdragndrop09:47
odyssey4methat repo was recently renamed (implied in https://review.openstack.org/#/c/168531/ )09:47
odyssey4meso I have a patch for that repo ready, but gerrit is rejecting the patch - very possibly because there is no .gitreview present09:48
robcresswellodyssey4me: Rename is here: https://review.openstack.org/#/c/167798/09:50
robcresswellodyssey4me: There are patches out to rename the Horizon imports and update global-reqs. Do they not solve the issues?09:51
odyssey4merobcresswell You mean https://review.openstack.org/169132 and https://review.openstack.org/169133 ?09:51
robcresswellodyssey4me: Indeed. Just trying to follow as best I can.09:51
*** cdent has joined #openstack-infra09:52
*** ildikov has quit IRC09:54
*** rlandy has joined #openstack-infra09:54
rdopieraI don't think those patches will merge with dvsm failing...09:55
rdopieraodyssey4me: you don't need the .gitreview file09:56
*** liusheng has quit IRC10:00
*** yamamoto has quit IRC10:01
*** liusheng has joined #openstack-infra10:01
*** Guest61703 has quit IRC10:01
*** annegentle has joined #openstack-infra10:01
AJaegerodyssey4me, 168531 is a patch for lrdgragndrop, nothing I can do10:01
AJaegerrdopiera, which patch is rejected?10:02
odyssey4meAJaeger ah, of course - let me track down the project cores10:03
*** dims has joined #openstack-infra10:03
AJaegerodyssey4me, https://review.openstack.org/#/admin/groups/385,members10:03
openstackgerritRob Cresswell proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913210:03
*** achanda has joined #openstack-infra10:06
*** annegentle has quit IRC10:06
*** ildikov has joined #openstack-infra10:09
*** jcoufal has joined #openstack-infra10:09
*** achanda has quit IRC10:10
*** ominakov has joined #openstack-infra10:13
*** ddieterly has joined #openstack-infra10:13
ekarlso4hmmms10:13
ekarlsodoesn't devstack work with pgsql ?10:13
*** ominakov has quit IRC10:13
ekarlso2015-03-31 10:11:10.249 | FATAL:  no pg_hba.conf entry for host "192.168.27.100", user "root", database "template1", SSL off10:13
*** dhritishikhar_ has joined #openstack-infra10:15
*** yfried|afk is now known as yfried|lunch10:16
*** Longgeek has joined #openstack-infra10:17
*** ddieterly has quit IRC10:17
*** yamamoto has joined #openstack-infra10:17
*** yfried|lunch is now known as yfried10:17
*** pc_m has joined #openstack-infra10:18
*** e0ne is now known as e0ne_10:19
*** e0ne_ is now known as e0ne10:23
*** markvoelker has joined #openstack-infra10:24
*** btully has joined #openstack-infra10:25
rdopieraAJaeger: the global-requirements.txt one: https://review.openstack.org/#/c/169132/10:26
*** dhritishikhar_ has quit IRC10:28
*** jlanoux_ has joined #openstack-infra10:28
*** markvoelker has quit IRC10:29
*** btully has quit IRC10:30
*** Longgeek has quit IRC10:30
ekarlsoso devstack is not tested with pgsql ?10:31
*** jlanoux has quit IRC10:31
*** teran has quit IRC10:31
*** samueldmq has joined #openstack-infra10:36
samueldmqmorning10:36
samueldmqso we already have something to solve early in the morning? :-)10:36
*** e0ne is now known as e0ne_10:37
*** yamamoto has quit IRC10:37
*** ghostpl_ has quit IRC10:38
*** e0ne_ is now known as e0ne10:40
yfriedAJaeger: any news on gate status?10:41
*** ghostpl_ has joined #openstack-infra10:42
*** jlanoux has joined #openstack-infra10:43
yfriedboris-42: see topic10:45
yfriedAJaeger: is there a way to disable "recheck" temporarily?10:46
*** jlanoux_ has quit IRC10:46
openstackgerritMarton Kiss proposed openstack-infra/system-config: Add backup scripts to ask.o.o site  https://review.openstack.org/16926010:49
*** claudiub has joined #openstack-infra10:52
AJaegeryfried, not sure about disabling ;(10:52
AJaegerrdopiera, do you know which changes we need? I've started creating an etherpad where we can collect all information: https://etherpad.openstack.org/p/lrdragndrop-stuck10:53
*** xnox has joined #openstack-infra10:55
robcresswellAJaeger: Those patches already listed seem key. I'm not sure if tests will have to be disabled to allow the merges though.10:56
*** gmann_ has joined #openstack-infra10:57
*** gmann_ has left #openstack-infra10:57
AJaegerrobcresswell, thanks10:57
robcresswelltbh, the old package should not have been taken down before the update10:57
AJaegerttx, flaper87 fungi, clarkb, jeblair: https://etherpad.openstack.org/p/lrdragndrop-stuck10:57
flaper87AJaeger: looking10:59
*** mmedvede has joined #openstack-infra11:00
* AJaeger is just trying to get facts together so that the infra admin team can quickly help...11:00
robcresswellAJaeger: Just adding any useful info I can think of.11:01
AJaegerrobcresswell, Thanks!11:01
robcresswellno problem :)11:01
AJaegerrobcresswell, that's what we need right now - all information easily accessible...11:02
*** pal has quit IRC11:02
*** annegentle has joined #openstack-infra11:02
robcresswellAJaeger: Yeah, understood. I've updated both patches to link the relevant bugs/ dependents. But they are failing the tests; I wonder if the tests may need disabling for the global-req change to pass, then reenabled for the horizon change.11:03
AJaegerrobcresswell, infra admins can approve without gating if really necessary.11:04
AJaegerI'm not an infra admin - so just help to collect data so that they can easier access the situation, they have far more knowledge here than I have...11:05
robcresswellAJaeger: Gotcha. I'm in the same boat. A little out of my depth, just trying to help out.11:06
*** yfried is now known as yfried|afk11:06
*** ildikov has quit IRC11:06
*** BharatK has quit IRC11:07
*** annegentle has quit IRC11:07
AJaegerrobcresswell, thanks for your help11:07
*** jcoufal has quit IRC11:08
*** ujuc has joined #openstack-infra11:08
*** jcoufal has joined #openstack-infra11:08
*** dmellado has quit IRC11:08
openstackgerritFlavio Percoco proposed openstack-infra/project-config: Install required services for Zaqar  https://review.openstack.org/16926911:10
*** jcoufal has quit IRC11:10
*** flaper87 has quit IRC11:10
*** yfried|afk is now known as yfried11:11
*** ddieterly has joined #openstack-infra11:13
*** teran has joined #openstack-infra11:15
*** tsufiev is now known as tsufiev_11:15
*** Longgeek has joined #openstack-infra11:17
*** ddieterly has quit IRC11:18
*** otter768 has joined #openstack-infra11:18
*** _nadya_ has joined #openstack-infra11:19
*** teran has quit IRC11:19
*** Hal has joined #openstack-infra11:20
*** Hal is now known as Guest4717711:20
rdopierarobcresswell: I wonder if I should just upload a package under the old name11:21
rdopierarobcresswell: to pypi11:21
*** ssam2 has quit IRC11:21
*** ildikov has joined #openstack-infra11:22
*** flaper87 has joined #openstack-infra11:22
*** otter768 has quit IRC11:23
*** mpaolino has joined #openstack-infra11:24
*** pblaho has joined #openstack-infra11:24
*** markvoelker has joined #openstack-infra11:25
*** teran has joined #openstack-infra11:25
*** flaper87 has quit IRC11:26
*** flaper87 has joined #openstack-infra11:26
*** salv-orlando has quit IRC11:26
AJaegerrdopiera, that'S one option out of the mess - the question is why was renaming needed then?11:26
AJaegerrdopiera, do you have permissions?11:26
*** hogepodge has quit IRC11:27
robcresswellrdopiera: That is probably the safest and most sensible solution.11:27
*** sc68cal has quit IRC11:27
openstackgerritJoshua Hesketh proposed openstack-infra/zuul: Add support for 'connection' concept  https://review.openstack.org/12152811:27
openstackgerritJoshua Hesketh proposed openstack-infra/zuul: Document the new connections in zuul  https://review.openstack.org/16927011:27
*** sc68cal has joined #openstack-infra11:28
openstackgerritMerged openstack-infra/project-config: Move aiopcpu to nova check queue  https://review.openstack.org/16904311:29
*** markvoelker has quit IRC11:29
*** changbl has quit IRC11:30
rdopierarobcresswell: I just did that, and discovered another problem11:31
rdopierarobcresswell: the code on stackforge is two versions behind the code on pypi11:31
AJaegerrdopiera, please update the etherpad with what you did and learnt11:31
AJaegerrdopiera, this one? https://pypi.python.org/pypi/XStatic-IrDragNDrop/1.0.2.2 That misses the Angular in the name11:32
*** jistr is now known as jistr|english11:32
*** jistr|english is now known as jistr|class11:33
rdopieraAJaeger: I just did11:33
rdopieraargh11:33
*** deepakcs has quit IRC11:33
*** ssam2 has joined #openstack-infra11:34
rdopieraAJaeger: ok, fixed11:34
AJaegerrdopiera, do you know why the package was renamed?11:34
robcresswellAJaeger: from 'I' to 'l'?11:34
AJaegerrobcresswell, yes11:35
robcresswellAJaeger: The original github repo is called lrdragndrop after the author11:35
rdopieraAJaeger: because the original project is named with l11:35
robcresswellat some point, someone saw the 'l' and though it was 'I'11:35
rdopieraAJaeger: and it was packager's mistake to use I11:35
AJaegerrdopiera, AH! Understood now.11:35
rdopieraand he forgot about the fundamental rule about releases11:36
rdopieraonce you release something, it stays releases11:36
rdopieraonce you release something, it stays released11:36
rdopierayou cannot unrealease anything11:36
AJaegerrdopiera, please tell him once he shows up ;)11:37
openstackgerritDerek Higgins proposed openstack-infra/zuul: Document ZUUL_CHANGES  https://review.openstack.org/12924311:37
robcresswellThe horizon patch has passed jenkins, so once the global-reqs passes it should be fine.The bugs are marked so should close themselves.11:37
rdopieraAJaeger: I will do better, I will add it to the docs about creating xstatic packages11:37
robcresswellJust need reviews once Jenkins has passed now :)11:37
AJaegerrdopiera, can we upload the xstatic packages via gerrit in the future?11:38
AJaegerrobcresswell, that quickly?11:38
rdopieraAJaeger: yes, but I couldn't get that to work, I would appreciate any help with it11:38
rdopieraAJaeger: whenever you or anyone else has time11:39
AJaegerrdopiera, could you join the infra meeting at 19:00 UTC today? Then let's add it as a topic there...11:39
robcresswellAJaeger: For some reason, the horizon patch had already passed, I have no idea why. The global reqs patch has a recheck running from rdopiera.11:40
*** ssam2 has quit IRC11:40
rdopieracertainly11:40
AJaegerrdopiera, are xstatics some how specail from packaging point of view?11:40
rdopieraAJaeger: no, they are regular python packages11:40
*** salv-orlando has joined #openstack-infra11:40
rdopieraAJaeger: as mundane, as possible11:40
AJaegerrdopiera, then it should be trivial to upload them - just add publish-to-pypi to them...11:40
AJaegerrdopiera, let me do a quick patch...11:41
rdopieraAJaeger: if you can make a patch on one, I can do the rest11:41
rdopieraAJaeger: thanks11:41
*** tsufiev_ is now known as tsufiev11:42
rdopieraAJaeger: by the way, if you see any other checks or anything that would be nice to have there11:42
AJaegerrdopiera, let's ask the other infra cores for suggestions as well... I need to do some other stuff ;)11:43
AJaegerbut will soon have a patch...11:43
openstackgerritAndreas Jaeger proposed openstack-infra/project-config: Setup publishing to pypi for xstatic-angular  https://review.openstack.org/16927411:44
AJaegerrdopiera, feel free to take my patch and finish it - I just took the first project to show what needs to be done11:44
AJaegerand if you do, add yourself as co-author11:44
AJaegerttx, you can send an allclear again and ask for rechecks...11:45
ttxAJaeger: just a sec11:46
AJaegerttx: rdopiera, gave us some breathing time ;)11:46
rdopieraAJaeger: how is the upload triggered?11:46
rdopieraAJaeger: by tagging a release?11:47
rdopieraAJaeger: or on every commit?11:47
*** dmellado has joined #openstack-infra11:47
ttx#status ok Check/Gate unstuck, feel free to recheck your abusively-failed changes.11:47
openstackstatusttx: sending ok11:47
AJaegerrdopiera, http://docs.openstack.org/infra/manual/drivers.html#tagging-a-release - tagging a release11:47
AJaegerttx, thanks!11:47
rdopieraAJaeger: ok11:48
ttxrdopiera: thx for saving the day!11:48
ttxAJaeger: and thanks for seeing this through.11:48
rdopierasorry for the kerfuffle, it's all caused by inadequate documentation and processes on our side11:48
AJaegerttx, you're welcome11:48
*** ChanServ changes topic to "Discussion of OpenStack Developer and Community Infrastructure | docs http://docs.openstack.org/infra/manual/ http://ci.openstack.org/ | bugs https://storyboard.openstack.org/ | source https://git.openstack.org/cgit/openstack-infra/"11:49
AJaegerrdopiera, let's follow the usual pypi release process as everybody else and we can all share and improve the same documentation ;)11:49
-openstackstatus- NOTICE: Check/Gate unstuck, feel free to recheck your abusively-failed changes.11:49
robcresswellrdopiera, AJaeger: Thanks for the fix.11:50
AJaegerI'm retriggering https://review.openstack.org/#/c/169132/ - it fails right now, the recheck was too early11:50
openstackgerritAndreas Jaeger proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913211:51
openstackstatusttx: finished sending ok11:51
robcresswellAJaeger: I think a recheck would've worked fine, rather than a new patch, no?11:51
robcresswelleither way, thanks11:52
AJaegerrobcresswell, no, it wouldn't - a recheck would be ignored right now.11:52
robcresswellAJaeger: Ah, okay11:52
flaper87w000t11:52
*** ihrachyshka has joined #openstack-infra11:52
flaper87thanks folks11:52
AJaegerThe change was failing and the fastest way to queue it in again is not waiting until it'S finished and then recheck but do a new patch...11:52
*** punith has quit IRC11:52
AJaegerflaper87, care to followup on your email with the all-clear?11:53
robcresswellAJaeger: Yep, I wasn;t sure whether recheck would reset a patch that was currently being checked.11:53
*** ssam2 has joined #openstack-infra11:53
*** weshay has joined #openstack-infra11:53
*** MaxV has quit IRC11:53
flaper87AJaeger: will do, right away11:54
flaper87AJaeger: mind to point me to the summary of the fix ?11:54
flaper87if there's any11:54
*** pal_ has joined #openstack-infra11:54
AJaegerflaper87, rdopiera uploaded the deleted package again ;)11:54
flaper87ooook11:54
AJaegerNOw we can do the change properly...11:54
*** ldnunes has joined #openstack-infra11:55
*** tiswanso has quit IRC11:55
flaper87sent11:55
openstackgerritAndreas Jaeger proposed openstack-infra/project-config: Setup publishing to pypi for xstatic-angular  https://review.openstack.org/16927411:56
*** mpaolino has quit IRC11:57
*** mpaolino has joined #openstack-infra11:59
*** mpavone has quit IRC12:00
*** woodster_ has joined #openstack-infra12:01
*** hdd has joined #openstack-infra12:02
*** annegentle has joined #openstack-infra12:03
*** ildikov has quit IRC12:03
openstackgerritFlavio Percoco proposed openstack-infra/project-config: Install required services for Zaqar  https://review.openstack.org/16926912:05
*** annegentle has quit IRC12:08
*** dprince has joined #openstack-infra12:09
*** erlon has joined #openstack-infra12:09
*** MaxV has joined #openstack-infra12:10
*** _nadya_ has quit IRC12:12
*** dkranz has joined #openstack-infra12:12
*** aysyd has joined #openstack-infra12:13
*** bswartz has quit IRC12:13
*** ddieterly has joined #openstack-infra12:14
*** mpavone has joined #openstack-infra12:14
*** mpaolino has quit IRC12:16
*** yamamoto has joined #openstack-infra12:16
*** hdd has quit IRC12:16
*** markvoelker has joined #openstack-infra12:17
*** yfried is now known as yfried|afk12:17
*** bhunter71 has joined #openstack-infra12:18
*** kgiusti has joined #openstack-infra12:18
*** ildikov has joined #openstack-infra12:18
*** ddieterly has quit IRC12:19
*** amotoki_ has quit IRC12:20
openstackgerritRadomir Dopieralski proposed openstack-infra/project-config: Setup publishing to pypi for xstatic-*  https://review.openstack.org/16927412:23
*** yfried|afk is now known as yfried12:24
*** dims has quit IRC12:25
*** adalbas has joined #openstack-infra12:25
*** dims has joined #openstack-infra12:25
*** claudiub has quit IRC12:27
*** claudiub has joined #openstack-infra12:27
*** mpaolino has joined #openstack-infra12:28
*** dboik has joined #openstack-infra12:30
*** dkliban_afk is now known as dkliban12:31
*** Guest47177 has quit IRC12:31
*** robcresswell has left #openstack-infra12:33
*** amotoki_ has joined #openstack-infra12:34
fungidoes anyone have a summary on how the package rename (which happened on friday) didn't cause any problems for integration tests until today?12:34
*** amotoki_ has quit IRC12:34
fungimanual actions taken on pypi.python.org is my take from briefly skimming the scrollback here12:36
*** yfried is now known as yfried|afk12:37
*** Hal has joined #openstack-infra12:37
*** Hal is now known as Guest166312:38
*** tiswanso has joined #openstack-infra12:38
*** yamamoto has quit IRC12:39
*** yfried|afk is now known as yfried12:41
*** e0ne is now known as e0ne_12:42
*** bknudson has joined #openstack-infra12:42
*** radez_g0n3 is now known as radez12:43
*** e0ne_ is now known as e0ne12:43
*** smccully has joined #openstack-infra12:47
fungiaha. mailing list thread says the original package name, which horizon was still relying on, got _deleted_ from pypi (seriously, who deletes packages from pypi like that?!?)12:50
*** gordc has joined #openstack-infra12:50
*** dboik has quit IRC12:50
mordredfungi: wow. that's special12:51
fungiindeed12:51
*** dboik has joined #openstack-infra12:52
*** yfried has quit IRC12:53
*** kerpan has joined #openstack-infra12:53
*** tnovacik has quit IRC12:53
*** chris613 has joined #openstack-infra12:53
*** bswartz has joined #openstack-infra12:54
*** ildikov_ has joined #openstack-infra12:58
*** jistr|class is now known as jistr12:58
*** ddieterly has joined #openstack-infra12:58
fungirdopiera: aha, it was you. i guess we each get to learn that lesson at some point! deleting releases from pypi isn't really ever a good idea12:59
fungithanks for straightening it out12:59
openstackgerritClaudiu Belu proposed openstack-infra/project-config: Adds cloud-init to StackForge  https://review.openstack.org/16929313:00
openstackgerritRadomir Dopieralski proposed openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913213:00
*** ildikov has quit IRC13:00
*** nilasae has quit IRC13:01
*** ajo has quit IRC13:01
*** ajo has joined #openstack-infra13:01
rdopierafungi: it's actually more complicated, we need to do the change in several steps...13:01
fungirdopiera: i expect so. figured renaming the git repo was only one of many13:02
*** annegentle has joined #openstack-infra13:02
*** smccully_ has joined #openstack-infra13:05
*** hogepodge has joined #openstack-infra13:05
*** baoli has joined #openstack-infra13:05
*** prmtl has quit IRC13:06
*** tonytan4ever has joined #openstack-infra13:06
*** subscope has joined #openstack-infra13:07
*** nmagnezi_ is now known as nmagnezi13:08
*** e0ne is now known as e0ne_13:08
AJaegerfungi, https://etherpad.openstack.org/p/lrdragndrop-stuck has all information13:08
fungithanks!13:09
*** eharney has joined #openstack-infra13:10
mrmartinfungi: I've added the last piece of ask.o.o backup https://review.openstack.org/#/c/169260/ but it is failing to build until the two other pgsql-backup puppet not gets an approval13:10
*** dboik has quit IRC13:11
AJaegerrdopiera, do you want to fix the xstatic patch or shall I?13:11
*** radez is now known as radez_g0n313:11
*** macjack has joined #openstack-infra13:11
rdopieraAJaeger: what do you mean?13:12
rdopieraaah13:12
rdopieraI see, I will fix it13:12
AJaegerrdopiera, thanks!13:12
mrmartinfungi: I cannot participate today in the infra meeting, but the ask.o.o seems to be ok, I cannot reproduce the Chinese lang solr search issue on the new instance and dev environment, so if the backup is passing, then I don't know about more open issues.13:12
AJaegerRemember adding yourself as "Co-Authored-By"13:12
*** e0ne_ is now known as e0ne13:13
openstackgerritRadomir Dopieralski proposed openstack-infra/project-config: Setup publishing to pypi for xstatic-*  https://review.openstack.org/16927413:13
*** peristeri has joined #openstack-infra13:13
ctlaughkrtaylor: I just saw your PM and responded.13:14
openstackgerritRadomir Dopieralski proposed openstack-infra/project-config: Setup publishing to pypi for xstatic-*  https://review.openstack.org/16927413:14
rdopieraAJaeger: roger13:14
*** btully has joined #openstack-infra13:14
AJaegerrdopiera, looks fine, thanks!13:15
*** ddieterly has quit IRC13:16
*** ddieterly has joined #openstack-infra13:17
*** btully has quit IRC13:19
*** otter768 has joined #openstack-infra13:19
*** mriedem_away is now known as mriedem13:20
*** ddieterly has quit IRC13:21
*** ayoung has joined #openstack-infra13:23
openstackgerritJoshua Hesketh proposed openstack-infra/zuul: Document the new connections in zuul  https://review.openstack.org/16927013:23
openstackgerritJoshua Hesketh proposed openstack-infra/zuul: Add support for 'connection' concept  https://review.openstack.org/12152813:23
openstackgerritJoshua Hesketh proposed openstack-infra/zuul: Add in extra connections tests  https://review.openstack.org/16930013:23
*** dustins has joined #openstack-infra13:24
*** otter768 has quit IRC13:24
*** wayneeseguin has quit IRC13:24
*** tnovacik has joined #openstack-infra13:25
*** unicell1 has joined #openstack-infra13:26
*** unicell has quit IRC13:26
*** mpavone has quit IRC13:27
*** mfink has joined #openstack-infra13:28
*** jaypipes has quit IRC13:29
*** xyang1 has joined #openstack-infra13:30
*** ssam2 has quit IRC13:30
*** dboik has joined #openstack-infra13:31
*** spzala has joined #openstack-infra13:31
*** garyh has joined #openstack-infra13:41
*** dhritishikhar_ has joined #openstack-infra13:42
*** jaypipes has joined #openstack-infra13:42
openstackgerritClaudiu Belu proposed openstack-infra/project-config: Adds cloud-init to StackForge  https://review.openstack.org/16929313:42
*** hdd has joined #openstack-infra13:43
*** macjack has quit IRC13:43
*** ssam2 has joined #openstack-infra13:43
*** ddieterly has joined #openstack-infra13:45
*** masoz has quit IRC13:46
*** tsg_ has joined #openstack-infra13:46
*** amitgandhinz has joined #openstack-infra13:46
openstackgerritJordan Pittier proposed openstack-infra/system-config: Third party CI: document how to properly format message with Gerrit Trigger  https://review.openstack.org/16771613:48
fungimrmartin: thanks for the update. sounds like we're close to being able to do a cut-over maintenance for it13:50
*** mwagner_lap has quit IRC13:51
*** amotoki_ has joined #openstack-infra13:51
rcarrillocruzhey folks, may I please get another core review on https://review.openstack.org/#/c/167553/ ? just need another one to bring the project to stackforge13:52
rcarrillocruzthanks13:52
*** ddieterly has quit IRC13:54
*** armax has joined #openstack-infra13:55
*** ildikov_ has quit IRC13:57
mordredfungi: you want an easy review? https://review.openstack.org/#/c/168671/1 :)13:57
fungimordred: any idea why the parent change wasn't approved already?13:59
*** sigmavirus24_awa is now known as sigmavirus2413:59
mordredfungi: nope. but I was going to approve it in a sec13:59
fungitoo late14:00
mordredwoot!14:00
mordredless things remaining in the queue == good14:00
*** wenlock has joined #openstack-infra14:01
mordredfungi: also, while I'm bugging you - can I get an opinion on https://review.openstack.org/#/c/168734/ ?14:02
mordredfungi: it's more urllib warnings14:02
*** andreykurilin has joined #openstack-infra14:03
*** andreykurilin has left #openstack-infra14:03
mordredfungi: in this case, the rackspace deprecated cert thing which makes using this stuff very verbose/chatty on a topic we can't fix14:03
sigmavirus24mordred: that's still not fixed?14:03
* sigmavirus24 goes to bug people14:03
fungisigmavirus24: you should totally bug people14:04
*** zz_dimtruck is now known as dimtruck14:04
*** sabeen has joined #openstack-infra14:04
mordredsigmavirus24: nope14:04
sigmavirus24It was supposed to have been fixed by last week14:04
* sigmavirus24 is tracking down the people who promised that14:04
*** mrmartin has quit IRC14:05
mordredsigmavirus24: oh - wait a sec ..14:05
jrollsigmavirus24: wow, my nags went nowhere :|14:05
*** ayoung has quit IRC14:05
fungi /usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/connection.py:251: SecurityWarning: Certificate has no `subjectAltName`, falling back to check for a `commonName` for now. This feature is being removed by major browsers and deprecated by RFC 2818. (See https://github.com/shazow/urllib3/issues/497 for details.)14:05
mordredsigmavirus24: I just retested without my silencing of it14:05
mordredfungi: did you get that recently?14:06
sigmavirus24fungi: I know the error14:06
* fungi is still getting that when testing with novaclient on rax just now14:06
sigmavirus24fungi: I added that warning myself14:06
*** gokrokve has joined #openstack-infra14:06
mordred:)14:06
fungiyep, so i gathered14:06
* sigmavirus24 is an evil person14:06
fungijust confirming it hasn't changed on us14:06
*** btully has joined #openstack-infra14:06
AJaegermordred, do you want an easy review? ;) https://review.openstack.org/168135 or https://review.openstack.org/168148 or https://review.openstack.org/168159 is my spring cleanup14:07
fungiit's sandwiched between a couple of the "you're running a python 2.x more than three months old, you should upgrade" warnings, so it's not as easy to spot as it could be14:07
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/system-config: Expose puppetmaster_server param on openstack_project::puppetmaster  https://review.openstack.org/16857214:08
*** yfried has joined #openstack-infra14:08
*** dkranz has quit IRC14:08
mordredfungi: you know, when I go to test that I'm getting it, I really should test against rax14:08
fungii wondered if you stopped seeing it because you were connecting to hpcloud ;)14:09
*** mpavone has joined #openstack-infra14:09
*** ChuckC has quit IRC14:09
*** mpaolino has quit IRC14:10
*** nelsnelson has joined #openstack-infra14:10
sigmavirus24fungi: that python warning isn't mine =P14:11
*** _nadya_ has joined #openstack-infra14:11
openstackgerritMonty Taylor proposed openstack-infra/shade: Disable warnings about old Rackspace certificates  https://review.openstack.org/16873414:11
fungisigmavirus24: yeah, that's bubbling up from urllib3 directly, i gather14:11
mordredok. I have a better way of filtering just that one14:11
mordredShrews: ^^ you'll like that better14:11
Shrewsmordred: i hate it.14:12
Shrewsthough i haven't looked yet14:12
* mordred throws a cow at Shrews14:12
*** baoli has quit IRC14:12
jrollmordred: "there is nothing a user can do" they can call up rackspace and complain loudly :)14:12
mordredjroll: we're done that already :)14:13
*** carl_baldwin has joined #openstack-infra14:13
*** dboik_ has joined #openstack-infra14:13
Shrewslet's all complain to jroll on a daily basis until it is fixed14:13
*** ildikov_ has joined #openstack-infra14:13
jrollmordred: not loudly enough, apparently :|14:13
sigmavirus24fungi: they both come from urllib3, but I only added one of them =P14:13
sdaguejroll: we should have the warning include the rax support line number then14:14
jrollheh14:14
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Disable warnings about old Rackspace certificates  https://review.openstack.org/16932714:14
mordredfungi: why not add it there too ^ ?14:14
*** dboik_ has quit IRC14:14
*** dboik_ has joined #openstack-infra14:15
*** dboik has quit IRC14:16
sigmavirus24so best estimate I can get now is early April (is when that certificate will work)14:16
sigmavirus24mordred: to be fair, the certificate isn't old14:16
openstackgerritMonty Taylor proposed openstack-infra/shade: Create swift container if it does not exist  https://review.openstack.org/16782314:17
*** dhritishikhar_ has quit IRC14:18
openstackgerritMonty Taylor proposed openstack-infra/shade: Fix a use of in where it should be equality  https://review.openstack.org/16910514:18
*** ayoung has joined #openstack-infra14:18
mordredsigmavirus24: fair. "deprecated" makes me think "old"14:19
openstackgerritMonty Taylor proposed openstack-infra/shade: Namespace caching per cloud  https://review.openstack.org/16861514:19
openstackgerritMonty Taylor proposed openstack-infra/shade: Allow for passing cache class in as a parameter  https://review.openstack.org/16214814:19
openstackgerritMonty Taylor proposed openstack-infra/shade: Add 'rebuild' to shade  https://review.openstack.org/16106714:19
openstackgerritMonty Taylor proposed openstack-infra/shade: Migrate API calls to task management  https://review.openstack.org/16866514:19
openstackgerritMonty Taylor proposed openstack-infra/shade: Add task management framework  https://review.openstack.org/16866414:19
sigmavirus24mordred: yeah I get that14:19
*** ZZelle has quit IRC14:19
mordredfungi, Shrews: that's all basically just rebasing that stack - I pulled three patches out and made them standalone and not in the dependency chain14:19
sigmavirus24Certificates without SANs are "okay" but every major browser is removing support for them because it's a good hardening measure14:19
sigmavirus24We're planning to do the same, but we're also not sure /when/ we'll do that14:20
*** ZZelle has joined #openstack-infra14:20
sigmavirus24Things like removing support for RC4 ciphers, however, are easy to decide to do on the other hand =P14:20
*** timcline has joined #openstack-infra14:21
Shrewsmordred: that container change... there was a change i made in Feb to remove the call to create the container within create_object(). It shouldn't be necessary. You're finding it is?14:21
*** dkranz has joined #openstack-infra14:21
Shrewsmordred: commit 91bc4014:22
*** carl_baldwin has quit IRC14:22
*** enikanorov has quit IRC14:23
*** mpaolino has joined #openstack-infra14:23
*** timcline has quit IRC14:24
*** sergsh has quit IRC14:24
*** timcline has joined #openstack-infra14:24
openstackgerritMerged openstack-infra/shade: Add api-level timeout parameter  https://review.openstack.org/16859614:26
*** timcline_ has joined #openstack-infra14:26
openstackgerritMerged openstack-infra/shade: Pass socket timeout to all of the Client objects  https://review.openstack.org/16867114:26
*** enikanorov has joined #openstack-infra14:27
*** tsg_ has quit IRC14:27
openstackgerritDavide Michelino proposed openstack-infra/project-config: Add Networking-portforwarding project  https://review.openstack.org/16933214:28
*** sdake has joined #openstack-infra14:29
mordredShrews: yes. it is necessary14:29
mordredShrews: on at least one of the clouds14:29
*** timcline has quit IRC14:30
*** dboik_ has quit IRC14:30
Shrewsmordred: poo. then let's put it back in create_object() instead14:30
mordredShrews: but - I agree it should not be in create_object in general - that was the wrong level ... it _should_ be in upload_image because the user shouldn't need to know they wanted a container14:30
mordredShrews: oh - ok14:30
mordredShrews: see, you'd convinced me on taht one :)14:30
*** zz_jgrimm is now known as jgrimm14:31
Shrewsi think your own argument aligns with my suggestion14:31
mordredhrm14:31
mordredyou're probably right14:31
Shrewsmordred: i wonder if it is an error to create a container that already exists14:32
*** baoli has joined #openstack-infra14:33
*** mattfarina has joined #openstack-infra14:36
Shrewsoh, create_container() takes care of that14:37
rcarrillocruzfungi: hiya, if you have +2 powers on project-config (i assume you do), could you have a look at https://review.openstack.org/#/c/167553/  pls ?14:38
openstackgerritSean M. Collins proposed openstack-infra/elastic-recheck: Add fingerprint for XStatic-Angular-Irdragndrop dependency failure  https://review.openstack.org/16915614:39
*** ChuckC has joined #openstack-infra14:39
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Add tests for shade cloud config parameter  https://review.openstack.org/16933914:41
fungircarrillocruz: out of curiosity, how is gozer leveraging autossh?14:41
mordredjeblair: I added tests for teh config validation as per ianw's request, but I did it in a follow on patch to reduce churn14:41
mordredfungi: I would counsel rcarrillocruz to not tell you that14:42
mordredfungi: only for the purposes of good though, I promise14:42
fungimordred: you and your corporate security ;)14:42
fungis/security/secrecy/14:42
rcarrillocruzmordred has spoken!14:43
mordredI will say that, in general, autossh does a great job at maintaining long-lived ssh connections14:43
openstackgerritRadomir Dopieralski proposed openstack-infra/project-config: Publish tuskar-ui-extras to pypi  https://review.openstack.org/16934114:43
*** pal has joined #openstack-infra14:43
*** pal_ has quit IRC14:43
rcarrillocruzit sure does14:44
fungii've never tried it, but i do know people who use it for personal purposes and are pleased with the result14:44
mordredianw, yolanda: speaking of - I adressed your concerns in https://review.openstack.org/#/c/168603/ but in a follow on patch14:44
claudiubfungi: hello14:44
fungiclaudiub: what have i broken now?14:44
claudiubfungi: I am having a little bit og trouble with a commit on project-config14:45
fungiclaudiub: which commit?14:45
claudiubfungi: I have a Jenkins -1 that I do not know how to solve: https://review.openstack.org/#/c/169293/214:45
openstackgerritMonty Taylor proposed openstack-infra/shade: Create swift container if it does not exist  https://review.openstack.org/16782314:45
mordredShrews: there you go ^^14:45
claudiubfungi: basically, it says: #cloud-init is not registered with ChanServ.14:45
* fungi checks14:46
Shrewsmordred: +1'd14:46
fungiclaudiub: yep, if you `/msg chanserv access #cloud-init list` you'll be told the same thing by chanserv14:47
fungiclaudiub: freenode's instructions point to http://blog.freenode.net/2008/04/registering-a-channel-on-freenode/14:47
*** amotoki_ has quit IRC14:49
claudiubfungi: aha, so basically, the channel exists, but it wasn't registered, correct?14:49
*** hashar is now known as hasharShopping14:49
fungiclaudiub: yes, in irc, a channel comes into existence the moment at least one user tries to join it14:50
TheJuliawin 514:50
fungiclaudiub: however, to register a channel with the network's services bots, you need to follow the additional steps documented there14:50
claudiubfungi: I see. Thanks for the help! :)14:51
*** ayoung has quit IRC14:51
openstackgerritClaudiu Belu proposed openstack-infra/project-config: Adds cloud-init to StackForge  https://review.openstack.org/16929314:51
fungihrm, minor hitch in my plan to start running jobs on devstack-.* workers instead of bare-.*14:53
anteayaoh?14:53
fungiwe don't actually _have_ any devstack-centos614:53
mordredoh14:53
anteayawell that would be a hitch yes14:53
*** jaypipes has quit IRC14:53
mordredfungi: we should make some14:53
fungiyep, shouldn't be hard as long as we don't actually expect devstack itself to work on them14:54
fungijust making a mental note that i'll need to add that14:54
anteayayup14:54
mordredfungi: maybe we should rename them14:54
*** carl_baldwin has joined #openstack-infra14:54
fungithat's the eventual plan. when bare-trusty for example goes away because we're running all its jobs on devstack-trusty, we can just rename that to trusty14:54
mordredyah. it'll be less confusing then14:55
rcarrillocruzthx fungi14:55
*** sputnik13 has joined #openstack-infra14:55
mordredfungi: fwiw, I recommend calling them "ubuntu-trusty" and not just trusty14:56
fungimordred: cool. and centos-6?14:56
mordredbecause that way we can call the centos and fedora nodes "centos-6" and "fedora-21"14:56
mordredyah14:56
*** marun has joined #openstack-infra14:56
mordredthat way it's "$DISTRO-$RELEASE" everywhere14:56
* rcarrillocruz finds disturbing that some manifests have a mix of '' and 'UNDEF' on class params...14:56
mordredfungi: I suppoe we could skip it and just have an image named "6" and one named "21"  :)14:57
openstackgerritAleksey Ripinen proposed openstack-infra/storyboard-webclient: Support for tags  https://review.openstack.org/16684714:57
*** devvesa has joined #openstack-infra14:58
openstackgerritMerged openstack-infra/project-config: Add stackforge/puppet-autossh  https://review.openstack.org/16755314:58
*** dangers_away is now known as dangers14:58
*** dustins_ has joined #openstack-infra14:58
*** dulek has quit IRC14:58
fungimordred: in that case, i may want to change how the version and release strings are emitted by bindep since i sort of chose them to align with the worker labels we're using (in absence of any existing format defined to express them)14:58
fungii guess we can try a quick bikeshed on standardizing our worker labels during today's meeting15:00
mordredfungi: each of the dib builds I've got now take each of those two things - largely because there is a base dib element you need for a given thing15:00
mordredbut then it takes a parameter for version15:00
*** bswartz has quit IRC15:00
*** devvesa has quit IRC15:00
mordredthere is a centos6 base element in the dib elements which kinda makes everything horky15:00
mordredso with the minimal elements, I kept it ${DISTRO}-minimal and each of those take a param for release so you can build what you want with a decent pattern15:01
*** baoli has quit IRC15:01
*** claudiub has quit IRC15:01
*** dustins has quit IRC15:01
fungieventual alignment between dib, bindep and our nodepool configuration might be nice, yeah15:01
*** dboik has joined #openstack-infra15:02
*** dboik has quit IRC15:02
*** cdent has quit IRC15:02
*** dboik has joined #openstack-infra15:03
*** rdopiera has quit IRC15:03
*** bswartz has joined #openstack-infra15:03
*** ayoung has joined #openstack-infra15:04
*** dims has quit IRC15:05
*** baoli has joined #openstack-infra15:05
openstackgerritAleksey Ripinen proposed openstack-infra/storyboard-webclient: Added query case to buildSignature  https://review.openstack.org/16936015:06
*** annegentle has quit IRC15:08
*** dustins_ has quit IRC15:08
*** dims_ has joined #openstack-infra15:08
*** pal has quit IRC15:08
*** mrmartin has joined #openstack-infra15:08
anteayafungi: do you think we need any discussion of the ldragndrop issue in today's meeting, in terms of messaging around renaming projects15:09
anteayaas you say it appears someone deletes from pypi just once in their life15:09
*** sputnik13 has quit IRC15:10
fungianteaya: i doubt it would be worth covering "don't delete releases from pypi" in the infra team meeting. i think we would be mostly preaching to the choir15:11
*** dustins has joined #openstack-infra15:11
*** MaxV has quit IRC15:11
fungiit wasn't any regular participant in the infra meetings who did it15:11
anteayayes, but in our messaging around renaming projects15:13
*** ajmiller has joined #openstack-infra15:13
anteayashould we do anything differently there15:13
anteayain terms of what we offer to others going through the process15:13
*** annegentle has joined #openstack-infra15:13
*** reed has joined #openstack-infra15:13
anteayaI'm fine if you disagree, just sharing a thought15:13
openstackgerritJames E. Blair proposed stackforge/gertty: Support searching by URL  https://review.openstack.org/16937115:15
*** zz_gondoi has joined #openstack-infra15:15
*** ajmiller_ has quit IRC15:15
*** MaxV has joined #openstack-infra15:15
*** zz_gondoi is now known as gondoi15:15
*** claudiub has joined #openstack-infra15:16
*** emagana has joined #openstack-infra15:18
*** MarkAtwood has joined #openstack-infra15:18
fungiit seems at most tangentially related to the topic of repo renames. also it was a stackforge project without any release integration in our infrastructure (someone performing their release process by hand) and therefore is just as likely to happen to us in situations completely unrelated to repo renaming15:18
fungieven by projects which are in no way involved in our community whatsoever15:18
*** carl_baldwin has quit IRC15:19
anteayaokay15:19
*** otter768 has joined #openstack-infra15:20
*** doug-fish has quit IRC15:20
jeblairi kind of think if there's a stackforge project in global requirements it's in the wrong place15:20
*** carl_baldwin_ has joined #openstack-infra15:20
openstackgerrityolanda.robla proposed openstack-infra/nodepool: Independent provider manager for polling tasks  https://review.openstack.org/16923515:21
yolandamordred, jeblair, proposal for independent manager ^15:21
*** zigo__ is now known as zigo15:23
*** dannywilson has joined #openstack-infra15:24
*** signed8bit has joined #openstack-infra15:24
*** dannywilson has quit IRC15:24
*** otter768 has quit IRC15:25
*** dannywilson has joined #openstack-infra15:25
timrcI started created a visualizer for our nodepool lifecycle events... it's far from complete, it may never be complete.. wondering if anyone else here is doing similar? http://15.125.65.125/15:25
openstackgerritMerged openstack-infra/elastic-recheck: Add fingerprint for XStatic-Angular-Irdragndrop dependency failure  https://review.openstack.org/16915615:26
fungitimrc: is that pulling from our nodepool stats counters in graphite.openstack.org?15:27
timrcfungi, No that is parsing debug.logs on our own instance.15:28
fungiahh15:28
fungitimrc: well, nodepool can be configured to emit stats events to a statsd15:29
*** wicope has joined #openstack-infra15:29
fungitimrc: so no need to parse the logs to get that data15:29
timrcI might do as cody-somerville suggested privated to do box and whisker.15:29
*** mrmartin has quit IRC15:29
*** stevemar has joined #openstack-infra15:29
*** jlibosva has quit IRC15:30
jeblairtimrc: and if a stat is missing from nodepool statsd, should be easy to add15:30
timrcfungi, Cool.  I haven't looked much into that.  It was convenient that our logs rotate daily and the events with timestamps were there.15:30
fungiand if there are events you find in the log which are useful to trend and don't have corresponding wrappers to emit stats, then yeah15:30
fungiwhat jeblair just said15:30
*** maurosr has quit IRC15:30
clarkbthe neat thing with graphite is it can likely do much of the graphing for you via its api (but it will also give you raw data in json iirc if doing it yourself is necessary)15:31
mordredI will echo the suggestion to add stats counters to nodepool - honestly being able to have a live graph of those sorts of times would be fantastic15:32
timrcclarkb, Yeah d3 made me want to punch a wall more than once.15:32
timrcI thought the graphs graphite generate were relatively static though.  I eventually want to make these more interactive.15:33
mordredin fact, you could really probably figure out a way to do it directly from TaskManager - since we already log the start/stop there15:33
*** dizquierdo has quit IRC15:33
timrcmordred, I'll look into what's there today and attempt to use it as a data source if everything I need is in place otherwise I'll submit a patch.15:33
mordredthere are other frontends that can work with the graphite data in dynamic ways15:33
timrcParsing logs and creating csv's is not my idea of a good time.15:34
*** scheuran has quit IRC15:35
mordredtimrc: http://jondot.github.io/graphene/ <-- I hear good things about this15:35
*** maurosr has joined #openstack-infra15:35
jeblairthey aren't dynamic in the sense of flashy javascript, but you can make any graph via url, so it's quite easy to use15:35
jeblairso we can still easily graph post-facto any of the data we collect15:36
*** YorikSar has quit IRC15:37
*** asselin has quit IRC15:38
*** jlanoux has quit IRC15:38
fungitimrc: yeah, parsing logs to generate stats is the sort of thing you do when you can't easily modify the application to give you the stats in a usable format15:38
*** gondoi is now known as zz_gondoi15:39
*** thedodd has joined #openstack-infra15:40
jeblairyolanda: what have you learned about the performance of 169235?15:41
timrcmordred, That does look pretty shiny.15:41
openstackgerritMerged openstack-infra/project-config: Add functional test job for python-saharaclient  https://review.openstack.org/16764115:42
*** harlowja_at_home has joined #openstack-infra15:42
*** sabeen3 has joined #openstack-infra15:42
openstackgerritClark Boylan proposed openstack-infra/nodepool: Support per provider image build settings  https://review.openstack.org/12674715:43
openstackgerritClark Boylan proposed openstack-infra/nodepool: Remove qemu-img-options  https://review.openstack.org/16938315:43
*** MaxV has quit IRC15:43
clarkbjeblair: ^ that doesn't detect when a new image is necessary due to a format change yet, but is that generally what you had in mind?15:43
*** sabeen3 has quit IRC15:43
*** sabeen has quit IRC15:43
*** MaxV has joined #openstack-infra15:44
*** sabeen has joined #openstack-infra15:44
*** harlowja_at_home has quit IRC15:44
*** nmagnezi has quit IRC15:44
fungiheading out for a lunch walk. back soon15:44
anteayaenjoy15:45
*** mestery has quit IRC15:45
*** dprince has quit IRC15:46
jogoclarkb: nice job with the 64bit live migration!15:46
clarkbjogo: the libvirt stop/start vs restart really tripped me up... Would've been done much sooner if I Had sorted that out15:47
*** skolekonov has quit IRC15:47
*** MaxV has quit IRC15:48
*** ghostpl_ has quit IRC15:49
*** ildikov_ has quit IRC15:49
*** dustins has quit IRC15:49
jeblairclarkb: why is DIB_IMAGE_CACHE a per-provider env var?15:49
*** ghostpl_ has joined #openstack-infra15:50
*** arxcruz has quit IRC15:50
clarkbjeblair: its currently per image15:51
clarkbjeblair: I thought you wanted to pull that stuff up out of images15:51
*** doug-fish has joined #openstack-infra15:51
clarkbit doesn't have to be though. The current code supports it either in the image or in the provider15:51
jeblairclarkb: i think things that affect the content of the filesystem created in the image should be attached to the diskimage thing, and things that affect the output format used by a provider should be attached to the provider...15:53
jeblairclarkb: i guess the cache dir doesn't fall into either category since it shouldn't affect either of those things, right?15:53
clarkbcorrect15:53
*** ala_ has quit IRC15:53
jeblairclarkb: so if we had to adjust those definitions to accomodate it, i figure it's more related to the content than the output format, inasmuch as it is used by the elements specified in the diskimage section...15:54
jeblairclarkb: so i'd lean toward saying that should be in the diskimage section15:54
jeblairclarkb: a second, related question -- why is compat an env variable?  i thought that was a qemu command line option?15:54
mordredjeblair: which change?15:54
mordredjeblair: it gets passed via env var15:55
jeblairmordred: 12674715:55
clarkbjeblair: it is a qemu-img command line option that is set via dib's env15:55
clarkbjeblair: which is why my next change in that stack is to remove the special qemu-img-options nodepool config since its basically redundant (we haven't used it that way either)15:55
jeblairclarkb, mordred: why is that not a command line argument to dib?15:56
jeblairclarkb, mordred: i mean, you tell dib what _major_ format to use for the output via command line, why would you not tell it what minor format to use?15:57
yolandajeblair, mordred, so that's etherpad for benchmarking , working on it right now15:57
yolandahttps://etherpad.openstack.org/p/Nodepool_benchmarks15:57
mordredjeblair: I think because it only is related to one of the image output formats - so if you did -t vhd the QEMU options wouldnt' be very meaninful?15:58
yolandathe traditional approach is over a more general period and a real usage, and the priorityqueue is for a shorter period, and on my test account, so it's not 100% reliable now15:58
clarkbjeblair: it can be; however, dib has a ton of options that can only be set via env var so to me it makes sense to use env vars as the preferred option there15:58
jeblairmordred: that is a problem many software systems have solved without resorting to environment variables15:58
mordredjeblair: dib drives a substantial number of things via env vars15:58
mordredjeblair: so it is consistent with overall operation15:58
greghaynesgenerally disk-image-create config is done by command line option and element config is done via env var15:59
greghaynesbecause otherwise wed need to invent some other way to tunnel that config through15:59
jeblairyolanda: cool, so i think that running equivalent experiments would be helpful as well as overall throughput figures15:59
mordredgreghaynes: well, isnt this a disk-image-create config?15:59
jeblairgreghaynes, mordred: i'm really confused. i would think that... exactly what mordred said.15:59
mordredgreghaynes: it's not really element config15:59
yolandayes, i know, i need to dedicate a bit more time15:59
mordredgreghaynes: should I make a patch?15:59
greghaynesyes, I think that wasnt entirely thought through here ;)15:59
greghaynesyes15:59
mordredk16:00
* mordred makes patch16:00
clarkbhold on16:00
clarkbignore dib for a second16:00
*** emagana has quit IRC16:00
mordred    echo "    --qemu-img-options -- option flags to be passed directly to qemu-img."16:00
clarkbmordred: yes it supports that see above16:00
clarkbwhat I am trying to say is that building images necessarily requires you set a half dozen env things16:00
jeblairclarkb, mordred: and nodepool uses that16:01
mordredclarkb: ah - you're saying that since we're setting env vars anyway16:01
clarkbso rather than support a half dozen nodepool config options that will only grow, it makes sense for nodepool to pass through an env setup16:01
clarkbmordred: yes16:01
mordredwe might as well set all the things that way16:01
clarkbyes16:01
mordredI can see that16:01
jeblairmakes sense, but i don't agree in this case16:01
annegentleget ready for a really out there git question. Is it possible to interact with git.openstack.org so that I could extract the exact URL of a source doc file? Such as http://git.openstack.org/cgit/openstack/openstack-manuals/tree/doc/playground-user-guide/source/backup_db_incremental.rst16:01
*** claudiub has quit IRC16:01
*** tonytan4ever has quit IRC16:01
jeblairclarkb: i believe that the diskimage entry in the config should support env vars16:01
jeblairclarkb: but there should be no per-provider env vars16:02
zaromorning16:02
clarkbjeblair: ok thats fine, I can just pull the per provider envvars stuff out16:02
jeblairclarkb: the provider should only specify the type and any additional options (like the qemu compat thing) related to that16:02
*** dboik_ has joined #openstack-infra16:02
jeblairclarkb: cool16:02
clarkbI do not think we should provide any qemu compat options16:02
jeblairer, don't we need to?16:02
clarkbbecause next week it will be vhd compat options16:02
clarkband the week after tarball flags and so on16:03
mordredhonestly, the qemu compat thing is about teh format16:03
*** tsg_ has joined #openstack-infra16:03
clarkbinstead we should just use the env to do this16:03
*** mestery has joined #openstack-infra16:03
mordredI'd go a completely different way and just have hardcoded logic that says "if format = qcow2, then qemu-img-options=compat"16:03
*** e0ne is now known as e0ne_16:03
mordredbut I'm wrongthinking many times16:03
* greghaynes was suggesting this16:04
mordredthat we have to specify this compat setting is a bug in something16:04
mordrednot a faeture16:04
jeblairclarkb: that's too generalized for nodepool to make decisions about when different images need to be made16:04
jeblairgreghaynes, mordred: do we know 0.10 works everywhere?16:04
clarkbjeblair: we do not16:04
mordredjeblair: well, there is only one cloud we have access to that uses qcow216:04
*** emagana has joined #openstack-infra16:04
mordredoh - wait - the tripleo cloud uses qcow216:05
jeblairi think tripleo and hpcloud yeah16:05
mordredgreghaynes: any chance you know if uploading an image to the tripleo cloud works if the image was built with 0.10?16:05
*** dboik has quit IRC16:05
*** tnovacik has quit IRC16:05
clarkbjeblair: its not too generalized because dib is making those decisions16:05
greghaynesmordred: I havent done it, no16:05
clarkbjeblair: so the qemu options are only valid when making qcow2, vhd stuff would only be valid for vhd images, tar flags would only be valid for tarballs16:05
mordredblerg16:05
jeblairclarkb: i'm saying that if you pass in any env variables, nodepool doesn't know which ones affect image output format16:05
clarkbso you can set all of them in the env and it will just work (tm)16:06
jeblairclarkb: only if we can set one value and have it work everywhere16:06
jeblairclarkb: in which case we may as well go with greghaynes idea and just set that value16:06
* SpamapS catches up16:06
mordredI say we _start_ with just setting the value until we actually have a real need to set it to different things16:06
mordredwe can add extra configurability later16:07
jeblairmordred: we allow setting it now.  you advocate for removing it.16:07
mordredI advocate for hardcoding it16:07
jeblairsure16:07
anteayaannegentle: I don't see that anyone else has replied to your question, I'm not sure what you mean by interact16:07
annegentleanteaya: ah, thanks for thinking through it with me. So is there an API for git.openstack?16:08
*** unicell1 has quit IRC16:08
anteayaannegentle: it is cgit so if cgit has an API then yes, but we didnt' set one up ourselves16:08
annegentleanteaya: I want to extract (did I say interact, sorry for imprecise language) a URL that poitns to a source file with preferrably the info from a conf.py file16:08
*** e0ne_ has quit IRC16:08
anteayaand I dont' recall that question before16:08
mordredanteaya: you can just do string substitution16:08
mordredgah16:08
annegentleanteaya: ok I'll look for cgit16:08
mordredannegentle:16:08
mordredthere is no api16:09
mordredbut16:09
*** medieval1 has joined #openstack-infra16:09
*** jistr has quit IRC16:09
jeblairwell, technically, there is a fully featured read-only REST API16:09
mordredthe base will always be http://git.openstack.org/cgit/16:09
mordredjeblair: :)16:09
annegentle:)16:09
mordredannegentle: then you'll always have the project name, then the word tree, then the path16:09
annegentleare there endpoints?16:09
*** carl_baldwin_ has quit IRC16:09
SpamapSMaybe we need to make image formats like elements16:09
annegentlemordred: ah, so glue it together with what I know16:09
mordredannegentle: http://git.openstack.org/cgit/{projectname}/tree/{path}16:09
mordredannegentle: yes16:09
annegentlemordred: lovely, you all know you are my rubber duckies that I talk to to solve my programming challenges :)16:10
SpamapSI'm not sure that solves this..16:10
annegentleanteaya: sometimes I just need help articulating desired outcome, thank you!16:10
mordredannegentle: in your example above- projectname=openstack/openstack-manuals and path=doc/playground-user-guide/source/backup_db_incremental.rst16:10
SpamapSbut I realized just now part of our frustration with this is the interface for image making is entirely different for everything else that we put into diskimage-builder16:10
mordredjust to be clear16:10
*** Sukhdev has joined #openstack-infra16:11
annegentlemordred: cool, and I think I can input projectname in the conf.py for doc builds16:11
*** whoops has joined #openstack-infra16:11
annegentleI think this scales across lots of doc builds16:11
mordredSpamapS: you'd have a vhd element and a qcow element? and if you specified both you'd get both things? that actually seems not great to me16:11
jeblairclarkb, mordred, greghaynes, SpamapS: let's address these questions one at a time:  regardless of mechanism, do we want qemu compat to be configurable in nodepool?16:11
anteayaannegentle: always happy to listen to you while you solve it yourself16:11
mordredjeblair: ++ to starting with that question16:11
mordredjeblair: I vote "no, not until we have a usecase for actually using it"16:12
mordredlargely because I want one image as much as possible16:12
mordredso if we had two qcow2 clouds16:12
greghaynessounds like there was a question of whether there is a single value that would work, seems like the issue we were running into can be worked around with a single value but it still is probably good to verify that16:12
mordredI'd rather build one qcow2 cloud and upload to both16:13
mordredotherwise we're going to need to put compat into the filename or something16:13
greghaynesyep, agreed16:13
mordredbeacuse we don't want to have file-per-provider (defeats the point)16:13
clarkbI am starting to come around to mordred here16:13
jeblairmordred: true, we would need to do that, though that's not super-hard.  we already have the format in the filename, we can add the compat flag too.16:14
clarkbbasically one qcow2 with the least common denominator version until necessary to do otherwise16:14
mordredjeblair: yes. we could also invent fake formats16:14
*** dprince has joined #openstack-infra16:14
mordredwe coould make a qcow2.0.10 format can a qcow2 format16:14
jeblairand to be clear, we're still talking identical images, just multiple files of them in different formats.16:14
mordredbut I thnk that gets lame16:14
jeblairyeah16:14
mordredyes16:14
*** e0ne has joined #openstack-infra16:14
greghaynesThe only downside I can think of is that were not testing qcow 0.20 and we dont get new 0.20 features, and I think both of those dont really matter to us16:15
mordredi agree16:15
jeblairokay, so it seems like we're cool with just lowest-common-denominatoring qcow16:15
jeblairand we have an idea of what the future might look like if we need to change our minds on that :)16:15
mordred++16:15
mordredSpamapS: now - for your element thing ... let me mention a think I suggested to greghaynes16:15
SpamapSmordred: so IMO, if we have two versions of qcow2 that are sometimes not compatible, those are not both "qcow2". However it also sounds like we are building a bridge to nowhere by pursuing this question?16:15
jeblairso having said that...16:15
SpamapSAh ok lag fail. What jeblair said. ;)16:16
jeblairi'd propose that we define env vars only in the nodepool diskimage section, define image types only in the provider section, and not specify qemu compat options anywhere16:16
*** ivar-lazzaro has joined #openstack-infra16:16
*** Guest1663 has quit IRC16:16
jeblairdo folks agree with that ^ ?16:16
*** ivar-lazzaro has quit IRC16:17
clarkbyes16:17
mordredjeblair: yes16:17
greghaynesSGTM16:17
jeblairwoot, i think that concludes nodepool bikeshedding.  i will write that up in the review.  feel free to proceed with dib bikeshedding :)16:17
*** ivar-lazzaro has joined #openstack-infra16:17
clarkbfwiw we can make the qemu version thing work today as is without any of that sillyness above, but that is because its set per diskimage and not per provider16:18
mordredSpamapS: I would like for dib to be able to not rebuild a new image, but instead just do a followup image conversion16:18
*** claudiub has joined #openstack-infra16:19
mordredSpamapS: as in "disk-image-create -n -t qcow2,raw -o ubuntu ubuntu-minimal vm infra ; disk-image-create --no-rebuild -n -t qcow2,raw -o ubuntu ubuntu-minimal vm infra"16:19
mordredSpamapS: gah16:19
mordredSpamapS: as in "disk-image-create -n -t qcow2,raw -o ubuntu ubuntu-minimal vm infra ; disk-image-create --no-rebuild -n -t qcow2,raw,vhd -o ubuntu ubuntu-minimal vm infra"16:19
SpamapSmordred: so be a thin layer in front of the vhd tools?16:19
mordredSpamapS: so the second command would notice that it already has an ubuntu.qcow2 and an ubuntu.raw so it does not start from scratch and would do the conversion it would have done the first time if I'd requested one16:20
greghayneson the second run, yes16:20
SpamapSI mean, realistically you don't need the element list in the second invocation.16:20
SpamapS--no-rebuild would just jump right to the image output step right?16:20
mordredyou don't - but if I don't have the images, I would expect it to build16:20
mordredSpamapS:  "disk-image-create -n -t qcow2,raw -o ubuntu ubuntu-minimal vm infra ; rm ubuntu* ; disk-image-create --no-rebuild -n -t qcow2,raw,vhd -o ubuntu ubuntu-minimal vm infra"16:20
mordredwould build thigns twice16:20
jeblairclarkb: i'm not sure i agree with that -- that still does not provide a way to map different qemu compat options to the qcow2 builds of a given image that might be used with multiple providers.16:21
greghaynesYea, so those are my questions - what level of cache invalidation do we want to do and do we want to store a fully copy on disk of the image16:21
mordredSpamapS: basically, make the second form idempotent16:21
*** bswartz has quit IRC16:21
SpamapSmordred: so we use the argument to -o as a cache key basically.16:21
*** derekh has quit IRC16:21
mordredSpamapS: yeah, basically16:22
mordredSpamapS: this came to mind in my brain16:22
mordredSpamapS: because I'd like to at some point make our own infra base images, and I'd like to have an ansible playbook that I can just run to make images as needed16:22
clarkbjeblair: oh right, we would have two or more different source images16:22
*** ildikov_ has joined #openstack-infra16:22
mordredSpamapS: once an image is made, I NEVER want to remake it16:22
mordredSpamapS: but, it's conceivable I might grow an additional cloud that wants an additional format16:22
mordredand it would be nice to be able to just add that format to the playbook16:23
SpamapSif [ "${rebuild_image}" = "1" ] || [ ! -e ${OUTPUT_BASE}.raw ] ; then do_all_the_build_things ; fi16:23
mordredand have the new cloud get properly formatted image16:23
greghaynesI think thatll work fine as long as we make it a --update-formats or similar rather than advertise it as a smart caching layer16:23
mordredgreghaynes: yah16:23
SpamapSmordred: that seems like a.. rare occurrance.16:23
mordredSpamapS: yes. if we never write this I will not die16:23
SpamapSmmk. I see all the merits, but place it in the Wishlist status unless there's some giant optimization I'm missing.16:24
greghaynesmordred: actually, one issue is it wont help with the img format thing16:25
mordredSpamapS: the main thing is that otherwise I'll have to add image conversion scripting to the ansible role which is sad because dib already knows how to go from raw to each of the formats my clouds want16:25
greghaynesmordred: because youll have to use a different name for the image16:25
mordredgreghaynes: what do you mean?16:25
greghayneser, qcow2 options16:25
mordredgreghaynes: I'm not talking qcow2 options - totally agree it won't help16:25
greghaynesok then16:25
mordredbut seriously - this is super wishlist -it's not really a problem right now16:25
mordredI just wanted to mention it because of SpamapS idea to make image formats more element like16:26
SpamapSmordred: I'm confused how that would need to be automated. You add clouds, what, once very 3 years?16:26
SpamapSalso btw, I may have accidentally shamed Cisco into wanting to provide a rack of servers for infra on Thursday night. ;)16:26
mordredSpamapS: I just grabbed myself a dreamhost account over the weekend16:26
mordredSpamapS: nice16:26
clarkbthat reminds me I have runabove instances I was playing with (not in relation to doing infra workloads)16:27
mordredSpamapS: but yeah - like I said - not a problem right now - I mainly wanted to inject the usecase in your  head so that didn't rework somethign to make doing that _harder_16:27
*** mtanino has joined #openstack-infra16:27
SpamapSAlso we need to update the infra slides to include the big tent.16:27
*** ivar-laz_ has joined #openstack-infra16:27
SpamapSmordred: cool. Image formats as elements would actually make it easier.16:28
mordredSpamapS: I believe you16:28
*** yfried is now known as yfried|afk16:29
*** yamahata has joined #openstack-infra16:29
* anteaya heads out for a walk16:29
*** ivar-lazzaro has quit IRC16:30
SpamapSmordred: but anyway, it's not a priority as image formats also come along about once every 3 years. ;)16:31
cody-somervillemordred: You should take a look at live-build from the Debian live project. It has a really nice architecture.16:31
mordredcody-somerville: why would I do that?16:31
SpamapScody-somerville: oh no you diii'nt  ;)16:31
SpamapScody-somerville: actually since you're paying attention... does live-build use something like schroot to build the OS level chroots?16:31
*** yfried|afk is now known as yfried16:31
SpamapSOne thing that has constantly made me go "hmmmmm" in dib is whether or not we should make it use schroot.. which would give us some really big optimizations for machines that do lots of builds (like using lvm and union filesystems)16:32
*** dhritishikhar_ has joined #openstack-infra16:32
SpamapSbut then I go "optimization trap" and move on.16:32
mordredcody-somerville: ok. I looked. it has terrible documentation and seems to have no interesting features16:33
*** carl_baldwin has joined #openstack-infra16:33
mordredcody-somerville: it's also a system designed to produce something that i don't want to produce16:33
mordredcody-somerville: so yeah, that was fun to look at, I now know that live-build is not useful to me16:34
openstackgerritClark Boylan proposed openstack-infra/nodepool: Support per provider image build settings  https://review.openstack.org/12674716:34
openstackgerritClark Boylan proposed openstack-infra/nodepool: Just set qcow2 version  https://review.openstack.org/16938316:34
clarkbjeblair: mordred greghaynes ^ I think those two patches address all of the above stuff including detection of missing image format requiring a rebuild16:34
*** unicell has joined #openstack-infra16:34
* greghaynes isnt exactly proud of the DIB docs yet16:34
clarkbits likely I will need to write more tests though16:34
cody-somervillemordred: I was just looking at the website as well. It used to have a very accessible manual.16:34
cody-somervillemordred: but I was suggesting looking at it from architecture perspective16:35
SpamapSgreghaynes: they're getting MUCH better every day. :)16:35
*** ujuc has quit IRC16:35
greghaynesSpamapS: Speaking of: https://review.openstack.org/#/c/16872716:35
*** baoli has quit IRC16:35
*** ssam2 has quit IRC16:36
mordredcody-somerville: sure - I don't think we're at the stage where we're really looking at architectures for build systems though16:36
SpamapSgreghaynes: yes, 3rd on my list ATM16:37
*** baoli has joined #openstack-infra16:37
greghaynes\O/16:37
*** annegentle has quit IRC16:40
*** mpaolino has quit IRC16:40
mordredgreghaynes: so - now that we have all of the unittesting - we should maybe live-test the resulting images on rax and hp again :)16:42
*** aduarte has joined #openstack-infra16:42
*** YorikSar has joined #openstack-infra16:42
greghaynesSounds like a good idea16:42
*** dangers is now known as dangers_away16:44
openstackgerritDavide Michelino proposed openstack-infra/project-config: Add Networking-portforwarding project  https://review.openstack.org/16933216:45
*** sputnik13 has joined #openstack-infra16:45
*** harlowja_away is now known as harlowja_16:45
*** SumitNaiksatam has quit IRC16:46
*** Swami has joined #openstack-infra16:46
*** hasharShopping has quit IRC16:47
*** jaypipes has joined #openstack-infra16:48
*** jlanoux has joined #openstack-infra16:48
openstackgerritAndreas Jaeger proposed openstack-infra/reviewstats: Update docs and security-docs teams  https://review.openstack.org/16829016:50
*** AJaeger_ has joined #openstack-infra16:50
*** dims_ has quit IRC16:51
*** lhcheng has joined #openstack-infra16:51
*** dims_ has joined #openstack-infra16:51
*** mwagner_lap has joined #openstack-infra16:52
*** yfried is now known as yfried|afk16:52
*** MaxV has joined #openstack-infra16:54
*** MarkAtwood has quit IRC16:54
*** mpaolino has joined #openstack-infra16:54
*** e0ne is now known as e0ne_16:56
*** bswartz has joined #openstack-infra16:57
*** tonytan4ever has joined #openstack-infra17:00
openstackgerritLuigi Toscano proposed openstack-infra/project-config: Make saharaclient-dsvm-functional voting  https://review.openstack.org/16942617:04
*** andreykurilin_ has joined #openstack-infra17:05
*** claudiub has quit IRC17:05
*** yfried|afk is now known as yfried17:06
*** lhcheng has left #openstack-infra17:06
*** e0ne_ is now known as e0ne17:06
*** tkelsey has quit IRC17:06
*** baoli has quit IRC17:08
*** tqtran has joined #openstack-infra17:09
*** dimtruck is now known as zz_dimtruck17:09
*** radez_g0n3 is now known as radez17:10
*** dustins has joined #openstack-infra17:12
*** tkelsey has joined #openstack-infra17:12
*** dtantsur is now known as dtantsur|afk17:14
*** baoli has joined #openstack-infra17:14
*** patrickeast has joined #openstack-infra17:14
*** SumitNaiksatam has joined #openstack-infra17:16
*** tkelsey has quit IRC17:17
*** johnthetubaguy is now known as zz_johnthetubagu17:18
*** funzo has joined #openstack-infra17:18
*** baoli has quit IRC17:19
*** wayneeseguin has joined #openstack-infra17:19
openstackgerritJoshua Harlow proposed openstack-infra/project-config: Remove the g-r requirements check  https://review.openstack.org/16943417:19
openstackgerritK Jonathan Harker proposed openstack-infra/jeepyb: Log project names as the projects are processed  https://review.openstack.org/16943517:19
*** baoli has joined #openstack-infra17:20
*** baoli has quit IRC17:20
*** otter768 has joined #openstack-infra17:21
cody-somervillehttp://lnav.org/ <-- this looks pretty cool. I wonder how well it would work with local Open Stack logs or something.17:22
*** achanda has joined #openstack-infra17:23
*** dboik_ has quit IRC17:25
*** otter768 has quit IRC17:26
*** dboik has joined #openstack-infra17:26
*** juggler has joined #openstack-infra17:26
SpamapScody-somerville: looks like an improvement on ccze :)17:27
*** markus_z has quit IRC17:30
jogoclarkb: how do I get the logs from the second node in aiopcpu into logstash?17:30
jogoclarkb: since directory is logs/*/n-cpu etc17:31
jogoinstead of logs/n-cpu17:31
*** mpavone has quit IRC17:32
clarkbwill need to update the log gearman client to get jobs submitted for them17:32
*** e0ne has quit IRC17:33
jogoclarkb: hmm all I know about is modules/openstack_project/files/logstash/jenkins-log-client.yaml17:33
*** Sukhdev has quit IRC17:35
greghaynesclarkb: I am noticing that hpcloud vs my dev box makes a huge difference in time to do qemu image conversions, which is probably why youall wanted that short-circuit so badly17:35
*** tkelsey has joined #openstack-infra17:36
*** ghostpl_ has quit IRC17:37
mordredgreghaynes: :)17:37
*** sdake_ has joined #openstack-infra17:37
clarkbwell we have to convert17:38
clarkbI wanted to stop copying files unnecessarily17:38
*** rhe00 has quit IRC17:39
*** jerryz_ has quit IRC17:39
jogoclarkb: so not sure how to update the log gearman client17:40
*** tkelsey has quit IRC17:40
*** annegentle has joined #openstack-infra17:41
*** doude has quit IRC17:41
*** sdake has quit IRC17:41
clarkbjogo: https://git.openstack.org/cgit/openstack-infra/puppet-log_processor/tree/files/log-gearman-client.py is the script17:42
clarkbjogo: however, I think it would be great if we could have a single n-cpu log file17:42
clarkbthough every time we try to use syslog it explodes on us so probably not worth the effort17:42
*** ayoung is now known as hopig17:43
*** baoli has joined #openstack-infra17:43
*** hopig is now known as ayoung17:43
*** bswartz has quit IRC17:44
clarkbjogo: I think our options are A) always put logs in known location B) grab index file and determine where subnode logs are C) use some sort of centralized log system to coalesce the logs17:46
*** annegentle has quit IRC17:46
mordredclarkb: d) give up and drink beer17:47
clarkbmordred: too early in the day for that17:48
fungioptions a,b and c can also be amended with "and drink beer"17:48
*** melwitt has joined #openstack-infra17:48
*** sdake has joined #openstack-infra17:48
fungii just always assumed it was an implied suffix17:48
openstackgerritValeriy Ponomaryov proposed openstack-infra/project-config: Add tempest + postgres job for manila  https://review.openstack.org/16593517:50
*** pelix has quit IRC17:50
SpamapSfungi: only 'and drink' is implied as it is an abstract method. You have to implement the actual method and populate with cups, location, company, and beverage of choice17:51
clarkbShrews: for 167371 what do you think about having shade do the logical "make networking work" which will create subnet, network, and router in one go17:52
mordredSpamapS: are you sure you don't want to implement a drink factory function that produces factories that produce drinks?17:52
fungigot it. i pretty much just used it as a class decorator17:52
clarkbShrews: as a user of shade I never want to think about routers ever17:52
sdakehey folks, rhallisey has created a channel for #kolla developers to congregate17:52
*** sdake_ has quit IRC17:52
mordredclarkb: I think we could add a function like that17:52
clarkbShrews: but not sure if this would be a good building block for what I decsribe or if we should change 167371 at all for that?17:52
sdakebut I'm concerned the channel wasn't registered propery17:52
mordredclarkb: similar to auto_ip17:52
mordredfor create_server17:52
sdakehe said he registered it with chanserv17:52
*** rhallisey has joined #openstack-infra17:52
*** doude has joined #openstack-infra17:52
fungisdake: checking now17:53
rhalliseysdake, hey17:53
sdakewhat is the appropriate technique to update teh chanserv settings?17:53
sdakehey rhallisey17:53
fungisdake: rhallisey: /msg chanserv access #kolla list17:53
fungisdake: rhallisey: that looks fine to me17:53
sdakefungi I don't have access17:53
Shrewsclarkb: that would assume everyone wants the same kind of network, no?17:53
mordredShrews: there is a basic kind of network that everyone wants17:54
mordredShrews: similar to the "this is a server and it has an IP"17:54
sdakefungi how do we get the status bot in the channel?17:54
Shrewsmordred: really? does everyone want the same subnet IP range???17:54
mordredif you want a different kind of thing, the lower-level primitives are there17:54
clarkbShrews: not the same range but thats just an option17:54
fungisdake: rhallisey: see http://ci.openstack.org/irc.html17:54
mordredShrews: maybe that's an argument to the function17:54
*** doude has quit IRC17:54
clarkbShrews: and if you need super fanc use the primitives. But imo its a major bug that a user has to configure 3 different resources to make logical networking work17:54
Shrewsmordred: what about multiple subnets???17:55
mordredclarkb: ++17:55
clarkbShrews: no17:55
mordredShrews: use the more complex things17:55
clarkbopenstack needs some opinions17:55
mordredthis is "I just want wroking network"17:55
mordredif you want fancy, you can always opt in17:55
clarkbworking network is one subnet, one network, and a router17:55
*** bswartz has joined #openstack-infra17:55
mordredyup17:55
*** pblaho has quit IRC17:55
mordredand those connected to each other17:55
clarkball hooked up and on the enxternal net if available17:55
mordredyup17:55
Shrewsi am not convinced17:55
clarkbShrews: you haven't clouded then17:56
clarkbthis is my biggest pain poit ever using openstack17:56
*** adalbas has quit IRC17:56
mordredyah. mine too17:56
sdakefungi thanks fore verifyign :)17:56
clarkboh also because order is important17:56
mordredalso, the docs for it are terrible17:56
fungisdake: rhallisey: oh, for the "openstackstatus" statusbot specifically the channel needs to be added to https://git.openstack.org/cgit/openstack-infra/system-config/tree/manifests/site.pp#n26617:56
clarkbso not only do I have to remember to do everything I have to be very careful about how I do it17:56
*** mtanino has quit IRC17:56
mordred"my cloud doesn't work" "why" "oh, I didn't create 6 different objects" *HEADDESK*17:56
*** zz_dimtruck is now known as dimtruck17:57
clarkbso anyways I think these router primitives are fine, having primitives is good but I think oen of the reasons shade exists is to serve the opinionated case so keeping that in mind as we go is good17:57
mordredShrews: also - the router api calls are all blocking?17:58
openstackgerritSpencer Krum proposed openstack-infra/system-config: Add a helpful friend to gerrit reviews  https://review.openstack.org/16901017:58
Shrewsclarkb: So, a good compromise is to allow full control by the user (what I'm creating), but we can also provide a do-it-all-for-me API method. I don't think that should be part of this review, though.17:58
mordredShrews: have I mentioned that I love the consistency17:58
*** Somay has joined #openstack-infra17:59
mordredShrews: yes. sorry - that's what IW as thinking17:59
mordredShrews: we make each of the apis like you're making17:59
clarkbShrews: yes thats what I am saying17:59
*** rlucio has joined #openstack-infra17:59
mordredShrews: also - is this correct or a typo: new_router = neutron.create_router(dict(router=router))17:59
clarkbalso at some point we need to have one change that adds mock as a dependency17:59
Shrewsclarkb: good, because i loathe reviews that are all "hey, add this other functionality that i just thought of"  :)18:00
clarkbthen rease all of the others on that otherwise we will end up with funny mock dep listings18:00
Shrewsmordred: maybe?18:00
mordredclarkb: yah - I'm going to rebase Shrews patch into the other series because I need to rebase the "migrate to tasks" patch on top of shrews' patch18:00
mordredclarkb: I mean, unless Shrews feels like doing that18:00
*** timcline_ has quit IRC18:01
mordredbut it seemed rude to ask him to18:01
Shrewsmordred: go for it18:01
mordredShrews: because passing in a dict as a positional parameter with a parameter of "router" which is a dict is CRAZY18:01
*** timcline has joined #openstack-infra18:01
sdakefungi thanks as usual the infrastructure team rocks :)18:01
mordredShrews: but, I would not put it past things for that to be the interface18:02
fungisdake: i like to think so, and you're welcome! ;)18:02
Shrewsmordred: the neutron api is a bit weird, yeah18:02
mordredShrews: OH DEAR GOD YOUR PATCH IS CORRECT18:02
mordred    def create_router(self, body=None):18:02
clarkbShrews: I am definitely not going to -1 the patch because of a missing feature. I just want shade to in general keep these things in mind as we make the primitives18:03
mordredclarkb: ^^ I'm going to guess that's been used by literally no humans ever18:03
Shrewsclarkb: agreed. i am not core, though, so you guys drive the direction  :-P18:03
clarkbmordred: create_router()?18:03
mordredclarkb: yeah18:03
mordredit takes "body" as a single arg, which is expected to be a dict18:04
clarkbmordred: I always use the horizon ui because I can never remember the entire list of things and what order to do them in18:04
Shrewsmordred: subnet stuff is the same way18:04
mordredShrews: it seems to be the pattern in the library18:04
Shrewsyup18:04
mordredthis may even win the prize for worst thing18:04
Shrewstook me a bit of digging to figure out wth is going on18:04
mordredit's a tough call between this and the glance task api18:04
*** jamielennox|away is now known as jamielennox18:04
mordredShrews: you win a cookie18:05
mordredShrews: it's a little used, sorry18:05
*** baoli has quit IRC18:05
clarkboh ya you have to enable dhcp too iirc18:05
clarkband set the range (rather than just inferring it from the subnet)18:05
asselin_are these commands correct? ./tools/zuul-changes.py  ... http://ci.openstack.org/zuul/zuul.html#starting-zuul18:06
openstackgerritRyan Hallisey proposed openstack-infra/system-config: Add status bot to kolla channel  https://review.openstack.org/16944518:07
*** MarkAtwood has joined #openstack-infra18:08
*** adalbas has joined #openstack-infra18:08
*** e0ne has joined #openstack-infra18:09
clarkbasselin_: https://git.openstack.org/cgit/openstack-infra/zuul/tree/setup.cfg#n23 is the mapping and Ithink that help text is generated by running that command18:09
clarkbasselin_: so yes?18:09
SpamapSmordred: Heat devs begged for changes to Neutron API and got told all the reasons they were wrong. Bring your A game. :)18:09
SpamapSmordred: and Neutron is by far the most difficult thing to orchestrate with Heat, so likely the same with something like Ansible.18:10
clarkbmordred: Shrews is the get_resource() pattern of iterating through a list rather than doing a show on said resource type iwth the name/id a thing in shade because not every resource supports shows on names?18:10
*** esker has joined #openstack-infra18:10
*** esker has quit IRC18:10
asselin_clarkb, seem's something not right: http://paste.openstack.org/show/197732/18:11
clarkboh zuul-changes is in the tools dir that different18:12
*** MarkAtwood has quit IRC18:12
Shrewsclarkb: yeah18:13
Shrewsconvenience method18:13
clarkbasselin_: https://git.openstack.org/cgit/openstack-infra/zuul/tree/tools/zuul-changes.py it doesn't take the 'rehcec' comment18:13
clarkbasselin_: it instead prints out zuul-enqueue commands so no gerrit stuff is involved18:13
Shrewsmordred: if you are doing mind-altering things with your tasks stuff, i'll hold off on my subnet patch until i see what you are doing18:14
mordredSpamapS: yes18:14
mordredgah18:14
mordredclarkb: yes18:14
mordredclarkb: also, many of those "show" on a name are actually doing a list in the client library18:15
clarkbmordred: nice18:15
mordredor, rather, you're talking about list(name="foo") - not show18:15
mordredshow is different18:15
mordredbut if we do the pattern, then we can control caching18:15
*** yfried is now known as yfried|afk18:15
mordredclarkb: there are so many things you learn in doing this that you never wanted to know18:15
clarkbShrews: does update router need to also take args to change the external net port? I am trying to remember how neutron exposes that18:16
*** rhallisey has left #openstack-infra18:16
*** yfried|afk is now known as yfried18:16
*** Longgeek has quit IRC18:16
*** Longgeek has joined #openstack-infra18:17
jogoclarkb: A sounds like the most straight forward18:17
jogoclarkb: instead of $IP-subnode, we can just use subnode18:17
clarkbjogo: yup18:17
jogoor something like that18:17
clarkbjogo: I would say subnode-118:18
*** tonytan4ever has quit IRC18:18
jogoand works for me18:18
*** e0ne is now known as e0ne_18:18
*** AJaeger_ has quit IRC18:18
*** tonytan4ever has joined #openstack-infra18:19
openstackgerritJoe Gordon proposed openstack-infra/devstack-gate: Make aiopcpu subnode directory more deterministic  https://review.openstack.org/16944918:19
clarkbShrews: ya looking at http://developer.openstack.org/api-ref-networking-v2-ext.html the external network id is a thing you can update. Is that something you think should go into this patch?18:19
jogoclarkb: like taht ^ ?18:19
clarkbjogo: no, you should do it where X is arbitrary based on the number of nodes18:20
*** _nadya_ has quit IRC18:21
clarkbalso for fun the network_id for external network id appears to be a dict18:21
Shrewsclarkb: i can add that pretty easily18:21
*** teran_ has joined #openstack-infra18:21
clarkbShrews: I am writing comments, I will probably +2 then let you decide if it needs to go in this change or a followup18:22
*** ihrachyshka has quit IRC18:22
Shrewsclarkb: i'll post a followup review then18:23
Shrewsclarkb: thanks18:23
openstackgerritMerged openstack/requirements: Updated incorrect name for angular lrdragndrop  https://review.openstack.org/16913218:23
*** e0ne_ has quit IRC18:23
openstackgerritJoe Gordon proposed openstack-infra/devstack-gate: Make aiopcpu subnode directory more deterministic  https://review.openstack.org/16944918:23
Shrewsjogo: how does one pronounce "aiopcpu" ????18:23
*** packet has joined #openstack-infra18:24
jogoShrews: mumble it18:24
Shrewsah, of course18:24
jogoall in one plus CPU18:24
*** tonytan4ever has quit IRC18:24
clarkbjogo: also I got dtroyer feedback on how to make my devstack change for the cpu_map update mergeable18:25
clarkbso going to do that next18:25
*** teran has quit IRC18:25
jogonice18:25
*** teran_ has quit IRC18:25
*** chris613 has quit IRC18:26
clarkbShrews: commented18:29
*** mpaolino has quit IRC18:29
fungiwow. we are somehow ending up with 167 additional packages on devstack-trusty in rax compared to hpcloud, but only 26 additional when comparing bare-trusty between the providers. this means we're somehow ending up with a ton of crap preinstalled on our devstack-trusty workers in rax18:31
fungithings which we presumably install when making bare-trusty workers18:31
clarkbfungi: rax makes the base images and probably adds a bunch of goodness for us18:32
fungiyeah, i'm guessing that the majority of that difference is stuff we wind up adding for bare-trusty on hpcloud but is already present for us in rax18:32
*** dkranz has quit IRC18:33
*** garyh has quit IRC18:33
fungithis adds an interesting extra challenge to the problem of validating that bindep is installing everything we need. it's easier for me to snag ready nodes in rax, much less so in hpcloud these days18:33
clarkbfungi: you could just boot the hpcloud images locally18:34
fungiyep, i may end up doing that18:34
openstackgerritDavid Shrewsbury proposed openstack-infra/shade: Let router update to specify external gw net ID  https://review.openstack.org/16945418:34
Shrewsclarkb: ^^^^18:34
clarkbfungi: https://github.com/greghaynes/dib-dev-deploy makes it easy18:34
fungior even just using them as chroots, not booting them18:34
clarkbfungi: ya I guess you aren't concerend about booting just package availability18:35
mordredfungi: I use them as chroots18:35
mordredfungi: also - this is one of the reasons I like my minimal base images18:35
clarkbI have typically been booting because booting has been the trouble area :)18:35
mordredfungi: it should help make sure that we actually _do_ list the things we need18:35
mordredclarkb: :)18:36
fungisince all i really want to compare is the package list from a devstack image after bindep runs against the package list from a bare image. so i can chroot into the devstack dib image and feed bindep results into the package manager in the chroot18:36
mordredclarkb: I've been booting them recently18:36
fungithis shouldn't theoretically have any bearing on booting18:36
*** e0ne has joined #openstack-infra18:36
mordredfungi: yup18:36
clarkbShrews: the neutron docs indicate that the network_id is a dict, do we want to make user pass in dict or make a dict for them?18:36
mordredmakea  dict for them18:36
mordredthat's a stupid interface18:36
clarkbShrews: ^ ok so thats a -1 from mordred :)18:36
*** eharney has quit IRC18:37
Shrewsclarkb: it's only a single value. let's not make it more difficult than it is  :)18:37
clarkbShrews: though I haven't confirmed that this is the interface in neutronclient18:37
fungimordred: bindep totally needs to drop all its command line options and configuration in favor of passing a dict as an envvar18:37
* clarkb goes to read neutronclient source18:37
mordredfungi: yes18:37
mordredclarkb: it's the only way to know how anything works18:37
fungiit's the interface of teh future18:37
fungisorta like dippin' dots18:38
mordredShrews: I like the patch - as long as it doesn't require that to be a dict :)18:38
Shrewsclarkb: good luck! neutronclient is stupidly absent of information18:38
Shrewsclarkb: try 'neutron help router-update'  or whatever the command is18:38
Shrewsno info on how to pass the update information18:38
clarkbShrews: wow yes it is18:38
Shrewsmajor client fail18:38
clarkbso I think the api docs are what we must look at and that says its a dict18:38
clarkbhrm maybe I should look in the neutron api server18:38
Shrewsclarkb: yup, i've been going by that18:39
mordredTHAT IS CRAZY18:39
clarkbShrews: rgr trying my hand at it now18:39
*** yfried is now known as yfried|afk18:41
clarkbhrm18:42
*** afazekas has quit IRC18:43
greghayneswow non kvm qemu is slow18:45
clarkbShrews: https://git.openstack.org/cgit/openstack/neutron/tree/neutron/extensions/l3.py#n105 so it is a dict18:46
greghaynestrying to make a dib test out of it, taking 10s of mins to boot a stock ubuntu vm18:46
clarkbShrews: I will -1 with a pointer there18:46
clarkbgreghaynes: I told you18:46
greghaynes:(18:46
*** dkranz has joined #openstack-infra18:46
Shrewsclarkb: i'm passing a dict... not sure what you are -1'ing18:46
clarkbits something like 45 seconds to boot ubuntu with openstack intsalled18:46
clarkbShrews: oh derp I was reading poorly18:47
Shrews:)18:47
*** Somay has quit IRC18:49
*** achanda has quit IRC18:49
Shrewsclarkb: and thanks for the reviews18:50
greghaynesclarkb: 45secs would be awesome18:51
clarkbgreghaynes: sorry minutes :)18:51
clarkbI just caught that typo myself :)18:51
greghaynes"what!? its faster than kvm!?"18:51
*** Rockyg has joined #openstack-infra18:51
*** dkranz has quit IRC18:51
*** annegentle has joined #openstack-infra18:52
*** dhritishikhar_ has quit IRC18:52
*** fbo has quit IRC18:52
*** tonytan4ever has joined #openstack-infra18:52
*** stevemar has quit IRC18:54
*** tnovacik has joined #openstack-infra18:54
*** pblaho has joined #openstack-infra18:55
openstackgerritMerged openstack-infra/shade: Disable warnings about old Rackspace certificates  https://review.openstack.org/16873418:56
*** yfried|afk is now known as yfried18:56
*** flaviosr has joined #openstack-infra18:58
tchaypoDoes DST flip this weekend in the US, or are you a few weeks later?18:58
openstackgerritMonty Taylor proposed openstack-infra/shade: Namespace caching per cloud  https://review.openstack.org/16861518:58
openstackgerritMonty Taylor proposed openstack-infra/shade: Allow for passing cache class in as a parameter  https://review.openstack.org/16214818:58
openstackgerritMonty Taylor proposed openstack-infra/shade: Add 'rebuild' to shade  https://review.openstack.org/16106718:58
openstackgerritMonty Taylor proposed openstack-infra/shade: Migrate API calls to task management  https://review.openstack.org/16866518:58
openstackgerritMonty Taylor proposed openstack-infra/shade: Add task management framework  https://review.openstack.org/16866418:58
mordredtchaypo: we've already flipped18:58
mordredclarkb, Shrews: I rebased on top of Shrews patch for routers and update the last patch to put those calls into the task framework18:59
dstufftDST is the worst18:59
mordredI also +A'd both of those patches, so they shoudl land soon18:59
Shrewshazzah18:59
openstackgerritMerged openstack-infra/shade: Add methods for logical router management  https://review.openstack.org/16737118:59
jeblairmeeting time!19:00
tchaypoThis year DST makes my Easter long weekend 97 hours instead of the usual 9619:00
*** jlanoux has quit IRC19:00
*** primemin1sterp is now known as primeministerp19:00
*** Somay has joined #openstack-infra19:01
*** Sukhdev has joined #openstack-infra19:01
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard-webclient: Fixed reference to experimental API  https://review.openstack.org/16945819:02
*** fbo has joined #openstack-infra19:04
*** dboik has quit IRC19:04
*** dkranz has joined #openstack-infra19:04
*** stevemar has joined #openstack-infra19:04
*** dboik has joined #openstack-infra19:04
ekarlsoanyone know how with sqla migrate to create a enum and then reference that enum in a table, then create the table with having it creating the enum again that's already created in pgsql ?19:05
*** Sukhdev has quit IRC19:05
openstackgerritMonty Taylor proposed openstack-infra/shade: Namespace caching per cloud  https://review.openstack.org/16861519:05
openstackgerritMonty Taylor proposed openstack-infra/shade: Allow for passing cache class in as a parameter  https://review.openstack.org/16214819:05
openstackgerritMonty Taylor proposed openstack-infra/shade: Add 'rebuild' to shade  https://review.openstack.org/16106719:05
openstackgerritMonty Taylor proposed openstack-infra/shade: Migrate API calls to task management  https://review.openstack.org/16866519:05
openstackgerritMonty Taylor proposed openstack-infra/shade: Add task management framework  https://review.openstack.org/16866419:05
*** tiswanso has quit IRC19:07
*** adalbas has quit IRC19:07
dansmithare there ceph tests in experimental or something?19:07
*** stevemar has quit IRC19:08
clarkbdansmith: there is a non voting devstack + ceph job on nova and cinder's check queue19:08
*** stevemar has joined #openstack-infra19:08
dansmithokay, thought I needed to hit the experimental button for that19:09
dansmithguess I need to pay more attention19:09
openstackgerritMerged openstack-infra/devstack-gate: Redirect subnode cleanup to devstack-gate-cleanup-host.txt  https://review.openstack.org/16810219:10
*** salv-orl_ has joined #openstack-infra19:11
*** garyh has joined #openstack-infra19:11
*** salv-orlando has quit IRC19:13
*** yfried is now known as yfried|afk19:14
*** Somay has quit IRC19:14
*** ZZelle_ has joined #openstack-infra19:16
openstackgerritStefano Maffulli 'reed' proposed openstack-infra/infra-manual: Creates a sandbox repo tutorial  https://review.openstack.org/13819519:19
*** medieval1 has quit IRC19:19
*** otter768 has joined #openstack-infra19:22
mtreinishclarkb: do you remember what that bug # was from the gate bug we were tracking a while ago that you hit on hp cloud?19:22
*** sarob has joined #openstack-infra19:22
jlvillaldhellmann: On https://github.com/dhellmann/ansible-znc-on-znc  Would you have any interest in patches for Fedora support?19:23
clarkbmtreinish: the neutron port leak bug? I do not remember the numer19:24
clarkbmtreinish: but I can find it if you like19:24
*** yfried|afk is now known as yfried19:24
*** adalbas has joined #openstack-infra19:24
clarkbmtreinish: https://bugs.launchpad.net/nova/+bug/135615719:25
openstackLaunchpad bug 1356157 in OpenStack Compute (nova) icehouse "make nova floating-ip-delete atomic with neutron" [High,Fix released] - Assigned to Aaron Rosen (arosen)19:25
mtreinishclarkb: yeah I think that was the one thanks19:25
*** mfink has left #openstack-infra19:26
*** otter768 has quit IRC19:26
*** achanda has joined #openstack-infra19:27
*** ynasser has joined #openstack-infra19:28
*** Sukhdev has joined #openstack-infra19:29
*** sdake_ has joined #openstack-infra19:31
openstackgerritChristoph Gysin proposed stackforge/gertty: Add support for auth-type=form  https://review.openstack.org/16946719:32
*** SumitNaiksatam has quit IRC19:33
openstackgerritChristoph Gysin proposed stackforge/gertty: Fix repository checking  https://review.openstack.org/16946819:33
openstackgerritChristoph Gysin proposed stackforge/gertty: Add config option for git clone URL  https://review.openstack.org/16946919:33
anteayajlvillal: he is on vacation, should be back maybe tomorrow or Thursday19:34
jlvillalanteaya: Thank you.19:34
anteayajlvillal: welcome19:34
*** sdake__ has joined #openstack-infra19:35
openstackgerritChristoph Gysin proposed stackforge/gertty: Add script to start gertty out of source dir  https://review.openstack.org/16947019:35
*** sdake has quit IRC19:35
*** reed has quit IRC19:36
*** reed has joined #openstack-infra19:36
*** Sukhdev has quit IRC19:36
*** Sukhdev has joined #openstack-infra19:37
*** tkelsey has joined #openstack-infra19:37
reedI fat-fingered killed X19:37
*** sdake_ has quit IRC19:38
*** Rockyg has quit IRC19:39
*** alexpilotti has joined #openstack-infra19:41
*** tkelsey has quit IRC19:41
*** SumitNaiksatam has joined #openstack-infra19:44
anteaya:(19:47
*** chris613 has joined #openstack-infra19:47
*** yfried is now known as yfried|afk19:47
*** chris613 has left #openstack-infra19:47
*** Rockyg has joined #openstack-infra19:47
fungidebian jessie release date set for april 25. sooner than i expected19:47
anteayagood to know19:48
*** tim_o has joined #openstack-infra19:49
*** thingee has joined #openstack-infra19:50
*** chris___2 has joined #openstack-infra19:50
*** annegentle has quit IRC19:51
*** Sukhdev has quit IRC19:54
*** gokrokve_ has joined #openstack-infra19:57
*** hashar has joined #openstack-infra19:58
*** gokrokve_ has quit IRC19:59
*** gokrokve_ has joined #openstack-infra19:59
*** dboik_ has joined #openstack-infra20:00
openstackgerritClark Boylan proposed openstack-infra/nodepool: Support per provider image build settings  https://review.openstack.org/12674720:00
openstackgerritClark Boylan proposed openstack-infra/nodepool: Just set qcow2 version  https://review.openstack.org/16938320:00
*** gokrokve has quit IRC20:00
openstackgerritClark Boylan proposed openstack-infra/nodepool: Use correct diskimage when building diskimages  https://review.openstack.org/16947420:00
anteayaso I was hoping we would hit open discussion but we did get through the agenda20:00
clarkbmordred: jeblair ^ that should pass tests now20:01
mordredwoot20:01
clarkbtime for foods20:01
greghaynesoo fun20:01
anteayaso do we want to talk about what to do about a summit etherpad thing?20:01
anteayaas many other teams are launching them20:01
*** yfried|afk is now known as yfried20:01
*** ibiris is now known as ibiris_away20:02
yolandajeblair, so per benchmarking etherpad, i'll be collecting more info tomorrow so we can compare properly20:03
*** dboik has quit IRC20:03
jeblairanteaya: not yet20:03
anteayajeblair: very good20:04
*** dkranz has quit IRC20:05
*** andreykurilin_ has quit IRC20:06
*** dkranz has joined #openstack-infra20:09
*** annegentle has joined #openstack-infra20:10
*** tonytan4ever has quit IRC20:10
*** tonytan4ever has joined #openstack-infra20:11
*** tim_o has quit IRC20:11
*** sabeen has quit IRC20:11
*** sabeen has joined #openstack-infra20:11
*** yfried is now known as yfried|afk20:13
*** eharney has joined #openstack-infra20:14
*** yfried|afk is now known as yfried20:17
*** ayoung has quit IRC20:17
*** hashar has quit IRC20:17
*** tiswanso has joined #openstack-infra20:19
*** achanda has quit IRC20:20
*** mfink has joined #openstack-infra20:20
*** tiswanso has quit IRC20:20
*** tiswanso has joined #openstack-infra20:21
*** alexpilotti has quit IRC20:21
*** dboik_ has quit IRC20:22
*** dboik has joined #openstack-infra20:23
*** baoli has joined #openstack-infra20:26
*** ibiris_away is now known as ibiris20:27
harlowja_mordred fyi; just saw this20:28
*** adalbas has quit IRC20:28
harlowja_[13:18:34]  <Odd_Bloke> smoser: Am I right in thinking that vendor-data simply isn't a thing in the version of cloud-init in precise?20:28
harlowja_[13:26:55]  <smoser> right . not at all.20:28
harlowja_[13:27:02]  <smoser> Odd_Bloke, i'd sponsor an sru20:28
harlowja_sooo mordred maybe that will help with HP cloud cloudinit?20:29
harlowja_or whatever20:29
mordredharlowja_: nope. I'm not using it20:29
mordredbut thanks20:29
harlowja_lol20:29
harlowja_mordred do u need a hug again20:29
*** teran has joined #openstack-infra20:29
mordrednope! I'm quite happy about it20:29
harlowja_hugggg20:29
harlowja_lol20:29
mordred:)20:29
harlowja_don't deny the hug man20:29
harlowja_jeeez20:29
harlowja_jeeeeezsh20:29
harlowja_lol20:29
mordredhugs are good20:29
harlowja_ha20:29
harlowja_hugs are made better by cloudinit20:30
harlowja_lol20:30
mordredharlowja_: and enhanced ever further by simple-init :)20:30
*** medieval1 has joined #openstack-infra20:30
*** annegentle has quit IRC20:30
reedhas anyone ever tried to abandon a change from CLI, using ssh as in https://review.openstack.org/Documentation/cmd-review.html?20:31
*** sarob has quit IRC20:32
openstackgerritSpencer Krum proposed openstack-infra/system-config: Revert "Add a helpful friend to gerrit reviews"  https://review.openstack.org/16950020:33
*** ayoung has joined #openstack-infra20:33
*** annegentle has joined #openstack-infra20:33
*** annegentle is now known as agentle_20:33
*** annegentle has joined #openstack-infra20:34
openstackgerritMerged openstack-infra/shade: Let router update to specify external gw net ID  https://review.openstack.org/16945420:37
sdaguereed: yes20:37
arosenquick questions, so this patch was just merged that makes congress an official openstack project: https://review.openstack.org/#/c/165204/20:37
*** ghostpl_ has joined #openstack-infra20:38
arosenI guess I need to now update project-config to move congress to openstack rather than stackforge?20:38
reedsdague, what do you think causes it to barf on my saying "fatal: "test" is not a valid patch set"20:38
reedssh -l smaffulli -p 29418 review.openstack.org 'gerrit review ' --project openstack-dev/sandbox.git -m "Abandon test patch" --abandon 16946020:38
*** dizquierdo has joined #openstack-infra20:38
*** baoli has quit IRC20:38
*** baoli has joined #openstack-infra20:39
anteayaarosen: yes that would be a good patch to offer20:39
sdagueyou need the patch level as well iirc20:39
sdagueor the idempotent id20:39
sdagueit's either X/Y or Ixxxxxxxxxxxxxxxxxxxxxxx20:39
anteayaarosen: then add a link to the patch to upcoming project renames: https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting20:40
anteayaarosen: so we can schedule a rename20:40
openstackgerritSpencer Krum proposed openstack-infra/system-config: Add a helpful friend to gerrit reviews  https://review.openstack.org/16901020:40
sdague169460 doesn't identify specifically enough20:40
*** david-lyle has quit IRC20:40
openstackgerritSpencer Krum proposed openstack-infra/system-config: Revert "Add a helpful friend to gerrit reviews"  https://review.openstack.org/16950120:41
*** e0ne has quit IRC20:42
reedsdague, I've tried with change-id too20:42
*** e0ne has joined #openstack-infra20:42
*** ghostpl_ has quit IRC20:42
reedsdague, AH!20:42
reedyou need to specify the patchset, too20:42
*** mrda-away is now known as mrda20:46
*** david-lyle has joined #openstack-infra20:49
*** ibiris is now known as ibiris_away20:50
*** claudiub has joined #openstack-infra20:51
*** e0ne is now known as e0ne_20:52
*** e0ne_ is now known as e0ne20:52
fungiwhich i find odd given that abandonment shouldn't need to take the patchset number into consideration, but i guess that's a non-optional part of their change manipulation api20:53
nibalizerjeblair: sdague can you review 169010 ?20:53
greghaynesone sec20:53
greghaynesI need to edit something :p20:53
openstackgerritStefano Maffulli 'reed' proposed openstack-infra/infra-manual: Creates a sandbox repo tutorial  https://review.openstack.org/13819520:53
greghayneswell, I could make my change dependent20:54
greghaynesgo for it20:54
*** ayoung has quit IRC20:54
*** yfried is now known as yfried|afk20:55
*** mrmartin has joined #openstack-infra20:56
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard-webclient: Added commandline commands to package.json  https://review.openstack.org/16848820:56
openstackgerritMichael Krotscheck proposed openstack-infra/storyboard-webclient: Removed grunt:server  https://review.openstack.org/16316520:56
*** dimtruck is now known as zz_dimtruck20:57
harlowja_mordred ha; maybe :-P20:58
*** zz_dimtruck is now known as dimtruck20:58
*** ibiris_away is now known as ibiris20:59
*** annegentle has quit IRC21:01
*** agentle_ has quit IRC21:01
*** annegentle has joined #openstack-infra21:01
anteayareed: did you get what you needed with that patch?21:02
reedanteaya, yes, I just submitted my final work. Have a look at it please, let me know what you think21:02
anteayareed: okay21:03
*** mrmartin has quit IRC21:03
*** dangers_away is now known as dangers21:04
anteayareed: you need to rebase https://review.openstack.org/#/c/104951/ on your new patch21:05
anteayasince that is the link to the tutorial21:05
reedah, so complicated :)21:06
openstackgerritJames E. Blair proposed openstack-infra/system-config: Revert "Add a helpful friend to gerrit reviews"  https://review.openstack.org/16950821:06
*** dkliban is now known as dkliban_afk21:06
*** annegentle has quit IRC21:07
*** ayoung has joined #openstack-infra21:07
anteayareed: yes21:08
fungireed: looks like a substantial addition. i look forward to reading through21:08
openstackgerritgreghaynes proposed openstack-infra/system-config: Give some actions to our friend  https://review.openstack.org/16950921:08
*** packet has quit IRC21:08
*** aduarte has quit IRC21:09
*** packet has joined #openstack-infra21:10
*** rfolco has quit IRC21:10
reedI think I'm messing things up21:12
*** aysyd has quit IRC21:13
*** tiswanso has quit IRC21:13
openstackgerritStefano Maffulli 'reed' proposed openstack-infra/infra-manual: Adds a link for new developers to use sandbox  https://review.openstack.org/10495121:14
openstackgerritStefano Maffulli 'reed' proposed openstack-infra/infra-manual: Creates a sandbox repo tutorial  https://review.openstack.org/13819521:14
*** achanda has joined #openstack-infra21:14
jogomordred jeblair: I thought I saw you guys say this morning that you don't think we are running out of nodes in nodepool is that correct?21:14
anteayareed: that should work, let's wait for docs to come back21:14
jeblairjogo: well, they are a renewable resource.  :)21:15
*** ldnunes has quit IRC21:15
fungijogo: that sounds like an overly broad assertion21:15
jeblairwe can harvest them, and they regrow at a rate which varies based on the overall health of the ecosystem21:16
jogofungi: it is overly broad21:16
fungiwe at times (for example right now) have jobs queued up waiting to eventually get an available node, and they would be done sooner in many cases if we had a larger quota21:16
jogofungi: right.21:16
ianwmy internet dropped before the open time in the meeting : can i get some eyes on the f20 removal patches https://review.openstack.org/#/q/status:open+topic:remove-f20,n,z21:16
jeblair(or clouds that worked)21:16
openstackgerritK Jonathan Harker proposed openstack-infra/project-config: Make the unrecognized exception more verbose  https://review.openstack.org/16951121:16
jogowe seem to hit that state daily now21:16
ianwone job to switch to centos, key people have +1'd that, then we can remove the f20 nodes.  one less problem to deal with21:16
jogojobs queued waiting for available nodes21:16
ianwand workarounds21:16
mordredjogo: yes, but we're working through some issues and changs to work on that - in general, I do not think we need to stop testing things because we're out of nodes21:17
fungijogo: yes, there are performance issues around turnover in our largest provider, and that seems to have us capped around 700-800 jobs per hour. that's still... a lot of jobs21:17
openstackgerritStefano Maffulli 'reed' proposed openstack-infra/infra-manual: Adds a link for new developers to use sandbox  https://review.openstack.org/10495121:17
*** irdkehn has joined #openstack-infra21:17
*** baoli has quit IRC21:17
jeblairjogo: also, if you look at the node graph, we have plenty of capacity overnight.  so even with our currently reduced capacity, our test activity is sustainable.21:18
*** baoli has joined #openstack-infra21:18
jogomordred: ahh when do you expect those issues to be resolved?21:18
*** bswartz has quit IRC21:19
*** david-lyle_ has joined #openstack-infra21:19
jogojeblair: that a great metric IMHO. When what should be a 45 minute test takes 2 or 3 hours to run21:19
jogoit significantly slows iteration time down21:19
mordredjogo: it does -but that can also be the case when we have inifinite nodes21:20
jeblairjogo: so in summary, i appreciate and thank you for your effort to find and remove jobs that are not actually testing anything.  but i don't think we need to remove jobs that are actually testing things, or avoid adding new jobs that test things.21:20
jogofungi: a lot is relative, we also have a lot of git repos21:20
jeblairjogo: do you mean that it is not a great metric?21:20
jeblairjogo: i'm having trouble following you.21:20
fungii agree that "jobs per hour" is not a great metric, but it's a metric we have21:20
jogojeblair: err my bad, that is "not a great metric"*21:20
jeblairjogo: we've _never_ promised that the system will always be instantly responsive.21:21
*** _nadya_ has joined #openstack-infra21:21
fungimore accurate is to say that at peak we have ~400 nodes in constant use and another ~450 in a state of overhead (being built or deleted)21:21
jeblairjogo: in fact, the metric we _most_ concern ourselves with is whether we are able to merge changes faster than they are added to the merge queue.21:22
jogojeblair: yeah trying to remove jobs that actually test nothing first. And not saying we *need* to drop useful tests. But we may need to in the future at this rate21:22
*** Sukhdev has joined #openstack-infra21:22
sdaguejeblair: I think the concern that jogo is expressing, which I share, is whether or not the system is believed to have headroom, when it's typically taking 3 hrs to return test results, it creates a climate that makes people not want to expand test scope21:22
jeblairjogo: as for anything else, the system is eventually consistent.21:22
jogojeblair: merge throughput is a useful metric, but that ignores the turn around time in development21:22
jeblairjogo: the primary goal of this system is not to outsource developer testing.21:23
*** otter768 has joined #openstack-infra21:23
jogosdague: correct, that and it has a direct impact on iteration time of work21:23
fungiit also looks like at the moment changes complete check in under 2 hours, and changes are taking at worst a couple hours to make it through the gate21:23
*** e0ne has quit IRC21:23
*** annegentle has joined #openstack-infra21:23
*** doug-fish has left #openstack-infra21:24
*** david-lyle has quit IRC21:24
jogofungi: yeah gate has been pretty solid recently.21:24
openstackgerritVictor Seva proposed openstack-infra/jenkins-job-builder: Allow use of aliases defined previously inside included files  https://review.openstack.org/15773521:24
jeblairthat sounds like well within what i think has traditionally been considered acceptable times for quite a while21:24
jogojeblair: right. How would you sate the primary goal of the system?21:24
fungithough it looks like glance unit tests may have a problem at the moment21:25
*** ibiris is now known as ibiris_away21:25
jogojeblair: right. How would you state the primary goal of the system?21:25
jeblairjogo: to ensure that only changes which pass tests merge21:25
jeblairjogo: to ensure that only changes which pass tests merge21:25
*** _nadya_ has quit IRC21:25
*** peristeri has quit IRC21:26
jogojeblair: my first line had a typo, your didn't :)21:26
fungia goal so nice he stated it twice!21:26
jogolol21:26
jeblairjogo: drat, didn't see it, otherwise i would have put one in! :)21:26
*** SumitNaiksatam has quit IRC21:26
jeblairjogo: i guess i was just 'satisfied' with your first question21:26
*** jgrimm is now known as zz_jgrimm21:26
fungiit passed his tests21:26
jeblair(or... 'sated')21:26
fungiohhh... indeed!21:27
*** otter768 has quit IRC21:27
*** mjturek1 has quit IRC21:28
jogoon a slightly tangential topic user survey results for postgres usage are fun http://superuser.openstack.org/articles/openstack-user-survey-insights-november-201421:28
jeblairjogo: anyway, i would like the system to be very fast and responsive, but when we're in a degraded state, i think it's reasonable for things to slow down a bit and for us to spread the load out over a larger time period21:28
jogojeblair: agreed, I didn't realize we were still in a degraded state21:29
*** pc_m has quit IRC21:30
jeblairjogo: if circumstances were slightly different, that's just about the most awesome thing you could have said.  :)21:30
jogojeblair: hehe true21:30
mordredjogo: nice survey21:30
mordredjogo: so, basically, PG is in the margin of error21:31
jeblairjogo: also, it looks like load is getting very close to capacity, so we should be getting back to instantaneous response territory soon.21:31
*** gulic has joined #openstack-infra21:31
*** Somay has joined #openstack-infra21:32
jogomordred: yup, more people use devstack that postgres21:34
*** asselin has joined #openstack-infra21:34
*** gordc has quit IRC21:34
mordredyah21:34
*** samueldmq has quit IRC21:34
*** dprince has quit IRC21:35
*** mwagner_lap has quit IRC21:35
ekarlsohave ATC codes been sent yet ?21:35
*** dkranz has quit IRC21:36
*** pblaho has quit IRC21:37
JayFYes,  a long ways back iirc21:37
*** asselin has quit IRC21:38
*** sdake has joined #openstack-infra21:38
fungiekarlso: in fact today is the last day to use them before the ticket price goes up and the discount code ceases to cover 100% of the cost21:39
ekarlsoany idea on what the subject looks like ? :p21:39
ekarlsoor what email their sent from21:39
ekarlsooh nmv !21:39
ekarlsofound it in the estack21:40
ekarlsonot haystack :P21:40
fungimake sure to use it immediately21:41
*** sdake__ has quit IRC21:42
*** david-lyle_ is now known as david-lyle21:43
jeblairanyone noticed gertty failing to do its periodic sync lately?21:44
jeblairi just noticed mine hadn't synced for a while, and i don't see an error21:44
jeblairi've added more logging locally to try to catch it21:44
jheskethMorning21:45
*** Somay has quit IRC21:45
*** mriedem is now known as mriedem_away21:45
openstackgerritMerged openstack-infra/system-config: Add a helpful friend to gerrit reviews  https://review.openstack.org/16901021:46
jeblairnibalizer: ^21:46
dstufftmordred: that makes me sad, postgres is a great db :D21:48
mordreddstufft: postgres is a great db!21:48
mordreddstufft: but most scalable applications are designed for _a_ db, not for _any_ db21:48
mordredin fact, I'd go so far as to say _all_21:48
dstufftyea21:48
mordredscalable appliations21:49
dstufftI think that pluggable DBs are a bad scene21:49
mordredso - I dont' want us to delete postgres support because it'sa  bad db21:49
dstufftexcept for frameworks and such21:49
*** ivar-laz_ has quit IRC21:49
mordredI want us to delete it because it's a stupid idea to support more than one db backend for non-frameworks21:49
dstufftmordred: to be clear, it makes me sad because i'd rather delete mysql support because of my own biases and the numbers don't back that up ;)21:49
fungi"we should also support postgres because someone may want to run openstack in a country where mysql is banned by law"21:49
mordred(I agree about framework pluggability - django supporting multi-databases is a great idea)21:49
mordreddstufft: :)21:49
openstackgerritmelanie witt proposed openstack-infra/project-config: Adjust regression exceptions for Nova Cells V1 job  https://review.openstack.org/16639621:49
mordreddstufft: that's fair - I want to delete postgres also because of my own biases21:50
mordreddstufft: I'm just lucky and have the numbers on my side21:50
jeblairmordred: that's not luck, that's skill21:50
mordredjeblair: :)21:50
lifelessmordred: I think we need to get community consensus about whether we're a product or a product toolkit21:50
dstufftalso even beyond scaling applications, supporting multi dbs means supporting some lowest common denominator subset21:50
nibalizerjeblair: awesome21:50
dstufftso you don't even get to use any of the cool features of a particular db21:50
lifelesswe're all over the map on that21:50
mordredlifeless: I believe that the foudnation has a very specific point of view on that already, actually21:50
nibalizerjeblair: my gertty always has syncing problems21:50
mordreddstufft: ++21:51
ekarlsowhat's bad with postgres ?21:51
nibalizerim always quitting and forcing syncing and stuff, sometimes it gets better21:51
mordredekarlso: nothing21:51
mordredekarlso: we'r enot going to have a db debate in here21:51
vishyher does anyone here know if the sysctl for net.bridge got removed in recent kernels?21:51
*** erlon has quit IRC21:51
*** stevemar has quit IRC21:51
mordredvishy: I do not know21:51
ekarlsomordred: ;)21:51
lifelessmordred: that may be, but the dev community still gets up in knots on questions in that space21:51
vishyrecently updated to kernel 3.19 and /proc/sys/net/bridge is gone21:51
fungivishy: bridge support compiled as a module? loaded?21:52
*** markvan has quit IRC21:52
lifelessmordred: and the DB question is one of those21:53
fungivishy: it's still in the 3.19 ip-sysctl.txt doc21:53
mordredlifeless: yes, I agree21:53
*** MaxV has quit IRC21:54
mordredlifeless: we still have not yet learned very well how to say no to things21:54
vishyfungi: yes the module is loaded21:54
vishylsmod shows it21:54
*** markvan has joined #openstack-infra21:54
fungiunfortunately i don't have any 3.19 systems handy21:54
vishyhttp://paste.openstack.org/show/197741/21:55
dstufftopenstack dictating a DB would likely be good for the project21:55
vishyis there something that needs to run post module load to create the psuedofilesystem entries?21:55
mordreddstufft: yes, it certainly would be21:55
mordreddstufft: however, so many other battles to fight21:56
fungivishy: not to my knowledge. /proc/sys is just a window into the kernel so it should reflect real-time what the kernel knows21:56
lifelessvishy: no, /proc is live from the kernel21:56
vishyso i am totally confused as to what might have happened then21:57
dstufftmordred: I am well acquainted with that feel21:57
*** baoli has quit IRC21:58
*** signed8b_ has joined #openstack-infra21:58
fungivishy: oh, did you use brctl to add a bridge? if there are no bridge interfaces that might not be present21:58
vishythere are bridge interfaces21:58
*** baoli has joined #openstack-infra21:58
*** mattfarina has quit IRC21:58
* fungi is out of ideas21:58
russellbprobably systemd's fault21:59
* fungi hands vishy a "blame lennart" tee shirt21:59
*** sdake_ has joined #openstack-infra21:59
vishypossibly21:59
vishythere are two possibilities21:59
vishya) something about running this new kernel on an old ubuntu build makes them not show up22:00
*** ociuhandu has quit IRC22:00
vishyb)  the /proc/sys/net/bridge/bridge-nf-* settings were removed from the bridge module22:00
*** tonytan_brb has joined #openstack-infra22:00
*** tonytan_brb has quit IRC22:00
*** signed8bit has quit IRC22:00
fungioh... /proc/sys/net/bridge is there but the bridge-nf-* knobs are absent?22:01
*** amitgandhinz has quit IRC22:01
*** tonytan_brb has joined #openstack-infra22:01
openstackgerritRamy Asselin proposed openstack-infra/zuul: Update zuul-changes.py doc to match usage  https://review.openstack.org/16952722:01
fungithe netfilter bridge support module is separate from the bridge interface support module. you might not have the netfilter one loaded22:01
*** david-lyle has quit IRC22:02
vishylololol22:02
vishy[  133.282713] bridge: automatic filtering via arp/ip/ip6tables has been deprecated. Update your scripts to load br_netfilter if you need this.22:02
vishyin dmesg22:02
fungilogs ftw!22:02
*** sdake has quit IRC22:03
*** rlandy has quit IRC22:03
*** tonytan4ever has quit IRC22:04
*** nelsnelson has quit IRC22:04
*** harlowja_ is now known as harlowja_away22:04
* ianw is trying very hard not to go postal at the tech support person telling me my ip address is 0.0.0.022:07
*** harlowja_away is now known as harlowja_22:07
tchaypojeblair: I can’t find the patches  I thought I read adding support for gertty using a proxy22:08
tchaypodid I dream about that?22:08
tchaypoianw: that’s … special..22:08
taronyou are a far better human than I then, I'd just lose it22:08
jeblairtchaypo: not ringing a bell, i think it was all a dream22:09
openstackgerritAaron Rosen proposed openstack-infra/project-config: Move congress projects from stackforge to openstack  https://review.openstack.org/16953022:09
tchaypoit was a pleasant dream22:09
tchaypoI wonder if requests has magical support?22:09
*** salv-orl_ has quit IRC22:09
*** salv-orlando has joined #openstack-infra22:13
anteayatchaypo: if you can get more nodes in your dream work, that is awesome22:13
anteayatchaypo: bring that code over to this side22:13
*** SumitNaiksatam has joined #openstack-infra22:14
fungitchaypo: requests has proxy support via the usual envvars afaik22:14
* fungi heads out to get dinner, back in a little while22:14
tchaypoyeah, it seems to be working22:15
*** mtanino has joined #openstack-infra22:15
tchaypobut it seems to not be a solution to my problem22:15
fungiproxies are far more often the source of problems than the solution to them22:15
tchaypofrom looking at the logs, I *think* that gertty might be slow because it opens many many connections to https://review.openstack.org22:15
tchaypoI was hoping to be able to use squid to persist a single connection and pipeline requests22:16
tchaypobut I’d forgotten about the ’s’ in https22:16
tchayposo I think my next bet is to look to see if requests has some kind of connection pooling, and if gertty already makes use of that22:17
*** jlvillal_ has joined #openstack-infra22:19
*** harlowja_ has quit IRC22:21
*** tonytan_brb has quit IRC22:23
*** asettle has joined #openstack-infra22:23
*** tnovacik has quit IRC22:23
*** medieval1 has quit IRC22:24
openstackgerritJames E. Blair proposed openstack-infra/nodepool: Add a simulator script  https://review.openstack.org/16953922:26
openstackgerritJames E. Blair proposed openstack-infra/nodepool: Add API sleeps to fakes  https://review.openstack.org/16954022:26
jeblairyolanda: ^22:26
*** harlowja has joined #openstack-infra22:26
jeblairyolanda: the first change is a very simple approximation of the nodepool api request queue model.  with it, you can make changes and run very approximate simulations quickly.22:27
jeblairyolanda: based on that, assuming i've modeled things close enough, i would expect the two-provider-queue change to result in a 60% increase in the average number of nodes we can produce over a 24 hour period under full load.22:28
*** ociuhandu has joined #openstack-infra22:28
jeblair(that script runs in a couple of seconds)22:28
jeblairyolanda: the second change will let you test out changes more accurately by using nodepool fakes instead of real quota.  It's not quite complete yet -- it needs delays added for some more API calls, but it's mostly there.22:29
jeblairyolanda: it runs in real-time though, so it's a better second-level test.22:29
*** ayoung has quit IRC22:29
jeblairmordred, clarkb, fungi: ^ fyi22:30
mordredjeblair: you are a flashing unicorn of pork22:30
tchayponope, as far as I can tell it’s already pipelining everything through a single connection22:30
*** carl_baldwin has quit IRC22:31
jeblaircody-somerville: ^22:31
jeblairmordred: i also noticed that the server list call is surprisingly slow...22:31
jeblairmordred: do you know if it suffers from the same lock issue?22:31
mordredjeblair: I do not know that22:32
jeblairit doesn't hurt us nearly as much as the delay in create/delete, because we cache it and use it for all of our servers22:32
jeblairbut it still takes 10 seconds22:32
*** carl_baldwin has joined #openstack-infra22:32
jeblair(which, btw, i think means that the priority queue change would in fact be very dangerous)22:32
mordredjeblair: agree22:32
mordredjeblair: well, I would imagine it's going to be listing that same table22:33
jeblair(i did not run that one through the simulator)22:33
mordredjeblair: so I _betcha_ if you stopped nodepool such that there was no activity on our account22:33
mordredand then ran a list (waiting a minute to let stale DB things fall away)22:33
mordredit would run more quickl22:33
mordredquickly22:33
mordredI'm betting its' table locks behind queued events22:33
mordred*guess*22:33
jeblair*nod*22:33
*** dizquierdo has quit IRC22:34
mordredjeblair: I think we can do much smarter caching aroudn that too, btw - but I have not _EVEN_ begun to code up examples of that22:35
*** timcline has quit IRC22:35
jeblairalso, it's worth noting that the simulator does not take into account the possibility that near-continuous use of the table lock by one task queue might cause even worse performance by the second (polling) task queue.  that we will have to observe experimentally.22:35
mordred++22:35
jeblairmordred: how?22:35
*** weshay has quit IRC22:35
mordredjeblair: I believe most of the time we know when we're manipulating somethign that should cause a change in teh status of a nova server22:36
jeblairmordred: the polling is about waiting for something on the cloud side to change22:36
mordredyes22:36
clarkbok I have train tickets to vancouver \o/ now if I can just manage to make the trip :)22:36
mordredbut we can poll with a show rather than a list22:36
anteayaclarkb: what are your chances?22:36
*** packet has quit IRC22:36
anteayaclarkb: I'm still going with you will have babies before summit22:36
mordredjeblair: and that operation always needs to be direct to the cloud, because it makes no sense for it not to be22:37
jeblairmordred: we list so that we can perform one call and get 600 results, rather than 600 calls22:37
mordredjeblair: oh right22:37
mordredjeblair: sorry - my brain was in a different context22:37
clarkbanteaya: last round of Dr appointments were still full of shrugs, basically we are doing our best but distinct possibility I don't go22:37
jeblairmordred: we used to hit the server directly, but we spent all of our time polling22:37
*** whoops has quit IRC22:37
mordredjeblair: ignore me22:37
anteayaclarkb: for which I will be happy for you (yay babies) and sad for me since you won't be at summit22:37
jeblairmordred: no prob22:37
*** aduarte has joined #openstack-infra22:37
anteayaclarkb: shrugs are good22:37
*** Rockyg has quit IRC22:37
mordredjeblair: if only we had some sort of notification system ...22:37
anteayaclarkb: means the babies could sit a while longer, better for them22:37
*** jlvillal_ has quit IRC22:38
*** annegentle has quit IRC22:38
jeblairmordred: some way to make messages accessible across the internet22:38
*** Krinkle has joined #openstack-infra22:38
anteayaclarkb: but yay tickets and I hope you get to attend summit22:38
mordredjeblair: I don't know - I'm not sure mongodb is designed for that22:38
jeblairhaha22:38
mordredjeblair: I mean, you could probably mongo the redis into a rabbit and then erlang something with a twisted deferred ...22:39
jeblairclarkb: i hope you make it but will also be happy for you if you don't :)22:39
KrinkleI'm looking for a way in Zuul to declare a post-merge job should doesn't queue. Similar to how Jenkins behaves by default when not allowing concurrency. Except that, due to the builds being parametized, this Jenkins behaviour doesn't happen. instead one gets a backlog of redundant post-merge jobs that run synchronously (e.g. generating documentation or doing code coverage). Basically to have it eithe22:40
Krinkler ignore triggers if one is already running, or dequeue/cancel existing builds and queue a new one (like the dependenant pipeline) either would be cool.22:40
greghaynesclarkb: 516 train?22:41
clarkbgreghaynes: ya22:41
greghaynessweet, me too22:41
clarkbon saturday22:41
*** medieval1 has joined #openstack-infra22:41
greghaynesyep22:41
*** agentle_ has joined #openstack-infra22:41
*** annegentle has joined #openstack-infra22:41
mordredKrinkle: as in, you want "build docs for $project for commit X" to skip commit Y if Z has already come in by the time you're ready to build it again, yes?22:42
*** sigmavirus24 is now known as sigmavirus24_awa22:42
Krinklemordred: Yeah, post-merge we trigger builds to generate documentation, but it takes 10 minutes, and if a minute later another one has been merged it should either cancel the running one or drop the event. Either would be fine.22:42
jeblairKrinkle: i don't think there's a way to do that out of the box; i think we'd need to add functionality to recognize that a commit is a later commit on the same ref.  that shouldn't be too hard to do.22:43
KrinkleBy target branch basically22:43
*** marcusvrn has quit IRC22:43
jeblairyeah22:43
Krinklemaking it refer to a branch instead of a zuul/ref. Just queue a build for 'master'. It would have the benefit of picking up the latest commit if it takes a while to get a free executor slot.22:44
jeblairKrinkle: you would need to implement the "isUpDateOf()" method on the Ref class in zuul/model.py22:44
jeblairer "isUpdateOf"22:44
jeblairthat currently always returns false22:44
*** andreykurilin_ has joined #openstack-infra22:45
KrinkleI noticed today we've got a release day (once a month) and there are lots of backports all being merged around the same time and our doc publisher is basically just spending the entire day generating documetnation for old commits.22:45
*** bhunter71 has quit IRC22:45
tchaypojeblair: so tailing the gertty logs, it looks like every time I restart gertty it starts scanning down the list of projects alphabetically - which means it spends a lot of time grovelling through all the diskimage-builder changes it last synced a few minute ago and never gets down to the tripleo projects it hasn’t synced in ages22:46
*** agentle_ has quit IRC22:47
clarkbKrinkle: as a simple alternative you could trigger those jobs on tags instead of merges22:47
*** marcusvrn has joined #openstack-infra22:47
clarkb(which is moving the problem not solving it)22:47
Krinkleclarkb: Hm.. that would work for release branches which eventually get tagged.22:47
*** dimtruck is now known as zz_dimtruck22:47
KrinkleWouldn't help for 'master' though.22:48
*** jlvillal_ has joined #openstack-infra22:48
*** jlvillal_ has quit IRC22:49
*** devlaps has joined #openstack-infra22:50
*** packet has joined #openstack-infra22:50
*** salv-orlando has quit IRC22:51
*** dustins has quit IRC22:52
*** sabeen has quit IRC22:52
openstackgerritAaron Rosen proposed openstack-infra/project-config: Move congress projects from stackforge to openstack  https://review.openstack.org/16953022:53
*** devlaps has quit IRC22:54
*** salv-orlando has joined #openstack-infra22:59
arosenarosen22:59
*** claudiub has quit IRC23:04
*** baoli has quit IRC23:05
*** dboik_ has joined #openstack-infra23:05
mordredmordred23:05
arosenwhoops *23:06
arosen:P23:06
arosenHi, just curious if you guys have seen these kernel issues in the upstream ci? http://logs2.aaronorosen.com/78/169278/1/check/dsvm-tempest-full-congress-nodepool/46c1873/logs/syslog.txt.gz#_Mar_31_22_53_4923:06
arosenI'm using the nodepool with dib and trusty23:07
*** thedodd has quit IRC23:07
*** dboik has quit IRC23:08
*** bswartz has joined #openstack-infra23:09
*** jaypipes has quit IRC23:09
*** dboik_ has quit IRC23:10
salv-orlandoI opened gerrit and clippy popped up! Is it a fool's day joke or do I have to burn my laptop?23:11
openstackgerritgreghaynes proposed openstack-infra/system-config: Give some actions to our friend  https://review.openstack.org/16950923:11
anteayasalv-orlando: you can review the patch to remove it23:11
salv-orlandoanteaya: ok... doing that ;)23:12
anteayasalv-orlando: https://review.openstack.org/#/c/169501/23:12
openstackgerritgreghaynes proposed openstack-infra/system-config: Give some actions to our friend  https://review.openstack.org/16950923:13
anteayasalv-orlando: you are also welcome to burn your laptop, send a photo if you do23:13
anteayanibalizer: is that the patch folks should be reviewing?23:13
*** annegentle has quit IRC23:13
nibalizeranteaya: there are two reverts in flight23:14
*** dims_ has quit IRC23:14
nibalizermine and jims23:14
anteayajeblair nibalizer who wins in the patch23:14
anteayayeah, can we pick one23:14
nibalizeri can abandon mine23:14
anteayaso salv-orlando's review hits the right one?23:14
nibalizerthough his has a -1 saying use mine so thats confusing23:14
anteayayes23:14
anteayaso jeblair how do you feel?23:14
nibalizermine is abandoned23:15
anteayajeblair: you have 508, nibalizer has 50123:15
anteayaokay23:15
*** davideagnello has quit IRC23:15
*** pcrews_ has joined #openstack-infra23:15
anteayasalv-orlando: review jim's at 508 please23:15
*** pcrews has quit IRC23:15
*** sputnik13 has quit IRC23:15
*** dims_ has joined #openstack-infra23:17
*** bhunter71 has joined #openstack-infra23:17
*** pcrews__ has joined #openstack-infra23:18
*** pcrews__ has quit IRC23:18
openstackgerritJames E. Blair proposed stackforge/gertty: Sync starred changes regardless of subscription  https://review.openstack.org/16850823:19
openstackgerritJames E. Blair proposed stackforge/gertty: Sync own changes regardless of subscription  https://review.openstack.org/16818123:19
openstackgerritJames E. Blair proposed stackforge/gertty: Support searching by URL  https://review.openstack.org/16937123:19
openstackgerritJames E. Blair proposed stackforge/gertty: Add checkout and cherry-pick to change list  https://review.openstack.org/16851023:19
openstackgerritJames E. Blair proposed stackforge/gertty: Add held changes  https://review.openstack.org/16783823:19
openstackgerritJames E. Blair proposed stackforge/gertty: Highlight starred changes in list  https://review.openstack.org/16657223:19
*** carl_baldwin has quit IRC23:20
*** pcrews_ has quit IRC23:20
*** mwagner_lap has joined #openstack-infra23:21
jeblairanteaya: i wasn't planning on merging the revert change until thursday23:21
*** Sukhdev has quit IRC23:22
*** marun has quit IRC23:22
*** medieval1 has quit IRC23:23
greghaynesI was just able to test 169509 in prod if theres any interest in adding some more life to our friend. I can update the revert once it goes in23:23
*** jaypipes has joined #openstack-infra23:23
*** bhunter71 has quit IRC23:23
*** otter768 has joined #openstack-infra23:24
*** hichihara has joined #openstack-infra23:24
*** asettle has quit IRC23:24
*** davideagnello has joined #openstack-infra23:24
jeblairgreghaynes: do you know if it works with old and new change screens?23:28
anteayajeblair: oh okay23:28
*** otter768 has quit IRC23:28
anteayasalv-orlando: burn your laptop23:28
reedis there documentation for the process (TC approval and all) to move from stackforge to openstack/?23:28
greghaynesjeblair: easy to test, I believe I am using the old one, how do I get to the new?23:29
openstackgerritNan Liu proposed openstack-infra/puppet-elasticsearch: Use archive to fetch/checksum instead of wget.  https://review.openstack.org/16490823:29
jeblairgreghaynes: https://review.openstack.org/#/settings/preferences23:30
nibalizergreghaynes: or s/c/c2/ in your url23:30
anteayareed: http://docs.openstack.org/infra/manual/creators.html#add-project-to-the-governance-repository23:30
*** melwitt has quit IRC23:30
anteayareed: is this for the new contributors work you are doing?23:30
reedanteaya, no, it's for this project ...  https://github.com/stackforge/networking-bgpvpn23:31
reedand the fact that they referenced github can be considered a sign23:31
*** andreykurilin_ has quit IRC23:31
reedbut the readme lists git.openstack as the source, which is good23:32
ianwwhy have we added something that auto-plays sound and can't be turned off ... it's rather obnoxious even as a joke.  at least you could turn the real clippy off23:33
jeblairnibalizer: there is sound?23:33
*** unicell has quit IRC23:34
anteayareed: why do they need to be in openstack?23:34
reedanteaya, that page on docs is not enough for my purpose. It doesn't describe or point to the requirements that the TC expects, like diversity of contributors and criteria for the voting etc23:34
*** unicell has joined #openstack-infra23:34
reedanteaya, I don't know, that was not their question :)23:34
jeblairreed: the tc does not require diversity of contributors23:34
jeblairreed: the tc requirements are in the governance repo23:34
anteayareed: that is the resolution in the governance repo23:34
jeblairreed: you can point there23:34
reedjeblair, anteaya: heading to the gov repo, thanks23:34
anteayais it taboo to ask people their motivation for their actions?23:35
* anteaya still does so23:35
nibalizerjeblair: yes23:35
clarkbjeblair: apparently there is sound. I operate with sound off as a general rule so didn't notice23:35
nibalizerand im not sure if we can turn that off23:35
*** wenlock has quit IRC23:35
reedanteaya, no, I just don't have time to engage at that level every time23:35
anteayareed: I can understand that23:35
jeblairnibalizer, clarkb: i think we need to revert then :(23:35
clarkbor turn sound off23:36
anteayareed: to be honest they should familiarize themselves with the whole governance repo23:36
reedhttp://governance.openstack.org/reference/new-projects-requirements.html23:36
jeblairclarkb: yeah, but nibalizer says that's an unknown23:36
reedthat's what I was looking for23:36
clarkbhttps://github.com/smore-inc/clippy.js/blob/master/src/animator.js#L160 just comment out that line23:37
greghaynesjeblair: does not work on new one, and looks like theres not really an easy way to make it work there23:38
*** changbl has joined #openstack-infra23:38
greghaynesjeblair: but, it also doesnt fail there, just doesnt do anything new23:38
*** tkelsey has joined #openstack-infra23:38
jeblairnibalizer: can you fix the sound thing quickly?23:38
nibalizerjeblair: i think so23:39
greghaynesalso, trying to figure out how gwt generated js works is a bit painful23:39
*** achanda has quit IRC23:41
jeblairi have to run now; unless another core is around to babysit the fix, i think we need to revert it.23:42
*** sputnik13 has joined #openstack-infra23:42
clarkbI can merge the fix if it hsows up23:42
jeblairkk23:42
*** tkelsey has quit IRC23:42
greghaynesnibalizer: I think I have a fix23:45
greghaynesnibalizer: if you want me to push23:45
openstackgerritAaron Rosen proposed openstack-infra/project-config: Move congress projects from stackforge to openstack  https://review.openstack.org/16953023:45
nibalizergreghaynes: doit23:45
*** signed8b_ is now known as signed8bit_ZZZzz23:45
nibalizeri tried just commenting out the line clarkb pointed out and minifying but thats failing on my testing rig23:45
nibalizerofc my testing rig is kinda janky so that could be just me23:45
*** pcrews has joined #openstack-infra23:47
openstackgerritgreghaynes proposed openstack-infra/system-config: Disable our friend's noise  https://review.openstack.org/16956723:47
clarkbgreghaynes: I assume you have tested?23:47
greghaynesYes, let me doubletest though23:48
clarkbkk23:48
greghaynesyep, no sound and seems to work23:49
greghaynes(if anyone else wants to test just paste the js into your browser console without date restriction)23:49
nibalizeri've got gregs deployed here and i can't get any sound out of it23:51
nibalizerhttp://reviewtest.cloud.nibalizer.com/#/c/123:51
clarkbits approved expect it to be everywhere shortlyish23:51
*** hdd has quit IRC23:54
*** dannywilson has quit IRC23:54
*** wicope has quit IRC23:54
*** hdd has joined #openstack-infra23:55
harlowjais review.openstack.org slow for anyone else?23:55
*** achanda has joined #openstack-infra23:55
harlowjathe images for reviews are taking forevdr to load23:55
harlowjasorta weird23:56
clarkbharlowja: it does that occasionally because memory leak23:56
harlowjaah23:56
clarkbI can check how it is doing23:56
clarkbhrm we don't appear to be leaking at least its not terribad like normal23:56
*** smccully has quit IRC23:56
harlowjahmmm, maybe just something else then, weird23:57
*** smccully_ has quit IRC23:57
*** ayoung has joined #openstack-infra23:57
clarkblooking at chromium dev tools it seems to be spending its time in gerrit api rpc calls23:59
clarkbalso a fair bit of time looking up avatars that don't exist for our users23:59
clarkbzaro: ^ any way to turn that off?23:59

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