Friday, 2013-07-19

*** openstack has joined #openstack-infra16:48
pleia2hooray!16:49
clarkbpleia2: safe for me to approve things that have the necessary reviews?16:49
pleia2clarkb: yes16:50
*** cppcabrera has joined #openstack-infra16:50
*** zaro0508 has joined #openstack-infra16:50
jeblairclarkb: so had i an opportunity to review I4b83c78d83efdd9da8f99c72e85097370dffddc3 i would have said "-1 can you put this in the devstack-logs and devstack-logs-periodic" macro instead?16:51
pleia2jeblair: hopefully I did the class thing right re: 3777016:51
openstackgerritA change was merged to openstack-infra/config: Add /cgit rewrite for git server docroot  https://review.openstack.org/3762316:51
*** Protux has joined #openstack-infra16:52
jeblairclarkb: from the shell logs in 37776, it looks like it's working as intended, so i think that's ready.16:52
clarkbjeblair: ok, I will propose a change to follow up and correct JJB if that helps16:52
jeblairclarkb: i'm on it.16:53
*** ladquin has joined #openstack-infra16:54
*** derekh has quit IRC16:54
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Move devstack testr log copying to macros  https://review.openstack.org/3794216:54
clarkbpleia2: https://review.openstack.org/#/c/37751/ I left a note there (and approved it)16:58
openstackgerritA change was merged to openstack-infra/jeepyb: Add mkdir of git directories to cgit script  https://review.openstack.org/3775116:59
pleia2clarkb: good to know, thanks :)16:59
openstackgerritVijendar Komalla proposed a change to openstack/requirements: Add python-troveclient to requirements  https://review.openstack.org/3768617:00
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Move devstack testr log copying to macros  https://review.openstack.org/3794217:00
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Set the Zuul/Jenkins base log URL in a function  https://review.openstack.org/3632017:00
*** niska` has quit IRC17:01
jeblairclarkb, fungi: ^ can you review those two?  I'd like to go ahead and merge them and start using the new feature.17:01
pleia2btw, the openstack blog interviewed me (it's up now), I said lots of nice things about you coremudgeons17:01
fungialready looking, jeblair17:02
clarkbpleia2: did you call us coremudgeons?17:02
pleia2clarkb: alas, no17:02
clarkb:(17:02
pleia2jeblair did a couple posts before though!17:02
clarkbpleia2: fungi https://review.openstack.org/#/c/37770/4/modules/openstack_project/manifests/git.pp line 58, authorized_keys don't come with a gerrit url do they? should that variable be removed fro mthat line?17:03
*** anteaya has joined #openstack-infra17:03
*** niska has joined #openstack-infra17:03
fungiclarkb: pleia2: correct it should just be the key17:03
pleia2clarkb: right (I swear I actually do know how to use ssh, this is what I get for copy pasta from a script that was using known_hosts to ssh in the opposite direction)17:03
*** Protux has quit IRC17:03
pleia2:)17:04
pleia2fixing17:04
SpamapShttp://paste.openstack.org/show/4094217:05
SpamapShalp ?17:05
openstackgerritElizabeth Krumbach Joseph proposed a change to openstack-infra/config: Add jeepyb & cgit repo creation script to git.o.o  https://review.openstack.org/3777017:05
clarkbSpamapS: upgrade distribute globally17:05
clarkbSpamapS: that appears to be the pain from distribute and setuptools merging17:06
clarkb(I think you can upgrade setuptools globally as well and get the same effect)17:06
fungior upgrade setuptools. either way17:06
*** nati_ueno has joined #openstack-infra17:06
* clarkb reviews jeblair's change now17:06
fungiyeah. the problem arises when distribute gets upgraded as part of a set of packages and then other packages try to install/upgrade before setuptools upgrades17:07
SpamapSI have setuptools 0.6c11 and distribute 0.7. No change.17:07
blamarhttp://logs.openstack.org/37921/1/check/gate-nova-docs/27861/console.html new bug? doesn't seem to be affecting other jobs so might have been a blip? couldn't find an error like it on the rechecks page17:07
fungiSpamapS: is that the latest setuptools?17:08
SpamapSno17:08
jeblairblamar: yeah, blip, sorry.  i'm moving the log storage around and permissions were wrong briefly17:08
SpamapShm, why didn't -U pick it up?17:08
SpamapSok now 0.7 of both17:08
jeblairblamar: i'd go with 'recheck no bug' since it's a known, nearly fixed problem.17:08
blamarjeblair: no worries, ty!17:08
SpamapShttp://paste.openstack.org/show/4094317:08
SpamapSslightly different similar fail17:09
openstackgerritRussell Bryant proposed a change to openstack-infra/config: Update the default extension  https://review.openstack.org/3775517:09
openstackgerritRussell Bryant proposed a change to openstack-infra/config: Update asterisk documentation  https://review.openstack.org/3776217:09
SpamapSall I wanted was a newer hacking. :-/17:10
openstackgerritElizabeth Krumbach Joseph proposed a change to openstack-infra/config: Add jeepyb & cgit repo creation script to git.o.o  https://review.openstack.org/3777017:12
fungiSpamapS: it's an upstream python distribute/setuptools growing pain point, to be sure. we hit it a week or two ago and promptly puppeted all our servers to install the newest setuptools as a workaround17:12
SpamapSI have 0.9.617:12
SpamapShm17:14
*** linuxmohan has joined #openstack-infra17:14
*** ewindisch has quit IRC17:15
*** ewindisch has joined #openstack-infra17:16
clarkbjeblair: reviewed. I found a typo that will need fixign17:16
clarkbfungi: I don't think that change has merged. I will hit the go button on it now17:16
clarkbunless we want to wait for zuul to be done17:16
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Add voipms configuration  https://review.openstack.org/3776517:17
fungiclarkb: which change hasn't merged?17:17
clarkbfungi: https://review.openstack.org/#/c/36708/17:17
fungiahh, right, that one17:18
*** cp16net is now known as cp16net|away17:19
*** sarob has joined #openstack-infra17:19
SpamapSok17:19
SpamapSso I had to rip out the system setuptools17:19
*** jjmb has joined #openstack-infra17:19
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Set the Zuul/Jenkins base log URL in a function  https://review.openstack.org/3632017:19
SpamapSworth noting that if anybody else gets stuck17:19
*** linuxmohan has quit IRC17:20
*** sarob has quit IRC17:20
clarkbSpamapS: system being apt installed?17:20
SpamapSright17:20
SpamapSwtf17:20
SpamapSI thought we were going to ignore periods on commit messages?!17:20
*** sarob has joined #openstack-infra17:21
SpamapS./setup.py:1:1: H803  git commit title ('Run forever if a command is configured.') should not end with period17:21
SpamapSgnar17:21
clarkbSpamapS: I think they added a rule so that you could choose one way or the other17:22
clarkbSpamapS: but you can easily add an ignore in tox.ini17:22
SpamapSthat17:22
SpamapSis17:22
SpamapSstupid17:22
SpamapSlets just pick one way, kthxbai!17:22
*** pabelanger has quit IRC17:22
SpamapS(and the one way we picked on the mailing list was to ignore entirely.17:22
SpamapSbut I digress17:22
SpamapS'tis friday, and I have a talk to get ready for17:23
* SpamapS ignores this insanity :p17:23
*** sarob has quit IRC17:25
*** sarob_ has joined #openstack-infra17:25
clarkbwe should probably change the mime type on our IRC log files17:26
clarkbso that the files open in your browser17:26
pleia2chrome is ok with it17:27
_TheDodd_Where can I find the actual gerrit hooks themselves? Such as change-merged, patchset-created, etc.17:28
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: xunit publisher was wrong with multiple types  https://review.openstack.org/3763417:28
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Add #savanna to eavesdrop  https://review.openstack.org/3794517:28
clarkb_TheDodd_: in the jeepyb repo. https://github.com/openstack-infra/jeeypb17:28
clarkboh nevermind17:29
clarkb_TheDodd_: the things that gerrit actually execs. One sec17:29
pleia2ah, the cgit rewrite needs an apache restart17:29
pleia2maybe make an exec that subscribes to the cgit apache file?17:29
clarkb_TheDodd_: https://github.com/openstack-infra/config/tree/master/modules/openstack_project/files/gerrit and https://github.com/openstack-infra/config/tree/master/modules/openstack_project/templates17:30
_TheDodd_clarkb, yes. The actual executable files. I've been searching around quite a bit, but I can't seem find them.17:30
_TheDodd_clarkb, thanks.17:30
_TheDodd_I'll take a look at them.17:30
jeblairpleia2: if you use the apache::vhost module, it does all that for you17:30
clarkbpleia2: not an exec. Instead add an apache service and make it subscribe to that file17:30
*** pabelanger has joined #openstack-infra17:30
clarkbpleia2: or what jeblair suggests17:30
mgagne_TheDodd_: not sure if it's the correct file: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/templates/gerrit_patchset-created.erb17:31
*** ^demon has joined #openstack-infra17:31
*** ^demon has joined #openstack-infra17:31
clarkbmgagne: that is one of them17:31
clarkbpleia2: chromium works too but firefox does not17:32
clarkbsilly firefox17:32
fungijeblair: spotted a tiny couple of omissions in 3632017:32
pleia2jeblair: yeah, poking around the module now17:33
clarkbpleia2: note we use a really old version because they made lots of incompatible changes and we never updated our end17:35
mordredclarkb: the apache module thing?17:36
clarkbmordred: ya17:36
pleia2thanks17:36
mordredclarkb: we should really go back and figure that out at some point17:36
clarkbmordred: we should17:36
*** Ryan_Lane has joined #openstack-infra17:36
*** Ryan_Lane has quit IRC17:36
*** Ryan_Lane has joined #openstack-infra17:36
openstackgerritJames E. Blair proposed a change to openstack-infra/config: Set the Zuul/Jenkins base log URL in a function  https://review.openstack.org/3632017:36
mgagnepleia2: they refactored the whole module a couple of weeks ago17:36
jeblairfungi: thanks. i believe those snuck in while this was in review.17:36
clarkbpleia2: you can blame hunner17:36
clarkbpleia2: but they are good changse and hunner did the right hting. its just slightly painful for us17:36
mordredclarkb, jeblair: btw - I realized yesterday that once pleia2 get dib gating working, there's a decent path forward for us to test changes to the content of the nodes we use in d-g17:37
mordredclarkb, jeblair: I'm going to write down what I'm thinking there, but I think there's a couple of really sexy possibilities17:37
jeblairmordred: ok.  what content?17:37
mgagneclarkb: lot of refactors coming I would say for a lot of modules, we got hit by the one for rabbitmq this week17:38
mordredjeblair: the stuff that's in the d-g machines- as opposed to the mechanism for launching them17:38
jeblairmordred: what stuff?17:38
*** SergeyLukjanov has quit IRC17:38
mordredjeblair: like the other day when we did the openjdk7 change and that affected the d-g nodes themselves17:39
mordred(that change was successful, clearly, just an example of the type of change I'm talking about)17:39
*** nati_ueno has quit IRC17:40
jeblairmordred: okay, well, maybe what you're writing down will help me appreciate what you're thinking; but my initial thought is that in general we don't attempt to gate the rest of the world...17:41
clarkbhttps://launchpad.net/ubuntu/precise/+source/openjdk-6 is still without a new release. ++ to upgrading to 717:41
jeblairmordred: and if it turs out our operating system is unable to host openstack development, we accept the risk that everything will stop working until we fix that, rather than try to artificially hold ourserves to a previous version17:42
mordredjeblair: yeah, working on writing it down right now. _definitely_ not talking about gating the rest of the world - purely talking about gating changes we decide to make.17:42
openstackgerritwill soula proposed a change to openstack-infra/jenkins-job-builder: Adding the Sauce OnDemand Plugin  https://review.openstack.org/3794717:42
*** nati_ueno has joined #openstack-infra17:42
mordredjeblair: we're missing each other. I'm sorry I brought it up without having writen down the thoughts first17:42
jeblairmordred: i'm not entirely sure we're missing each other.  could you accomplish what you're thinking by adding 'puppet agent --test' to devstack-gate and running that in response to changes to infra/config?17:45
ekarlsowin 317:45
openstackgerritA change was merged to openstack-infra/config: Add jeepyb & cgit repo creation script to git.o.o  https://review.openstack.org/3777017:47
openstackgerritA change was merged to openstack-infra/config: Move devstack testr log copying to macros  https://review.openstack.org/3794217:49
reedcan we have stats for the IRC channel logs?17:49
openstackgerritA change was merged to openstack-infra/config: Set the Zuul/Jenkins base log URL in a function  https://review.openstack.org/3632017:49
jeblairbreak-everything friday continues ^17:49
reedoh, no17:49
clarkbjeblair: I am at my battle station17:50
reedno commits on friday :)17:50
clarkbreed: what kind of stats?17:50
reedstuff like http://stats.kopweb.net/17:50
clarkbthe logs are pretty raw, youwould have to do processing on them I think17:50
reedclarkb, yes, there are tools for that :)17:50
jeblairreed: i don't see why not.  that sounds like a good low-hanging-fruit bug for openstack-ci17:52
reedjeblair, will file a request, thanks17:53
jog0mordred: this entry point stuff is funny17:53
openstackgerritA change was merged to openstack-infra/devstack-gate: Grab testr logs even when testr fails.  https://review.openstack.org/3777617:55
openstackgerritwill soula proposed a change to openstack-infra/jenkins-job-builder: Adding the Sauce OnDemand Plugin  https://review.openstack.org/3794717:59
*** sandywalsh has quit IRC18:01
reedhttps://bugs.launchpad.net/openstack-ci/+bug/120313718:03
clarkbwow pisg is mean18:03
jeblairclarkb: ?18:03
*** amotoki has quit IRC18:03
clarkbI ran it really quickly over today's infra log and got "Is jeblair stupid or just asking too many questions? 33.3% lines contained a question!"18:04
*** SergeyLukjanov has joined #openstack-infra18:04
pleia2poor jeblair18:04
jeblairclarkb: i'm not sure we want that?18:04
annegentle_clarkb: lol18:05
clarkbjeblair: yeah I don't think we want that either. but pisg was the example reed provided18:05
clarkbjeblair: it also spat out "pleia2 brings happiness to the world. 13.3% lines contained smiling faces. :)"18:05
*** pabelanger has quit IRC18:05
annegentle_clarkb: lol?18:05
pleia2go me18:05
jeblairclarkb: maybe we should look at something else?18:05
annegentle_oh I crack myself up18:05
annegentle_jeblair: ROTFL18:05
jeblairclarkb: or maybe it has an option not to be rude?18:05
clarkbjeblair: I think we should. I will update the bug that pisg is a little harsh18:05
clarkbjeblair: oh maybe /me reads more docs18:05
pleia2I think it's configurable for those things18:05
pleia2(I use it for a few non-foss channels)18:05
reedI put superseriousstats on the bug report18:05
reedpisg seems lame18:06
reedand I hate perl18:06
pleia2hah18:06
clarkbreed: but it supports supybot out of the box so you never have to touch the perl18:06
pleia2pisg is pretty solid though, it has been running on one of my servers for years without messing with it (I forgot it existed)18:06
*** arborism has joined #openstack-infra18:06
reedclarkb, it remains lame-oh18:06
reed:)18:07
*** melwitt has joined #openstack-infra18:07
*** pcrews has joined #openstack-infra18:07
reedthere is irssistats too18:07
pleia2it is very community amusing-oriented though, with quotes and all, not just raw stats18:07
reedless lame than pisg, it seems18:07
reedhttp://royale.zerezo.com/irssistats/18:08
reedhttp://royale.zerezo.com/irssistats/irssi.php18:09
jeblairi think zuul may need a restart to pick up this config correctyl.18:09
*** nati_ueno has quit IRC18:09
clarkbjeblair: because it is a function?18:10
clarkbI have updated the bug with notes on pisg so that we don't have mean stats18:11
jeblairclarkb: i think so18:11
jeblairclarkb: i think the reload method may need to reimport imported modules18:11
*** nati_ueno has joined #openstack-infra18:11
clarkbjeblair: I think you are correct18:11
fungii use reload(modulename) for that, though haven't checked to see if that's outdated methodology18:13
jeblairi rechecked things that were in the check queue18:13
jeblairfungi: yeah, zuul uses that in unit tests18:13
fungiahh, good to know i'm not doing something too stupid where i've used it then18:14
jeblairthe currently running jobs have a reasonable looking BASE_LOG_PATH18:14
*** sandywalsh has joined #openstack-infra18:14
jeblairand i just checked one job in jenkins and it uses BASE_LOG_PATH in the scp section18:15
*** linuxmohan has joined #openstack-infra18:15
clarkb\o/18:15
clarkbjeblair: this should make our filesystems much happier18:15
clarkbthank you for getting it done18:15
mordredw00t!18:15
jeblairclarkb: yep.  6 months from now we'll be able to run 'ls'.  :)18:16
* mordred hands jeblair a cookie18:16
jeblairCOOKIE18:16
clarkbI should keep an eye on the logstash gearman client18:16
jeblairclarkb: does that need to be restarted?18:16
*** jjmb has quit IRC18:16
clarkbjeblair: it does, I am going to do it now18:18
clarkbI will lose whatever logs are queued in the gearman server. but I can live with that18:18
*** linuxmohan has quit IRC18:20
*** jjmb has joined #openstack-infra18:20
jeblairi believe there is a bug in that the requirements jobs seem to be always running, ignoring the file matching pattern.18:20
pleia2bah, my special class for jeepyb is messed up, looking into18:21
jeblairhowever, the correct jobs appear to be voting, which is a similar configuration construct.18:21
cppcabrerafungi: The only catch with using reload(module) is that this doesn't work in Python 3. It that case, you'd have to `import imp; imp.reload(module)`18:21
cppcabreras/It/In18:21
*** kgriffs has joined #openstack-infra18:21
*** kgriffs has left #openstack-infra18:22
sdaguefungi: so you have some time to talk about the log filter / color stuff on the log server?18:22
fungicppcabrera: indeed, i've run into that already since i was using it in a py3k-supporting project18:22
*** linuxmohan has joined #openstack-infra18:22
fungisdague: yeah, i started seeing if i could figure out a good conditional rewrite to apply to that, but a lot of it will have to do with what makes clarkb happy/sad18:22
jeblairsdague: aside: you mentioned mod_python the other day; why not use that?18:23
sdagueno worries18:23
sdaguejeblair: because I wasn't sure if it was an option18:23
fungisdague: we could go with a shadow path to the same filenames, or a separate vhost even if we wanted, rather than dealing with rewrites too18:23
sdagueif you are cool with installing it in that environment, I can take a run at that18:23
sdagueI was trying to make this a least intrusive as possible18:23
*** cp16net|away is now known as cp16net18:24
fungisdague: i don't see any reason to forbid mod_python on static.o.o, as long as we're careful that it doesn't burst into flames when loaded18:24
pleia2jeblair: my fix re: your first comment https://review.openstack.org/#/c/37770/3/modules/openstack_project/manifests/git.pp was all wrong, which class did you mean?18:24
sdaguefungi: ok, it will add substantial weight to all the apache process, but if the server can handle it, I'm cool with figuring out how to do it that way18:25
sdagueit should make things more flexible for sure18:25
jeblairi've been mulling over the idea of having log storing and log retrieving apps.  i don't have a fully formed plan yet, but i think both could be useful in the long run, and this could be a start of the second.18:25
*** dprince has quit IRC18:25
fungijeblair: swift backend!18:26
jeblair(in particular, i'm thinking that apps can help with alternate storage backends, additionaly security (containers for different kinds of artifacts), validation of upload permissions, etc)18:26
* fungi ducks18:26
jeblairfungi: exactly18:26
clarkblogstash is currently in a weird place where jobs that are finishing started before the JJB config changes so they don't have the new parameter18:26
*** linuxmohan has quit IRC18:26
jeblairclarkb: oh, yeah.18:27
clarkbbut once they flush out everything should be happy on the logstash side18:27
*** linuxmohan has joined #openstack-infra18:27
jeblairso i think the bug in zuul relates to multiple metajobs matching; i think there may be a bit of last-job-wins18:28
*** nati_ueno_2 has joined #openstack-infra18:29
jeblairwe should fix this immediately as it could mean some job configuration options are wrong18:29
* jeblair hacks18:30
*** pabelanger has joined #openstack-infra18:33
*** krtaylor has quit IRC18:35
devanandahi! anything particularly off with jenkins today?18:36
mordreddevananda: there are some things that are being worked through right now18:38
devanandak18:38
devanandahttps://review.openstack.org/#/c/37763/ seems permanently LOST, but i've seen a few other things temporarily LOST recently too18:38
openstackgerritClark Boylan proposed a change to openstack-infra/config: Properly root log path in logstash gearman client.  https://review.openstack.org/3795318:39
clarkb^ that fixes one problem with the logstash + new log path stuff18:39
*** hashar has joined #openstack-infra18:39
clarkbthat is the only problem I currently see. I have manually applied that and disabled puppet. I will probably just push it thorugh after lunch if that hasn't happened sooner18:39
notmynamejeblair: do you have a twitter account?18:41
jeblairnotmyname: nope18:41
clarkbdevananda: we had a huge backlog of changes that couldn't go in around H2 and now that H2 is over the floodgates have opened18:42
notmynamejeblair: k. my searches were getting odd results (https://twitter.com/Blairgun)18:42
notmynamejeblair: ;-)18:42
openstackgerritElizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix jeepyb class issue on git.o.o  https://review.openstack.org/3795418:42
*** boris-42 has quit IRC18:42
clarkbdevananda: these are good things that should make the gate and logging and stuff scale further18:42
devanandaclarkb: awesome. yea, i've noticed you guys working on those things :)18:43
clarkbsdague: mtreinish did the stop logging glance sqlalchemy at DEBUG level change ever go through?18:43
openstackgerritJames E. Blair proposed a change to openstack-infra/zuul: Apply metajobs in order  https://review.openstack.org/3795518:45
jeblairclarkb, fungi, mordred: ^18:45
mordredjeblair: looking18:45
mordredjeblair: looks reasonable to me18:46
clarkbreviewing18:47
*** hashar has quit IRC18:48
fungilgtm18:48
fungiwe need another quick outage as soon as we merge that, right?18:48
jog0I have a strange question about copyright18:48
jog0https://review.openstack.org/#/c/35380/7/nova/tests/virt/xenapi/test_driver.py18:48
jog0see line 418:48
jog0is that allowed?18:48
jeblairnotmyname: i'm also not https://twitter.com/jeblair  who totally gets what twitter is for.18:49
clarkbjeblair: +2'd will let you approve and control the push process18:49
notmynamejeblair: lol18:49
clarkbjog0: I am not a lawyer, but yes I think it is allowed. Open Source licenses don't invalidate copyright they depend on it18:51
clarkbjog0: so asserting that all rights are reserved by the copyright holder is fine18:51
jeblairjog0: it is... confusing.18:51
jog0clarkb: they can reserve all rights doesn't conflict with open sourcing?18:51
clarkbjog0: no, because open sourcign depends on strong copyright...18:51
jeblairjog0: because it is clearly falsified by the next statements which declares that some rights, in fact, are not reserved.18:52
mtreinishclarkb: nope18:52
clarkbjeblair: apache2 doesn't undermine copyright rights though which that line refers to18:52
jeblairhowever, some licenses have that in their standard text18:52
mtreinishclarkb: https://review.openstack.org/#/c/37123/ still sitting there with 3 +1s18:52
jeblaireg, bsd.18:52
clarkbmtreinish: damn18:52
openstackgerritElizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix jeepyb class issue on git.o.o  https://review.openstack.org/3795418:53
jog0jeblair: clarkb any links explaining this18:53
jog0that I can defer to18:53
fungijog0: it was a phrase required by the buenos aires convention and later obsoleted by the berne convention18:54
jog0fungi: that means nothing to me18:54
jog0:/18:54
* fungi gets international copyright reference out real quick18:54
mordredjog0: http://en.wikipedia.org/wiki/All_rights_reserved18:55
fungior wikipedia ftw18:55
jeblairjog0: i think the short version of what fungi is saying is that it doesn't matter, it's sort of a no-op.  combine that with the fact that some floss licenses have it, and some don't, i think it's "mostly harmless".18:55
mordred"The requirement to add this notice became obsolete and essentially deprecated on August 23, 2000 when Nicaragua became the final member of the Buenos Aires Convention to also become a signatory to the Berne Convention. As of that date, every country that was a member of the Buenos Aires Convention (which is the only copyright treaty requiring this notice to be used) was also a member of Berne, which18:55
mordredrequires protection be granted without any formality of notice of copyright."18:55
jeblairit is not in the standard apache license, nor is it in fsf licenses; bsd yes, and some others.18:56
sdagueyeh, it doesn't really matter if the apache header is in the file18:56
jog0ahh so it means nothing18:56
jog0thanks18:56
mordredsdague: that's not the same discussion18:56
jeblairmordred: i think you had a parse error18:56
sdaguemordred: yes, sorry, I think it was a parse error18:57
jeblairmordred: i think sdague said "as long as the apache header is in the file, also having ARR doesn't really matter"18:57
sdaguecorrect18:57
sdaguethat's what I meant at least18:57
mordredah. I thought he was bringing up the "do we need the headers in each file" thing again. :)18:57
jeblair(as compared to having ARR in the file and not having the apache header, which would be a bad situation)18:57
sdaguemordred: no way, I just landed H10218:57
mordredsdague: w00t18:57
sdagueto enforce it18:58
clarkbjeblair: very bad18:58
*** arborism has quit IRC18:58
*** ewindisch has quit IRC18:58
sdagueomg - http://status.openstack.org/zuul/ the devstack gate is empty18:59
fungion purpose i hope18:59
sdagueyeh, I guess it's the day of rest after H218:59
*** krtaylor has joined #openstack-infra18:59
jeblairyeah, i don't even think that's an error.  :)18:59
jeblairwell, it's break-everything day around here.  :)18:59
sdagueok, fixed that. (sent in a devstack change)19:00
sdaguesomething just doesn't feel right to me if that queue is empty19:00
clarkbI am going to grab lunch. Will pay attention to IRC so ping if things go further sideways19:00
*** arborism has joined #openstack-infra19:01
openstackgerritA change was merged to openstack-infra/zuul: Apply metajobs in order  https://review.openstack.org/3795519:03
*** hashar has joined #openstack-infra19:04
*** hashar has quit IRC19:05
*** hashar has joined #openstack-infra19:05
*** linuxmohan has quit IRC19:05
*** ewindisch has joined #openstack-infra19:07
*** anteaya has quit IRC19:18
*** nati_ueno has quit IRC19:21
*** nati_ueno has joined #openstack-infra19:22
*** sarob_ has quit IRC19:25
*** sarob has joined #openstack-infra19:26
*** sarob has quit IRC19:30
jeblairrestarted zuul19:31
jeblairrechecked/reverified19:31
jeblairthe requirements jobs seem to be correctly not running this time19:32
openstackgerritwill soula proposed a change to openstack-infra/jenkins-job-builder: Adding the Sauce OnDemand Plugin  https://review.openstack.org/3794719:40
*** mikal has quit IRC19:41
*** SergeyLukjanov has quit IRC19:41
*** vipul is now known as vipul-away19:41
*** SergeyLukjanov has joined #openstack-infra19:43
*** mikal has joined #openstack-infra19:43
*** ladquin is now known as ladquin_afk19:45
fungitoo awesome19:47
*** vipul-away is now known as vipul19:52
*** sarob has joined #openstack-infra19:53
*** cppcabrera is now known as cppcabrera_afk19:53
jeblairhttps://review.openstack.org/#/c/37329/19:59
jeblairThis change was unable to be automatically merged with the current state of the repository and the following changes which were enqueued ahead of it: 37207. Please rebase your change and upload a new patchset.19:59
jeblairfirst change to hit the new check!20:00
sdaguesweet20:00
fungii can't wait to see how much that improves gate throughput/parallelism20:02
sdagueyeh, on busy days that's going to be huge20:02
sdaguejeblair: did that also fix the kick out on first fail issue?20:02
*** ewindisch has quit IRC20:03
jeblairsdague: i believe that was fixed in the scheduler refactoring; i'm pretty sure we've spotted changes since being correctly being severed afteir their first job fails.20:03
sdagueok cool20:03
*** ewindisch has joined #openstack-infra20:07
*** sarob has quit IRC20:11
*** sarob has joined #openstack-infra20:12
*** changbl has quit IRC20:12
*** linuxmohan has joined #openstack-infra20:16
*** sarob has quit IRC20:16
openstackgerritA change was merged to openstack-dev/hacking: Add information about the Google style guide  https://review.openstack.org/3699720:17
*** plomakin has quit IRC20:18
*** plomakin has joined #openstack-infra20:19
*** linuxmohan has quit IRC20:22
openstackgerritA change was merged to openstack-infra/config: Add a node pattern for precise3k slaves.  https://review.openstack.org/3760920:24
pleia2so, using apache::vhost is the way we should do things, but cgit doesn't strictly have a docroot (it's a cgi script) so I'll have to take time to look into what I should populate it with20:24
pleia2but right now I need to head over to the airport20:25
lifelessttx: mordred: which guys?20:25
jeblairpleia2: you can probably just set it to an empty dir in /var/www/something20:25
jeblairpleia2: have a nice flight!20:25
pleia2jeblair: noted, thanks!20:26
SlickNikhey guys20:26
clarkbingy says duplicate keys in yaml should be an error20:26
SlickNikWhat's the status of https://review.openstack.org/#/c/37903/20:26
clarkbalso .yaml is preferred to .yml20:26
SlickNikIn the review it says: Starting gate jobs. http://status.openstack.org/zuul/20:26
clarkbpleia2: see what we did for gerrit and jenkins20:27
clarkbpleia2: mordred passes in something like "INVALID ARGUMENT"20:27
SlickNikBut if I look at that URL, there is no mention of 37903.20:27
SlickNikAny ideas what's going on?20:27
jeblairSlickNik: then it probably got lost in a recent zuul restart; 'reverify no bug'20:27
jeblairclarkb: "ingy says duplicate keys in yaml should be an error" context?20:28
SlickNikjeblair: Thanks for the update. Done!20:28
clarkbjeblair: pyyaml and consequently JJB allow dupliacate keys in your yaml and I think first one wins. eg if you define a template with the same name twice. Ingy was just in the office giving a talk on stuff and afterwards I asked him what the proper behavior should be and that is to be an error20:28
jeblairclarkb: *nod*20:29
lifelessmordred: clarkb: whats the account name for the infra HPCloud account?20:31
lifelesse.g. the one you want glance enabled on20:32
clarkblifeless: openstackjenkins2 and openstackci2 but the second one needs mordred to bless it with not costing my credit card money20:33
lifelessopenstackjenkins2 ack20:33
clarkblifeless: in other news I Think I saw it mentioned that the double cuonting of tests doesn't affect just tempest20:34
lifelessack20:35
lifelesswill look into it20:35
*** ewindisch has quit IRC20:36
lifelessclarkb: or should I ask for both ?20:37
lifelessclarkb: note that HP Cloud's glance only does one-user or all-user images at the moment20:38
lifelessclarkb: so if we enabled openstackjenkins2 images uploaded there won't be visible to openstackci220:38
lifelessclarkb: looking for an ack - one or both ?20:38
clarkblifeless: will enabling it break our ability to take snapshots and use hp provided images?20:39
clarkblifeless: devstack-gate depends on those services so we don't want to flip the switch if we will berak that20:40
clarkbhttp://engineering.cloudscaling.com/happy-3rd-birthday-openstack/20:40
clarkbI don't know how tripleo got in there but I will take the praise :)20:40
lifelessclarkb: no20:40
clarkbjeblair: fungi do you guys have an opinion on where we want to enable it?20:41
clarkbI am leaning towards both but lets see what they have to say20:41
* fungi reads back up to see what "it"is20:41
lifelessglance upload20:41
jog0why don't we have a sqlalchemy-migrate launchpad thingy yet?20:41
jog0for bug tracking etc20:41
lifelessHP cloud can permit end users to use it20:41
lifelessbut you need to submit a support request20:41
lifelessand some internal foo20:42
jeblairclarkb, lifeless: sounds great; why?20:42
lifelessI've been chasing the thread to understand it20:42
jog0clarkb: haha20:42
lifelessjeblair: tripleo gating will be massively easier if we can use the dib workflow directly.20:42
lifelessjeblair: which is glance based20:42
jeblairlifeless: is it also being enabled on rackspace?20:42
*** hashar has quit IRC20:42
*** sarob has joined #openstack-infra20:42
lifelessjeblair: Alex_Gaynor is chasing that20:42
jog0jeblair mordred: ^, bug tracking for sqla-migrate https://review.openstack.org/#/c/37656/20:43
lifelessjeblair: HPCloud support told me they plan to open glance up broadly eventually but no ETA yet20:43
clarkbthe nova unittests over time graph is nice because that wouldn't be possible without testr20:43
fungiclarkb: if the question is whether we want an openstack feature available to all our accounts on one of our openstack-based public cloud providers, i'm pretty sure the answer is ye20:44
fungis20:44
fungi(free newline for the taking)20:44
clarkbI will treasure that byte20:45
jeblairlifeless: ok, cool.  i agree with clark: both, but disruption would be bad.  and at the risk of reiterating something that may have been said already -- i'm not comfortable using it in production until we can use it globally20:45
fungithe safety factor could be mitigated by initially enabling it for openstackci2 and testing what we want there while it's not being used yet, then adding it for openstackjenkins2 once we think it checks out20:46
lifelessjeblair: so, I don't know how long the lead time is. I'd rather ask for it and not use it for a bit (or only use it for testing whether a new job works?)20:46
lifelessthan not ask for it until its available everywhere.20:46
jeblairlifeless: yeah, i'm totally cool with getting it in there and starting to work with it.20:46
*** plomakin has quit IRC20:47
lifelessok20:47
jeblairlifeless: i would personally love that feature.  just want to be clear.  :)20:47
*** cp16net is now known as cp16net|away20:47
*** plomakin has joined #openstack-infra20:47
*** cp16net|away is now known as cp16net20:48
lifelessjeblair: mordred: you have mail now about this.20:50
*** sarob has quit IRC20:50
jeblairfungi, clarkb, mordred: i'm going to get certs for review, jenkins, etherpad, wiki.20:52
fungijeblair: great. i know we're down to about a week left now20:53
jeblair(those are the ones that expire in the next 6 months -- i'd like to align their renewals for efficiency (and safety))20:53
fungion a couple of them at least20:53
openstackgerritAntoine Musso proposed a change to openstack-infra/storyboard: gitignore /*.egg  https://review.openstack.org/3796920:53
jeblairshould i also get one for git.o.o?20:53
clarkbjeblair: ++ I was going to suggest that20:53
jeblairclarkb: suggest alignment or git?20:53
fungijeblair: i think it would be awesome to do https on git.o.o20:53
clarkbjeblair: sure? though I foresee that git:// will be the popular protocol there which won't be secure in any way20:53
clarkbjeblair: alignment20:54
jeblairtrue.  kernel.org's cgit service is behind https20:54
jeblairlifeless: thanks20:54
pabelangergit.o.o? Are back up for github or migrating everything to it?20:54
fungipabelanger: our github presence is merely a mirror. this will be another, but more officially openstack, mirror20:55
jeblairpabelanger: we're going to refer to it in all our docs and scripts;  we'll continue to mirror to github because it's useful for a lot of ppl.20:55
fungipabelanger: afaik github mirrors will also remain in place20:55
fungiwhat jeblair said ;)20:55
jeblair3 coremudgeons agree.  :)20:55
pabelangerTrue, gerrit would be the official I guess, everything a mirror :)20:56
fungijeblair: that is a strict majority. so be it20:56
clarkbpabelanger: it will allow us to host our own git daemon to use the git protocol20:56
fungipabelanger: yep, we just don't want to point the entire world at gerrit for cloning/pulling because... java20:56
clarkbpabelanger: which will hopefully speed up some of our testing20:56
jeblairfungi, clarkb: 2 year 2048?20:57
fungijeblair: sounds fine to me20:57
*** sarob has joined #openstack-infra20:58
fungianything >1024 is safe for quite a while most likely20:58
clarkbjeblair: sounds good20:58
fungimodulo new and interesting ways to efficiently factor very large numbers20:58
clarkbfungi: you better be knocking on wood20:59
pabelangerclarkb: cool, does that mean by passing gerrit actually running the git server?20:59
*** mgagne has quit IRC20:59
fungiclarkb: i for one welcome our new factorizing masters20:59
pabelangerTrying to understand where the performance impact is20:59
clarkbpabelanger: we already bypass gerrit in most places and hit an http mirror so yes and yes20:59
pabelangerI see20:59
clarkbpabelanger: git over http is much faster now than it used to be but it is still the slowest protocol behind ssh and git20:59
annegentle_does any other project in OpenStack delegate approvers for a particular directory within a repo? Or do they just start a new repo with "core" reviewers? Asking on behalf of the training manuals idea-guys.20:59
clarkbpabelanger: so the performance benefit is in talking a more efficient protocol when updating workspaces for jobs and so on21:00
fungipabelanger: in many ways, this is just us moving the local apache-based mirror we already run on our gerrit server today to a separate host with a distinct name and nifty additional features21:00
clarkbannegentle_: I think that feature has been requested but gerrit doesn't support it directly21:00
clarkbannegentle_: instead these things are managed by social convention21:00
pabelangerclarkb: fungi: Cool!  Need to logoff now, but will chat more later21:01
pabelangercya21:01
*** pabelanger has quit IRC21:01
fungiannegentle_: in places where we need to ensure that, we go with separate repositories/projects with distinct groups21:01
fungiannegentle_: but i agree with clarkb, letting social pressures dictate that sort of thing leads to improved harmony amongst devs in most cases21:02
openstackgerritAntoine Musso proposed a change to openstack-infra/storyboard: projects view now accepts projects with spaces  https://review.openstack.org/3797721:03
*** ewindisch has joined #openstack-infra21:03
clarkbI am going to try doing another code review pass before the weekend21:03
clarkbanything in particular folks want me to not miss?21:04
fungiclarkb: nothing specific from me21:04
*** rcleere has quit IRC21:05
fungii'm heading out to grab some dinner in a bit, then coming back to add precise3k slaves and confirm pbr/oslo.config unit tests on them21:06
openstackgerritA change was merged to openstack-infra/config: Properly root log path in logstash gearman client.  https://review.openstack.org/3795321:06
openstackgerritClark Boylan proposed a change to openstack-infra/jenkins-job-builder: Tests: Use a testscenarios for running tests.  https://review.openstack.org/3699021:08
clarkbfungi: jeblair: I am going to readvertise the series here https://review.openstack.org/#/c/14893 ^ should make them both mergable21:09
lifelessjeblair: fungi: clarkb: so we need to be in the us-east beta region for glance access.21:09
lifelessthats just another region which you can activate via the console21:09
clarkblifeless: I don't think we want beta21:09
clarkblifeless: oh21:09
clarkblifeless: well, we want it to homogenous across the AZs but I suppose we can go ahead and enable it in one for testing21:10
lifelessright21:10
clarkbjeblair: fungi ^ sane?21:10
fungiso we can try out glance in us-east on our accounts, sounds like. which means it probably also is unlikely to break our production zones anyway. wfm21:10
jeblairclarkb, lifeless: what's the correlation between AZs and regions?21:10
jeblair"az1, az2, az3, us-east": one of these is not like the others.21:10
lifelessjeblair: us-west is the current hpcloud region21:11
lifelesshttps://account.hpcloud.com/services21:11
clarkbjeblair: az1 az2 and az3 are in us-west21:11
jeblairk, so az1-3 are in us-west21:11
lifelessUS West AZ 121:11
clarkbjeblair: regions house AZs21:11
jeblairwhat azs are in us-east-beta?21:11
lifelesscompute 13.5 is in us-east, and may have a different region/az layout21:11
lifelessjeblair: us-east21:11
lifelessjeblair: there is no us-east non-beta az atm21:12
jeblairok.  sounds nicely separate from everything we're doing now; enabling it sounds sane.21:12
lifelesscool21:12
*** dirk has joined #openstack-infra21:12
lifelessI suspect you may need to request access to us-east from both accounts before the glance request can go in.21:12
jeblairclarkb: want to take care of that?21:13
clarkbya, I will do that now21:13
clarkblifeless: there is no option to do this in the web console21:15
lifelessclarkb: https://account.hpcloud.com/services21:15
lifelessclarkb: bottom of the page 'Beta Services'21:15
lifelessclarkb: Compute US East v13.5 not activated Request Access21:15
clarkbgah why is it under beta21:15
clarkbput it in compute and note it is beta21:15
lifelessclarkb: possibly because naive users click on things without understanding21:16
openstackgerritDirk Mueller proposed a change to openstack-dev/hacking: Don't crash on tests when git is not installed  https://review.openstack.org/3787021:16
lifelessclarkb: possibly because they think naive users ...21:16
lifelessclarkb: who knows :>21:16
*** ewindisch has quit IRC21:16
jog0mordred: ping on entrypoints21:17
fungii can confirm naive users *definitely* click on things without understanding21:18
*** cp16net is now known as cp16net|away21:18
*** ewindisch has joined #openstack-infra21:19
fungievery time i check to see whether any users have tried to add the internal system cla and usg cla in gerrit, in deference to the very shouty warnings on and in them, i find a handful21:19
*** dhellmann is now known as dhellmann_21:19
clarkblifeless: requests have been submitted for accounts and I have noted the desire to use glance in the requests21:19
fungithough i think in that case it's mostly people with a very tenuous grasp of english getting desperate when they start having issues with the icla21:20
*** krtaylor has quit IRC21:21
lifelessclarkb: cool21:21
* fungi dines... bbl21:21
lifelessclarkb: the request for glance is in, I believe. jeblair and mordred are being copied on everythnig21:22
clarkblifeless: perfect, thank you21:22
lifelessis there an alias for 'this gets to -infra' that I could use rather than a cc-list ?21:22
clarkblifeless: not really21:22
clarkbthere is the infra maillist but it requires membership21:22
lifelessjust saying:>21:23
clarkbpleia2: does your plane have wifi?21:27
clarkbpleia2: I have questions21:27
*** rnirmal has quit IRC21:27
clarkbmeh I will leave them in the review21:27
clarkbjeblair: any particular reason for not approving https://review.openstack.org/#/c/35360/3 ? (maybe the soft freeze)21:31
*** mrodden has quit IRC21:31
reeddo you know what a deployment advisor is? https://ask.openstack.org/question/3232/deployment-advisor-in-openstack/21:31
jeblairclarkb: so you and fungi could look at it21:32
clarkbjeblair: ok, I will let fungi eye it over after dinner21:33
*** sarob has quit IRC21:33
*** sarob has joined #openstack-infra21:33
Shrewsmordred: or any pbr guys... this look familiar to anyone? http://paste.openstack.org/show/40967/21:34
*** ewindisch has quit IRC21:35
clarkbShrews: ya, I think that is a case of needing latest setupttools/distribute21:36
*** sarob has quit IRC21:36
clarkbShrews: the merging of those two projects has caused some headaches and upgrading smooths it out21:36
*** sarob_ has joined #openstack-infra21:36
*** Ryan_Lane has quit IRC21:36
pleia2clarkb: hey, I have a half hour before boarding21:37
Shrewsclarkb: setuptools was upgraded in an attempt to fix it. should distribute be installed as well?21:37
pleia2clarkb: yeah, so they are required for one of the .erb files that jeepyb populates21:37
clarkbShrews: they should be the same at this point, SpamapS mentioned that he had to remove the apt installed version of setuptools too21:38
clarkbpleia2: thanks21:38
jeblaircan't we just define them inside the class?21:38
pleia2jeblair: what class?21:38
jeblairas in, if they aren't used, why do they need to be class parameters?21:38
jeblairpleia2: openstack_project::git21:39
clarkbjeblair: we can do that21:39
LinuxJediShrews: just tried upgrading distribute too, still not working :(21:39
*** vipul is now known as vipul-away21:41
*** avtar has quit IRC21:41
jog0Shrews: I think I did a rm -rf .tox and upgraded pip and distribute etc to the latest and reran21:41
clarkbjeblair: fungi mordred I am writing up a list of changes that I plan on merging this afternoon if no additional reviews are posted https://etherpad.openstack.org/infra-reviews21:42
clarkb*no additional negative reviews21:42
*** ewindisch has joined #openstack-infra21:42
*** _TheDodd_ has quit IRC21:42
LinuxJedijog0: this is on a deployment, no .tox21:43
*** lcestari has quit IRC21:43
jeblairclarkb: lgtm21:43
*** gyee has quit IRC21:43
*** mriedem has quit IRC21:43
jog0LinuxJedi: ouch21:44
LinuxJediShrews: figured out a fix, but it is sort of dangerous21:45
jog0anyone ex Drizzlers here at the moment?21:45
LinuxJedijog0: me, Shrews, mordred, pcrews21:45
Shrews3 of those are quite busy, atm21:46
jog0so the sqlite migration snafu21:46
jog0since someone stuck me as a sqlalchemy-migrate maintainer I started  digging into this one21:46
*** vipul-away is now known as vipul21:47
LinuxJedijog0: we are in the middle of a huge deployment, give us a couple of hours21:47
jog0LinuxJedi: sure21:47
jog0the TL;DR is can we actaully replace sqlite with drizzle as our testing backend21:47
jog0but it can wait till monday21:47
openstackgerritElizabeth Krumbach Joseph proposed a change to openstack-infra/config: Fix jeepyb class issue on git.o.o  https://review.openstack.org/3795421:54
*** odyssey4me has quit IRC21:54
*** ewindisch has quit IRC21:55
*** koolhead17 has quit IRC21:56
*** sarob_ has quit IRC21:57
*** koolhead17 has joined #openstack-infra21:57
openstackgerritSergey Lukjanov proposed a change to openstack-infra/config: Add more jobs for Savanna projects  https://review.openstack.org/3798721:58
*** sarob has joined #openstack-infra21:58
clarkbhttps://pypi.python.org/pypi/xattr/0.7.0 is a new version that appears to have broken the gate22:01
clarkbsdague: jeblair fungi ^22:01
clarkbnot sure how that snuck paste the mirror jobs22:02
clarkbs/paste/past/22:02
*** vipul is now known as vipul-away22:02
*** sarob has quit IRC22:02
clarkbwell thats unfortunate https://jenkins.openstack.org/job/periodic-mirror-python27/140/console22:03
*** bknudson has joined #openstack-infra22:04
* clarkb attempts a fix22:05
mordredjog0: yes. I agree. I thnk it will be easy22:06
*** cp16net|away is now known as cp16net22:06
jog0mordred: got another q for you: https://pypi.python.org/pypi/zc.buildout/2.2.022:07
jog0that came recomended as a what to try instead of entry_points via pkg_resources22:07
jog0which slow down the more things are in python22:07
mordredoh god. no22:08
jog0so the question is have you ever heard of / used that?22:08
mordredwe are not switching to a completely new build and packaging system22:08
mordred:)22:08
clarkbmordred: jeblair why do C libs that python pacakges need to compile against not get isntalled by puppet on the d-g nodes?22:08
jog0running 'time python -c "import pkg_resources"' on a devstack instance takes 0.2 seconds or so22:08
mordredclarkb: because we expect devstack to request them22:08
clarkbmordred: jeblair: I think the fix for xattr here is to make sure that libffi-dev is installed on the nodes and I am trying to figure out if it needs to go on the d-g hosts too22:08
mordredjog0: grok proble. I'd rather go back to hand-coded bin/ scripts22:09
jog0in tripleo it takes .08 seconds in the bigger venvs22:09
clarkbmordred: ok, so devstack will need to be patched independently if necessar22:09
mordredthan switch to zc.buildout22:09
jog0mordred: thats what I figured22:09
mordredjog0: zc.buildout is a parallel system to pip/virtualenv22:09
mordredit would change EVERYTHING22:09
jog0ohhh  hahaha22:09
*** krtaylor has joined #openstack-infra22:10
openstackgerritClark Boylan proposed a change to openstack-infra/config: Install libffi-dev on our slaves.  https://review.openstack.org/3799022:10
clarkbmordred: jeblair fungi ^ once that is merged and applied the mirror script will need to be rewritten22:10
jog0mordred: I included you on the thread that lifeless started about this22:10
clarkbmordred: jeblair fungi also why is the mirror script uploading stuff if it fails to install packages? I thought it was supposed to protect us from that22:10
lifelessjog0: ?22:11
* clarkb investigates that problem22:11
lifelessmordred: what project were you referring to above btw ?22:11
jog0lifeless: the rootwrap thing22:11
lifelessjog0: ah yes22:11
mordredclarkb: can you describe the problem you are seeing a little more?22:12
clarkbmordred: https://jenkins.openstack.org/job/gate-keystone-pep8/6418/console is happening because https://jenkins.openstack.org/job/periodic-mirror-python27/140/console did not properly fail22:12
lifelessmordred: ttx: to be clear, I"m not talking about rootwrap, something else - about 5 horus back22:13
mordredclarkb: ok. so we didn't successfully install libffi22:13
lifelessyou say I could get a release 'of that lib'22:13
mordredon the mirror builder22:13
lifelessbut not what lib that is.22:13
mordredbut then we uploaded libffi to our mirror?22:14
clarkbmordred: correct, but we should never have rsync'd xattr to the mirror22:14
clarkbmordred: no we uploaded xattr to the mirror which had dependencies that failed to install22:14
mordredAH22:14
mordredweird22:14
clarkbthe dependencies failed to install because libffi-dev was not present22:14
mordredhow did xattr get ... oh jeez22:14
* mordred groks the logic error22:14
clarkbmordred: so immediate fix is the one I have proposed above22:14
clarkbbut we need to fix the mirror jobs too22:14
mordredwell, it's not going to be a tiny change22:15
clarkb:/22:15
clarkbmaybe I will let it slide until later22:15
mordrednot HUGE - but not a one liner22:15
mordredfile a bug or whatnot - it's important22:15
clarkbwill do22:15
mordredbut yeah, right now, workaround is a good idea22:15
clarkbmordred: can I get reviews on that workaround?22:15
pleia2ok, plane now, hopefully that change i made gets things sorted22:15
mordred+2'd22:15
*** ladquin_afk has quit IRC22:16
mordredclarkb: because we process the pip cache...22:16
mordredclarkb: things will get downloaded22:16
mordredinto the cache22:16
*** linuxmohan has joined #openstack-infra22:16
mordredoh wait.22:16
mordredno, that's not right22:16
clarkbmordred: is that cache persistent? it is right?22:16
mordredthe double download/freeze cycle should handle this22:17
mordredjog0: where's the thread lifeless started about this?22:18
*** Ryan_Lane has joined #openstack-infra22:19
clarkbhttps://bugs.launchpad.net/openstack-ci/+bug/120319822:19
jog0mordred: I sent it to your private email22:20
*** linuxmohan has quit IRC22:20
mordredjog0: gotit22:20
*** datsun180b has quit IRC22:21
clarkbI have approved my workaround. I will run puppet on the mirror hosts once that is in and happy then trigger the mirror jobs again22:21
clarkbby the time that is all done normal puppet should've puppetted the other slaves22:21
jog0http://www.mail-archive.com/distutils-sig@python.org/msg17454.html22:21
jog0mordred: thats where it came from22:22
mordredjog0: ok. I just responded22:22
openstackgerritA change was merged to openstack-infra/config: Install libffi-dev on our slaves.  https://review.openstack.org/3799022:22
mordredjog0: I believe we could VERY easily fix this in pbr22:23
*** mrodden has joined #openstack-infra22:23
clarkb#status notice Projects with a dependency on xattr are failing tests due to unresolved xattr dependencies. Fix should be in shortly22:23
openstackstatusNOTICE: Projects with a dependency on xattr are failing tests due to unresolved xattr dependencies. Fix should be in shortly22:23
mordredjog0:  still specifying the entrypoint console_script as we do now - but intercept it and install a boilerplate script like the entrypoints one22:23
mordredjog0: but that does an import and not a pkg_resources lookup22:23
jog0mordred: I like the words 'Very easily fix'22:23
mordredjog0: I did not want to be that specific on the mailing list22:24
*** mrodden1 has joined #openstack-infra22:24
mordredjog0: http://paste.openstack.org/show/40974/ is what testr does22:25
jog0cool that should be much faster22:25
mordredjog0: I think we can do that without checking in 1000 boilerplate crappers into our trees22:25
mordredjog0: gimme a sec22:25
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Tests for publishers  https://review.openstack.org/1489322:26
jog0well we only *really* need it or any binary that is short lived and we expect to call more then once a second22:26
*** linuxmohan has joined #openstack-infra22:26
jog0for any*22:26
openstackgerritA change was merged to openstack-infra/config: Add python unittests to JJB.  https://review.openstack.org/3761722:27
mordredjog0: sure. but it's a super-simple interface - so if we solve it once, we might as well just solve it across the board22:27
openstackgerritA change was merged to openstack-infra/jenkins-job-builder: Tests: Use a testscenarios for running tests.  https://review.openstack.org/3699022:27
*** mrodden has quit IRC22:27
jog0mordred: true, although I wonder if there is any side effect IE why use pkg_resorces for this at all22:27
*** ewindisch has joined #openstack-infra22:28
jog0maybe it does some neato trick or something22:28
mordredjog0: why us? or why in general?22:28
mordredit does not22:28
mordredhttp://paste.openstack.org/show/40975/22:28
mordredthere is what you get from entrypoints22:28
mordredthe main difference is checking for version22:28
jeblairclarkb: does the mirror script ever exit nonzero?22:29
mordredbut, you know what? I don't think we need to check that at runtime. it's silly22:29
openstackgerritA change was merged to openstack-infra/config: Fix blueprint and bug parser to require a separator  https://review.openstack.org/3685322:29
*** prad has quit IRC22:29
jog0mordred: and that magic load_entry_point command22:30
jog0but yes, why in general and why us22:31
*** linuxmohan has quit IRC22:31
jog0I just don't want to fix one problem and create a worse one22:31
mordredjog0: why in general is so that we have the bulk of our code in nova/*22:31
mordredand not have real logic in a bin/*py script22:31
mordredwhere it's hard to unittest22:31
clarkbjeblair: no I don't think it does22:31
clarkbjeblair: it will need to track the status of each install and if one fails exit(1)22:32
mordredor, well, really, it's the same answer for both :)22:32
jog0mordred: I understand why we do it, but why  is there taht load_entry_point magic22:32
mordredwell, that's just how setuptools and friends do this22:32
clarkbjeblair: maybe. I think we wanted a single failure to not break everything else22:32
clarkbexcept a single failure did that by getting past the mirror script22:32
dstufftin a console_script?22:32
mordreddstufft: yeah22:32
dstufftBecause setuptool supports multi version installs22:32
dstufftso you have multiple versions of things installed to the same python22:32
dstufftand the load_entry_point stuff activates the appropiate versions22:33
jog0dstufft: so even multiple versions of dependencies and whatnot?22:33
dstufftjog0: sure22:33
dstufftInstalling with pip forces multi version stuff to go away because it's a tremendous hack22:33
mordreddstufft: where would it put such a script?22:33
*** ^demon has quit IRC22:33
mordredah22:33
clarkbpuppet is running on the mirror slaves now22:33
jog0dstufft: ahh so that the  magic it adds22:33
dstufftthere's actually a discussion on distutils-sig about this right now22:33
mordredok. so, the fact that we're pip heavy means that the benefit of the complexity is lost on us22:33
*** ^demon has joined #openstack-infra22:34
jog0wait, so entry points do somethign that pip prevents from happening22:34
jog0mordred: python fail22:34
dstufftentry points themselves are just progrmatic hooks22:34
jog0that have support for multiple versions of everything which presumablly is why importing pkg_resources is slow: it walks the full tree22:35
dstufftOh is this a speed thing22:35
jog0dstufft: *nod*22:35
clarkbmirror job builds have started22:35
dstufftIt's looking in every .egg-info directory for a entry_points.txt22:35
clarkbhopefully this will all be behind us in about 10 minutes22:35
jog0in devstack: time python -c "import pkg_resouces"22:36
dstufftwhich it then loads22:36
dstufftloads ~= opens + reads22:36
dstufftit's an ini file I believe22:36
dstufftdoes openstack support windows?22:36
jog0dstufft: http://paste.openstack.org/show/40952/22:36
clarkbdstufft: some pieces of it22:36
clarkbdstufft: a small subset22:36
dstufftIf you don't need windows support you can approximate the same thing without a dependency on pkg_resources just by shipping a python script using the distutils scripts element22:37
*** danger_fo is now known as danger_fo_away22:37
dstufftthe problem is windows22:37
dstufftbecasue you need an exe for it to work on windows22:37
jog0dstufft: can I quote you on that "the problem is windows"22:37
*** Ryan_Lane is now known as gerrit-wm22:37
mordredwe just shipped scripts until this last cycle22:38
jeblairpretty sure rootwrap isn't expected to work on windows22:38
jog0jeblair: good point22:38
mordredwhich I believe will be the solution to our probleem22:38
mordredm22:38
dstufftIf it doesn't need windows support you can just ship a regular python file without an extension and get an appromxiation of what console_scripts gives you without the machinary22:38
jog0\o/22:39
mordredright. I'm on it. I have a solutoin in mind. I am hacking on it right now22:39
jog0faster22:39
dstufftif it does need windows you can still do that, you just need to ship an exe like setuptools has22:39
dstuffthttps://bitbucket.org/pypa/setuptools/src/9dc434ac0308749d564d721a19ee412c2e79754f/launcher.c?at=default22:40
dstufftis the exe setuptools ships22:40
dstufftwell the code for it22:40
*** ewindisch has quit IRC22:40
dstufftbasically it just loads a .py file named the same as the exe22:40
mordredI'm going to intercept the console_scripts on linux and ship a simple script22:40
clarkbwell darn https://jenkins.openstack.org/job/periodic-mirror-python27/141/console22:40
mordredand let it passthrough on windows22:40
*** gerrit-wm has quit IRC22:40
mordredso that setuptools will do the right thing there22:40
*** UtahDave has quit IRC22:40
mordredand the day we care about speed on windows, we can revisit22:40
clarkbI am not super interested in making cffi compile on ubuntu precise. should we just pin xattr?22:41
mordredclarkb: yes22:41
dstufftclarkb: strange, cffi works fine for me on precise22:41
dstufftthough I haven't tested xattr22:41
*** Ryan_Lane has joined #openstack-infra22:42
clarkbdstufft: is there another missing lib?22:42
dstufftmordred: fwiw distutils-sig will have a better solution to this sooner or later22:42
* clarkb does a little more investigating22:42
dstufftclarkb: Not sure offhand, however!22:42
dstufftpaging Alex_Gaynor :D22:42
dstufftpretty sure he did the cffi port of xattr22:43
Alex_Gaynordstufft: what'd I do?22:43
mordreddstufft: great! I'll have one today :)22:43
dstufftAlex_Gaynor: cffi + xattr borken for openstack22:43
Alex_GaynorCan you better specific borken?22:43
dstuffthttps://jenkins.openstack.org/job/periodic-mirror-python27/141/console22:43
Alex_GaynorI did port xattr to cffi22:43
clarkbAlex_Gaynor: first it was missing libffi-dev22:43
mordredAlex_Gaynor:22:43
Alex_Gaynorand then bob did a release22:43
mordredAlex_Gaynor: https://jenkins.openstack.org/job/periodic-mirror-python27/141/console22:43
clarkbAlex_Gaynor: but now with libffi-dev installed we get ^22:43
jog0mordred: nice solution22:44
* clarkb enters a holding pattern in hopes Alex_Gaynor will fix all the things22:44
*** SergeyLukjanov has quit IRC22:44
Alex_Gaynorhmm, looking22:44
clarkbit does the same on centos6 https://jenkins.openstack.org/job/periodic-mirror-python26/139/console22:45
Alex_Gaynorlooks like XATTR_MAXNAMELEN, XATTR_NOFOLLOW, XATTR_NOSECURITY are only defined on some systems22:46
clarkbdon't tell me you only tested on OS X :{22:46
clarkber :P22:46
dstufftThere are other OS's besides OSX? :V22:47
mordredOSX is an OS?22:47
clarkbfwiw we only test on centos and precise and the poor BSDs and OS X get broken sometimes22:47
*** cppcabrera_afk is now known as cppcabrera22:48
jeblairclarkb: openstack runs on bsd and osx?22:48
mordredjeblair: no22:48
clarkbjeblair: vishy does all of his testing without virtualenvs on OS X22:48
clarkband people use *BSD for stuff apparently22:48
clarkbso ya kinda22:48
jeblairclarkb: yes, but is there an OSX openstack cloud?  public or private?22:48
dstufftFreeBSD supremacy22:48
clarkbjeblair: not that I am aware of22:48
clarkbAlex_Gaynor: so, I am guessing that won't be a quick fix? I suppose I should pin xattr for now?22:49
mordredjeblair: no. because not even apple uses it as a server os22:49
jeblairclarkb: we test all the platforms we support.22:49
*** pabelanger_ has quit IRC22:49
clarkbjeblair: good point22:49
dstufftmordred: you mean a bunch of mac minis shoved in a rack doesn't count as servers?22:49
Alex_Gaynorclarkb: Define quick? I think it'll be fixed in <1.5 hours :}22:49
* mordred belly laughs22:49
*** gyee has joined #openstack-infra22:49
*** pabelanger has joined #openstack-infra22:49
jeblairclarkb: +1pin22:50
dstufftI'd be happy if Ic ould legally run OSX on the cloud22:50
clarkbok whipping that up now. Going to simply delete the package from the mirror first22:50
*** cp16net is now known as cp16net|away22:50
mordreddstufft: honestly - not trolling but a) why b) yeah, I spoke to the apple folks about that - their buildfarm guys agree with you :)22:51
jeblairclarkb, mordred: i'm trying to figure out exactly how wrong/right the mirror script was...22:51
jeblairclarkb, mordred: it's really nuanced...22:51
vishyclarkb: why ya gotta tell everyone about that22:51
vishy:)22:51
dstufftmordred: the clue is in the second point22:51
clarkbvishy: because its crazy :)22:51
mordreddstufft: yeah. I used to have osx builders in the drizzle build farm. of course, we had cloud builers for every other os22:51
mordreddstufft: but for osx, we had a mac pro tower in my living room22:52
jog0btw there were apple people at the last summit planning on using OpenStack internally on Oracle RHEL22:52
dstufftmordred: my dream is to someday have an OSS buildfarm for PyPI, push up a sdist 2.0, plop out wheels for all the various archs22:52
dstufftmordred: this pretty much requires the ability to use a VM to run OSX though :[22:52
clarkbmordred: jeblair: so actually Alex_Gaynor thinks <1.5 hours will be fixed I think I won't pin it for now and just rely on the fact that the mirrors don't update very often22:52
jeblairclarkb, mordred: on the one hand, 'external package changes and everything blows up and we pin the reqs repo' is something that we currently (say that) we want.22:52
*** SergeyLukjanov has joined #openstack-infra22:52
mordreddstufft: well, I might know some people who know a lot about really scalable build farms who take submissions from untrusted sources22:52
clarkbmordred: jeblair since I removed the package from the mirror by hand we should be good until Alex_Gaynor has pushed a fix22:52
jeblairclarkb, mordred: on the other hand, if this had been a reqs repo change, it may have "passed tests" and been permitted22:53
*** vipul-away is now known as vipul22:53
mordredjeblair: I think that in this case though the question is logically, how did xattr actually wund up in th eupload22:53
mordredaside from whether we want or don't want the break22:53
mordredas I understand the current mirror logic, it makes  no sense to me22:53
jeblairclarkb, mordred: and as one of you pointed out earlier; if bar is permanently broken on pypi, we don't want ignore updates to foo.22:54
mordredjeblair: ++22:54
jeblairmordred, clarkb: i think before we fix it, we need to start documenting exactly what we want it to do in all cases, maybe on this wiki page: https://wiki.openstack.org/wiki/Requirements22:54
mordredjeblair: +10022:55
jeblairmordred, clarkb: because i'm almost certain we'll end up with wack-a-mole otherwise22:55
Alex_Gaynorclarkb: ok push is committed, I just realized I don't have pypi push rights, so I'm asking the maintainer22:55
clarkbjeblair: ++22:56
clarkbAlex_Gaynor: ok, no rush now as I have a temprorary fix22:56
dstufftmordred: this is down the road stuff so it's not needed yet22:57
mordreddstufft: totes. just saying ... I might be interested in hacking on that with you22:57
Alex_Gaynorclarkb: release done!22:58
dstufftmordred: cool sounds good22:58
*** nati_ueno has quit IRC22:59
openstackgerritDirk Mueller proposed a change to openstack/requirements: Raise eventlet to 0.13.0  https://review.openstack.org/3799622:59
openstackgerritA change was merged to openstack-infra/config: Add tripleo-incubator to openstack.  https://review.openstack.org/3536023:00
jeblairah, i'm having trouble getting ssl certs because the foundation executive director forwarded hostmaster@openstack.org to his account23:01
clarkbAlex_Gaynor: rerunning mirror jobs now23:03
Alex_Gaynorclarkb: cool23:03
clarkbjeblair: :/23:03
*** mrodden1 has quit IRC23:05
*** mrodden has joined #openstack-infra23:05
*** locke1051 has quit IRC23:05
lifelessAlex_Gaynor: oh hi :)23:05
*** locke105 has joined #openstack-infra23:05
lifelessAlex_Gaynor: I've got a request happening to get the infra team access to our glance in the new US-East region; but jeblair etc can't use that in prod until all the infra clouds support it - so this is me pinging you about the same for Rackspace :>23:06
lifelessmordred: jeblair: I just had an evil idea; what if there was a takeovernode backend for glance23:08
openstackgerritA change was merged to openstack-infra/config: Ensure latest setuptools is present.  https://review.openstack.org/3670823:08
clarkbAlex_Gaynor: running builds look much better23:10
Alex_Gaynorclarkb: <phew>23:10
openstackgerritA change was merged to openstack-infra/config: Fix jeepyb class issue on git.o.o  https://review.openstack.org/3795423:11
openstackgerritClark Boylan proposed a change to openstack-infra/config: Tune ElasticSearch settings.  https://review.openstack.org/3759823:12
lifelessmordred: jeblair: specifically, a infra-owned glance instances running in a region; upload an image to it, and it spawns a vm, does takeovernode for you, snapshots, done.23:14
clarkbxattr 0.7.1 is in the mirror now. I think we should be fine23:15
jeblairlifeless: that sounds wonderfully rube-goldberg, but seems like it would work.23:16
mordred:)23:16
mordredI can't imagine ANYTHING that would go wrong ;)23:17
lifelessmordred: I can :>23:17
dirkclarkb: does xattr 0.7.1 fix http://logs.openstack.org/91/37991/2/check/gate-ceilometer-python26/3487/console.html ?23:17
*** arborism has quit IRC23:18
*** openstackgerrit has quit IRC23:18
clarkbdirk: it should23:18
jeblaircerts acquired.23:19
*** openstackgerrit has joined #openstack-infra23:19
clarkbdirk: well that particular error was something else. there were two related failures but the other failure should also be correct now23:19
zaroclarkb: looks like this merged... https://review.openstack.org/#/c/37529/23:19
openstackgerritA change was merged to openstack-infra/config: Optimize ElasticSearch indices later.  https://review.openstack.org/3739723:19
clarkbdirk: tl;dr try it again and let us know if you see any more breakage23:19
zaroclarkb: now what do i need to do to tag?23:19
openstackgerritA change was merged to openstack-infra/config: Tune ElasticSearch settings.  https://review.openstack.org/3759823:20
clarkbzaro: we will need to populate that group then you should be able to push tags23:21
clarkbzaro: I am doing about 4 different things at the moment but can add that to the list23:21
zaroclarkb: cool. thanks.23:21
*** arborism has joined #openstack-infra23:22
clarkbmordred: fungi: I think the jeepy install on review.o.o must've hit that setuptools thing23:23
clarkbmordred: fungi: can one of you guys confirm? and perhaps fix it? fixing it will make it so that gerrit groups are created as being self owned23:24
*** nati_ueno has joined #openstack-infra23:25
jeblair(i wish i could do file insertions in hieradata yaml; the ssl cert stuff is unweildy)23:26
jeblair(maybe i will write a python script to edit the yaml)23:26
clarkbjeblair: hiera does support a bunch of backends. there may be a clean way to do that23:26
*** SergeyLukjanov has quit IRC23:28
jeblairclarkb: there seems to be a 3rd party mysql one at least.23:29
jeblairi think i'll write a quick script for now though.23:29
clarkbzaro: can you tell me what your gerrit account id is for the account you are using?23:30
clarkbzaro: its under the settings page23:30
clarkbzaro: and if you like I can walk you through the process of tagging a thing on Monday23:30
clarkbzaro: its a bit painful to correct if smoething goes wrong23:30
*** michchap has joined #openstack-infra23:31
fungiosx/*bsd support... not for server projects, but it's great that people can potentially run the clients there23:37
fungiclarkb: review.o.o was where we hit it (and fixed it) originally. i'm wondering how setuptools/distribute could have downgraded since then23:39
zaroclarkb: account id is 698723:39
zaroclarkb: i will wait for your guidance.  thanks.23:39
jeblairfungi: maybe we should start testing the clients on bsd23:40
fungijeblair: i take it jonathan's e-mail isn't over quota and bouncing like todd's then, at least (lloyd said he got an over-quota bounce for hello@o.o which forwarded to todd@o.o)23:40
jeblairfungi: i unforwarded the account and asked the ca to resent the dlv emails23:41
jeblairfungi: and sent an email to jonathan letting him know23:41
fungioh, even better ;)23:41
clarkbfungi: oh, maybe it was fixed and jeeepyb just wasn't updated?23:41
clarkbfungi: could be a different problem but my jeepyb fix to group creation doesn't seem to be on review.o.o23:41
*** michchap has quit IRC23:42
* fungi looks23:42
clarkbzaro: gearman-plugin-core is ready to go. We can push a tag on monday23:43
openstackgerritMonty Taylor proposed a change to openstack-dev/pbr: Replace entry_points console_scripts  https://review.openstack.org/3800023:43
*** gyee has quit IRC23:43
mordredjog0, dstufft: ^^23:43
mordredI need to add unittests23:43
clarkbjeblair: ninja email admin23:43
mordredbut I wanted to show you the patch first23:43
fungiclarkb: ahh, looks like we have distribute installed from pip (and at the latest version) but setuptools only installed from ubuntu's precise deb23:44
clarkb#status notice Project affected by the xattr cffi dependency issues should be able to run tests and have them pass. xattr has been fixed and the new version is on our mirror.23:47
openstackstatusNOTICE: Project affected by the xattr cffi dependency issues should be able to run tests and have them pass. xattr has been fixed and the new version is on our mirror.23:47
*** ^demon has quit IRC23:47
openstackgerritJeremy Stanley proposed a change to openstack-infra/config: Pull toci into openstack-infra  https://review.openstack.org/3743223:47
*** pcrews has quit IRC23:48
fungirebased due to merge conflict ^23:48
openstackgerritMonty Taylor proposed a change to openstack-dev/pbr: Replace entry_points console_scripts  https://review.openstack.org/3800023:49
mordredlifeless: ^^ also, if you have snarky or evil commends on the above patch, I'd appreciate them23:49
*** dkehn is now known as dkehn_away23:49
lifelessmordred: while I have your attention23:52
mordredlifeless: uhoh23:52
lifeless04:26 < ttx> mordred: i don't know any of those guys. lifeless probably knows them though23:52
lifeless04:27 -!- vogxn [~Adium@49.205.181.238] has quit [Quit: Leaving.]23:52
lifeless04:28 < mordred> lifeless: ^^ if you could poke someone, a new release of that lib with that patch landed would23:52
lifeless                 be helpful23:52
mordredlifeless: yes please23:52
lifelessmordred: *which project*23:52
mordredlifeless: https://launchpad.net/django-openid-auth23:52
lifelessthank you23:52
lifelessI shall see23:52
mordredlifeless: if you know them - but I assume you know everyone :)23:53
lifelessmordred: review on the console scripts thing23:53
mordredlifeless: thank you23:54
mordredlifeless: good question. lemme check23:54
lifelessmordred: so, you need a release done ?23:54
lifeless[of d-o-a ?23:54
mordredlifeless: yes. it should get a usable thing for develop - that magic should be done by mucking with the lib install .pth stuff23:55
mordredlifeless: yes. the current one is broken23:55
*** nati_ueno has quit IRC23:55
mordredlifeless: https://code.launchpad.net/~mordred/django-openid-auth/zip-safe/+merge/17587523:56
mordredlifeless: setuptools can't autosense the use of data files in the tree because the calls to use them are inside of django, not in the current source tree23:56
mordredso being explicit is needed if you want to be able to install_requires on that package23:56
clarkbI have reenabled puppet in all of the places that I had turned it off (mirror slaves, logstash.o.o, and the elasticsearch nodes)23:58
*** retr0h_ is now known as retr0h23:59

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