Thursday, 2015-05-07

« Wednesday, 2015-05-06 Index Friday, 2015-05-08 »
*** abitha has quit IRC00:20
*** yingjun has joined #openstack-rally00:27
*** dmorita has joined #openstack-rally00:28
*** yingjun has quit IRC02:26
*** yingjun has joined #openstack-rally02:30
*** softCloud has quit IRC02:47
*** panbalag has quit IRC02:54
*** yingjun has quit IRC03:50
*** pradeep has joined #openstack-rally03:56
*** yfried has joined #openstack-rally04:07
*** abitha has joined #openstack-rally04:18
*** abitha has quit IRC04:19
*** tfreger has joined #openstack-rally04:28
*** kiran-r has joined #openstack-rally04:51
*** yfried has quit IRC04:54
*** yingjun has joined #openstack-rally04:59
*** coolsvap|afk is now known as coolsvap05:03
*** yingjun has quit IRC05:04
*** yingjun has joined #openstack-rally05:05
*** yingjun has quit IRC05:13
*** yingjun has joined #openstack-rally05:13
*** yingjun has quit IRC05:18
*** nkhare has joined #openstack-rally05:20
*** rdas has joined #openstack-rally05:50
*** neeti has joined #openstack-rally05:53
*** yingjun has joined #openstack-rally05:57
*** agarciam has joined #openstack-rally05:58
*** neeti has quit IRC06:03
*** neeti has joined #openstack-rally06:05
*** yingjun has quit IRC06:13
*** anshul has joined #openstack-rally06:17
*** yingjun has joined #openstack-rally06:19
*** amalinow has joined #openstack-rally06:20
*** kairat_kushaev has quit IRC06:23
*** yfried has joined #openstack-rally06:25
*** kairat_kushaev has joined #openstack-rally06:26
*** exploreshaifali has joined #openstack-rally06:29
*** exploreshaifali has quit IRC06:40
*** exploreshaifali has joined #openstack-rally06:40
*** yfried is now known as yfried|afk07:02
*** yfried|afk is now known as yfried07:03
*** exploreshaifali has quit IRC07:18
openstackgerritMikhail Dubov proposed openstack/rally: Release 0.0.4  https://review.openstack.org/18049807:18
*** frobware_ has quit IRC07:19
openstackgerritMikhail Dubov proposed openstack/rally: Release 0.0.4  https://review.openstack.org/18049807:20
openstackgerritMikhail Dubov proposed openstack/rally: Release 0.0.4  https://review.openstack.org/18049807:21
*** jose_lausuch has joined #openstack-rally07:22
fricklerstpierry: that will be me, added some comments to the review07:35
*** yingjun has quit IRC07:47
*** yingjun has joined #openstack-rally07:54
*** dmorita has quit IRC07:59
*** amaretskiy has joined #openstack-rally08:02
*** karimb has joined #openstack-rally08:08
openstackgerritYair Fried proposed openstack/rally: Split SLA to base and plugins  https://review.openstack.org/18050908:12
openstackgerritYair Fried proposed openstack/rally: Split runners to base and plugins  https://review.openstack.org/18090408:12
yfriedboris-42: ^ please check out my newest try.08:12
*** yingjun has quit IRC08:14
yfriedamaretskiy: boris-42: how do I "recheck" Mirantis CI (not jenkins)08:16
amaretskiyyfried: type "recheck rally"08:17
*** fhubik has joined #openstack-rally08:18
*** yingjun has joined #openstack-rally08:18
*** e0ne has joined #openstack-rally08:23
*** igormarnat is now known as imarnat08:45
*** fhubik_afk has joined #openstack-rally08:49
*** fhubik_afk is now known as fhubik_meeting08:49
yfriedamaretskiy: tnx. do you know why https://review.openstack.org/#/c/180509/7 failed?08:49
*** fhubik has quit IRC08:49
amaretskiyyfried: afaik gates are dead for now :(08:50
yfriedamaretskiy: so system failure, and not mine? great!08:51
amaretskiyyfried: i think we just have to wait08:51
yfriedamaretskiy: https://review.openstack.org/#/c/180508/3 andreykurilin09:03
*** e0ne is now known as e0ne_09:03
yfriedamaretskiy: btw "recheck rally" triggered jenkins as well09:04
amaretskiyyfried: great! I believe I will review this soon09:04
yfriedamaretskiy: don't get too excited. this patch only creates 3 __init__.py files :)09:04
yfriedboris-42: are you around?09:05
amaretskiyyfried: oh yes.... :)09:05
*** Miouge has quit IRC09:08
*** aix has joined #openstack-rally09:14
*** yfried is now known as yfried|afk09:17
*** Miouge has joined #openstack-rally09:23
*** Miouge has quit IRC09:24
*** tosky has joined #openstack-rally09:25
*** cdent has joined #openstack-rally09:30
*** yingjun has quit IRC09:32
*** e0ne_ is now known as e0ne09:33
*** redixin has joined #openstack-rally09:35
*** fhubik_meeting is now known as fhubik09:38
openstackgerritRoman Vasilets proposed openstack/rally: Add deploy environment murano benchmark  https://review.openstack.org/17162509:38
openstackgerritRoman Vasilets proposed openstack/rally: Add MuranoPackage context  https://review.openstack.org/13765009:38
*** fhubik has quit IRC09:38
*** fhubik has joined #openstack-rally09:39
*** yfried|afk is now known as yfried09:41
*** yfried is now known as yfried|afk09:51
*** fhubik is now known as fhubik_afk09:53
*** fhubik_afk is now known as fhubik09:54
*** nkhare has quit IRC09:55
*** imarnat is now known as igormarnat09:59
*** nkhare has joined #openstack-rally09:59
*** yfried|afk is now known as yfried10:16
*** pradeep has quit IRC10:18
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241810:20
yfriedamaretskiy: is the gate still broke?10:21
openstackgerritMikhail Dubov proposed openstack/rally: Release 0.0.4  https://review.openstack.org/18049810:22
amaretskiyyfried: I don't know, I usually type "recheck" and see what is happening10:23
yfriedamaretskiy: http://logs.openstack.org/49/168849/6/check/gate-rally-dsvm-rally/99ed110/console.html10:23
yfriedamaretskiy: I think we have a bug in the gate10:23
yfriedamaretskiy: see the Traceback10:23
amaretskiyyfried: something is wrong with heat10:25
*** igormarnat is now known as imarnat10:28
*** fhubik is now known as fhubik_afk10:35
*** Miouge has joined #openstack-rally10:37
*** yfried_ has joined #openstack-rally10:38
*** yfried has quit IRC10:38
*** coolsvap is now known as coolsvap|afk10:43
*** Miouge has quit IRC10:45
amaretskiyyfried: this seems to be a cause of broken gates: https://review.openstack.org/#/c/170608/10:48
*** frobware_ has joined #openstack-rally10:50
*** agarciam has quit IRC10:50
*** agarciam has joined #openstack-rally10:51
*** fhubik_afk is now known as fhubik10:54
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241810:57
*** Miouge has joined #openstack-rally10:58
amaretskiyyfried: here is fix https://review.openstack.org/#/c/180937/10:58
*** Miouge has quit IRC11:03
*** panbalag has joined #openstack-rally11:07
*** pradeep has joined #openstack-rally11:16
*** agarciam has quit IRC11:25
*** agarciam has joined #openstack-rally11:25
*** e0ne is now known as e0ne_11:27
openstackgerritAlexander Maretskiy proposed openstack/rally: [Gates] Disable Heat scenarios  https://review.openstack.org/18094811:29
openstackgerritAlexander Maretskiy proposed openstack/rally: [Gates] Revert `Disable Heat scenarios'  https://review.openstack.org/18094911:35
*** yfried__ has joined #openstack-rally11:44
dmelladoboris-42: pong11:47
dmelladoare you around?11:47
*** yfried_ has quit IRC11:48
*** agarciam has quit IRC11:49
*** agarciam has joined #openstack-rally11:49
*** fhubik is now known as fhubik_afk11:50
*** e0ne_ is now known as e0ne11:51
*** openstackgerrit has quit IRC11:52
*** openstackgerrit has joined #openstack-rally11:52
*** msdubov_ has joined #openstack-rally11:54
openstackgerritMikhail Dubov proposed openstack/rally: Release 0.0.4  https://review.openstack.org/18049811:57
*** softCloud has joined #openstack-rally11:57
*** kiran-r has quit IRC12:00
*** nkhare has quit IRC12:00
*** dpaterson has joined #openstack-rally12:02
yfried__amaretskiy: andreykurilin: redixin: I'm seeing something weird: https://github.com/openstack/rally/blob/master/tests/unit/benchmark/scenarios/mistral/test_workbooks.py#L5112:06
yfried__when I'm running tests via "tox -e py27" it works fine12:07
yfried__everything passes12:07
*** stpierre has joined #openstack-rally12:07
yfried__but executing this test returns AttributeError: 'MistralWorkbooks' object has no attribute 'create_delete_workbook'12:07
yfried__which seems reasonable since it really doesn't have that attribute. anyone knows what's up with that?12:08
yfried__akuznetsova: ping12:08
akuznetsovayfried__, pong12:09
yfried__akuznetsova: https://review.openstack.org/#/c/144320/12:09
yfried__is this your patch?12:09
akuznetsovayfried__, yes12:09
*** dmellado has quit IRC12:09
yfried__akuznetsova: https://github.com/openstack/rally/blob/master/tests/unit/benchmark/scenarios/mistral/test_workbooks.py#L51 please see my above question12:10
boris-42yfried__: hi there12:11
boris-42yfried__: what's up?12:11
*** dmellado has joined #openstack-rally12:11
yfried__boris-42: working on the "move"12:11
boris-42yfried__: do not forget one thing12:11
yfried__boris-42: trying to follow your guide re backword comp12:11
boris-42yfried__: you should import all modules*12:11
yfried__boris-42: what do you mean?12:11
boris-42yfried__: https://github.com/openstack/rally/blob/master/rally/benchmark/__init__.py12:12
boris-42yfried__: regarding to backward compability12:12
boris-42yfried__: we should take care only about base.py modules (to make sure that htey are on the same place)12:12
*** MaxPC has joined #openstack-rally12:13
yfried__boris-42: no idea what that import does/mean12:13
yfried__akuznetsova: ?12:13
amaretskiyyfried__: this test is broken12:13
yfried__amaretskiy: the Mistral test?12:14
*** imarnat is now known as igormarnat12:14
amaretskiyyfried__: mistral tests are not executed at all because there is no __ini__.py12:14
amaretskiy__init__.py12:14
amaretskiyin tests/unit/benchmark/scenarios/mistral/12:14
amaretskiy:)12:14
yfried__amaretskiy: ok. so I should make sure there's no init file in test/unit/.../mistral?12:15
boris-42amaretskiy: it can't be borken12:15
boris-42amaretskiy: or maybe12:15
amaretskiyhttps://github.com/openstack/rally/tree/master/tests/unit/benchmark/scenarios/mistral12:15
boris-42amaretskiy: seems you are right12:16
akuznetsovaoops12:16
boris-42=(12:16
boris-42sad that we merged that..12:17
amaretskiyboris-42: we need a checker for __init__.py on gates12:17
yfried__amaretskiy: boris-42: akuznetsova: now they are being executed in my patch (plugin migration). the __init__ file is still missing, so that's not the reason they are skiiped12:17
*** fhubik_afk is now known as fhubik12:18
amaretskiyyfried__: I run tox on master - result is OK, then add __init__.py and got expected error12:19
yfried__amaretskiy: init in mistral dir or in tests dir?12:19
yfried__amaretskiy: anyway that doesn't work for me12:20
amaretskiyyfried__: so if your tests do fail on master, then your way of running tests is different12:20
yfried__amaretskiy: I'll push the patch and you'll see12:20
yfried__tox -e py2712:20
amaretskiy$ touch tests/unit/benchmark/scenarios/mistral/__init__.py12:20
akuznetsovaI will prepare fix for test12:20
amaretskiyyfried__: I run "tox -e py27" and all results are as expected12:20
openstackgerritRoman Vasilets proposed openstack/rally: Add deploy environment murano benchmark  https://review.openstack.org/17162512:23
yfried__amaretskiy: me too. but on my work branch it doesn't work the same12:24
amaretskiyyfried__: maybe __init__.pyc still exist12:25
amaretskiyyfried__: anyway the main question about test looks solved12:25
yfried__amaretskiy: I'll take a look12:26
openstackgerritRoman Vasilets proposed openstack/rally: Add deploy environment murano benchmark  https://review.openstack.org/17162512:27
*** pradeep has quit IRC12:27
*** igormarnat is now known as imarnat12:27
akuznetsovayfried__, boris-42 do I need to report bug first or I can just create a commit ?12:28
dmelladohey boris-4212:29
dmelladodid you ping me yesterday? ;)12:29
boris-42akuznetsova: create commit12:30
boris-42dmellado: yep we implemented register method in osclients12:30
dmelladooh great, I'll have a look12:31
dmelladowould you have a link to the commit?12:31
openstackgerritRoman Vasilets proposed openstack/rally: Add deploy environment murano benchmark  https://review.openstack.org/17162512:31
yfried__boris-42: akuznetsova: well, it's a bug, so for tracking reasons it would be better to create a bug (1 line of description) so your commit can point to it as Closes-bug12:32
akuznetsovayfried__, yes, I meant it12:33
yfried__akuznetsova: then go ahead12:34
akuznetsovayfried__, I am already in progress12:34
boris-42dmellado: https://review.openstack.org/#/c/180937/12:34
dmelladothanks boris-4212:35
*** imarnat is now known as igormarnat12:38
*** msdubov_ has quit IRC12:38
*** igormarnat is now known as imarnat12:39
openstackgerritAnastasia Kuznetsova proposed openstack/rally: Fix Mistral unit tests  https://review.openstack.org/18097412:39
akuznetsovaboris-42, yfried__ , amaretskiy patch is ready ^12:40
amaretskiyok12:40
*** tfreger has quit IRC12:42
*** pradeep has joined #openstack-rally12:42
yfried__boris-42: https://review.openstack.org/#/c/180508/ please comment if you think these __init__ files should contain some imports12:43
*** msdubov_ has joined #openstack-rally12:54
*** yingjun has joined #openstack-rally13:07
*** fhubik has quit IRC13:11
*** kiran-r has joined #openstack-rally13:12
*** fhubik has joined #openstack-rally13:14
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241813:16
*** imarnat is now known as igormarnat13:20
*** igormarnat is now known as imarnat13:20
*** imarnat is now known as igormarnat13:20
*** yingjun has quit IRC13:23
*** pradeep has quit IRC13:25
*** openstackstatus has joined #openstack-rally13:26
*** ChanServ sets mode: +v openstackstatus13:26
*** pradeep has joined #openstack-rally13:26
*** rdas has quit IRC13:27
*** yfried__ has quit IRC13:28
*** pradeep has quit IRC13:31
*** igormarnat is now known as imarnat13:33
*** pradeep has joined #openstack-rally13:34
*** e0ne is now known as e0ne_13:34
*** e0ne_ is now known as e0ne13:37
*** msdubov_ has quit IRC13:39
*** pradeep has quit IRC13:45
*** kiran-r has quit IRC13:51
*** fhubik has quit IRC13:59
openstackgerritAndrey Kurilin proposed openstack/rally: Implement subunit_parser  https://review.openstack.org/18014414:05
openstackgerritAndrey Kurilin proposed openstack/rally: Implement subunit_parser  https://review.openstack.org/18014414:10
*** imarnat is now known as igormarnat14:10
*** neeti has quit IRC14:17
*** agarciam has quit IRC14:18
openstackgerritChris St. Pierre proposed openstack/rally: Fix security groups cleanup  https://review.openstack.org/17942814:21
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241814:22
*** pradeep has joined #openstack-rally14:25
panbalagamaretskiy, Hi14:37
amaretskiypanbalag: hi14:37
panbalagamaretskiy, I'm responding to one of your comments "Restructured text provides numeric lists:#. Item 1#. Item 2" in https://review.openstack.org/#/c/166487/6/doc/specs/in-progress/common-rally-verify.rst14:38
panbalagamaretskiy, what exactly did you mean? did you want to replace 1. 2. etc with # item 1, # item 2 etc?14:38
amaretskiypanbalag: RestructuredText is a markup language that is useful for generating docs in different formats (usually to HTML) - so there is special markup for lists generation14:40
amaretskiypanbalag: open http://rst.ninjs.org/14:40
amaretskiypanbalag: and play with lists14:41
amaretskiypanbalag: also there is a reference http://docutils.sourceforge.net/docs/user/rst/quickref.html#enumerated-lists14:41
panbalagamaretskiy, I'll follow this example. Thanks14:42
dmelladopanbalag: I highly recommend you to take a look at this too :)14:46
dmelladohttp://sphinx-doc.org/14:46
dmelladoyou can get there examples with rst14:46
fricklerstpierre: we are still getting python3.4 errors: http://paste.openstack.org/show/216216/ (and Hi, I'm Jens ;)14:51
frickleralso Mirantis CI seems broken, http://185.8.56.88/logs/TrXpnGDIAbhaSlMw/tox-ecli-py27-postgres-dsvm/ gives me a 40414:52
*** fhubik has joined #openstack-rally14:52
*** fhubik has quit IRC14:53
*** fhubik has joined #openstack-rally14:54
panbalagdmellado, Thanks. I'll try it out14:54
*** klindgren__ has joined #openstack-rally14:56
openstackgerritRoman Vasilets proposed openstack/rally: Add deploy environment murano benchmark  https://review.openstack.org/17162514:57
*** anshul has quit IRC14:57
*** klindgren_ has quit IRC14:58
panbalagamaretskiy, I forgot the sequence of commands to send out the file for review..is it git add , git commit and then git review?15:03
*** fhubik_afk has joined #openstack-rally15:03
*** fhubik has quit IRC15:04
amaretskiypanbalag: yes, but do not forget to be sure that your patch is rebased to up-to-date master, and run tox before `git review' to be sure that everything is fine with tests15:04
*** pboros has joined #openstack-rally15:05
panbalagwhats the command to rebase? git pull?15:05
panbalagamaretskiy,  whats the command to rebase? git pull?15:05
amaretskiy`git rebase master'15:05
amaretskiyyes, you need to checkout master and run git pull15:05
amaretskiyand then checkout back to patch branch and run rebase if required15:06
*** mwagner_lap has quit IRC15:10
openstackgerritRoman Vasilets proposed openstack/rally: DO NOT MERGE. Test  https://review.openstack.org/18103115:11
*** amalinow has quit IRC15:14
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241815:16
fricklergate-rally-dsvm-rally is still not running with heat, it seems15:18
panbalagamaretskiy, there are few tox tests that fail because of  "ImportError: No module named 'boto'" . It is probably some package missing in my python installation..any idea what this is?15:20
amaretskiyrm -rf .tox15:20
amaretskiyso next time when tox is run it reinstalls updated dependencies from test-requirements.txt15:21
panbalagamaretskiy, ok. let me try  that15:24
*** pradeep has quit IRC15:25
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241815:28
openstackgerritChris St. Pierre proposed openstack/rally: Fix security groups cleanup  https://review.openstack.org/17942815:32
panbalagamaretskiy, that worked. Thanks15:36
*** pboros has quit IRC15:42
*** pradeep has joined #openstack-rally15:42
*** pboros has joined #openstack-rally15:49
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241815:57
*** jose_lausuch has quit IRC15:58
*** mwagner_lap has joined #openstack-rally16:00
*** e0ne is now known as e0ne_16:08
openstackgerritIgor Degtiarov proposed openstack/rally: [WIP][Ceilometer] Add  scenario for events and traits  https://review.openstack.org/18051416:12
*** e0ne_ is now known as e0ne16:17
*** pbandzi has quit IRC16:41
*** amaretskiy has quit IRC16:46
fricklerso the infra ppl confirmed that the rally job should be running with heat enabled, still it fails here: http://logs.openstack.org/28/179428/7/check/gate-rally-dsvm-rally/3a24ab4/console.html#_2015-05-07_15_54_32_55316:49
fricklerhttp://logs.openstack.org/28/179428/7/check/gate-rally-dsvm-rally/3a24ab4/console.html#_2015-05-07_15_54_16_076 does not see heat in the services list, though16:54
openstackgerritPrasanth Anbalagan proposed openstack/rally: Specs to modify rally verify  https://review.openstack.org/18107816:58
*** exploreshaifali has joined #openstack-rally17:00
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241817:01
boris-42frickler: we know17:02
boris-42frickler: they deleted it17:03
boris-42frickler: from default and didn't say17:03
boris-42frickler: to rally team17:03
boris-42frickler: I don't know the polite word to describe what I think about the person who did this17:04
*** e0ne has quit IRC17:11
panbalagboris-42, regarding the new review request. Since rally changed  from stackforge to openstack, amaretskiy suggested cloning the repo again from https://github.com/openstack/rally and copying files over there.17:11
boris-42panbalag: so that was the bad idea 2 times17:12
boris-42panbalag: first of all the only thing that you need to do with old repo is to remove gerrit remote17:12
boris-42panbalag: and run "git review -s"17:13
panbalagboris-42, oh I didnt know that17:13
boris-42panbalag: after git pull all changes17:13
boris-42panbalag: the second bad advice was to copy changes17:13
boris-42panbalag: if you chose to keep new repo do the next thing17:13
boris-42panbalag: git checkout master17:13
boris-42panbalag: git checkout -b your_new_branch17:13
boris-42panbalag: and on gerrit page of all patch find the "cherry-pick" tab17:14
boris-42panbalag: copy-paste this command and run inside your repo17:14
panbalagboris-42, which command ?17:17
panbalagboris-42, I still have the old repo (just renamed it to a temp directory..did not delete it)..can I just change the gerrit remote there and run "git review -s" ? and is there a way to delete the new one?17:18
fricklerboris-42: yes, but the patch to restore it was merged and yet we are still seeing errors17:20
*** yfried__ has joined #openstack-rally17:21
boris-42 frickler because it doens't return17:25
boris-42heat17:25
panbalagboris-42, what does the command inside "cherry-pick" tab do?17:25
boris-42panbalag: it copy-paste your commit from gerrit17:26
boris-42panbalag: to your local repo17:26
boris-42panbalag: in the rigth way17:26
panbalagboris-42, I'm a bit confused. So are we trying to overwrite the new review request with the old review request? or somehow copy the comments from old one to new one?17:28
panbalagboris-42, I would rather prefer keeping the old review request because all the comments are there .17:29
boris-42panbalag: so to continue work on old review17:32
boris-42panbalag: you need to keep only one thing in commit message17:32
boris-42panbalag: change-id17:32
boris-42panbalag: but proper way to get in new repo patch17:32
boris-42panbalag: is to cherry-pick it17:33
boris-42panbalag: and you have link in your old review that allows you to do this17:33
*** yfried__ has quit IRC17:34
*** yfried__ has joined #openstack-rally17:35
panbalagboris-42, ok. also what should be the new gerrit remote path? ssh://<user>@review.openstack.org:29418/openstack/rally.git?17:37
*** karimb has quit IRC17:38
boris-42panbalag: you should do the next17:42
boris-42git remote remove gerrit17:42
boris-42git checkout master17:42
boris-42git pull17:42
boris-42git review -s17:42
boris-42this will do all that you need and setup new remote17:42
boris-42don't do this by hands17:42
panbalagboris-42, ok let me do that17:43
fricklerhmm, looking at https://review.openstack.org/#/c/169709/1/stackrc maybe we need "h-eng,h-api,h-api-cfn,h-api-cw" back in enabled_services instead of "heat"?17:44
fricklerboris-42: ^^17:44
boris-42frickler: we are already working on this17:44
boris-42frickler: redixin actually17:45
fricklerk, thx17:45
boris-42frickler: https://review.openstack.org/#/c/181088/17:45
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241817:53
*** e0ne has joined #openstack-rally18:07
*** gema has quit IRC18:16
*** pradeep has quit IRC18:16
openstackgerritPrasanth Anbalagan proposed openstack/rally: Rally verify support for any in-tree Openstack project functional tests  https://review.openstack.org/16648718:16
openstackgerritPrasanth Anbalagan proposed openstack/rally: Addressed comments from patchset 6.  https://review.openstack.org/18110318:16
panbalagboris-42, I did use the cherry-pick from old review request. But looks like again a new request has been generated.18:17
stpierreyou need to amend the commit18:18
boris-42stpierre: actually cherry-pick should be ok18:18
stpierrecherry-pick, amend, review18:18
* stpierre prefers 'git review -d' to cherry-pick, but that's an underwear problem. in the end you need to review a single commit -- that's a hard requirement18:19
panbalagboris-42, stpierre, how do I delete the requests https://review.openstack.org/181103 and https://review.openstack.org/181078.18:20
stpierrethere's an "Abandon Change" button18:20
stpierrepanbalag: once you get used to gerrit it's actually pretty slick, but it's got a doozy of a learning curve.18:23
panbalagstpierre, Thanks. yeah. I'm still getting used to it.18:24
boris-42stpierre: heeh18:24
boris-42stpierre: +118:24
boris-42stpierre: but overall after you learn it it's nice18:24
boris-42stpierre: https://twitter.com/PavlovicBoris/status/59637689156259840018:25
boris-42panbalag: ^18:25
boris-42WOOT18:25
*** gema_ has joined #openstack-rally18:25
boris-42frickler: okay patch that fix rally gates18:25
boris-42isg going to be merged soon18:25
stpierresweet!18:26
stpierre(on both counts)18:26
panbalagboris-42, stpierre, so everytime I want to modify the patch and post it for review, I need to do a cherry-pick from previous version , git amend and git review?18:26
boris-42panbalag: nope18:27
boris-42panbalag: cherry-pick is just copy-paste function in git18:28
boris-42panbalag: so you copy patch from review and paste inside your local repo18:28
boris-42panbalag: after that you should just amend + send on review18:28
stpierrepanbalag: you've got a number of possible workflows: 1) keep a persistent branch for work on a given feature/bug; amend the HEAT commit and review for each patchset; 2) 'git review -d' to download the current patch in gerrit, amend, then 'git review -f' to review it and delete the temporary branch; 3) keep a persistent branch for work on a given feature/bug; make new commits to that branch, then squash the branch to master to review.18:28
stpierrei'm sure there are others. i use a mix of 1 and 2, but i know 3 has its adherents18:28
stpierre2 is mainly useful when you're collaborating with someone else on a single patch, because you always get the latest version of the patch in gerrit18:29
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241818:30
stpierrejust remember: nearly every git problem can be solved with MOAR BRANCHES18:30
*** pradeep has joined #openstack-rally18:32
*** igormarnat is now known as imarnat18:34
*** exploreshaifali_ has joined #openstack-rally18:44
*** exploreshaifali has quit IRC18:46
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241818:52
openstackgerritOmar Khalid proposed openstack/rally: Changes for scenario for VM-VM throughput L2/L3 For now it supports iperf/TCP but plan is to support nuttcp by adding a new nuttcp class Once my other boot_hypervisor_test review is merged i will provide ability to control same/different host  https://review.openstack.org/15190519:02
panbalagboris-42, do you know how I can execute "gate-rally-dsvm-rally" test locally?19:04
panbalagboris-42, the logs show "Benchmark HeatStacks.list_stacks_and_resources[0] has wrong configuration.. Invalid scenario argument: 'Service is not available: heat'". I dont understand what the specs has got anything to do with this19:06
*** karimb has joined #openstack-rally19:10
boris-42panbalag: Rally gates are broken19:13
boris-42panbalag: they should be soon fixed19:13
boris-42stpierre> just remember: nearly every git problem can be solved with MOAR BRANCHES19:13
boris-42LOL19:13
panbalagboris-42, now if I just want to re execute the tests once gates are fixed, how do I do that?19:14
panbalagboris-42, does jenkins do it automatically?19:15
boris-42panbalag: you should write "recheck" in commit message19:16
*** mpopow has joined #openstack-rally19:32
*** imarnat is now known as igormarnat19:40
boris-42HOLA19:55
boris-42gates are unblocked19:55
boris-42finally19:55
*** frobware_ has quit IRC20:10
*** pothole is now known as ptoohill20:19
*** exploreshaifali_ has quit IRC20:26
stpierreare we blocked on something else now? it looks like the tox-ecli-py**-postgres-dsvm tests are reliably failing with no obvious error message20:28
boris-42stpierre: so I think it's the different issue20:30
boris-42stpierre: I hope redixin will fix it20:30
boris-42stpierre: however it's not blocking like Jenkins20:30
boris-42stpierre: so 2 patches are going to be merged20:30
*** ibiris is now known as ibiris_away20:40
*** msdubov_ has joined #openstack-rally20:46
boris-42stpierre: turned it off20:47
boris-42stpierre: unit it will be fixed20:47
*** e0ne has quit IRC20:51
stpierrecool20:51
*** pboros has quit IRC20:53
boris-42stpierre: seems like we should add more jobs20:56
boris-42stpierre: and organize better where benchmarks are run *20:56
*** andreykurilin__ has joined #openstack-rally20:58
openstackgerritMerged openstack/rally: Fix broken samples scenario for CinderVolumes.create-and-attach-volume  https://review.openstack.org/18020520:59
*** baker has joined #openstack-rally21:04
*** exploreshaifali_ has joined #openstack-rally21:06
openstackgerritMerged openstack/rally: Remove sla from Heat scenarios in MOS job  https://review.openstack.org/18046721:13
*** cdent has quit IRC21:21
*** msdubov__ has joined #openstack-rally21:24
*** msdubov_ has quit IRC21:27
*** msdubov__ has quit IRC21:33
*** exploreshaifali_ has quit IRC21:35
openstackgerritMerged openstack/rally: Updated from global requirements  https://review.openstack.org/18060521:44
*** stpierre has quit IRC21:44
*** mwagner_lap has quit IRC21:45
*** MaxPC has quit IRC21:50
openstackgerritBoris Pavlovic proposed openstack/rally: Speed up and reorganize rally jobs  https://review.openstack.org/18116321:52
openstackgerritOmar Khalid proposed openstack/rally: Changes for scenario for VM-VM throughput L2/L3 For now it supports iperf/TCP but plan is to support nuttcp by adding a new nuttcp class Once my other boot_hypervisor_test review is merged i will provide ability to control same/different host  https://review.openstack.org/15190521:56
*** softCloud has quit IRC21:56
openstackgerritOmar Khalid proposed openstack/rally: Changes for scenario for VM-VM throughput L2/L3 For now it supports iperf/TCP but plan is to support nuttcp by adding a new nuttcp class Once my other boot_hypervisor_test review is merged i will provide ability to control same/different host  https://review.openstack.org/15190522:08
openstackgerritMerged openstack/rally: Fix RALLY_SCENARIO filename in rally-gate.py  https://review.openstack.org/18052522:15
openstackgerritMerged openstack/rally: Creates "plugins" tree  https://review.openstack.org/18050822:20
*** baker has quit IRC22:20
*** karimb has quit IRC22:20
*** mpopow has quit IRC22:24
*** mpopow has joined #openstack-rally22:38
*** tosky has quit IRC22:40
*** redixin has quit IRC23:24
openstackgerritBoris Pavlovic proposed openstack/rally: Speed up and reorganize rally jobs  https://review.openstack.org/18116323:40
*** abitha has joined #openstack-rally23:41
abithaboris-42: Hi Boris, i have a question about adding test package to rally23:43
boris-42abitha: hi there23:44
abithaone of the test scenario im doing requires the use of scp. I see that rally has paramiko but no scp... how do i go about adding scp support?23:44
abithado we just add it to our test-requirement.txt?23:45
boris-42abitha: first of all which library is it?23:45
abithahttps://pypi.python.org/pypi/scp23:45
boris-42abitha: the second thing test-requirments.txt are requirements required during the testing23:46
boris-42abitha: no such lib here https://github.com/openstack/requirements/blob/master/global-requirements.txt23:46
*** mpopow has quit IRC23:46
boris-42abitha: so it will be quite problematic to add it23:47
abithaok so this package needs to be in the global requirement23:47
boris-42abitha: yep otherwice our CI will fail23:47
abithaok ill find another way then23:48
abithathanks23:48
boris-42abitha: np23:49
boris-42abitha: you can calll it from shutil23:49
boris-42abitha: or something like that23:49
abithaoh ok. thanks :)23:50
« Wednesday, 2015-05-06 Index Friday, 2015-05-08 »

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