Monday, 2016-06-27

*** tmckay has joined #openstack-sahara01:00
*** tmckay has quit IRC01:47
*** houming has joined #openstack-sahara02:20
*** SergeyLukjanov has quit IRC03:23
*** SergeyLukjanov has joined #openstack-sahara03:30
*** nkrinner_afk is now known as nkrinner03:56
*** links has joined #openstack-sahara04:15
*** pgadiya has joined #openstack-sahara04:41
*** sverma has joined #openstack-sahara05:14
*** witlessb has joined #openstack-sahara05:47
*** Poornima has joined #openstack-sahara06:03
*** sgotliv__ has joined #openstack-sahara06:17
*** rcernin has joined #openstack-sahara06:27
*** pcaruana has joined #openstack-sahara06:41
*** tesseract- has joined #openstack-sahara06:59
*** jamielennox is now known as jamielennox|away07:01
*** anshul has joined #openstack-sahara07:21
*** anshul is now known as Guest5996307:22
*** dmk0202 has joined #openstack-sahara07:31
*** permalac has joined #openstack-sahara07:56
*** itisha has joined #openstack-sahara08:28
*** vgridnev_ has joined #openstack-sahara08:31
*** vgridnev_ has quit IRC08:52
*** esikachev has joined #openstack-sahara08:53
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Tests for Sahara CLI  https://review.openstack.org/33436209:10
*** jamielennox|away is now known as jamielennox09:26
*** _degorenko|afk is now known as degorenko09:33
*** abalutoiu has joined #openstack-sahara09:40
*** pgadiya has quit IRC09:43
*** tosky has joined #openstack-sahara09:47
*** Poornima has quit IRC10:07
openstackgerritVitaly Gridnev proposed openstack/sahara-image-elements: build all images for ambar in gate  https://review.openstack.org/33438810:11
openstackgerritVitaly Gridnev proposed openstack/sahara-image-elements: build all images for ambari in gate  https://review.openstack.org/33438810:13
openstackgerritMichael Ionkin proposed openstack/sahara: testcommit  https://review.openstack.org/33439610:14
*** Poornima has joined #openstack-sahara10:16
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Added tests with "node group" commands for Sahara cLI  https://review.openstack.org/33436210:18
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: [do not review] testcommit  https://review.openstack.org/32326510:32
*** Poornima has quit IRC10:33
openstackgerritMichael Ionkin proposed openstack/sahara: testcommit  https://review.openstack.org/33439610:34
*** Poornima has joined #openstack-sahara10:39
*** Poornima has quit IRC10:39
*** links has quit IRC10:43
*** houming has quit IRC10:50
*** houming has joined #openstack-sahara10:51
*** links has joined #openstack-sahara10:55
openstackgerritMerged openstack/sahara-specs: Allow creation of python topologies for Storm  https://review.openstack.org/33300310:59
*** esikachev has quit IRC11:16
*** Poornima has joined #openstack-sahara11:30
*** houming has quit IRC11:42
*** agireud has quit IRC11:45
*** esikachev has joined #openstack-sahara11:46
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: [do not review] freeze warlock  https://review.openstack.org/33443411:57
openstackgerritTang Chen proposed openstack/python-saharaclient: Use osc_lib instead of cliff  https://review.openstack.org/33443812:00
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818012:02
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: [do not review] freeze warlock  https://review.openstack.org/33443412:04
*** rodrigods has quit IRC12:06
*** rodrigods has joined #openstack-sahara12:06
openstackgerritTang Chen proposed openstack/python-saharaclient: Use osc_lib instead of cliff  https://review.openstack.org/33443812:07
vgridnevpgadia, around?12:13
vgridnevhm, seems no12:13
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818012:15
*** agireud has joined #openstack-sahara12:16
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818012:16
toskyuhm, what if we move sahara_cli_tests under sahara_tempest_plugin/tests/cli ? We could then share the configuration for the tests that could require it (like the plugin used)12:16
vgridnevtosky, what kind of configuration we want to share?12:25
toskyvgridnev: the plugin used; see my comment in https://review.openstack.org/33436212:25
vgridnevuh, fake plugin12:26
toskythe idea is to make the (future) tests working with any available plugins12:27
vgridnevI guess it will be available for cluster creation since newton12:27
toskymoreover, in fact those tests are tempest tests (using tempest APIs)12:27
toskyalso, related to this, we could import the tests in sahara (sahara/tests/tempest/scenario/data_processing), so that we could have only one tempest plugin12:28
toskynow the question is the usual: cherry-pick or proper history import if SergeyLukjanov is available?12:29
vgridnevtosky, I not sure we will find him, he is busy with internal tasks mostly12:30
toskyI can imagine12:30
vgridnevhow many commits do we have?12:30
tosky1912:31
vgridnevit's much if we just pushing that12:31
vgridnevtosky, can we just enable simple job (pep8 I guess) and cherry-pick changes?12:33
vgridnevor it's bad idea?12:34
toskywell, if there are no other infra-cores available...12:34
toskyunless I can get some of them12:35
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Added tests with "node group" commands for Sahara CLI  https://review.openstack.org/33436212:35
openstackgerritMikhail Lelyakin proposed openstack/sahara-dashboard: Resolve bug with long time for loading node group template page  https://review.openstack.org/33327312:37
toskyone thing at a time; let me see if I can find some infra-cores12:37
toskybut for the CLI tests, so, what do you think about this possible move?12:37
vgridnevtosky, how to run them (CLI tests)12:38
vgridnev?12:38
vgridnevas separate job, or with tempest tests?12:38
openstackgerritMikhail Lelyakin proposed openstack/sahara-dashboard: Resolve bug with long time for loading node group template page  https://review.openstack.org/33327312:38
toskyvgridnev: like any other tempest tests; if you want to run only them you need to specify a custom pattern12:39
toskyso we could keep a separate job too12:39
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818012:41
vgridnevbtw, If we have them in same place, we should be sure that we can run them together (no collision in names, for example)12:42
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: [do not review] testcommit  https://review.openstack.org/32326512:42
*** tmckay has joined #openstack-sahara12:43
toskyack, sure; I don't think it should be a problem, as afaik the name comes also from the path12:44
*** _crobertsrh is now known as crobertsrh12:57
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818013:02
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818013:04
openstackgerritVitaly Gridnev proposed openstack/sahara: plugins api impl  https://review.openstack.org/33337413:05
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818013:12
*** nkrinner is now known as nkrinner_afk13:13
openstackgerritVitaly Gridnev proposed openstack/sahara: plugins api impl  https://review.openstack.org/33337413:14
openstackgerritMerged openstack/sahara-ci-config: Added publisher script and job  https://review.openstack.org/32818013:17
*** egafford has joined #openstack-sahara13:32
*** sverma has quit IRC13:42
vgridnevegafford, you are welcome to review base patches of plugins api https://review.openstack.org/#/q/status:open+project:openstack/sahara+branch:master+topic:bp/plugin-management-api13:47
egaffordvgridnev: Cool; I'll look at those today (probably in about 2 hours.) Thanks!13:48
*** sverma has joined #openstack-sahara13:55
*** tosky has quit IRC13:56
openstackgerritMikhail Lelyakin proposed openstack/sahara-dashboard: Fix bug with detailed error on node groupe deletion  https://review.openstack.org/33392614:07
openstackgerritVitaly Gridnev proposed openstack/sahara: sleep before waiting requests  https://review.openstack.org/33417814:10
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Added tests with "node group" commands for Sahara CLI  https://review.openstack.org/33436214:13
*** vgridnev has quit IRC14:23
*** Poornima has quit IRC14:27
*** vgridnev has joined #openstack-sahara14:28
openstackgerritMerged openstack/sahara-image-elements: build all images for ambari in gate  https://review.openstack.org/33438814:47
openstackgerritMikhail Lelyakin proposed openstack/sahara: Code refactoring in oozie Workflow creators  https://review.openstack.org/33039814:48
*** sverma has quit IRC14:55
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Fix command for running publisher script  https://review.openstack.org/33451814:55
*** esikachev has quit IRC14:59
openstackgerritVitaly Gridnev proposed openstack/sahara: plugins api impl  https://review.openstack.org/33337415:01
*** esikachev has joined #openstack-sahara15:04
*** krotscheck is now known as krotscheck_dcm15:05
*** tellesnobrega is now known as tellesnobrega_af15:20
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Fix command for running publisher script  https://review.openstack.org/33451815:22
*** dmk0202 has quit IRC15:30
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: [do not review] testcommit  https://review.openstack.org/32326515:32
*** pcaruana has quit IRC15:38
*** tesseract- has quit IRC15:40
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Fix typpo in glance command  https://review.openstack.org/33454015:46
*** links has quit IRC15:52
*** tosky has joined #openstack-sahara15:52
openstackgerritMikhail Lelyakin proposed openstack/sahara-dashboard: Fix service configuration view for node groups  https://review.openstack.org/33454515:56
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Added tests with "node group" commands for Sahara CLI  https://review.openstack.org/33436215:57
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Added tests with "node group" commands for Sahara CLI  https://review.openstack.org/33436216:07
openstackgerritMerged openstack/sahara-ci-config: Fix command for running publisher script  https://review.openstack.org/33451816:20
openstackgerritMerged openstack/sahara-ci-config: Fix typpo in glance command  https://review.openstack.org/33454016:38
*** Guest59963 has quit IRC16:46
*** degorenko is now known as _degorenko|afk16:54
*** abalutoiu has quit IRC17:01
openstackgerritAlina Nesterova proposed openstack/sahara-tests: Added cli-tests for cluster and cluster template commands  https://review.openstack.org/33456717:01
openstackgerritMichael Ionkin proposed openstack/sahara: Fix glanceclient.v2.images  https://review.openstack.org/33456917:06
*** _elmiko is now known as elmiko17:07
elmikovgridnev: hey, heads up, i will miss the meeting this week. i am at a conference17:07
*** ukaynar has joined #openstack-sahara17:10
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Removing of Mapr 5.5.0 for scripts  https://review.openstack.org/33457417:13
*** itisha has quit IRC17:20
*** akuznetsov has joined #openstack-sahara17:28
*** esikachev has quit IRC17:31
*** abalutoiu has joined #openstack-sahara17:37
egaffordvgridnev: Wow, sahara.plugins.labels is super dense. Haven't had a really challenging review in a while. :)17:52
*** egafford has quit IRC18:01
*** egafford has joined #openstack-sahara18:07
*** elmiko is now known as _elmiko18:26
*** akuznetsov has quit IRC18:33
*** egafford has quit IRC18:34
*** rcernin has quit IRC18:34
*** zemuvier has quit IRC18:36
*** tellesnobrega_af is now known as tellesnobrega18:44
*** esikachev has joined #openstack-sahara18:48
*** egafford has joined #openstack-sahara18:53
*** egafford has quit IRC18:54
*** egafford has joined #openstack-sahara18:56
*** egafford has quit IRC18:56
*** egafford has joined #openstack-sahara18:56
*** rcernin has joined #openstack-sahara19:03
*** tmckay has quit IRC19:08
*** ukaynar has quit IRC19:14
*** tellesnobrega is now known as tellesnobrega_af19:27
vgridnevelmiko, ack19:40
vgridnevegafford, maybe. the only though that I have, that probably labels.json is not good idea19:41
egaffordvgridnev: Yeah, I'm not a huge fan of labels.json.19:42
vgridnevabout your note at https://review.openstack.org/#/c/332043/1 , None will returned if there is no matching version19:42
egaffordIt seems to me that most of that should be pushed into the plugins (and just kept as flat constants without label substitution.)19:42
egaffordRight, that's a really really minor note.19:42
egaffordI understand the flow (and that None will be returned.) The method just doesn't do anything with 'res' unless version exists in the plugin, so there's not a reason to call as_resource unless the version exists.19:43
egaffordIt's incredibly trivial, and not worth worrying about unless someone has negative feedback.19:43
vgridnevegafford, I did not got you at all19:48
vgridnevhow you will write this method?19:49
egaffordvgridnev: I looked at it again and understood what you're saying. Removed that objection; +2.19:49
egaffordStill have some concerns about some complexity that doesn't seem needed in the api patch; the others lgtm.19:50
*** esikachev has quit IRC19:53
*** tmckay has joined #openstack-sahara19:56
*** esikachev has joined #openstack-sahara19:56
vgridnevegafford, could you please take a look on https://etherpad.openstack.org/p/sahara-review-priorities list, I still think it's good instrument to highlight important reviews for us20:10
egaffordvgridnev: ack20:11
vgridnevI also thinking about topic for changes that already got +2 from core. For example: sahara-final-core-vote20:11
vgridnevso, when core just putted +2 for change it might be a good to change topic to highlight that there are few good changes20:12
egaffordvgridnev: Makes sense to me; I can start posting new changes to that.20:13
*** tellesnobrega_af is now known as tellesnobrega20:14
*** esikachev has quit IRC20:24
*** tmckay has quit IRC20:38
openstackgerritVitaly Gridnev proposed openstack/sahara: plugins api impl  https://review.openstack.org/33337420:44
vgridnevegafford, I thinking that now it should be cleaner20:45
vgridnev^^20:45
egaffordvgridnev: I'll take a look; thanks!20:46
egaffordvgridnev: Yup; this is way cleaner. :D20:47
egaffordvgridnev: So we need to implement plugin.get_labels() for each plugin in a future patch (or patches), right?20:49
vgridnevyep, but it's required with default value20:50
egaffordYeah this is much better-encapsulated and very clean and readable. Thanks!20:51
*** catintheroof has joined #openstack-sahara20:55
openstackgerritMerged openstack/sahara: Fix the ca certificate handling in the client sessions  https://review.openstack.org/33063520:55
*** tmckay has joined #openstack-sahara20:58
*** sgotliv__ has quit IRC21:01
openstackgerritMerged openstack/sahara-specs: Adding ability to export and import templates via CLI  https://review.openstack.org/27354721:04
openstackgerritVitaly Gridnev proposed openstack/sahara-image-elements: remove hdp from elements  https://review.openstack.org/32608021:13
*** crobertsrh is now known as _crobertsrh21:14
vgridnevegafford, you are welcome https://bugs.launchpad.net/sahara/+bug/159671521:33
openstackLaunchpad bug 1596715 in Sahara "[DOC][DIB] readme is too old and should updated properly" [Medium,Confirmed] - Assigned to Ethan Gafford (egafford)21:33
egaffordvgridnev: :D21:33
* egafford is actually laughing out loud a bit.21:33
*** permalac_ has joined #openstack-sahara21:35
*** permalac has quit IRC21:36
*** witlessb has quit IRC21:54
*** catintheroof has quit IRC21:55
openstackgerritEthan Gafford proposed openstack/sahara-image-elements: Fixing diskimage-create.sh README.rst  https://review.openstack.org/33468522:07
openstackgerritMerged openstack/python-saharaclient: print statment has become a function in py3  https://review.openstack.org/33207022:08
*** egafford has quit IRC22:13
*** jamielennox is now known as jamielennox|away22:21
*** abalutoiu_ has joined #openstack-sahara22:42
*** abalutoiu has quit IRC22:43
*** abalutoiu_ has quit IRC22:48
*** jamielennox|away is now known as jamielennox23:43
*** tosky has quit IRC23:51

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