Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2015-04-23

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

All times shown according to UTC.

Time Nick Message
00:32 yingjun joined #openstack-rally
00:34 openstackgerrit Boris Pavlovic proposed openstack/rally: [Feature request] Specify what benchmarks to execute in task  https://review.openstack.org/176460
00:55 echoingumesh joined #openstack-rally
02:02 davideagnello joined #openstack-rally
02:07 openstackgerrit Li Yingjun proposed openstack/rally: Set updated_at to now if it's None for task list  https://review.openstack.org/176583
02:39 yingjun joined #openstack-rally
03:45 yingjun joined #openstack-rally
03:53 yingjun joined #openstack-rally
04:07 davideagnello joined #openstack-rally
04:46 nkhare joined #openstack-rally
04:47 yingjun joined #openstack-rally
05:04 rdas joined #openstack-rally
05:21 stpierre joined #openstack-rally
05:53 kiran-r joined #openstack-rally
05:59 yingjun_ joined #openstack-rally
06:33 anshul joined #openstack-rally
07:00 neeti joined #openstack-rally
07:18 davideagnello joined #openstack-rally
07:26 yfried|afk joined #openstack-rally
07:32 psd joined #openstack-rally
07:33 pbandzi joined #openstack-rally
07:33 arxcruz joined #openstack-rally
07:37 karimb joined #openstack-rally
07:41 e0ne joined #openstack-rally
08:17 fhubik joined #openstack-rally
08:23 yingjun joined #openstack-rally
08:25 yfried|afk joined #openstack-rally
08:27 andreykurilin__ joined #openstack-rally
08:33 pradeep joined #openstack-rally
09:08 davideagnello joined #openstack-rally
09:13 e0ne joined #openstack-rally
09:15 xylan joined #openstack-rally
09:16 xylan hi, anyone here?
09:16 xylan I installed rally using docker through this doc: https://registry.hub.docker.com/u/rallyforge/rally/
09:17 xylan but i met a problem when i run 'rally-manage db recreate', see the error here: http://paste.openstack.org/show/205168/
09:19 andreykurilin__ xylan: hi!
09:20 xylan andreykurilin__: hi
09:20 msdubov_ joined #openstack-rally
09:20 xylan andreykurilin__: any suggestion?
09:21 andreykurilin__ xylan: first of all, can you file a bug at https://bugs.launchpad.net/rally/+bugs ?
09:21 xylan andreykurilin__: sure, i am just wondering if it's a bug, or, i do the wrong step..
09:22 andreykurilin__ xylan: looks like a bug:)
09:22 andreykurilin__ xylan: try launch "chmod 777 ~/rally_home"  and than go to docker image again and start `rally-manage db recreate`
09:22 xylan andreykurilin__: ok, give me a sec
09:24 xylan andreykurilin__: same error, :(
09:24 andreykurilin__ hm...
09:24 xylan andreykurilin__: rally_home, you mean my local host or in the container
09:24 andreykurilin__ xylan: in your local host
09:25 xylan andreykurilin__: yep, it's what i did
09:25 xylan andreykurilin__: maybe i should submit a bug for that
09:25 andreykurilin__ xylan: what command was used to launch a container?
09:26 xylan docker run -t -i -v ~/rally_home:/home/rally -v ~/rally_db:/var/lib/rally/database rallyforge/rally
09:26 xylan andreykurilin__: the same with the doc
09:26 xylan andreykurilin__: i pulled the image just now
09:27 andreykurilin__ ok
09:27 exploreshaifali joined #openstack-rally
09:29 andreykurilin__ xylan: it looks like you read not the latest doc :) latest revision doesn't include mounting of rally_db directory. poosible solutions: launch chmod for `~/rally_db` dir or launch a container by `docker run -t -i -v ~/rally_home:/home/rally rallyforge/rally`
09:30 xylan andreykurilin__: ooh, you mean, https://registry.hub.docker.com/u/rallyforge/rally/ is obsolete?
09:31 xylan andreykurilin__: you're right, http://rally.readthedocs.org/en/latest/install.html#rally-docker, the content incloude what you mentioned
09:32 andreykurilin__ xylan: hm...strange. when I opened https://registry.hub.docker.com/u/rallyforge/rally, it doesn't include mounting rally_db dir:(
09:33 openstackgerrit Arx Cruz proposed openstack/rally: Adding FunctionalMixin class  https://review.openstack.org/146838
09:33 frobware is there an alternative for not having hard coded values in the scenario benchmarks?
09:33 andreykurilin__ frobware: What values do you mean?
09:34 frobware andreykurilin__, number of iterations, number of tenants, etc
09:35 aix joined #openstack-rally
09:36 andreykurilin__ frobware: you can replace all values by {{ some_var_1 }}, {{ some_var_2 }} and than, launch scenario with `rally task start my.json --task-args '{"some_var_1": 2, "some_var_2": 3}'`
09:37 andreykurilin__ frobware: task scenario is a simple jinja template, so it supports a lot of features(variables, loops and etc)
09:38 xylan andreykurilin__: thanks very much! it works!
09:39 xylan and btw, frobware, you can refer to http://rally.readthedocs.org/en/latest/tutorial/step_4_task_templates.html
09:39 andreykurilin__ xylan: nice to hear that:)
09:39 xylan frobware: you're welcome
09:43 tosky joined #openstack-rally
09:44 anshul joined #openstack-rally
09:45 frobware andreykurilin, thanks will take a look
09:46 openstackgerrit Nikita Konovalov proposed openstack/rally: [Sahara] Fix the config descriptions  https://review.openstack.org/176321
10:02 openstackgerrit Nikita Konovalov proposed openstack/rally: [Sahara] Fix the config descriptions  https://review.openstack.org/176321
10:02 fhubik joined #openstack-rally
10:06 andreykurilin frobware: btw, jinja templates supports "if...else..." statements too. For example, we are using "if...else" for configuration network context at our lab:)
10:09 e0ne joined #openstack-rally
10:09 openstackgerrit Nikita Konovalov proposed openstack/rally: [Sahara] More debug logging  https://review.openstack.org/170491
10:13 yingjun joined #openstack-rally
10:14 frobware andreykurilin, do you have an example I can cargo cult? ;)
10:16 frobware andreykurilin, actually, I'm being lazy. Just started reading the docs and it look straightforward - for what I want initially. thanks.
10:17 openstackgerrit Nikita Konovalov proposed openstack/rally: [Sahara] More debug logging  https://review.openstack.org/170491
10:18 andreykurilin frobware: http://paste.openstack.org/show/205189/
10:18 frobware andreykurilin, thank you!
10:18 andreykurilin frobware: np
10:32 cdent joined #openstack-rally
10:43 nkhare joined #openstack-rally
10:56 davideagnello joined #openstack-rally
11:01 nkhare joined #openstack-rally
11:19 openstackgerrit Boris Pavlovic proposed openstack/rally: Adds Nova floating IPs bulk tests  https://review.openstack.org/168054
11:26 BaQs joined #openstack-rally
11:26 BaQs o/
11:27 BaQs hi all, can't find out how to tell rally to not check ssl certificates when curling APIs endpoints
11:27 BaQs would any of you know the magic option ? (if any :) )
11:40 xylan andreykurilin: ping
11:41 xylan boris-42: ping
11:41 xylan does anybody know the password for user 'rally' in the rally docker image?
11:42 xylan because i want to install some python package, but without the right permission
11:43 andreykurilin xylan: pong
11:44 xylan andreykurilin: hi, glad you're still here
11:44 andreykurilin xylan: rally user doesn't have password:(
11:44 andreykurilin xylan: what packages do you want to install?
11:44 xylan andreykurilin: python-mistralclient
11:45 andreykurilin ж(
11:45 andreykurilin ;(
11:45 xylan andreykurilin: actually, with the docker image, we can not install anything
11:45 xylan andreykurilin: under the rally user
11:45 xylan andreykurilin: when using sudo, the password will be asked
11:47 xylan andreykurilin: any suggestion? i have been working on this for a whole afternoon
11:48 xylan andreykurilin: or, maybe i should install rally from the scratch
11:48 xylan in a real vm or physical host
11:49 andreykurilin xylan: you can try to build your own container. you need to edit only two lines
11:50 xylan andreykurilin: do you have the dockerfile?
11:50 andreykurilin xylan: https://github.com/openstack/rally/blob/master/Dockerfile
11:50 andreykurilin xylan: you can add a password or remove rally user at all and launch container with root user
11:50 softCloud joined #openstack-rally
11:50 openstackgerrit Boris Pavlovic proposed openstack/rally: Fix coverage job to compare proper git diff  https://review.openstack.org/176703
11:51 boris-42 xylan: ping
11:51 xylan boris-42: hi
11:51 boris-42 xylan: why do you need it?
11:51 xylan boris-42: i'm testing mistral using rally
11:52 xylan boris-42: can we add python-mistralclient in the requirement file?
11:52 xylan boris-42: dependency
11:52 boris-42 xylan: as far as it will be in global requriments
11:52 redixin joined #openstack-rally
11:52 andreykurilin boris-42: we can add it in docker image
11:53 boris-42 redixin: xylan needs root access in docker image
11:53 boris-42 rvasilets: to install muranoclient
11:53 boris-42 redixin: to install mistralclient**
11:53 xylan boris-42: mistral
11:53 boris-42 rvasilets: sry
11:53 boris-42 xylan: yep yep typo
11:54 redixin docker run -u root -t -i rallyforge/rally
11:54 rvasilets o/
11:54 redixin you got root. what next?
11:54 xylan boris-42: actually, i need mistral client to be installed ready when i use rally image
11:54 rvasilets any questions?
11:54 andreykurilin redixin, boris-42: maybe we should provide root access in docker image?
11:54 redixin omg
11:54 redixin man docker
11:55 redixin andreykurilin, docker run -u root -t -i rallyforge/rally
11:55 xylan redixin: hi, are you sure?
11:55 xylan redixin: wait a min, i'll try
11:55 redixin xylan, you can install anything in that way but
11:55 andreykurilin redixin: why this command is not mentioned in out docker readme?:)
11:56 xylan root@openlab:/opt/kong/docker/rally# docker run -u root -t -i --name docker -v /opt/kong/docker/rally/rally_home:/home/rally rallyforge/rally
11:56 xylan root@1882eccfd66d:~#
11:56 redixin xylan, you should create image based on modifying image
11:56 xylan redixin: it works!
11:56 xylan redixin: thanks very much! and you boris-42 and andreykurilin
11:57 redixin xylan, please read docker manual. Things are turned a bit complex when you trying to modify image
11:57 redixin xylan, all modifications are lost after you exited that session
11:57 xylan redixin: thanks for your advise. what i need is just add a package.. then i can commit my modification
11:58 xylan redixin: and have my own version :)
11:58 xylan for now
11:58 redixin xylan, oh, ok. so you know how to commit :)
11:58 xylan redixin: absolutely
11:58 openstackgerrit Boris Pavlovic proposed openstack/rally: Fix coverage job to compare proper git diff and make it more strict  https://review.openstack.org/176703
11:58 openstackgerrit Boris Pavlovic proposed openstack/rally: [DO NOT MERGE] Testing coverage job  https://review.openstack.org/175558
11:58 xylan redixin: although i'm a beginner to docker...
11:58 redixin xylan, me too
11:59 xylan redixin: but, anyway, thanks!
11:59 boris-42 redixin: andreykurilin msdubov_ fix for coverage job ^
11:59 boris-42 that is quite important
11:59 redixin another fix O_O
12:00 aix joined #openstack-rally
12:00 aix joined #openstack-rally
12:00 xylan boris-42: hi, one more question, do you have a plan to add python-mistralclient to requirements.txt? what do you mean by 'global requirement'?
12:01 xylan boris-42: i saw designate, trove, etc. are already included
12:02 msdubov_ boris-42, Yep, saw that, thanks
12:03 andreykurilin xylan: we have mistralclient in optional-requirements https://github.com/openstack/rally/blob/master/optional-requirements.txt#L1
12:04 xylan andreykurilin: oh, i see
12:04 andreykurilin xylan: when mistralclient will be in https://github.com/openstack/requirements/blob/master/global-requirements.txt , rally team will have ability to add mistalclient to main requirements
12:04 andreykurilin xylan: without changes in global-requirements, we cann't add mistalclient to main requirements. this is OpenStack policy:(
12:05 xylan andreykurilin: got it. i'm reading https://github.com/openstack/requirements, to see how can i add mistralclient dependency
12:05 redixin probably we should do pip install optioinal-requirements for docker image
12:05 boris-42 redixin: makes sense
12:05 boris-42 xylan: you can fix our docker image
12:06 redixin just small change in rally's Dockerfile
12:06 boris-42 xylan: https://github.com/openstack/rally/blob/master/Dockerfile#L19
12:06 boris-42 xylan: yep somewhere there ^
12:07 xylan boris-42 redixin : ok, i'll try, btw, i was a contributor to Rally...
12:08 andreykurilin cool
12:08 xylan andreykurilin: :)
12:14 cdent joined #openstack-rally
12:31 BaQs hep
12:32 BaQs is there any option or config to allow rally to bench endpoints using self-signed certs plz ?
12:37 neeti_ joined #openstack-rally
12:45 davideagnello joined #openstack-rally
12:53 boris-42 BaQs: hi
12:56 BaQs hi boris-42
13:00 BaQs do you have any idea how to add "--insecure" switch to curl calls ?
13:02 openstackgerrit svasheka proposed openstack/rally: Add keystone benchmark scenarios for roles  https://review.openstack.org/165409
13:19 openstackgerrit svasheka proposed openstack/rally: Add keystone benchmark scenarios for roles  https://review.openstack.org/165409
13:22 boris-42 BaQs: u can pass ssl options
13:22 boris-42 BaQs: https://github.com/openstack/rally/blob/master/samples/deployments/existing.json#L11-L12
13:24 openstackgerrit svasheka proposed openstack/rally: Add keystone benchmark scenarios for roles  https://review.openstack.org/165409
13:27 e0ne joined #openstack-rally
13:30 aswadr joined #openstack-rally
13:36 kiran-r joined #openstack-rally
13:43 BaQs boris-42: nice! testing right now
13:45 boris-42 BaQs: =)
13:56 msdubov__ joined #openstack-rally
13:57 BaQs boris-42: works!
14:01 openstackgerrit Merged openstack/rally: [Feature request] Specify what benchmarks to execute in task  https://review.openstack.org/176460
14:04 nkhare joined #openstack-rally
14:05 boris-42 BaQs: great
14:35 openstackgerrit Boris Pavlovic proposed openstack/rally: Fix coverage job to compare proper git diff and make it more strict  https://review.openstack.org/176703
14:35 openstackgerrit Boris Pavlovic proposed openstack/rally: [DO NOT MERGE] Testing coverage job  https://review.openstack.org/175558
14:37 anshul joined #openstack-rally
14:38 openstackgerrit Merged openstack/rally: Fix creating user in keystone v3 wrapper  https://review.openstack.org/175504
14:59 openstackgerrit Lingxian Kong proposed openstack/rally: Install optional-requirement packages for Rally docker image  https://review.openstack.org/176794
15:00 pradeep joined #openstack-rally
15:07 openstackgerrit Lingxian Kong proposed openstack/rally: Install optional-requirement packages for Rally docker image  https://review.openstack.org/176794
15:09 csoukup joined #openstack-rally
15:10 BaQs boris-42: I may have a stupid question but... playing with scenarios, I find some interesting for me. how do I start it ? I have to pass full path of scenario json file ?
15:10 msdubov__ joined #openstack-rally
15:13 boris-42 BaQs: nope just class_name.method_name
15:13 boris-42 BaQs: or you mean path to file?
15:13 boris-42 BaQs: so it can be relative or absolute path
15:16 BaQs strange:
15:17 BaQs (rally)admin@rally:~$ rally task  start Authenticate.keystone
15:17 BaQs -------------------------------------------------------------------------------- Preparing input task
15:17 BaQs --------------------------------------------------------------------------------
15:17 BaQs [Errno 2] No such file or directory: u'Authenticate.keystone'
15:17 BaQs (rally)admin@rally:~$
15:22 boris-42 BaQs: you missed the point of rally
15:22 boris-42 BaQs: rally becnhamrk scenario is just one part of benchmark
15:22 boris-42 BaQs: take a look here https://rally.readthedocs.org/en/latest/tutorial/step_1_setting_up_env_and_running_benchmark_from_samples.html
15:23 boris-42 BaQs: and this https://rally.readthedocs.org/en/latest/tutorial/step_2_running_multple_benchmarks_in_a_single_task.html
15:24 boris-42 BaQs: you should put path to file with input task (not just scenario name)
15:24 boris-42 BaQs: like this https://github.com/openstack/rally/blob/master/samples/tasks/scenarios/authenticate/keystone.yaml
15:31 BaQs ok I see, checking all of that
15:40 davideagnello joined #openstack-rally
15:43 openstackgerrit Chris St. Pierre proposed openstack/rally: Add Nova scenario to boot multiple servers in one request  https://review.openstack.org/176858
16:14 exploreshaifali joined #openstack-rally
17:00 apal joined #openstack-rally
17:01 openstackgerrit Merged openstack/rally: Fix coverage job to compare proper git diff and make it more strict  https://review.openstack.org/176703
17:34 kiran-r joined #openstack-rally
17:35 boris-42 BaQs: any progress?)
17:35 boris-42 xylan: thanks for patch
17:53 e0ne joined #openstack-rally
18:06 davideagnello boris-42: when adding a plugin to Rally, can I separate my plugin into an additional base class which inherits from Rally's base.Scenario?  (similar to Rally built in scenarios for Nova, Neutron, etc..)
18:07 boris-42 davideagnello: yep
18:07 boris-42 davideagnello: we fixed recently small issue with relative imports
18:07 boris-42 https://github.com/openstack/rally/commit/6bb572e046e4090a3a979440dcafd3d9b9e712b1
18:08 boris-42 davideagnello: ^ so now it should work
18:08 davideagnello boris-42: my plugin was discoverable by rally when it was in a single file/class, now that I have separated it into base.scenario <-- CueScenario <-- CueClusters
18:08 davideagnello boris-42: ok will update my rally
18:08 boris-42 davideagnello: it will be discovered and you can split it in 2 files*
18:08 tosky_ joined #openstack-rally
18:08 davideagnello boris-42: ok great!
18:09 boris-42 https://www.irccloud.com/pastebin/Iiu8Dloo
18:09 boris-42 davideagnello: ^
18:09 davideagnello boris-42: when was this merged?
18:09 davideagnello boris-42: ok cool :)
18:10 boris-42 davideagnello: day ago=)
18:10 boris-42 davideagnello: I just discovered this bug recently =(
18:10 davideagnello boris-42: syncing now, thanks!
18:12 boris-42 davideagnello: btw
18:12 boris-42 davideagnello: you should make coverage job in cue as well
18:13 boris-42 davideagnello: it's perfect
18:13 boris-42 =)
18:13 davideagnello boris-42: sweet, does this get coverage on your target service through rally?
18:13 e0ne joined #openstack-rally
18:13 boris-42 davideagnello: nope it's not related to Rally
18:14 boris-42 davideagnello: it's just coverage unit test job
18:14 boris-42 davideagnello: it's put -1 if proposed change is not fully covered by unit tests http://logs.openstack.org/58/175558/17/check/rally-coverage/d389ba6/console.html#_2015-04-23_14_43_34_162
18:15 boris-42 davideagnello: and shows diff=)
18:15 davideagnello boris-42: ok cool, will take a look for our project. thanks :)
18:16 davideagnello boris-42: where do I find more info on how to incorporate it?
18:17 boris-42 davideagnello: take a look here https://github.com/openstack/rally/blob/master/tox.ini#L30-L31
18:17 boris-42 davideagnello: and this script
18:17 boris-42 davideagnello: https://github.com/openstack/rally/blob/master/tests/ci/cover.sh
18:21 davideagnello boris-42: thank you!
18:22 boris-42 davideagnello: and you should add this job in openttack project config
18:22 davideagnello boris-42: ok
18:22 boris-42 davideagnello: we are running it only on checks* but I think we will run it as well in gate pipeline
18:23 davideagnello boris-42: makes sense, we will be adding it as a CI gate too
18:27 boris-42 davideagnello: I calculated this job saves about 25 hrs of my life / release=)
18:30 davideagnello boris-42: lol that sounds like a good investment!
18:33 boris-42 davideagnello: yep automation is key of scaling work on project
18:33 boris-42 davideagnello: but pssss don't say that to nova team
18:34 boris-42 =)
18:34 boris-42 davideagnello: you can take a look at this thread: [openstack-dev][all][code quality] Voting coverage job (-1 if coverage get worse after patch)
18:34 davideagnello boris-42: hahaha sounds right to me
18:34 boris-42 davideagnello: =)
18:34 boris-42 davideagnello: and have fun=)
18:47 openstackgerrit Boris Pavlovic proposed openstack/rally: Finish support of benchmarking with existing users  https://review.openstack.org/168524
18:47 openstackgerrit Boris Pavlovic proposed openstack/rally: Fix rally show commands for existing users in deployment  https://review.openstack.org/168636
18:48 echoingumesh joined #openstack-rally
19:03 mwagner_lap joined #openstack-rally
19:36 davideagnello boris-42: with the built-in admin_client, how do I get the admin session?  I would have to supply my own auth token? (keystoneclient.auth.identity.v2.Password)
19:39 boris-42 davideagnello: admin_client is method
19:39 boris-42 davideagnello: that uses instance of rally.oslcients.Clients
19:40 boris-42 instance_of_client.<first_arg>(<second_arg>)
19:40 boris-42 davideagnello: ^
19:40 boris-42 why you can just use
19:40 boris-42 self.admin_clients("keystone")
19:40 boris-42 and use python kyestone client?
19:40 boris-42 davideagnello: btw what about extedning OSclients
19:41 boris-42 davideagnello: we talked about this before but ..
19:44 echoingumesh joined #openstack-rally
20:14 davideagnello boris-42: I will be looking at extending OSclients next, but right now trying to get a base working for the rest of my team to add scenario tests in
20:18 cdent joined #openstack-rally
20:25 echoingumesh joined #openstack-rally
20:27 e0ne joined #openstack-rally
20:28 andreykurilin__ joined #openstack-rally
20:38 openstackgerrit Chris St. Pierre proposed openstack/rally: Add Nova scenario to boot multiple servers in one request  https://review.openstack.org/176858
20:47 boris-42 davideagnello:
20:47 boris-42 davideagnello: so "self.admin_clients("keyston")" -will return inited python keystone client
20:47 boris-42 davideagnello: after that you can get from keytsotne client everything that you need
20:49 davideagnello boris-42: ok, getting a keystoneclient session, I would have to call self.admin_clients("keystone").session.Session(auth=myauth)?
20:49 boris-42 davideagnello: so not exprt with session
20:49 boris-42 davideagnello: try it
20:50 davideagnello boris-42: ok, was wondering if I could get the admin session directly since keystone was already instantiated that way
20:51 davideagnello boris-42: until yesterday I was using "auth = keystoneclient.auth.identity.v2.Password(...)  but it doesn't seem to work anymore
20:51 davideagnello boris-42: I now get: Failed to load module with plugins /home/vagrant/.rally/plugins/cue_scenarios/cue_utils.py: 'module' object has no attribute 'auth'
20:54 boris-42 hm
20:54 boris-42 davideagnello: paste the code
20:54 davideagnello boris-42: ok
20:54 davideagnello import keystoneclient
20:55 boris-42 davideagnello: hm
20:55 davideagnello https://gist.github.com/anonymous/f086968bf301f420ba76
20:55 davideagnello boris-42: that's what I had working until yesterday
20:55 boris-42 so you are doing something illegal
20:55 boris-42 davideagnello: ^
20:55 boris-42 davideagnello: it's not how rally was designed
20:55 boris-42 davideagnello: to do*
20:56 davideagnello boris-42: ok
20:57 davideagnello boris-42: I basically need my admin session to provide to our Cue client
20:57 boris-42 you are doing this in wrong way
20:57 boris-42 it shouldn't be in module
20:58 boris-42 davideagnello: will back latter
20:58 boris-42 later*
20:58 boris-42 davideagnello: and will help you
20:59 davideagnello boris-42: ok, thank you.  Appreciate it.  I would like to ideally make use of the framework in the proper way.  At the same time, I am trying to interface to a service running in Openstack that of course is currently not in Rally
21:02 davideagnello boris-42: my current patch for this is:  https://review.openstack.org/#/c/175524/
21:10 openstackgerrit Carlos L. Torres proposed openstack/rally: [report] Improve reports data and units  https://review.openstack.org/161037
21:13 openstackgerrit Carlos L. Torres proposed openstack/rally: [report] Improve reports data and units  https://review.openstack.org/161037
21:31 boris-42 davideagnello: reviewing
21:32 davideagnello boris-42: thank you, noticed an issue with atomic_action_timer, I had atomic.action_timer
21:33 boris-42 davideagnello: that code just doens't work
21:33 boris-42 davideagnello: if it works I don't know why=)
21:33 boris-42 davideagnello: I will put long descriiption
21:33 boris-42 davideagnello: in review
21:33 davideagnello boris-42: yes, got it working now, with atomic_action_timer
21:34 davideagnello boris-42: thank you
21:44 boris-42 davideagnello: so atomic actions are OK
21:44 boris-42 davideagnello: but your code is not thread safe and won't work
21:44 davideagnello boris-42: ok
21:44 boris-42 davideagnello: you can take a look at my comment and fix it quick
21:44 davideagnello boris-42: because of the way I am using the keystone client?
21:44 boris-42 davideagnello: mostly
21:45 davideagnello boris-42: ok thank you, appreciate your feedback
21:45 boris-42 davideagnello: you shouldn't have any open files sockets or something like that in global space
21:45 boris-42 davideagnello: that will lead unexpected hardly understandable and absolutelly uncatchable errors
21:45 boris-42 that can't be debuged=)
21:47 davideagnello boris-42: thanks, I started with this as a demonstration of using Cue with Rally.  Since we will be using Rally, it's time to harden the integration :)
21:49 boris-42 davideagnello: so let me try to find the way
21:49 boris-42 davideagnello: to make a proper client initialziation
21:49 boris-42 davideagnello: btw
21:51 boris-42 davideagnello: so I can help you a bit with setting up dsvm jobs
21:51 boris-42 davideagnello: in infra so you will run them on each patch
21:51 boris-42 davideagnello: I see you are not running them
21:52 boris-42 davideagnello: https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L5001-L5009
21:52 boris-42 davideagnello: or you are going to do it by your self?
21:53 davideagnello boris-42: will be developing a few integration and scenario tests first, make sure they are stable then will integrate them to our CI/gates.  That should be next week sometime :)
21:54 boris-42 davideagnello: that is very bad approach
21:54 boris-42 =)
21:54 boris-42 davideagnello: it's better to put non-voting job
21:54 boris-42 davideagnello: on each patch, make non-voting job pass always
21:54 boris-42 davideagnello: and make it voting
21:54 boris-42 =)
21:55 boris-42 davideagnello: let me help you with infra stuff
21:55 davideagnello boris-42: ok thank you, appreciate it.  It's all new to me
21:56 boris-42 davideagnello: yep it's a bit hard first time
21:56 boris-42 davideagnello: but when you know what to do it takes just minutes
21:57 davideagnello boris-42: yeah that seems like the case
22:04 echoingumesh joined #openstack-rally
22:08 boris-42 davideagnello: tox doesn't use in cue venv??
22:10 davideagnello boris-42: tox creates a virtual env and runs tests in it
22:11 boris-42 davideagnello: OSError: [Errno 13] Permission denied
22:11 boris-42
22:11 boris-42 davideagnello: just got  this
22:12 davideagnello boris-42: from running tox for project Cue?
22:12 echoingumesh joined #openstack-rally
22:13 boris-42 davideagnello: yep
22:15 boris-42 davideagnello: ah it's my bad
22:15 boris-42 davideagnello: sorry
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Add `LogCatcher' context  https://review.openstack.org/174454
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Fix `sshutils' to execute commands with args  https://review.openstack.org/177009
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Split validation.file_exists, allow `required' arg  https://review.openstack.org/177010
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Introduce command-specifying dictionary  https://review.openstack.org/177011
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Introducing `valid_command' validator  https://review.openstack.org/177012
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Extend `sshutils` with `put_file'  https://review.openstack.org/177013
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Make `boot_runcommand_delete' accept command-dict  https://review.openstack.org/177014
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Add command-dict option to upload a local command  https://review.openstack.org/177015
22:26 openstackgerrit Pavel Boldin proposed openstack/rally: Add command-dict option to specify command args  https://review.openstack.org/177016
22:29 openstackgerrit Carlos L. Torres proposed openstack/rally: [report] Improve reports data and units  https://review.openstack.org/161037
22:34 boris-42 davideagnello: https://review.openstack.org/177020
22:36 davideagnello boris-42: awesome, thank you!  will try it out and see how it works
22:36 echoingumesh joined #openstack-rally
22:38 boris-42 davideagnello: https://review.openstack.org/177022
23:01 jseutter joined #openstack-rally
23:41 openstackgerrit Pavel Boldin proposed openstack/rally: Add command-dict option to specify command args  https://review.openstack.org/177016
23:41 openstackgerrit Pavel Boldin proposed openstack/rally: Introducing `valid_command' validator  https://review.openstack.org/177012
23:41 openstackgerrit Pavel Boldin proposed openstack/rally: Extend `sshutils` with `put_file'  https://review.openstack.org/177013
23:41 openstackgerrit Pavel Boldin proposed openstack/rally: Make `boot_runcommand_delete' accept command-dict  https://review.openstack.org/177014
23:41 openstackgerrit Pavel Boldin proposed openstack/rally: Add command-dict option to upload a local command  https://review.openstack.org/177015

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