Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:01 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Removed admin_user from benchmark.runners.base  https://review.openstack.org/120278
00:03 RainbowBastion boris-42 What should I change this line to, since admin no longer exists? https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/base.py#L179
00:10 aswadr joined #openstack-rally
00:16 openstackgerrit Walter A. Boring IV (hemna) proposed a change to stackforge/rally: WIP: Add a cinder volume attachment scenario  https://review.openstack.org/120283
00:19 boris-42 RainbowBastion do nothing
00:19 boris-42 RainbowBastion it exist
00:19 boris-42 RainbowBastion who said that it doesn't exist?
00:29 dmorita joined #openstack-rally
00:29 RainbowBastion boris-42 I mean that since admin/endpoint will be removed in this coming patch, what should I do about that line.
00:30 RainbowBastion boris-42 *?
00:32 boris-42 RainbowBastion it won't be removed...
00:32 boris-42 RainbowBastion it won't be removed from context...
00:32 boris-42 RainbowBastion here we are preparing context
00:32 boris-42 RainbowBastion https://github.com/stackforge/rally/blob/master/rally/benchmark/engine.py#L188
00:33 boris-42 RainbowBastion here we are passing https://github.com/stackforge/rally/blob/master/rally/benchmark/runners/base.py#L214 that context
00:34 RainbowBastion boris-42 ok, I see.  Then the context isn't getting passed properly in test_base? https://jenkins01.openstack.org/job/gate-rally-python26/1539/console
00:34 boris-42 RainbowBastion and if you take carefully here https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/base.py#L179
00:37 RainbowBastion boris-42 yep, just confirmed it isn't, I'll go add those fields into the context object in test_run in base.py
00:37 boris-42 RainbowBastion ya that should help
00:39 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Removed admin_user from benchmark.runners.base  https://review.openstack.org/120278
00:43 boris-42 RainbowBastion ^ okthat should work now
00:44 RainbowBastion boris-42 awesome
00:47 boris-42 RainbowBastion at least it passed py26
00:47 boris-42 RainbowBastion so if no pep8 errors
00:47 boris-42 RainbowBastion everything will be OK
00:49 boris-42 RainbowBastion wanna one more micro patch ?
00:54 boris-42 RainbowBastion so I am going to sleep but task is quite simple
00:54 boris-42 RainbowBastion move this method https://github.com/stackforge/rally/blob/master/rally/benchmark/scenarios/base.py#L174-L187
00:55 boris-42 RainbowBastion here https://github.com/stackforge/rally/blob/master/rally/benchmark/types.py#L36
00:55 boris-42 RainbowBastion and fix import in benchmark.runner.base and tests
00:56 RainbowBastion boris-42 will do
00:57 boris-42 RainbowBastion just in separated patch
00:57 boris-42 yingjun hi
00:57 yingjun boris-42, hi
00:57 boris-42 yingjun how are you?
00:57 yingjun boris-42, good, thanks
00:58 boris-42 yingjun btw are you going to be on Paris summit?
00:58 yingjun boris-42, no
00:58 boris-42 yingjun =(
00:58 yingjun boris-42, :((
00:59 boris-42 yingjun issues with visa?
01:00 yingjun boris-42, nope, no plan for the Paris summit from my boss..
01:01 boris-42 yingjun ah sad =(
01:01 boris-42 yingjun hope to see you one some other summit/meetup
01:01 boris-42 yingjun so you miss some meetings, so probably you don't know that we have CLI cleanup week
01:01 yingjun boris-42, me too, maybe in the next summit.
01:01 yaguang joined #openstack-rally
01:02 boris-42 yingjun and we are making google doc
01:02 boris-42 yingjun https://docs.google.com/a/mirantis.com/document/d/1iZa9V8C3Z09JTC3x2MGpBq-FEszwRmVCYoxCbN1zHgo/edit#heading=h.e8w122cqt4oh
01:02 boris-42 yingjun with ideas
01:02 boris-42 yingjun so if you have some ideas or found some bugs with CLI
01:02 boris-42 yingjun you can put there
01:02 boris-42 yingjun or even just make a patch that fixes
01:03 boris-42 yingjun as well seems like this is good candidate for merging https://review.openstack.org/#/c/120278/
01:03 yingjun boris-42, ok
01:03 boris-42 yingjun we need  just to wait for gates a bit
01:04 boris-42 yingjun so sorry but I need to sleep now=) it's a bit too late here 5 a.m. no jokes=)
01:04 yingjun ok, night
01:04 boris-42 yingjun see you later
01:20 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
01:30 jjmb joined #openstack-rally
01:31 jjmb1 joined #openstack-rally
01:32 jjmb2 joined #openstack-rally
01:42 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Store sla result to db  https://review.openstack.org/118243
01:56 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
02:08 openstackgerrit A change was merged to stackforge/rally: Removed admin_user from benchmark.runners.base  https://review.openstack.org/120278
02:26 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
02:35 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
02:42 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
02:45 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
02:47 Poornima joined #openstack-rally
03:39 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
03:41 openstackgerrit Oleh Anufriiev proposed a change to stackforge/rally: Autoadd user stories,samples and feature requests to sphinx  https://review.openstack.org/113454
04:40 yingjun joined #openstack-rally
05:05 nkhare joined #openstack-rally
05:16 Poornima joined #openstack-rally
05:45 k4n0 joined #openstack-rally
06:50 stannie joined #openstack-rally
07:02 mmalam joined #openstack-rally
07:21 k4n0 joined #openstack-rally
08:09 rediskin joined #openstack-rally
08:54 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job Binary internals cleanup  https://review.openstack.org/120083
08:56 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Job Binary internals cleanup  https://review.openstack.org/120083
09:12 mmalam|2 joined #openstack-rally
09:12 mmalam|2 Why I am getting this error, any idea:
09:13 mmalam|2 Command failed, please check log for more info 2014-09-10 13:22:59.920 1993 CRITICAL rally [-] Conflict: Conflict occurred attempting to store project - Duplicate Entry (HTTP 409) (HTTP 409) 2014-09-10 13:22:59.920 1993 TRACE rally Traceback (most recent call last):
09:15 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Collect time duration for Context setup/cleanup  https://review.openstack.org/119606
09:16 mmalam|2 boris-42: Hi
09:18 mmalam|2 Any help on the following error: http://paste.openstack.org/show/109379/
09:35 boris-42 mmalam|2 you are using old novaclinet
09:35 boris-42 nikhil_k you need to install fresh one
09:35 boris-42 nikhil_k sorry
09:35 boris-42 mmalam you need to install fresh one
09:36 mmalam nova client link?
09:36 boris-42 mmalam ?
09:36 boris-42 mmalam https://github.com/openstack/python-novaclient
09:36 mmalam but boris, isnt some network issue
09:37 boris-42 mmalam nope it is not
09:40 boris-42 mmalam I need this patch https://review.openstack.org/#/c/119776/
09:40 boris-42 mmalam in openstack/requirments
09:41 boris-42 mmalam after that will be merged you won't' have issues with novaclient
09:41 mmalam you mean, i just cherry-pick the patch
09:41 mmalam and thats it
09:41 boris-42 mmalam believe update novaclient
09:42 boris-42 mmalam if you don't believe you can debug
09:43 mmalam Ok BOSS
09:43 mmalam let me try it.
09:45 boris-42 mmalam you should have python-novaclinet with version >=2.18
09:45 boris-42 mmalam currently I believe version is 2.17
09:45 mmalam ok
09:47 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
09:47 andreykurilin joined #openstack-rally
09:50 yingjun joined #openstack-rally
10:02 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Fix disassociation of nova-networks from tenants  https://review.openstack.org/120026
10:12 boris-42 nkhare your bug is https://bugs.launchpad.net/rally/+bug/1367677
10:12 boris-42 nkhare is related to novaclient =(
10:13 nkhare is it ?
10:13 nkhare boris-42, is it ?
10:13 boris-42 nkhare yep  it is
10:13 tzabal joined #openstack-rally
10:13 boris-42 nkhare you need fresh one
10:13 boris-42 nkhare rally doesn't work with 2.17
10:18 nkhare boris-42, thanks. upgrading to latest novaclient solved the issue.
10:18 nkhare boris-42, any related bz by which I can why this happened
10:22 k4n0 boris-42, ping
10:23 msdubov k4n0 Can I help you?
10:23 k4n0 msdubov, hey
10:24 k4n0 msdubov, I was looking at andreykurilin comment on https://review.openstack.org/#/c/120052/
10:25 k4n0 msdubov, I will add "rally-manage tempest set-config" in a new review instead of this one, and will fix current review as per 1st line of andreykurilin comment
10:25 msdubov k4n0 Yep I also think that we shouldn't rewrite the autogenerated file
10:25 msdubov k4n0 It's bad practice
10:25 k4n0 msdubov, ok
10:29 msdubov tuvenen, Are you here?
10:32 tuvenen msdubov, yep
10:33 msdubov tuvenen, Would you like to make a contribution to Rally? We have actually several very simple tasks to start with
10:33 penguinRaider joined #openstack-rally
10:35 tuvenen msdubov, it could be interesting :). The main problem is that I don't have time right now since I'm working on two big issues. Can you point me the tasks to have a look at?
10:36 msdubov tuvenen Thanks, sure. E.g. this one: https://trello.com/c/khALzeni/224-easy-move-the-preparation-of-the-cli-output-for-rally-info-in-a-separate-utils-module
10:37 msdubov tuvenen It is about the "rally info find <query>" command which can be used from CLI to get info about different benchmark scenarios by their name etc.
10:37 msdubov tuvenen The task is basically just to move a couple of code lines to a separate method to make them reusable
10:37 msdubov tuvenen So pretty simple :)
10:39 msdubov tuvenen To be more general, there is a Trello board about Rally: https://trello.com/b/DoD8aeZy/rally
10:39 msdubov tuvenen It has a "Pool of tasks" section - so you're welcome to take any of these
10:42 tuvenen msdubov, thanks for the link. I'm connected to trello and I will have a look for sure. But as said previously I'm a quite busy right now...
10:42 msdubov tuvenen Okay! Looking forward to collaborating with you.
10:50 yingjun joined #openstack-rally
10:54 boris-42 k4n0 pong
11:17 rook-out joined #openstack-rally
11:26 Poornima joined #openstack-rally
11:37 ekarlso- joined #openstack-rally
11:50 mmalam|2 joined #openstack-rally
12:06 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Allow user specified tempest config file  https://review.openstack.org/120052
12:26 cdent joined #openstack-rally
12:34 boris-42 cdent so actually it will be nice to fix 2 broken ceilometer benchmarks
12:34 boris-42 cdent cause I am not a big specialist in ceilometer=)
12:36 msdubov tzabal Hi, around?
12:36 tzabal msdubov hi
12:37 mwagner_lap joined #openstack-rally
12:38 msdubov tzabal How are you doing?
12:38 tzabal msdubov good thanks
12:38 tzabal msdubov you?
12:38 msdubov tzabal I just wanted to ask you whether you could take this small task: https://trello.com/c/khALzeni/224-easy-move-the-preparation-of-the-cli-output-for-rally-info-in-a-separate-method
12:38 msdubov tzabal If you aren't buzy
12:38 msdubov tzabal Me too, thanks :)
12:39 tzabal msdubov sure
12:39 msdubov tzabal Great!
12:41 mwagner_lap joined #openstack-rally
12:52 cdent i'll see what I can do boris-42, if it is something clear I might be able to figure it out
12:52 boris-42 cdent https://bugs.launchpad.net/rally/+bug/1354683
12:52 boris-42 cdent this bug
12:57 openstackgerrit Mikhail Dubov proposed a change to stackforge/rally: Fix syntax errors in JSON doc samples  https://review.openstack.org/116217
13:30 openstackgerrit Rohan Kanade proposed a change to stackforge/rally: Allow user specified tempest config file  https://review.openstack.org/120052
13:34 stannie joined #openstack-rally
13:50 jjmb joined #openstack-rally
13:51 jjmb1 joined #openstack-rally
13:52 openstackgerrit Tzanetos Balitsaris proposed a change to stackforge/rally: Minor modification of the `rally info find` command  https://review.openstack.org/120416
14:08 jjmb joined #openstack-rally
14:10 openstackgerrit Nikita Konovalov proposed a change to stackforge/rally: Sahara Cluster scaling benchmark  https://review.openstack.org/120430
14:20 openstackgerrit A change was merged to stackforge/rally: Fix syntax errors in JSON doc samples  https://review.openstack.org/116217
14:34 openstackgerrit Li Yingjun proposed a change to stackforge/rally: Add task file verify command  https://review.openstack.org/120303
14:51 yingjun joined #openstack-rally
14:51 openstackgerrit A change was merged to stackforge/rally: Sahara Job Binary internals cleanup  https://review.openstack.org/120083
15:40 andreykurilin joined #openstack-rally
16:08 openstackgerrit A change was merged to stackforge/rally: Fix disassociation of nova-networks from tenants  https://review.openstack.org/120026
16:25 marcoemorais joined #openstack-rally
16:27 cdent_ joined #openstack-rally
16:36 marcoemorais joined #openstack-rally
16:36 marcoemorais joined #openstack-rally
16:45 openstackgerrit A change was merged to stackforge/rally: Allow user specified tempest config file  https://review.openstack.org/120052
16:55 marcoemorais joined #openstack-rally
17:31 marcoemorais joined #openstack-rally
17:33 marcoemorais joined #openstack-rally
17:33 mwagner_lap joined #openstack-rally
17:33 marcoemorais joined #openstack-rally
17:46 openstackgerrit Sergey Skripnick proposed a change to stackforge/rally: Cleanup refactoring(WIP)  https://review.openstack.org/116269
17:51 RainbowBastion joined #openstack-rally
18:14 cdent boris-42: you still around?
18:14 cdent I think the problem with the broken ceilometer scenarios may be a ceilometerclient bug, but I need some confirmation that it is not just my environment
18:15 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
18:35 stannie joined #openstack-rally
18:38 tzabal left #openstack-rally
18:46 boris-42 cdent me around
18:46 boris-42 cdent so these benchmarks are run in gates
18:46 boris-42 cdent on every patch
18:47 cdent boris-42: https://bugs.launchpad.net/python-ceilometerclient/+bug/1367867
18:48 cdent that was created after getting confirmation from some other ceilo folk. So the problem isn't in the rally client, it is doing the right thing. There's a bug either in ceilometerclient or in the server, and we're currently debating which
18:49 boris-42 cdent just for you info
18:49 boris-42 cdent you can open any patch
18:49 boris-42 cdent https://review.openstack.org/#/c/120416/
18:49 boris-42 cdent and click on gate-rally-dsvm-rally
18:50 boris-42 cdent in "Becnhamrking logs"
18:50 boris-42 cdent http://logs.openstack.org/16/120416/1/check/gate-rally-dsvm-rally/ccee5b3/console.html
18:50 cdent that's very cool
18:50 boris-42 cdent you can find erors
18:51 boris-42 cdent by searching for benchmark/sceanrio
18:51 boris-42 cdent as well you have ceilometer logs and so on
18:51 boris-42 cdent we are running this task https://github.com/stackforge/rally/blob/master/rally-scenarios/rally.yaml
18:51 boris-42 cdent on every patch in rally
18:52 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
18:57 boris-42 cdent btw this job can be add someday to ceilometer
18:57 boris-42 cdent if it will be reasonable
18:57 cdent yeah we hope to do that at some point
19:07 boris-42 cdent it's actually very simple oepration
19:07 * boris-42 cdent https://rally.readthedocs.org/en/latest/rally_gatejob.html
19:07 cdent yeah, but first we have to get the client working :)
19:07 cdent yeah, I've seen that
19:07 boris-42 cdent did you run  at least something successfully?)
19:08 boris-42 cdent cause something should work=)
19:08 cdent oh yeah, I was able to make the get stuff work just fine
19:08 cdent but in order to really test reads I need to be able to automate writes
19:09 boris-42 cdent so you have issues with these 2 benchmarks?
19:09 boris-42 cdent actually when we added them they worked
19:10 cdent can you pinpoint a time when they broke?
19:10 boris-42 cdent nope
19:10 boris-42 cdent I can point just when we added them
19:10 cdent as that will probably help us fix the problem in the client or server
19:10 boris-42 cdent heh
19:10 cdent presumably some window of time though: they worked at time X, and then somebody reported they were broken at time Y
19:11 boris-42 cdent I'll try to do my best
19:11 cdent thanks
19:12 boris-42 cdent so we saw that they don't work ~ 1 Aug
19:12 boris-42 cdent maybe few days before
19:13 cdent and they were created and working around when?
19:13 boris-42 cdent looking...
19:15 boris-42 cdent so lemme check
19:15 boris-42 cdent end of May
19:15 cdent excellent, thank you, I suspect that will be quite helpful
19:16 boris-42 cdent this is result
19:16 boris-42 cdent http://logs.openstack.org/30/92030/15/check/check-rally-dsvm-rally/98e4bed/rally-plot/results.html.gz
19:16 boris-42 cdent when we have all Ceilometer benchmarks
19:16 boris-42 cdent so you can try to get the same results in the middle
19:16 boris-42 cdent between Jun - Aug
19:16 boris-42 e.g. 1 Jul
19:17 boris-42 cdent we didn't have at that moment SLA checks
19:17 boris-42 cdent so our job was not -1 it always +1 =)
19:18 boris-42 so we didn't get the exactly moment
19:18 * cdent nods
19:19 boris-42 cdent Jun 20 everythign works
19:19 boris-42 so between Jun 20 - 1 Aug
19:20 boris-42 Jul 9 - 1 Aug
19:22 boris-42 Jul 25 - 1 Aug
19:23 boris-42 oh no sorry
19:23 boris-42 it was Jul 9
19:25 cdent so up to Jul 9 it was working, and then after that, no good?
19:29 boris-42 cdent so nope
19:29 boris-42 cdent I found precise numbers
19:33 boris-42 cdent so I found
19:34 boris-42 cdent betwenn 6-7 Aug
19:35 boris-42 cdent so 7 Aug
19:36 boris-42 cdent between 11 a.m and 1 p.m. ^_^
19:37 cdent wow
19:37 cdent that's very precise!
20:09 rook-out joined #openstack-rally
20:17 RainbowBastion joined #openstack-rally
20:24 RainbowBastion boris-42 Moving preprocess over is done
20:26 marcoemorais joined #openstack-rally
20:28 marcoemorais joined #openstack-rally
20:30 marcoemorais joined #openstack-rally
20:41 RainbowBastion https://review.openstack.org/#/c/120296/ anyone have time to review a tiny patch?
20:43 boris-42 RainbowBastion unit tests required
20:44 RainbowBastion boris-42 Do I need to make a test_types.py?  Or just add the preprocess test to test_base.py?
20:45 boris-42 RainbowBastion it already exists
20:45 boris-42 RainbowBastion you need to put https://github.com/stackforge/rally/blob/master/tests/benchmark/test_types.py
20:45 boris-42 RainbowBastion to put there ^ tests for this method
20:45 boris-42 RainbowBastion they were missing before.. =(
20:47 RainbowBastion boris-42 ah okay, I'll start working on it now
20:48 boris-42 RainbowBastion great
21:04 marcoemorais joined #openstack-rally
21:06 marcoemorais joined #openstack-rally
21:08 marcoemorais joined #openstack-rally
21:14 andreykurilin joined #openstack-rally
21:22 asalkeld joined #openstack-rally
21:40 oanufriev joined #openstack-rally
21:46 openstackgerrit A change was merged to stackforge/rally: Minor modification of the `rally info find` command  https://review.openstack.org/120416
22:05 rediskin left #openstack-rally
22:20 marcoemorais joined #openstack-rally
22:21 marcoemorais joined #openstack-rally
22:30 openstackgerrit Walter A. Boring IV (hemna) proposed a change to stackforge/rally: WIP: Add a cinder volume attachment scenario  https://review.openstack.org/120283
22:37 openstackgerrit Kevin Tsang proposed a change to stackforge/rally: Moved preprocess function from base to types  https://review.openstack.org/120296
23:05 RainbowBastion boris-42 Apart from the pep8 errors, does this look ok? ^
23:06 boris-42 RainbowBastion nope
23:07 boris-42 RainbowBastion types.preprocess() is not mock object
23:07 boris-42 RainbowBastion so there won't be method assert_called_once_with
23:07 RainbowBastion boris-42 ok
23:07 boris-42 RainbowBastion you need to mock https://review.openstack.org/#/c/120296/6/rally/benchmark/types.py
23:07 boris-42 RainbowBastion osclients
23:08 boris-42 RainbowBastion and probably   base.Scenario.meta
23:08 boris-42 RainbowBastion put proper values as a results of calls of those methods
23:09 boris-42 RainbowBastion and check that they are properly called with proper arugments
23:09 RainbowBastion boris-42 Would you rather I use the import mock or should I use the "with mock.patch" syntax?
23:09 boris-42 RainbowBastion and that returned value is proper
23:09 boris-42 RainbowBastion you should use as in the rest of code..
23:09 boris-42 RainbowBastion @mock.patch()
23:10 openstackgerrit OpenStack Proposal Bot proposed a change to stackforge/rally: Updated from global requirements  https://review.openstack.org/120615
23:29 yingjun joined #openstack-rally
23:48 harlowja_ joined #openstack-rally
23: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
23:59 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