Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2014-09-11

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

All times shown according to UTC.

Time Nick Message
00:04 yingjun joined #openstack-rally
00:24 dmorita joined #openstack-rally
00:28 harlowja joined #openstack-rally
00:49 yaguang joined #openstack-rally
01:04 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
01:55 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
02:37 yingjun joined #openstack-rally
03:20 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
03:29 Poornima joined #openstack-rally
03:59 yingjun joined #openstack-rally
04:15 nkhare joined #openstack-rally
04:54 jaypipes joined #openstack-rally
04:57 Poornima joined #openstack-rally
06:32 rediskin joined #openstack-rally
06:34 k4n0 joined #openstack-rally
06:41 tuvenen joined #openstack-rally
06:42 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
07:27 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
07:56 andreykurilin joined #openstack-rally
08:05 stannie joined #openstack-rally
08:09 asalkeld joined #openstack-rally
08:24 yingjun joined #openstack-rally
08:27 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
08:38 Ch00k joined #openstack-rally
08:43 yingjun joined #openstack-rally
08:44 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Adds unit tests for rally.cmd.cliutils  https://review.openstack.org/116958
08:59 nkhare joined #openstack-rally
09:08 andreykurilin joined #openstack-rally
09:14 openstackgerrit A change was merged to stackforge/rally: Updated from global requirements  https://review.openstack.org/120615
09:15 asalkeld joined #openstack-rally
10:01 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Collect time duration for Context setup/cleanup  https://review.openstack.org/119606
10:55 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Cleanup refactoring  https://review.openstack.org/116269
11:05 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job sequence benchmark added  https://review.openstack.org/120071
11:05 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Cluster and EDP contexts  https://review.openstack.org/115962
11:05 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job Execution benchmark  https://review.openstack.org/116881
11:05 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Adding optional parameters to Sahara cluster  https://review.openstack.org/119778
11:05 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Cluster scaling benchmark  https://review.openstack.org/120430
11:15 yingjun joined #openstack-rally
11:15 penguinRaider joined #openstack-rally
11:15 nkhare joined #openstack-rally
12:02 RaginBajin joined #openstack-rally
12:08 nkhare joined #openstack-rally
12:12 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Fix cli job(WIP)  https://review.openstack.org/120766
12:15 mwagner_lap joined #openstack-rally
12:18 boris-42 ekarlso- around?
12:22 aswadr joined #openstack-rally
12:31 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job sequence benchmark added  https://review.openstack.org/120071
12:35 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Cluster and EDP contexts  https://review.openstack.org/115962
12:35 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job Execution benchmark  https://review.openstack.org/116881
12:35 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Adding optional parameters to Sahara cluster  https://review.openstack.org/119778
12:35 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job sequence benchmark added  https://review.openstack.org/120071
12:57 tzabal joined #openstack-rally
13:13 rlrossit joined #openstack-rally
13:38 rlrossit left #openstack-rally
13:44 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: Rally cli functional testing  https://review.openstack.org/118714
14:08 andreykurilin_ joined #openstack-rally
14:32 ekarlso- boris-42: ?
14:32 ekarlso- what's up :p
14:35 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: User_stories and feature requests appended to sphinx  https://review.openstack.org/113454
14:35 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: Doc samples to rst  https://review.openstack.org/120794
14:39 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: Rally cli functional testing  https://review.openstack.org/118714
14:42 Ch00k_ joined #openstack-rally
14:49 openstackgerrit Walter A. Boring IV (hemna) proposed a change to stackforge/rally: WIP: Add a cinder volume attachment scenario  https://review.openstack.org/120283
14:55 cdent joined #openstack-rally
15:06 boris-42 ekarlso- hi there
15:07 boris-42 ekarlso- could you make feature request?)
15:07 ekarlso- what for again :p
15:08 boris-42 ekarlso- for "chain-of-benchmarks"
15:12 tuvenen_ joined #openstack-rally
15:48 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Fix CIDR overlapping in _generate_subnet_cidr  https://review.openstack.org/119787
15:57 openstackgerrit A change was merged to stackforge/rally: Adds unit tests for rally.cmd.cliutils  https://review.openstack.org/116958
15:57 Ch00k joined #openstack-rally
16:17 rook boris-42 for the cleanup phase rally runs at the end, how do i ensure things that I have added get removed (I have added code for routers -- set gateway, add subnet port).. However the cleanup phase doesn't seem to get rid of the routers.
16:17 yingjun joined #openstack-rally
16:18 rook instead of me searching through the code, I figure I would ask
16:20 marcoemorais joined #openstack-rally
16:28 RainbowBastion joined #openstack-rally
16:29 RainbowBastion joined #openstack-rally
16:31 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
16:33 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
16:41 openstackgerrit Walter A. Boring IV (hemna) proposed a change to stackforge/rally: Add a cinder volume attachment scenario  https://review.openstack.org/120283
16:41 boris-42 rook hm
16:41 boris-42 rook so rediskin is working on refactoring cleanup stuff
16:41 rook hey
16:41 boris-42 rook and adding mechanism that actually does dictater cleanup (so we will able as well to check that everything is cleanup up)
16:41 boris-42 rook for now we have some bugs in that mechanism and I know that sux=)
16:41 boris-42 rook we even have feature request for that https://github.com/stackforge/rally/blob/master/doc/feature_request/production_ready_cleanup.rst
16:42 rook okay..
16:42 rook boris-42 I am going to see about pushing my added functions for neutron
16:42 boris-42 rook so lemme point you
16:43 boris-42 rook so cleanup is here
16:43 boris-42 rook https://github.com/stackforge/rally/tree/master/rally/benchmark/context/cleanup
16:43 rook boris-42 i have experimental code that does the following : https://gist.github.com/anonymous/dcb07386c09c4349dcab
16:43 rook I need to clean it up, and push it upstream for you guys to bark at it ;)
16:44 boris-42 rook oh nice
16:45 boris-42 rook so you can push to review
16:45 boris-42 rook and we can comment on early steps
16:46 boris-42 rook so what resource is not cleaned?
16:46 rook boris-42: yup - I need to pull and update my current repo, then integrate my changes.
16:46 rook boris-42 routers are not deleted
16:47 rook boris-42 but, i suspect it is because I am setting the gateway and not clearing the gateway
16:48 rook boris-42: and since the routers never get deleted, the networks I attach to the routers never get removed
16:48 rook chicken and egg
16:49 boris-42 rook https://github.com/stackforge/rally/blob/master/rally/benchmark/context/cleanup/utils.py#L127
16:49 boris-42 rook so probably you nee dot add something here=)
16:49 rook boom, exactly what i was looking for boris-42 =)
16:49 boris-42 rook great
16:49 boris-42 +)
16:50 rook thank you sir!
16:50 boris-42 rook I'll be back in few hrs =)
16:50 boris-42 rook see you later
16:50 rook roger, take care
17:21 openstackgerrit OpenStack Proposal Bot proposed a change to stackforge/rally: Updated from global requirements  https://review.openstack.org/120847
17:26 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
17:27 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
17:47 openstackgerrit joined #openstack-rally
17:53 kevtsang_ joined #openstack-rally
18:06 marcoemorais joined #openstack-rally
18:07 marcoemorais joined #openstack-rally
18:08 marcoemorais joined #openstack-rally
18:21 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
18:33 marcoemorais joined #openstack-rally
18:38 marcoemorais joined #openstack-rally
18:57 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
19:30 rook does tox run reliably for everyone?
19:37 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: Rally cli functional testing  https://review.openstack.org/118714
19:37 temujin joined #openstack-rally
19:40 openstackgerrit A change was merged to stackforge/rally: Updated from global requirements  https://review.openstack.org/120847
20:32 RainbowBastion joined #openstack-rally
20:37 andreykurilin joined #openstack-rally
20:37 marcoemorais joined #openstack-rally
20:37 oanufriev joined #openstack-rally
20:40 marcoemorais joined #openstack-rally
20:45 marcoemorais1 joined #openstack-rally
20:53 RainbowBastion boris-42 In addition to the meta and osclients.Clients methods, do I also have to mock up the preprocessor.transform method?
21:06 rook RainbowBastion ping , maybe you can help me -- trying to determine what tox is looking for, These scenarios don't have samples
21:06 rook MismatchError: set(['VMTasks.network_boot_runcommand']) != set([]): These scenarios don't have samples: set(['VMTasks.network_boot_runcommand'])
21:11 RainbowBastion rook hi
21:11 rook RainbowBastion: i think i got it ;)
21:11 rook need to add the tests
21:12 RainbowBastion rook Okay, good, I've been submitting so much stuff for review recently because I can't get tox to work on my local machine, lol
21:12 rook omg
21:12 rook i had the same issue
21:12 rook tox was blowing up for me
21:12 rook i had to do a fresh pull, merge my changes and re-try
21:12 rook i had to also clean the cache
21:12 rook huge pita
21:13 RainbowBastion rook I gave up on getting tox to work because I'm only at my job for a few more days before I head back to school
21:13 rook oh, well good luck at school! ;)
21:15 RainbowBastion rook thanks!  Do you know if I need to mock up the transform function in order to get the preprocess unit test working? https://review.openstack.org/#/c/120296/
21:16 RainbowBastion rook specifically in test_types.py
21:18 rook i am not the right person to ask about mock/test code... I am far from understanding it
21:21 RainbowBastion rook From what I understand, there are 3 phases:
21:21 RainbowBastion rook 1: Create the mock objects/set up what needs to be passed to the method being tested
21:22 RainbowBastion rook 2: call the method and make appropriate changes to the mock objects indicative of their final results
21:22 RainbowBastion rook 3: do asserts to make sure that everything got properly called and has the right state at the end of the method
21:23 rook yeah, it makes sense when speaking about it, now writing the code to support it, is a different story ;)
21:23 rook I just need to stare at it for a while.
21:23 rook which i do not have the time
21:23 rook so I just ctrl-c ctrl-v for the test code
21:23 rook :)
21:23 rook jk, but seriously
21:26 boris-42 RainbowBastion hey there
21:26 boris-42 RainbowBastion I'm back
21:26 RainbowBastion boris-42 hi
21:26 boris-42 rook around?
21:26 boris-42 rook tox said to you that you didn't' write https://github.com/stackforge/rally/tree/master/doc/samples/tasks/scenarios
21:26 boris-42 rook samples for your scenario
21:26 boris-42 rook around?
21:26 rook yup - i saw that... tox was just hating on me. I had to clear the cache
21:32 marcoemorais joined #openstack-rally
21:34 marcoemorais1 joined #openstack-rally
21:35 marcoemorais1 joined #openstack-rally
21:36 marcoemorais1 joined #openstack-rally
21:36 rediskin left #openstack-rally
21:37 marcoemorais1 joined #openstack-rally
21:37 marcoemorais joined #openstack-rally
21:38 marcoemorais joined #openstack-rally
21:40 RainbowBastion boris-42 could you leave a couple comments on how I can get the unit test for preprocess working? https://review.openstack.org/#/c/120296
21:44 asalkeld joined #openstack-rally
21:47 boris-42 RainbowBastion one second
21:48 rook boris-42 i am going to push this patch, but there isn't much for the test code.
21:48 rook i might need some assistance with that.
21:48 rook i need to roll for the day.
21:49 boris-42 rook-roll just push it on review
21:49 boris-42 rook-roll as is
21:49 boris-42 rook-roll and continue work on it
21:50 rook-roll rgr
21:52 openstackgerrit Joe Talerico proposed a change to stackforge/rally: Patch to have a VMTask scenario: Create a router Create a network & subnet Attach the network to the router Set external network as gateway Launch a guest Test for connectivity Run a command and exit.  https://review.openstack.org/120921
21:53 rook-roll :) thanks boris-42
21:53 rook-roll ttyl
22:01 boris-42 rook-roll thank you for contribution=)
22:09 mwagner_lap joined #openstack-rally
22:12 rook-roll np boris-42!
22:16 openstackgerrit Joe Talerico proposed a change to stackforge/rally: Patch to have a VMTask scenario: Create a router Create a network & subnet Attach the network to the router Set external network as gateway Launch a guest Test for connectivity Run a command and exit.  https://review.openstack.org/120921
22:19 openstackgerrit Joe Talerico proposed a change to stackforge/rally: Patch to have a VMTask scenario: Create a router Create a network & subnet Attach the network to the router Set external network as gateway Launch a guest Test for connectivity Run a command and exit.  https://review.openstack.org/120921
22:39 openstackgerrit_ joined #openstack-rally
22:42 RainbowBastion left #openstack-rally
22:42 RainbowBastion joined #openstack-rally
22:47 openstackgerrit A change was merged to stackforge/rally: Fix CIDR overlapping in _generate_subnet_cidr  https://review.openstack.org/119787
22:48 boris-42 RainbowBastion still commenting your patch..
22:55 RainbowBastion boris-42 thanks
23:13 boris-42 RainbowBastion heh that was hard
23:14 boris-42 RainbowBastion I left comment
23:14 boris-42 RainbowBastion I hope it's clear now
23:17 RainbowBastion boris-42 Cool, I see, I should have used JSON objects instead of mocks for a lot of the tests.
23:17 boris-42 RainbowBastion seems like you have issues with understanding how mocks stuff works
23:18 boris-42 RainbowBastion I know they are quite tricky
23:18 boris-42 RainbowBastion I have same issues when I started =)
23:18 boris-42 RainbowBastion so the idea is that you are changing some fucntions/class with fake objects with specified behavrious
23:18 boris-42 RainbowBastion so when you are calling it you can specify what it returns for example
23:19 boris-42 RainbowBastion so you can isolate code that you are testing from rest of code
23:21 RainbowBastion boris-42 Do I also need to mock up the .transform function within preprocess?  Since every other external call was mocked up.
23:23 RainbowBastion boris-42 Oh, I see, you define the preprocessor class within mock_meta
23:23 boris-42 RainbowBastion mock.meta will return it
23:23 boris-42 RainbowBastion so nope
23:26 RainbowBastion boris-42 So @mock.patch doesn't take a local file path as its argument?  Is it there more for readability so you know what classes that the external calls are coming in from?
23:26 boris-42 RainbowBastion ?
23:26 boris-42 RainbowBastion not sure that I get
23:26 boris-42 RainbowBastion @mock.patch will replace some global variable inside specified module
23:26 boris-42 like you would like to mock something in rally.benchmarks.types
23:27 boris-42 so you need to specify "rally.benchmark.types" + this global object
23:27 boris-42 like "rally.benchmark.types.osclients"
23:27 boris-42 it will be osclients inside rally.benchmark.types
23:27 boris-42 instead of real osclients we will have there mock.MagicMock instace
23:31 RainbowBastion boris-42 ok, that makes sense now
23:38 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
23:52 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296

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