Wednesday, 2014-03-19

« Tuesday, 2014-03-18 Index Thursday, 2014-03-20 »
*** ErikB1 has quit IRC00:00
*** ErikB1 has joined #openstack-sahara00:18
*** ErikB1 has quit IRC00:20
*** coolsvap has quit IRC02:39
*** ErikB1 has joined #openstack-sahara02:50
*** witlessb has joined #openstack-sahara02:56
*** bradd1 has quit IRC03:00
aignatovJust woke up and realised that voting is merged! Congrats to all!  :)03:16
*** ErikB1 has quit IRC03:35
*** IlyaE has joined #openstack-sahara04:42
openstackgerritA change was merged to openstack/sahara: Rename 'self.savanna' to 'self.sahara' in integration tests  https://review.openstack.org/8137205:47
*** IlyaE has quit IRC06:04
*** IlyaE has joined #openstack-sahara06:05
*** akuznetsov has quit IRC06:08
openstackgerritSergey Reshetnyak proposed a change to openstack/sahara: Prepare integration tests for use for hadoop 2  https://review.openstack.org/7890006:19
*** dmitryme has joined #openstack-sahara06:21
openstackgerritSergey Reshetnyak proposed a change to openstack/sahara: Add integration tests for vanilla 2 plugin  https://review.openstack.org/7708906:25
*** akuznetsov has joined #openstack-sahara06:27
*** Ch00k has joined #openstack-sahara06:49
*** dmitryme_ has joined #openstack-sahara06:52
*** dmitryme has quit IRC06:54
*** dmitryme_ is now known as dmitryme06:54
*** IlyaE has quit IRC07:11
openstackgerritSergey Reshetnyak proposed a change to openstack/sahara: Add integration tests for vanilla 2 plugin  https://review.openstack.org/7708907:15
aignatovtmckay: here?07:43
*** ylobankov has joined #openstack-sahara08:09
*** ylobankov has quit IRC08:11
*** ylobankov has joined #openstack-sahara08:11
*** Ch00k has quit IRC08:12
openstackgerritFengqian Gao proposed a change to openstack/sahara: Keep python 3.X compatibility for xrange  https://review.openstack.org/7984908:17
*** Ch00k has joined #openstack-sahara08:48
openstackgerritYaroslav Lobankov proposed a change to openstack/sahara: Integration test for a transient cluster was added  https://review.openstack.org/7622708:51
*** witlessb has joined #openstack-sahara08:57
*** Ch00k has quit IRC09:52
*** Ch00k has joined #openstack-sahara09:53
*** dmitryme has quit IRC10:07
*** Ch00k_ has joined #openstack-sahara10:37
*** Ch00k has quit IRC10:38
*** themistymay has joined #openstack-sahara10:44
*** Ch00k_ has quit IRC10:45
*** Ch00k has joined #openstack-sahara10:45
*** dmitryme has joined #openstack-sahara10:46
*** tosky has joined #openstack-sahara10:55
*** themistymay has joined #openstack-sahara11:42
tmckayaignatov, here now11:49
*** _crobertsrh has quit IRC11:51
aignatovtmckay: ok, what would be our next steps in the renaming11:57
*** ErikB has joined #openstack-sahara11:57
tmckayaignatov, well, right now I'm setting up my laptop to run a tiny integration test from sahara (3 nodes) so that I can verify changes quickly.  My thought was to grep integration tests until there are no more savannas, run tests to verify, and then do the same in the main sahara source.  Lots of little things left, like config parameter names, filenames, etc12:00
*** mattf is now known as _mattf12:02
tmckayaignatov, maybe not so scientific :)12:02
aignatovtmckay: I see, so I can take all entry points in setup.cfg, and also strings in text12:04
tmckayaignatov, okay.  I'm working from sahara/tests down12:05
openstackgerritTomasz 'Zen' Napierala proposed a change to stackforge/puppet-sahara: Fixes installing development builds  https://review.openstack.org/8151212:06
openstackgerritVadim Rovachev proposed a change to openstack/sahara-dashboard: Add sleep in await state cluster method  https://review.openstack.org/8123112:14
openstackgerritTomasz 'Zen' Napierala proposed a change to stackforge/puppet-sahara: Fixes installing development builds  https://review.openstack.org/8151212:23
aignatovoh, the patch https://review.openstack.org/#/c/81319/ is still not merged, so we can't merge all Chad's dashboard changes12:29
aignatovboring12:29
SergeyLukjanovaignatov, yup12:30
tmckayaignatov, I would like to change PROP_TAG in utils/openstack/images.py as part of an integration test CR to move the tags to "_sahara_tag..."12:31
tmckayhmm, and PROP_DESCR and PROP_USERNAME too12:31
openstackgerritTomasz 'Zen' Napierala proposed a change to stackforge/puppet-sahara: Fixes installing development builds  https://review.openstack.org/8151212:31
tmckayjust so that we don't both change it :)12:32
tmckaymaybe I'll make a "change tags" CR12:32
tmckay"smart watch?"  Beam me up, Scotty.12:33
tmckayI just saw a bunch of grandfather clocks from the 18th century.  Now, there was a timepiece.12:33
* tmckay is listening to the radio12:34
aignatovtmckay: heh, ok, I will not change these tags12:35
tmckaythanks12:36
openstackgerritTomasz 'Zen' Napierala proposed a change to stackforge/puppet-sahara: Fixes installing development builds  https://review.openstack.org/8151212:46
*** tmckay is now known as tmckay_kid_taxi13:10
*** tmckay_kid_taxi has quit IRC13:11
*** crobertsrh has joined #openstack-sahara13:19
*** rhodgin has joined #openstack-sahara13:21
openstackgerritSergey Reshetnyak proposed a change to openstack/sahara: Add Job History Server process to vanilla 2 plugin  https://review.openstack.org/8053513:31
*** Ch00k has quit IRC13:45
*** Ch00k has joined #openstack-sahara13:45
*** dmitryme has quit IRC14:08
*** tmckay has joined #openstack-sahara14:18
*** dmitryme has joined #openstack-sahara14:25
aignatovtmckay: I'm renaming files with *savanna* pattern in it.14:46
aignatovhope I'm not duplication your job14:46
tmckayaignatov, great14:46
aignatovhow is you integration instllation going? :)14:47
tmckayaignatov, no duplication yet.  I have a test running right now :) Just tox -e integration with a local itest.conf and modified cluster templates.  Testing that "sahara_" image tags work.14:48
tmckayIf I need to rebase on your change, that's no problem14:48
tmckayaignatov, hmm, is ci using the newest client yet?14:54
tmckaylooks like not, because test_requirements still has 0.5.0.  Because "savanna_url" needs to switch to "sahara_url" when test_requirements changes14:56
*** bradd1 has joined #openstack-sahara14:56
tmckayI can try that locally, too14:57
SergeyLukjanovfolks, I'm canceling tomorrow meeting15:00
crobertsrhok :)15:01
SergeyLukjanovdue to the Mirantis team unavailability, mattf said that he'll be unavailable too15:01
SergeyLukjanovtmckay, eanxgeek15:01
tmckaySergeyLukjanov, I see, thanks15:01
*** IlyaE has joined #openstack-sahara15:01
*** dmitryme has quit IRC15:22
openstackgerritTrevor McKay proposed a change to openstack/sahara: Move integration tests to python-saharaclient 0.6.0  https://review.openstack.org/8155615:30
tmckayaignatov, ^^15:31
*** themistymay has quit IRC15:31
aignatovtmckay: great15:33
aignatovwaiting for the saharaclient released15:33
aignatovoh, how this is looooong...15:33
tmckayaignatov, has the client not been released?  When I changed my test requirements locally, it pulled down 0.6.015:35
aignatovah, yes, I was wrong, client is being install successfully but15:37
aignatovopenstack Jenkins has global requirements check15:37
aignatovand the 0.6.0 version is not there15:37
aignatovI mean requirement check is failing with new saharaclient15:38
tmckayah15:39
aignatovas result Jenkins will put -1 and not merge to each patch with saharaclient>=0.6.015:39
*** dmitryme has joined #openstack-sahara15:39
tmckayaignatov, okay, can we change the global requirements?15:39
tmckayCan't remember if I've done that before15:40
aignatovSergeyLukjanov did already it, bit his match is under gate hell15:40
tmckaylol15:40
tmckayokay15:40
aignatovit's being merged for about 50 hours already15:40
tmckaywow15:40
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change "_savanna_" image properties to "_sahara_"  https://review.openstack.org/8156615:49
*** ErikB has quit IRC15:49
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change "_savanna_" image properties to "_sahara_"  https://review.openstack.org/8156615:50
*** coolsvap has joined #openstack-sahara15:54
openstackgerritAlexander Ignatov proposed a change to openstack/sahara: Reaning files with savanna words in it's names  https://review.openstack.org/8156916:02
aignatovtmckay, ^^16:02
tmckayaignatov, -1, your title has a typo :)16:03
aignatovdoh16:03
* tmckay applies for newspaper editor job16:03
*** Ch00k has quit IRC16:04
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change lock string to 'sahara-' for migration tests  https://review.openstack.org/8157116:04
openstackgerritAlexander Ignatov proposed a change to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156916:04
aignatovhehe :)16:04
aignatovbtw, Trevor, we have some validation bugs related to EDP, I think we should fix it during icehouse16:05
aignatovthey are pretty simple, just not to forget16:06
tmckayaignatov, agreed.  Hopefully we can get the renaming done soon16:06
*** ErikB has joined #openstack-sahara16:12
openstackgerritAlexander Ignatov proposed a change to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156916:15
dmitrymetmckay: regarding https://review.openstack.org/#/c/81566/216:36
tmckaydmitryme, hi16:36
dmitrymewe need to change tags in our CI lab in order for that patchset to pass integration tests16:36
tmckaydmitryme, okay16:36
dmitrymewe will ask Sergey Kolekonov, who takes care of that stuff, to do that tomorrow16:37
tmckaydmitryme, okay, thanks, should be fine.  +1 for small isolated changes16:38
*** akuznetsov has quit IRC16:58
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change remaining 'savanna' references in sahara/tests  https://review.openstack.org/8158217:12
tmckayaignatov, dimitryme, okay, it's getting tough to break changes down at this point.  I put all the remaining test changes here, let me know if you would like something broken out ^^17:12
tmckayaignatov, we might be bumping into each other, fixing the tests meant changing some strings in non-test files17:14
*** witlessb has quit IRC17:14
*** witlessb has joined #openstack-sahara17:15
tmckaybut, ack -i savanna under sahara/tests shows only client and tag issues left, covered in the other 2 CRs17:15
*** tosky_ has joined #openstack-sahara17:16
tmckayoops, did that on master17:18
*** tosky has quit IRC17:19
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change remaining 'savanna' references in sahara/tests  https://review.openstack.org/8158217:20
*** dmitryme has quit IRC17:21
*** IlyaE has quit IRC17:23
*** Ch00k has joined #openstack-sahara17:26
aignatovtmckay: yes, that might happen and the winner will be the man which patch is merged firstly :)17:28
tmckayheh17:29
aignatovwinner in bumping17:29
tmckayI was just going to add the alembic README to your file rename patch.  It references savanna-db-manage17:29
tmckayJust text, no functional change17:29
tmckaysince that is where setup.cfg is modified17:30
aignatovok, I think rebase will resolve all conflicts17:30
tmckaystill lots of little odds and ends left over, who would have imagined?17:30
aignatovyes, but before that there were opinions that renaming could be just 's/savanna/sahara'17:34
aignatovdmitryme, here?17:36
tmckayaignatov, well, it could have I suppose.  Everything would have broken all at once.  I feel better about having things work by stages17:37
aignatovdmitryme, integration tests in my latest patch failed with issue around  'savanna_subprocess' renamed to the new name17:38
tmckaymy favorite command ack -i savanna --ignore-dir=doc17:38
aignatov:)17:38
aignatovomg, I found a code where _savanna_subrocess could be _savanna-subprocess17:41
aignatov'_' and '-', and I renamed only with '_'17:41
tmckayoops17:42
tmckayI think lawyers should have to do this, maybe they would think twice about lawsuits over such things17:42
openstackgerritAlexander Ignatov proposed a change to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156917:43
*** akuznetsov has joined #openstack-sahara17:44
openstackgerritTrevor McKay proposed a change to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156917:50
tmckaydarn, missed one17:52
*** crobertsrh has quit IRC17:52
openstackgerritTrevor McKay proposed a change to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156917:54
aignatovtmckay: agree about lawyers :)17:54
tmckaylol17:54
*** IlyaE has joined #openstack-sahara17:55
*** Ch00k has quit IRC17:56
*** crobertsrh has joined #openstack-sahara17:58
aignatovtmckay: lol, in a minute I think I'm crazy or openstackgerrit got crazy looking at your name on path set above18:01
tmckayaignatov, hehe, no I just stole your glory18:02
tmckaymaybe there is a way to do the commit as someone else...18:03
tmckayaignatov, hmm, I made a change to run_tests.sh for coverage which got rid of some "savanna" errors, but it still fails to load plugins... not sure why18:04
*** Ch00k has joined #openstack-sahara18:10
aignatovtmckay: let me check this18:12
tmckayok18:12
tmckayaignatov, here is what I changed18:13
tmckayhttp://www.fpaste.org/86821/52813139/18:13
tmckaybrb18:25
*** akuznetsov has quit IRC18:45
*** ErikB has quit IRC18:53
*** ErikB1 has joined #openstack-sahara18:55
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change 'savanna' references in tools  https://review.openstack.org/8160019:06
aignatovJenkins put me -1 :(19:17
tmckayaignatov, hmm, "sahara.readthedocs" doesn't seem to be there yet.  Is that on the radar?19:23
aignatovwhere did you find it?19:24
aignatovin the openstack logs?19:25
tmckayI modified *.rst in sahara/, then checked with rst2html19:25
tmckayeverthing works but the readthedocs links19:26
tmckayI'll make the CR and add a comment19:27
aignatovah, ok19:27
aignatovI just thought that you are looking at the failed tempest results in my patch set and found something wrong with docs :)19:28
tmckayheh, no, separate topic.  I can take a look at your tempest logs, too19:28
tmckaytempest broke on me the other day with client stuff19:29
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change savanna references in top level docs  https://review.openstack.org/8160619:29
*** tosky_ has quit IRC19:30
aignatovhehe, probably I found a reason19:33
aignatovhttps://github.com/openstack-dev/devstack/search?q=savanna&ref=cmdform19:33
tmckayaignatov, well, if that is not *the* reason, it is still *a* reason :)19:34
aignatovexactly :)19:34
aignatovI think we can't just remove etc/savanna completely19:34
openstackgerritAlexander Ignatov proposed a change to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156919:43
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change 'savanna' references in tools  https://review.openstack.org/8160019:45
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change 'savanna' references in tools  https://review.openstack.org/8160020:01
*** dmitryme has joined #openstack-sahara20:03
openstackgerritA change was merged to openstack/sahara: Change remaining 'savanna' references in sahara/tests  https://review.openstack.org/8158220:09
aignatovoops, am I right here? https://review.openstack.org/#/c/80987/2/requirements.txt20:10
tmckayaignatov, hmm, yes, I think it should be python-saharaclient 0.6.020:11
aignatovcrobertsrh: we need to update your sahara-dashbord patch20:13
crobertsrhAh, yes.  I was guessing when I did the patch.20:13
openstackgerritChad Roberts proposed a change to openstack/sahara-dashboard: Adapting to use the python-saharaclient library  https://review.openstack.org/8098720:16
openstackgerritChad Roberts proposed a change to openstack/sahara-dashboard: Changing configuration from SAVANNA to SAHARA  https://review.openstack.org/8102620:22
openstackgerritChad Roberts proposed a change to openstack/sahara-dashboard: Changing savanna-db to internal-db  https://review.openstack.org/8102920:22
openstackgerritChad Roberts proposed a change to openstack/sahara-dashboard: Remainder of renaming changes from savanna to sahara  https://review.openstack.org/8103820:22
crobertsrhOk, I think I've managed to rebase the whole chain after the tweak to requirements.txt20:23
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change remaining savanna namespaces in setup.cfg  https://review.openstack.org/8161720:26
tmckayaignatov, no reason not to change openstack-common.conf at this point, is there?20:28
tmckayMaybe that should be added to your file renaming CR20:28
tmckaybase=savanna20:29
aignatovhehe, I don't know what this base parameter mean :)20:30
tmckayIt's the line from openstack-common.conf.  I think it means where to put the openstack common modules20:30
dmitrymewhen you use update.py script from oslo-incubator, it places openstack.common package inside 'base' package20:30
dmitrymei.e. before renaming we had savanna.openstack.common...20:31
tmckayif we've changed setup.cfg to name the project sahara, it should change I think20:31
aignatovok, so now it should be shara20:31
aignatov*sahara.openstack.common20:31
tmckayright.  Do you want to add to your CR with setup.cfg20:31
dmitrymeaignatov: right20:31
dmitrymewe do need to change it at some point, though it can be done later20:32
aignatovtmckay: I have a patch with cleaning setup.cfg from savanna words20:32
tmckayyes, that's the one20:32
aignatovi think I'll add this there too20:32
aignatovdmitryme: tmckay are you ok if'll do that way?20:33
tmckayoh yes, sure.  I'm just looking through my ack list :)20:33
tmckayI have some dep changes on that one20:34
aignatovaaaa20:34
aignatovtmckay: I see20:34
tmckayI think it's a fine place20:34
aignatovyou were the first20:34
tmckaynot sure what to change next20:34
dmitrymeaignatov: sure20:34
aignatovtmckay: i didn't see your https://review.openstack.org/#/c/81617/20:34
aignatovand told about my the same patch :)20:35
tmckayoh, lol20:35
aignatovtmckay: so you can add openstack-common.conf renaming there20:35
aignatovalso as I remember20:36
aignatovwe decided to remove the....20:36
aignatovsahara = sahara.service.direct_engine:DirectEngine20:36
aignatovonly direct or heat20:36
aignatovnot sure when we decided20:36
tmckayaignatov, okay, great, I'll do it in a moment20:37
aignatovthx20:37
tmckaydimitryme, do you know about openstack/common/gettextutils.py?20:38
tmckayit's in common, but it has savanna references in it.... is it autogenerated?20:38
tmckayis it local?20:38
tmckayhmmm, I think it's local.  it's not in openstack-common.conf20:39
tmckaySo, we put it in common, but it does not come from there (openstack).  A little bit confusing.20:39
openstackgerritAndrew Lazarev proposed a change to openstack/sahara: Renamed 'idh' integration tests to 'idh2'  https://review.openstack.org/8040520:40
aignatovtmckay: see comments in this patch https://review.openstack.org/#/c/79460/ :)20:41
tmckayah20:42
aignatovas I understand it is not autogenerated :)20:42
tmckayyes, I think someone typed it ;-)20:42
*** dmitryme has quit IRC20:43
aignatovwell well, sharaclient is about to merge20:44
aignatovhttp://status.openstack.org/zuul/20:44
tmckaywoohoo20:44
*** dmitryme has joined #openstack-sahara20:45
aignatovdmitryme: you have slow net?20:45
dmitrymeaignatov: no, I accidentally broke my connection20:46
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change remaining savanna namespaces in setup.cfg  https://review.openstack.org/8161720:49
tmckayoops20:49
openstackgerritTrevor McKay proposed a change to openstack/sahara: Change remaining savanna namespaces in setup.cfg  https://review.openstack.org/8161720:50
*** coolsvap has quit IRC20:51
*** ErikB1 has quit IRC20:53
aignatovtmckay: well, it seems your renaming is better than mine :)20:54
*** ErikB1 has joined #openstack-sahara20:54
tmckayaignatov, it's like someone blew up a bag of marbles and said "Okay, pick them all up!"  I just grabbed a different handful of marbles20:56
tmckayor, I'm afraid we'll end up with spare parts, like when you take apart something big and have pieces left over20:57
tmckay"Hmm, it works without that.  I wonder what it's for"20:57
*** crobertsrh is now known as _crobertsrh20:59
aignatovtmckay: I'm just stopped any coding :)21:02
tmckayack21:02
tmckayI'm still trying to figure out openstack common, I think some things *have* to be autogenerated21:03
openstackgerritRobert Nettleton proposed a change to openstack/sahara-image-elements: Removes unnecessary/incorectly-named packages from HDP install script  https://review.openstack.org/8163021:11
aignatovtmckay: we add tags to savanna-ci images, and I retriggegred it in your patch21:15
tmckayoh, great21:15
*** IlyaE has quit IRC21:17
*** IlyaE has joined #openstack-sahara21:20
aignatovtmckay: did you see that? https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator21:25
tmckayaignatov, yes.  looks like the version changed since I looked21:28
aignatovseems to be openstack-common.conf is just self-described config which helps to sync oslo code easily21:29
*** dmitryme has quit IRC21:29
aignatovas I understood it correctly :)21:29
tmckayaignatov, I think so.  I just want to be sure what is custom in that dir and what's not.  there are some savanna references that update.sh makes, I think21:29
tmckayfor instance21:30
tmckaydb/sqlalchemy/session.py21:30
tmckay293:               default='savanna.sqlite',21:30
tmckay841:            index = filename.rfind('savanna')21:30
tmckayI want to make sure I know where those 'savanna' strings came from :)21:30
tmckaymaybe I can try it locally and see what it does...21:34
aignatovtmckay: It came from here21:36
aignatovhttps://review.openstack.org/#/c/29936/21:36
tmckaygit blame?  I should have thought of that21:38
aignatovI think this default can easily changed to sahara.db21:38
aignatovsahara.sqlite21:38
tmckayYes, it looks like newer oslo code breaks out the options to a separate package21:38
tmckaywell, file21:38
tmckayaignatov, do you think I should just commit the results from update?21:39
aignatovyes21:39
tmckaybecause it gets rid of all the savanna references under common :)21:40
tmckayokay21:40
openstackgerritTrevor McKay proposed a change to openstack/sahara: Update from openstack-common from oslo  https://review.openstack.org/8164221:42
tmckayhmmm, it even changed gettextutils which isn't even on the modules list21:43
tmckaymaybe it's smart21:43
tmckayaignatov, okay, that's enough for me.  (You must be tired :) )  We'll see what's broken and do more tomorrow21:44
*** witlessb has quit IRC21:48
*** tmckay has quit IRC21:48
aignatovtmckay: yes, will see more tommorow21:48
*** Ch00k has quit IRC22:20
openstackgerritA change was merged to openstack/sahara: Renaming files with savanna words in its names  https://review.openstack.org/8156922:46
*** bradd1 has quit IRC23:10
*** rhodgin has quit IRC23:12
*** IlyaE has quit IRC23:28
*** IlyaE has joined #openstack-sahara23:58
« Tuesday, 2014-03-18 Index Thursday, 2014-03-20 »

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