Wednesday, 2015-02-11

openstackgerritAndrew Lazarev proposed openstack/python-saharaclient: Added --bypass-url support for keystone 3  https://review.openstack.org/15471600:02
openstackgerritMerged openstack/sahara: Config parameters beginning with "oozie." should be in job properties file  https://review.openstack.org/15421400:10
openstackgerritMerged openstack/sahara: Add efficient method for detecting installed packages  https://review.openstack.org/15453100:11
*** sgotliv__ has quit IRC00:18
*** egafford has joined #openstack-sahara00:26
*** amcrn has quit IRC02:01
*** fvollero has quit IRC02:09
*** Nikolay_St has quit IRC02:23
*** Nikolay_St has joined #openstack-sahara02:25
*** macjack has quit IRC02:29
*** macjack has joined #openstack-sahara02:40
*** hdd has quit IRC02:42
*** hdd has joined #openstack-sahara02:47
*** hdd has quit IRC03:13
*** hdd has joined #openstack-sahara03:26
*** hdd has quit IRC03:41
Nikolay_Stelmiko: tmckay: hi03:44
*** hdd has joined #openstack-sahara03:44
Nikolay_Stguys, just a quick notice about https://review.openstack.org/#/c/154037. I think we should first end with the spec review here: https://review.openstack.org/#/c/15444803:45
Nikolay_Stand make it clear which messages in Sahara should be on what logging level. I will introduce this in next patchset03:46
*** Nikolay_St has quit IRC04:06
*** ViswaV has quit IRC04:07
*** ViswaV has joined #openstack-sahara04:09
*** coolsvap_ is now known as coolsvap04:48
*** hdd has quit IRC05:21
*** Nikolay_St has joined #openstack-sahara05:36
*** sgotliv__ has joined #openstack-sahara05:39
*** ghenriks has quit IRC05:44
*** hdd has joined #openstack-sahara06:04
*** Longgeek has joined #openstack-sahara06:31
*** hdd has quit IRC06:51
openstackgerritNikolay Starodubtsev proposed openstack/sahara-specs: Sahara new style logging  https://review.openstack.org/15444806:54
openstackgerritArtem Osadchiy proposed openstack/sahara: Refactor MapR plugin for Sahara  https://review.openstack.org/14787207:13
*** sgotliv__ has quit IRC07:46
openstackgerritKen Chen proposed stackforge/sahara-ci-config: Add tag 5.3.0 for CDH image registry  https://review.openstack.org/15477707:49
openstackgerritKen Chen proposed stackforge/sahara-ci-config: Add tag 5.3.0 for CDH image registry  https://review.openstack.org/15477707:52
*** chlong has quit IRC08:00
*** witlessb has joined #openstack-sahara08:16
openstackgerritKen Chen proposed stackforge/sahara-ci-config: Add tag 5.3.0 for CDH image registry  https://review.openstack.org/15477708:19
openstackgerritlu huichun proposed openstack/sahara: Add impala shell solr package in the cdh plugin  https://review.openstack.org/15478508:53
*** IvanBerezovskiy has joined #openstack-sahara09:13
openstackgerritNikolay Starodubtsev proposed openstack/python-saharaclient: Add how to participate doc  https://review.openstack.org/12389809:23
*** tosky has joined #openstack-sahara09:28
openstackgerritVitaly Gridnev proposed openstack/sahara: Improve messages for validation  https://review.openstack.org/15448309:45
openstackgerritSergey Kolekonov proposed stackforge/sahara-ci-config: Use Ubuntu Trusty for Jenkins slaves  https://review.openstack.org/15480009:55
openstackgerritDenis Egorenko proposed stackforge/sahara-ci-config: Set to lower case 'image_os' variable  https://review.openstack.org/15480210:00
openstackgerritMerged stackforge/sahara-ci-config: Set to lower case 'image_os' variable  https://review.openstack.org/15480210:11
*** DenisEgorenko has joined #openstack-sahara10:14
*** sgotliv__ has joined #openstack-sahara10:14
openstackgerritDenis Egorenko proposed stackforge/sahara-ci-config: Specify branches for Vanilla 2.x jobs  https://review.openstack.org/15415010:28
*** skolekonov has joined #openstack-sahara10:37
openstackgerritDenis Egorenko proposed stackforge/sahara-ci-config: Specify branches for Vanilla 2.x jobs  https://review.openstack.org/15415010:38
openstackgerritVitaly Gridnev proposed openstack/sahara: Make status description field more useful  https://review.openstack.org/15401410:39
openstackgerritVitaly Gridnev proposed openstack/sahara: Make status description field more useful  https://review.openstack.org/15401410:41
openstackgerritMerged openstack/sahara-image-elements: Add solr-doc and search packages  https://review.openstack.org/15432410:45
openstackgerritMerged openstack/sahara-image-elements: Turn off iptables via 'disable-firewall' element  https://review.openstack.org/15318912:30
openstackgerritVitaly Gridnev proposed openstack/sahara: [Vanilla2] Open ports for hive  https://review.openstack.org/15484312:46
openstackgerritKen Chen proposed stackforge/sahara-ci-config: Add tag 5.3.0 for CDH image registry  https://review.openstack.org/15477712:54
openstackgerritVitaly Gridnev proposed openstack/sahara: Make status description field more useful  https://review.openstack.org/15401412:57
openstackgerritMerged stackforge/sahara-ci-config: Add tag 5.3.0 for CDH image registry  https://review.openstack.org/15477713:02
openstackgerritMerged stackforge/sahara-ci-config: Specify branches for Vanilla 2.x jobs  https://review.openstack.org/15415013:03
*** Nikolay_St has quit IRC13:24
openstackgerritAndrey Pavlov proposed openstack/sahara: Adding validation check for Spark plugin  https://review.openstack.org/15400113:50
tellesnobregatmckay, ping13:58
tmckaytellesnobgrega, hi14:00
openstackgerritVitaly Gridnev proposed openstack/sahara: Make status description field more useful  https://review.openstack.org/15401414:01
*** crobertsrh has joined #openstack-sahara14:02
openstackgerritAndrey Pavlov proposed openstack/sahara: Fixing log messages to avoid information duplication  https://review.openstack.org/14750414:05
tmckaytellesnobrega, oops, spelled your nick wrong14:09
tellesnobregatmckay, hi, i was testing storm here and i found that the machines cant communicate with each other, they are asking for the permission key14:10
tellesnobregashouldnt sahara enable this communication?14:10
openstackgerritMerged openstack/sahara: Refactor event-log code  https://review.openstack.org/15319314:11
tmckaytellesnobrega, hmm, interesting.  Yes, ssh keys should be set up so that you can ssh between them.  So, on a standard  ubuntu image you should be able to ssh around in the cluster as the ubuntu user.14:12
tellesnobregatmckay, that it what i was supposing it happened14:13
tmckayand, of course, you can get into the cluster with the keypair from outside14:13
tmckayall clusters work that way, afaik.  I don't *think* there is anything to do in the plugin to make that happen ...14:13
tellesnobregatmckay, i can get into the cluster from outside with the keypair14:13
tellesnobregabut the cluster itself is not communicating14:14
tellesnobregatmckay, there is no ssh keys authorized in the .ssh/authorized_keys14:15
tellesnobregatmckay, weird, right?14:15
tmckaytellesnobrega, ah, I was going to ask about that14:15
tmckayoooo14:15
tmckaytellesnobrega, grep (I use ack) for authorized_keys in Sahara14:15
tmckayapparently you might have to do some work to set that up14:16
tmckayvanilla1 and spark both at least touch it14:16
tellesnobregatmckay, i guess so. I need a way to make it configure it self without having to set it up by hand14:16
tmckaytellesnobrega, although service/engine.py touches it too14:16
tellesnobregafor the user it will be a pain14:16
tmckaymaybe something is going wrong in that code14:17
tmckaytellesnobrega, should definitely be possible, I've just never investigated that code because it's always worked14:17
tellesnobregatmckay, can be that storm wasnt using the engine.py (we had no edp) and it was not configuring it?14:17
tmckaywell, that's a different engine.py I think.  Taking a quick look now.14:19
tellesnobregatmckay, ok14:21
tmckaytellesnobrega, okay, direct engine is still the default provisioning engine. Is that what you used? (yes, unless you specifically tried to use heat)14:26
tellesnobregatmckay, yes, didnt try anything different from the default14:27
openstackgerritTelles Mota Vidal Nóbrega proposed openstack/sahara-image-elements: Changing Zookeeper folder name to not use version  https://review.openstack.org/15488914:28
tmckaytellesnobrega, okay, I would start then with a breakpoint (or extra log message) in _wait_until_accessible in service/engine.py.  According to that, an instance won't be accessible until authorized_keys is created14:28
tellesnobregatmckay, ok14:29
tmckaytellesnobrega, two possibilities come to mind -- the auth_keys was there, but something in provisioning overwrote it, or 2) it's in a different user from where you are looking for some reason14:29
tmckaytellenobrega, or 3) something recently changed and there is a bug so that the cluster is coming up "active" even when the nodes weren't set up correctly14:30
tellesnobregatmckay, i think it is actually a problem with storm image14:34
tellesnobregatmckay, the storm image has a storm user14:34
tmckaytellesnobrega, is that the user that the machines use to communicate?14:35
tmckaybase image is what, ubuntu?14:35
tellesnobregawhich should be responsible for deploying and managing the cluster14:35
tellesnobregayes14:36
tmckaytellesnobrega, so if there is an "ubuntu" user and a "storm" user, then the storm user probably needs extra auth_key set up in the plugin during provisioning.  Or ....14:36
tellesnobregatmckay, i have to check which user is configured by the plugin14:36
tmckaymaybe if you set the user to "storm" for the image when you register it, Sahara can provision with the storm user?  Just a wild guess, I don't know if that would work14:37
tmckaystorm user would have to have sudo14:37
tellesnobregayes14:38
tellesnobregai will double check that14:38
tmckayNikolay_St, thanks for comments, reading back ...14:39
tmckaytellesnobrega, interesting case14:39
tellesnobregatmckay, yeah14:40
*** sgotliv has joined #openstack-sahara14:45
*** sgotliv__ has quit IRC14:48
openstackgerritTelles Mota Vidal Nóbrega proposed openstack/sahara: Changing zookeeper to not use version number  https://review.openstack.org/15490214:52
*** Nikolay_St has joined #openstack-sahara14:55
*** Longgeek has quit IRC15:03
*** Longgeek has joined #openstack-sahara15:03
*** Longgeek has quit IRC15:04
*** Nikolay_St has quit IRC15:05
openstackgerritSergey Reshetnyak proposed openstack/sahara: New integration tests - base functional  https://review.openstack.org/14849115:22
openstackgerritSergey Reshetnyak proposed openstack/sahara: New integration tests - scaling  https://review.openstack.org/14893515:25
*** Networkn3rd has joined #openstack-sahara15:57
openstackgerritVitaly Gridnev proposed openstack/sahara: Add provision steps to Spark Plugin  https://review.openstack.org/15213215:59
*** vgridnev_ has joined #openstack-sahara15:59
vgridnev_Hi tmckay, i repaired problem with 'is' in my patch16:00
tmckayvgridnev_, thanks!  I think that came originally from a comment by elmiko, btw :)16:01
tmckayvgridnev_, I learned something about "is".  We should always stomp it out with integers from now on16:02
vgridnev_agree16:02
tmckayI could see how it would be easy to type "is 0" though since we are used to typing "is None" and sometimes the brain will equate 0 and None16:02
elmikoyea, that was a bad suggestion on my part =(16:03
elmikoi think "is" is only supposed to be used against None16:03
vgridnev_but  there are no problems with "is None", yes?16:04
elmikoright16:04
elmikothere shouldn't be a problem with "is 0" because numnbers are immutable, but None is a fixed value16:04
tmckayvgridnev_, right, because None is always the same object16:04
*** Longgeek has joined #openstack-sahara16:04
tmckayelmiko, is 0 is fine, but the danger is that some noob sees that and doesn't understand that you can't use it in general, only -5 to 25616:05
openstackgerritVitaly Gridnev proposed openstack/sahara: Add provisioning steps to Storm plugin  https://review.openstack.org/15214316:05
elmikotmckay: yes16:05
elmikois16:05
elmikoer, "is" can be dangerous if it isn't understood. it's an object comparison, not value.16:06
tmckayelmiko, I think this is what Bill Clinton was talking about16:06
elmikoha!16:07
elmikoi might have been a little too fancy when suggesting "if var is 0"16:08
openstackgerritSergey Reshetnyak proposed openstack/sahara: New integration tests - EDP  https://review.openstack.org/15004216:08
tmckayelmiko, it looks nice, I'll grant you16:08
tmckayI didn't know about the nuance til I read up on it16:08
elmikoyea, it's good to know about16:08
tmckaytested it out in the Py27 shell, sure enough it breaks16:08
*** Longgeek has quit IRC16:09
elmikohow did it break?16:10
elmikonvm, that's weird about comparing numbers beyond -5 and 255, i didn't know that16:11
tmckayyeah, completely unexpected16:13
tmckaypylint should warn about is and numeric literals, I wonder if it does16:13
elmikoagreed, i thought numbers were stored like string and immutable. guess not.16:13
elmikooh well, sorry vgridnev_ for the confusion16:14
elmikotmckay: that's a good idea16:14
tmckaythey could be, though.  I suppose it could even be implementation defined (unless the spec calls specifically for that kind of storage)16:14
vgridnev_elmiko, all right.16:16
vgridnev_Now we should be more careful with this16:16
elmikodefinitely16:17
*** IvanBerezovskiy has left #openstack-sahara16:27
*** coolsvap has quit IRC16:37
*** hdd has joined #openstack-sahara16:49
*** coolsvap has joined #openstack-sahara16:55
*** tosky has quit IRC16:56
*** coolsvap is now known as coolsvap_17:02
*** skolekonov has quit IRC17:02
*** jodah has quit IRC17:15
*** hdd has quit IRC17:16
*** hdd has joined #openstack-sahara17:18
*** jodah has joined #openstack-sahara17:20
*** tosky has joined #openstack-sahara17:41
*** coolsvap_ is now known as coolsvap17:47
*** sgotliv has quit IRC17:48
*** Longgeek has joined #openstack-sahara17:54
*** Longgeek has quit IRC17:58
*** Networkn3rd has quit IRC17:59
*** Networkn3rd has joined #openstack-sahara18:04
*** Nikolay_St has joined #openstack-sahara18:12
*** Nikolay_St has quit IRC18:19
*** Nikolay_St has joined #openstack-sahara18:20
*** Nikolay_St has quit IRC18:24
*** Nikolay_St has joined #openstack-sahara18:25
*** saurabhs has joined #openstack-sahara18:27
*** saurabhs has quit IRC18:27
*** saurabhs has joined #openstack-sahara18:32
tellesnobregaegafford, hey, thanks for the review18:35
tellesnobregaegafford, should the dependency be the patch from sahara depends on the sahara-image-elements or the other way around18:36
toskytellesnobrega: I don't know this specific case, so please wait for egafford, but I would say that the question is: which patch works without the other?18:38
tellesnobregasahara ones works only if the image-elements get in18:38
tellesnobregai guess that answers my question18:38
tellesnobregatosky, to add dependency i use review -d #patch-number right?18:39
tellesnobregatosky, or is it just a special word in the commit message?18:39
toskyuhm, not really an expert on that; git review -d download the other page18:40
toskyare the patches on different repositories18:40
tosky?18:40
tellesnobregayes, and this wont work here18:40
tellesnobregaexactly18:40
elmikotellesnobrega: to make a dependency, get the change you want to depend on, then rebase on top of it, then git-review18:41
toskyadd a WIP to the second one, or ask for a temporary -1; when the first one is merged, unlock the second18:41
toskyelmiko: different repositories18:41
elmikotosky: oh, woops18:41
elmikonevermind me then =)18:41
* elmiko backs into the shadows18:41
tellesnobregathanks18:42
openstackgerritTelles Mota Vidal Nóbrega proposed openstack/sahara: Changing zookeeper to not use version number  https://review.openstack.org/15490218:43
*** sgotliv has joined #openstack-sahara18:45
*** tellesnobrega has quit IRC18:50
*** coolsvap has quit IRC18:51
*** ViswaV has quit IRC19:01
*** tosky has quit IRC19:01
*** ViswaV has joined #openstack-sahara19:02
*** hdd has quit IRC19:05
*** Nikolay_St has quit IRC19:07
egaffordJuno Stable lives! https://review.openstack.org/#/c/153352/19:12
*** Nikolay_St has joined #openstack-sahara19:12
openstackgerritVitaly Gridnev proposed openstack/sahara: Add provision steps to Spark Plugin  https://review.openstack.org/15213219:12
openstackgerritVitaly Gridnev proposed openstack/sahara: Add provisioning steps to Storm plugin  https://review.openstack.org/15214319:15
*** tellesnobrega has joined #openstack-sahara19:16
*** sgotliv has quit IRC19:17
*** ViswaV has quit IRC19:21
*** ViswaV has joined #openstack-sahara19:26
*** macjack has quit IRC19:30
*** amcrn has joined #openstack-sahara19:42
*** Longgeek has joined #openstack-sahara19:42
*** Networkn3rd has quit IRC19:53
*** Nikolay_St has quit IRC19:58
*** Networkn3rd has joined #openstack-sahara20:00
*** Longgeek has quit IRC20:04
*** saurabhs has quit IRC20:11
*** Networkn3rd has quit IRC20:17
*** Networkn3rd has joined #openstack-sahara20:20
*** Networkn3rd has quit IRC20:21
*** Networkn3rd has joined #openstack-sahara20:21
*** ViswaV has quit IRC20:28
*** hdd has joined #openstack-sahara20:43
elmikohey folks, gonna need lots of help with this one, https://review.openstack.org/#/c/155052/20:57
*** tellesnobrega has quit IRC20:57
*** tellesnobrega has joined #openstack-sahara21:00
*** saurabhs has joined #openstack-sahara21:01
*** ViswaV has joined #openstack-sahara21:12
*** Networkn3rd has quit IRC21:19
*** Networkn3rd has joined #openstack-sahara21:26
*** Networkn3rd has quit IRC21:30
*** Longgeek has joined #openstack-sahara21:49
*** crobertsrh is now known as _crobertsrh21:57
*** hdd has quit IRC22:08
*** Longgeek has quit IRC22:11
*** egafford has quit IRC22:43
openstackgerritAndrew Lazarev proposed openstack/python-saharaclient: Removed cluster validation on client side  https://review.openstack.org/15509222:54
*** vgridnev_ has quit IRC23:01
*** saurabhs has quit IRC23:06
*** saurabhs has joined #openstack-sahara23:10
*** hdd has joined #openstack-sahara23:20
*** saurabhs has quit IRC23:20
*** saurabhs has joined #openstack-sahara23:30
*** chlong has joined #openstack-sahara23:37
openstackgerritMichael McCune proposed openstack/sahara-image-elements: Adding new update option to diskimage-create.sh  https://review.openstack.org/15109923:40
*** tmckay has quit IRC23:42
*** Longgeek has joined #openstack-sahara23:56

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