Wednesday, 2015-05-27

mordredclarkb: yah00:00
jrollharlowja_: it's interesting to me that oslo.concurrency doesn't provide anything for actual distributed locking between two computers00:01
jrollharlowja_: where it seems like that's tooz's primary goal00:01
*** emagana has quit IRC00:01
*** camunoz has quit IRC00:02
harlowja_jroll yup, tooz was made to be that00:02
harlowja_*or part of it was00:02
harlowja_again goes back to the current idea, make generic libraries, if openstack needs special config or whatever, make a oslo.* library for just that stuff00:03
jrollharlowja_: right, so maybe I'm misunderstanding the problem here, but it doesn't seem like those really overlap00:03
harlowja_i think the discussion is just about 'oslo.* does what, wait a second it doesn't do much but wrap something else, whatttttt'00:03
harlowja_and that devolves into well than what is oslo...00:04
harlowja_*which i can only offer opinions on :-P00:04
*** annegentle has quit IRC00:04
clarkbharlowja_: and specifically should we be pushing these "special" things upstream00:04
harlowja_others may (and afaik do) have different opinions00:04
harlowja_depends on how special, lol00:05
harlowja_not everyone wants to see your ugly stepchild00:05
harlowja_*even if they are special00:05
* harlowja_ oh man, bad analogy, lol00:05
clarkbharlowja_: but thats an indication that it doesn't belong in any lib00:05
clarkbharlowja_: eg if its ugly then all of openstack should not be using it00:05
harlowja_u saying my ugly stepchild isn't pretty00:06
harlowja_how dare u00:06
harlowja_lol00:06
harlowja_clarkb all valid points00:06
harlowja_if i could of done it all differently, i might have :-P00:06
harlowja_time travel not working00:06
harlowja_ha00:06
fungigah, it already merged00:07
clarkbfungi: its ok we can follow up with cleanup00:07
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Simplify nodepool diskimage logging  https://review.openstack.org/18578800:08
fungiclarkb: that's ^ what we want? ^00:08
clarkbfungi: ya I think so00:08
fungii was hoping to use the pending change rebased on that to prove it didn't add new sections00:08
fungithat explains why gerrit was giving me fits trying to submit the rebased version00:08
clarkboh :(00:08
fungii wasn't going crazy after all ;)00:08
clarkbbut the removals from the logging conf look correct to me00:09
*** annegentle has joined #openstack-infra00:09
harlowja_bnemec probably has some ideas/comments on all this to00:09
harlowja_*if he's around00:09
harlowja_we chatted in #openstack-oslo this morning (my time) about that same question ;)00:09
harlowja_mordred where's that mini-cloud-init repo u have00:10
*** ZZelle_ has quit IRC00:10
clarkbharlowja_: glean00:11
harlowja_thx00:11
*** nelsnelson has quit IRC00:11
clarkbmordred: I think you can make a new shade now00:11
fungiharlowja_: http://git.openstack.org/cgit/openstack-infra/glean/tree/00:11
harlowja_yuppers, thx, mordred  have u and jroll been in contact about the network format changes (that i think u mentioned)00:12
harlowja_i think jroll was (or knows the person) who was proposing one (maybe the same one?)00:12
jrollharlowja_: what about it00:12
jrollharlowja_: yeah, that's my team working on it, both nova and cloud-init side00:12
clarkbchanges atop what already is used?00:12
jrollprimarily JoshNang00:12
clarkbor changes being changes to nova00:12
harlowja_jroll k, the same one mordred  was talking about (?)00:12
jrollclarkb: it *still* isn't in nova00:12
harlowja_*if u know00:12
jrollyes00:12
harlowja_cools00:12
mordredformat change?00:13
mordredlike, other than what's in the nova tree?00:13
jrollharlowja_: assuming that's the stuff running in rax cloud00:13
harlowja_mordred something something, network config json00:13
jrollI'm not aware of schema changes in quite a while00:13
mordredyah00:13
jrollbut yes, that00:13
mordredI test for network-config and also for vendor.json00:13
mordredbtw - I have functional tests for all known forms of this00:13
harlowja_right, k00:13
mordredhttp://git.openstack.org/cgit/openstack-infra/glean/tree/glean/tests/fixtures00:13
harlowja_cool00:13
mordredhttp://git.openstack.org/cgit/openstack-infra/glean/tree/glean/tests/fixtures/liberty/mnt/config/openstack/latest/network_info.json00:14
mordredfor instance00:14
harlowja_cool beans00:14
mordredand http://git.openstack.org/cgit/openstack-infra/glean/tree/glean/tests/fixtures/rax/mnt/config/openstack/latest/vendor_data.json00:14
mordredso - PLEASE feel free to steal/use them in cloud-init v200:14
jrollharlowja_: we have patches for cloud-init v1 if that's useful to you00:14
openstackgerritColleen Murphy proposed openstack-infra/project-config: Install zlib1g-dev for puppet beaker jobs  https://review.openstack.org/18579100:14
harlowja_so the next question (applicable to glean, cloud-init) is nova (or neutron) going to update that :-/00:15
harlowja_*when hotplugs happen...00:15
*** camunoz has joined #openstack-infra00:15
clarkbharlowja_: config drive is never updated aiui so no?00:15
mordredah - well, glean does not support hotplug/config-drive changes00:15
jrollharlowja_: config drive is versioned00:15
jrollharlowja_: so if we change it, it will be a new configdrive version00:15
harlowja_mordred right, thats one of the major reasons why scott wants new cloud-init i think00:15
*** markvoelker has joined #openstack-infra00:16
harlowja_for that damn hotplug stuff (which apparently works on amazon?)00:16
mordredah00:16
clarkbharlowja_: with config drive?00:16
mordredwell, I do not want to support it myself00:16
mordredI am of the opinion that if you are adding devices to your cloud instance00:16
harlowja_ya, except its the cloud, things are dynamic, annnd magic!00:16
jrollI'm not sure how configdrive would support hotplug, but hey00:16
mordredthen you are doing it from config management00:16
clarkbjroll: right I don't think it can00:16
mordredand you should be able to reach-in and do it00:16
fungii would be surprised to see a nodepool-booted diskimage-builder instance survive long enough to see a hotplug event, nor would we be too concerned if it went toes-up00:16
clarkbso this would be a metadata only use case00:16
jrollyou'd have to unmount, rebuild, and remount00:17
clarkbjroll: which is not something that nova does00:17
mordredthis is why I do NOTHING in glean other than get a node ssh-able00:17
*** sigmavirus24_awa is now known as sigmavirus2400:17
clarkbjroll: and I think config drive is a nova thing?00:17
mordredanything past that can be managed by config management00:17
mordredand by config management, I mean ansible :)00:17
clarkbmordred: can has shade release?00:17
mordredclarkb: yes - one sec00:17
jrollclarkb: harlowja_ though I assume all this metadata stuff has the same schema in config drive and the metadata service, so maybe it's relevant00:17
clarkbjroll: aiui no00:17
clarkbjroll: and thats part of the prblem with what rax is doing on config drive?00:17
jrollx.x00:18
openstackgerritJames E. Blair proposed stackforge/gertty: Advance cursor on change list toggle  https://review.openstack.org/18569800:18
openstackgerritJames E. Blair proposed stackforge/gertty: Allow reviewing one change in change list  https://review.openstack.org/18569700:18
openstackgerritJames E. Blair proposed stackforge/gertty: Fix updating flags on threaded changes  https://review.openstack.org/18569600:18
openstackgerritJames E. Blair proposed stackforge/gertty: Add ability to review multiple changes at once  https://review.openstack.org/18569500:18
jrollclarkb: no clue if that's a problem00:18
clarkbmordred would know but I thought this was a completely new schema00:18
mordredit's a new schema in liberty00:19
mordredin rax it's in vendor-data (appropriately)00:19
clarkband since metadata server is an ec2 thing its not really something you can just add to/change and make happ00:19
jrollmordred: *hopefully* in liberty :P00:19
clarkbmordred: ah ok00:19
mordredjroll: if it's not in liberty I will personally kill every single member of the nova core team with my tongue00:19
clarkbmordred: and metadata service supports it in liberty?00:19
jrollclarkb: aha, I see00:19
mordredclarkb: you CAN add to ec2 metadata service00:19
jrollmordred: right, I'd like to keep them around so I hope it's in liberty :P00:19
mordredclarkb: is the same as config drive - it has openstack stanzas in the path00:20
clarkbmordred: gotcha00:20
mordredclarkb: however, I know little about it - because it's terrible00:20
*** david-lyle has quit IRC00:20
openstackgerritJames E. Blair proposed openstack-infra/project-config: Update infra-specs voting rules  https://review.openstack.org/18579200:20
clarkbmordred: ya me too, I know more about how it works than the schema provided to VMs00:20
clarkbits basically lots of NAT00:20
clarkband crying00:20
fungior drinking00:20
mordredyah00:20
mordredit's a great example of why "because Amazon" is a terrible idea00:21
mordredit's a great example of why "because Amazon" is a terrible reason00:21
*** annegentle has quit IRC00:21
harlowja_openstack also has a metadata service that isn't the ec2 one (that has a similar format as the layout on the config drive)00:21
fungijeblair: 185792 is neat... i didn't realize project-specific review labels were possible00:22
jeblairfungi: yep, also 18578500:22
*** Sukhdev has joined #openstack-infra00:22
*** sdake has quit IRC00:22
clarkbfungi: its new in gerrit 2.8 iirc00:22
jeblairfungi: check out https://review-dev.openstack.org//#/c/5308/1,publish to see it in action00:22
fungineat-o!00:22
clarkbso part of our migration to 2.8 was update all-projects with our review labels instead of relying on the DB for it00:22
harlowja_btw, would u guys mind getting in https://review.openstack.org/#/c/185095/ (thats been blocking tooz for a while)00:23
*** davideagnello has quit IRC00:23
*** markvoelker has quit IRC00:23
harlowja_i'm getting kazoo to release (After its current travis weirdness is resolved)00:23
*** markvoelker has joined #openstack-infra00:23
fungiharlowja_: i take it the global requirements reviewers are on hiatus?00:23
jeblairyou can also turn Code-Review off completely, but i chose to re-use the category in both those changes, just with altered definitions (it seemed close enough)00:23
harlowja_fungi probably in the mountains around vancover or something00:24
jeblairi smoked some pork shoulder yesterday.  i'm going to eat some more of it now.00:24
clarkbjeblair: nice00:25
*** salv-orlando has joined #openstack-infra00:25
*** puranamr has quit IRC00:25
*** sarob has quit IRC00:25
*** davideagnello has joined #openstack-infra00:25
clarkbok shade upgraded on nodepool.o.o I am going to restart nodepool now00:26
*** sputnik13 has quit IRC00:26
*** puranamr has joined #openstack-infra00:26
clarkbthats done, we should be ready for uploading images tomorrow00:26
clarkbfungi: how is that pvmove?00:26
fungiclarkb: right at the 2/3 mark now00:27
clarkbawesome so that should be done prior too00:27
fungiprobably another hour and it should wrap up00:27
* clarkb is happy to have gotten this done today00:27
*** annegentle has joined #openstack-infra00:27
clarkbjhesketh: you around? before I run off wanted to talk about the os loganalyze pass through now that shade/nodepool are sorted00:28
fungii almost feel like looking into nodepool problems has gotten me slightly closer to testing the bindep-related things i wanted to test today00:28
clarkbjhesketh: the idea I had was to have apache call htmlify/foo on things that need htmlification, then have a passthrough/foo for things that should be passed through as is00:28
clarkbjhesketh: I think it would be very simple to get that change in and since its just an apache rewrite rule we can test it with logs-dev prior to updating logs.o.o00:29
jheskethclarkb: I'm around but in a videoconf, I'll ping back in 15min?00:29
*** mohitsharma has joined #openstack-infra00:29
*** davideagnello has quit IRC00:29
clarkbjhesketh: sure00:29
*** puranamr has quit IRC00:30
*** davideagnello has joined #openstack-infra00:33
*** ddutta has joined #openstack-infra00:34
*** mohitsharma has quit IRC00:34
fungijeblair: on 185792 how is the normal requirement for at least one code-review +2 satisfied?00:35
fungijeblair: or is the requirement just that there's at least one vote of the maximum defined value for every label?00:36
*** annegentle has quit IRC00:36
clarkbfungi: thats what it is, the gerrit "function" is at least one maxmimum for every label and no mins00:36
fungiaha, got it00:37
fungiso by not definining the code-review label's vote range beyond +1, a +1 will satisfy00:37
clarkbyup00:37
jeblairfungi, clarkb: actually i'm setting code-review's function to 'noop'00:37
jeblairwhich means it's effectively ignored by gerrit00:37
clarkbah ok00:38
jeblairthe normal function is "MaxWithBlock"00:38
clarkbI think there are 3 built in functions, ^, Max, and noop00:38
funginot i see the fnords00:38
fungier, now00:38
jeblairhttps://review.openstack.org/Documentation/config-labels.html#label_function00:38
*** otter768 has joined #openstack-infra00:39
clarkbok rax is giving us 413s for image uploads00:40
*** btully has quit IRC00:40
clarkbHTTPOverLimit retry at: footime00:40
jroll:|00:41
clarkbmordred: do you know if we need quota bumps?00:41
clarkbor is that purely a rate limit action?00:41
jrollclarkb: rate limit bumps are a thing, too00:41
clarkbjroll: well we are very "idle" right now so any rate limiting that is happening is very weird00:41
clarkbOH! its using glance v100:42
clarkbso the error messge is terrible because we are doing the wrong thing completely00:42
jroll:|00:42
clarkbI think the fix is to set the cloud: value in nodepool to rackspace00:42
fungiif rackspace: tasks(badnez)00:43
*** otter768 has quit IRC00:44
clarkbI am writing a nodepool.yaml patch now00:44
clarkbits a really weird error to give when an endpoint is not useable though00:44
*** yamahata has quit IRC00:44
fungimakes me wonder how they "disabled" the v1 api (max rate of 0.0/s?)00:45
*** nelsnelson has joined #openstack-infra00:46
clarkbI am trying to figure out where shade decides to do the glance v2 + tasks thing00:46
clarkbto make sure that my fix is sufficient00:46
crinklenibalizer: clarkb I commented on https://review.openstack.org/#/c/185754/ , I don't think adding a new hypervisor is the way to go but I think I have workarounds for getting latest beaker to work00:46
fungiany time we talk about interesting openstack service design choices, i now see mordred's worstcat slide in my mind00:47
*** tiswanso has joined #openstack-infra00:47
openstackgerritClark Boylan proposed openstack-infra/system-config: Tell nodepool/shade that rax clouds are rax  https://review.openstack.org/18579500:49
clarkbnow that I think about it there is probably already a change to set ^00:49
clarkbhttps://review.openstack.org/#/c/168602/4/modules/openstack_project/templates/nodepool/nodepool.yaml.erb there is but I think the cloud value used is wrong00:50
clarkbmordred: ^ reading os-c-c and nodepool's cloud.yaml I believe the cloud name should be rackspace not rax. Can you confirm?00:50
*** mwhagedorn has joined #openstack-infra00:51
jheskethclarkb: pong00:54
clarkbjhesketh: o/00:54
clarkbjhesketh: just trying to think about how we might be able to speed up the swift data pass through feature to os loganalyze00:55
jheskethso re your os-log note, yes, I agree00:55
clarkbjhesketh: and one idea I had was to use logs-dev.o.o which we can do if we make it controllable via the apache vhost00:55
jheskethclarkb: right.. so the issue was that the refactor I did broke something so it was reverted00:56
jheskethI never got the logs to look into what went wrong exactly00:57
clarkbjhesketh: ya neither did I00:57
jheskethbut the plan was to make better testing to catch the issue00:57
clarkbaiui that all happened when you and I were sleeping00:57
jheskeththen re-do the refactor00:57
jheskethto be clear, the revert was the right move00:57
clarkbjhesketh: and I think having better testing is fine, my concern is that every patch we make has gotten -1 we should go redo this other complicated and long term thing00:57
clarkbjhesketh: which worries me because the goal here is a simple passthrough which we can test effectively without all that00:57
clarkball we need is to have code that is dormant in prod because htmlify/ is used then test a passthrough/ root on logs-dev then add that to logs.o.o00:58
fungiclarkb: are you sure re rax vs rackspace? it looked like nodepool was matching its cloud parameter to keys in the "clouds" list, not in the "cloud" subkeys of the associative arrays within them00:58
*** ddutta has quit IRC00:58
clarkbfungi: I am not sure00:59
fungii'm trying to renavigate that now to confirm00:59
clarkbfungi: but the value in cloud.yaml seems to be rackspace and the value at https://git.openstack.org/cgit/openstack/os-client-config/tree/os_client_config/vendors.py#n26 is rackspace too00:59
*** ddieterly has joined #openstack-infra00:59
clarkbfungi: mordred's change to do similar uses rax and I think that is wrong based on ^01:00
jheskethclarkb: so I'd rather not throw the work out that we did to refactor os-loganalyze, but I'm not sure how we merge it without the better testing01:00
clarkbjhesketh: have it switch based on the root url path, and have a unittest that covers the handling of the siwtching01:01
clarkbjhesketh: that allows us to test via unittests the selection method, then test the actual passthrough with logs-dev.openstack.org01:01
clarkbso the code paths for htmlify/ should be changing much at all (iirc the refactor doesn't touch those much)01:02
clarkband the refactor is reasonably well tested via the unittests01:02
*** alexsyip has quit IRC01:02
*** salv-orlando has quit IRC01:03
jheskethclarkb: the patch that failed didn't do any of the pass through stuff. It was a refactor as prereq to it https://review.openstack.org/#/c/172711/401:04
jheskethso it should have been tested by the unit tests, but we didn't catch something01:04
*** asselin_ has joined #openstack-infra01:04
jheskethapparently related to the wsgi app01:04
fungiclarkb: based on my reading of os-client-config, the function shade is calling is returning a dict of the keys/values in the "clouds" associative array from clouds.yaml, so "rax" and "hpcloud" not "rackspace" and "hp"01:04
jheskethif we want that refactor to go in we need to find what was wrong and (hopefully) test it01:05
fungiclarkb: but i agree this is confusing enough that i'm not 100% sure01:05
clarkbfungi: clouds.yaml is rackspace too iirc01:05
openstackgerritMerged openstack/requirements: Kazoo 2.1 has a bug which is breaking tooz gate  https://review.openstack.org/18509501:05
fungiclarkb: http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/templates/nodepool/clouds.yaml.erb says "rax"01:06
fungias one of the keys in the "clouds" array01:06
clarkbfungi: oh I see what you are saying I would be very very very sad if the cloud: foo in cloud.yaml isn't meant to match cloud:foo in nodepool.yaml01:06
fungithen its "cloud" key maps to "rackspace"01:06
clarkbso much so that I think next step is to fix that01:06
fungiclarkb: we have cloud being used at two levels, so conflating the meaning of each01:07
clarkbhow to be confusing, use the same key twice with two different semantic meanings :/01:07
clarkbfungi: yup01:07
fungiyeah01:07
fungionce is meant to be "type of cloud/provider environment" and one is meant to be "some specific credential set"01:07
clarkbjhesketh: so your preference would be to keep trying to make the devstack integration test work?01:07
clarkbfungi: ok I can abandon my change, https://review.openstack.org/#/c/168602/4/modules/openstack_project/templates/nodepool/nodepool.yaml.erb is the one to review01:08
clarkbcan worry about unconfusing the meaning of cloud later01:09
fungiyep, agreed. i already looked over it which is what was spurring me on to try to figure this out by relearning the codebases involved01:09
ianwgreghaynes: has anyone ever considered some sort of conditional matching on element-deps, such that different elements are chosen based on the release?01:09
clarkbfungi: I am rechecking it, the nodepool test should pass now as cloud is a valid config parameter01:09
ianwi am considering the best way to handle differences with 'dnf' on f2201:10
clarkbfungi: then I think gerrit will want a rebase01:10
clarkbbecause it depends on an outdated change01:10
fungiclarkb: also 168602 does say "hpcloud" rather than "hp" so this seems to agree with my interpretation of how this is translating in the chain of nodepool->shade->os-client-config01:10
jheskethclarkb: I'm not sure. I want to be pragmatic about it too. I think perhaps we either need the devstack integration testing or otherwise find the bug locally, fix it, and be satisfied it's going to work somehow (which possibly means untested). Otherwise we could drop the refactor and just look at getting passthrough working01:10
clarkbfungi: ya I think you are correct01:10
clarkbjhesketh: I was fine with the integration testing until it too fell victim ot "hey lets massively rewrite this other thing"01:11
fungiclarkb: it's not going to merge without a rebase, so rebasing it now01:11
*** dims has quit IRC01:12
fungimerge conflict on nodepool.yaml.erb01:12
clarkbjhesketh: though mtreinish thought that it was possibly already something devstack supported so I may be overreacting to that (sdagues comment implied that we would need to add a new feature to devstack)01:13
*** spzala has quit IRC01:13
jheskethclarkb: well this here should add support for having os-loganalyze installed in devstack: https://review.openstack.org/#/c/180553/01:13
fungitripped up by the api-timeout additions01:14
jheskethclarkb: so now we just need a test to hit the server and check the file is there... that should confirm that apache/wsgi/os-loganlayze are playing nicely togethe01:14
jheskethr01:14
*** patrickeast has quit IRC01:14
jheskethclarkb: so it may not be that much more work01:14
clarkbjhesketh: no, the adding magic to requirements got shot down with a suggetion that we make devstack support that for devstack plugsin01:14
jheskethbut I don't know yet because I haven't done it01:14
jheskethoh, I see now :-s01:15
clarkbhttps://review.openstack.org/#/c/180552/01:15
clarkbwhich is a dep01:15
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Turn on persistent dbm-based caching in shade  https://review.openstack.org/16863201:15
clarkband why I am frustrated :/01:15
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Switch to consuming cloud auth from clouds.yaml  https://review.openstack.org/16860201:15
fungiclarkb: ^ updated the remainder of that stack with a rebase to master tip01:15
clarkbfungi: thanks01:15
*** btully has joined #openstack-infra01:15
greghaynesianw yea, we explicitly try not to use element deps for element selection, only for actual dependencies01:16
jheskethclarkb: so we could work on fixing the refactor without tests, get enough buy in that it's good, merge it and when requirements/d-g/whatever figure out how to install magic then we add the test01:16
jheskethclarkb: it's not ideal, but it's a way forward01:16
clarkbfungi: thoughts on approving 168602 now? it will either just work or it will completely break nodepool :)01:16
*** mohitsharma has joined #openstack-infra01:17
jheskethclarkb: otherwise the most direct route is to drop the refactor and just do the minimal amount for a passthrough01:17
fungiclarkb: wouldn't be the first time today?01:17
clarkbfungi: nah nodepool is mostly working today, just image uploads are broken :)01:17
clarkbfungi: but I guess I am around so can babysit01:17
greghaynesianw: The idea is the composition is supposed to be as independent as possible. There are some cases where meta elements abd stuff crop up though01:17
ianwgreghaynes: what i mean is that DIB_RELEASE=22 for fedora should probably select a dnf element, rather than yum01:18
clarkbjhesketh: ya so I was trying to figure a way we could sanely test and I rmembered logs-dev exists01:18
ianwgreghaynes: although, i think KISS at this point and a few branches will do us ... still testing01:18
clarkbjhesketh: probably iwth that is it shares the same install so its mostly only good for checking vhost changes01:18
*** SumitNaiksatam has quit IRC01:18
clarkbjhesketh: we could chnage it to install to a venv and run out of there separately then use it01:18
jheskethclarkb: not sure how that'll help test it unless we manually patch logs-dev?01:19
clarkbjhesketh: ya we would either manually update the venv or have puppet install a non merged version of os-loganalyze01:19
*** btully has quit IRC01:20
*** tiswanso has quit IRC01:20
clarkb(ftr I don't think we should throw out the work that has been done, testing is good, I just don't want to wait for devstack to figure out how plugins can/should affect requirements)01:21
*** tiswanso has joined #openstack-infra01:21
jheskethclarkb: okay, so maybe we do the plan where we find the bug, (confirm it works in logs-dev) merge it without tests then once devstack stuff is figured out add them01:22
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Turn on persistent dbm-based caching in shade  https://review.openstack.org/16863201:22
fungifixed a typo i found in that ^ one01:22
*** jamielennox|away is now known as jamielennox01:23
clarkbjhesketh: sounds good to me01:24
*** spzala has joined #openstack-infra01:24
*** sigmavirus24 is now known as sigmavirus24_awa01:24
clarkbI need to pop out now for dinner (will keep an eye on nodepool since I approved that change), but I think next step for that is deploy the reverted chagne to logs-dev via a virtualenv, there is a puppet module to manage venvs so shouldn't be too bad01:25
clarkbjhesketh: I can take a stab at ^ tomorrow likely01:25
jheskethclarkb: okay thanks, sounds good01:25
greghaynesianw: ah, yea thats an annoying case. I cant remember what all the yum element does but I wonder if we cant make it just DTRT depending on distro version?01:25
jheskethclarkb: I'll push up the revert revert and see if I can replicate locally01:26
clarkbjhesketh: kk01:26
fungiokay, pvmove just completed. i'll clean up the sata pv now that it's all running from ssd01:26
greghaynesI'm on my way to airport so only half here btw01:26
*** mohitsharma has quit IRC01:26
openstackgerritJoshua Hesketh proposed openstack-infra/os-loganalyze: Add support for setting filters and views per file  https://review.openstack.org/17332901:27
openstackgerritJoshua Hesketh proposed openstack-infra/os-loganalyze: Allow the config to set the filter and view  https://review.openstack.org/17332801:27
openstackgerritJoshua Hesketh proposed openstack-infra/os-loganalyze: Reapply tidy up generators into contained objects  https://review.openstack.org/18580401:27
ianwgreghaynes: OK, no worries, just didn't want to go doing things that have been rejected before.  i'll start out by keeping the yum element & just trying some s/yum/dnf/01:27
*** annegentle has joined #openstack-infra01:27
*** annegentle has quit IRC01:28
ianwif it gets ridiculous, which i think over time it will, even if not right now, separate elements is probably the way to go01:28
greghaynesianw: yea, were generally fearful of adding complexity to our dependencies, so id do that as the last resort01:28
clarkbwe could just use zypper everywhere and call it good01:29
greghaynesLets do it01:29
fungiokay, nodepool.o.o now has only one cinder volume and it's ssd. the old one is vgreduced, pvremoved, nova volume-detached and cinder deleted01:30
*** ayoung has joined #openstack-infra01:33
*** spzala has quit IRC01:34
greghaynesvroom vroom01:34
*** david-lyle has joined #openstack-infra01:35
fungialso "we moved our server's data from sata to ssd and never took it offline" (openstack: surprising useful!)01:35
*** dahoo has joined #openstack-infra01:35
fungithough really that was all thanks to the magic of lvm2, except the bits where we hotplugged virtual disks into the domu. but cinder++01:36
*** dahoo is now known as jerryz01:37
*** puranamr has joined #openstack-infra01:38
openstackgerritMerged openstack-infra/system-config: Switch to consuming cloud auth from clouds.yaml  https://review.openstack.org/16860201:39
*** zhiwei has joined #openstack-infra01:40
fungilet the broken commence01:41
mordredwoot01:41
*** koolhead17 has quit IRC01:42
*** puranamr has quit IRC01:42
openstackgerritTony Breeds proposed openstack-infra/irc-meetings: Update Chef cookbooks meeting  https://review.openstack.org/18292601:46
openstackgerritTony Breeds proposed openstack-infra/irc-meetings: Change refstack Chair  https://review.openstack.org/18292501:46
openstackgerritTony Breeds proposed openstack-infra/irc-meetings: Remove Storyboard Meeting.  https://review.openstack.org/18580801:46
openstackgerritTony Breeds proposed openstack-infra/irc-meetings: Update Docs team meeting  https://review.openstack.org/18580901:46
openstackgerritTony Breeds proposed openstack-infra/irc-meetings: Update heat meeting  https://review.openstack.org/18581001:46
openstackgerritTony Breeds proposed openstack-infra/irc-meetings: Add Ironic/Neutron meeting  https://review.openstack.org/18581101:46
*** zz_dimtruck is now known as dimtruck01:47
fungimordred: on 185701 would returning "" instead of "None" have worked too? at least the former is a string which also evaluates false01:48
fungior does it also need to be nonempty?01:48
*** annegentle has joined #openstack-infra01:49
*** Sukhdev has quit IRC01:51
*** jyuso has quit IRC01:51
*** yamahata has joined #openstack-infra01:53
*** koolhead17 has joined #openstack-infra02:02
greghaynesDidbwe verify that keystone token timeouts was the nodepool issue?02:03
*** ivar-lazzaro has quit IRC02:03
*** salv-orlando has joined #openstack-infra02:04
greghaynesseems scary to pile the clouds.yaml change on top before weve gotten the other issue fix...02:05
fungion the gerrit 2.10 front, looks like the jgit patch we probably want (d906214) is merged and covered by today's v4.0.0.201505260635-rc2 tag02:05
*** mwhagedorn has quit IRC02:05
greghaynesah, yes, that does look like the patch we want02:07
clarkbgreghaynes we didnt test but know glanceclient wont get new token02:08
greghaynesyea02:08
clarkband we need cloud change for rax02:08
clarkbdoesnt work otherwise. uses glance v102:08
clarkband get rate limited which is lol02:08
greghaynesaye02:09
*** salv-orlando has quit IRC02:09
greghaynes:/02:09
greghaynestheres way too many layers of bugs were working through02:09
greghaynesalthough, the rax thing isbt an issue until we start usibg dib there yes?02:10
*** unicell has quit IRC02:11
*** yamamoto has quit IRC02:12
fungiwhich is trying to be nowish02:13
greghaynesoh, wow02:13
fungibecause... why not?02:13
greghaynesheh02:13
greghayneswell, we were beibg all conservative about the first patch02:14
fungiit's at least for label types we're not actively running jobs on yet02:14
greghaynesI certainly dont mind just going for it though02:14
fungiso if they don't build/upload, meh02:14
clarkbgreghaynes: the first patch was the scary one02:15
clarkbgreghaynes: but now that we are using shade its all much less scary02:15
fungi(for some definitions of scary)02:15
clarkband using cloud does appear to have broken the world02:16
clarkbOpenStackCloudException: Error getting compute endpoint: Error constructing auth plugin: password __init__() got an unexpected keyword argument 'project-id'02:16
greghaynesthat tends to happen when you use clouds02:16
clarkbthat looks like a "you gave me a kwarg I didn't want so here have an exception" exception02:17
greghayneshah, I wonder if project -> tenant rename not happening?02:17
greghayneswe map between those two properties in a few places02:17
clarkbwell looking at where the exception comes from we don't pass a project id so I have no idea02:18
clarkbwe don't pass a tenant either02:18
clarkbbut cloud.yaml has project-id02:18
clarkbgreghaynes: any idea what it should be? tenant?02:19
* clarkb can ninja that change in02:19
clarkband see if it all gets happy02:19
clarkbmaybe its project_id?02:20
*** salv-orlando has joined #openstack-infra02:21
*** taron has quit IRC02:21
clarkband of course it read that config once and now we have to restart nodepool to get it to reread02:21
clarkbmordred: ^ that is a regression btw02:21
greghaynesclarkb: looks like it can be any of project_id, tenant_id, tenant_name, project_name02:22
*** taron has joined #openstack-infra02:22
clarkbya02:23
clarkbso the - was the problem, I think rh1 has similar problems with auth-url02:23
*** ddieterl_ has joined #openstack-infra02:23
greghayneslooks like also project_name is what we should really be calling it02:23
clarkband we need to make it so that its reread02:24
clarkbbut I think thats a shade thing so not sure how to influence from nodepoo02:25
clarkb(we can probably check file mod time)02:25
greghayneshrm02:25
*** salv-orlando has quit IRC02:25
clarkbthat is honestly probably the biggest problem here02:25
greghaynesseems like a good test02:26
*** ddieterly has quit IRC02:27
clarkbgreghaynes: project_name is the least performant option02:27
greghayneshah02:27
clarkbtenant_id the most02:27
clarkb(its a small number of iterations so meh, just pointing that out)02:27
greghaynesso the os-c-c code makes it read like they are all just backwards compat options for project_name02:28
clarkbI will just s/-/_/02:28
greghaynesyea02:28
greghaynesThe amount it matters asymptotically approaches 002:28
clarkbok I have puppet stopped so it won't break my manual edits and puppet change in a moment02:28
openstackgerritClark Boylan proposed openstack-infra/system-config: Use _ instead of - in clouds.yaml  https://review.openstack.org/18582402:29
clarkbfungi: mordred greghaynes ^ tahts manually in place and seems to make nodepool happy02:29
greghaynesclarkb: you checked auth_url has same issue?02:30
asselin_greghaynes, does this stacktrace mean anything to you? http://paste.openstack.org/show/238941/02:30
clarkbgreghaynes: yup, rh1 was failing until I made that change (and restarted nodepool again, grumble grumble)02:31
clarkbactually thinking about this more I wonder if we should just make the nodepool cloud: value mean what oscc value rather than the clouds.yaml value02:31
greghaynesasselin_: We just ran into that error and pushed up a patch to fix02:31
clarkbif we do that then we don't have any of this restarting nonsense02:31
asselin_greghaynes, ok great!02:32
greghaynesasselin_: youll need to grab latest shade02:32
clarkbasselin_: ya you want to upgrade to latest shade02:32
asselin_ok thanks02:32
greghaynesclarkb: huh?02:32
*** david-lyle has quit IRC02:33
greghaynesclarkb: os-c-c just pulls out of clouds.yaml02:33
clarkbgreghaynes: so I have discovered (and fungi helped me grok) that the oscc and nodepool and shade config is all needless confusing and needs a serious overhaul imo02:33
clarkbgreghaynes: nodepool.yaml has a cloud: foo config option02:33
greghaynesyea, that maps to an os-c-c cloud02:33
clarkbgreghaynes: no it doesn't :)02:33
nikeshmhi02:33
greghaynesok, so the terms here suck02:33
clarkbgreghaynes: it instead maps to a key in your clouds.yaml02:33
greghaynesthat maps to a clouds.yaml cloud02:33
greghaynesyea02:34
clarkbgreghaynes: that key then maps to a oscc cloud02:34
clarkbgreghaynes: and this is the problem02:34
nikeshmi am getting this issue on nodepool image-upload http://paste.openstack.org/show/238941/02:34
clarkbgreghaynes: if instead of relying on clouds.yaml at all which fails to reload our config properly when changes are made we kept everything but the static oscc cloud data in nodepool.yaml we could make cloud point to oscc clouds properly02:34
nikeshmthis is my yaml for nodepool http://paste.openstack.org/show/238951/02:34
clarkbgreghaynes: then we stop being confusing and have the benefit of loading our config properly02:34
*** SumitNaiksatam has joined #openstack-infra02:35
greghaynesos-c-c just does config overlay, one of those steps overlays the clouds.yaml config, which cloud in clouds.yaml is indexed by that clouds parameter02:35
clarkbgreghaynes: well its done poorly then02:35
*** dkranz has quit IRC02:35
clarkbgreghaynes: it should be transparent based on name02:35
clarkbnikeshm: yes see the response to asselin_ , you need to upgrade to latest shade02:35
clarkbgreghaynes: because as is it is incredibly confusing and fails to handle configs properly in nodepool02:35
*** gyee has quit IRC02:36
greghaynesI think a lot of this comes down to the use of the word 'cloud' being the worst choice possible02:36
greghaynesthere literally cannot be a more overloaded term here02:36
clarkbgreghaynes: yes, but also the configs are not reloaded properly by shade/oscc02:36
clarkbgreghaynes: I can fix both if we make a single definition of cloud: in nodepool.yaml02:36
greghaynesyea, thats a solvable thing which can be transparent to the user02:36
greghaynesI think I need to see a config or something to grok what youre suggesting02:37
clarkbwhich is load static cloud config data that oscc knows about, then continue to load the non static info from nodepool.yaml02:37
greghayneswhat are the non-static bits?02:37
clarkbgreghaynes: literlaly https://review.openstack.org/#/c/185795/1/modules/openstack_project/templates/nodepool/nodepool.yaml.erb02:37
asselin_nikeshm, sent you the details in -cinder02:37
clarkbgreghaynes: auth url, tenant, password etc02:37
clarkbgreghaynes: those can and do change02:37
clarkbgreghaynes: so put them in the config that is checked and reread and reprocessed if necessary, only read the static cloud: foo bits from oscc02:38
clarkbgreghaynes: which are things like what version of glance do I need t ouse02:38
greghaynesclarkb: yea, so you can do that - it should overlay what you specify in nodepool.yaml ontop of the os-c-c config (which will be backed by clouds.yaml if you specify cloud:)02:38
greghaynesI think theres a test for that functionality too02:38
*** annegentle has quit IRC02:38
clarkbgreghaynes: not really because the way nodepool uses cloud today is to look at cloud.yaml02:39
clarkbgreghaynes: my suggestion is to kill cluod.yaml and delete it02:39
clarkbits redundant and adds no value02:39
clarkball it did was confuse both fungi and I and has regressed our config update handling02:39
greghaynesclarkb: http://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/tests/test_shade_integration.py#n42 is what youre describing I think02:40
clarkbgreghaynes: specifically what I am saying is load cloud foo directly from oscc02:40
*** otter768 has joined #openstack-infra02:40
*** annegentle has joined #openstack-infra02:40
greghaynesclarkb: that is overlaying ontop of the config specified in a clouds.yaml02:40
clarkbgreghaynes: do not involve clouds.yaml at all02:40
clarkbgreghaynes: right but I am telling you clouds.aml is the problem02:40
clarkbgreghaynes: because it does not know how to be reread AND it overloads "cloud" to the point of confusion02:40
clarkbif we just stop using it, then there is no config update problem, and there is no confusion about what "cloud" means02:41
clarkband it will just work02:41
*** claudiub has quit IRC02:41
greghaynesfor nodepool I really am ambivilent on whether it uses clouds.yaml... I dont think it buys us much02:42
greghaynesI do like the clouds.yaml thing in general though02:42
greghaynesso meh02:42
greghaynesYouall already have the composition logic for secrets via puppet02:42
greghaynesso its just not an issue for you02:42
greghaynesAlso, I would like to propose we rename the os-c-c term 'cloud' to be 'profile'02:43
clarkbthat seems sane02:43
*** otter768 has quit IRC02:44
* morganfainberg looks at the clouds.yaml closely.02:48
morganfainberggreghaynes: there is some talk of client config moving. If it does, a rename/update might be easier than doing it in-place.02:49
morganfainberg*shrugs*02:49
greghaynesmorganfainberg: yea, I talked about thatvwith jamie a bunch at summit02:50
morganfainbergYep.02:50
greghaynesmoving os c c that is02:50
morganfainbergI support it moving btw.02:50
greghaynesw00t02:50
morganfainbergBut I am not the final say.02:51
*** fedexo has joined #openstack-infra02:51
* greghaynes plugs ears and goes lalalala02:51
* morganfainberg snerks.02:52
*** gokrokve has joined #openstack-infra02:52
clarkbbut if clouds.yaml is to stay we have to figure out updates to it in shade/oscc02:52
greghaynesyep02:52
greghaynesI dont think itll be too hard of a problem to fix02:53
*** nelsnelson has quit IRC02:53
*** asselin_ has quit IRC02:54
*** nelsnelson has joined #openstack-infra02:54
mordredwhat's the problem?02:55
mordredand not it's not redundant02:55
clarkbmordred we dont reload clouds.yaml02:55
greghaynesmordred: changes to clouds.yaml arent reflected in nodepool until restart02:55
clarkbwhich is a regression02:56
clarkbits redundant for nodepool02:56
clarkbnot for other things02:56
mordredok. we can fix the file reload thing02:57
mordredI dont' think it's redundant because it allows us to use other tools from the same config02:57
mordredbut reverting for now is fine02:57
clarkbwad going to let it run02:57
clarkbalreadu restarted 3 times02:57
mordredkk02:58
*** spzala has joined #openstack-infra02:58
mordredbtw - renaing things in clouds.yaml is a non-starter without a deprecation path02:58
clarkbsee 18582402:58
mordredso renaming cloud to profile would need to have completely working both things - which obviously isn't that hard02:59
greghaynesmordred: just support both namings02:59
clarkbmordred just have an alias like tenant and project02:59
mordredbut it's out in the wild02:59
mordredsure02:59
mordredjust saying02:59
greghaynesyes, deprecate for sure02:59
clarkbsure, but it will help parsing significantly02:59
mordredit's production now, which means we must be very very careful02:59
mordredwanna here a funny?03:03
mordredhear03:03
mordredso - my roku decided to unpair with showtimeanytime tonight03:03
mordredwhich means you have to go back to a browser somewhere and activate03:03
mordredBUT03:03
mordredshowtime has deployed an invalid SSL cert, so firefox is complaining03:03
mordredand one let you do the thing03:03
mordredwon't03:03
*** btully has joined #openstack-infra03:04
clarkbsounds fun03:04
mordredsolution: spin up Windows VM and try with IE - which, as expected, doesn't know how security works03:04
mordredno warning from IE AT ALL about the broken cert03:04
clarkbmordred: I think the confusion is clouds.yaml has a list of clouds under clouds:03:04
*** fedexo has quit IRC03:05
clarkbmordred: then inside each cloud we specify a cloud03:05
greghaynesI heard you like clouds03:05
clarkband that cloud is different than the cloud we are describing under clouds: and it just gets the brain confused03:05
mordredagree - I never could come up with a better name for that than "vendor" or something03:05
mordredexcept for people using private clouds03:05
mordredit might not be a vendor - it might just be a configured cloud03:05
StevenKprovider ?03:05
mordredso "profile" works fine03:05
greghaynesThey are config profiles from the clients perspective03:05
mordredor provider03:05
mordredyah03:05
StevenKThat might an overloaded term, though03:05
mordredI like profile03:05
clarkbI think one of my nodepool restarts may have made rax ubuntu-trusty uploads unhappy, going to try deleting one of them to see if it starts again happy03:06
mordredalso - I agree - we should s/-/_/ everywhere03:06
*** gokrokve has quit IRC03:06
mordredI _thought_ we already were03:06
mordredmaybe we're not doing that transform on the auth dit?03:06
mordreddict?03:06
clarkbmordred: definitely aren't with latest sahde and latest oscc03:06
mordredkk03:07
*** gokrokve has joined #openstack-infra03:07
clarkbmordred: I can get tracebacks from nodepool logs if you want them03:07
mordredwe can fix that03:07
mordrednah03:07
mordredif we're not - it's certainly not hard to :)03:07
mordredwe process all the dicts and whatnot - so doing a key transform is cake03:07
mordredbtw - for hpcloud it shold be project_name not project_id03:08
*** btully has quit IRC03:08
mordredand looks like it for tripleo as well03:08
*** SumitNaiksatam has quit IRC03:08
clarkbmordred: those are aliases03:08
clarkbmordred: so it doesn't matter and project_name is the least best option according to oscc code03:09
*** SumitNaiksatam has joined #openstack-infra03:09
*** dpaterson has quit IRC03:09
clarkb(its listed last so you have to check all of them before you get to that one)03:09
mordredit should be the most preferred03:09
mordredreally?03:09
clarkbyes I can get a link to the line, one sec03:09
mordredhrm. well, ANYTHING with tenant is broken and evil and shoudl die in a fire03:09
greghayneseh, all the otger options are listed as backwards compat options03:09
greghaynesbut yes, you will loop over them first03:10
mordredbetween project_id and project_name, I believe you can pass a project_id to the project_name  param but you cannot pass a name to the id param03:10
mordredthey will all wind up in project_name at the end of the day03:10
clarkbhttps://git.openstack.org/cgit/openstack/os-client-config/tree/os_client_config/config.py#n23503:10
mordredmorganfainberg: ^^ remind me to punch someone about this, btw03:10
mordred"so that project_name is the ultimate winner"03:10
clarkbmordred: so oscc should change the order if project_name is preferred with tenant* being least preferred03:10
mordredit is03:11
mordredit doesn't bail on a match03:11
* morganfainberg reads backscroll03:11
mordredit keeps looping03:11
mordredand will overwrite anything it finds03:11
greghaynesI think clarkb is uber optimizing03:11
clarkbmordred: wat03:11
*** gokrokve has quit IRC03:11
mordredso project_name being checked last makes it most preferred03:11
mordredwe do it that way03:11
clarkbthats but haivng >1 doesn't make sense03:11
*** david-lyle has joined #openstack-infra03:11
clarkbyou can only have one03:11
mordredrather than searching and bailing03:11
mordredbecause we want to delete anything that is less preferred from the dict so that we only have one answer03:12
*** spzala has quit IRC03:12
mordredthe job of this library isn't to yell at you - it's to do its best to work around the fact that openstack's story on project and tenant has been broken for years03:12
clarkbthis is the sort of thing we should keep a link to for whenever people want to change rest apis03:12
morganfainbergOh god. The name vs ID thing?03:12
mordredand it's ENTIRELY expectable that someone might have both set in an env var03:12
clarkbmorganfainberg: and tenant vs project03:12
morganfainbergThat has annoyed me a ton.03:13
clarkbso we have broken this on two axies03:13
clarkbwhich is O_O03:13
morganfainbergclarkb: if I could make project never had happened. I would have.03:13
mordredso, since someone could now set OS_PROJECT_NAME and OS_PROJECT_ID in env vars means we have to do the crazy above03:13
mordredI do apologize03:13
morganfainbergTenant was a much more descriptive / better term.03:13
mordredit COULD do with a better comment03:13
mordredmorganfainberg: tenant makes no sense to me, btw03:13
clarkbmordred: and OS_TENANT_ID and OS_TENANT_NAME03:13
mordredmorganfainberg: I think NEITHER make any sense, tbh03:13
mordredclarkb: yah03:13
mordredclarkb: a person could set all four and expect the world to work03:13
morganfainbergmordred: better because of less overload. But domain should have been realm. Anyway.03:14
greghaynesyou know what makes the least sense though - 4 options03:14
mordredgreghaynes: yah03:14
morganfainbergThe id to name transform is just weird.03:14
mordredmorganfainberg: TELL ME ABOUT IT03:14
clarkbanyways, I don't want to change id to name because that will require another nodepool restart03:14
mordredyah. sure03:14
clarkbso my vote is stick with simple s/-/_/03:14
mordredit's not a problem - at _some_ point we may want to do that just to clean up our file so that it's better correct documentation for other people03:15
mordredtotally not urgent03:15
morganfainbergclarkb: please stick with simple! :)03:15
clarkbmordred: it might be good for oscc to document the "correct" thing and then "this is acceptable for raisins"03:15
mordred++03:15
mordredclarkb: I'm taking notes on a few different things we need better docs on03:15
mordredthis, and also the dogpile key creation03:15
*** tiswanso has quit IRC03:16
* morganfainberg renames03:16
morganfainbergProjects to "cloudythings"03:16
mordredclarkb: for noticing the restarts ... that probably wants to be a nodepool thing, but perhaps getting info about what file to watch for changes from shade?03:16
clarkbwe should just use uuids for everything then ship a lookup table book03:16
mordredclarkb: don't say that outloud03:17
mordredclarkb: someone will think you're serious03:17
mordredclarkb: youknow, because cloud03:17
morganfainbergclarkb: can we use a pdf format for that. That is all in comic sans.03:17
clarkbmordred: but then we can print and bind some really awesome uuids mapping books03:17
mordredclarkb: with LaTeX03:17
clarkbmorganfainberg: no I want hardcover textbook size book03:17
mordredI know someone who does book binding :)03:17
clarkbso that you can't just copy paste uuids03:17
clarkbyou have to type them in by hand or figure out ocr03:17
morganfainbergOh crap. Wait. Did irccloud just change "comic sans" in line on my phone. *snicker*03:18
mordredclarkb: ok - I have notes on the two bugs and the two docs update we need03:18
morganfainbergyes it did. Ahahhahahahah. Someone has a sense of humor.03:18
mordredclarkb: any other issues I might have missed in scrollback to tackle in the morning?03:18
clarkbmordred: we will have to wait and see how the rax-ord ubuntu-trusty upload goes03:18
clarkbmordred: its the one I restarted for fear that a nodepool restart got the initial uploads into a funny state03:19
mordredk03:19
clarkbit thinks its still "building"03:19
mordredbut so far nothing vomited like the -id did03:19
clarkbnope its running without tracebacks as far as I can see03:19
mordredossum03:19
*** rossella_s has quit IRC03:19
mordredwell, the other 4 are easy to do in the morning03:19
mordredwell, 3 of them are03:19
mordredI'm going to need a combination of SpamapS and morganfainberg to write the docstrings about why we're doing the dogpile key stuff03:19
clarkbmordred: for reloads I think oscc/shade need to expose a flag that says "my config is updated" then the application consuming the libs can make a decision on whether or not to handle that03:19
*** rossella_s has joined #openstack-infra03:20
mordredclarkb: awesome. I will do that03:20
clarkbmordred: then you do if shade.config_updated: nodepool handle application logic here03:20
*** kaisers has quit IRC03:20
mordredyup03:20
clarkbmordred: probably need to make it a little smarter than that so we don't reconfigure the whole world03:20
morganfainbergmordred: yes. I will help doc that stuff.03:20
clarkbmordred: but thats the general idea of what I think makes this better for nodepool03:20
mordredmorganfainberg: tl;dr - clarkb looked at the code and had a WTF why are we doing all of this? earlier today03:20
*** salv-orlando has joined #openstack-infra03:21
mordredclarkb: I'm going to start with dumb "my config changed"03:21
*** panda has quit IRC03:21
clarkbmordred: speaking of, I did eventually go digging into the dogpile docs and the default make a key function is undocumented :/03:21
mordredclarkb: yah - this is why SpamapS and morganfainberg have to write the shade doc on the subject03:21
clarkbmordred: so not only does it apparently not do the right thing, I have no idea what it actually does03:21
mordredclarkb: only morganfainberg and SpamapS do03:21
mordred(understand what upstream does)03:21
*** panda has joined #openstack-infra03:21
morganfainbergmordred: and zzzeeekk03:21
*** samueldmq has quit IRC03:21
mordredyah. him too03:21
morganfainbergIf I was less ptl-ing I have like 10+ backlogs fixes for dogpile.03:22
morganfainbergThis being related to one of them.03:22
*** fawadkhaliq has joined #openstack-infra03:22
mordredok. I'm going back to the TVs03:22
morganfainbergI will help document at the very least *plans to corner SpamapS to work on doc things soon*03:23
clarkbbut I have learned a lot about dogpile (that was fun) and a lot about oscc (which is basically shade lite) and hopefully won't be learning everything there is to know about rax's glance task image upload process tomorrow03:23
clarkbmordred: rax-ord upload just failed, trying to find a traceback now03:24
clarkbmordred: ClientException: Object POST failed returned a 504 Gateway Time-out03:27
clarkbwhen uploading to swift03:27
*** yamamoto has joined #openstack-infra03:27
greghaynesclarkb: nice!03:27
*** salv-orl_ has joined #openstack-infra03:28
clarkbso thats progress I guess03:28
clarkbnow time for me to play these new wiiu games03:29
clarkb(nodepool should be fine with that being the only issue on an unused image)03:29
* greghaynes -> airborne03:29
*** salv-orlando has quit IRC03:29
*** salv-orl_ has quit IRC03:32
*** annegentle has quit IRC03:32
*** changbl has joined #openstack-infra03:32
*** jerryz has quit IRC03:32
*** bhunter71 has quit IRC03:32
*** koolhead17 has quit IRC03:33
*** stevemar has joined #openstack-infra03:33
nikeshmclarkb: hi i tried as rasselin said,but i am not getting any /opt/shade03:35
nikeshmand still same error03:35
*** salv-orlando has joined #openstack-infra03:36
cody-somerville"Currently ansible only supports HTTP proxies" <-- seriously?03:37
*** annegentle has joined #openstack-infra03:37
*** dalgaaf has quit IRC03:38
clarkbnikeshm pip install shade==0.6.3 should work03:40
clarkbansible only supports http proxies for what particular functionality?03:40
clarkbansible istelf uses ssh/fireball03:41
*** salv-orlando has quit IRC03:41
*** salv-orl_ has joined #openstack-infra03:43
*** salv-orl_ has quit IRC03:43
nikeshmclarkb: shade==0.6.1 is already there,is 0.6.3 is required03:44
clarkbyes you need the latest version03:46
nikeshmso again do we have to run install_master,sh03:47
*** sarob has joined #openstack-infra03:49
*** camunoz has quit IRC03:49
jheskethclarkb: hmm, so I can't reproduce the problem with os-loganalyze... I've deployed 185804 on an instance and it serves up content just fine from disk and swift03:50
jheskethwell the simple console.log I was testing03:50
jheskethbut I don't have much more to go on in terms of what broke03:50
clarkbnikeshm no just run the pip install command I gave you03:51
*** emagana has joined #openstack-infra03:51
*** sarob has quit IRC03:53
*** sarob has joined #openstack-infra03:53
*** armax has quit IRC03:54
*** Sukhdev has joined #openstack-infra03:58
*** yfried has joined #openstack-infra04:00
*** jerryz has joined #openstack-infra04:01
*** achanda has joined #openstack-infra04:02
*** krtaylor has joined #openstack-infra04:03
*** gnuoy has quit IRC04:05
*** jamespage has quit IRC04:05
*** gnuoy has joined #openstack-infra04:06
*** jamespage has joined #openstack-infra04:06
*** dbyte has joined #openstack-infra04:10
*** dimtruck is now known as zz_dimtruck04:15
*** links has joined #openstack-infra04:15
nikeshmclarkb: thanks, is this nodepool yaml file ok http://paste.openstack.org/show/238951/  i want to use same image on two cloud-provider04:17
*** ddieterl_ has quit IRC04:18
nikeshmclarkb: one cloud provider i will use for FC drivers tempest and another for iSCSI driver tempest04:18
nikeshmjob04:18
*** xyang1 has quit IRC04:20
mordredcody-somerville: what other sorts of proxies do you need?04:22
mordredcody-somerville: also - ansible knows how to handle ssh jump hosts too04:22
mordredclarkb: just on ord? or on iad and dfw too? I think I've seen issues with ORD swift in the past too04:23
*** annegentle has quit IRC04:26
*** maishsk_afk has joined #openstack-infra04:31
*** maishsk_afk is now known as maishsk04:31
*** markvoelker has quit IRC04:31
*** maishsk has quit IRC04:33
*** sarob has quit IRC04:34
*** yamamoto has quit IRC04:34
*** unicell has joined #openstack-infra04:34
*** puranamr has joined #openstack-infra04:34
*** amotoki_ has joined #openstack-infra04:35
*** sks has joined #openstack-infra04:39
*** yfried has quit IRC04:40
*** otter768 has joined #openstack-infra04:41
*** jerryz has quit IRC04:44
*** otter768 has quit IRC04:45
*** abregman has joined #openstack-infra04:48
nikeshmclarkb: thanks04:49
nikeshmthats issue solved04:49
*** btully has joined #openstack-infra04:52
*** sputnik13 has joined #openstack-infra04:55
*** gokrokve has joined #openstack-infra04:56
*** harlowja_at_home has joined #openstack-infra04:57
*** btully has quit IRC04:57
*** sputnik13 has quit IRC04:58
*** sputnik13 has joined #openstack-infra04:59
*** sputnik13 has quit IRC04:59
*** ParsectiX has quit IRC05:13
*** gokrokve has quit IRC05:14
*** btully has joined #openstack-infra05:15
*** salv-orlando has joined #openstack-infra05:16
*** sdake has joined #openstack-infra05:18
*** leopoldj has joined #openstack-infra05:18
*** ddieterly has joined #openstack-infra05:18
*** achanda has quit IRC05:19
*** salv-orlando has quit IRC05:21
*** ddieterly has quit IRC05:23
*** harlowja_at_home has quit IRC05:24
*** deepakcs has joined #openstack-infra05:28
*** cloudm2 has quit IRC05:32
*** Sukhdev has quit IRC05:34
*** yamamoto has joined #openstack-infra05:34
*** fawadk has joined #openstack-infra05:37
*** fawadk has joined #openstack-infra05:37
*** fawadk is now known as fawadkhaliq_05:37
*** fawadkhaliq has quit IRC05:39
*** sdake has quit IRC05:40
*** salv-orlando has joined #openstack-infra05:44
*** salv-orl_ has joined #openstack-infra05:47
*** yamamoto has quit IRC05:47
*** salv-orlando has quit IRC05:48
*** Ala has joined #openstack-infra05:51
*** achanda has joined #openstack-infra05:52
*** puranamr has quit IRC05:52
*** rlandy has joined #openstack-infra05:54
*** salv-or__ has joined #openstack-infra05:55
*** salv-orl_ has quit IRC05:55
*** salv-orl_ has joined #openstack-infra05:57
*** achanda_ has joined #openstack-infra05:58
*** achanda has quit IRC05:59
*** salv-or__ has quit IRC06:00
*** emagana has quit IRC06:00
*** koolhead17 has joined #openstack-infra06:01
*** emagana has joined #openstack-infra06:01
*** koolhead17 has quit IRC06:01
*** koolhead17 has joined #openstack-infra06:01
*** kaisers has joined #openstack-infra06:01
*** e0ne has joined #openstack-infra06:01
*** salv-orlando has joined #openstack-infra06:01
*** salv-orl_ has quit IRC06:02
*** kaisers has quit IRC06:02
*** yfried has joined #openstack-infra06:03
*** fawadkhaliq has joined #openstack-infra06:03
*** flashgordon is now known as jogo06:04
*** yamamoto has joined #openstack-infra06:04
*** jerryz has joined #openstack-infra06:05
*** emagana has quit IRC06:05
*** fawadkhaliq_ has quit IRC06:05
*** salv-orlando has quit IRC06:06
*** fawadk has joined #openstack-infra06:06
*** fijal has quit IRC06:06
*** kaisers has joined #openstack-infra06:06
*** salv-orlando has joined #openstack-infra06:06
*** BharatK has joined #openstack-infra06:08
*** fawadkhaliq has quit IRC06:08
*** unicell has quit IRC06:09
*** unicell has joined #openstack-infra06:09
*** e0ne has quit IRC06:11
*** fijal has joined #openstack-infra06:12
*** yamamoto has quit IRC06:14
*** unicell1 has joined #openstack-infra06:14
*** achanda_ has quit IRC06:15
*** unicell has quit IRC06:16
*** stevebaker has quit IRC06:16
*** stevebaker has joined #openstack-infra06:16
*** ildikov has quit IRC06:18
*** SumitNaiksatam has quit IRC06:19
*** hdd has joined #openstack-infra06:19
*** ddieterly has joined #openstack-infra06:19
*** SumitNaiksatam has joined #openstack-infra06:20
*** salv-orlando has quit IRC06:21
*** ayoung has quit IRC06:22
*** ddieterly has quit IRC06:24
*** salv-orl_ has joined #openstack-infra06:28
*** stevebaker has quit IRC06:30
*** kaisers has quit IRC06:30
*** AzherKhan has joined #openstack-infra06:30
*** salv-orlando has joined #openstack-infra06:31
*** stevebaker has joined #openstack-infra06:33
*** salv-or__ has joined #openstack-infra06:33
*** salv-orl_ has quit IRC06:33
*** shardy has joined #openstack-infra06:35
*** salv-orlando has quit IRC06:35
flip214Hi. I've got a devstack plugin, but check-tempest-dsvm-full-drbd-devstack says "Cloning not allowed in this configuration". what am I doing wrong?06:36
*** hdd has quit IRC06:36
*** salv-or__ has quit IRC06:37
*** ZZelle_ has joined #openstack-infra06:39
*** maishsk_afk has joined #openstack-infra06:39
*** oomichi has joined #openstack-infra06:41
*** otter768 has joined #openstack-infra06:42
*** AzherKhan has quit IRC06:42
*** soren_ has joined #openstack-infra06:44
*** AzherKhan has joined #openstack-infra06:44
*** otter768 has quit IRC06:46
*** yfried is now known as yfried|afk06:48
*** salv-orlando has joined #openstack-infra06:49
*** yamamoto has joined #openstack-infra06:50
*** fawadkhaliq has joined #openstack-infra06:51
*** salv-orl_ has joined #openstack-infra06:53
*** oomichi has quit IRC06:53
*** salv-orlando has quit IRC06:53
*** fawadk has quit IRC06:54
*** yfried|afk is now known as yfried06:56
*** koolhead17 has quit IRC06:56
*** salv-orl_ has quit IRC06:58
*** salv-orlando has joined #openstack-infra06:58
*** mpaolino has joined #openstack-infra06:59
*** salv-orl_ has joined #openstack-infra07:00
*** achanda has joined #openstack-infra07:02
*** salv-orlando has quit IRC07:03
*** mpaolino has quit IRC07:04
*** claudiub has joined #openstack-infra07:04
*** salv-orl_ has quit IRC07:05
*** dbyte has quit IRC07:06
*** dbyte has joined #openstack-infra07:07
*** dbyte has quit IRC07:07
*** dbite has quit IRC07:07
*** dguitarbite has quit IRC07:07
*** dguitarbite has joined #openstack-infra07:07
*** yolanda has quit IRC07:10
*** mpavone has joined #openstack-infra07:11
*** fijal_ has joined #openstack-infra07:11
*** k4n0 has joined #openstack-infra07:11
*** fijal has quit IRC07:14
*** AzherKhan has quit IRC07:15
*** markvoelker has joined #openstack-infra07:17
openstackgerritSergey Kraynev proposed openstack-infra/project-config: Enable Heat services for functional tests  https://review.openstack.org/18297407:18
*** stevemar has quit IRC07:20
*** ddieterly has joined #openstack-infra07:20
*** panda has quit IRC07:21
*** panda has joined #openstack-infra07:22
*** fijal_ is now known as fijal07:22
*** jtomasek has quit IRC07:23
*** salv-orlando has joined #openstack-infra07:23
*** AzherKhan has joined #openstack-infra07:23
*** stevebaker has quit IRC07:24
*** stevebaker has joined #openstack-infra07:24
*** ddieterly has quit IRC07:25
*** emagana has joined #openstack-infra07:25
*** annegentle has joined #openstack-infra07:27
*** salv-orlando has quit IRC07:27
*** salv-orlando has joined #openstack-infra07:27
*** emagana has quit IRC07:30
*** salv-orlando has quit IRC07:31
*** sshnaidm has joined #openstack-infra07:31
*** annegentle has quit IRC07:32
*** salv-orlando has joined #openstack-infra07:32
*** markus_z has joined #openstack-infra07:32
*** arxcruz has joined #openstack-infra07:33
*** pasquier-s has joined #openstack-infra07:34
*** salv-orl_ has joined #openstack-infra07:35
*** salv-orlando has quit IRC07:36
*** achanda has quit IRC07:36
*** Hal has joined #openstack-infra07:37
*** dtantsur|afk is now known as dtantsur07:37
*** Hal is now known as Guest3618207:38
*** achanda has joined #openstack-infra07:38
*** dims has joined #openstack-infra07:38
*** dguerri`away is now known as dguerri07:40
*** jlanoux has joined #openstack-infra07:42
*** dims has quit IRC07:44
*** chlong has quit IRC07:45
*** achanda has quit IRC07:47
*** dalgaaf has joined #openstack-infra07:48
*** dizquierdo has joined #openstack-infra07:49
*** fawadkhaliq has quit IRC07:50
lxslihttps://review.openstack.org/185620 https://review.openstack.org/18563207:51
lxsli^-- couple of elastic recheck queries07:51
*** jistr has joined #openstack-infra07:52
*** devvesa has joined #openstack-infra07:56
*** achanda has joined #openstack-infra08:00
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Add an acceptance test with beaker for puppet-zuul  https://review.openstack.org/18021408:03
*** emagana has joined #openstack-infra08:04
*** matbu_afk has quit IRC08:05
*** AzherKhan has quit IRC08:06
*** emagana has quit IRC08:09
*** matbu has joined #openstack-infra08:10
*** HeOS has quit IRC08:11
*** matbu has quit IRC08:11
*** matbu has joined #openstack-infra08:14
*** e0ne has joined #openstack-infra08:15
*** fijal has quit IRC08:15
BobBallI have a very stupid question... How is the pypi mirror setup for nodepool nodes?  I see the configure_mirror script but that only seems to be called by multinode_setup which I'm definitely not calling... Yet my pypi mirrors are trying to use the openstack.org ones which is wrong08:15
*** e0ne has quit IRC08:19
*** ddieterly has joined #openstack-infra08:21
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Add an acceptance test with beaker for puppet-zuul  https://review.openstack.org/18021408:22
*** ildikov has joined #openstack-infra08:22
*** btully has quit IRC08:25
*** ddieterly has quit IRC08:25
*** ZZelle has quit IRC08:27
*** ZZelle has joined #openstack-infra08:27
*** mpavone has quit IRC08:31
maishsk_afkQ: is it possible to get a list of all etherpads under https://etherpad.openstack.org ?08:34
*** maishsk_afk is now known as maishsk08:34
*** heyongli has joined #openstack-infra08:34
*** fijal has joined #openstack-infra08:35
*** mpavone has joined #openstack-infra08:37
*** markus_z has quit IRC08:38
*** otter768 has joined #openstack-infra08:43
*** jerryz has quit IRC08:44
*** otter768 has quit IRC08:47
*** markus_z has joined #openstack-infra08:51
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Fix wrong indent reported by recent puppet-lint  https://review.openstack.org/18588908:52
*** fhubik has joined #openstack-infra08:53
*** mrmartin has joined #openstack-infra08:53
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Add an acceptance test with beaker for puppet-zuul  https://review.openstack.org/18021408:54
*** u_glide has joined #openstack-infra08:54
*** shardy_ has joined #openstack-infra08:55
*** shardy has quit IRC08:57
*** achanda has quit IRC08:58
*** ihrachyshka has joined #openstack-infra08:58
*** emagana has joined #openstack-infra08:59
GheRiveromorning all08:59
*** lifeless_ has joined #openstack-infra09:00
*** lifeless has quit IRC09:00
*** shardy_ has quit IRC09:01
*** dizquierdo has quit IRC09:01
*** shardy has joined #openstack-infra09:01
*** emagana has quit IRC09:03
*** mpavone has quit IRC09:04
openstackgerritJoshua Hesketh proposed stackforge/gertty: Only enqueue a task if its not already there  https://review.openstack.org/18589009:06
openstackgerritJoshua Hesketh proposed stackforge/gertty: Perform multiple sync tasks asynchronously  https://review.openstack.org/18589109:06
*** freerunner has joined #openstack-infra09:07
*** teran has quit IRC09:09
*** fhubik is now known as fhubik_afk09:12
*** HeOS has joined #openstack-infra09:14
electricalMorning GheRivero09:14
*** amotoki_ has quit IRC09:14
*** freerunner has quit IRC09:16
*** ekarlso has quit IRC09:16
*** rossella_s has quit IRC09:19
*** rossella_s has joined #openstack-infra09:19
*** ddieterly has joined #openstack-infra09:22
*** ekarlso has joined #openstack-infra09:22
*** e0ne has joined #openstack-infra09:23
*** amotoki_ has joined #openstack-infra09:23
*** e0ne is now known as e0ne_09:23
*** ddieterly has quit IRC09:26
*** jtomasek has joined #openstack-infra09:27
*** shardy_ has joined #openstack-infra09:28
*** hichihara has quit IRC09:28
*** shardy has quit IRC09:30
*** e0ne_ is now known as e0ne09:31
*** fhubik_afk is now known as fhubik09:33
*** mrmartin has quit IRC09:33
*** shardy_ has quit IRC09:34
*** shardy has joined #openstack-infra09:34
openstackgerritDavid TARDIVEL proposed openstack-infra/project-config: Create new stackforge project python-watcherclient  https://review.openstack.org/18500609:34
*** dosaboy_ has quit IRC09:36
*** dosaboy has joined #openstack-infra09:36
*** derekh has joined #openstack-infra09:37
*** amotoki_ has quit IRC09:39
*** kaisers has joined #openstack-infra09:39
*** mattfarina has joined #openstack-infra09:41
*** mattfarina has quit IRC09:42
*** mpavone has joined #openstack-infra09:42
u_glideHello all! Could anyone help me with strange ECONNREFUSED error in gate?09:43
openstackgerritMerged openstack-infra/irc-meetings: updated swift meeting time  https://review.openstack.org/18544709:43
u_glide#link http://logs.openstack.org/78/184778/7/check/gate-manila-tempest-dsvm-neutron/d7cea82/logs/screen-m-shr.txt.gz09:44
*** dizquierdo has joined #openstack-infra09:44
u_glidelooks like apache refuses connection to keystone09:45
u_glideoh, I figured out problem :)09:47
*** nikeshm has quit IRC09:48
*** Guest36182 has quit IRC09:51
*** emagana has joined #openstack-infra09:53
*** Hal has joined #openstack-infra09:56
*** Hal is now known as Guest5985809:56
*** emagana has quit IRC09:57
openstackgerritMerged openstack-infra/irc-meetings: Remove Storyboard Meeting.  https://review.openstack.org/18580809:58
openstackgerritMerged openstack-infra/irc-meetings: Update Docs team meeting  https://review.openstack.org/18580909:58
openstackgerritMerged openstack-infra/irc-meetings: Update Chef cookbooks meeting  https://review.openstack.org/18292609:59
*** maishsk has quit IRC10:00
openstackgerritThierry Carrez proposed openstack-infra/irc-meetings: Change refstack Chair  https://review.openstack.org/18292510:03
*** maishsk_afk has joined #openstack-infra10:04
openstackgerritThierry Carrez proposed openstack-infra/irc-meetings: Update heat meeting  https://review.openstack.org/18581010:04
*** freerunner has joined #openstack-infra10:04
openstackgerritThierry Carrez proposed openstack-infra/irc-meetings: Add Ironic/Neutron meeting  https://review.openstack.org/18581110:05
openstackgerritMerged openstack-infra/irc-meetings: Change refstack Chair  https://review.openstack.org/18292510:07
openstackgerritMerged openstack-infra/irc-meetings: Update heat meeting  https://review.openstack.org/18581010:07
*** teran has joined #openstack-infra10:08
openstackgerritMerged openstack-infra/irc-meetings: Add Ironic/Neutron meeting  https://review.openstack.org/18581110:09
*** dims has joined #openstack-infra10:09
electricalSome one is having a merge party :p10:09
*** achanda has joined #openstack-infra10:09
*** mfmcdonagh has joined #openstack-infra10:09
*** yamamoto has quit IRC10:10
*** teran_ has joined #openstack-infra10:11
*** teran has quit IRC10:13
*** fhubik has quit IRC10:14
*** pblaho has quit IRC10:14
*** fhubik has joined #openstack-infra10:14
*** pblaho has joined #openstack-infra10:15
*** linuxmaniac has quit IRC10:21
*** ddieterly has joined #openstack-infra10:23
*** ddieterly has quit IRC10:27
*** _nadya_ has joined #openstack-infra10:33
*** linuxmaniac has joined #openstack-infra10:36
*** matbu has quit IRC10:38
*** matbu has joined #openstack-infra10:39
*** fhubik is now known as fhubik_afk10:42
*** akrivoka has joined #openstack-infra10:43
*** otter768 has joined #openstack-infra10:44
*** yfried is now known as yfried|afk10:44
*** ParsectiX has joined #openstack-infra10:45
*** zhiwei has quit IRC10:45
*** yfried|afk is now known as yfried10:45
*** emagana has joined #openstack-infra10:47
*** emagana has quit IRC10:48
*** otter768 has quit IRC10:48
*** emagana has joined #openstack-infra10:48
*** pc_m has joined #openstack-infra10:50
akrivokawe seem to have a problem in with stable/kilo horizon patches10:53
*** emagana has quit IRC10:53
akrivokahttps://review.openstack.org/#/q/status:open+project:openstack/horizon+branch:stable/kilo,n,z10:53
akrivokagrenade is failing for all of them in the same way, unrelated to the code in the patches10:54
akrivokaI think it is a package version conflict http://logs.openstack.org/56/183656/4/check/check-grenade-dsvm/3acba73/logs/grenade.sh.txt.gz#_2015-05-27_07_15_12_72610:55
akrivokabut I don't know how to fix that issue - any help would be appreciated10:55
*** samueldmq has joined #openstack-infra11:00
samueldmqmorning11:00
*** ildikov has quit IRC11:01
jklarehi, does anybody here know if a project renaming / move has to happen if one is currently under stackforge/.. and has become part of the big tent ?11:01
*** weshay has joined #openstack-infra11:01
*** mrmartin has joined #openstack-infra11:04
*** freerunner has quit IRC11:10
*** freerunner has joined #openstack-infra11:12
*** freerunner has quit IRC11:15
*** ildikov has joined #openstack-infra11:16
*** dguitarbite has quit IRC11:18
*** achanda has quit IRC11:20
*** ddieterly has joined #openstack-infra11:23
*** achanda has joined #openstack-infra11:25
*** e0ne is now known as e0ne_11:26
*** ddieterly has quit IRC11:29
*** openstack has joined #openstack-infra11:39
*** sshnaidm has quit IRC11:39
*** emagana has joined #openstack-infra11:43
mancdazhi peeps - any chance we could get some eyes on this review: https://review.openstack.org/#/c/153057/11:44
mancdazit's been kicking around for quite a while11:44
*** emagana has quit IRC11:48
*** dprince has joined #openstack-infra11:52
*** sshnaidm has joined #openstack-infra11:55
*** yfried is now known as yfried|afk11:56
*** deepakcs has quit IRC11:56
*** yfried|afk is now known as yfried11:58
*** yamamoto has joined #openstack-infra11:59
*** baoli has joined #openstack-infra12:01
*** BharatK has quit IRC12:01
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Add an acceptance test with beaker for puppet-zuul  https://review.openstack.org/18021412:03
*** yamamoto has quit IRC12:04
*** markvoelker has quit IRC12:04
*** markvoelker has joined #openstack-infra12:04
*** freerunner has joined #openstack-infra12:05
*** ildikov has quit IRC12:07
*** e0ne is now known as e0ne_12:09
*** e0ne_ is now known as e0ne12:14
*** aysyanne has joined #openstack-infra12:14
*** BharatK has joined #openstack-infra12:15
openstackgerritFabien Boucher proposed openstack-infra/project-config: Add beaker-rspec jobs to puppet-zuul repo  https://review.openstack.org/18595212:16
*** bswartz has quit IRC12:17
*** amotoki_ has joined #openstack-infra12:19
*** BharatK has quit IRC12:19
*** achanda has quit IRC12:20
*** ildikov has joined #openstack-infra12:21
*** ddieterly has joined #openstack-infra12:24
*** ddieterly has quit IRC12:29
*** yolanda has joined #openstack-infra12:29
yolandahi, i have an user that created a project with a non-accessible branch, and now he cannot land any change due to Remote %s did not have any references12:30
yolandahow can i fix that? (not deleting the project)12:30
*** mwagner_lap has joined #openstack-infra12:31
*** dims has quit IRC12:31
*** mwagner_lap is now known as mwagner12:31
*** miqui has joined #openstack-infra12:32
*** olaph has joined #openstack-infra12:32
*** dims has joined #openstack-infra12:32
*** ParsectiX has quit IRC12:33
*** rbradfor_ has quit IRC12:34
*** rbradfor_ has joined #openstack-infra12:35
*** emagana has joined #openstack-infra12:37
yolandafungi, you know? ^12:39
*** akshai has joined #openstack-infra12:40
*** eharney has quit IRC12:40
electricalyolanda: is that in gerrit? can't change the branch name?12:40
yolandaelectrical, yes , it's happening on gerrit12:40
*** mrmartin has quit IRC12:40
yolandamaster branch12:41
yolandajust any change can land due to that, for that project12:41
*** emagana has quit IRC12:41
electricalyolanda: hm okay. and you can't change it in gerrit any more then after creating the project?12:41
*** annegentle has joined #openstack-infra12:41
yolandaif i create a new branch, it tries to point to HEAD and shows invalid revision12:42
yolandasomething went really wrong in the creation of that project, i'm just thinking in removing12:42
electricalAh okay, i see. hmm. good question.12:42
*** mrunge has joined #openstack-infra12:44
*** mrunge_ has joined #openstack-infra12:44
*** otter768 has joined #openstack-infra12:44
electricalso the master branch doesn't exist anymore right? i think it means that in the .git/config file there is a reference to the master branch. you could edit that file and change it. but not sure if that's possible with gerrit :-(12:44
*** mrunge has quit IRC12:45
*** mrunge_ has quit IRC12:45
electricalmust admit i only used gerrit so far as a user ( for reviews )12:45
*** kaisers has quit IRC12:45
*** mrunge has joined #openstack-infra12:45
TheJuliayolanda: It kind of sounds like their git repo is in a very bad state12:46
yolandaso yes, very bad state12:47
*** fijal has quit IRC12:47
yolandai removed the project and running manage-projects again12:48
*** julim has joined #openstack-infra12:48
*** otter768 has quit IRC12:49
*** prad has joined #openstack-infra12:49
*** fijal has joined #openstack-infra12:50
*** marcusvrn has joined #openstack-infra12:52
*** ekarlso has quit IRC12:53
*** ekarlso has joined #openstack-infra12:53
*** xyang1 has joined #openstack-infra12:56
*** sks has quit IRC12:56
*** gordc has joined #openstack-infra12:58
electricalyolanda: stupid computers ;-)12:59
*** fawadkhaliq has joined #openstack-infra13:00
yolandaelectrical , there are these days when nothing wants to work...13:01
electricalknow that feeling yeah13:01
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Consume os-client-config directly  https://review.openstack.org/18597213:02
*** gaelL has quit IRC13:02
*** mwhagedorn has joined #openstack-infra13:02
mordredfungi, clarkb: ^^ that should take care of the "notice when the config has changed without a restart" thing13:04
*** ayoung has joined #openstack-infra13:04
*** dboik has joined #openstack-infra13:05
*** annegentle has quit IRC13:06
*** bknudson has joined #openstack-infra13:06
*** hashar has joined #openstack-infra13:06
*** macjack has joined #openstack-infra13:07
*** sks has joined #openstack-infra13:10
*** ddieterly has joined #openstack-infra13:11
waynrelectrical: yo13:12
electricalwaynr: hiya bud13:12
waynryou mind if i take a crack at the bug you reported yesterday? not sure if you wanted to work on it yourself13:12
*** radez_g0n3 is now known as radez13:13
openstackgerritMonty Taylor proposed openstack-infra/shade: Turn server object to dict so we can pop things  https://review.openstack.org/18597613:13
*** ddieterly has quit IRC13:13
*** shardy_ has joined #openstack-infra13:14
*** shardy has quit IRC13:15
openstackgerritChristian Berendt proposed openstack-infra/system-config: Use the correct name for the "Repository Creator's Guide"  https://review.openstack.org/18597813:15
*** salv-orl_ has quit IRC13:17
*** mriedem_away is now known as mriedem13:17
*** pradk has joined #openstack-infra13:18
*** shardy_ has quit IRC13:19
*** shardy has joined #openstack-infra13:20
*** tiswanso has joined #openstack-infra13:22
*** dpaterson has joined #openstack-infra13:22
fungimordred: i spy a typo in your 185972 change13:23
*** doug-fish has joined #openstack-infra13:23
fungiyolanda: replacing the backend git repo sounds like the right solution, if it only got as far as importing/creating it (or perhaps not even that far?)13:25
fungieither replacing it by hand or letting manage-projects reclone it13:25
yolandafungi, i removed the project, just deleting from gerrit, and executed manage-projects13:25
yolandabut it is is buggy anyway13:25
yolandait creates a project with branch=HEAD, revision=master13:26
yolandaand cannot create new branches13:26
fungidid it import a repo lacking a "master" branch?13:26
*** dustins has joined #openstack-infra13:26
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Consume os-client-config directly  https://review.openstack.org/18597213:26
*** macjack has left #openstack-infra13:26
fungii thought we had safeguards in m-p against that (we'd previously just said we didn't support importing projects with no master branch)13:26
yolandayes, it did13:26
yolandamaybe we have an outdated version of jeepyb, but i woudn't say13:27
fungiso, anyway, the solution is to fix the repo being imported (rename their default branch to master or something)13:27
fungiand then delete it from gerrit and replace it or let manage-projects replace it13:27
yolandafungi, they just removed the upstream ref from that project13:27
yolandaas it was pointing to a pass-protected repo, that was the root cause13:28
yolandabut seems i cannot get it back to life13:28
*** annegentle has joined #openstack-infra13:28
openstackgerritMonty Taylor proposed openstack-infra/shade: Turn server object to dict so we can pop things  https://review.openstack.org/18597613:29
*** _nadya_ has quit IRC13:29
*** eharney has joined #openstack-infra13:30
*** annegentle has quit IRC13:30
*** abregman has quit IRC13:30
*** _nadya_ has joined #openstack-infra13:31
*** _nadya_ has quit IRC13:31
*** oomichi has joined #openstack-infra13:31
*** emagana has joined #openstack-infra13:31
*** sdake_ has joined #openstack-infra13:31
*** amitgandhinz has joined #openstack-infra13:33
*** _nadya_ has joined #openstack-infra13:33
*** annegentle has joined #openstack-infra13:33
electricalwaynr: sorry, had a meeting. please go ahead. I've been looking at it my self but no idea where to start tbh13:33
*** yfried is now known as yfried|afk13:33
waynrelectrical: this code is seriously tangled, i don't think the bug is actually in applyDefaults13:33
*** _nadya_ has quit IRC13:33
*** jtomasek has quit IRC13:34
electricalwaynr: i tend to agree. it picks up most things from like namespace 'puppet' and if its not there it picks up global name space correctly13:34
electricalso i think its the sequence of the other parts thats wrong13:34
waynri believe the bug may be somewhere in expandYaml or expandYamlForTemplatejob13:34
*** _nadya_ has joined #openstack-infra13:34
*** emagana has quit IRC13:35
waynrright now i am just pprinting stuff that gets passed into applyDefaults and i can see at least one instance where the "data" parameter appears to be a "project" dictionary13:36
*** jsavak has joined #openstack-infra13:36
waynrwait scratch that i am just confused13:37
electricalwaynr: i did some extra logging as well when running my tests and it gets very confusing at some point. I was digging into the expandYamlForTemplateJob code di13:38
*** zz_jgrimm is now known as jgrimm13:38
*** peristeri has joined #openstack-infra13:39
waynrokay yeah it looks like the project named "project" in concat_defaults004 is being passed as the "data" to applyDefaults13:39
electricalwaynr: the template var in there holds the correct vars but it gets lost somewhere at some point.13:39
*** dkranz has joined #openstack-infra13:41
*** links has quit IRC13:41
openstackgerritThierry Carrez proposed openstack-infra/irc-meetings: Remove redundant/stale scheduling info from desc  https://review.openstack.org/18598813:42
*** cloudm2 has joined #openstack-infra13:42
openstackgerritJohn Griffith proposed openstack-infra/project-config: Revert "Stop running non-voting docker job on nova"  https://review.openstack.org/18599213:43
electricalwaynr: 004 barfs at expanded = deep_format(template, params, allow_empty_variables)13:44
electricalWhere params: {'name': 'project', 'description': 'my project description'}13:45
*** abregman has joined #openstack-infra13:45
electricaland template; {'confversion': 'v0.42', 'fooversion': 'v0.10', 'name': 'job-template', 'builders': [OrderedDict([('shell', "echo 'confVersion: {confversion} & fooVersion: {fooversion}'")])], 'defaults': 'myproject', 'description': 'my project description'}13:45
waynrfg13:46
waynrderp13:46
*** sdake has joined #openstack-infra13:47
*** _nadya_ has quit IRC13:48
*** jtomasek has joined #openstack-infra13:50
*** _nadya_ has joined #openstack-infra13:50
*** sdake_ has quit IRC13:51
*** annegentle has quit IRC13:52
*** annegentle has joined #openstack-infra13:53
*** amotoki_ has quit IRC13:54
*** dustins_ has joined #openstack-infra13:56
*** pradk has quit IRC13:56
*** sks has quit IRC13:58
*** abregman has quit IRC13:59
openstackgerritThierry Carrez proposed openstack-infra/irc-meetings: Small improvements in template  https://review.openstack.org/18600413:59
electricalwaynr: getJobTemplate does get all the right defaults as far as i can see.13:59
*** dustins has quit IRC14:00
waynryeah14:00
*** matbu has quit IRC14:00
waynri'm gonna try reverting this allow_empty_variables patch just out of curiosity14:00
*** pradk has joined #openstack-infra14:01
waynrd'oh...that came before the code reorganization14:02
*** yfried|afk is now known as yfried14:02
*** k4n0 has quit IRC14:02
*** zz_dimtruck is now known as dimtruck14:03
electricalwaynr: i could for fun apply https://review.openstack.org/#/c/179459/ and see what happens?14:04
electricalor the part with your comment14:04
*** chlong has joined #openstack-infra14:05
*** oomichi_ has joined #openstack-infra14:05
waynrgo for it14:06
*** sigmavirus24_awa is now known as sigmavirus2414:06
*** matbu has joined #openstack-infra14:07
electricalstill fails14:07
waynryeah all that patch does is reorder parameter overrides14:07
electricalit has to do with the deep_merge thing. it doesn't seem to merge it correctly14:08
*** dalgaaf has quit IRC14:08
electricalsorry, deep_format14:08
*** tonytan4ever has joined #openstack-infra14:09
*** oomichi has quit IRC14:09
waynri played around with the concat_defaults004.yaml example and it ceases to faile when setting the "myproject" defaults on either the project or the job-group level14:09
electricalso it works on job-template?14:09
waynryeah, the defaults are being applied correctly on job-template14:10
electricalyou have made a change then i assume? because with current master it fails14:10
*** dustins_ is now known as dustins14:10
waynryeah what i am saying is that i modify the testcase concat_defaults004.yaml14:10
*** dustins is now known as dustins_away14:10
*** bswartz has joined #openstack-infra14:10
waynrjust to try different situations to get a feel for what might be going wrong14:10
*** abregman has joined #openstack-infra14:11
waynrwhenever i set "defaults: myproject" on either the project JJB object or the job-group JJB object the example passes14:11
*** koolhead17 has joined #openstack-infra14:11
electricalah okay. 004, 005 and 006 do the same but only with a moved defaults location ( job template, job-group and project )  and they all fail with current master14:11
clarkbmordred: looks like all 3 regions have gateway timeouts so not just an ord thing14:11
waynrthe problem isn't in the deep_format function itself bu somewhere in expandYamlForTemplateJob14:12
waynrhmm i just deleted all the other examples in your patch other than 00414:12
electricalah okay14:12
openstackgerritFlavio Percoco proposed openstack-infra/project-config: Disable py26 jos for glance_store  https://review.openstack.org/18601114:13
electrical002 and 003 are a clone of 001 but with the defaults part moved.. 004, 005 and 006 tests are the same but also with moved default location and no defaults set in the global namespace14:13
waynrelectrical: i think 005 and 006 fail because no defaults are set on the job-template itself which is a differnt bug14:13
electricalyeah. the job-group and project don't pass the defaults correctly further down.14:14
openstackgerritMerged openstack-infra/devstack-gate: Add manila-ui to cloned projects  https://review.openstack.org/18478814:14
electricalwhich is an other bug indeed.14:14
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Alter governance repo voting rules  https://review.openstack.org/18578514:14
electrical004 is sort of a clone of 001 ( which passes ) but without the vars in the global name space14:14
waynri think the real bug is all the lack of intentional design put into scoping of various JJB constructs throughout the YamlParser14:15
electricalhmm yeah..14:15
*** btully has joined #openstack-infra14:15
*** oomichi has joined #openstack-infra14:16
waynrit'd be really nice to just to come up with a well-defined spec that describes the behavior of JJB Yaml and re-build the implementation to match that spec from the ground up14:16
electricalyeah, but that's not something that can be done overnight :-)14:16
waynrevery time somone adds a new feature or fixes an old bug there is some corner case that no one considered that can lead to error states14:16
waynrs/can/will/14:17
*** annegentle has quit IRC14:17
electricalfrom what i noticed with this bug at least is that it tries to merge myproject defaults with global defaults.. and if it can't find the key in the global defaults it will just ignore it ( or something like this )14:17
waynrwe could at least put a moratorium on new yamlparser features until we have a chance to rewrite the current implementation14:17
*** leopoldj has quit IRC14:17
*** oomichi_ has quit IRC14:18
openstackgerritAndreas Jaeger proposed openstack-infra/project-config: Add project-team-guide  https://review.openstack.org/18574614:18
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Update infra-specs voting rules  https://review.openstack.org/18579214:19
clarkbmordred: my hunch is this is related to image size and upload time vs whatever proxy is sitting infront of swift14:19
clarkbbut considering that we got rate limit 413s asthe error when using glance v1 that is probably completely wrong14:19
mordredclarkb: AWESOME14:21
electricalwaynr: i agree..  your https://review.openstack.org/#/c/182542/ should spark the rewrite part i guess.14:21
mordredI'm going to go bug the swift people14:21
*** timcline has joined #openstack-infra14:21
fungiclarkb: any attempt to reason about openstack error codes is likely doomed to failure and madness14:22
waynrelectrical: well that's mostly targeted at solidifying the programmatic API, i have no intention of rewriting the YamlParser in relation to that spec14:22
electricalwaynr: ah okay, i see. well, perhaps we should include that part in there for a 2.0 release :-)14:23
waynrwell once the API is solidified we should be able to rewrite the yamlparser more easily without worrying about how its implementation affects the rest of the program14:23
clarkbmordred: as an aside, if we get image uploads to dfw working then get swift object passthrough on logs.o.o working we will have solved the image publishing problem for the most part14:23
mordredya14:23
mordredI'll be super happy about that14:24
electricalwaynr: ah i see, okay14:24
mordredclarkb: btw - I have patches up for all of the things14:24
clarkbmordred: http://logs.openstack.org/images/ will host our images, but we won't do index generation for it so indexing is the last bit to figure out14:24
waynrso we can release 2.0.0 then begin working on yamlparser rewrite for a minor release of 2.x series that is backwards-compatible with 2.0.014:24
clarkbmordred: ya, the nodepool one failed tests pretty hard, still digging through oscc before getting to that one14:24
*** jokke_ has quit IRC14:24
*** jokke_ has joined #openstack-infra14:24
mordredclarkb: I did not do your suggestion on the config refresh because of how nodepool wants to think about it14:24
mordredclarkb: although I did also start to add the plumbing to do the way you suggested14:25
clarkbmordred: you actually did my original suggestion14:25
mordredI did?14:25
mordredneat14:25
waynralthough to be honest once the API is cleaned up i am hoping to begin working on a pure-python approach to configuring jenkins jobs14:25
clarkbmordred: which was that nodepool should talk to oscc directly14:25
waynrwhich will remove the need for me to worry about the yamlparser14:25
mordredah. I did not catch that as a suggestion. and yes - I completely agree having dug into that bit of code14:25
electricalwaynr: how do you mean pure python approach ?14:25
openstackgerritYanis Guenane proposed openstack-infra/tripleo-ci: Pin puppetlabs-mysql  https://review.openstack.org/18601614:25
clarkbmordred: it was part of my why do we have a clouds.yaml anyways. Though I did think we should do it that way so that nodepool's cloud key could point to what may soon be called profile14:26
waynri mean like "job = JenkinsJob(some_dict)" where some_dict contains the basic key-value pair necessary to generate XML for a job14:26
ttxmordred: ok, I think with those two things we are ready to go live on irc-meetings. preparing launch email. Already updated wiki page14:27
*** rbradfor_ is now known as rbradfor14:27
ttxI'll wait for the new template to be picked up14:27
openstackgerritMerged openstack-infra/irc-meetings: Remove redundant/stale scheduling info from desc  https://review.openstack.org/18598814:27
waynrthis JenkinsJob object (subclass of OrderedDict) can then be passed to the upcoming XmlBuilder class to actually generate the correct XML14:27
openstackgerritMerged openstack-infra/irc-meetings: Small improvements in template  https://review.openstack.org/18600414:27
mordredclarkb: oh - so - the other reason we want clouds.yaml and not just direct config in nodepool.yaml is so that we can handle cloud providers that do more advanced keystone things without having to teach nodepool all of that logic14:28
*** ddieterly has joined #openstack-infra14:28
electricalwaynr: ah, i think i understand what you mean.14:28
waynrI haven't put a great deal of thought into the programmatic interface I want to use for these python classes but they should be arbitrarily extensible using the full power of the Python language14:28
mordredclarkb: for instance, unitedstack is a keystone v3 cloud, so you have to pass user_domain_id and project_domain_id as well14:28
mordredbut you only need to do that for them14:28
*** wenlock has joined #openstack-infra14:29
mordredor if we grow a cloud provider that decides to use a different keystone auth plugin than password14:29
waynrit will also allow for arbitrary abstractions to represent relationships between jobs such as a SerialParameterizedPipeline class which could be a sublcass of "list"14:29
mordred(we could decide to use kerberos for infra cloud :) )14:30
clarkbmordred: except we are teaching nodepool about all that in your change aren't we?14:30
mordredwe are not14:30
clarkbmordred: its the way we figure out if we need a new provider manager14:30
fungimordred: kerberos is keystone v4 from what i hear14:30
waynrrather than trying to squeeze all kinds of strange and competing logic in the YamlParser14:30
*** bnemec has quit IRC14:30
mordredclarkb: we compare cloud configs in my change14:30
openstackgerritMerged openstack-infra/project-config: Add barbican-dogtag flag  https://review.openstack.org/18518314:30
electricalwaynr: ah okay. well, i think we can agree that in the current state of the code it will be near impossible to fix this defaults merging behaviour right?14:30
waynroh i don't know about that14:30
mordredthe direct username/password comparison is in the old code14:30
*** claudiub has quit IRC14:31
ttxfungi: I don't remember if we wanted to CNAME eavesdrop before we announce...14:31
waynri think we can probably fix it but it's just really confusing14:31
waynrwell14:31
*** liuxg has joined #openstack-infra14:31
electricalwaynr: hehe i already gave up after a few hours of debugging this morning.14:31
fungittx: we don't have any different vhost for it right now anyway14:31
mordredclarkb: in my change we pass the username/password/auth-url to the client config constructor, and then compare old and new client configs - that means ultimately the only thing the infra production case will be testing will be client configs14:31
clarkbmordred: I see so you are saying the only way we would support non passwd auth is via oscc clouds.yaml? (your change does directly handle passwords)14:31
waynri have an idea or two i wanna try, will know in 20 minutes or so if it works14:31
fungittx: we can always run a url redirect at the old name if we decide to change it later14:31
mordredclarkb: yes14:31
electricalwaynr: cool :-)14:31
waynrbut i'm only gonna try to fix the 004 example14:31
waynrwell maybe 005 and 00614:32
electricalyeah indeed14:32
mordredclarkb: or, we'd have to teach nodepool all about the ins and outs of all the different ways to configure different keystone auth types14:32
ttxfungi: so I should just point to eavesdrop for the moment ?14:32
mordredwhich I promise we do not want to do14:32
electricalif you would solve 005/006 then you would also fix 002/003 since they represent the same issue ( location of the defaults definition )14:32
clarkbmordred: ok, so we keep what it knows today for backward compat and simplicity, but moving forward extra fancyness stays in oscc14:32
mordredyah14:32
*** annegentle has joined #openstack-infra14:32
fungittx: yeah, http://eavesdrop.openstack.org/ seems fine for now. if people really want a different name, i can do whatever's necessary to make that happen later once the bikeshedding has subsided14:32
waynroh14:32
electrical004 with the defaults in the job-template would be easiest them i guess.14:32
ttxack14:33
mordredclarkb: I'll try to capture that in a "why" comment somewhere14:33
*** esker has joined #openstack-infra14:33
*** mestery has quit IRC14:33
*** mtanino has joined #openstack-infra14:33
clarkbmordred: fwiw I think pep8 test may have caught the reason the tests failed14:34
mordredclarkb: oh - and wow, yeah. nodepool tests failed HARD. fixing now14:34
mordredclarkb: well, they failed because I suck14:34
*** mestery has joined #openstack-infra14:34
clarkbwell specifically because an undefined name was used :)14:34
fungittx: who added the timeanddate.com hyperlinks? that's pretty nifty14:34
clarkbmordred: btw mysql in docker for nodepool tests is pretty handy14:34
mordredclarkb: ++14:35
*** bnemec has joined #openstack-infra14:35
mordredclarkb: I'm a big fan of the idea of docker containers with our database configs in them14:35
ttxfungi: they were there on the original template I wrote14:35
*** dims has quit IRC14:35
mordredclarkb: and kinda want to publish docker containers with the mysql and postgres databases with openstackci user added to them14:35
fungithis is probably the first time i've looked at it while not braindead14:35
yolandahi, we need to have this setting on place : VIRTUALENV_NO_PIP=114:36
mordredclarkb: I mean, if there were a simple way like that to spin one up for unittests - I'm pretty sure we could kill sqlite with fire14:36
yolandabut seems as tox overwrites it14:36
clarkbmordred: if you show me how to make the db just start at boot without me making custom init changes I would be so happy, but its been such a low priority I have never bothered to figure it out14:36
mordredclarkb: but - I don't have time to think about the full stry yet there14:36
mordredclarkb: yup. same here14:36
yolandado you know how to mak tox to don't install pip, but use the one from the system?14:37
waynrelectrical: fixed 00414:37
electricalwaynr: that's quick :-)14:37
clarkbyolanda: if using tox >= 2.0 it cleans the env prior to testing. You can whitelist the env var if you need to14:37
fungiyolanda: did this start breaking for you with pip 2.0? if so, you probably need a passenv added to your tox.ini14:37
clarkbyolanda: its also possible that tox supports that functionality as a config in tox.ini directly14:37
yolandafungi, clarkb, we see that virtuaenv installs an old pip version 1.514:38
yolandaand we need newer ones14:38
yolandapip 6 or pip 714:38
electricalwaynr: diff? would love to try it out14:38
*** nelsnelson has quit IRC14:38
mordredyolanda: that soudns like very old virtualenv14:38
fungiyolanda: virtualenv provides its own bundled pip version14:38
clarkbya, you can just install a newer virtualenv14:38
fungipip 1.5 would mean you have virtualenv 1114:38
yolandalet me check, last time i looked it brought a very old version14:39
fungiif memory serves14:39
mordredyolanda: current virtualenv has current pipi14:39
mordredyolanda: they release together14:39
fungi"very old" in this case means _months_ old14:39
yolandaok, let me install latest14:39
yolandaand check14:39
*** achanda has joined #openstack-infra14:40
*** emagana has joined #openstack-infra14:40
waynrelectrical: sure, one moment14:41
yolandafungi, i did a pip install --upgrade for virtualenv and it only raises to 1.11.614:41
yolandahow do you install them, and which version do you have upstream?14:41
*** dangers_away is now known as dangers14:41
mordredyolanda: mordred@camelot:~$ virtualenv --version14:42
mordred13.0.014:42
*** eharney has quit IRC14:42
fungiyolanda: 13.0.1 is the latest on pypi14:42
mordredyolanda: are you pointing at a stale mirror?14:42
fungiyolanda: are you using an outdated mirror14:42
fungi?14:42
yolandalooks like14:42
yolandayep, our mirror14:42
*** kgiusti has joined #openstack-infra14:43
waynrelectrical: http://paste.openstack.org/show/240096/14:43
yolandamordred, fungi, you rock14:43
fungiyolanda: i take it you don't use bandersnatch to generate your mirror14:44
waynri left in some debug print statements14:44
yolandafungi, you are right14:44
*** salv-orlando has joined #openstack-infra14:45
*** otter768 has joined #openstack-infra14:45
openstackgerritAndreas Jaeger proposed openstack-infra/project-config: Sort PO files to avoid random reorder  https://review.openstack.org/18602614:46
clarkbmordred: https://review.openstack.org/#/c/185824/1 any chance we can get that in so I can reenable puppet on nodepool.o.o14:49
*** yfried is now known as yfried|afk14:49
*** otter768 has quit IRC14:50
*** hemnafk is now known as hemna14:50
mordredclarkb: done14:51
*** patrickeast has joined #openstack-infra14:51
*** achanda has quit IRC14:52
*** zz_ja is now known as zz_zz_ja14:52
clarkbdanke14:53
*** nelsnelson has joined #openstack-infra14:53
*** fijal has quit IRC14:53
yolandafungi, mordred, looking at system-config, it says desired_virtualenv = 12.1.114:53
yolandahow are you on 13 ?14:54
clarkbyolanda: we are not14:54
*** fijal has joined #openstack-infra14:54
yolanda12.1.1 then?14:54
electricalwaynr: looks like you solved it indeed :-)14:54
clarkbya that is what we are using14:54
yolandamordred, why did you told me 13 ?! is that your local node?14:54
clarkbyolanda: yes camelot is mordred's laptop14:54
yolandaok14:54
mordredoh - sorry - yeah, that's what I have locally14:55
yolandai'll stick on what upstream has14:55
yolandamordred, you are lying to me :)14:55
*** Kennan has quit IRC14:55
*** amotoki_ has joined #openstack-infra14:55
*** dims_ has joined #openstack-infra14:55
mordredyolanda: I do that :)14:55
ttxAlright we are LIVE. Thx everyone who helped making this happen14:56
*** Kennan has joined #openstack-infra14:56
yolandaclarkb, which pip brings 12.1.1?14:56
fungi12.1.1 was the latest when we updated our puppet to install latest virtualenv. ages ago. like a couple of weeks already14:56
mordredttx: w00t!14:56
clarkbmordred: fungi: any thoughts on further debugging of swift image uploads? if not I guess next step is a rackspace support ticket14:56
mordredclarkb: I asked in the swift channel14:56
mordredno response yet14:56
clarkbyolanda: I think its latest pip 6.x14:56
fungiyolanda: pip 6.1.114:56
yolandaok14:56
yolandathx14:56
*** pblaho has quit IRC14:57
fungiyolanda: virtualenv_version - 6 = pip_version14:57
waynrelectrical: i am close to solving the others14:57
clarkbmordred: oh cool I can wait for a response there14:57
*** pblaho has joined #openstack-infra14:57
electricalwaynr: i wonder where the actual bug is then why it doesn't merge both ways. so confusing :-)14:57
fungi(starting with virtualenv 12 and pip 6 anyway)14:57
fungiyolanda: so virtualenv 13.0.1 vendors a copy of pip 7.0.114:57
yolandafungi, that's more risky14:58
yolandaare you going to update?14:58
fungiyolanda: definitely. just no idea when14:58
fungiclarkb: what is the current error we're getting now that we're using glance v2?14:58
clarkbfungi: likely after we get image builds fixed14:58
clarkbfungi: ClientException: Object POST failed: 504 Gateway Time-out is the edited error message14:59
*** maishsk_afk has quit IRC14:59
fungi504. thanks14:59
waynrelectrical: the bug for 002 and 003?14:59
clarkbif the dib builds for hpcloud successfully upload today then I think we can worry about latest virtualenv14:59
*** gokrokve has joined #openstack-infra14:59
electricalwaynr: the bug for 00414:59
waynroh14:59
*** freerunner has quit IRC14:59
*** amotoki_ has quit IRC15:00
electricali do see what you are doing but i wonder where its going wrong. or its something thats not being handled at all anywhere in the code15:00
*** claudiub has joined #openstack-infra15:00
fungiclarkb: are you able to reproduce that error when posting the same file to swift manually?15:00
clarkbfungi: no I haven't attempted a manual upload yet, I will be honest. It is because I have no idea what that entails as its "complicated"15:00
*** salv-orlando has quit IRC15:00
clarkbmaybe mordred or greghaynes can tl;dr it for me and I can give it a go15:00
*** ddieterly has quit IRC15:01
waynrwell the problem is that there is no way for the "project" or "job-group" to receive parameters set at the job-template level15:01
clarkbfungi: you basically have to do completely undocumented things (because its not upstreamed yet) with swift uploads and glance tasks15:01
*** timcline has quit IRC15:01
waynrit has to happen on a job-template by job-template basis as in the patch i linked you to15:01
clarkbfungi: I could probably figure it out by reading shade code15:01
*** armax has joined #openstack-infra15:01
openstackgerritMerged openstack-infra/system-config: Use _ instead of - in clouds.yaml  https://review.openstack.org/18582415:01
mordredI have an ansible playbook15:01
*** timcline has joined #openstack-infra15:02
mordredthat will do it15:02
mordredalthough it relies on ansible code that is not yet upstreamed :)15:02
waynrthat "params" dict is being used to populate template strings on the merged job-template/job-group/project15:02
fungiclarkb: oh. it's unfortunate that rackspace isn't using openstack. again15:02
*** jamespage_ has joined #openstack-infra15:02
waynrthe "params" is the merged set of job-group+project parameters15:02
mordredoh - do you mean testing doing it without shade? I do have shell scripts you can use to do that15:02
clarkbmordred: yes, we should probably try it without shade entirely15:03
clarkbbut as I said thats "complicated"15:03
fungimordred: just wondering if we have distilled a reproducible use case that doesn't start out "install and run this code we wrote"15:03
mordredclarkb: https://review.openstack.org/#/c/140840/13/tools/HOW-TO-TEST.rst15:03
mordredif you grab 140840 you'll get some new scripts in the tools dir15:03
mordredthey can be used to upload to rackspace using just normal command line clients15:04
mordredthe instructions on how are in that file15:04
mordredthe swift command line for just doing the swift upload is in tools/upload-rax.sh15:04
mordredI think just trying that first is likely a good first test15:04
*** asselin_ has joined #openstack-infra15:04
*** Ala has quit IRC15:04
mordredswift upload --object-name ${IMAGE}.vhd images ${IMAGE}.vhd15:05
electricalwaynr: feels like there is a bug then in params = self.applyDefaults(params, template) handling. template should be the override dict in the applyDefaults function which does the same as you done.15:06
clarkbOH! I think we are trying to upload qcows still :)15:06
clarkbder15:06
mordredreally? that's no fun15:06
mordredbut it was a swift gateway timeout that we got, no?15:06
clarkbI mean it doesn't matter because swift fails15:06
mordredyah15:06
fungiclarkb: aha. missing the convert step? or just passing the wrong filename to upload?15:06
clarkbfungi: I think both15:07
mordredyou want I should try the swift upload?15:07
*** dprince has quit IRC15:07
clarkbmordred: ya if you want to at least see if swift upload can be made to work with the qcow2 that would be good15:07
electricalwaynr: only difference is that you do a if key not in params:    and applyDefaults does a if key in params:15:07
*** maishsk_afk has joined #openstack-infra15:08
openstackgerritChristian Berendt proposed openstack-infra/irc-meetings: Whitelist mkdir as external in the tox.ini file  https://review.openstack.org/18603215:08
*** ddieterly has joined #openstack-infra15:08
*** freerunner has joined #openstack-infra15:09
mordredclarkb: trying now15:09
clarkbI will write a change for the file format issue15:09
electricalwaynr: seems i was wrong. broke more things then i fixed with it :p15:09
katyaferventHi everyone! please help us to make stable/juno working https://review.openstack.org/#/c/184809/15:09
*** bswartz has quit IRC15:10
electricalwaynr: i would have assumed that https://git.openstack.org/cgit/openstack-infra/jenkins-job-builder/tree/jenkins_jobs/parser.py#n137 would take care of the part you just fixed15:11
electricalvery confusing.15:11
waynrelectrical: agreed, very confusing15:12
*** bswartz has joined #openstack-infra15:12
waynrbut that code block addresses a different issue related to parameter interpolation15:12
openstackgerritClark Boylan proposed openstack-infra/system-config: Set image-type to vhd for rax nodepool providers  https://review.openstack.org/18603415:13
electricalwaynr: yeah indeed.15:13
*** maishsk_afk_ has joined #openstack-infra15:13
*** e0ne is now known as e0ne_15:13
clarkbmordred: ^ I think we may also need to be telling rax to trea these at pvhvm somehow, but that can happen after we have working images15:13
*** maishsk_afk has quit IRC15:13
*** maishsk_afk_ is now known as maishsk_afk15:13
electricalwaynr: in any case a good rewrite is mandatory for the project to move forward :-)15:14
waynragreed!15:15
mordredclarkb: yes, that's right - we need the properties to be set15:15
mordredclarkb: I think that support is already added to nodepool?15:15
waynri wonder how we should go about socializing the need for a moratorium on new JJB yamlparser features15:15
clarkbmordred: ya I think we can set it on the image, but would have to go double check15:15
clarkbmordred: its definitely less urgent than make swift go15:15
clarkbno pun intended15:15
mordredclarkb: ok. so - uploading via swift command line client worked15:16
mordredclarkb: so there is a different problem15:16
mordredclarkb: I'll start digging in15:16
electricalwaynr: a large banner in the top of the file and be done? ;-)15:16
waynryeah, i guess i'll keep commenting on new features indicating that we should wait for a rewrite15:17
waynr(i personally don't ready comments at the top of files very often)15:17
*** hashar has quit IRC15:17
electricalwaynr: hehe very true :-)15:17
electricalat least the testing framework is easy to use, that we don't have to change i hope :p15:18
openstackgerritJames E. Blair proposed openstack-infra/project-config: Add project-team-guide  https://review.openstack.org/18574615:18
waynryeah15:19
*** spzala has joined #openstack-infra15:19
*** _nadya_ has quit IRC15:19
electricalwaynr: with 004 solved at least i won't have to duplicate entries from other name spaces into global, so that will be much cleaner15:20
*** rlandy has quit IRC15:20
*** sslypushenko has joined #openstack-infra15:20
jeblairmordred, SergeyLukjanov: if you have a minute to leave a vote on https://review.openstack.org/182811 that would be great15:21
*** panda has quit IRC15:21
*** u_glide has left #openstack-infra15:21
*** cody-somerville has quit IRC15:21
*** acruz_ has quit IRC15:22
*** yfried|afk is now known as yfried15:22
jeblairclarkb: revised gertty migration work okay?15:22
mordredjeblair: done15:22
clarkbjeblair: yup, appears to be up and happy15:22
*** e0ne_ is now known as e0ne15:22
*** panda has joined #openstack-infra15:22
openstackgerritYanis Guenane proposed openstack-infra/tripleo-ci: Pin puppetlabs-mysql  https://review.openstack.org/18601615:22
*** koolhead17 has quit IRC15:22
*** claudiub has quit IRC15:23
liuxghi, my CI failed with test case " test_minimum_basic_scenario" for establish authenticated ssh connection to server failed15:23
*** dtantsur is now known as dtantsur|afk15:23
liuxgwho can give me a help15:23
*** koolhead17 has joined #openstack-infra15:24
openstackgerritMerged openstack-infra/infra-specs: Add priority efforts section  https://review.openstack.org/18333715:24
*** nelsnelson has quit IRC15:24
waynrelectrical: hmm looking more closely at 002, i don't think this is an example we should support before the yamlparser rewrite15:24
*** rbrooker_ has joined #openstack-infra15:25
waynrthat is to say, we shouldn't expect non-global defaults to merge into global defaults15:25
openstackgerritDavanum Srinivas (dims) proposed openstack-infra/irc-meetings: Switch oslo chair  https://review.openstack.org/18603815:25
asselin_liuxg, you can join us in -meeting415:25
*** dprince has joined #openstack-infra15:25
waynrthe way defaults are designed is as an "either-or" arrangement15:25
waynreither you use global defaults, or you use an alternate defaults15:25
liuxgasselin: thanks, I will join the meeting15:26
waynralthough in general i think in the future JJB should support something like 002/005 and 003/00615:26
waynrso you were correct earlier ;)15:27
*** gokrokve has quit IRC15:27
waynr(correct that we leave the fix at targeting 004 for now)15:27
*** gokrokve has joined #openstack-infra15:28
*** bhunter71 has joined #openstack-infra15:28
*** devvesa has quit IRC15:28
waynrelectrical: you want to update the patch to remove 002,003,005, and 006 and add the fix for 004?15:28
*** maishsk_afk has quit IRC15:28
clarkbmordred: is this possibly related to how that account has a different project_name for swift than it does for nova?15:29
*** koolhead17 has quit IRC15:29
mordredclarkb: it shoudl not - I'm using the same keystone creds in each case15:29
clarkbmordred: the url in the errors seem to have the correct mosso files path so I don'y think that is it15:29
mordredclarkb: I've made a small python script to do just the upload using shade and those creds since the swift command line worked15:30
*** maishsk_afk has joined #openstack-infra15:30
mordredclarkb: and I'm running that with debug on15:30
mordredwe'll see what, if anything, that shows15:30
*** marun has joined #openstack-infra15:31
*** eharney has joined #openstack-infra15:32
openstackgerritJames E. Blair proposed stackforge/gertty: Fix searching for message  https://review.openstack.org/17266715:32
openstackgerritJames E. Blair proposed stackforge/gertty: Advance cursor on change list toggle  https://review.openstack.org/18569815:32
openstackgerritJames E. Blair proposed stackforge/gertty: Allow reviewing one change in change list  https://review.openstack.org/18569715:32
openstackgerritJames E. Blair proposed stackforge/gertty: Fix updating flags on threaded changes  https://review.openstack.org/18569615:32
openstackgerritJames E. Blair proposed stackforge/gertty: Make "limit" a noop in queries  https://review.openstack.org/17267015:32
openstackgerritJames E. Blair proposed stackforge/gertty: Fix searching by reviewer account id  https://review.openstack.org/17266915:32
openstackgerritJames E. Blair proposed stackforge/gertty: Batch sync change by commit tasks  https://review.openstack.org/17269115:32
openstackgerritJames E. Blair proposed stackforge/gertty: Fix age searching  https://review.openstack.org/17266815:32
openstackgerritJames E. Blair proposed stackforge/gertty: Fix refresh on project and change lists  https://review.openstack.org/18388715:32
openstackgerritJames E. Blair proposed stackforge/gertty: Fix diff display of deleted empty files  https://review.openstack.org/18054615:32
openstackgerritJames E. Blair proposed stackforge/gertty: Add database pruning  https://review.openstack.org/17998115:32
openstackgerritJames E. Blair proposed stackforge/gertty: Attach comments to files  https://review.openstack.org/17998015:32
openstackgerritJames E. Blair proposed stackforge/gertty: Add files table  https://review.openstack.org/17997915:32
openstackgerritJames E. Blair proposed stackforge/gertty: Add ability to review multiple changes at once  https://review.openstack.org/18569515:32
*** gokrokve has quit IRC15:32
SergeyLukjanovjeblair, looking15:32
mordredit's the jeblair patch bomb!15:32
*** liuxg has left #openstack-infra15:32
jeblairit's going to be followed by a merge bomb :)15:33
zigoStevenK: May I help to get the packages that you need into Debian?15:33
*** mrmartin has joined #openstack-infra15:33
zigoZuul maybe?15:33
zigoAnd nodepool?15:33
clarkbI think hashar has both of those in debian already15:33
zigoOh ok.15:33
zigoclarkb: https://packages.debian.org/search?keywords=zuul&searchon=names&suite=all&section=all15:34
zigoNo zuul in Debian...15:34
zigoNo nodepool either.15:34
clarkbhuh, I was pretty sure hashar had done packaging work for both15:34
*** BharatK has joined #openstack-infra15:34
openstackgerritMerged stackforge/gertty: Add support for SQLAlchemy 1.0.4  https://review.openstack.org/18576115:34
zigoclarkb: Problably, but it was never uploaded.15:35
jeblairmaybe just wikimedia-local packaging?15:35
fungizigo: also pretty sure StevenK is a dd and can upload on his own, though not sure if you were offering packaging help15:35
zigofungi: I can help anyone who ask for such things.15:35
mordredwe're popular15:35
rcarrillocruzasselin: hiya!15:36
rcarrillocruzyou around?15:36
fungizigo: ahh, yep, he's a.k.a. stevenk@debian.org15:36
zigofungi: I just found out on db.debian.org! :)15:36
*** claudiub has joined #openstack-infra15:37
zigoI wonder why zuul and nodepool were never uploaded then.15:37
asselin_rcarrillocruz, yes....in 2 meetings...15:37
asselin_rcarrillocruz, hi :)15:37
rcarrillocruzduh, sorry about that :/15:37
*** ParsectiX has joined #openstack-infra15:37
rcarrillocruzso, we are prepping our Gerrit upgrade15:37
rcarrillocruzand turns out our internal manifest won't work on Trusty systems15:37
rcarrillocruzwe really want to do upgrade to 2.8 on Trusty15:38
SergeyLukjanovjeblair, +2 on the new team structure - very clear and reasonable! thx15:38
rcarrillocruzi'll take a stab at puppet-gerrit module15:38
rcarrillocruzdo I need to add myself to the summit etherpad?15:38
*** pradk has quit IRC15:38
rcarrillocruzor is it put on read-only ?15:38
openstackgerritChristian Berendt proposed openstack-infra/irc-meetings: Use valid HTML5 syntax in the meetingindex.jinja template  https://review.openstack.org/18604415:38
clarkbrcarrillocruz: fwiw we did 2.4 -> 2.8 on precise, then upgraded to trusty15:38
clarkbrcarrillocruz: so that is known to work15:38
jeblairSergeyLukjanov: cool, thanks!15:38
*** koolhead17 has joined #openstack-infra15:39
*** harlowja_at_home has joined #openstack-infra15:40
rbradforI have documented my limited understanding of the Fedora image used by Magnum at https://blueprints.launchpad.net/magnum/+spec/cache-required-fedora-image   I have requested input from the Magnum team and would value any Infra knowledge that can help achieve the objective15:40
rcarrillocruzclarkb: did you do an upgrade on the same instance? or you did a spin of a new instance, import of mysqldump and gerrit repos and then did the upgrade on new instance?15:40
clarkbrcarrillocruz: we used a new instance that we switched dns to point to15:40
*** tim_o has joined #openstack-infra15:41
rcarrillocruzso you did the mysql export/import then...15:41
*** gokrokve has joined #openstack-infra15:42
jeblairrbradfor: ack, i'll try to take a look at it in a bit15:42
mordredclarkb: found the swift issue and have learned somethign about swift15:43
mordredclarkb: we were actually failing in the post-upload metadata update15:43
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Cleanup std dev backfill in run_time graph  https://review.openstack.org/18604615:43
mordredclarkb: turns out that's much more expensive than one might expect15:43
clarkbfungi: devstack-trusty image build was about 30 minutes today15:43
mordredclarkb: so I will now work up a patch to set the metadata at object creation time15:43
clarkbmordred: oh? due to hashing?15:43
*** maishsk_afk has quit IRC15:44
fungiclarkb: and was an hour yesterday?15:44
notmynamein many swift deployments (including rackspace), updating metadata is a whole object copy operation (server-side). therefore it's expensive15:44
clarkbfungi: ya working to confirm exact numbers15:44
clarkbfungi: its all logged, just have to get the correct log15:45
*** yfried has quit IRC15:45
fungiclarkb: so cocktail napkin analysis says that sata is 25% as fast as ephemeral and ssd is 50% as fast as ephemeral?15:45
*** nelsnelson has joined #openstack-infra15:46
clarkbfungi: old build was ~49 minutes, new build ~32 minutes15:46
clarkband ya ephemeral was ~1515:46
*** wznoinsk_ has joined #openstack-infra15:47
openstackgerritJames E. Blair proposed stackforge/gertty: Advance cursor on change list toggle  https://review.openstack.org/18569815:47
openstackgerritJames E. Blair proposed stackforge/gertty: Allow reviewing one change in change list  https://review.openstack.org/18569715:47
openstackgerritJames E. Blair proposed stackforge/gertty: Fix updating flags on threaded changes  https://review.openstack.org/18569615:47
openstackgerritJames E. Blair proposed stackforge/gertty: Add ability to review multiple changes at once  https://review.openstack.org/18569515:47
*** annegentle has quit IRC15:47
jeblairfound a bug in the mass-review feature15:48
*** wznoinsk_ is now known as wznoinsk15:48
*** Guest59858 has quit IRC15:48
openstackgerritMonty Taylor proposed openstack-infra/shade: Set metadata headers on object create  https://review.openstack.org/18604815:49
pabelangerSo, I'm not part of the openstack-infra-packagers but need to get some patches merged.  Could a core please approve the following?  https://review.openstack.org/#/c/59590/15:49
pabelangersame with https://review.openstack.org/#/c/59591/15:49
electricalwaynr: yeah will do. thanks15:49
pabelangerthis is for the zuul-packaging repo15:49
*** Sukhdev has joined #openstack-infra15:50
*** ihrachyshka has quit IRC15:51
openstackgerritMerged stackforge/gertty: Support regexes in search  https://review.openstack.org/17264915:51
openstackgerritMerged stackforge/gertty: Do not enqueue duplicate tasks  https://review.openstack.org/17265015:51
openstackgerritMerged stackforge/gertty: Fix searching for message  https://review.openstack.org/17266715:51
openstackgerritMerged stackforge/gertty: Fix age searching  https://review.openstack.org/17266815:51
openstackgerritMerged stackforge/gertty: Fix searching by reviewer account id  https://review.openstack.org/17266915:51
openstackgerritMerged stackforge/gertty: Make "limit" a noop in queries  https://review.openstack.org/17267015:51
clarkbfungi: its a decent improvement that we can likely live with particularly as we trim down the number of images we host15:51
*** ParsectiX has quit IRC15:51
openstackgerritMerged stackforge/gertty: Batch sync change by commit tasks  https://review.openstack.org/17269115:51
openstackgerritMerged stackforge/gertty: Add files table  https://review.openstack.org/17997915:51
openstackgerritMerged stackforge/gertty: Attach comments to files  https://review.openstack.org/17998015:52
openstackgerritMerged stackforge/gertty: Add database pruning  https://review.openstack.org/17998115:52
openstackgerritMerged stackforge/gertty: Fix diff display of deleted empty files  https://review.openstack.org/18054615:52
openstackgerritMerged stackforge/gertty: Fix refresh on project and change lists  https://review.openstack.org/18388715:52
openstackgerritMerged stackforge/gertty: Add ability to review multiple changes at once  https://review.openstack.org/18569515:52
openstackgerritMerged stackforge/gertty: Fix updating flags on threaded changes  https://review.openstack.org/18569615:52
openstackgerritMerged stackforge/gertty: Allow reviewing one change in change list  https://review.openstack.org/18569715:52
openstackgerritMerged stackforge/gertty: Advance cursor on change list toggle  https://review.openstack.org/18569815:52
jeblairfungi, crinkle: that was mass-reviewed ^15:53
*** lifeless_ is now known as lifeless15:53
*** jamespage_ has quit IRC15:54
crinklejeblair: :D15:54
*** jamespage_ has joined #openstack-infra15:54
fungiheh15:54
openstackgerritMonty Taylor proposed openstack-infra/shade: Set metadata headers on object create  https://review.openstack.org/18604815:55
fungiclarkb: we have a few diskimages we're no longer using still in the config, right? maybe we should go ahead and cull those now that we're moving actual production labels to dib anyway?15:55
openstackgerritRichard Pijnenburg proposed openstack-infra/jenkins-job-builder: Adding tests surrounding handling of defaults values  https://review.openstack.org/18566115:55
mordredclarkb, fungi, notmyname: ^^ I have tested that that can upload one of our images15:55
*** amotoki_ has joined #openstack-infra15:56
*** arxcruz has joined #openstack-infra15:56
*** arxcruz has quit IRC15:56
electricalwaynr: thank you for fixing the bug :-)15:56
*** dannywilson has joined #openstack-infra15:56
clarkbfungi: its possible, we may want to sync up with ianw as those are largely images he has in progress15:56
clarkbthough I think devstack keeps breaking on fedora and centos for unrelated reasons making it more complicated15:56
electricalclarkb: fungi mordred https://review.openstack.org/#/c/185661/   interesting fix. would love your feedback.15:56
*** fhubik is now known as fhubik_afk15:57
fungiclarkb: oh, i meant some of the duplicate "foo-dib" labels15:57
mordredclarkb, fungi, notmyname: for reference, this is the test script I used: http://paste.openstack.org/show/240206/15:57
clarkbfungi: ya I think I got rid of the true duplicates, the others are the ones that ianw was working on and aren't really duplicates but I maybe missing something15:57
*** tiswanso has quit IRC15:58
mordredI did the md5/sha256 params to skip the calculation of them - they are not required15:58
mordred(pasting that both to show what I verified, and the simple case of how to do this task from python)15:58
mordredShrews: ^^15:59
*** jamespage_ has quit IRC15:59
*** deepakcs has joined #openstack-infra16:00
*** baoli has quit IRC16:00
*** emagana has quit IRC16:00
*** julim has quit IRC16:00
*** yamahata has quit IRC16:00
*** zz_zz_ja is now known as zz_zz_zz_ja16:01
*** ildikov has quit IRC16:01
*** amotoki_ has quit IRC16:01
*** yamahata has joined #openstack-infra16:01
openstackgerritMerged stackforge/gertty: Be more verbose on non-tagged versions  https://review.openstack.org/17977616:01
openstackgerritMerged stackforge/gertty: Make ButtonDialog scrollable  https://review.openstack.org/18380916:01
openstackgerritMerged stackforge/gertty: Explain how to install on Arch Linux  https://review.openstack.org/17500816:01
*** baoli_ has joined #openstack-infra16:02
*** emagana has joined #openstack-infra16:02
mtreinish^^^ \o/16:02
*** tiswanso has joined #openstack-infra16:02
jeblairmordred: what version of pbr should gertty be using?16:03
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck: Add query for zake 0.2.2 stable break bug 1459322  https://review.openstack.org/18605516:03
clarkbjeblair: >=1.0,<2.0 iirc16:03
openstackbug 1459322 in tooz "stable/juno broken with zake 0.2.2 release" [Undecided,New] https://launchpad.net/bugs/145932216:03
clarkbjeblair: global requirments should say16:03
*** julim has joined #openstack-infra16:03
*** dustins_away is now known as dustins16:03
jeblairclarkb: was 1.0 the one that fixed the alembic thing?16:04
clarkbjeblair: 1.0 is the pbr release that added the semver enforcement16:04
jeblairglobal says pbr>=0.11,<2.016:04
jeblairso maybe it was 0.11 that fixed alembic?16:04
* jeblair digs16:04
mordredyes16:05
mtreinishjeblair: I just switched subunit2sql to >=1.0.0 but that's because I started using extras16:05
mordredjeblair: the fix for alembic is in 0.1116:05
*** ashleighfarnham has joined #openstack-infra16:05
jeblairmordred: i agree, found "Don't avoid egg-info quite so often16:06
jeblairin that tags history16:06
jeblairso yeah pbr>=0.11,<2.0 seems reasonable16:06
*** oomichi has quit IRC16:06
*** jistr has quit IRC16:07
lxslihttps://review.openstack.org/185620 https://review.openstack.org/18563216:07
lxslimtreinish: ^-- couple of elastic recheck queries16:07
openstackgerritJames E. Blair proposed stackforge/gertty: Update PBR requirement to >=0.11  https://review.openstack.org/18605716:07
jeblairmordred: ^16:07
*** amitgandhinz has quit IRC16:08
clarkbpuppet is reenabled on nodepool.o.o16:08
mtreinishlxsli: nice, 98k hits on that second one16:08
openstackgerritMerged openstack-infra/elastic-recheck: Add query for zake 0.2.2 stable break bug 1459322  https://review.openstack.org/18605516:09
openstackbug 1459322 in tooz "stable/juno broken with zake 0.2.2 release" [Undecided,New] https://launchpad.net/bugs/145932216:09
mordredjeblair: ++16:09
openstackgerritMerged stackforge/gertty: Update PBR requirement to >=0.11  https://review.openstack.org/18605716:09
*** rcarrillocruz has left #openstack-infra16:09
pleia2good morning16:09
*** mpavone has quit IRC16:10
*** jlanoux has quit IRC16:10
openstackgerritTravis Tripp proposed openstack-infra/irc-meetings: Add Search Team Meeting  https://review.openstack.org/18606016:10
anteayamorning pleia216:10
*** asselin__ has joined #openstack-infra16:11
*** stevemar has joined #openstack-infra16:11
*** amitgandhinz has joined #openstack-infra16:11
mtreinishttx, devananda: it looks like python-ironicclient's lp isn't a member of the openstack group16:12
*** asselin_ has quit IRC16:12
SpamapSmorganfainberg: when you have like, 20 minutes, we should actually discuss why the generic dogpile.cache key generator can't be made to work as well as the one we have in shade.16:12
*** harlowja_at_home has quit IRC16:12
mtreinishor at least e-r tests are complaining about it: http://logs.openstack.org/20/185620/2/check/gate-elastic-recheck-tox-queries/e8d8b3c/console.html#_2015-05-26_14_40_17_06416:12
mtreinishlxsli: ^^^16:12
*** unicell1 has quit IRC16:12
morganfainbergSpamapS: let me write some16:12
morganfainbergEmails and I have time.16:12
morganfainbergSpamapS: :)16:12
ttxmtreinish: will fix it if this page ever loads16:13
pabelangerttx, With eavesdrop.o.o do you see adding the ability to have individual iCals for teams?  On top of the single iCal for all teams?16:13
morganfainbergSpamapS: maybe pre lunch but post breakfast?16:13
ttxpabelanger: I see it. tonyb suggested added that capability to yaml2ical16:13
*** freerunner has quit IRC16:13
mordredjeblair: if you have a sec: https://review.openstack.org/#/c/186048/ is needed for dib-nodepool to get us past the current sad16:13
*** jogo has quit IRC16:13
lxslimtreinish: nice spot16:13
*** freerunner has joined #openstack-infra16:13
clarkbjeblair: mordred https://review.openstack.org/#/c/186034/ too16:14
pabelangerttx, Nice. Good to know16:14
fungipabelanger: if you get bored, you could just submit a change to generate them, and another to link them in the template16:15
pabelangerfungi, so, patches welcome? ;)16:15
clarkbmordred: also your nodepool change is still failing pep8 due to the undefined name16:15
*** alexsyip has joined #openstack-infra16:16
mtreinishttx: thanks16:16
fungipabelanger: that is an excellent distillation of what i said, yep! ;)16:16
pabelangerhehe16:16
*** abregman has quit IRC16:16
openstackgerritMatt Riedemann proposed openstack/requirements: Raise cap on tooz to <0.13  https://review.openstack.org/18606516:16
*** hdd has joined #openstack-infra16:16
*** nikeshm has joined #openstack-infra16:16
ttxmtreinish: should be good now16:17
*** jogo has joined #openstack-infra16:17
*** achanda has joined #openstack-infra16:17
*** rbrooker_ has quit IRC16:17
mordredclarkb: yah - I'm looking in to that now. there's something else broken after I fixed that16:18
pleia2ooh bug 1226855 is getting some attention again, hooray for the rise of containers? :)16:18
openstackbug 1226855 in lxc (Ubuntu) "Cannot use open-iscsi inside LXC container" [Wishlist,Confirmed] https://launchpad.net/bugs/122685516:18
openstackgerritJames E. Blair proposed openstack-infra/irc-meetings: Remove Install Guide Update Meeting  https://review.openstack.org/18606616:18
pleia2seems some folks at canonical want to raise priority, so fingers crossed that serge can make time for it now16:18
*** fhubik_afk is now known as fhubik16:18
*** gokrokve has quit IRC16:19
SpamapSmorganfainberg: just when you have some time. I think I understand, but I suspect you have it in a much more succinct package than my ramblings.16:19
openstackgerritJames E. Blair proposed openstack-infra/irc-meetings: Remove Networking Guide Update Meeting  https://review.openstack.org/18606716:19
openstackgerritSean M. Collins proposed openstack-infra/irc-meetings: Remove the IPv6 subteam meeting  https://review.openstack.org/18606816:19
*** gokrokve has joined #openstack-infra16:19
morganfainbergSpamapS: sure! I think I know what the issue(s) are will ping you post breakfast.16:19
openstackgerritMerged openstack-infra/elastic-recheck: gate-heat-python27 #1458399  https://review.openstack.org/18563216:19
nikeshmhi,i am tring to run dsvm-tempest-ful job in jenkins slaves,but its failing from yesterday night16:20
nikeshmhttp://paste.openstack.org/show/240214/16:20
*** vahidh has joined #openstack-infra16:20
openstackgerritMerged openstack-infra/system-config: Update team structure and add council  https://review.openstack.org/18281116:21
*** gokrokve has quit IRC16:21
*** derekh_ has joined #openstack-infra16:21
nikeshmasselin: hi16:21
asselin__hi16:21
asselin__nikeshm, look at your devstack.log file16:22
clarkbpleia2 I saw that, may make testing nested openstacks much simpler in the near future16:22
pleia2clarkb: yep16:23
mordredclarkb: so - I fixed a couple of things and now am hitting a unit test fail that I think I need help with16:23
asselin__line 226 of your post says to look there16:23
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Consume os-client-config directly  https://review.openstack.org/18597216:23
asselin__nikeshm, logs in logs/devstacklog.txt.gz16:23
openstackgerritMerged openstack-infra/irc-meetings: Remove Install Guide Update Meeting  https://review.openstack.org/18606616:23
clarkbmordred, ok I have the tea kettle on but can dig in once my tea is madr16:23
mordredclarkb: I'm still looking- but a second set of eyes would now be helpful16:23
openstackgerritMerged openstack-infra/irc-meetings: Remove Networking Guide Update Meeting  https://review.openstack.org/18606716:23
mordredclarkb: awesome16:23
*** ashleighfarnham has quit IRC16:23
*** claudiub has quit IRC16:24
mtreinishttx: cool, thanks16:24
*** derekh has quit IRC16:25
openstackgerritMerged openstack-infra/elastic-recheck: gate-python-ironicclient-python34 #1454051  https://review.openstack.org/18562016:26
lxslimtreinish: thanks :)16:27
akrivokaanyone have any idea how to fix the jenkins grenade failure all stable/kilo patches are getting? e.g. https://review.openstack.org/#/c/183656/16:28
zaromorning16:28
Davieyakrivoka: it is in progress16:28
openstackgerritAnita Kuno proposed openstack-infra/irc-meetings: Correct an error in a meeting description.  https://review.openstack.org/18607516:29
anteayamorning zaro16:29
Davieyakrivoka: details on -dev ML and https://etherpad.openstack.org/p/stable-tracker , being discussed in #openstack-stable16:29
akrivokaDaviey: thanks for the info!16:29
*** fawadkhaliq has quit IRC16:30
*** julim has quit IRC16:31
nikeshmasselin_ : oh i was not getting this file in my logs directory http://115.119.152.41/dothill-logs/logs/$LOG_PATH/logs/  ,BTW,just curious to know if anything break in upstream,becuase i was able to run this earlier but from yesterday night its not working16:32
*** dannywilson has quit IRC16:32
asselin__nikeshm, not sure why you have $LOG_PATH in your url...something not right.16:36
asselin__nikeshm, my system's running..you'll need that log file to see why your ci is failing16:36
pabelangersigh, I posted this into the wrong channel. So I apologizes for the duplication if you are in the other channel.16:36
pabelangerWe talked a little about this yesterday, revolving around having nodepool create build environments.  I was envisioning a new prepare_node_pkg.sh script (calling slave.pp and slave_pkg.pp), which would be its own package image, to allow jenkins to run the tooling.  This was simple to remove some of the dependencies that prepare_node_base.sh adds.  However, I'm not apposed to using prepare_node_bare.sh and adding into it build16:36
pabelanger dependencies, however seems to add more overhead into that image.16:36
*** maishsk_afk has joined #openstack-infra16:37
anteayattx: we seem to have lost the ability to link from the meeting wikipage to individual meeting wikipages with agendas16:37
mordredpabelanger: yeah - I think we want to avoid having multiple images - and instead just have an ubuntu-trusty image and a fedora-22 image, etc.... all of those images should have nothing installed but a maximal set of things pre-cached16:38
openstackgerritRichard Pijnenburg proposed openstack-infra/jenkins-job-builder: Adding tests surrounding handling of defaults values  https://review.openstack.org/18566116:38
clarkbmordred: it looks like the provider manager thread is dying for some reason16:38
clarkbmordred: unfortunately it doesn't seem to haev logged the reason?16:38
mordredclarkb: yah. that's how it's looking to me too16:39
electricalzaro: uploaded new patch set with hopefully a better explanation of the bug :-)16:39
clarkbmordred: I am going to guess somethingrelated to the chnage in https://review.openstack.org/#/c/185972/3/nodepool/provider_manager.py16:39
anteayattx: I can find the information in the history of the meetings wikipage, but where should the links to meeting agendas live?16:39
mordredclarkb: yah. let me try something16:39
jeblairanteaya: it looks like you can link within the meeting descriptions... at least some of them do16:39
anteayajeblair: do you see an example? I don't see one yet16:40
jeblairanteaya: barbican links to blueprints16:40
pabelangermordred, Okay, so for the moment I am actually using Ubuntu 14.04 to do sbuild(debian) and mock(fedora) builds; for the POC.  However, like you said, I could see somebody wanting a fedora base image.  Getting a chroot into the base OS, is pretty easy.16:40
mordredpabelanger: yah. having a fedora base image - totally awesome16:41
jeblairclarkb, mordred: gah -- nodepool should never hit an exception that isn't logged, this is extremely troubling16:41
*** pblaho has quit IRC16:41
anteayajeblair: so they do16:41
clarkbjeblair: ya not sure how iut happened16:41
pabelangermordred, so, would you see a packaging-trusty and packaging-fedora image in nodepool then?16:41
mordrednope16:42
mordredpabelanger: I would see an ubuntu-trusty and fedora-22 image in nodepool16:42
mordredI do not think we want/need a different image for packaging16:42
pabelangerokay16:42
jeblairyeah, we really want to drive the number of images down to minimal16:42
*** unicell has joined #openstack-infra16:42
flip214can somebody please explain me the meaning of "Cloning not allowed in this configuration" for https://review.openstack.org/#/c/178049/ (devstack plugin to install drbd)? I understand the literal meaning, but not the reason or how to fix it.16:43
openstackgerritMerged openstack-infra/shade: Set metadata headers on object create  https://review.openstack.org/18604816:43
jeblairclarkb: why do you think it died?16:43
*** jtomasek has quit IRC16:43
*** julim has joined #openstack-infra16:43
jeblairflip214: you may need to add it to PROJECTS in the job configuration; there should be other examples related to devstack plugins16:43
pabelangerso, with that in mind, how could pre-caching of chroots happen atop of ubuntu-trusty?  If I understand properly, once the jenkins slave came online, the chroots would be built then.16:44
openstackgerritAnita Kuno proposed openstack-infra/irc-meetings: Correct an error in a meeting description.  https://review.openstack.org/18607516:44
clarkbjeblair: my hunch is an uncaught exception in the main runloop of the thread16:44
pabelangerunless we add packaging bits into the ubuntu-trusty dib?16:44
flip214jeblair: I'll try to find that, thanks for the hint.16:45
*** soren_ has quit IRC16:45
mordredpayes16:45
mordredgah16:45
*** julim has quit IRC16:45
greghaynes'morning16:45
mordredpabelanger: yes. I think bulding the chroots in the ubuntu-trusty dib is a totally decent idae16:45
greghaynesclarkb: we broke nodepool?16:46
openstackgerritAnita Kuno proposed openstack-infra/irc-meetings: Correct an error in a meeting description.  https://review.openstack.org/18607516:46
jeblairclarkb: are we running head?16:46
pabelangermordred, okay. let me see how that would look16:46
clarkbgreghaynes: not really, things that didn't work in the past continue to not work. The only real regression is the config reloading16:46
clarkbjeblair: yes16:46
mordredpabelanger: we'll want to take care to not leave the tools needed to build the chroot in the image itself, since that would pollute the minimal image16:46
*** otter768 has joined #openstack-infra16:46
pabelangermordred, right, that is what I was thinking too16:46
jeblairclarkb: did you do a thread dump and see the thread was absent?16:46
clarkbjeblair: note its only broken on mordreds change16:46
mordredpabelanger: it's possible taht you may want to brainmeld with greghaynes to figure out how the dib caching mechanism might be used, and/or also how the building of the chroot might be done image-host side16:46
clarkbHEAD is fine16:47
clarkband so is production16:47
greghaynesoh, dib things16:47
jeblairoh, you're running 185972 in local testing and seeing it break16:47
clarkbsee https://review.openstack.org/#/c/185972/316:47
mordredjeblair: yah - this is us trying to debug why my change is breaking all the tests16:47
jeblairi missed that part :)16:47
fungithe pyyaml.org trac has been overrun by commentspam16:48
fungithat's unfortunate16:48
*** radez is now known as radez_g0n316:48
clarkbhrm the run method has a catch all Exception catcher16:48
*** annegentle has joined #openstack-infra16:48
clarkboh wait I lied16:48
pabelangermordred, one issue to keep in mind is tools like mock require user group changes, but that _could_ be pre-baked into dib.  EG: adding jenkins to the mock group.16:49
pabelangerQ: Is there any existing job that launches puppet before actually doing a gate test?16:50
*** timcline has quit IRC16:50
openstackgerritClark Boylan proposed openstack-infra/nodepool: Debug dying task managers  https://review.openstack.org/18607916:51
clarkbmordred: ^ I think that will log the info you need16:51
*** yamahata has quit IRC16:51
greghaynespabelanger: do sbuild and mock already support feeding them a pre-made chroot?16:51
clarkbpabelanger: you have sudo, you should affect that change at job run time if necessar16:51
*** otter768 has quit IRC16:51
flip214jeblair: in the infra-project-config, do you mean in jenkins/jobs/….yaml an 'export PROJECTS="..."' ?16:51
pabelangergreghaynes, define feeding?  building a chroot outside of mock or sbuild tooling?16:52
flip214because we don't even have a job file in there yet.16:53
greghaynespabelanger: im just trying to figure out, without dib how would you cache a chroot and have those tools use it16:53
*** annegentle has quit IRC16:53
*** emagana has quit IRC16:53
pabelangergreghaynes, right now, I do mock -r flavor init, which bootstraps my chroots16:53
openstackgerritJames E. Blair proposed openstack-infra/project-config: Upload ical output to log server  https://review.openstack.org/18608116:54
Clintsbuild can operate on a chroot that you've built16:54
Clintor an lvm snapshot or whatever16:54
mordredgreghaynes, pabelanger: naive/stupid version is an element that does "yum install mock ; mock -r flavor init ; yum remove mock"16:54
pabelangerthen, once my job launches from jenkins, I'd re-update the chroot (ensuring the latest) then do builds16:54
greghaynesI wonder if we should make the chroot with dib16:54
jeblairflip214: hrm.  let me look.16:55
pabelangermordred, Right, something like that16:55
clarkbI think it would be good to see the non optimized version of the job16:55
clarkbthen worry about image changes16:55
clarkbbecause right now we all seem to be doing a lot of hand waving at each other16:56
greghaynesYea. Sounds like it shouldnt be challenging to turn that into an element after the fact16:56
greghayneswhich will make it happen at build time16:56
clarkband having a concrete job that is running would help all involved16:56
mordredclarkb: AHA!16:57
greghaynesmordred: Re: nodepool consuming os-c-c directly - can we not make that a shade feature?16:57
mordredclarkb: I have foudn it I think16:57
*** amotoki_ has joined #openstack-infra16:57
mordredgreghaynes: what?16:57
clarkbmordred: did my change help? I am waiting for jenkins to finish running the test16:57
mordredgreghaynes: (moar words please)16:57
mordredclarkb: no - I found it with print statements16:57
mordredclarkb: I think16:57
greghaynesmordred: where shade will reload clients if the clouds.yaml it is using changes16:57
pabelangergreghaynes, Right, okay. So let me see how to add them into dib for ubuntu-trusty16:57
pabelangerI need to get my local nodepool running again16:58
*** claudiub has joined #openstack-infra16:58
mordredgreghaynes: _possibly_ ... although we need to think about how to interface with other things like nodepool that might also be managing that16:58
greghaynesmordred: oh, right, were exposing clients entirely16:58
mordredgreghaynes: so for now doign this seems safer - but I think it's TOTALLY something we should think about a design for16:58
greghaynesmordred: ok, yea, nodepool for now16:58
*** asahlin has quit IRC16:59
greghaynespabelanger: Yea, I think what clarkb said is probably right though - first iteration as a normal old job should work fine and be a ton easier to debug / figure out16:59
*** ashleighfarnham has joined #openstack-infra17:00
jeblairflip214: to use the existing templates, your project should have been called "stackforge/devstack-plugin-drbd"17:00
*** asahlin has joined #openstack-infra17:00
clarkbmordred: nope no "manager died." string in the test output, this is weird17:00
jeblairflip214: since you named it "stackforge/drbd-devstack" you will need to make your own job definitions17:00
clarkbmordred: curious to see what you have found out, otherwise my next step may be pdb17:00
jeblairflip214: see this line: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n193417:01
pabelangergreghaynes, clarkb: yes, I agree.  Just making sure we are all on same page17:01
clarkbmordred: oh except I see a none type is not callable exception in your provider manager hange17:01
*** amotoki_ has quit IRC17:01
jeblairflip214: you could also rework that template to be more flexible for project names17:01
flip214jeblair: can I rename a project? I didn't know about that requirement.17:02
*** HeOS has quit IRC17:02
clarkbmordred: the config value passed to shade there is supposed to have a callable member17:02
jeblairflip214: yes, it will take a few weeks.17:02
pabelangerSo, next step, installing packaging tool bits. To me, having puppet do that makes sense, eg: install mock, sbuild.  However, I don't know if we have any jobs that are launching puppet as the first step for a jenkins slave?17:02
pabelangerother wise, we put the bits into a bash script17:02
clarkbmordred: http://paste.openstack.org/show/240318/17:02
pabelangerwhich, is meh for me17:02
greghaynespabelanger: most people seem to just have a bash script in a job17:02
*** deepakcs has quit IRC17:02
greghaynespabelanger: there is work on using bindep as a general solution, but thats not done yet17:03
clarkbnot using puppet is more friendly to the masses17:03
flip214jeblair: ;[17:03
clarkbbut if puppet is the best tool then ok17:03
pabelangergreghaynes, right17:03
*** hemna is now known as hemnafk17:03
mordredclarkb: right. so - the question is how is self.provider existing without getting a cloud_config object on it17:03
flip214jeblair: would you recommend creating a new project with the correct name instead?17:03
mordredclarkb: which may just mean that a fake isn't faking enough things17:03
clarkbmordred: no I think the type is wrong entirely17:03
*** shardy has quit IRC17:03
clarkbmordred: oh wait its none so duh17:04
mordredright17:04
mordrednow - you may be right about the type ...17:04
pabelangerclarkb, well, to me puppet makes it easy to add support for more OS's. But that is my personal opinion.  Not a deal breaker for me to use a bash script launched from jenkins job.17:04
clarkbpabelanger: sure except tomorrow you will be field questions from all sides on why you didn't use chef/ansible/salt/cfengine/etc17:05
pabelangerright17:05
fungiflip214: please don't create new repos to correct misnaming. if it really needs to be renamed then put it in the queue to go through the renaming process, but in the meantime adding a new job-template or making the existing one more flexible should be a much faster solution17:05
clarkbpabelanger: and in a couple months there will be a separate branch for all the builds to implement in ^17:05
clarkbpabelanger: if you just use bash no one seems to complain, its really weird17:05
jeblairflip214: no.  i'd recommend: 1) update the template to allow repo names that don't fit the pattern (do this in any case to keep things moving for now).  2) optionally rename your repo if you want to be like the others.  3) if you do 2, consider putting the template back after the rename17:05
pabelangerclarkb, Ya, i see that.17:05
pabelangerokay, so for the simple fact we don't run puppet apply on jenkins jobs launching, is likely a reason not to use puppet.   However, it _could_ be added17:06
flip214jeblair: fungi: thank you for your help.17:06
flip214If only I knew all the interrelations and dependencies here...17:07
clarkbmordred: my guess is that we don't have clouds.yaml in place and without it oscc doesnt dothe right thing merging items?17:07
mordredoscc should TOTALLY do the right thing without a clouds.yaml in place17:07
clarkbmordred: then get_one_cloud returns None17:07
mordredeverythign in the universe breaks if it doesn't17:07
*** dboik has quit IRC17:07
flip214thanks a lot, will try that when I'm back.17:07
jeblairflip214: you're welcome; sorry that didn't get caught in review.17:08
*** fhubik is now known as fhubik_afk17:08
mordredget_one_cloud will overlay the passed in parameters on top of an empty cloud if there is no clouds.yaml17:08
clarkbmordred: hrm ya get_one_cloud returns a CloudConfig always17:08
*** sshnaidm has quit IRC17:09
*** annegentle has joined #openstack-infra17:10
mordredclarkb: so - I'm getting a None returned from self._getClient() in provider_manager17:10
mordredclarkb: inside of resetClient17:10
mordredoh - hang on - might be debugging bong17:11
*** dboik has joined #openstack-infra17:12
*** fhubik_afk is now known as fhubik17:12
*** ociuhandu has joined #openstack-infra17:12
*** tiswanso has quit IRC17:12
*** baoli_ has quit IRC17:12
*** annegentle has quit IRC17:13
clarkbmordred: random thought oscc should be a dep in requiremnts.txt17:13
*** markus_z has quit IRC17:14
*** sdake_ has joined #openstack-infra17:14
*** julim has joined #openstack-infra17:14
pabelangergreghaynes, are the puppet infra bits will on the dib after the puppet element is run?17:14
clarkbmordred: also the inequality check on CloudConfig objects isn't going to work I don't think17:14
pabelangerI _think_ so17:14
mordredclarkb: yes. I agree17:15
mordredclarkb: why not?17:15
clarkbpabelanger: ya we don't clean up /etc/puppet/modules or the puppet install17:15
mordredclarkb: well, I mean, it will once my patch to add equality support to CloudConfig objects lands17:15
clarkbmordred: bceause its just going to compare objects but we always get a new object so it will always be unequal17:15
clarkboh is that part of the oscc stack?17:15
pabelangerclarkb, okay, that might work for my POC17:15
clarkbmordred: I wonder if the unittests installing last oscc release is related?17:16
clarkbmordred: maybe try manually installing your oscc changes to the tox venv then rerun17:16
mordredclarkb: yes. I have already installed tip of oscc in my local tox env17:16
*** dboik has quit IRC17:16
clarkbkk17:16
*** e0ne is now known as e0ne_17:16
*** kutija has joined #openstack-infra17:16
*** gokrokve has joined #openstack-infra17:17
*** sdake has quit IRC17:17
greghaynesclarkb: mordred do youall have a testname youall are using to debug?17:18
*** SumitNaiksatam has quit IRC17:18
mordredgreghaynes: I'm running nodepool.tests.test_nodepool.TestNodepool.test_dib_node17:18
clarkbgreghaynes: nodepool.tests.test_shade_integration.TestShadeIntegration.test_nodepool_provider_config17:18
*** puranamr has joined #openstack-infra17:18
clarkbI think ^ is the test that most closely targets the broken code17:18
greghaynesand thats just after checking out https://review.openstack.org/#/c/185972/3 ?e17:19
clarkbgreghaynes: ya17:19
clarkbmordred: ok, there is a type issue with the config var17:20
clarkbmordred: shade expects it to be an OpenStackConfig but get_one_cloud returns CloudConfig17:20
*** e0ne_ is now known as e0ne17:20
mordredclarkb: aha! thank you17:20
clarkbmordred: I think thats where the None type comes from17:20
fungijeblair: mordred: clarkb: SergeyLukjanov: jhesketh: pleia2: i'm going over https cert expirations coming up in july... jenkins.o.o is among them. any objections to me replacing that with a self-signed cert like we have on jenkinsNN.o.o servers?17:21
clarkbbecause CloudConfig.get_one_cloud is none and thats not callable17:21
openstackgerritMatt Riedemann proposed openstack/requirements: Add tooz to projects.txt  https://review.openstack.org/18609017:21
mordred++17:21
pleia2fungi: no objections17:21
clarkbfungi: the only concern there is possibly that we use that node for "trusted" builds, but its easy enough to confirm validity of artifacts otherwise (and https on jenkins.o.o likely doesn't help that too much)17:22
*** fawadkhaliq has joined #openstack-infra17:22
*** krtaylor has quit IRC17:22
* jogo wonders how https://review.openstack.org/#/c/185365/ is different then all the other CI related stuff we have17:22
jeblairfungi: nope17:22
greghaynesso, are the nodepool tests actually finding fails?17:22
greghaynesthis is crazypants17:22
*** puranamr has quit IRC17:22
clarkbgreghaynes: yup17:22
clarkbthe test suite is actually reasonably robust at this point. Not perfect but our coverage is decent17:23
*** fhubik has quit IRC17:23
fungijogo: looks like a nih zuul?17:23
clarkbfungi: jogo yes thats what it is17:23
greghaynesmaybe request a project rename?17:23
* greghaynes tries to be less snarky17:23
clarkbI talked to them briefly about it, they didn't want to have to run jenkins (you don't have to), wanted lightweight python proess (we have that), so was trying to stear towards zuul but :/17:23
fungii find it amusing that changes proposed to the nih zuul will be coordinated by... actual zuul ;)17:24
*** yamahata has joined #openstack-infra17:24
jogofungi: hehe17:24
*** puranamr has joined #openstack-infra17:24
*** fhubik has joined #openstack-infra17:24
jogothanks, just checking if it was just me or not17:24
jeblairfungi: well, i mean, they would theoretically if we approved that change17:24
clarkbsteer? english is hard17:24
jeblairi am inclined to -2 it.17:25
*** e0ne has quit IRC17:25
clarkbI think one is a cow the other is a driving direction thing17:25
*** yamamoto has joined #openstack-infra17:25
*** baoli has joined #openstack-infra17:26
*** tiswanso has joined #openstack-infra17:26
mordredjeblair: I'm inclined to abstain indefinitely17:26
fungiwould be nice to see the rally-ci authors just help us solve their use case with zuul17:26
jeblairmordred: why abstain?  if you do, someone will approve that change.17:26
clarkboh I remember the other objects was the complicated config for zuul17:26
clarkber other objection17:27
jeblairclarkb: where was that objection voiced?17:27
clarkbI think they are interested in a daemon that does the one thing that they want.17:27
*** fawadkhaliq has quit IRC17:27
*** dizquierdo has quit IRC17:27
clarkbjeblair: in a rally meeting a few weeks back, I can dig it up on eavesdrop17:27
jeblairwell, i mean, they could have talked to us17:27
*** mrunge has quit IRC17:27
fungii find rally confusing and hard to implement. i'm not going to let them know that, but i'm going to go ahead and write a replacement17:28
clarkbhttp://eavesdrop.openstack.org/meetings/rally/2015/rally.2015-04-21-17.02.log.txt there17:28
*** ParsectiX has joined #openstack-infra17:29
*** yamamoto has quit IRC17:30
openstackgerritMerged openstack-infra/project-config: Clean up run-tox.sh to make it easier to reuse  https://review.openstack.org/16141417:30
fungii already -2'd. rally is an official openstack project. ci systems are well outside the scope they presented to the tc17:30
*** alexsyip has quit IRC17:30
*** emagana has joined #openstack-infra17:31
clarkbmordred: I wonder if CloudConfig should go away and OpenStackConfig be the object17:31
jeblairfungi: well, their change had an error.  it was _mostly_ proposing stackforge, except the acl was mistakenly in openstack/17:31
*** puranamr has quit IRC17:31
clarkbmordred: and for backward compat just have CloudConfig inherit OpenStackConfig with no overrides17:32
jeblairfungi: even if we assume it's going to stackforge, i'm still opposed to it -- it duplicates a project in the infra program, which is official.17:32
clarkbmordred: or monkey patch it17:32
*** puranamr has joined #openstack-infra17:32
clarkbmordred: otherwise shade has to know about this in weird weays17:32
jeblair(also, i'm becoming increasingly -1 on new stackforge projects)17:32
jeblairfungi: i have -2d it on the grounds they should work with zuul.17:32
*** alexsyip has joined #openstack-infra17:33
clarkbmordred: it isn't clear to me how an openstack config is different than a cloud config within in the context of openstack client config. They should be the same?17:33
mordredclarkb: an OpenStackConfig represents the total amount of config found on the system. a CloudConfig is the config of one cloud17:34
clarkbmordred: hrm, in that case I think shade is expecting the wrong config object type? since thats config for one cloud17:35
mordredyes. I'm fixing right now17:35
clarkbcool17:35
fungii agree they should at least discuss zuul-related options with us and clear up any misconception before running off to redesign it in a vacuum, but remain unconvinced that we should reject stackforge projects only on the grounds that they duplicate functionality of existing official projects (no matter how hypocritical the choice)17:35
*** julim has quit IRC17:35
jogoclarkb: what a classic line "the only problem with zuul is too complex for configuration" replace zuul with anything else17:35
jogoX is too big and complex, lets re-write it as Y and make it simple. 2 years later Y is too big and complex17:36
fungibut yes if we start down the road of "all projects in our gerrit must join the big tent or get out" then the waters get a little more muddy there17:36
jeblairfungi: fungi "17:36
jeblairfung: "Where it makes sense, the project cooperates with existing projects rather than gratuitously competing or reinventing the wheel"17:36
dstufftjogo: kill all configuration17:36
jeblairfungi: is the tc stance on such things17:36
*** alexsyip has quit IRC17:37
mordredjeblair: otoh, the TC has explicitly declined governance on stackforge. we have not yet gotten the agreement that stackforge is only a historical bug17:37
jeblairfungi: and yes, i think we should both apply that to stackforge (stackforge is not the place to be in openstack while escaping the requirements of being in openstack)17:37
*** yfried has joined #openstack-infra17:38
mordredjeblair: I think this might be an excellent motivating factor to get that agreement17:38
fungijeblair: i'd love to see us start applying that to stackforge projects too, yes. do we need a tc resolution to grandfather existing stackforge projects into the big tent, or just shut it down officially where new project requests are concerned and grandfather the original ones into continuing their own wild west?17:38
jogodstufft: everything should just have this UI: https://i.imgur.com/iMQcodA.jpg17:38
jeblairmordred: the TC has declined governance, but it also created stackforge17:38
mordredjeblair: yes indeed17:38
dstufftjogo: unironically agree17:38
dstufftjogo: too bad reality17:38
mordredjeblair: I agree with your position - I just think we need to actually get the TC to decide it17:38
jogodstufft: silly reality17:38
*** julim has joined #openstack-infra17:39
*** hemnafk is now known as hemna17:39
jeblairmordred: so while i agree with you on moving forward for clarity, i also think that even in the current murky situation i think we can apply some judgement and say "it does not make sense for the project to negate its own goals by supporting this"17:39
dstufftjogo: I've recently taken a view point that every configuration option should be thought of as some sort of "reality debt" or something, and projects (not openstack specific, just projects in general) should strive to eliminate them where possible. Too often things, especially OSS things, use configuration as an excuse tnot to make a decision17:39
mordredjeblair: totally17:40
mordredjeblair: also, I reserve the right to -2 things because and only because I have exercised personal judgement17:40
openstackgerritgordon chung proposed openstack/requirements: Pin pyngus version as python-qpid-proton is not in pypi  https://review.openstack.org/18609817:40
jeblairmordred: indeed17:40
mordredwithout ANY policy backing me up17:40
jogodstufft: I don't disagree but the devil is in the details17:40
mordredI am more than happy if people want to complain17:40
dstufftjogo: for sure :)17:40
jeblairmordred, fungi: i will work on a resolution to clarify the future of stackforge, but i think the -2 should stand on this because i think the goals we have set as a community are important.17:41
mordredjeblair: agree17:41
dstufftjogo: maybe it was only me that the "OSS uses configuration to avoid making a choice" thing was a realization/revelation and not completely obvious :)17:41
*** sputnik13 has joined #openstack-infra17:41
jogodstufft: so the inverse is a problem too: X doesn't support my use case and isn't willing to so I now will go and re-invent the wheel but with more knobs17:41
*** teran_ has quit IRC17:42
devanandaquestion on reviewstats -- for projects that have discrete review teams for discrete subprojects, what's the best way to represent that?17:42
fungijeblair: i'm willing to go along with whatever the tc decides on it, but also completely support your autonomy to -2 anything you judge problematic17:42
devanandaeg, ironic has ironic-core, ironic-python-agent-core, ironic-discoverd-core, bifrost-core, and if you count drivers, also proliantutils-core17:43
devananda(those are all gerrit groups)17:43
dstufftjogo: yea :/17:43
mordredI think there shold be One Clear Default value that makes sense for anything in a config17:44
fungidevananda: it's tricky. bknudson started https://review.openstack.org/139727 in an attempt to use my who-approves.py to dynamically generate/update review teams17:44
mordredrequiring config for basic operation is fail17:44
jeblairdstufft, jogo: to that end, our configuration of zuul is exceedingly complex.  however, the basic configuration of zuul is quite simple, and, mordred, yeah, the defaults are, i think, sane.17:44
mordredhowever, exposing config to solve jogo's thing17:44
mordredjeblair: ++17:44
*** SumitNaiksatam has joined #openstack-infra17:44
jeblairour unit tests still test the same configuration we ran with zuul in 3 years ago.17:44
devanandafungi: oh that looks like fun :-/17:44
jogo++ for sane defaults, this really sounds like fear of using other peoples code because it is unknown17:45
fungidevananda: that is to say, it's currently not a solved problem. the infra projects currently just mark the infra-core team as core reviewers in reviewstats config, which is suboptimal for the reasons you mention17:45
devanandafungi: but doesn't address the question - it assumes an answer17:45
*** dustins has quit IRC17:45
*** redixin has joined #openstack-infra17:45
devanandayea. there are and will continue to be people with approval rights on subprojects, and more projects are adopting this "delegation of trust" model17:46
bknudsonfungi: devananda: I didn't get too far on that... the issues that I ran into are mentioned at the top of the part: https://review.openstack.org/#/c/139727/1/tools/who-approves/reviewstats_projects.py17:46
*** ildikov has joined #openstack-infra17:46
fungidevananda: the real problem, i think, is that reviewstats assumes one particular team structure when across openstack we have a variety of team composition models17:46
jeblairjogo: and yeah, reading the meeting logs they clearly didn't know about major features of zuul.  i honestly think they could do what they want with zuul with a minimum of fuss.  which clarkb attempted to convey but they were not receptive.17:46
bknudsonand it's the subprojects part that is not encoded anywhere.17:46
devanandafungi: yes17:46
jeblairthey said they would look into it and regroup at the next meeting.  they did not.17:46
*** koolhead17 has quit IRC17:46
devanandafungi: and different projects are adopting different approaches to suit their unique/evolved cultures17:46
bknudsonHopefully people are using reviewstats as a guide and not as having all the info.17:48
fungidevananda: i'm also not sure that it even makes sense to try to enumerate all the ways in which subteams might be structured and interrelated. the closest we can come with any accuracy is introspecting gerrit to identify accounts with workflow +1 privs on a per-repo basis17:48
fungiwe can sort of infer team relationships from gerrit groups17:49
greghaynesmordred: hah, you renamed cloud to profile already17:49
*** dboik has joined #openstack-infra17:49
mordredgreghaynes: :)17:49
devanandafungi: thus my question :)17:49
*** emagana has quit IRC17:50
*** Sukhdev has quit IRC17:50
*** asselin__ has quit IRC17:51
devanandafungi: for example, neutron seems to be splitting along the driver boundary, with one team per driver. OTOH, Ironic teams are split along API boundaries, with subteams for different services17:51
fungidevananda: but if we redesign, it probably makes the most sense to analyze these sorts of team structures objectively (things we can actually query/measure), rather than based on some assertion they make about their structures (which may be based entirely on human judgement and therefore can't be easily represented via metadata)17:51
*** ivar-lazzaro has joined #openstack-infra17:51
devanandafungi: I'm not sure yet what the desired representation even is, to be honest17:52
devanandaso let's not redesign anything just yet :)17:52
*** dannywilson has joined #openstack-infra17:52
fungidevananda: third data point: the infra-council https://review.openstack.org/18281117:52
mriedemfungi: we need a ninja merge to unblock all of stable/juno and stable/kilo17:53
*** chlong has quit IRC17:53
devanandahehe17:53
jogojeblair: agreed17:53
mriedemfungi: clarkb: jeblair: https://review.openstack.org/#/c/186084/ needs ninja merge17:53
mriedemdims_: dhellmann: harlowja_: ^17:53
fungimriedem: do you have a brief summary of the deadlock?17:53
mriedemfungi: yeah, https://etherpad.openstack.org/p/stable-tracker17:54
mriedemtop thing under kilo17:54
fungithanks17:54
mriedemand https://bugs.launchpad.net/python-tooz/+bug/145932217:54
openstackLaunchpad bug 1459322 in tooz "stable/juno broken with zake 0.2.2 release" [Undecided,New]17:54
mriedemoh crap17:55
fungimriedem: i'm having a hard time deriving a summary from that17:55
mriedemhttps://review.openstack.org/#/c/186084/ depends on a thing that17:55
*** sdake has joined #openstack-infra17:55
*** dannywil_ has joined #openstack-infra17:55
mriedemfungi: tooz has uncapped requirements on kazoo and zake in 0.12 which is what juno pulls in17:55
mriedemzake 0.2.2 released today with a block on the version of kazoo that is used in juno17:55
fungithat helps17:55
mriedemso we have to cap kazoo and zake in tooz on stable/juno17:55
mriedemand then left the cap on tooz in g-r on stable/juno to get a new release of tooz with those caps17:55
mriedemhowever, i either need to squash or re-order https://review.openstack.org/#/c/186084/ and it's dependent change for .gitreview17:56
mriedemdhellmann: do you have a preference there? ^17:56
*** dannywilson has quit IRC17:56
mriedem*lift the cap17:56
devanandafungi: right - that's neat. and another good data point17:56
*** aduarte has joined #openstack-infra17:57
devanandafungi: in the meantime, I'd like to not have approvers on subprojects be not counted. if you have no opinions at the moment, i'll just throw something up that adds them, and we'll sort out details later17:57
*** timcline has joined #openstack-infra17:57
*** sarob has joined #openstack-infra17:58
*** sdake_ has quit IRC17:58
fungidevananda: i think it's entirely reasonable that if ironic is okay with the tracking disparity then their team in reviewstats represents all reviewers with approval on at least one repo within that project17:58
mordreddevananda: http://stackalytics.com/report/contribution/ironic-group/3017:59
mordreddevananda: tracks contributions across all of the ironic projects17:59
fungithat looks surprisingly like reviewstats mostly reimplemented inside of stackalytics. neat17:59
devanandaohyah. except it looks like it's pulling from gerrit groups and handling inheritance18:00
mordredyes. people said they wanted the data that was in reviewstats. stackalytics has had that for at least 1.5 years18:00
devanandaeg, it's showing TheJulia as a core, because bifrost-core is a subgroup of ironic-core18:00
devanandaneat18:00
mordredwell, actually18:00
mordredwhat it does is loko for people who have cast +2 or -2 votes18:00
mordredsince you cannot do that if you are not core18:00
devanandamordred: on which projects?18:01
mordreddevananda: on the projects defined in the governance repo as ironic's projects18:01
devanandaoh18:01
clarkbmordred: stackalytics should serve http://en.wikipedia.org/wiki/Loco_moco18:01
nikeshmhi18:01
* devananda forgets all about reviewstats18:01
devanandathis is actually smarter18:01
nikeshmin my nodepool vm18:01
nikeshmram size is always 6gb18:01
pleia2the stats are slightly different between stackalytics and reviewstats, so it's interesting to browse them both18:02
nikeshmeither i am selecting 8gb ram or 16 ram flavor18:02
*** dims_ has quit IRC18:02
fungiclarkb: the tiki bar across the street from me has that, but they call it a "garbage plate"18:02
nikeshmpatrickeast: hi18:02
fungigoes great with beer18:02
mordredclarkb: I want to eat that18:03
*** achanda_ has joined #openstack-infra18:03
*** dims_ has joined #openstack-infra18:03
*** fhubik has quit IRC18:04
*** sarob_ has joined #openstack-infra18:04
mriedemfungi: oooo18:04
mriedemupstate new york18:05
mriedemhome of the garbage plate :) or so i'm told18:05
* Clint squints.18:05
clarkbmordred: clearly we need to find a way to go to hawii18:05
pleia2++18:05
clarkbmordred: we can eat all of the amazing foods, but mostly just poke because it wins18:05
*** achanda has quit IRC18:05
*** e0ne has joined #openstack-infra18:06
*** e0ne is now known as e0ne_18:06
*** sarob has quit IRC18:06
* fungi loves poke salad18:06
*** ParsectiX has quit IRC18:06
fungiwhich is two things in two completely different places in the world, but both great18:07
*** e0ne_ is now known as e0ne18:07
clarkbwhen I go to hawaii I buy a tub of poke and it it three meals a day18:08
fungii would totally do that too18:08
pleia2tub of poke <318:08
*** Sukhdev has joined #openstack-infra18:08
fungimuch like i subsist on ceviche in other places18:09
clarkbfungi: and rum and coke18:09
openstackgerritSpencer Krum proposed openstack-infra/project-config: Add beaker-localhost and beaker-nodepool hypervisors  https://review.openstack.org/18575418:09
clarkbfungi: you can get that in hawaii too18:09
* fungi doubts the hawaiians call it a "cuba libre"18:09
openstackgerritK Jonathan Harker proposed openstack-infra/jeepyb: Swap logging levels for command and its output  https://review.openstack.org/18611018:10
fungibut as hawaii usually has decent sake on hand, i'd much prefer that over rum and coke18:10
clarkbfungi: I think they call it a mai tai and drop the coke18:10
openstackgerritMerged openstack/requirements: Block oslo.vmware 0.13.0 due to a backwards incompatible change  https://review.openstack.org/18574818:10
fungiheh18:12
clarkbthough I suppose mai tai is more than just lime18:13
*** Sukhdev has quit IRC18:13
Clintand a cuba libre is more than just rum&coke18:14
clarkbya thats there the lime comes in18:14
fungiindeed. it's also false hope and dashed dreams18:14
*** doug-fish has quit IRC18:14
* Clint sheds a solitary tear18:15
fungimriedem: did you come to a conclusion on the tooz change?18:15
clarkbfungi: mriedem rereading the tooz thing, couldnt we just update the pin in tooz and release?18:15
clarkbI am not grokking why it requires force merging18:16
*** krtaylor has joined #openstack-infra18:16
mriedemclarkb: in g-r?18:16
clarkbmriedem: and in tooz18:16
mriedemcircular dep18:16
clarkbmriedem: where?18:16
mriedemhttps://review.openstack.org/#/c/186065/18:16
mriedemthat's g-r on stable/juno18:16
mriedemwe need https://review.openstack.org/#/c/186084/ to release tooz 0.12.118:16
mriedemwhich will make https://review.openstack.org/#/c/186065/ pass18:16
clarkboh because we failed at pinning :/18:16
mriedemyes18:16
mriedemwe borked ourselves18:16
mriedemfungi: i'm good with https://review.openstack.org/#/c/186084/ if you're ok with merging that and the dependent change which is just a .gitreview update18:17
clarkbya, I see now18:17
mriedemfor the new branch18:17
mriedemdhellmann: are you happy with this now? https://review.openstack.org/#/c/186084/18:17
clarkbif we force merge the requirements update can we then let 186084 go through normally?18:18
fungimriedem: well, you need to do something about the parent commit first if we're going to circumvent gating18:18
mriedemclarkb: i think it has to be the other way around18:18
dhellmannmriedem: sorry, got side-tracked with another issue, catching up now18:18
mriedemclarkb: we need https://review.openstack.org/#/c/186084/ merged to tag and release tooz 0.12.118:18
mriedemclarkb: which will then make https://review.openstack.org/#/c/186065/ pass18:19
dhellmannmriedem: squashing the commits makes sense to me18:19
mriedemdhellmann: fungi: ok, squashing now18:19
clarkbmriedem: I think they will end up passing if either one merges first18:19
*** maishsk_afk has quit IRC18:19
clarkbmriedem: because you break the cycle in either case18:19
mriedemclarkb: not sure how that's true in the tooz patch18:19
clarkbut I have to think about it a bit more and figure out where the fails are18:19
*** BharatK has quit IRC18:20
clarkbmriedem: http://logs.openstack.org/84/186084/2/check/gate-tempest-dsvm-neutron-src-tooz/d369f8a/logs/devstacklog.txt.gz#_2015-05-27_17_50_16_85618:20
fungiunless the requirements change passing needs a tooz release first18:20
dhellmannit will18:20
dhellmannif we can get tooz itself unblocked, we can land the requirements changes cleanly18:21
clarkbdhellmann: right my suggestion is to force merge the requirements change then actually test that tooz can be unblocked18:21
mriedemdhellmann: done https://review.openstack.org/#/c/186084/18:21
dhellmannclarkb: interesting18:21
clarkbdhellmann: bceause force merging requirements and letting jenkins test tooz is preferable to making a tooz release then finding out we haven't fix it18:21
clarkbdhellmann: because if that happens then we do this dance again18:21
clarkbbut I am not sure if we can test it that way because requirements needs a new release of tooz in the first place18:22
*** dboik has quit IRC18:22
clarkb(consuming from releases vs consuming from git is what makes this complicated)18:22
mriedemclarkb: but http://logs.openstack.org/84/186084/2/check/gate-tempest-dsvm-neutron-src-tooz/d369f8a/logs/devstacklog.txt.gz#_2015-05-27_17_50_16_856 is not pulling in the capped zake from the change itself18:22
clarkbmriedem: ya which tells me there may be something else at fault18:22
mriedemso even if we force merge the g-r raised cap on tooz, there is no tooz 0.12.1 to test it18:22
*** kmartin has quit IRC18:22
clarkbmriedem: because thats the src job18:22
clarkbmriedem: right but the src job is the one failing18:23
mriedemclarkb: i think that's because ceilometer is pulling in tooz 0.1218:23
mriedemhttp://logs.openstack.org/84/186084/2/check/gate-tempest-dsvm-neutron-src-tooz/d369f8a/logs/devstacklog.txt.gz#_2015-05-27_17_47_30_78718:23
mriedemthat's another part of this onion, once we've updated g-r we have to sync g-r to ceilometer on stable/juno and merge that too18:23
clarkbhrm, in that case I think the src job is broken but we can come back to that later18:23
mriedemyeah, those are weeds i'm not familiar with18:24
clarkbso ya I think you may be correct about needing tooz release and force merge especially if the src job has that interaction18:25
openstackgerritMerged openstack-infra/project-config: Upload ical output to log server  https://review.openstack.org/18608118:26
* dhellmann wonders if the src job installs a package, then uninstalls it and replaces it with src later18:26
clarkbdhellmann: oh I bet thats it, but since it doesn't do a pip install -U the old conflicting version is still in place18:27
clarkbthis is a corner case that lifeless' pip fixes should fix I think18:27
dhellmannor it hasn't gotten to that point when it fails18:27
clarkbas it should detect that the resolution of that package has changed and install a newer version18:27
dhellmannwe should make sure we install the libs from src before anything else in devstack18:27
openstackgerritMerged openstack-infra/tripleo-ci: Remove nova temprevert  https://review.openstack.org/18269518:27
clarkbdhellmann: ++18:28
*** freerunner has quit IRC18:28
dhellmannbut in the short term, what's safer? force merging the requirements update? and does that actually let us land the tooz fix?18:28
mriedemdhellmann: that doesn't help18:28
*** apmelton is now known as apmelton_away18:29
clarkbdhellmann: I believe mriedem is correct and it won't help18:29
dhellmannso we have to do both?18:29
mriedemwe need ninja merge tooz fix, release tooz 0.12.1, recheck g-r change, sync to ceilometer in juno and merge that18:29
dhellmannforce both, that is?18:29
mriedemthen garbage plates all around18:29
dhellmannk18:29
clarkbdhellmann: no, the original plan of ninjamerge tooz, release 0.12.1 should work18:29
mriedemdhellmann: only one force18:29
clarkbyup18:29
dhellmanncool18:29
fungisounds right. so who wants to push the button of shame this time? shall i, or are you doing that now clarkb?18:30
clarkbfungi: I can do it unless you want to18:30
*** dboik_ has joined #openstack-infra18:30
fungigo for it18:30
clarkb(at this point I understand and am happy to)18:30
clarkbkk18:30
clarkbthough one more question18:31
clarkbmriedem: dhellmann the zake pin is essentially doing what g-r does which we now recognize as broken18:31
clarkbkazoo too18:31
dhellmannharlowja_: ^^18:32
clarkbshould we be using <X.Y.Z instead of <=X.Y.Z-1 for those?18:32
*** ashleighfarnham has quit IRC18:32
dhellmannare we pinning or excluding?18:32
*** ashleighfarnham has joined #openstack-infra18:32
mriedemhttps://review.openstack.org/#/c/186084/3/requirements-py3.txt18:32
dhellmanncapping18:32
clarkbdhellmann: ya but the cap is inclusive, which means if we need to release a point relase for fixes we have to do the dance again18:33
fungijeblair: is christopheraedo who you were talking to about puppeting apps.openstack.org?18:33
dhellmannyeah18:33
mriedemi just copied what was in g-r, but yeah18:33
mriedemi could see zake<0.218:33
dhellmannmriedem: I think clarkb is right18:33
*** BharatK has joined #openstack-infra18:33
clarkbmriedem: oh if this is already in g-r then lets fix it after we are unwedged18:33
*** dustins has joined #openstack-infra18:33
mriedemclarkb: the tooz caps are the same as what they are in g-r on stable/juno18:34
clarkbas we can do that with normal process, if no one disagress I can force merge now18:34
jeblairfungi: i think so, mordred? ^18:34
mriedemsince stable/juno for tooz was just created today and isn't synced up18:34
fungijeblair: mordred: also was the conversation just "thins would be a good next step" or do you know of any work already started in that direction?18:34
mordredjeblair: yes18:34
*** kmartin has joined #openstack-infra18:34
clarkbmriedem: dhellmann: that sound reasonable?18:34
mordredfungi: no work that we know of18:35
dhellmannclarkb: yep18:35
fungik, thx18:35
clarkb(I want to make sure I don't conflate fixing the wedge with making things better overall)18:35
*** maishsk_afk has joined #openstack-infra18:35
dhellmannclarkb: agreed18:35
clarkbok merging now18:35
*** Somay has joined #openstack-infra18:36
clarkbdhellmann: mriedem done18:37
openstackgerritAnita Kuno proposed openstack-infra/irc-meetings: Correct an error in a meeting description.  https://review.openstack.org/18607518:37
dhellmannclarkb: thanks18:37
mriedemclarkb: double thanks18:38
*** jtomasek has joined #openstack-infra18:40
*** maishsk_afk has quit IRC18:43
*** mattfarina has joined #openstack-infra18:43
*** doug-fish has joined #openstack-infra18:44
openstackgerritMatt Riedemann proposed openstack/requirements: Leave wiggle room for zake and kazoo fix releases  https://review.openstack.org/18612318:44
mriedemdhellmann: clarkb: fyi https://review.openstack.org/#/c/186123/18:45
*** maishsk_afk has joined #openstack-infra18:45
*** amotoki_ has joined #openstack-infra18:46
*** maishsk_afk is now known as maishsk18:46
jogohttps://github.com/osops18:46
mtreinishdhellmann: were you really eager to announce the new tooz release? :)18:47
jogoeven being shared out by foundation staff18:47
pleia2huh18:47
*** achanda_ has quit IRC18:47
mriedemmtreinish: that's automated :)18:47
*** otter768 has joined #openstack-infra18:47
mordredwow18:47
*** sdake_ has joined #openstack-infra18:47
jogomtreinish: you should add your own words to the script18:47
mtreinishmriedem: heh, I know I'm just giving dhellmann a hard time :)18:47
mriedemstill have to wait for it to hit the mirrors18:48
mordredhogepodge: heya18:48
mriedem'sorry about burning your house down, here is a new release of x!18:48
jogomordred: context http://lists.openstack.org/pipermail/openstack-operators/2015-May/007060.html18:48
*** BharatK has quit IRC18:48
*** Sukhdev has joined #openstack-infra18:49
clarkbjogo: are you on that list? you can respond with the elk stack we use18:50
jeblairclarkb: he already did! :)18:50
clarkbwhich includes filters for most openstack services18:50
clarkbawesome18:50
clarkbI see it now18:50
fungiyeah, the last time this same topic came up on the ops list i provided links to our logstash filters18:50
clarkb(maybe I should subscribe to that list too)18:50
*** mattfarina has quit IRC18:50
*** HeOS has joined #openstack-infra18:50
pleia2Clint: wow, nice work digging up these ancient reviews (clearly it's been some time since I've made it that far back in my queue)18:50
*** maishsk has quit IRC18:50
*** amotoki_ has quit IRC18:51
jeblairclarkb: and the PR to the osops repo was to add a link to the stack in infra18:51
fungia new thread seeking logstash filters seems to crop up about every 6 months on that ml18:51
jogoI was just about to say that myself18:51
*** maishsk has joined #openstack-infra18:51
*** sdake has quit IRC18:51
jogofungi: but what I find interesting is github osops18:51
Clintpleia2: well, pabelanger was grexing18:51
clarkbspeaking of github, I am (trying) to get this etherpad-lite fix merged and I have no idea why people like this workflow18:52
*** otter768 has quit IRC18:52
pleia2Clint: rightly so :)18:52
clarkbgrabbing the PR is really hard18:52
fungijogo: some of our operator community seem allergic to doing things which might make them seem like developers, but apparently pushing their configs into a github repo isn't one of those things18:52
clarkband even though a PR is an issue you can add a PR to an issue18:52
pabelangerClint, pleia2 well, mostly because openstack-infra-packagers group does not exist yet!  Hopefully somebody will get +2 support to help merge that stuff18:53
pleia2I believe it does some magical linking if you mention the issue in the commit message18:53
clarkbthankfully I have sorted most of that out now and am only running into my lack of js knowledge18:53
pleia2but I don't like the workflow either18:53
clarkbpleia2: it does, but PRs are issues18:53
clarkbpleia2: so why can't I just say attach this code to issue X when making a pull request18:53
pleia2ah, right18:53
*** gokrokve has quit IRC18:53
clarkbin fact funny story, there are hacks in our PR closer bot because of this18:54
clarkbwe disable issues, except there is no close PR api call bceause they are issues, you are supposed to close the issue instead18:54
*** mattfarina has joined #openstack-infra18:54
* pleia2 nods18:55
clarkbwhich confuses the python lib, this leads to getting the PR as a PR then pretending its an issue and closing it18:55
* clarkb notes that the osops filters are already very similar to ours18:56
*** gokrokve has joined #openstack-infra18:56
openstackgerritGeorge Peristerakis proposed openstack-infra/system-config: Moved the server class out of the status class  https://review.openstack.org/18612718:56
*** dboik_ has quit IRC18:57
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add release notes for recent releases  https://review.openstack.org/18365718:57
*** dboik has joined #openstack-infra18:57
clarkbfun thing I have noticed about this test suite, is they have flaky tests too :) sometimes 3 tests fail but usually 2 tests fail18:58
*** e0ne has quit IRC18:59
*** bdemers has joined #openstack-infra19:00
*** achanda has joined #openstack-infra19:00
*** fawadkhaliq has joined #openstack-infra19:01
*** fawadkhaliq has quit IRC19:06
*** abregman has joined #openstack-infra19:06
*** sdake has joined #openstack-infra19:06
*** eharney has quit IRC19:08
dhellmannmtreinish: have you ever seen me eager about anything? ;-)19:09
*** sdake_ has quit IRC19:10
*** mattfarina has quit IRC19:12
*** fawadkhaliq has joined #openstack-infra19:14
*** eharney has joined #openstack-infra19:15
*** jtomasek has quit IRC19:16
*** gyee has joined #openstack-infra19:16
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Consume os-client-config directly  https://review.openstack.org/18597219:17
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Debug dying task managers  https://review.openstack.org/18607919:17
jeblairdhellmann: i've seen you eager to _avoid_ things :)19:17
mordredclarkb: ok - I fixed some things ^^ but I'm back to not being able to see why the providerManager is going away19:17
mordredjeblair: I rebased on clark's patch to capture that, but there is still something going wrong that's not logging19:18
mordredjeblair: I'm not convinced that's a bug in nodepool per-se though - I _think_ it may be fake constructure19:18
kashyapHi folks, maybe I'm missing something very trivial, `git review` fails at this step - `git rebase -p -i remotes/gerrit/master` when I try to submit a review from a local branch.19:18
kashyapAny hints what am I missing? http://paste.openstack.org/show/240517/19:19
clarkbmordred: you should look at the provider config test19:19
clarkbmordred: that one should blow up if providers are still breaking (I would hope)19:19
clarkbkashyap: it means your change has a merge conflict and you should resolve it19:20
*** yamamoto has joined #openstack-infra19:20
*** spredzy is now known as spredzy|afk19:20
clarkbkashyap: if you don't want to resolve it yet (note that gerrit cannot merge if you push this way) you can `git review -R` to push it up without a rebase19:20
*** nolafleur has quit IRC19:20
kashyapclarkb: Oh, I could swear I did rebase it? But I came back to this commit after a while, let me check again.19:20
*** kmartin has quit IRC19:20
jeblairoh, maybe we should detect that case and have git-review say "your change has a merge conflict and you should resolve it, if you don't want to resolve it yet (note that gerrit cannot merge if you push this way) you can `git review -R` to push it up without a rebase"19:20
*** jtomasek has joined #openstack-infra19:21
clarkbjeblair: ya that may be a friendler message, and possibly git rebase --abort for the user too19:21
kashyapjeblair: Yeah, current error seems to come from  - https://github.com/openstack-infra/git-review/blob/master/git_review/tests/test_git_review.py#L15019:21
mordredclarkb: which test is the provider config test?19:21
clarkbmordred: nodepool.tests.test_shade_integration.TestShadeIntegration.test_nodepool_provider_config19:22
mordredoh - got it19:22
mordredthat works19:22
clarkbso after all this fighting node.js I finally checked out HEAD^ and reran test, those fai ltoo19:22
clarkbso I think their tests are broken19:22
jeblairclarkb: ga19:22
jeblairclarkb: the abort does happen19:23
clarkbjeblair: hrm?19:24
jeblairoh, wait, i may not fully understand that yet.19:24
clarkbjeblair: sorry I am etherpad-lite debugging too19:24
jeblairclarkb: (i'm switching back to git-review.  i'm going to implement my suggestion.  i'll let you know when i have something.)19:24
clarkbbecause I want to get our fixed merged and I was asked to make sure their tests pass (they don't) and just now realized thats because master doesn't work19:25
anteayamtreinish: this patch adds a neutron linuxbridge job to tempest's experimental queue, could you add it to your review queue please? https://review.openstack.org/#/c/184830/19:25
jeblairclarkb: *nod*19:25
clarkbjeblair: oh you mean abort in git-review, I thought it didn't abort but maybe I need to upgrade locally19:25
jeblairclarkb: it sometimes does.  i'm working out the threads now.19:25
mtreinishanteaya: sigh, yet another dsvm job on the overloaded tempest queue19:27
clarkbmordred: so that may have been one issue but not the only issue. Now to track down that no longer running traceback19:27
mordredyah19:27
mtreinishsc68cal: why not just do the sorta reckless thing and switch it over without telling anyone19:27
mordredclarkb: gut hunch is that we're not faking out the right things19:27
*** ashleighfarnham has quit IRC19:27
mordredclarkb: greghaynes figured it out last time - maybe he's got the magic touch19:28
* mordred looks at greghaynes ...19:28
*** jsavak has quit IRC19:28
mtreinishsc68cal: I would support that approach19:28
clarkbmordred: so that is raised if you call stop() on the manager19:28
*** jsavak has joined #openstack-infra19:28
clarkbmordred: which we would be calling around rebuilding the managers. So I think it is failing due to the inequalit checks not working right19:28
mordredoh!19:28
anteayamtreinish: well I've been guiding him to the non-reckless approach19:29
clarkbmordred: we basically make new managers and the tests never have a manger long enough to effect change in the system19:29
*** sarob_ has quit IRC19:29
mordredclarkb: but if we're not faking that out appropriate ... let me add some prints ...19:29
anteayamtreinish: what tempest queue would you prefer to see the job? it is non-voting19:29
clarkbmordred: I don't think we fake it at all19:29
mordredwell, we need to now19:29
clarkbmordred: no19:30
mordredno?19:30
clarkbmordred: we need to make the inequality checks work so that we don't rebuild managers every pass through the loop19:30
mordredright19:30
mordredthey work in os-client-config now19:30
clarkbmordred: and that shouldn't require faking19:30
*** timcline has quit IRC19:30
*** ashleighfarnham has joined #openstack-infra19:30
mordredwhat I'm saying is that I'm guessing somewhere now that we're actually directly instantiating that, that we're either faking too much or not enough things, such that the inequality checks are triggering19:30
*** timcline has joined #openstack-infra19:31
*** mattfarina has joined #openstack-infra19:31
mtreinishanteaya: it's on experimental which is the only one it makes sense. I'm just looking at the job count in all the queues and sighing19:31
anteayamtreinish: I'm getting kind of tired of guiding someone in a process that is then circumvented/re-written due to overload19:31
anteayamtreinish: ah yes, I definitely agree with your motivation for sighing19:31
anteayaand we again have the constant problem of it works for n things we we never imgaged that x^2 would want to use it19:32
anteayan^219:32
clarkbmordred: I really think the fakes are sufficient19:33
anteayamtreinish: I'm open to a discussion about how to solve the job count, if you want to have one19:33
mordredclarkb: I'm sure they are19:33
*** abregman has quit IRC19:33
clarkbmordred: we fake the clients themselves, the problem is the provider managers which are not faked are rebuilt each time19:33
*** fawadkhaliq has quit IRC19:33
clarkbmordred: and we shouldn't need to fake that out because its all internal process bits, no network or other service interaction there.19:33
*** mattfarina has quit IRC19:34
mordredwell, other tahn the fact that it will search for env vars or system config files19:34
mordredbut I don't think that's the problem19:34
mordredI'm starting from that inequality check and working outwards to see where things are falling over19:35
electricalOla all im back.19:35
mordredclarkb: yes -that is showing up as unequal - hypothesis confirmed19:36
mordrednow ... why19:36
*** jgriffith has quit IRC19:36
mtreinishanteaya: well in this case, I don't know why this job needs to be on tempest19:36
openstackgerritJames E. Blair proposed openstack-infra/git-review: Provide an explanation on failed rebase  https://review.openstack.org/18613919:36
jeblairclarkb, kashyap: ^19:36
*** jgriffith has joined #openstack-infra19:36
mtreinishbecause running with a different driver hopefully shouldn't change neutron's behavior sufficiently to break a tempest run (assuming it's properly configured)19:37
anteayamtreinish: okay well that would be a fix for now, if sc68cal agrees19:37
anteayamtreinish: I can't assume such a thing with neutron19:37
* kashyap clicks19:37
openstackgerritJames E. Blair proposed openstack-infra/git-review: Provide an explanation on failed rebase  https://review.openstack.org/18613919:37
anteayamtreinish: perhaps sc68cal can19:37
jeblairclarkb: forgot your co-authored-by ^ :)19:37
clarkbjeblair: ha19:37
mtreinishanteaya: oh, I know that's an incredibly naive statement I just made19:37
mtreinishbecause I've used neutron and worked with it before19:38
mordredBad file descriptor (bundled/zeromq/src/epoll.cpp:81)19:38
mordred:)19:38
mordredclarkb: I got a new error ^^ ?19:38
anteayamtreinish: but motivated by trying to reduce job count, I understand19:38
clarkbmordred: I have never seen that before19:38
mordredclarkb: I'm going to blame something else on that one19:38
anteayamtreinish: yeah, so I guess I wait to hear from sc68cal and hear his thoughts19:38
*** yamamoto has quit IRC19:38
mordredclarkb: I'm also going to go with "maybe I didn't actually have the os-client-config patch installed like I thought"19:38
*** yamamoto has joined #openstack-infra19:38
anteayasince we do want neutron linuxbridge tested19:39
clarkbmordred: :)19:39
anteayamtreinish: however that needs to happen19:39
*** yamamoto has quit IRC19:40
mtreinishanteaya: sure, but I just don't think running with linuxbridge on tempest commits adds anything. Neutron *should* be abstracting away the differences between backends19:41
* mtreinish buries his head in sand19:41
mordredmtreinish: well - except there are some differences and we're happy about them19:41
*** timcline has quit IRC19:42
kashyapI wish Gerrit allows one to correct grammatical mistakes after sending hit too soon on a comment.19:42
mordredmtreinish: like the difference between an install that needs floating-ips, for instance19:42
*** timcline has joined #openstack-infra19:42
kashyaphitting send too soon* ie.  :-)19:42
mordredmtreinish: but I dont know that that would appear to tempest19:42
mtreinishmordred: no, I realize that. Look how many config options we have on tempest to try and map all the different neutron deploy configs out there19:42
mordrednod19:43
clarkbkashyap: I am actually a fan of immutable history there, its not always the cleanest ( and I make my fair share of mistakes ) but keeps everything honest and above board19:43
clarkbmordred: linux bridge would not affect need for floating ips19:43
mordredclarkb: zomg. I need to define __ne__ don't I?19:43
clarkbmordred: yes19:43
mordred*headdesk*19:43
*** cody-somerville has joined #openstack-infra19:43
*** cody-somerville has quit IRC19:43
*** cody-somerville has joined #openstack-infra19:43
kashyapclarkb: Yeah, I fully concur. It shows the human aspect too, as opposed to fixing everyone's OCD19:44
clarkbmordred: or use a "isEqual" metho19:44
mtreinishmordred: I'm just trying to figure out when we would need to run check experimental on a tempest commit to test a change against a linux bridge backed neutron19:44
mordredwell, I've already got an __eq__19:44
clarkbmordred: I personally am a fan of explicit methods simply because operator overloading is weird19:44
clarkbmordred: right and this is why its weird :)19:44
mtreinishand whether that edge case is justification enough for adding yet another job to the tempest queue19:44
mordredk. one sec19:44
mordredI mean, it'll be is_equal - but let's not quibble19:44
pabelangerwoah, that took a few minutes! But local nodepool running again, now run with dibs19:46
*** abregman has joined #openstack-infra19:46
clarkbarg I really need to subscribe to this list, jogo maybe you can respond to the logstash thing and say pull requests welcom19:46
clarkbjogo: the only reason its greedy is because the data in the message is rather unstructured so being not greedy is hard19:46
mordredmaybe point out that fixing the ones infra uses is important because they're the ones the dev use to find problems19:47
mordredso the more they're the same thing, the more the experience the operators have will be reflected back to the developers and the easier it will be to talk about the same problems19:47
clarkband now I am subscribed19:48
clarkbmordred: +100019:48
*** jgrimm is now known as zz_jgrimm19:49
openstackgerritRyan Petrello proposed openstack-infra/project-config: Test pecan against barbican, magnum, and gnocchi.  https://review.openstack.org/18614119:49
*** yamamoto has joined #openstack-infra19:50
jogoclarkb: you want to respond as you are the expert?19:50
clarkbjogo: I can't because I only just subscribed19:50
clarkbotherwise you end up breaking the thread an all sorts of unfun19:51
jogoclarkb: ack, will respond19:51
*** timcline has quit IRC19:51
clarkbI am sure jeblair will say there is a way to get around that but my mail foo is such that if I attempt to I will likely just make a new dangling thread and it will be unfun19:52
*** timcline has joined #openstack-infra19:52
jeblairclarkb: easiest way is to ask someone to bounce the message to you19:52
jeblairclarkb: hardest (but best) way is what mordred wants to do -- set up an nntp server ;)19:52
clarkbalso I have to pop out now or will be late for doctor19:52
clarkbhopefully back late this afternoon to check in on nodepool progress19:53
jeblairthe doctor is usually late anyway (despite having a time machine)19:53
jeblair[it's also possible i'm confused about exactly what clarkb is doing when he goes to see 'the doctor']19:54
clarkb:)19:54
*** ociuhandu has quit IRC19:55
*** freerunner has joined #openstack-infra19:55
*** gokrokve has quit IRC19:55
jeblairclarkb, fungi, mordred, marun: I merged https://review.openstack.org/161414 which is a potential "break everything on tomorrow's image builds" change; so if something goes wrong with tox jobs, we'll need to drop the new images.19:58
fungijeblair: thanks for the heads up. i'll check when i wake up19:59
jogoclarkb: responded hopefully I didn't miss anything19:59
*** freerunner has quit IRC19:59
*** mestery_ has joined #openstack-infra20:00
*** kmartin has joined #openstack-infra20:00
*** olaph is now known as olaf20:00
*** olaf is now known as Guest2941920:00
*** Guest29419 is now known as olaph20:00
*** cody-somerville has quit IRC20:00
*** freerunner has joined #openstack-infra20:00
*** ajmiller_ is now known as ajmiller20:01
dhellmannjeblair: you've seen to the truth of the matter regarding my eagerness20:01
*** erikmwilson_ has joined #openstack-infra20:01
*** Rockyg has joined #openstack-infra20:01
*** jtomasek has quit IRC20:01
*** ashleighfarnham has quit IRC20:02
*** mestery has quit IRC20:03
*** tonytan4ever has quit IRC20:05
nikeshmhi20:05
nikeshmi am using trusty-server-cloudimg-amd64-root.tar.gz image in nodepool20:06
nikeshmbut nodepool vm is not  having RAM greater than 6GB20:06
nikeshmis there any kernel-limit in trusty-server-cloudimg-amd64-root.tar.gz image20:07
*** sdake_ has joined #openstack-infra20:07
nikeshmwith cirros image i am able to get RAM of any size20:07
nikeshmbut with this nodepool image i am getting RAM of only 6GB20:08
*** kgiusti has left #openstack-infra20:08
nikeshmi tried many flavors with nodepool image20:08
nikeshmbut its not taking RAM more than 6GB20:08
*** jtomasek has joined #openstack-infra20:08
*** sdake has quit IRC20:11
*** tonytan4ever has joined #openstack-infra20:11
*** alexsyip has joined #openstack-infra20:12
*** achanda has quit IRC20:14
*** tiswanso has quit IRC20:14
*** tiswanso has joined #openstack-infra20:15
funginikeshm: nodepool tells nova to boot instances from the image. in theory you should be able to nova boot that image and confirm whether or not your problem has anything to do with nodepool (i'm pretty sure it does not, but would need to know a lot more about how nodepool is configured to update your images to say that for certain)20:16
asselinnikeshm, try booting the base image in nova and see if that works as expected20:18
*** esker has quit IRC20:19
asselinnikeshm, actually if I remember right, nodepool dib picks the image for you...."release: trusty" http://paste.openstack.org/show/240583/20:20
*** ashleighfarnham has joined #openstack-infra20:20
*** slagle_ is now known as slagle20:20
nikeshmasselin:right20:21
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add attachemnts table to database  https://review.openstack.org/18615120:21
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add support to subunit2sql for storing attachments  https://review.openstack.org/18615220:21
mtreinishpcrews: ^^^ I figured since your job change you dropped that, so I hacked something together. I still need to test it though20:22
*** mrmartin has quit IRC20:26
*** julim has quit IRC20:34
*** amotoki_ has joined #openstack-infra20:34
*** akrivoka has quit IRC20:35
*** annegentle has joined #openstack-infra20:37
*** claudiub has quit IRC20:38
openstackgerritAnita Kuno proposed openstack-infra/irc-meetings: Remove on hold meeting  https://review.openstack.org/18615820:39
*** amotoki_ has quit IRC20:39
*** tiswanso has quit IRC20:40
openstackgerritSylvain Bauza proposed openstack-infra/irc-meetings: Rename Gantt meeting to Nova Scheduler  https://review.openstack.org/18615920:41
*** stevemar has quit IRC20:43
*** dizquierdo has joined #openstack-infra20:44
*** ldnunes has quit IRC20:44
SomayHello everyone, I just started out as a summer intern with OpenDaylight on the Jenkins Job Builder project. I would be really grateful if someone reviewed the following patch - https://review.openstack.org/#/c/185445/20:45
openstackgerritSridhar Ramaswamy proposed openstack-infra/project-config: Add tacker-horizon repo for Tacker VNF Manager UI  https://review.openstack.org/18616120:45
*** esker has joined #openstack-infra20:46
anteayahi Somay I hope your internship is going well20:47
anteayaSomay: if you go into your gerrit ui and you are signed in, navigate to people > list groups and search for jenkins-job-builder-core20:49
*** Sukhdev has quit IRC20:49
anteayaSomay: while others can review your patch too, those are the folks you should spend some time getting to know20:49
*** bswartz has quit IRC20:50
openstackgerritMerged openstack/requirements: Raise cap on tooz to <0.13  https://review.openstack.org/18606520:51
Somayanteaya: Thanks, I’ll have a look there20:52
anteayaSomay: welcome20:52
*** TravT_ has joined #openstack-infra20:53
electricalSomay: i ave a question about your patch. Any specific reason to do negative comparisons? ( the not parts )20:55
openstackgerritSumit Naiksatam proposed openstack-infra/irc-meetings: Remove Neutron Adv Services meeting  https://review.openstack.org/18616320:56
electricaljust something I'm wondering because i personally try to avoid them :-)20:56
Somayelectrical: Yes, the checkboxes in the Jenkins UI are stated as “Disable automatic site…”20:56
*** baoli has quit IRC20:56
*** sridhar_ram has joined #openstack-infra20:56
Somayand in JJB, I wished to keep the description as to enable the feature20:57
electricalAh okay i see. Any specific reason for not calling them disable-automatic-site-archiving ?20:58
SomayDisable sounds good with a checkbox, but for yaml, we should have keywords which describe the feature, hence the negaton20:58
SomayI thought automatic-site-archiving is a better keyword20:58
openstackgerritSumit Naiksatam proposed openstack-infra/irc-meetings: Remove Neutron Adv Services meeting  https://review.openstack.org/18616320:58
Somaydisable-automatic-site-archiving would be less user friendly IMHO20:59
electricalgood point20:59
Somayand by the same argument, we should then have enable-disable for all keywords, which is not nice20:59
electricaljust wanted to understand the reasoning behind it but you obviously thought about it :-)21:00
Somayyes, not an issue :)21:00
*** jerryz has joined #openstack-infra21:00
*** soren_ has joined #openstack-infra21:01
*** ociuhandu has joined #openstack-infra21:01
electricalI've seen developers do weird thing because they were stuck in a certain thought pattern ;-)21:01
*** maishsk_ has joined #openstack-infra21:01
openstackgerritSumit Naiksatam proposed openstack-infra/irc-meetings: Fix typo in name  https://review.openstack.org/18616721:01
*** esker has quit IRC21:01
Somayelectrical: yes, that happens all the time. I agree with you21:02
*** ociuhandu has quit IRC21:02
*** dustins has quit IRC21:02
*** ildikov has quit IRC21:03
*** maishsk has quit IRC21:03
*** maishsk_ is now known as maishsk21:03
*** sdake has joined #openstack-infra21:04
*** ociuhandu has joined #openstack-infra21:04
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Publish git-review documentation  https://review.openstack.org/18616821:05
zxiiroAny JJB reviewers available to review this patch, just needs one more +2 and we'd like to have it in sooner rather than later? https://review.openstack.org/#/c/185445/121:05
zxiiroclarkb: ^ ?21:06
*** jsavak has quit IRC21:06
zxiiroclarkb: woops sorry i copyed the wrong patch number i mean https://review.openstack.org/#/c/182116/21:07
openstackgerritSumit Naiksatam proposed openstack-infra/irc-meetings: Fix GBP meeting details  https://review.openstack.org/18617121:07
*** btully has quit IRC21:07
nikeshmasselin: tried with trusty-server-cloudimg-amd64-disk1.img  its working good with this,but nodepool is trying with trusty-server-cloudimg-amd64-root.tar.gz'21:08
* zxiiro has had a long day :)21:08
*** sdake_ has quit IRC21:08
*** btully has joined #openstack-infra21:09
*** notnownikki has quit IRC21:10
*** samueldmq has quit IRC21:11
*** miqui has quit IRC21:11
*** mestery_ is now known as mestery21:12
electricalI'm patiently waiting for 2 reviews as well :-)21:13
*** dtantsur|afk has quit IRC21:13
* electrical thinks its time to implement a 'review friday' :p21:13
zxiiroelectrical: I usually patiently wait too but this particular patch we need sooner rather than later. I can't wait the usual 2-3 months that my patches usually take to have someone look at it...21:14
*** notnownikki has joined #openstack-infra21:14
electricalzxiiro: yeah indeed.21:14
anteayaelectrical zxiiro Somay you have all reviewed each other's patches, yeah?21:15
*** dizquierdo has quit IRC21:16
*** dkranz has quit IRC21:16
zxiiroanteaya: I've reviewed Somay's patch since I'm his mentor :)21:16
electricalanteaya: i did review it but haven't left my +1 there yet.21:16
anteayazxiiro: great21:16
mordredclarkb: ok - I just released os-client-config with the fixes needed for nodepool21:16
openstackgerritDavide Guerri proposed openstack-infra/shade: Add port resource methods  https://review.openstack.org/17996821:16
*** mwagner has quit IRC21:16
anteayaelectrical: wonderful, it helps to leave a comment that at least says "I've looked at this."21:16
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Consume os-client-config directly  https://review.openstack.org/18597221:17
electricalanteaya: will do21:17
*** jtomasek has quit IRC21:17
mordredclarkb: and I marked a version dep on the new version21:17
anteayaelectrical: thank you21:17
zxiiroanteaya: we plan to provide many more patches over the summer21:18
anteayazxiiro: awesome21:18
*** bswartz has joined #openstack-infra21:18
zxiiroI just wish there was a way to make a specific patch as something we need soon. most of the time I don't care how long it takes but for this particular one it's blocking us from some work at LF21:19
anteayazxiiro: if you and Somay could also make a point of reviewing all the open jjb patches in gerrit it is possible one or both of you might be in a position to become a core reviewer on jjb, many considerations pending21:19
anteayabut it would speed up the time your patches spend in review21:19
zxiiroanteaya: I do do the occassional patch review and plan to increase my time as JJB si something we need21:20
electricalanteaya: i've been doing the same lately ( reviewing open changes )21:20
anteayazxiiro: the best way I know is to become a proficient reviewer21:20
anteayazxiiro electrical awesome21:20
anteayathat really helps the rest of the jjb review team21:20
waynranteaya: i'm interested in becomming a core reviewer on JJB although i tend to only review core library changes rather than module changes21:20
anteayawaynr: great21:20
anteayawaynr: I think a familiarity with the entire repo will help21:21
anteayaexpertise in a certain area is great21:21
anteayabut it is good to have some of each21:21
*** ildikov has joined #openstack-infra21:21
waynri am quite familiar with the entire repo, having implemented both yamlparser features and module features21:21
anteayawonderful21:22
zxiiroanteaya: FWIW I've been contributing since December. Trying to get more active but it's hard to stay engaged when you feel ignored.21:22
anteayaelectrical zxiiro Somay have you all reviewed waynr's patches too?21:22
anteayazxiiro: I hear that21:22
anteayazxiiro: this is why I suggest helping each other21:22
anteayafinding peers who have the same interest is important21:22
electricalanteaya: partially. We worked together on a weird bug that waynr in the end managed to solve :-)21:22
anteayawaiting for cores is often frustrating21:22
zxiiroOur priority has upped too since we hear JJB is becoming removed from OS's priorities from this past summit21:23
anteayaelectrical: wonderful21:23
zxiiroI worry that any core reviewers are even still interested in reviewing and maintaining JJB if they are removing it from OpenStack's infra21:23
anteayazxiiro: ahhhh, let's hear from jeblair on that as I am not feeling it is being removed as much as repurposed21:23
electricalanteaya: i'm working on getting JJB implemented at current job ( elastic ) so i got good reasons to get things reviewed faster ;-) hehe21:23
anteayabut it is possible I am incorrect21:23
anteayaelectrical: sounds good21:24
anteayaour job > zuul workflow is definitely changing21:24
* anteaya digs up the etherpad21:24
zxiiroyeah that's what I got from one of the fishbowl sessions at the summit last week21:24
anteayaso here is the etherpad: https://etherpad.openstack.org/p/YVR-infra-zuulv321:25
anteayazxiiro: you were at summit?21:25
anteayadid we meet?21:25
mordredyah - so we will be moving away from at at $future_date - however, enough people are happily using it that I'd expect we can make more people core on it and have it have a happy life21:25
zxiiroanteaya: indeed i was. not sure, i met zaro though21:25
anteayazxiiro: ah glad you met zaro21:25
zxiiroi'll be at the Jenkins Conference in June too.21:26
mordredwe certainly don't want to make life hard for people who love jjb - we think it's great too21:26
anteayaline 14 Stop using JJB in favor of ansible playbooks21:26
zxiiromordred: that's good to know21:26
waynrinteresting, how do ansible playbooks replace JJB?21:27
anteayaso yes according to the etherpad the intent is to move away from jjb and implement ansbile playbooks for jobs21:27
anteayacreating the config for a job21:27
mordredwaynr: its all part of teh zuul v3 work21:27
*** dannywil_ has quit IRC21:27
waynrare there more details on the etherpad?21:27
*** bdemers has quit IRC21:27
mordredwaynr: on infra's side, we basically use jenkins for 2 things currently: 1) log uploading 2) remote execution of shells21:27
mordredwe don't use any of its other features21:27
anteayaso yes, stepping up involvement on jjb so that you can continue to use it for your work is a good idea21:27
*** dannywilson has joined #openstack-infra21:27
*** peristeri has quit IRC21:27
mordredand we're _very_ close to not using its log/artifact uploading either21:27
mordredwaynr: let me find you the zuulv3 spec21:28
waynroh is it in infra-specs?21:28
anteayahttps://review.openstack.org/#/c/164371/3/specs/zuulv3.rst21:28
mordredwaynr: https://review.openstack.org/#/c/164371/21:28
waynri have that open in a tmux session somewhere21:28
mordredyah21:28
anteayaline 33421:28
anteaya33521:28
mordredbut read the whole thing - it gives context on the why and everything21:28
mordredI do want to be clear that I think jenkins is an awesome technology and meets the needs of many people21:29
waynrokay this seems very important, as i have been working on getting zuul to work for $dayjob21:29
mordredwe have just happened to hit the sscaling inflection points where that is not true for us any longer21:29
waynr(using github)21:29
mordredwaynr: yes. I'm very excited about your github work21:29
* mordred has evil plans for it21:29
anteayaso waynr zxiiro electrical your goals to be more active in jjb are great, and it would serve all your purposes to get to know each other well and to review each other's patches regularly21:29
electricalanteaya: that's on my schedule for tomorrow afternoon :-) ( reviews )21:30
electricalfor now its bed time for me though :-)21:30
waynranteaya: okay, I am reviewing Somay's patch now21:30
anteayaelectrical: understood21:30
anteayawaynr: thank you21:30
*** samueldmq has joined #openstack-infra21:30
electricalTake care all. catch you all online tomorrow again21:30
mordredelectrical: sleep well21:31
waynrit's sad that i can be more active on JJB on my vacation (right now) than i can while at $dayjob because i manage CI pipelines for several dozen projects using JJB21:31
*** akshai has quit IRC21:31
anteayaelectrical: https://www.youtube.com/watch?v=Rp317mfo3Qg make reviewing a daily habit21:31
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Publish git-review documentation  https://review.openstack.org/18616821:31
zxiiroanteaya: i think the reviews statitics is broken http://stackalytics.com/?metric=commits&user_id=thanh.ha@linuxfoundation.org&project_type=all&release=all21:31
* anteaya feels bad she hasn't been true to that herself lately21:31
zxiiroanteaya: it seems tho think i have no reviews when I know i've reviewed a few in the past21:32
anteayazxiiro: it has you listed as an independent21:32
anteayaif you are Thanh Ha21:32
openstackgerritDawid Malinowski proposed openstack-infra/jenkins-job-builder: Extend support for join-trigger  https://review.openstack.org/18617621:33
zxiiroyeah i should probably submit a patch to stackalytics to set that right21:33
anteayaso for whatever reason stackalytics is confused about who you work with21:33
anteayazxiiro: that would be my suggested course of action21:33
SomaySorry, I wasn’t around. I read the whole conversation. Yes, we should review each other’s patches. It’s quite new to me that JJB is being removed from the priorities. Also, I’m quite new to JJB, so I’ll try to review as many patches as I can.21:34
Somayanteaya: thanks for the suggestion21:34
anteayaSomay: welcome21:35
anteayaSomay: you might find some value here, when getting started reviewing a patch: http://anteaya.info/blog/2013/03/21/reviewing-an-openstack-patch/21:36
SomaySure, i’ll read this21:38
anteayaI hope it is helpful21:39
*** dpaterson has quit IRC21:39
*** dtantsur has joined #openstack-infra21:40
*** gordc has quit IRC21:41
openstackgerritDavide Guerri proposed openstack-infra/shade: Add port resource methods  https://review.openstack.org/17996821:41
zxiiroanteaya: done https://review.openstack.org/18617821:41
*** asahlin is now known as asahlin_afk21:42
zxiiroare there infra meetings i should be aware of / attend?21:42
Clintevery tuesday21:42
openstackgerritMateusz Matuszkowiak proposed openstack-infra/project-config: Added new SF repository for Calamari Fuel plugin  https://review.openstack.org/18617921:43
zxiiroclarkb: noted... i assume this one? https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting21:43
*** dtantsur has quit IRC21:45
jeblairelectrical, waynr, zxiiro: yeah, i have a todo item to respond to the mailing list thread about the proposed jjb rewrite.  the short version is that even if infra stops using jjb, i want to make sure the project continues.  we will not pull support from jjb; instead we will work out the best way for jjb to continue to live beyond our use of it.  because of the summit i have been too busy to give that the attention it deserves, but hope to d21:45
anteayazxiiro: that is the one21:45
openstackgerritDoug Hellmann proposed openstack-infra/yaml2ical: Allow extra settings for meetings to be used in templates  https://review.openstack.org/18618321:45
jeblairelectrical, waynr, zxiiro: in other words, i don't think there's a hurry; we have time to work out a good transition plan21:46
zxiirojeblair: thanks, in hindsight i probably should have introduced myself to you during the summit. Maybe next time :)21:46
jeblairzxiiro: see you in tokyo? ;)21:47
zxiirojeblair: maybe, I'm also attending the Jenkins Conference next month if you're attending that21:47
zxiirojeblair: I think you were the guy leading the fishbowl sessions?21:47
*** ddieterly has quit IRC21:48
anteayazxiiro: is hard to miss, follow the bow tie21:48
anteayajeblair is hard to miss21:48
jeblairzxiiro: some of them, yes. i also had a hat.21:48
zxiirocool yeah i'm pretty sure I know who you are21:49
jeblairuh oh :)21:49
*** mrmartin has joined #openstack-infra21:49
openstackgerritDoug Hellmann proposed openstack-infra/irc-meetings: Add meeting_id and agenda_url support  https://review.openstack.org/18618421:49
jeblairanteaya: ^21:49
*** dboik has quit IRC21:49
*** mriedem has quit IRC21:51
lifelessis the requirements wedge fixed, or can I help?21:51
anteayajeblair: doug's changes?21:51
*** pc_m has quit IRC21:52
jeblairanteaya: yeah, i think it may implement a feature you implicitly requested :)21:52
waynrjeblair: i am looking at the zuul3 spec right now, it looks very good! actually this is exactly the direction i was hoping zuul would go eventually (moving away from a reliance on jenkins as a job runner)21:52
clarkbI am not here but can we make sure packaging + delorean doesnt reinvent zuul21:52
clarkbbecause it sounds like they are reinventing zuul21:52
fungilifeless: i think it's settled since the tooz patch and release... at least i haven't heard any further screaming but you might check in #openstack-stable21:52
*** dizquierdo has joined #openstack-infra21:52
anteayajeblair: it does, I'll redo mine as a dependency of doug's patch21:53
fungiclarkb: i may have only skimmed... seemed like they said it grokked zuul refs and so could be run on our job workers21:53
jeblairwaynr: cool, glad you like it.  i think i have one or two things to tweak, but i think it's pretty close in principle to being ready to get started on.  still time to leave feedback if you have some.21:53
clarkbno they have to add zuul support and it does its own triggering21:53
clarkbso no zuul at all you rundelorean21:53
fungioh. ew21:53
*** rguillebert has joined #openstack-infra21:53
fungiderekh_: ^ ?21:54
clarkbat least thats how I read21:54
*** dannywilson has quit IRC21:54
clarkb"it monitors git repositories"21:54
waynrjeblair: yeah i might not be able to leave feedback until next week but i am somewhat concerned since $dayjob may need a transition period between jenkins and non-jenkins architecture if i can convince my team that zuul is the tool for us21:54
*** dannywilson has joined #openstack-infra21:55
waynr(concerned about backwards compatibility with the jjb/jenkins world that is)21:55
*** mattfarina has joined #openstack-infra21:55
*** mjturek1 has quit IRC21:55
fungiwaynr: at least for now, we have a jenkins plugin which allows it to speak gearman, and use that to bridge zuul to jenkins. keep in mind that jenkins-only to jenkins+zuul is a transition we've already undergone21:56
clarkbalso we dont need anew namespace or distro specific repos21:56
* clarkb stops readinf email and goes back to hospitalling21:57
*** dtantsur has joined #openstack-infra21:57
jeblairwaynr: yeah, we've made a lot of transitions in a stepwise manner and supporting backwards compat (as fungi described), but for this one, i think it will be very hard to do that, so i am proposing a break.  we will probably switch all our jobs at once.  we will probably write a tool to do the bulk of the conversion automatically, but i don't know that it will end up being generalizable.21:57
fungiclarkb: right. derekh_ suggested asking infra, so i was going to wait for them to do that bit21:57
clarkbfungi well they have asked... anyways hard to be on irc, back later21:57
jeblairwaynr: i think it will be possible/easy to switch projects one zuul shared-change-queue at a time21:57
waynryeah it probably won't be generalizable given what little i know of openstack's JJB usage vs my own usage21:58
jeblairwaynr: we might take advantage of that to switch infra first before openstack21:58
waynr(your auto conversion tool that is)21:58
jeblairthat might work for you depending on how segmented your configuration is21:58
fungiclarkb: best of luck! stop looking at openstack things and focus on hospital21:58
*** mwagner has joined #openstack-infra21:58
*** mrmartin has quit IRC21:58
*** dangers is now known as dangers_away21:58
waynri was actually hoping to convert all my JJB yaml to JJB python code using classes that could be used later to target something other than JJB for job definitions21:59
anteayaclarkb: what fungi said21:59
derekh_clarkb: fungi yup its not currently ZUUL aware i.e. it currently only builds after a commit has gone into the branch its tracking, but I'm open to adding support for reading ZUUL REFs if it looks like we're going in that direction21:59
jeblairalso, if someone comes up with a clever way to make the transition easier, i'm all for entertaining it.  i just sensed that for this instance, it could greatly extend the implementation time and we would be able to more easily absorb a cutover rather than extended transition.21:59
zxiirojeblair: when Zuulv3 is out would we still be able to use Jenkins along with it if we wanted to? or would support for it be dropped completely?22:00
waynryeah i don't think my approach would help you necessarily, but I think that using python to define job configurations is something i want to continue doing beyond a JJB/jenkins world--i also considered implementing a zuul layout generator using these same classes22:00
zxiiroas far as I'm aware it's gearman that talks to Jenkins so that shouldn't be a problem no?22:01
fungiderekh_: yeah, if delorean could just be reduced into something which can run on demand on a job worker and let us fire it from zuul and pass in the usual refs and have it understand those, i think it ought to work fine. though at that point it seems like it's just a standardized set of package building instructions22:01
anteayahttps://review.openstack.org/#/c/186184/1 failed with no logs22:01
*** gokrokve has joined #openstack-infra22:02
waynrzxiiro: in my understanding there are some important parameters passed from zuul to jenkins that tell jenkins where to get the git ref it uses for cloning the project being tested22:02
jeblairwaynr: one thing i want to do in particular is try to reduce the repetitive constructs that made jjb necessary in the first place.  we might fail to achieve perfection there and need something like what you describe :)22:02
*** akshai has joined #openstack-infra22:03
*** mriedem has joined #openstack-infra22:03
anteayafungi: sorry fungi I rechecked your recheck22:03
*** hrubi_ has quit IRC22:03
fungianteaya: zuul will ignore it--no worries22:03
anteayak22:04
*** dtantsur has quit IRC22:04
derekh_fungi: yes we should be able todo that, I don't see any reason it wouldn't work and your right that would just be a set of build instructions, but its only half of what delorean does, the other half is keeping track of builds for all of the projects22:04
*** Sukhdev has joined #openstack-infra22:04
fungiwaynr: in fact, it's just adding those as parameters in jenkins which show up in the calling environment on the worker, and so it's really passing that to whatever your job payload/script is, just via jenkins22:04
derekh_fungi:  so that after each commit we have a repository containing all of the most recent successfully built packages22:05
*** amotoki_ has joined #openstack-infra22:05
fungianteaya: https://jenkins03.openstack.org/job/gate-irc-meetings-tox-ical/17/console22:05
jeblairzxiiro: i believe it will theoretically be possible, but i think it would require a bit more work.  my intention is for us to use ansible in openstack, and to design the system to support that use case, but also that part should be pluggable.  i think that with some changes to the gearman-plugin, it could be done.22:05
derekh_fungi: in other words you end up with a "current" repo that contains current openstack http://trunk.rdoproject.org/centos70/current/22:05
fungiderekh_: got it. so customized publisher too22:06
derekh_fungi: yup22:06
*** bknudson has quit IRC22:06
waynrjeblair: yeah that's my goal also--reduce repitition. i've found so often while developing templates and job groups using JJB yaml that what i really want is a turing complete language to allow me to achieve a greater degree of configuration efficiency using conditionals, subclasses, and more natural parameterization constructs22:06
anteayafungi: is that a failure to connect to swift?22:07
zxiirojeblair: guess we'll have to see, i discussed this a bit with zaro during the summit as well. I hope it can be done since we at LF do not yet have any plans to add Ansible to our setup but we do want to give zuul a try.22:07
openstackgerritDavide Guerri proposed openstack-infra/shade: Add port resource methods  https://review.openstack.org/17996822:07
fungianteaya: nope, http://logs.openstack.org/84/186184/1/check/gate-irc-meetings-tox-ical/ccc4920/console.html is working22:07
fungianteaya: i think something's wrong with the swift directory index generation. maybe this is more widespread. checking now22:08
jeblairwaynr: let's not put this in the critical path of zuulv3, but i could imagine making part of zuul pluggable so that instead of loading job definitions from yaml, it hands off to something else to produce the needed output.  that could invoke the sort of thing you are describing.22:08
fungialso possible it's something gone wrong with serving directory indexes from os-loganalyze22:08
jeblairwaynr: i think if we keep that in the back of our minds, we can probably avoid designing something that would break that, and then add it later.22:09
dougwighi infra, can i get some project-config eyeballs on this experimental lbaas job? it's for moving the tempest tests out of tempest, per qa's request.  https://review.openstack.org/#/c/184870/22:09
*** dtantsur has joined #openstack-infra22:09
zxiirogotta step out for dinner. If any JJB core reviewers has some time to review this patch I would be very appreciative ;)  https://review.openstack.org/#/c/182116/22:10
*** hrubi has joined #openstack-infra22:10
*** amotoki_ has quit IRC22:10
waynrjeblair: sounds good! i'll keep an eye on the spec and the upcoming zuul patches22:10
jeblairwaynr: i'm anticipating this is a full 6-month cycle project.  other people tell me they think it will take longer.  :)22:11
jeblair(i don't believe them)22:11
*** Rockyg has quit IRC22:11
mordredjeblair: I've got it specced at 2 months in my head22:12
openstackgerritMerged openstack/requirements: Add glance_store to projects.txt  https://review.openstack.org/18267422:12
zarozxiiro: slow now because pelix is out for a week.22:12
fungianteaya: aha!22:14
zarojeblair, clarkb, fungi : i believe i was able to repro the gerrit issue on review-dev.  http://paste.openstack.org/show/240745/22:14
*** timcline has quit IRC22:14
anteayafungi: that sounds like you found something useful22:14
fungianteaya: it's caused by that change, but also an oversight on how we publish the test artifacts for that job22:14
anteayawonderful22:14
fungianteaya: we upload output/index.html which is generated as part of the job22:14
anteayaglad it flushed out an oversight22:14
zaroi'm thinking my next step is to validate this fix https://git.eclipse.org/r/#/c/48288/22:15
fungianteaya: _that_ is also what teh swift upload job would create to provide a directory tree index22:15
anteayazaro: yay22:15
zarounless you feel that we need to do something else?22:15
anteayafungi: name clash22:15
*** pradk has joined #openstack-infra22:15
fungianteaya: in this particular case, the proposed change is broken in such a way that output/index.html is 0 bytes in length22:15
anteayanot a helpful file length22:16
fungianteaya: then the job helpfully uploads that as index.html for the logs files22:16
anteayazaro: I'm supportive of your direction22:16
anteayafungi: I think I follow22:16
jeblairzaro: awesome, i agree you have reproduced the error and next step is validate the fix.  yay!22:16
fungizaro: yeah, i saw it ended up in the 4.0.0 rc2 tag yesterday, so hopefully gerrit will also be picking that up soon22:16
greghayneszaro: nice!22:16
waynrjeblair: if i can get my team on board with moving to zuul2 i should eventually be able to participate in zuul3 efforts22:17
anteayawaynr: one step at a time, motivating a team to change takes work22:17
zarocool, now need to figure out how to package that change into gerrit22:17
* mordred finds bottles of liquor help22:17
*** erikmwilson_ has quit IRC22:17
*** rguillebert has quit IRC22:17
jeblairwaynr: if offers of smoked meat help, i feel confident we can oblige22:18
anteayawaynr: we welcome your efforts but don't think moving a team in a different direction will be easy22:18
jeblairmordred: also that22:18
mordredmmm. smoked meat22:18
waynranteaya: my team is more than ready to move away from our current Jenkins-centric CI architecture, the trick will be convincing them in the next month that zuul2 is the right next step for us22:18
*** ashleighfarnham has quit IRC22:18
*** dguerri is now known as dguerri`away22:18
* fungi finds that smoked meat goes will with bottles of liquor22:19
fungier, well22:19
anteayawaynr: ah22:19
anteayawaynr: have any of your team spent any time in this channel?22:19
waynrwe just finished up an 18-month long term plan that necessitates being able to scale beyond what Jenkins has to offer22:19
waynranteaya: not that i am aware of, i finally got one of them to join me on JJB reviews last week though :)22:19
anteayawaynr: they are welcome to pop in and ask us our opinion of moving22:19
anteayawaynr: yay, progress22:20
*** annegentle has quit IRC22:20
zxiirozaro: ah that explains why i couldn't find pelix in this channel :)22:21
waynri am hoping to have a full proof of concept of zuul2 for our most common workflows by the end of June, this will be accompanied by a detailed technical overview/evaluation of zuul in the context of the user stories we have collected from our wider engineering org22:21
zxiirofungi: zaro: Gerrit should pick it up soon. The JGit and Gerrit developers work very closely, some of them are even committers on both projects22:21
anteayawaynr: I hope you are getting all you need from us to accomplish your goals22:21
anteayaincluding offers of liquor and smoked meat22:23
*** annegentle has joined #openstack-infra22:24
*** amitgandhinz has quit IRC22:24
fungijhesketh: i'm currently reading through zuul_swift_upload.py to figure out how to specify that i want to rename a file on upload, but if your around and have a moment to short-circuit me to the answer for that i would be most obliged22:27
fungier, you're22:27
fungioh, i bet i could just do that as part of the job instead22:28
waynranteaya: ha! thanks, i will probably be increasingly active here asking questions for my POC work22:28
mordredfungi clarkb: I have pushed an 0.6.4 tag for shade which includes the swift upload fix22:28
anteayawaynr: of course, glad to know the overall plan22:28
mordredfungi, clarkb: I have also pushed a 1.1.0 tag for os-client-config which includes the oscc fixes needed22:29
anteayamordred: if I read through the lines correctly it appears clarkb is at the hospital and latest report is he is offline for a bit22:29
mordredanteaya: yes22:29
mordredanteaya: I agree with your lines22:30
anteayajust in case you were expecting a reply22:30
anteayakk22:30
mordrednope. just leaving him nuggets he can digest upon his return22:30
anteayak22:30
*** dizquierdo has quit IRC22:31
*** dannywil_ has joined #openstack-infra22:32
*** nelsnelson has quit IRC22:32
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Rename irc-meetings index.html in job logs  https://review.openstack.org/18620222:33
fungianteaya: i think this ^ should do what we want22:34
*** kmartin_ has joined #openstack-infra22:34
*** fawadkhaliq has joined #openstack-infra22:34
*** kmartin has quit IRC22:34
*** patrickeast_ has joined #openstack-infra22:34
* anteaya looks22:34
*** soren_ has quit IRC22:34
*** dimtruck is now known as zz_dimtruck22:35
*** dannywilson has quit IRC22:35
anteayalooks like it should do the job22:35
anteayamordred: can you cast an eye on 186202 please?22:36
*** patrickeast has quit IRC22:36
*** patrickeast_ is now known as patrickeast22:36
*** tonyb has joined #openstack-infra22:37
tonybAm I in trouble?22:37
anteayanot at all22:37
tonyb\o/22:37
tonybHow can I help?22:37
anteayawanted to share the irc-meeting job situation22:37
anteayahttps://review.openstack.org/#/c/186202/22:37
anteayaif you could review that, that would be great22:37
anteayabasically the swift upload failed since the logs were called the same name as the file the swift job creates22:38
anteayaso that patch renames the file created by the ical job for the tests22:38
anteayato avoid a name clash22:38
*** fawadkhaliq has quit IRC22:38
fungiwell, the upload didn't exactly fail22:38
openstackgerritMatt Riedemann proposed openstack/requirements: Leave wiggle room for zake and kazoo fix releases  https://review.openstack.org/18612322:39
* anteaya listens while fungi fixes what she said22:39
fungiin fact the upload successfully replaced the file index for the logs22:39
jlvillalI think this is a simple change to requirements: https://review.openstack.org/185770   But I would like to get feedback if this is the correct route.  At the moment we are changing tox.ini in openstack/ironic to work-around the issue.22:40
fungibut ideally we'd like to get the file index for the job logs rather than the irc index being (sometimes unsuccessfully if the change is bad) generated by tox22:40
tonybfungi: Yeah.  I was reviewing a few irc-meetings changes and it was bit of a pain.22:41
*** salv-orlando has joined #openstack-infra22:41
tonybfungi: Thanks for the fix.22:41
anteayaah I hadn't looked at any irc-meeting logs before22:41
anteayayes, http://logs.openstack.org/75/186075/4/check/gate-irc-meetings-tox-ical/e0043c6/ is kind of useless in debugging a job22:42
tonybanteaya: +122:42
*** derekh_ has quit IRC22:42
fungii'm hoping the approach of renaming it as the first publisher task will work around the fact that we need it to have that name when the scp publisher pushes it to the eavesdrop server but can't have that name when the swift publisher uploads it on check results22:42
anteayatonyb: I just wanted to include you so you knew what was happening going forward22:42
*** aysyanne has quit IRC22:43
anteayafungi: it will be fun to track22:43
fungianyway, no fire, just resulted in some mild confusion over broken change https://review.openstack.org/18618422:43
anteayamordred: thank you22:44
tonybThanks all.  I need to take the kids to school.22:44
*** timcline has joined #openstack-infra22:44
anteayatonyb: enjoy22:45
fungijlvillal: can you elaborate on "supposed to be installed by WSME, but for some reasons it was missed in py3 env"?22:45
fungijlvillal: does wsme generally have this failing on py3k?22:45
jlvillalfungi: The openstack/ironic gate started failing.22:46
*** annegentle has quit IRC22:46
fungidhellmann: ^ 185770 may point to a deeper problem22:46
jlvillalAdding the 'simplegeneric' to our tox.ini fixed it.  I didn't do the initial patch.22:46
jlvillalfungi: I'm not sure why it started failing.  I'm copying the text that was put into the patch which fixed the issue to make the gate work.22:47
dhellmannjlvillal: is simplegeneric in the requirements list for wsme?22:47
fungijlvillal: right, but you asked what the correct approach is to solving this. i think it starts with having a solid understanding of what's broken22:47
jlvillaldhellmann: I think so.  When I do the install without this fix on my system it works.22:47
fungiso we can figure out where it should be fixed22:47
jlvillalfungi: I am unsure what is the underlying cause.  It is only happening on Python 3.22:48
fungijlvillal: do you have a change proposed to trove to revert the workaround so i can see it failing?22:48
dhellmannjlvillal: ok, it shouldn't be necessary to add it to ironic's requirements, then. Do you have a link to logs for a failed job or a bug?22:48
jlvillalfungi: I do not.22:48
fungithat's probably a great place to start then. wip revert of the workaround change22:49
jlvillaldhellmann: It was failing around Tuesday of last week.  So the hack was put into make it workaround.22:49
fungiwhen we see how it breaks, we'll know more22:49
jrollthis may have been the one where us cores said "f it, let's fix the gate and then file a bug"22:49
dhellmannfungi: the dependency on simplegenerics is the same in requirements.txt and requirements-py3.txt (no range on either)22:49
*** otter768 has joined #openstack-infra22:49
*** timcline has quit IRC22:49
jrollthat's totally my fault :(22:49
*** kutija has quit IRC22:49
jlvillalfungi: dhellmann: This is the patch: https://review.openstack.org/#/c/184429/22:49
dhellmannjlvillal, jroll : ok, you should try reverting that and if you see it continue to fail we can help you debug it now that the summit is over22:50
jrollfungi: iirc, the theory (or actual behavior?) was it being uninstalled due to not being in requirements.txt22:50
jrolldhellmann: yeah, +122:50
jrolljlvillal: you got this? I was just passing by :)22:50
jlvillaldhellmann: fungi jroll : I will try a revert patch and see what happens.22:50
dhellmannjroll: hmm, I'm not aware of anything that uninstalls packages like you describe unless it's also trying to upgrade22:50
* jlvillal has to walk home first.22:50
*** krtaylor has quit IRC22:50
jrollsweet, thanks22:51
dhellmannjlvillal: ping me when you've got some logs and I'll see if I can help you out22:51
*** afazekas has quit IRC22:51
jrolldhellmann: I think there's code around "this is in reqs.txt but not in g-r" that does something like that22:51
jlvillaldhellmann: thanks22:51
jrolldhellmann: but then I went to a summit and forgot everything I had learned shortly before that :P22:51
dhellmannjroll: on a sync it does, but not when installing stuff22:51
jrolldhellmann: ok, my memory is poor atm22:52
fungialso that shouldn't in theory be happening on unit tests, which i think is the only place python 3.x comes into play?22:52
dhellmannjroll: and that's just to your project's requirements*.txt, not to wsme's -- pip should still be dragging in the secondary dependency22:52
*** akshai has quit IRC22:52
dhellmannfungi: good point22:52
jrollah, true. hm.22:53
* jroll shrugs22:53
jrolllet's see where the revert goes22:53
fungibut yeah, the quick-fix change lacks any debugging details about why it was implemented22:53
anteayafungi: seems publishers can't talk shell: jenkins_jobs.errors.JenkinsJobsException: Unknown entry point or macro 'shell' for component type: 'publisher'.22:53
*** camunoz has joined #openstack-infra22:53
jheskethMorning22:53
anteayamorning jhesketh22:53
anteayaand what great timing, we could use your help22:53
anteayajhesketh: let us know when you are appropriately coffeed22:54
fungianteaya: thanks. jhesketh have suggestions for how i can best do https://review.openstack.org/186202 ?22:54
*** otter768 has quit IRC22:54
jrollfungi: yeah, it's weird what happens when we talk about changes face-to-face :)22:54
jheskethanteaya: heh, not sure how much coffee that'll require ;-). What's up?22:54
jheskethfungi: looking22:54
anteayajhesketh: right now we are getting http://logs.openstack.org/59/186159/1/check/gate-irc-meetings-tox-ical/77ac141/ as output for the job run, we would like to have the job logs22:55
jheskethfungi: what you have done seems sensible to me22:56
anteayapublishers can't talk shell though22:56
anteayajenkins fails on that approach22:56
*** dannywil_ has quit IRC22:57
fungiinterestingly, it looks like a publisher can be a builder22:57
anteayabuilders can talk shell22:57
fungiyeah, i'm just now reverse-engineering our publisher for swift uploads22:57
jheskethYeah we can use a publisher builders to do shell22:57
fungipostbuildscript builders22:58
fungifound it. thanks22:58
fungithe zuul-swift-devstack-logs-with-console publisher is the example i wanted22:58
jheskethfungi: alternatively, and perhaps better, we can upload the whole output folder as a folder and train zuul-swift-upload to not generate an index of one already exists22:58
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Rename irc-meetings index.html in job logs  https://review.openstack.org/18620222:59
openstackgerritAdam Gandelman proposed openstack-infra/system-config: Add #akanda to openstack bot's channel list  https://review.openstack.org/18236823:00
fungijhesketh: great idea. in the meantime there's the simple hack23:00
lifelessjeblair: dhellmann: IIRC you took photos of the whiteboard on the requirements BOF; could you send me copies?23:00
*** angela-s has quit IRC23:00
openstackgerritJoe Gordon proposed openstack-dev/hacking: Add example to versioning section of README  https://review.openstack.org/18525223:01
openstackgerritJoe Gordon proposed openstack-dev/hacking: Revise adding additional checks section of README  https://review.openstack.org/18525323:01
openstackgerritJoe Gordon proposed openstack-dev/hacking: Take advantage of RST formatting for links  https://review.openstack.org/18525723:01
*** doug-fish has left #openstack-infra23:01
fungijhesketh: though... i think we still want file indexes on those. for example it's nice to be able to browse the irc-meetings.ical file in the output directory even if generating output/index.html goes haywire and ends in a 0-byte file23:01
*** Rockyg has joined #openstack-infra23:01
jheskethfungi: hmm, that's a good point. Maybe zuul-swift-upload needs a feature to detect and move indexes out of the way23:03
fungiyou can still of course get to irc-meetings.ical by blindly feeding in the url in that case, but less confusing if just trying to follow the link to site-index.html results in a blank page (or in this recent case your browser offering to let you download a null-length file of unknown mime type)23:03
*** sarob has joined #openstack-infra23:03
*** mriedem has quit IRC23:04
jeblairlifeless: i don't think i did.  i only have pictures of float planes.  i believe dhellmann did.23:04
*** gokrokve has quit IRC23:04
*** pradk has quit IRC23:04
openstackgerritSpencer Krum proposed openstack-infra/puppet-log_processor: Build worker config from template  https://review.openstack.org/18621423:05
fungijeblair: you should have said "sure! hold on..." and then sent him pictures of float planes instead23:05
anteayawere they selfies with float planes?23:05
jeblairfungi, anteaya: there's a good chance that's actually what dhellmann was doing23:05
*** chlong has joined #openstack-infra23:05
anteayaha ha ha23:05
lifelessjeblair: cool; I'll await my float plane photobomb23:06
nibalizerclarkb: interested in your feedback on 18621423:07
*** wuhg has joined #openstack-infra23:07
anteayanibalizer: he might or might not be offline for a while23:07
nibalizeryea for to do baby things23:08
anteayajust by way of managing your turn around expectations23:08
anteayajhesketh: care to review now that Jenkins likes this? https://review.openstack.org/#/c/186202/23:10
*** sarob_ has joined #openstack-infra23:10
*** tonytan4ever has quit IRC23:10
mordredlifeless: I took pictures in my mind23:10
*** redixin has quit IRC23:10
anteayaI'm not sure I want to see those23:11
anteayado the lines stay still in your mind pictures?23:11
openstackgerritJames E. Blair proposed openstack-infra/project-config: Update infra project ACLs  https://review.openstack.org/18621723:11
jeblairthat's a fun change ^23:12
*** sarob has quit IRC23:12
*** sarob_ is now known as sarob23:12
jeblairi could do that as 120 changes if people prefer.23:13
*** salv-orlando has quit IRC23:14
anteayaI don't prefer 120 changes23:14
anteayawhat do you want to do with projects that noone is really working on?23:14
anteayaare folks working on openstack-nose for instance?23:14
*** koolhead17 has joined #openstack-infra23:15
*** koolhead17 has quit IRC23:15
*** koolhead17 has joined #openstack-infra23:15
jogomordred: wow this is painful: http://git.openstack.org/cgit/openstack/os-client-config/tree/os_client_config/vendors.py23:15
jeblairanteaya: we might consider retiring openstack-nose if we're done with it, but we can handle that separately.23:15
jeblairjogo: yeah, i believe it represents a failure to achive our goal as a project.23:15
*** sdake_ has joined #openstack-infra23:15
*** notnownikki has quit IRC23:16
anteayajeblair: okay23:16
mordredjogo: yah23:16
jogojeblair: agreed23:17
openstackgerritDoug Wiegley proposed openstack-infra/project-config: Add second job for lbaasv1 API  https://review.openstack.org/18487023:17
jeblair(and to be really specific, i think it's in direct contradition to 'ubiquitous' in the mission statement)23:17
jogomordred: good response to Keith23:17
mordredjogo: thanks!23:18
mordredjogo: hopefully it came across as a yes, and23:18
*** heyongli- has joined #openstack-infra23:18
nibalizerya there are a couple puppet modules that could be nominated for attic or whatever other disposal process is available to us23:19
*** yamamoto has quit IRC23:19
*** ChuckC has joined #openstack-infra23:19
nibalizerbut as jeblair said, different change23:19
*** sdake has quit IRC23:19
clarkbjhesketh: with the ongoing nodepool dib things then being afk I haven't gotten to the logs-dev stuff yet and likely wont today23:20
jeblairnibalizer: in infra, i expect it will be a README change to the project, and perhaps marking the project read-only if we feel really strongly about it.  :)  i don't buy into the idea that we need to change the repo name when we stop working on a project.23:20
*** hdd has quit IRC23:21
fungior worse, when we have changes in governance of a project23:21
* nibalizer nod23:21
jeblairyeesh, indeed.23:21
clarkbnibalizer: 186214 doesn't produce a working worker config so not sure what the intent is there23:21
anteayajeblair: I'm wondering about all the puppet modules23:22
jeblairanteaya: i think they're great23:22
jeblairand getting better all the time23:22
clarkbnibalizer: oh wait its the worker not the client I may be mistaken23:22
anteayajeblair: I see the same puppet folks working on all of them, would it make sense to have a puppet-core team for all of them?23:22
*** ZZelle_ has quit IRC23:22
*** TravT_ has quit IRC23:22
jeblairanteaya: yes, but i'm going to do that with gerrit groups rather than acl files23:22
openstackgerritDoug Hellmann proposed openstack-infra/release-tools: Add make_feature_branch.sh  https://review.openstack.org/18622223:22
anteayajeblair: I'm just talking acl changes atm23:22
nibalizerclarkb: yea, the names in this section of -infra are not ideal23:22
nibalizerat least I get confused a lot23:23
clarkbnibalizer: gearmanisms23:23
anteayaoh so the puppet-core is a memeber of each individual puppet module23:23
jeblairanteaya: that's what i meant by the sentence "by using group inclusion in Gerrit"23:23
jeblairanteaya: yep23:23
clarkbnibalizer: so I think my initial review is why, the change doesn't capture any of that23:23
anteayaokay thanks23:23
jeblairanteaya: groups can be members of groups, so that's easy to do23:23
clarkbnibalizer: if I understand correctly this doesn't make the worker more or less configurable23:23
jeblairthat's what i'm working on next, and want to get in place before we merge that change23:23
clarkbnibalizer: it just changes the api, so capturing the reason behind that would be good23:23
anteayajeblair: yes, was just wondering what you were planning, so thanks for letting me know23:24
jeblairanteaya: but if there's someone who wants to be puppet-zuul core but doesn't care about any other modules, we can do that just as easily in the new system23:24
nibalizerclarkb: kindof, it moves the configuration closer to the thing doing the configuriaton23:24
anteayaright23:24
*** hichihara has joined #openstack-infra23:24
anteayajust thinking the puppet people tend to move as a group, that I see23:24
clarkbnibalizer: but is it worth breaking everyone using hte module? in theory just us but who knows23:25
clarkbnibalizer: good to justify that23:25
nibalizerclarkb: yea I can get in to that23:25
nibalizeroh gross puppet doesn't like variables with dashes23:25
clarkbnibalizer: correct23:25
clarkbthis is why our projects are named puppet-foo_br23:25
nibalizerso things like output-host are gonna be output_host in puppet23:25
nibalizer:/23:25
clarkbyes23:25
nibalizeri kenw about modules, I didn't think it mattered for variables23:26
*** tim_o has quit IRC23:26
funginibalizer: lots of languages dislike hyphens in variable names since long ago the authors of ascii decided to use one byte for both the subtraction sign and the hyphen23:26
*** abregman has quit IRC23:27
jeblairfungi: except of course the one true language23:27
*** hdd has joined #openstack-infra23:27
fungiyes23:27
clarkbfungi: in fairness old typewriters didn't have 1 keys (supposedly this is why 0 cames after 9 but they didn't have 0 keys either in many cases)23:28
crinkleyeah the module names and variable names are different stories23:28
clarkbfungi: so its good we got different characters in ascii for those23:28
fungiagreed. also glad lisp treats l/1 and 0/O differently ;)23:28
jheskethclarkb: no trouble.. unfortunately I was unable to reproduce the issue locally23:29
jeblairfungi: i think you just poked the eyes out of anyone reading this in droid sans23:29
clarkbjeblair: I am reading in droid l and 1 are distinct but 0/O are not23:30
fungi...and nothing of value was lost23:30
*** mwhagedorn has quit IRC23:31
jeblairclarkb: sure, but can you distinguish l/l ?23:31
clarkbjeblair: nope /me copies into vim to figure out what is going on there23:31
* fungi just realized he lacks "the critic" on dvd. will correct that posthaste23:31
clarkbvim assures me its the same ascii character23:31
* jeblair goes back to work in the other desktop before clarkb figures it out23:31
clarkb:P23:31
openstackgerritSpencer Krum proposed openstack-infra/puppet-log_processor: Build worker config from template  https://review.openstack.org/18621423:32
nibalizerclarkb: updated23:32
jeblairfungi: it stinks!23:32
clarkbmordred: ok so oscc and shade have new releases with nodepool needed fixes, rechecking your nodepool change should pass now?23:32
clarkbmordred: but jenkins still says no23:33
anteayajeblair: still on the acl change, why is openstack/openstack-planet acl part of that patch?23:34
*** asselin_ has joined #openstack-infra23:34
jeblairanteaya: it's an infra repo23:34
mordredclarkb: yah - I pushed that up and rechecked because I'm stumped again and was hoping it was just a local problem23:34
clarkbmordred: http://logs.openstack.org/72/185972/5/check/gate-nodepool-python27/968cb96/console.html#_2015-05-27_23_25_57_541 looks like a new error23:34
clarkbmordred: provider doesn't have region_name23:34
mordredclarkb: thank you23:35
*** prad has quit IRC23:35
*** wenlock has quit IRC23:35
clarkbthen later mysql says deadlock detected but I think thats likely due to earlier errors23:35
*** docaedo has joined #openstack-infra23:36
*** nelsnelson has joined #openstack-infra23:36
anteayajeblair: yes you an monty are core: https://review.openstack.org/#/admin/groups/93,members but so are anne reed and ttx23:37
clarkbmordred: so I think region_name needs to remain on the provider object in nodepool.yaml23:37
clarkbmordred: that is a small fix23:37
mordredclarkb: yah - that's totally fine23:37
jeblairanteaya: yes, and they'll end up in the new core group too23:37
mordredclarkb: lemme do that real quick23:37
*** dims_ has quit IRC23:38
clarkbjhesketh: interesting, maybe we broke the non swift case? did you test that too?23:38
*** fawadkhaliq has joined #openstack-infra23:38
openstackgerritMonty Taylor proposed openstack-infra/nodepool: Consume os-client-config directly  https://review.openstack.org/18597223:38
mordredclarkb: ^^ I reorged slightly too - so that the patch is cleaner to look at and the provider mangling is in a function23:39
*** krtaylor has joined #openstack-infra23:39
jlvillaldhellmann, fungi, So reverting that simplegeneric patch failed the testing:  https://review.openstack.org/#/c/186208/23:39
jlvillaldhellmann, fungi, http://logs.openstack.org/08/186208/1/check/gate-ironic-python34/b3618cd/23:40
*** dims_ has joined #openstack-infra23:40
jheskethclarkb: yep23:40
clarkbmordred: yup looks good if jenkins is happy with it23:40
jheskethclarkb: I only tried one file, but it worked from disk and swift23:40
fungijlvillal: thanks. seeing now if i can interpret why23:41
clarkbjhesketh: maybe fungi remembers? iirc he and sdague were around for the breaking23:41
jheskethclarkb: and the timestamps etc all marked-up correctly23:41
*** ajmiller has quit IRC23:42
jlvillalfungi, Thanks23:43
nibalizerclarkb: is the ascii diagram in http://docs.openstack.org/infra/system-config/logstash.html correct?23:43
nibalizerspecificlly 1:1 ratio of logstash indexers to log-gearman workers?23:44
nibalizerit looks to me like there are 4 gear workers per logstash indexer23:45
openstackgerritMerged openstack-infra/git-review: Isolate tests from user/system git configuration  https://review.openstack.org/17995423:45
clarkbnibalizer: its correct in that its not meant to imply any ratios23:45
*** timcline has joined #openstack-infra23:46
clarkbnibalizer: we run 4:1, you can run N:123:46
fungidhellmann: jlvillal: it looks like wsme 0.6.4 is installed according to tox/py34-1.log and while it has simplegeneric in the install_requires within its setup.py that doesn't seem to get pulled in23:46
anteayajeblair: okay there it is in governance: http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml#n59523:46
nibalizeroooo23:46
anteayaI couldn't figure out what made it an infra repo23:46
jlvillalfungi, Yes, I think that is what we were seeing.  And why they put in the change to tox.ini23:46
jlvillalfungi, Why it isn't being pulled in, not sure....23:47
nibalizerI think "Currently we have multiple Logstash worker nodes that pair a log pusher with a Logstash indexer." is why I thought 1:123:47
nibalizerdid it used to be 1:1 ?23:47
lifelessinstall_requires in setup.py? Is it a pbr using project?23:47
fungilifeless: as of 0.6.4 it was not, no23:47
lifelesspbr sets install_requires from static data, setup.py's values will be ignored.23:47
clarkbnibalizer: it was at one point but not for very long23:47
nibalizerwell, and the diagram23:47
fungilifeless: no, wait it was indeed using pbr at that point23:47
openstackgerritJamie Lennox proposed openstack-infra/project-config: Ignore requirements job on keystoneauth branch  https://review.openstack.org/18622823:47
fungilifeless: http://git.openstack.org/cgit/stackforge/wsme/tree/setup.py?id=0.6.4 for reference23:47
fungilifeless: so perhaps new-world pbr has broken that23:48
fungianteaya: it was previously an orphaned repo until infra adopted it out of the pile of unclaimed repos back when we started having formal programs23:49
anteayaah okay thanks23:49
fungianteaya: it wasn't a perfect fit for any particular program in my opinion, but it needed a home23:49
anteayagot it23:49
anteayaglad it has a roof and food23:49
*** timcline has quit IRC23:50
fungiand since it's configuration for a service being hosted on our servers, infra was a better home than the other options23:50
*** ddutta has joined #openstack-infra23:50
*** yamamoto has joined #openstack-infra23:50
jeblairalso, some of us were reviewing it23:51
fungiyep, that too23:51
lifelessfungi: I don't think it ever worked23:51
*** amotoki_ has joined #openstack-infra23:51
lifelessfungi: we assign to the setup.py kwargs in utils.py23:51
fungilifeless: dhellmann: jlvillal: so i wonder if tagging a wsme 0.7 is in order?23:51
fungiit looks much more pbresque23:52
fungimaster does, i mean23:52
fungi0.6.4 was ~6mos ago23:52
*** nelsnelson has quit IRC23:52
fungiit's going on 25 changes since then23:53
lifelessactually no, yes I'm sure pbr regressed this - but we never claimed to support it (and never had tests) - and update.py enforces a standard setup.py23:53
lifelessso while we might want to fix it (please file a bug), we should fix wsme IMO23:54
dhellmannlifeless: wsme isn't an openstack project, it's just trying to use some of the same tools23:54
dhellmannyeah, I agree23:54
dhellmannfungi, lifeless, jlvillal: how about if I make a note to release wsme on monday? do we think the current master fixes this?23:54
fungidhellmann: it looks like it should. though who knows what new shadows lurk in the heart of 0.7.023:55
jlvillalfungi, dhellmann, if a new wsme fixes it then sounds good to me :)23:55
dhellmannfungi: right, so no releases this late in the week :-)23:55
clarkbmordred: down to one failure now http://logs.openstack.org/72/185972/6/check/gate-nodepool-python27/be02e34/console.html#_2015-05-27_23_44_50_777 so close23:55
fungijlvillal: yeah, i guess keep your workaround in place and recheck 186208 once that happens23:55
jlvillalfungi, Okay.23:56
*** amotoki_ has quit IRC23:56
dhellmannjlvillal: do you have a bug open for this?23:56
fungijlvillal: and if it passes, then update the commit message with some better detail about the wsme release and get it merged23:56
samueldmqanteaya, hello, could you please revisit 179661 ? (Experimental Devstack job for Identity v3 API only)23:57
samueldmqanteaya, you already had a +2 there, but it needed a rebase23:57
samueldmqanteaya, the devstack patch in which it depends on already started gate jobs23:57
fungidhellmann: the change that added the workaround in ironic said a bug report was desired, but it was also during the summit so...23:57
dhellmannfungi: yeah23:57
dhellmannjlvillal: please file a bug against wsme so we have something to track this with23:57
anteayasamueldmq: sure23:58
samueldmqanteaya, great thanks23:58
*** ddutta has quit IRC23:58
*** julim has joined #openstack-infra23:59
openstackgerritSpencer Krum proposed openstack-infra/puppet-log_processor: Build worker config from template  https://review.openstack.org/18621423:59
anteayasamueldmq: let's just get some fresh jenkins output on it first, I just rechecked the patch23:59
samueldmqanteaya, nice, makes sense23:59

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