Perl 6 - the future is here, just unevenly distributed

IRC log for #openstack-rally, 2016-06-22

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

All times shown according to UTC.

Time Nick Message
00:12 yingjun joined #openstack-rally
00:47 catintheroof joined #openstack-rally
00:50 kuladeep joined #openstack-rally
01:02 openstackgerrit yuyafei proposed openstack/rally: Make dict.keys() PY3 compatible  https://review.openstack.org/332593
01:39 yanyanhu joined #openstack-rally
01:48 ilbot3 joined #openstack-rally
01:48 Topic for #openstack-rally is now ☁ Rally RoadMap: http://goo.gl/JZkmwY ☁ Review Dashboard: http://goo.gl/YuCJfl ☁ IRC logs http://irclog.perlgeek.de/openstack-rally ☁ Key persons to ask:  boris-42, yfried, msdubov, rediskin, andreykurilin, amaretskiy  ☁ Documentation: https://rally.readthedocs.org/en/latest/ ☁ To publish changes to Rally:  https://rally.readthedocs.org/en/latest/contribute.html
02:00 yingjun joined #openstack-rally
02:03 yingjun joined #openstack-rally
02:16 yamamot__ joined #openstack-rally
02:57 caowei joined #openstack-rally
03:01 yingjun joined #openstack-rally
03:17 yingjun joined #openstack-rally
03:23 openstackgerrit lvfm proposed openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/332021
03:33 openstackgerrit lvfm proposed openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/332021
03:44 M00nr41n joined #openstack-rally
03:44 catintheroof joined #openstack-rally
04:18 yingjun joined #openstack-rally
04:52 shausy joined #openstack-rally
04:53 yamamot__ joined #openstack-rally
04:54 yingjun joined #openstack-rally
04:56 rcernin joined #openstack-rally
04:58 openstackgerrit lvfm proposed openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/332021
05:10 KevinE joined #openstack-rally
05:23 yingjun joined #openstack-rally
05:23 M00nr41n joined #openstack-rally
05:30 caowei joined #openstack-rally
05:45 yingjun joined #openstack-rally
05:55 rcernin joined #openstack-rally
06:13 rdas joined #openstack-rally
06:36 caowei joined #openstack-rally
06:47 tesseract- joined #openstack-rally
06:48 rvasilets_ joined #openstack-rally
06:48 igajsin1 joined #openstack-rally
06:52 dmellado_ joined #openstack-rally
07:10 amaretskiy joined #openstack-rally
07:14 yanyanhu joined #openstack-rally
07:22 pcaruana joined #openstack-rally
07:26 adiantum joined #openstack-rally
07:35 ushkalim joined #openstack-rally
07:51 adiantum1 joined #openstack-rally
08:06 e0ne joined #openstack-rally
08:11 yingjun joined #openstack-rally
08:35 alexchadin joined #openstack-rally
08:42 bapalm joined #openstack-rally
08:44 yingjun joined #openstack-rally
08:52 mvk joined #openstack-rally
08:53 abalutoiu joined #openstack-rally
08:54 yingjun joined #openstack-rally
09:07 anshul joined #openstack-rally
09:14 igajsin1 left #openstack-rally
09:23 yingjun joined #openstack-rally
09:30 adiantum joined #openstack-rally
09:44 tosky joined #openstack-rally
09:46 yingjun joined #openstack-rally
09:54 bapalm joined #openstack-rally
09:56 mvk joined #openstack-rally
10:19 openstackgerrit Merged openstack/rally: Make dict.keys() PY3 compatible  https://review.openstack.org/332593
11:16 caowei joined #openstack-rally
11:27 zackf joined #openstack-rally
12:03 ushkalim joined #openstack-rally
12:14 e0ne joined #openstack-rally
12:15 maeca1 joined #openstack-rally
12:19 dave-mccowan joined #openstack-rally
12:21 shausy joined #openstack-rally
12:37 alexchadin joined #openstack-rally
12:41 alexchadin joined #openstack-rally
13:02 yamamoto joined #openstack-rally
13:07 permalac joined #openstack-rally
13:55 openstackgerrit Illia Khudoshyn proposed openstack/rally: Fix 'existing_users' context against keystone v3  https://review.openstack.org/332204
13:58 dpaterson joined #openstack-rally
14:20 tosky joined #openstack-rally
14:26 arnoldje joined #openstack-rally
14:35 KevinE joined #openstack-rally
15:01 openstackgerrit Illia Khudoshyn proposed openstack/rally: Fix 'existing_users' context against keystone v3  https://review.openstack.org/332204
15:10 bapalm joined #openstack-rally
15:11 yarkot1 joined #openstack-rally
15:16 bkopilov joined #openstack-rally
15:17 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
15:24 rcernin joined #openstack-rally
15:29 openstackgerrit Andrey Kurilin proposed openstack/rally: [ci] rename rally-gate.py to wip-rally-gate.py  https://review.openstack.org/332873
15:36 pcaruana joined #openstack-rally
15:41 zackf joined #openstack-rally
15:49 catintheroof joined #openstack-rally
15:58 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
16:04 adiantum joined #openstack-rally
16:13 adiantum joined #openstack-rally
16:18 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
16:20 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
16:44 raddaoui joined #openstack-rally
16:45 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
17:01 sai joined #openstack-rally
17:20 rcernin joined #openstack-rally
17:28 alexchadin joined #openstack-rally
17:41 mvk joined #openstack-rally
17:56 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
18:30 e0ne joined #openstack-rally
18:53 mvk_ joined #openstack-rally
19:02 mkrcmari__ joined #openstack-rally
19:12 openstackgerrit Boris Pavlovic proposed openstack/rally: [spec] Making DB scalable and flexible enough  https://review.openstack.org/167060
19:16 mvk joined #openstack-rally
19:17 rook KevinE getting back to your question, I don't feel like that is a better solution.
19:18 KevinE rook: say what?
19:18 rook [06/21/16 16:01] <KevinE> rook: andreykurilin__ what would happen if I just set the src path to {{iperf-path}} and went that route for merging, just let the user deal with it?
19:22 openstackgerrit Kevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/326701
19:23 KevinE rook: oh that, well I had already changed that early this morning. I'm just cleaning things up in the meantime and we're trying to figure out what to do
19:24 yamamoto joined #openstack-rally
19:28 pvaneck joined #openstack-rally
20:00 openstackgerrit Merged openstack/rally: [ci] rename rally-gate.py to wip-rally-gate.py  https://review.openstack.org/332873
20:01 mvk_ joined #openstack-rally
20:02 openstackgerrit Merged openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/332021
20:12 openstackgerrit Merged openstack/rally: [Core] Introduce class-based scenarios  https://review.openstack.org/322825
20:12 openstackgerrit Merged openstack/rally: [spec] Making DB scalable and flexible enough  https://review.openstack.org/167060
20:51 rook joined #openstack-rally
20:56 andreykurilin__ KevinE: sorry for not-reviewing. will try to look at your patch tomorrow
20:57 KevinE andreykurilin__: No problem, keep working on things. I was actually gonna message you because I think you'll be able to help a lot more than others
20:57 KevinE andreykurilin__: Hopefully I'll have it pretty cleaned up tonight so it makes sense :)
20:59 openstackgerrit Merged openstack/rally: [Plugins] Improve scenario output from boot_runcommand_delete  https://review.openstack.org/328369
21:01 rook joined #openstack-rally
21:39 abalutoiu joined #openstack-rally
21:39 openstackgerrit Boris Pavlovic proposed openstack/rally: Use pytest instead of testr  https://review.openstack.org/328672
21:44 openstackgerrit Kevin Esensoy proposed openstack/rally: Add scenarios to VM  https://review.openstack.org/326701
21:46 openstackgerrit Kevin Esensoy proposed openstack/rally: Add scenarios to VM  https://review.openstack.org/326701
21:56 openstackgerrit Boris Pavlovic proposed openstack/rally: [Plugins] Class-based scenario DummyFailure  https://review.openstack.org/322829
22:04 andreykurilin__ KevinE: btw, maybe I'll not have enough time for keystone endpoint_type issue. Do you want to fix it?
22:04 andreykurilin__ *will not have enough time this week
22:04 KevinE andreykurilin__: uhh how, didn't you say you had stuff done on it too?
22:05 andreykurilin__ I know how to fix it:)
22:05 KevinE andreykurilin__: well I don't :P you kind of ran off and did your own thing
22:06 andreykurilin__ KevinE: heh. Yeah, I tried several changes before I got the final idea of fix.
22:07 KevinE andreykurilin__: I don't know what you expect me to say when confronted with "I know how to fix it, it's not what you're thinking, wanna figure it out?" :p
22:10 zackf joined #openstack-rally
22:11 andreykurilin__ KevinE: the fix is based on your first patch and with some modifications it will pass our gates:) and I want to hear from you something like "Yeah, I have enough time to make world better, to fix this issue. Tell me more about fix" lol
22:18 andreykurilin__ KevinE: okay, I'll try to find someone else
22:18 KevinE andreykurilin__: well I have all the time in the world, with 0 skills, and I assumed that having no time to do the fix correlated with not having time to explain :)
22:18 KevinE andreykurilin__: I would absolutely love to do it. Am I ABLE to do it? I have no clue tell me what your plan is!
22:21 andreykurilin__ KevinE: so the explanation of fix=! fix:) and this task can increase your skills and in future you will be able to do more difficult stuff and I'm interested in contribution to rally:)
22:23 KevinE andreykurilin__: I don't even know what to say lol
22:23 andreykurilin__ So your patch(first version) failed on gates, because "default" cloud restricts access to keystone throw public endpoint. In real word such behavior can be changed(like on your configuration), so it would be nice to have ability to configure endpoint types separately for keystone and for other services in rally
22:25 andreykurilin__ You need to extend rally deployment config with new variable "keystone_endpoint_type" with default value None(it means that keystone will use its inner default type) and transmit it to keystoneclient like in your change
22:28 andreykurilin__ KevinE: the first part of change- extend config schema like https://github.com/openstack/rally/blob/master/rally/deployment/engines/existing.py#L99-102
22:28 andreykurilin__ and transmit new variable into Credentials object at L124
22:29 andreykurilin__ Also you need to extend Credentials object with new variable
22:29 andreykurilin__ And, of course, you need to transmit this new variable into keystoneclient like in your previous change
22:30 andreykurilin__ Btw, you will need to fix unit tests, but I'll help you with suggestions
22:31 andreykurilin__ KevinE: how does it sound for you?:)
22:33 KevinE andreykurilin__: I understood like half of that :/
22:34 KevinE so you're saying to edit the Credential's endpoint_type to allow for admin & internal as well, then also create a keystone_endpoint_type and add it to that list of variables, setting the default to none?
22:35 KevinE andreykurilin__: That'll take like 2 minutes so I must be missing something :)
22:36 andreykurilin__ KevinE: almost. You shout not change Credentials's endpoint_type, just add new variable there
22:36 andreykurilin__ *should
22:37 andreykurilin__ Yeah, this change is not so big, but you will need to fix existing tests
22:37 adiantum joined #openstack-rally
22:37 andreykurilin__ Btw, sorry for my English:)
22:38 KevinE andreykurilin__: pshh, better than my *insert your native language*
22:39 andreykurilin__ Heh
22:40 andreykurilin__ KevinE: so can I hope to see a fix from you?)
22:42 KevinE andreykurilin__: http://pastebin.com/reQb3FaG
22:43 KevinE andreykurilin__: lines 77 and 109?
22:44 andreykurilin__ This part is correct
22:45 andreykurilin__ Also, credentials object should accept this new variable
22:45 KevinE andreykurilin__:  Then "kwargs["interface"] = kwargs["endpoint_type"]" --> "kwargs["interface"] = kwargs["keystone_endpoint_type"]" ?
22:46 andreykurilin__ Yes
22:46 KevinE andreykurilin__: I'm not sure what you mean by the credentials object accepting the new variable :)
22:46 KevinE andreykurilin__: I'm learning I promise, I'm just learning from 0.000
22:46 andreykurilin__ https://github.com/openstack/rally/blob/master/rally/common/objects/credential.py#L21
22:47 andreykurilin__ You should add new argument in init method
22:48 andreykurilin__ Also, to_dict method of Credential object should be extended
22:49 KevinE andreykurilin__: http://pastebin.com/Abq5cxXE lines 6 and 32?
22:51 andreykurilin__ Credential instance should include this variable too, so look at L16
22:52 andreykurilin__ After you finish this, try to launch 'tox -epy27' locally. It will launch unit tests. I think they will fail after your modifications;) try to fix them. If you will not succeed, push change without fixing unit tests and I'll help at gerrit
22:52 andreykurilin__ And now I need to go sleep, it is too late
22:52 KevinE so insert a "self.keystone_endpoint_type = keystone_endpoint_type"
22:53 KevinE andreykurilin__: okay I'll work on it, and bug you tomorrow! hehe
23:16 KevinE_ joined #openstack-rally

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