Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2015-01-20

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

All times shown according to UTC.

Time Nick Message
00:00 yingjun joined #openstack-rally
00:06 nkhare joined #openstack-rally
00:28 dmorita joined #openstack-rally
01:18 boris-42 jlk: that patch fucked me=)
01:18 boris-42 jseutter: one day
01:18 boris-42 jlk: one day*
01:18 boris-42 jseutter: sry*
01:18 boris-42 jseutter: we still have in rally hack
01:19 boris-42 jlk: https://github.com/stackforge/rally/blob/master/rally/osclients.py#L49-L50=)
01:34 boris-42 jlk: rmk guys what do you think about this change http://osdir.com/ml/openstack-dev/2015-01/msg01004.html ?
01:51 openstackgerrit Merged stackforge/rally: Restructure Rally docs  https://review.openstack.org/120018
02:42 openstackgerrit Merged stackforge/rally: Removes duplicate for generate_random_name  https://review.openstack.org/148258
02:47 ilbot3 joined #openstack-rally
02:47 Topic for #openstack-rally is now ☁ Rally RoadMap: http://goo.gl/JZkmwY ☁ Rally IRC chat logs http://irclog.perlgeek.de/openstack-rally ☁ Key persons to ask:  boris-42, msdubov, rediskin, andreykurilin, amaretskiy  Ã¢ËœÂ 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
03:00 jlk boris-42: I like that change
03:03 boris-42 jlk: could you reply in mailing list?)
03:03 boris-42 jlk: I need to convince msdubov=)
03:03 boris-42 jlk: btw msdubov finished new rally readthedocs=) https://rally.readthedocs.org/en/latest/
03:03 boris-42 jlk: now they are much simpler for understanding=)
03:04 jlk oooh I'll rad up
03:04 jlk read up
03:05 boris-42 jlk: so we have some small tutorial here https://rally.readthedocs.org/en/latest/
03:05 boris-42 jlk: here https://rally.readthedocs.org/en/latest/tutorial/step_4_working_with_multple_openstack_clouds.html **
03:05 boris-42 jlk: we merged a bit RAW patch
03:05 boris-42 jlk: cause it was hmm tooo big
03:06 boris-42 jlk: https://review.openstack.org/#/c/120018/ cause it was +2k -1k LOC=)
03:06 boris-42 but overall new structure seems more reasonable at least for me
04:04 boris-42 yfried|afk: hey
04:30 nkhare joined #openstack-rally
04:39 yfried_ boris-42: just woke up. I'll be in the office in 90min (god and daughter willing...)
04:39 boris-42 yfried_: =)
04:40 boris-42 yfried_: ok
05:03 nkhare_ joined #openstack-rally
05:09 jaypipes joined #openstack-rally
05:41 openstackgerrit Zhongcheng Lao proposed stackforge/rally: Disable security group for some neutron driver  https://review.openstack.org/148429
05:43 prmtl joined #openstack-rally
05:45 yfried_ andreykurilin: https://review.openstack.org/#/c/147492/3..4/rally/verification/tempest/tempest.py,cm it's kinda hard to work when getting 2 conflicting comments   :)
05:45 yfried_ andreykurilin: what do you thing I should do?
05:49 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Removing "failed" column entirely from rally cli as well as db  https://review.openstack.org/147962
06:06 rdas joined #openstack-rally
06:41 prmtl joined #openstack-rally
06:46 neeti joined #openstack-rally
06:48 yfried_ joined #openstack-rally
06:52 yfried_ joined #openstack-rally
06:54 yfried_ boris-42: andreykurilin: what should I do with https://review.openstack.org/#/c/147492 ?
06:59 yfried_ boris-42: I also fail to understand your comment here https://review.openstack.org/#/c/147571/2/rally/benchmark/context/tempest.py,cm
07:08 nmagnezi joined #openstack-rally
07:08 nmagnezi_ joined #openstack-rally
07:22 yfried_ boris-42: https://review.openstack.org/147962 could you please comment on my concern?
07:29 prmtl joined #openstack-rally
07:49 psd Hey yfried_
07:49 yfried_ psd: Hi
07:49 psd I thought of what you have pointed out
07:49 yfried_ psd: re?
07:50 psd but then decided against it, since I thought tests try and verify the programs by posing as a user
07:50 psd yfried_, regarding  https://review.openstack.org/147962
07:50 psd (correct me if I am wrong)
07:51 openstackgerrit Sergey Kraynev proposed stackforge/rally: Re-enabling Heat/Neutron resources test  https://review.openstack.org/148457
07:54 yfried_ psd: ?
07:55 yfried_ psd: oh, sorry
07:55 yfried_ psd: I'm not sure how you define " posing as a user"
07:56 yfried_ psd: what "user" would input a raw string? and what if tomorrow we replace the string value of "failure" with something else?
07:56 yfried_ psd: since we already have a const for that, I see no reason to input values directly
07:58 psd yfried_, Okay, I am not sure how to put my point
07:58 psd but I am convinced :)
07:58 yfried_ psd: well, we can agree to disagree :)
07:58 psd let's probably wait for boris-42, and then I can upload the new patch :)
07:58 psd Hehe sure :)
07:58 yfried_ psd: tnx for the discussion though
08:00 e0ne joined #openstack-rally
08:01 psd thanks for pointing it out :)
08:04 exploreshaifali joined #openstack-rally
08:04 arxcruz joined #openstack-rally
08:20 Miouge joined #openstack-rally
08:31 yfried_ arxcruz: https://review.openstack.org/#/c/141813/ could you please rebase?
08:31 yfried_ arxcruz: I'd like to review
08:31 arxcruz yfried_, sure, I was about to do that :)
08:32 yfried_ boris-42: andreykurilin: https://github.com/stackforge/rally/blob/master/rally/consts.py#L27-L39
08:33 yfried_ is partially duplicating https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128
08:34 yfried_ boris-42: andreykurilin is there a way to check if "mystring" in consts.ServiceType.<some form of iterator>?
08:44 openstackgerrit Arx Cruz proposed stackforge/rally: Add a validator for neutron networks  https://review.openstack.org/141813
08:44 arxcruz yfried_, ^
08:44 yfried_ arxcruz: looking
08:51 yfried_ arxcruz: there are a lot of cosmetic changes that are distracting from your actual work. could you please take a look at your patch and make sure you submitted only what you wanted and not some leftovers from the rebase?
08:52 arxcruz yfried_, you meant the ValidationResult(True) ?
08:52 yfried_ arxcruz: partly
08:55 arxcruz yfried_, please, can you comment in the code? :) it would be better so I know exactly what you mean...
08:55 yfried_ arxcruz: commenting on code, but I was hoping you might notice some unintended changes and provide me with a cleaner version
08:56 arxcruz the ValidationResults was asked in this patch, that's why is there
08:56 arxcruz also, removing the ValidationResults() in some returns was intended as well
08:57 arxcruz can you check the discussion in the other patch sets ? there are a lot of discussions about what change and what not
08:57 arxcruz brb, czech lessons
08:58 yfried_ arxcruz: you're touching a lot of code outside the scope of your patch. it's really distracting
08:58 yfried_ arxcruz: all the  ValidationResult(True) changes
08:59 arxcruz yfried_, I understand, so, please comment on the patch if you don't mind, so everyone that was involved can see, and we can start a discussion :)
09:00 yfried_ arxcruz: sure
09:08 yfried_ arxcruz: review done
09:09 amaretskiy joined #openstack-rally
09:10 yfried_ boris-42: andreykurilin: amaretskiy: any of you around?
09:10 amaretskiy yfried_ hi
09:10 andreykurilin_ joined #openstack-rally
09:11 yfried_ amaretskiy: https://github.com/stackforge/rally/blob/master/rally/consts.py#L27-L39 is partially duplicating https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128
09:11 yfried_ amaretskiy: is there a way to check if "mystring" in consts.ServiceType.<some form of iterator>?
09:11 amaretskiy i don't think so
09:12 yfried_ amaretskiy: also, https://github.com/stackforge/rally/blob/master/rally/consts.py#L130-L149 what does this do, overwrite the values in https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128 ?
09:12 amaretskiy since service type (means "OpenStack service") and tempest tests cases seems to be completely different things
09:14 amaretskiy there are services types (internal OpenStack entity) and there are proper service types implementations, examples: identity->keystone, network->neutron
09:14 amaretskiy consts provides a mapping for them
09:14 amaretskiy but tempest tests cases are something different
09:15 yfried_ amaretskiy: actually, not
09:15 amaretskiy yfried_  I'm not a tempest man :) maybe I am mistaken aout tempest
09:15 yfried_ amaretskiy: tempest api test sets are each mapped to service type
09:15 amaretskiy but consts look fine for me
09:16 amaretskiy yfried_ I recommend to discuss that with andreykurilin
09:16 yfried_ amaretskiy: my concern is https://github.com/stackforge/rally/blob/master/rally/verification/tempest/tempest.py#L176-L177
09:17 yfried_ amaretskiy: this works fine if set_name is in ServiceType but not for "full", "smoke", or "scenario"
09:17 amaretskiy I think that the minus of such mapping is that tempest does not refer to rally code
09:18 amaretskiy so any mapping is a potential problem
09:18 amaretskiy I mean that future tempest changes can break that
09:20 yfried_ amaretskiy: ok
09:32 andreykurilin_ yfried_: hi! yesterday I broke some lab in mirantis, so I was busy repairing it and forgot to answer you, sorry :)
09:32 andreykurilin_ yfried_: now, I can try to answer whatever you want.
09:38 yfried_ https://github.com/stackforge/rally/blob/master/rally/consts.py#L27-L39 is partially duplicating https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128  is there a way to check if "mystring" in consts.ServiceType.<some form of iterator>?
09:38 yfried_ also, https://github.com/stackforge/rally/blob/master/rally/consts.py#L130-L149 what does this do, overwrite the values in https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128 ?
09:38 yfried_ andreykurilin_: ignore the last one ^
09:38 yfried_ https://github.com/stackforge/rally/blob/master/rally/verification/tempest/tempest.py#L176-L177 this works fine if set_name is in ServiceType but not for "full", "smoke", or "scenario"
09:42 andreykurilin_ yfried_: TEMPEST_TEST_SETS cannot be replaced by _ServiceType, because we don't want to mix tempest-specific consts with others. TEMPEST_TEST_SETS is a list of all modules from tempest.api (plus full and smoke set name)
09:44 andreykurilin_ yfried_: I'm not tempest guru, so I'mm scared a bit to mix them:)
09:44 yfried_ andreykurilin_: ok
09:45 openstackgerrit Yair Fried proposed stackforge/rally: Adds "scenario" to available verify sets  https://review.openstack.org/147581
09:45 yfried_ andreykurilin_: ^
09:45 andreykurilin_ yfried_, about iterator: you can do something like consts.ServiceType.__dict__
09:49 andreykurilin_ yfried_: please see my comment for https://review.openstack.org/#/c/147581/
09:50 openstackgerrit Yuriy Nesenenko proposed stackforge/rally: Add extend_volume scenario for Cinder test  https://review.openstack.org/147489
09:51 yfried_ amaretskiy: https://review.openstack.org/#/c/147581/3/rally/consts.py,cm
09:51 yfried_ amaretskiy: what's your concern here?
09:52 andreykurilin_ yfried_: about https://review.openstack.org/#/c/147581/ it would be better to ask boris-42 :)
09:52 amaretskiy (1,) + (2,) == (1, 2)
09:53 yfried_ amaretskiy: A + B returns a list/tuple
09:53 yfried_ amaretskiy: itertools.chain returns an iterator
09:53 amaretskiy iter( (1,) + (2,) )
09:54 yfried_ amaretskiy: so that's what itertools.chain does
09:54 andreykurilin_ yfried_: about comment conflicts. maybe I'm wrong(I'm not JG, so I can do some mistakes). please, fix that patch as msdubov noted
09:54 yfried_ andreykurilin_: JG?
09:55 yfried_ andreykurilin_: already did
09:55 andreykurilin_ yfried_: Joe Gordon:)
09:55 amaretskiy yfried_ iter() is built-in but itertools is imported module - no sense to import extra module if we can do this in pure python
09:55 boris-42 yfried_: andreykurilin_ what?
09:55 andreykurilin_ boris-42: <yfried_> also, https://github.com/stackforge/rally/blob/master/rally/consts.py#L130-L149 what does this do, overwrite the values in https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128 ?
09:56 boris-42 yfried_: so what did you fail to understand?
09:58 yfried_ boris-42:  https://github.com/stackforge/rally/blob/master/rally/consts.py#L112-L128 defines class level values
09:58 yfried_ boris-42: and  https://github.com/stackforge/rally/blob/master/rally/consts.py#L130-L149 overwrites them
09:58 yfried_ boris-42: wonder why do the first block at all
09:59 oanufriev joined #openstack-rally
09:59 openstackgerrit Yair Fried proposed stackforge/rally: Adds "scenario" to available verify sets  https://review.openstack.org/147581
09:59 amaretskiy yfried_: by the way, we have, finally, merged https://review.openstack.org/#/c/144337/  so your patch sets related to VMTasks are not blocked anymore
10:00 yfried_ amaretskiy: yeah. I saw that. tnx
10:00 * yfried_ dreads the rebase...
10:00 amaretskiy :)
10:01 openstackgerrit Yair Fried proposed stackforge/rally: Adds "scenario" to available verify sets  https://review.openstack.org/147581
10:02 psd_ joined #openstack-rally
10:02 afazekas joined #openstack-rally
10:04 boris-42 yfried_: as far as I remember that was the idea of amaretskiy =)
10:04 amaretskiy ???
10:05 boris-42 andreykurilin_: with 2 types services and serviceTypes
10:05 boris-42 amaretskiy: ^
10:05 boris-42 andreykurilin_: sry
10:05 yfried_ boris-42: could you please clarify your comment https://review.openstack.org/#/c/147571/2
10:05 yfried_ ?
10:05 amaretskiy >> andreykurilin_: with 2 types services and serviceTypes
10:05 amaretskiy yes
10:05 amaretskiy I initially divided these two sets of entities
10:06 boris-42 yfried_: you can use just
10:06 boris-42 yfried_: self.task["deploymnet_uuid"]
10:06 boris-42 yfried_: instead of self.task.task.deployment_uuid
10:06 boris-42 yfried_: seems nicer for me
10:07 yfried_ boris-42: so you want me to uset the dict instead of attribute? just making sure
10:07 boris-42 yfried_: I dislike sefl.task.task
10:08 boris-42 when you can do self.task["uuid"]
10:08 boris-42 task["deployment_uuid"]
10:08 boris-42 that is all
10:10 arxcruz boris-42, around?
10:11 arxcruz boris-42, regarding the Assert SLA you ask me, in the SLA have a way to get the exceptions? x["error"][something] perhaps ?
10:12 openstackgerrit Yair Fried proposed stackforge/rally: Allow user to choose Tempest remote repo  https://review.openstack.org/147492
10:12 openstackgerrit Yair Fried proposed stackforge/rally: Adds Tempest source repo support for "Tempest" context  https://review.openstack.org/147571
10:12 openstackgerrit Yair Fried proposed stackforge/rally: Adds support for multiple tempest sources  https://review.openstack.org/148187
10:12 yfried_ boris-42: https://review.openstack.org/#/c/143849/ do you need me to resume working on this?
10:16 e0ne joined #openstack-rally
10:19 yfried_ boris-42: I started doing some reviews today but most of the queue has either failed jenkins or failed rebase
10:23 andreykurilin_ yfried_: about our yesturday talk. I lost my doc with ideas of improvement verification:( so I create a bp with information about improvement of "rally-manage tempest" cli
10:24 andreykurilin_ yfried_: https://blueprints.launchpad.net/rally/+spec/improve-tempest-manage-cli
10:28 psd_ joined #openstack-rally
10:44 openstackgerrit Yuriy Nesenenko proposed stackforge/rally: Add list_snapshots scenario for Cinder test  https://review.openstack.org/146580
10:47 andreykurilin_ joined #openstack-rally
10:51 e0ne joined #openstack-rally
11:03 yfried|lunch boris-42: didn't you move cinder tests to unstable?
11:06 andreykurilin_ yfried: yes: https://review.openstack.org/#/c/148288/
11:07 yfried andreykurilin_: so why do I fail http://logs.openstack.org/81/147581/5/check/gate-rally-dsvm-rally/b8b8663/rally-plot/results.html.gz ?
11:08 andreykurilin_ yfried: good question:)
11:08 yfried andreykurilin_: how about a good answer? ;)
11:10 andreykurilin_ yfried: It seems to me, this scenario is not failed often, so it was not mpved to unstable.
11:14 openstackgerrit Yuriy Nesenenko proposed stackforge/rally: Add extend_volume scenario for Cinder test  https://review.openstack.org/147489
11:16 andreykurilin_ yfried: I need to go, will be available later.
11:22 cdent joined #openstack-rally
11:24 tkammer joined #openstack-rally
11:24 yfried tkammer: ping
11:27 akuznetsova_ joined #openstack-rally
11:31 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Removing "failed" column entirely from rally cli as well as db  https://review.openstack.org/147962
11:31 psd_ yfried, ^, :)
11:32 Miouge_ joined #openstack-rally
11:34 prmtl joined #openstack-rally
11:38 yfried psd: I thinkg you've missed some
11:41 psd oh sorry, I think I missed that file
11:42 openstackgerrit Anton Arefiev proposed stackforge/rally: Change cinder scenario: create and delete volume  https://review.openstack.org/146820
11:42 openstackgerrit Anton Arefiev proposed stackforge/rally: Add cinder scenario: create volume from volume(clone)  https://review.openstack.org/147449
11:46 yfried amaretskiy: can we mark wrapper functions as "atomic"?
11:46 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Removing "failed" column entirely from rally cli as well as db  https://review.openstack.org/147962
11:49 yfried amaretskiy: boris-42: cinder is doing some bad stuff http://logs.openstack.org/87/148187/3/check/gate-rally-dsvm-neutron-rally/b8d8dfd/rally-plot/results.html.gz#/
11:51 pboros joined #openstack-rally
11:52 psd thanks yfried :)
11:56 obutenko joined #openstack-rally
12:03 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Publishing functional tests html reports  https://review.openstack.org/147098
12:08 msdubov_ joined #openstack-rally
12:13 nkhare joined #openstack-rally
12:17 pcaruana joined #openstack-rally
12:18 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Publishing functional tests html reports  https://review.openstack.org/147098
12:20 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Publishing functional tests html reports  https://review.openstack.org/147098
12:28 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Removing "failed" column entirely from rally cli as well as db  https://review.openstack.org/147962
12:30 msdubov_ joined #openstack-rally
12:32 RaginBajin joined #openstack-rally
12:48 e0ne joined #openstack-rally
12:49 msdubov_ joined #openstack-rally
12:52 nkhare joined #openstack-rally
13:02 openstackgerrit Oleh Anufriiev proposed stackforge/rally: Publishing functional tests html reports  https://review.openstack.org/147098
13:07 prmtl joined #openstack-rally
13:10 yfried amaretskiy: here? I wonder if I can make floatingip commands in wrapper into atomic actions
13:11 exploreshaifali joined #openstack-rally
13:15 amaretskiy yfried: hi again
13:16 amaretskiy yfried I think atomic actions can be embedded in wrapper but they must be enabled by specific argument (disabled by default)
13:16 amaretskiy also - this question is a serious topic to discus swith boris-42
13:23 aarefiev joined #openstack-rally
13:26 yfried amaretskiy: boris-42: so - we have the net-wrapper
13:26 yfried but we also have nova.create-floatingip and other similar atomic actions
13:27 yfried a better practice would be to define net_wrapper.allocate_fip_to_server(server)
13:27 yfried that would:
13:27 tkammer joined #openstack-rally
13:27 yfried 1. create fip
13:28 yfried 2. fetch the server address
13:28 yfried 3. associate fip with address
13:28 yfried and will do 1 and 3 in atomic actions
13:29 yfried so I wonder if we define everything in net_wrapper
13:29 yfried and define the atomic actions for the specific implementations of the wrapper
13:29 yfried amaretskiy: boris-42: ^ what do you think?
13:31 msdubov_ joined #openstack-rally
13:32 amaretskiy >> a better practice would be to define net_wrapper.allocate_fip_to_server(server)
13:32 amaretskiy not sure about that - I always suggest to implement object-related actions as this object methods (actually that is an OOP) - example: server.allocate_fip()
13:32 amaretskiy instead of some_utils.do_smth_with_server(server)
13:33 amaretskiy s/server.allocate_fip()/server.associate_fip(fip)/
13:35 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Improving error message  https://review.openstack.org/148308
13:36 yfried amaretskiy: how does server know net_wrapper?
13:40 yfried msdubov_: could you please review the rest of https://review.openstack.org/#/q/status:open+project:stackforge/rally+branch:master+topic:tempest_source,n,z ?
13:43 msdubov_ joined #openstack-rally
13:44 msdubov_ yfried: Will do today!
13:50 e0ne joined #openstack-rally
13:56 yfried msdubov_: tnx
14:00 boris-42 yfried: so me back finally
14:00 boris-42 yfried: sorry today i have workout=)
14:01 * boris-42 reading
14:18 mwagner_lap joined #openstack-rally
14:21 amaretskiy >> yfried: amaretskiy: how does server know net_wrapper?
14:21 amaretskiy server should know only several data (floating_ip_address, fixed_ip_address, etc) - it does not need network wrapper to associate fip with itself
14:23 amaretskiy yfried: I have a question to you regarding https://review.openstack.org/#/c/146814/
14:23 amaretskiy yfried: ping ^^^
14:24 yfried amaretskiy: I want a highlevel floating ip function that creates and associates a fip
14:24 yfried amaretskiy: shoot
14:25 yfried amaretskiy: I have absolutely no knowledge with displays so I just hacked around
14:26 amaretskiy >> I want a highlevel floating ip function that creates and associates a fip
14:26 amaretskiy yfried, ok, just give a patch set for that and we can see explicit advantages/disadvantages of that approach
14:26 amaretskiy yfried: regarding https://review.openstack.org/#/c/146814/
14:26 amaretskiy I have a draft patch
14:26 openstackgerrit Yuriy Nesenenko proposed stackforge/rally: Add list_snapshots scenario for Cinder test  https://review.openstack.org/146580
14:26 amaretskiy that prepares data and draws a chart
14:26 amaretskiy for parallel iterations
14:27 amaretskiy I can send you a diff file
14:27 amaretskiy or just submit a patch set over your patch
14:27 amaretskiy so in this case i will be a co-author
14:27 amaretskiy what option do you prefer?
14:27 yfried amaretskiy: you should run over my patch, it was just rough sketch
14:28 amaretskiy okay
14:28 yfried amaretskiy: I'm no way near ready to work on it
14:28 amaretskiy so there will be a new patch set for this patch
14:28 amaretskiy and then you can examine and continue work on this patch
14:28 yfried amaretskiy: re fip
14:28 amaretskiy my patch is a draft but it already working
14:29 yfried amaretskiy: is it ok to define atomic actions in network wrapper?
14:29 yfried amaretskiy: great. I'd like to take a look
14:29 amaretskiy we should discuss atomic actions with bores-42
14:29 amaretskiy *boris-42
14:29 amaretskiy because atomic actions are the point of future refactoring
14:32 amaretskiy imho AtomicMixin is better choice for future http://paste.openstack.org/show/155752/
14:33 amaretskiy we need unified and flexible implementation of atomic actions both for context and scenarios
14:33 amaretskiy so that is not only a question of wrapper
14:34 amaretskiy wrapper should be able store atomic action for context and same for scenario - with different names
14:38 amaretskiy yfried: also please confirm that i can delete this document https://docs.google.com/a/mirantis.com/document/d/1p1473pRwBqthIa7tlbVAbm5OZT6RS4mgVDvR1p-e2EM/edit
14:40 yfried amaretskiy: what do you mean different names?
14:40 amaretskiy take a look at http://paste.openstack.org/show/155752/
14:40 yfried amaretskiy: why delete this doc? I thought we are going to use it to plan future improvement of network
14:41 amaretskiy same action is called both in context and scenario
14:41 amaretskiy my patch is merged so you can rebase your - there is no conflict
14:42 amaretskiy we have road map for future planning
14:43 yfried amaretskiy: roadmap?
14:43 amaretskiy the reason why this document was created is that we had patched with huge changes of same code - so we had to organize ourself to avoid unnecessary work
14:44 yfried amaretskiy: yeah
14:44 amaretskiy now your patch is not blocked
14:44 yfried amaretskiy: but we (you and I) discussed future work on network and I thought we can coordinate it in a new doc
14:44 yfried amaretskiy: -- in the same doc
14:44 amaretskiy okay, let's start new doc :)
14:45 amaretskiy but take a look at road map
14:45 amaretskiy the url is in this chat's header
14:46 amaretskiy so - i delete  https://docs.google.com/a/mirantis.com/document/d/1p1473pRwBqthIa7tlbVAbm5OZT6RS4mgVDvR1p-e2EM/edit
14:46 amaretskiy right?
14:54 tkammer joined #openstack-rally
15:04 psd joined #openstack-rally
15:04 openstackgerrit Merged stackforge/rally: Allow user to choose Tempest remote repo  https://review.openstack.org/147492
15:09 yfried amaretskiy: ack
15:12 yfried andreykurilin: still here? about tempest remove/reinstall
15:12 Miouge joined #openstack-rally
15:13 yfried andreykurilin: if we remove the repo from tempest base - what happens to all the deployment specific repos
15:16 tkammer joined #openstack-rally
15:17 tkammer yfried, ping
15:17 yfried tkammer: ack
15:19 msdubov_ joined #openstack-rally
15:20 msdubov_ joined #openstack-rally
15:20 yfried amaretskiy: can we do server related ops like you suggested?
15:21 yfried amaretskiy: server is implemented in nova.client, isn't it?
15:25 amaretskiy >> yfried: amaretskiy: can we do server related ops like you suggested
15:25 amaretskiy I'm not a dictator, you can submit any patch set and then team can review and comment it, everything is possible
15:25 amaretskiy >> yfried: amaretskiy: server is implemented in nova.client, isn't it?
15:25 amaretskiy yes, that is a nova stuff
15:26 yfried amaretskiy: I'm not asking for permission
15:26 amaretskiy i understand but i can mistaken
15:26 yfried amaretskiy: I'm asking if rally has a server object extension
15:26 amaretskiy anyone from our team can have another opinoion
15:26 yfried amaretskiy: if not, then adding server methods is a little problematic
15:26 amaretskiy so i can not block your ideas
15:27 amaretskiy but nova server already has a method to associate floating ip
15:27 yfried amaretskiy: but not to create
15:28 amaretskiy creating of floating ip imho a job of network wrapper only
15:28 amaretskiy but association - not sure
15:28 yfried amaretskiy: ok, so about creation - should the network-wrapper make it an atomic action?
15:29 amaretskiy we must have an ability to measure everything
15:29 amaretskiy but specific implementation of this atomic action is the question
15:29 yfried amaretskiy: also - atomic actions are defined in scenario.base
15:29 amaretskiy let's discuss atomic actions with boris-42
15:29 yfried boris-42: here?
15:35 pboros_ joined #openstack-rally
15:41 Miouge joined #openstack-rally
15:51 psd joined #openstack-rally
15:58 jaypipes joined #openstack-rally
16:00 boris-42 yfried: meetings hell
16:00 boris-42 yfried: sry
16:00 boris-42 yfried: I will reply later today
16:02 openstackgerrit Yuriy Nesenenko proposed stackforge/rally: Add list_snapshots scenario for Cinder test  https://review.openstack.org/146580
16:02 openstackgerrit Wataru Takase proposed stackforge/rally: Add cold migrate scenario for Nova test  https://review.openstack.org/148307
16:09 yfried amaretskiy: http://paste.openstack.org/show/155752/ fyi - starting to work on this
16:10 amaretskiy yfried: ok, but keep in mind that boris-42 should approve this approach
16:11 amaretskiy this paste was just my idea
16:11 boris-42 yfried: btw are you going to take a part in todays rally meeting?
16:11 boris-42 yfried: one more meeting mrrreee meeting=)
16:12 yfried boris-42: when is it?
16:12 boris-42 yfried: in 48 minutes
16:12 yfried yfried: probably not. that's 19:00 my time, and I should be heading home shortly
16:13 yfried yfried: is it a weekly thing?
16:13 boris-42 yfried: heh it's 20:00 here..
16:13 yfried boris-42: aren't you guys rally-core? I'd expect you to hold the mtg on your own working hours...
16:13 boris-42 yfried: we can move it actually
16:14 boris-42 yfried: the idea is to be possilbe to make one meeting for all countries
16:14 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Improving error message  https://review.openstack.org/148308
16:15 tkammer joined #openstack-rally
16:20 openstackgerrit Alexander Maretskiy proposed stackforge/rally: (WIP) Add chart `Parallel iterations' to report  https://review.openstack.org/146814
16:21 amaretskiy yfried: take a look at https://review.openstack.org/#/c/146814/
16:21 dpaterson joined #openstack-rally
16:22 msdubov_ joined #openstack-rally
16:23 psd joined #openstack-rally
16:26 andreykurilin_ joined #openstack-rally
16:37 andreykurilin__ joined #openstack-rally
16:45 andreykurilin__ yfried: I here for several minutes)
16:45 andreykurilin__ yfried: >if we remove the repo from tempest base - what happens to all the deployment specific repos
16:46 andreykurilin__ yfried: all deployment specific repositories are independen from base repo. vase repo is used only during installation
16:46 andreykurilin__ *base repo is used ...
16:50 yfried andreykurilin__: yeah,
16:50 yfried andreykurilin__: so I understand "remove"
16:51 yfried andreykurilin__: but what happens to the "daughter repos" on re-install?
16:52 rvasilets joined #openstack-rally
16:52 andreykurilin__ yfried: imo, "remove" should remove not base repo, but specific to deployment files(repo, config, logs files  and etc)
16:54 yfried andreykurilin__: and re-install?
16:54 openstackgerrit Yair Fried proposed stackforge/rally: Refactor AtomicAction into AtomicMixin  https://review.openstack.org/148637
16:54 yfried amaretskiy: boris-42^
16:56 openstackgerrit Yuriy Nesenenko proposed stackforge/rally: Add extend_volume scenario for Cinder test  https://review.openstack.org/147489
17:00 psd joined #openstack-rally
17:00 andreykurilin_ joined #openstack-rally
17:00 msdubov_ joined #openstack-rally
17:00 andreykurilin_ yfried: I don't know correct way for re-install. It can be done in next way: remove all deployment specific files and launch tempest.install. or: remove all deployment specific files(except config file in case, rally-manage tempest configure should help use, if he will want to update it) and launch tempest.install
17:06 msdubov_ joined #openstack-rally
17:08 openstackgerrit Merged stackforge/rally: Removing "failed" column entirely from rally cli as well as db  https://review.openstack.org/147962
17:09 openstackgerrit Merged stackforge/rally: Re-enabling Heat/Neutron resources test  https://review.openstack.org/148457
17:32 psd joined #openstack-rally
17:36 msdubov_ joined #openstack-rally
17:49 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano benchmarks  https://review.openstack.org/137661
17:49 openstackgerrit Roman Vasilets proposed stackforge/rally: Add murano base for murano benchmarks  https://review.openstack.org/137650
18:35 openstackgerrit Merged stackforge/rally: Add extend_volume scenario for Cinder test  https://review.openstack.org/147489
19:08 fhubik_lunch joined #openstack-rally
19:13 yfried joined #openstack-rally
19:20 cdent joined #openstack-rally
19:21 openstackgerrit Yair Fried proposed stackforge/rally: Refactor AtomicAction into AtomicMixin  https://review.openstack.org/148637
19:21 yfried boris-42: ^
19:23 yfried boris-42: andreykurilin: this might be a stupid question, but why can't I see ^ in zuul?
19:25 jaypipes all: great work on the new docs. really, really superb.
19:27 andreykurilin_ joined #openstack-rally
19:32 andreykurilin_ yfried, about zuul: you should go to http://status.openstack.org/zuul/ and enter patch-id (https://review.openstack.org/#/c/<patch-id>/) in filter field. For your patch - 148637. Your patch now in "check" queue:)
19:33 exploreshaifali joined #openstack-rally
19:33 yfried andreykurilin_: I know. it wasn't when I asked the question.
19:34 yfried andreykurilin_: it took unusual 5min
19:34 andreykurilin_ yfried: hm
19:34 andreykurilin_ yfried: magic)
19:35 andreykurilin_ jaypipes: yeah, msdubov done a great work on it:)
20:22 psd joined #openstack-rally
21:16 pboros___ joined #openstack-rally
21:23 psd_ joined #openstack-rally
21:25 Prabhjyot__ joined #openstack-rally
21:28 psd_ joined #openstack-rally
21:28 openstackgerrit Jesse Keating proposed stackforge/rally: Update location to sample script  https://review.openstack.org/148726
21:41 openstackgerrit Prabhjyot Singh Sodhi proposed stackforge/rally: Improving error message  https://review.openstack.org/148308
21:48 Prabhjyot__ joined #openstack-rally
22:19 e0ne joined #openstack-rally
23:45 jlk boris-42: so on benchmarks like CinderVolumes.create_snapshot_and_attach_volume, how do I pass along my nic info to the nova boot? I tried putting the argument in the serviers: block, but that got kicked back as invalid :/

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