Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2014-08-08

| Channels | #openstack-rally index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:15 yingjun joined #openstack-rally
01:47 ilbot3 joined #openstack-rally
01:47 Topic for #openstack-rally is now ☁ Rally IRC chat logs http://irclog.perlgeek.de/openstack-rally ☁ Key persons to ask:  boris-42, rediskin, andreykurilin, k4n0  Ã¢ËœÂ Open tasks https://trello.com/b/DoD8aeZy/rally ☁ To publish changes to Rally:  https://wiki.openstack.org/wiki/Rally/Develop#How_to_contribute   ☁ Rally weekly updates:  https://wiki.openstack.org/wiki/Rally/Updates
01:59 AlexF_ joined #openstack-rally
02:11 yingjun joined #openstack-rally
02:12 yaguang joined #openstack-rally
02:18 AlexF_ joined #openstack-rally
02:26 yingjun joined #openstack-rally
02:27 yingjun_ joined #openstack-rally
02:28 yingjun joined #openstack-rally
02:31 yingjun_ joined #openstack-rally
02:33 AlexF_ joined #openstack-rally
02:35 yingjun joined #openstack-rally
02:37 yingjun__ joined #openstack-rally
02:43 yingjun joined #openstack-rally
02:44 yingjun_ joined #openstack-rally
03:12 yingjun joined #openstack-rally
04:27 AlexF_ joined #openstack-rally
04:45 lordd joined #openstack-rally
04:59 Poornima joined #openstack-rally
05:03 AlexF_ joined #openstack-rally
05:16 nkhare joined #openstack-rally
05:27 AlexF_ joined #openstack-rally
05:40 ajayaa joined #openstack-rally
05:54 vaidyanath joined #openstack-rally
06:02 k4n0 joined #openstack-rally
06:21 penguinRaider joined #openstack-rally
06:50 yingjun joined #openstack-rally
07:06 andreykurilin_ joined #openstack-rally
07:29 vaidyanath left #openstack-rally
07:33 lordd joined #openstack-rally
07:54 amaretskiy joined #openstack-rally
07:55 lordd joined #openstack-rally
08:21 nkhare joined #openstack-rally
08:24 rediskin joined #openstack-rally
08:34 andreykurilin k4n0, coolsvap : hi! do you have a bit of free time?:)
08:34 k4n0 andreykurilin, bit busy today
08:34 k4n0 andreykurilin, but we can discuss task, ill start on Monday
08:35 andreykurilin k4n0, I want to ask you review a big patch:)
08:36 k4n0 andreykurilin, sure, i can do that
08:36 k4n0 andreykurilin, please pass me the link
08:36 andreykurilin k4n0, https://review.openstack.org/#/c/66919
08:36 k4n0 andreykurilin, ok i am on it
08:37 yingjun joined #openstack-rally
08:37 andreykurilin k4n0, this is base for rallyclient. most of loc in related to common code(rallyclient/openstack/* and tools/*), so the patch is not so big as it seems at first
08:44 rook joined #openstack-rally
08:45 k4n0 andreykurilin, got it
08:45 k4n0 andreykurilin, ill start after lunch
08:45 andreykurilin k4n0, thanks
08:53 nkhare joined #openstack-rally
08:57 openstackgerrit Andrey Kurilin proposed a change to stackforge/python-rallyclient: Add RallyShell  https://review.openstack.org/112612
09:05 openstackgerrit Andrey Kurilin proposed a change to stackforge/python-rallyclient: Add RallyShell  https://review.openstack.org/112612
09:08 yingjun joined #openstack-rally
09:26 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: All scenario runs time measurement  https://review.openstack.org/105318
09:27 vaidyanath joined #openstack-rally
09:32 openstackgerrit Andrey Kurilin proposed a change to stackforge/python-rallyclient: Add RallyShell  https://review.openstack.org/112612
09:56 aswadr joined #openstack-rally
09:56 lordd joined #openstack-rally
10:04 coolsvap andreykurilin, i was following the rallyclient changes yesterday but was a bit busy with a fix, I will see all the changes today
10:09 AlexF_ joined #openstack-rally
10:24 AlexF_ joined #openstack-rally
10:25 davidlenwell joined #openstack-rally
10:41 nkhare_ joined #openstack-rally
10:52 vaidyanath joined #openstack-rally
10:52 tnurlygayanov joined #openstack-rally
10:53 ajayaa joined #openstack-rally
11:02 boris-42 k4n0 ping
11:02 boris-42 k4n0 could you address comment here https://review.openstack.org/#/c/106813/3
11:03 boris-42 k4n0 and I will merge cause andreykurilin seems like need this patch
11:03 boris-42 k4n0 so I will be ok to remove -1, but Li made good point
11:15 k4n0 boris-42, yes
11:15 k4n0 boris-42, checking
11:16 boris-42 k4n0 thanks
11:30 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Adds --json,--pprint flags to cmd  https://review.openstack.org/106813
11:34 openstackgerrit A change was merged to stackforge/rally: Amended sphinx documents  https://review.openstack.org/110738
11:35 k4n0 boris-42, done
11:35 k4n0 boris-42, andreykurilin , do we agree on the decorator ?
11:36 jjmb joined #openstack-rally
11:52 ajayaa joined #openstack-rally
11:58 lordd joined #openstack-rally
12:08 AlexF_ joined #openstack-rally
12:10 nkhare_ joined #openstack-rally
12:10 nkhare joined #openstack-rally
12:16 Sahil___ joined #openstack-rally
12:18 rook nkhare:  ping
12:18 lordd joined #openstack-rally
12:18 rook nkhare: do you know if Rally has a API service?
12:19 rook boris-42: lordd ^
12:19 lordd hey
12:20 rook Yo
12:20 rook Good Morning
12:20 rook if it is morning for you
12:20 lordd yeah, it's 2pm and I'm in Spain, so it's early morning
12:20 lordd haha
12:20 rook lol, it is 0820 here
12:20 rook My guess there is no API service
12:21 lordd no API service yet
12:21 rook "Yet"
12:21 rook is there a blueprint
12:23 lordd yep, one sec
12:23 rook Because right now, I need to automate increasing load, so I would need to rally task start <mytask> , grep for the task_id, then watch over rally task list for completion, then modify the json to increase things
12:23 rook simple sed
12:24 lordd the base: https://blueprints.launchpad.net/rally/+spec/api-base
12:24 lordd in fact there are like 3 blueprints related to API
12:24 lordd but this one should be the starting point
12:27 lordd so once the api-base is done, work on api-rest will start
12:27 lordd well start, be increased, as it is dependent on this api-base
12:34 andreykurilin joined #openstack-rally
12:35 amaretskiy joined #openstack-rally
12:38 boris-42 rook we are working on that
12:38 boris-42 rook hi there
12:38 boris-42 rook let me point you to some patches
12:39 boris-42 rook https://review.openstack.org/#/c/103377/10
12:39 boris-42 https://review.openstack.org/#/c/112181/
12:39 boris-42 https://review.openstack.org/#/c/66919/
12:40 boris-42 https://review.openstack.org/#/c/112612/
12:40 boris-42 rook ^ so this all is about as a Service stuff
12:45 nkhare rook, sorry .in a meeting now
12:58 boris-42 nkhare I already replied =)
12:58 nkhare boris-42, thanks
13:35 zarric joined #openstack-rally
13:51 AlexF_ joined #openstack-rally
13:53 ajayaa joined #openstack-rally
14:04 openstackgerrit A change was merged to stackforge/rally: All scenario runs time measurement  https://review.openstack.org/105318
14:13 openstackgerrit Andrey Kurilin proposed a change to stackforge/rally: WIP: fix ceilometerclient  https://review.openstack.org/112883
14:20 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Fix rally-gate.sh exit status  https://review.openstack.org/112887
14:24 boris-42 andreykurilin ^ this should failed without your patch
14:24 boris-42 =)
14:24 boris-42 andreykurilin fail*
14:25 andreykurilin boris-42 :)
14:31 openstackgerrit A change was merged to stackforge/rally: Adds --json,--pprint flags to cmd  https://review.openstack.org/106813
14:32 boris-42 k4n0 OOHH
14:32 boris-42 k4n0 http://logs.openstack.org/84/112684/2/check/gate-rally-dsvm-neutron-rally/1078983/console.html
14:33 boris-42 k4n0 dsvm-neutron doesn't have nano image
14:33 boris-42 rediskin so about cleanup context
14:33 boris-42 rediskin there are couple nits otherwise we can merge it
14:34 boris-42 rediskin could you address them?
14:36 rediskin sure
14:39 boris-42 coolsvap hi
14:39 boris-42 coolsvap any progress of refactoring validation stuff?
14:44 boris-42 rediskin andreykurilin so guys
14:44 boris-42 rediskin andreykurilin  seems like there are some conernc
14:44 boris-42 about naming
14:44 boris-42 rediskin andreykurilin coolsvap  this alias
14:44 boris-42 https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/cinder/volumes.py#L16
14:45 boris-42 i think that @scneario_base.scenario is to long
14:45 boris-42 and 2 times using scenario word
14:45 boris-42 is as well to explicit
14:45 boris-42 thougths?
14:45 andreykurilin boris-42, +1
14:45 boris-42 coolsvap ^ what do you think?
14:53 boris-42 rmk rook aswadr ^ btw guys what do you think as well?
14:54 tzabal joined #openstack-rally
14:54 boris-42 tzabal hi
14:54 tzabal boris-42 hi
14:56 boris-42 tzabal we are discusing stuff here
14:56 boris-42 tzabal http://irclog.perlgeek.de/openstack-rally/2014-08-08
15:01 tzabal boris-42 just finished reading it
15:04 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Split user and admin cleanup to 2 separate classes  https://review.openstack.org/100590
15:07 boris-42 ajayaa hi how are you?
15:07 ajayaa boris-42, Hi
15:08 ajayaa I am good man.
15:09 ajayaa boris-42, Can I help you in some way?
15:09 boris-42 ajayaa contribut to rally?)
15:09 boris-42 ajayaa did you try it?)
15:09 boris-42 ajayaa do you have any UX and so on?
15:09 ajayaa Yes. I have tried rally already. It is awesome.
15:09 ajayaa UX?
15:10 boris-42 ajayaa user experience
15:10 ajayaa no man!
15:10 boris-42 ajayaa =)
15:10 ajayaa Just python, c++
15:10 boris-42 ajayaa by UX I mean something like e.g. I wasn't able to do this with Rally => rally suxxxx
15:10 boris-42 ajayaa =)
15:12 ajayaa nope. okay. I am not a power user of rally yet, Our cloud is yet to come to life. So no.
15:26 openstackgerrit Andrey Kurilin proposed a change to stackforge/rally: Fix the way of obtaining ceilometerclient  https://review.openstack.org/112883
15:31 jaypipes joined #openstack-rally
15:34 coolsvap boris-42, sorry was away, reading
15:34 boris-42 coolsvap no worries
15:36 coolsvap boris-42, I pushed the patch for refactoring without tests, I didn't had a chance to talk to msdubov
15:36 boris-42 coolsvap ?
15:36 boris-42 coolsvap I mean we are speaking about refactoring  and t @scneario_base.scenario
15:36 boris-42 coolsvap that using 2 times scenario in this is too explicit
15:36 boris-42 andreykurilin rediskin and me at least have such opinion
15:38 coolsvap boris-42, the same will be there at all places, would you guys recommend to shorten it somehow like scen_base or something?
15:38 boris-42 coolsvap just use @base
15:38 boris-42 coolsvap without alias
15:39 boris-42 coolsvap and only in place where we have multiiple bases
15:39 boris-42 coolsvap just couple of files we can have aliases
15:39 boris-42 @base.scneario looks nice
15:39 boris-42 @base.atomic_action_timer as well
15:39 coolsvap boris-42, :) okay will do that no issues
15:39 boris-42 coolsvap thank you
15:40 coolsvap i just wanted it to be explicit to know what you are using
15:45 boris-42 coolsvap I know, but in such case when it is always similiar
15:45 boris-42 coolsvap and there is no other variants it's better to be short
15:45 coolsvap boris-42, hmm
15:45 boris-42 coolsvap I mean in all benchmarks we are using only one module called base
15:45 boris-42 coolsvap so person learns it only 1 time
15:46 boris-42 coolsvap and then it is always the same, and being short here is okay
15:46 coolsvap boris-42, sure I will push the patch for that in sometime
15:46 boris-42 coolsvap thank you
15:51 penguinRaider joined #openstack-rally
15:56 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Fix rally-gate.sh exit status  https://review.openstack.org/112887
15:58 lordd joined #openstack-rally
16:24 mux joined #openstack-rally
16:32 boris-42 coolsvap ohhh
16:32 boris-42 coolsvap I didn't saw this https://review.openstack.org/#/c/111989/
16:32 boris-42 seen*
16:33 coolsvap boris-42, yup  I will do that
16:33 boris-42 coolsvap going to review it
16:39 jjmb joined #openstack-rally
16:48 marcoemorais joined #openstack-rally
16:59 openstackgerrit Swapnil Kulkarni proposed a change to stackforge/rally: Update the scenari_base to base in benchmarks  https://review.openstack.org/112973
17:00 coolsvap boris-42, ^^
17:07 openstackgerrit Swapnil Kulkarni proposed a change to stackforge/rally: Update the scenario_base to base in benchmarks  https://review.openstack.org/112973
17:31 marcoemorais joined #openstack-rally
17:39 openstackgerrit Swapnil Kulkarni proposed a change to stackforge/rally: Add Rally create keypair CLI command  https://review.openstack.org/98992
17:45 openstackgerrit Tzanetos Balitsaris proposed a change to stackforge/rally: Modify config semantic validation of benchmark engine  https://review.openstack.org/112981
17:46 tzabal boris-42 if you can please check the above review ^^ and tell me if you are ok with that way. If we are ok, I will continue with the unit tests.
17:48 openstackgerrit Yan Ning proposed a change to stackforge/rally: changed image name for nova test  https://review.openstack.org/112983
17:48 openstackgerrit Swapnil Kulkarni proposed a change to stackforge/rally: Add Rally create keypair CLI command  https://review.openstack.org/98992
18:00 openstackgerrit Swapnil Kulkarni proposed a change to stackforge/rally: Fix copyright in doc/source/config.py  https://review.openstack.org/112987
18:04 lordd joined #openstack-rally
18:14 lordd joined #openstack-rally
18:14 openstackgerrit Yan Ning proposed a change to stackforge/rally: changed image name for nova test  https://review.openstack.org/112989
18:34 RaginBajin joined #openstack-rally
18:49 openstackgerrit Joseph Bajin proposed a change to stackforge/rally: Lowercases every item from the role list that is compared with admin https://bugs.launchpad.net/rally/+bug/1354549  https://review.openstack.org/112999
19:01 boris-42 coolsvap thnaks
19:01 boris-42 RaginBajin hi
19:01 boris-42 RaginBajin thanks for bug fix
19:02 RaginBajin boris-42: Hi there.  No problem. I just started looking at using rally for some status alerting and wanted to try it out. That's when I ran into this issue. It was an easy fix
19:10 boris-42 RaginBajin one small nit with commit message
19:10 RaginBajin ok
19:10 RaginBajin This is my first commit so be gentle :)
19:14 boris-42 RaginBajin so one thing then
19:14 boris-42 RaginBajin don't touch pls cahgne-id
19:14 boris-42 RaginBajin it's very important to keep it
19:15 RaginBajin I didn't.  Git recommended I add a githook to add that change id. I amended the commit message, which added it and then reran the git review -R
19:17 boris-42 RaginBajin yep
19:17 boris-42 RaginBajin that's proper way to do things
19:18 RaginBajin Ok, so what I did was fine then?
19:18 boris-42 RaginBajin I made here review https://review.openstack.org/#/c/112999/
19:18 boris-42 RaginBajin you need to push Change-set 2 with fixes
19:18 boris-42 you should*
19:18 boris-42 RaginBajin and to do this properly, you shouldn't change cahnge-id in commit message
19:19 boris-42 RaginBajin use git commit --amend
19:19 boris-42 RaginBajin and re run git review
19:21 RaginBajin So basically remove the link that I had in there, change it to Closes-bug: #1354549, and then add a little bit more detail? Is that correct?
19:24 openstackgerrit Joseph Bajin proposed a change to stackforge/rally: Fixes the check for admin role to be case-insenstive. Lowercases every item from the role list that is compared with admin Closes-bug: #1354549  https://review.openstack.org/112999
19:29 boris-42 RaginBajin last one nit, sorry=)
19:30 boris-42 RaginBajin we need to put somewhere instruction about commit syntax
19:30 RaginBajin boris-42: Yes please because I'm following all the directions, and basically just did what you asked and it's somehow still wrong
19:31 RaginBajin So, any examples of EXACTLY what you are looking for would be fantastic
19:31 boris-42 RaginBajin just one empty line
19:31 boris-42 RaginBajin after first line with title
19:31 RaginBajin What title
19:31 boris-42 RaginBajin and it will be perfect
19:32 RaginBajin My commit message didn't have a title, and the example you provided wasn't the title of my bug
19:32 boris-42 RaginBajin title of commit message is first line of it
19:33 boris-42 RaginBajin e.g. it's showed everywhere in github, in git as a title of commit
19:33 RaginBajin So what is the rest of the commit message suppose to have then.
19:33 boris-42 RaginBajin it's description of what you have done or issue
19:34 boris-42 RaginBajin so it helps reviewers to understand what does patch
19:34 boris-42 1) So one short line to understand what about is patch
19:34 boris-42 2) one empty line
19:34 boris-42 3) Any amount of lines that describes issue/improvmetns/what was done
19:34 RaginBajin ok
19:35 boris-42 4) closes-bug: #XXX, or bp name_of_bp_on_launchpad
19:35 openstackgerrit Joseph Bajin proposed a change to stackforge/rally: Fixes the check for admin role to be case-insenstive.  https://review.openstack.org/112999
19:35 boris-42 5) Change-id: XXX
19:35 boris-42 RaginBajin that's is structure of commit message for all openstack projects, not just rally
19:36 boris-42 RaginBajin actually they are too nerdy and they have quite long instruction
19:36 boris-42 RaginBajin https://wiki.openstack.org/wiki/GitCommitMessages
19:37 RaginBajin Thanks for the link. I'll look at it some more. I appreciate the feedback and patience with me
19:37 boris-42 RaginBajin +2ed your change
19:37 boris-42 RaginBajin now we should find second maintainer to approve it
19:38 boris-42 rediskin ping are you around?
19:43 penguinRaider joined #openstack-rally
19:43 AlexF_ joined #openstack-rally
20:08 openstackgerrit Swapnil Kulkarni proposed a change to stackforge/rally: Refactor benchmark/context/quota.py  https://review.openstack.org/113011
20:12 coolsvap boris-42, gn! see ya mostly on monday!
20:12 boris-42 coolsvap thanks for patches
20:12 boris-42 coolsvap have a good weekend
20:24 coolsvap joined #openstack-rally
20:35 andreykurilin_ joined #openstack-rally
21:41 penguinRaider joined #openstack-rally
22:39 openstackgerrit joined #openstack-rally
22:46 openstackgerrit Yan Ning proposed a change to stackforge/rally: try proper work flow  https://review.openstack.org/112684
23:23 openstackgerrit A change was merged to stackforge/rally: Split user and admin cleanup to 2 separate classes  https://review.openstack.org/100590
23:34 tzabal left #openstack-rally
23:55 aswadr joined #openstack-rally

| Channels | #openstack-rally index | Today | | Search | Google Search | Plain-Text | summary