Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2014-12-26

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

All times shown according to UTC.

Time Nick Message
00:02 jjmb joined #openstack-rally
00:29 openstackgerrit li,chen proposed stackforge/rally: Use from six.moves.urllib.parse instead of urlparse  https://review.openstack.org/143914
01:04 jjmb joined #openstack-rally
01:04 jjmb joined #openstack-rally
01:26 yingjun joined #openstack-rally
01:48 klindgren__ joined #openstack-rally
01:50 jjmb joined #openstack-rally
01:53 openstackgerrit Li Yingjun proposed stackforge/rally: Move files to common lib(Part 3)  https://review.openstack.org/143938
02:31 yaguang joined #openstack-rally
02:36 jjmb joined #openstack-rally
02:38 coolsvap joined #openstack-rally
02:41 akuznetsova_ joined #openstack-rally
02:46 rvcehimanshu joined #openstack-rally
03:22 Poornima joined #openstack-rally
04:24 rdas joined #openstack-rally
04:58 rvcehimanshu joined #openstack-rally
05:22 Poornima joined #openstack-rally
05:41 neeti joined #openstack-rally
05:46 yfried|afk joined #openstack-rally
05:51 exploreshaifali joined #openstack-rally
05:56 boris-42 joined #openstack-rally
06:13 yingjun joined #openstack-rally
06:48 nkhare joined #openstack-rally
07:35 rvcehimanshu joined #openstack-rally
08:06 rvcehimanshu joined #openstack-rally
08:27 rvcehimanshu joined #openstack-rally
08:43 oanufriev joined #openstack-rally
09:06 andreykurilin joined #openstack-rally
09:06 amaretskiy joined #openstack-rally
09:06 rvasilets joined #openstack-rally
09:10 oanufriev joined #openstack-rally
09:19 yfried joined #openstack-rally
09:23 andreykurilin_ joined #openstack-rally
09:30 andreykurilin_ joined #openstack-rally
09:37 yingjun joined #openstack-rally
09:39 rvcehimanshu joined #openstack-rally
09:55 pboldin joined #openstack-rally
09:59 nmagnezi joined #openstack-rally
10:17 pcaruana joined #openstack-rally
10:19 openstackgerrit Merged stackforge/rally: Use from six.moves.urllib.parse instead of urlparse  https://review.openstack.org/143914
10:45 pboldin joined #openstack-rally
10:47 openstackgerrit Oleh Anufriiev proposed stackforge/rally: WIP!!!! flakes plugin for single quotes  https://review.openstack.org/144118
10:54 openstackgerrit Pavel Boldin proposed stackforge/rally: Add the flavor context  https://review.openstack.org/141670
10:54 openstackgerrit Pavel Boldin proposed stackforge/rally: Add the generic script-based benchmark for the VMs  https://review.openstack.org/141671
10:54 openstackgerrit Pavel Boldin proposed stackforge/rally: Add the context benchmark_image  https://review.openstack.org/138466
10:54 openstackgerrit Pavel Boldin proposed stackforge/rally: Tests for rally gates  https://review.openstack.org/143469
10:59 pboldin1 joined #openstack-rally
12:11 jjmb joined #openstack-rally
13:33 openstackgerrit li,chen proposed stackforge/rally: Use six.moves.map instead of itertools.imap  https://review.openstack.org/144143
13:39 openstackgerrit Nikita Konovalov proposed stackforge/rally: Neutron network context support  https://review.openstack.org/143671
13:50 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
13:51 pboldin joined #openstack-rally
13:52 jjmb joined #openstack-rally
14:03 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
14:17 pboldin joined #openstack-rally
15:06 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Report generation improvement  https://review.openstack.org/143866
15:28 jjmb joined #openstack-rally
15:33 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
15:51 msdubov_ joined #openstack-rally
16:14 jjmb joined #openstack-rally
16:15 pboldin please review https://review.openstack.org/#/c/141670
16:27 jjmb1 joined #openstack-rally
16:28 openstackgerrit Merged stackforge/rally: Use six.moves.map instead of itertools.imap  https://review.openstack.org/144143
16:47 exploreshaifali joined #openstack-rally
16:48 jjmb joined #openstack-rally
17:50 boris-42 paboldin: you should ping somebody lol
18:39 exploreshaifali joined #openstack-rally
19:14 tkammer joined #openstack-rally
19:29 openstackgerrit Abhishek Chanda proposed stackforge/rally: Move to uuidutils from oslo.utils  https://review.openstack.org/144175
19:48 dspano joined #openstack-rally
19:55 dspano Hello all. I was looking through the rally source, and it looks like you can't currently test private flavors in scenarios. Is that correct, or am I totally off-base?
19:55 boris-42 dspano: hi hi
19:56 boris-42 dspano: what do you mean by private flavors?
19:56 dspano Ones that aren't public.
19:57 boris-42 dspano: ah so they are tenant related
19:57 dspano Yup.
19:58 boris-42 dspano: so basically it's not hard to exnend rally to test it
19:58 dspano I see that when a task runs it creates a random tenant, but I can't see where one could assign a flavor to the tenant.
19:58 boris-42 dspano: so we have mechanism of "context"
19:59 boris-42 dspano: you can create it for private flavors
19:59 dspano Aaargh. I had a feeling.
19:59 boris-42 dspano: actually pboldin is already working on it https://review.openstack.org/#/c/141670/
20:00 boris-42 dspano: but he is covereing another case (like just creating public flavors)
20:00 boris-42 dspano: I think it makes sense to add review to that patch and ask him to add argument
20:00 boris-42 dspano: like "private"
20:01 boris-42 dspano: True/Falce if it is enabled it will create all flavors for each tenant
20:01 boris-42 dspano: the another way that will be supported soon is https://docs.google.com/a/mirantis.com/spreadsheets/d/16DXpfbqvlzMFaqaXAcJsBzzpowb_XpymaK2aFY2gA2g/edit#gid=0
20:01 boris-42 dspano: to wait feature "support-exisitng-users"
20:01 dspano boris-42: Taking a look now
20:04 dspano boris-42: Am I reading "task-verification-levels" right? Soon admin user won't be needed for tasks that don't require it?
20:04 boris-42 dspano: yep that is the idea
20:04 dspano boris-42: Awesome.
20:04 boris-42 dspano: actually i am working on it exactly now*
20:04 dspano boris-42: That is extremely helpful.
20:05 boris-42 dspano: yep I am working on it already 8 months=)
20:05 dspano boris-42: Lol. I know how that goes.
20:06 dspano I will most likely start contributing once I can wrap my head around things a little more.
20:06 boris-42 dspano: actually we have a trillions of tasks for that
20:06 boris-42 dspano: I mean simple tasks to start understanding code better
20:07 dspano Cool.
20:08 boris-42 dspano: so if u are interested I can help you to find something interesting
20:11 dspano boris-42: Sure.\
20:11 boris-42 dspano: so let's start from refactoring of ScenarioRunners
20:12 boris-42 dspano: take a look at this patch https://review.openstack.org/#/c/137502/
20:13 boris-42 dspano: I introduce such method https://review.openstack.org/#/c/137502/3/rally/benchmark/context/base.py
20:13 boris-42 dspano: def context(....)
20:13 boris-42 dspano: we need absolutelly the same thing but for scenarioRunners
20:13 boris-42 dspano: https://github.com/stackforge/rally/tree/master/rally/benchmark/runners
20:14 boris-42 dspano: so there are only few runners and we don't need to change so much here
20:22 dspano boris-42: Thank you.
20:23 boris-42 dspano: btw did you contribute to any other OpenStack project before?
20:27 dspano boris-42: Long-time user, first-time contributor.
20:28 boris-42 dspano: so it's a bit different to other github.com projects
20:28 boris-42 dspano: cause OpenStack has own CI and github.com is just mirror
20:29 boris-42 dspano: actual repo is here http://git.openstack.org/cgit/stackforge/rally
20:29 boris-42 dspano: this can help you a bit to start https://wiki.openstack.org/wiki/Rally/Develop#How_to_contribute
20:32 dspano boris-42: It looks like a lot less steps to go through than a commit to OpenStack.
20:33 boris-42 dspano:  ?)
20:33 dspano boris-42: I spoke too soon. I see. The workflow is pretty much the samne.
20:33 boris-42 dspano: worklow to what?
20:34 dspano Submitting a patch.
20:34 boris-42 dspano: it similiar to any other project in OpenStack*
20:34 dspano boris-42: That's why I said I spoke too soon. :)
20:34 boris-42 dspano: and it actually sux a bit=)
20:35 boris-42 dspano: to hard to contribute =(
20:35 boris-42 too*
20:35 dspano boris-42: This will be a lof fun. Thanks for the links.
20:35 boris-42 dspano: =)
20:36 boris-42 dspano: it'a actually hard only first time
20:36 boris-42 dspano: then it's super simple=)
20:36 dspano boris-42: Yeah, that's what I figured.
20:37 boris-42 dspano: actually we can make even simpler patch as first
20:37 boris-42 dspano: just to get this workflow done
20:38 boris-42 dspano: like move these lines https://github.com/stackforge/rally/blob/master/rally-jobs/rally.yaml#L667-L828
20:38 boris-42 dspano:  to this file https://github.com/stackforge/rally/blob/master/rally-jobs/rally-neutron.yaml
20:39 boris-42 dspano: these tasks are run on every commit in rally in our CI
20:39 boris-42 dspano: so rally-neutron.yaml is taking about 5-10 minutes less then rally.yaml (so we should make some kind of balanicng)
22:16 andreykurilin_ joined #openstack-rally
22:26 openstackgerrit Abhishek Chanda proposed stackforge/rally: Moved module "uuidutils" from oslo-incubator to oslo.utils  https://review.openstack.org/142722
22:38 openstackgerrit Dave Spano proposed stackforge/rally: rally-neutron.yaml is taking about 5-10 minutes less then rally.yaml to run, so some of the work was moved for load-balancing  https://review.openstack.org/144180
22:38 openstackgerrit Abhishek Chanda proposed stackforge/rally: Moved module "uuidutils" from oslo-incubator to oslo.utils  https://review.openstack.org/142722
22:39 dspano boris-42: Thanks for all the tips! I'm going to look into the change you suggested earlier a little later.
22:40 boris-42 dspano: heh something went wrong https://review.openstack.org/#/c/144180/
22:40 boris-42 dspano: did you see amount of files?
22:41 dspano boris-42: wonderful.
22:41 boris-42 dspano: I even don't know how to help you...
22:41 boris-42 dspano: let's try the simplest one approach
22:41 boris-42 dspano: could you please do the next set of commands
22:41 boris-42 git checkout master
22:41 boris-42 git remote update
22:41 boris-42 git pull
22:42 boris-42 git checkout -b some_new_branch_where_youll_do_all_changes
22:42 boris-42 # change files
22:42 boris-42 git add rally-jobs/rally.yaml rally-jobs/rally-neutron.yaml
22:42 boris-42 git commit
22:43 boris-42 dspano: that may help ^
22:43 boris-42 dspano: as well commit messages should be formated well
22:43 boris-42 first line (less than 80 symbols)
22:43 boris-42 empy line
22:44 boris-42 <body> (every line less than 80 symbols)
22:48 openstackgerrit Abhishek Chanda proposed stackforge/rally: Moved module "uuidutils" from oslo-incubator to oslo.utils  https://review.openstack.org/142722
22:51 pcaruana joined #openstack-rally
23:10 openstackgerrit Dave Spano proposed stackforge/rally: Move CinderVolumes tasks from rally.yaml to rally-neutron.yaml  https://review.openstack.org/144183
23:10 dspano boris-42: That's better.
23:11 boris-42 dspano: much better
23:12 boris-42 dspano: so as far as you know how to contribute=)
23:12 dspano boris-42: Lol.
23:12 boris-42 dspano: you can work on bigger patch that we disucss before
23:13 boris-42 dspano: btw if you keep in commit message line "Change-id"
23:13 boris-42 dspano: you will make new Patch sets (instead of new reviews)
23:13 boris-42 dspano: like now
23:13 dspano boris-42: I'll do that next time.
23:14 boris-42 dspano: ya it was just adivce=)
23:14 boris-42 dspano: in case if you didn't know that=)
23:14 dspano boris-42: Is there a way to get rid of the junk review?
23:14 dspano boris-42: I didn't.
23:15 boris-42 dspano: you already did
23:15 boris-42 dspano: just press button abandon
23:15 boris-42 dspano: btw there is a small nit in your patch
23:15 boris-42 dspano: extra spaces that should be removed..
23:15 boris-42 dspano: so u should just remove them
23:16 boris-42 run "git add ..."
23:16 boris-42 "git commit --amend"
23:16 boris-42 keep that line with change-id
23:16 boris-42 and run
23:16 boris-42 git review -R
23:16 boris-42 dspano: ^
23:20 dspano boris-42: The empty line at 569?
23:21 boris-42 dspano: you can leave empty line
23:21 boris-42 dspano: but you should remove spaces
23:22 boris-42 dspano: they are display as red space on review
23:23 dspano boris-42: Not sure why vi did that.
23:23 dspano boris-42: Sorry about that.
23:25 boris-42 dspano: cuase it doesn't have special plugin
23:25 boris-42 dspano: that cuts trailing spaces on save
23:27 boris-42 dspano: so just remove them and publish new patch set
23:29 openstackgerrit Dave Spano proposed stackforge/rally: Move CinderVolumes tasks from rally.yaml to rally-neutron.yaml  https://review.openstack.org/144183
23:30 boris-42 dspano: heh seems like it will be easier to install plugin ;)
23:30 boris-42 dspano: I had the same issues with spaces lol
23:30 boris-42 dspano: half of day spanding to remove all of them=)
23:30 boris-42 dspano:  https://review.openstack.org/#/c/144183/2/rally-jobs/rally-neutron.yaml now you have here in 568 line
23:31 dspano boris-42: I see.
23:32 boris-42 dspano: OK I need to sleep a bit
23:33 boris-42 dspano: I'll review your patch tomorrow
23:33 openstackgerrit Dave Spano proposed stackforge/rally: Move CinderVolumes tasks from rally.yaml to rally-neutron.yaml  https://review.openstack.org/144183
23:35 openstackgerrit Dave Spano proposed stackforge/rally: Move CinderVolumes tasks from rally.yaml to rally-neutron.yaml  https://review.openstack.org/144183
23:35 dspano boris-42: Have a good one. Thanks again.
23:36 boris-42 dspano: Ok nice you did it!
23:36 boris-42 dspano: +2 from me
23:36 boris-42 andreykurilin: ^ take a look pls as well
23:36 dspano Hahahahaha
23:37 boris-42 dspano: it's always hard to start=)
23:37 boris-42 dspano: I have issues with starting my blog, I am already trying to do that for an half of year=)
23:38 boris-42 dspano:  current progress is boris-42.me lol
23:39 dspano boris-42: Just glad to have the first one done. It's will only get easier from here.
23:39 dspano boris-42: Time for a beer.
23:39 boris-42 dspano: I am already drunk so no thanks=)
23:40 boris-42 dspano: yep the hardest thing is done=)
23:40 boris-42 dspano: see you later=)
23:40 dspano boris-42: See you later :)

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