Wednesday, 2016-06-22

« Tuesday, 2016-06-21 Index Thursday, 2016-06-23 »
*** yingjun has joined #openstack-rally00:12
*** maeca1 has quit IRC00:34
*** catintheroof has joined #openstack-rally00:47
*** kuladeep has joined #openstack-rally00:50
openstackgerrityuyafei proposed openstack/rally: Make dict.keys() PY3 compatible  https://review.openstack.org/33259301:02
*** catintheroof has quit IRC01:07
*** KevinE has quit IRC01:10
*** raddaoui has quit IRC01:17
*** yanyanhu has joined #openstack-rally01:39
*** ilbot3 has quit IRC01:45
*** dpaterson has quit IRC01:48
*** ilbot3 has joined #openstack-rally01:48
*** pvaneck has quit IRC01:51
*** yingjun has quit IRC02:00
*** yingjun has joined #openstack-rally02:00
*** yingjun has quit IRC02:02
*** yingjun has joined #openstack-rally02:03
*** yingjun has quit IRC02:07
*** yamamot__ has joined #openstack-rally02:16
*** kuladeep has quit IRC02:32
*** caowei has joined #openstack-rally02:57
*** yingjun has joined #openstack-rally03:01
*** yingjun has quit IRC03:16
*** yingjun has joined #openstack-rally03:17
openstackgerritlvfm proposed openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/33202103:23
openstackgerritlvfm proposed openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/33202103:33
*** M00nr41n has joined #openstack-rally03:44
*** catintheroof has joined #openstack-rally03:44
*** dave-mcc_ has quit IRC03:48
*** njohnston|afk is now known as njohnston04:01
*** catintheroof has quit IRC04:02
*** yamamot__ has quit IRC04:10
*** jrcloud has quit IRC04:15
*** yingjun has quit IRC04:18
*** yingjun has joined #openstack-rally04:18
*** yingjun has quit IRC04:22
*** M00nr41n has quit IRC04:30
*** shausy has joined #openstack-rally04:52
*** yamamot__ has joined #openstack-rally04:53
*** yingjun has joined #openstack-rally04:54
*** rcernin has joined #openstack-rally04:56
openstackgerritlvfm proposed openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/33202104:58
*** njohnston is now known as njohnston|afk05:01
*** caowei has quit IRC05:01
*** KevinE has joined #openstack-rally05:10
*** KevinE has quit IRC05:15
*** rcernin has quit IRC05:16
*** yingjun has quit IRC05:22
*** yingjun has joined #openstack-rally05:23
*** M00nr41n has joined #openstack-rally05:23
*** yingjun has quit IRC05:27
*** caowei has joined #openstack-rally05:30
*** yingjun has joined #openstack-rally05:45
*** rcernin has joined #openstack-rally05:55
*** caowei has quit IRC05:56
*** rdas has joined #openstack-rally06:13
*** caowei has joined #openstack-rally06:36
*** tesseract- has joined #openstack-rally06:47
*** rvasilets_ has joined #openstack-rally06:48
*** igajsin1 has joined #openstack-rally06:48
*** dmellado_ has joined #openstack-rally06:52
*** yanyanhu has quit IRC07:04
*** rvasilets__ has quit IRC07:04
*** dmellado has quit IRC07:04
*** permalac_ has quit IRC07:06
*** amaretskiy has joined #openstack-rally07:10
*** yanyanhu has joined #openstack-rally07:14
*** pcaruana has joined #openstack-rally07:22
*** adiantum has joined #openstack-rally07:26
*** ushkalim has joined #openstack-rally07:35
*** adiantum has quit IRC07:51
*** adiantum1 has joined #openstack-rally07:51
*** adiantum1 is now known as adiantum07:54
*** e0ne has joined #openstack-rally08:06
*** adiantum has quit IRC08:09
*** yingjun has quit IRC08:11
*** yingjun has joined #openstack-rally08:11
*** alexchadin has joined #openstack-rally08:35
*** bapalm has quit IRC08:37
*** bapalm has joined #openstack-rally08:42
*** yingjun has quit IRC08:43
*** yingjun has joined #openstack-rally08:44
*** yingjun has quit IRC08:48
*** mvk has joined #openstack-rally08:52
*** abalutoiu has joined #openstack-rally08:53
*** yingjun has joined #openstack-rally08:54
*** anshul has joined #openstack-rally09:07
*** anshul is now known as Guest4082109:07
*** igajsin1 has left #openstack-rally09:14
*** yingjun has quit IRC09:23
*** yingjun has joined #openstack-rally09:23
*** adiantum has joined #openstack-rally09:30
*** mvk has quit IRC09:35
*** tosky has joined #openstack-rally09:44
*** yingjun has quit IRC09:46
*** yingjun has joined #openstack-rally09:47
*** bapalm has quit IRC09:48
*** yingjun has quit IRC09:53
*** bapalm has joined #openstack-rally09:54
*** mvk has joined #openstack-rally09:56
*** alexchadin has quit IRC10:01
openstackgerritMerged openstack/rally: Make dict.keys() PY3 compatible  https://review.openstack.org/33259310:19
*** caowei has quit IRC10:24
*** yanyanhu has quit IRC10:30
*** caowei has joined #openstack-rally11:16
*** shausy has quit IRC11:19
*** zackf has joined #openstack-rally11:27
*** zackf has quit IRC11:31
*** ushkalim has quit IRC11:50
*** yamamot__ has quit IRC11:57
*** ushkalim has joined #openstack-rally12:03
*** e0ne has quit IRC12:13
*** e0ne has joined #openstack-rally12:14
*** maeca1 has joined #openstack-rally12:15
*** dave-mccowan has joined #openstack-rally12:19
*** shausy has joined #openstack-rally12:21
*** rdas has quit IRC12:23
*** alexchadin has joined #openstack-rally12:37
*** alexchadin has quit IRC12:41
*** alexchadin has joined #openstack-rally12:41
*** yamamoto has joined #openstack-rally13:02
*** permalac has joined #openstack-rally13:07
*** njohnston|afk is now known as njohnston13:30
openstackgerritIllia Khudoshyn proposed openstack/rally: Fix 'existing_users' context against keystone v3  https://review.openstack.org/33220413:55
*** dpaterson has joined #openstack-rally13:58
*** tosky has quit IRC14:18
*** tosky has joined #openstack-rally14:20
*** arnoldje has joined #openstack-rally14:26
*** KevinE has joined #openstack-rally14:35
*** abalutoiu has quit IRC14:36
*** Guest40821 has quit IRC14:43
*** alexchadin has quit IRC14:55
*** bkopilov has quit IRC14:59
openstackgerritIllia Khudoshyn proposed openstack/rally: Fix 'existing_users' context against keystone v3  https://review.openstack.org/33220415:01
*** permalac has quit IRC15:08
*** rcernin has quit IRC15:09
*** bapalm has quit IRC15:10
*** bapalm has joined #openstack-rally15:10
*** yarkot1 has quit IRC15:10
*** tesseract- has quit IRC15:10
*** yarkot1 has joined #openstack-rally15:11
*** bkopilov has joined #openstack-rally15:16
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670115:17
*** pcaruana has quit IRC15:22
*** rcernin has joined #openstack-rally15:24
openstackgerritAndrey Kurilin proposed openstack/rally: [ci] rename rally-gate.py to wip-rally-gate.py  https://review.openstack.org/33287315:29
*** caowei has quit IRC15:32
*** pcaruana has joined #openstack-rally15:36
*** zackf has joined #openstack-rally15:41
*** adiantum has quit IRC15:41
*** catintheroof has joined #openstack-rally15:49
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670115:58
*** amaretskiy has quit IRC16:04
*** adiantum has joined #openstack-rally16:04
*** adiantum has quit IRC16:10
*** adiantum has joined #openstack-rally16:13
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670116:18
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670116:20
*** dpaterson has quit IRC16:27
*** e0ne has quit IRC16:37
*** rcernin has quit IRC16:40
*** pcaruana has quit IRC16:43
*** raddaoui has joined #openstack-rally16:44
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670116:45
*** shausy has quit IRC16:58
*** sai has joined #openstack-rally17:01
*** sai has quit IRC17:02
*** mvk has quit IRC17:11
*** rcernin has joined #openstack-rally17:20
*** alexchadin has joined #openstack-rally17:28
*** tosky has quit IRC17:36
*** alexchadin has quit IRC17:37
*** mvk has joined #openstack-rally17:41
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670117:56
*** yamamoto has quit IRC18:23
*** e0ne has joined #openstack-rally18:30
*** adiantum has quit IRC18:38
*** mvk_ has joined #openstack-rally18:53
*** mvk has quit IRC18:55
*** mkrcmari__ has joined #openstack-rally19:02
*** mvk_ has quit IRC19:02
*** sshnaidm_ has quit IRC19:08
openstackgerritBoris Pavlovic proposed openstack/rally: [spec] Making DB scalable and flexible enough  https://review.openstack.org/16706019:12
*** e0ne has quit IRC19:12
*** mkrcmari__ has quit IRC19:15
*** mvk has joined #openstack-rally19:16
rookKevinE getting back to your question, I don't feel like that is a better solution.19:17
KevinErook: 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:18
openstackgerritKevin Esensoy proposed openstack/rally: [DO NOT MERGE] Add scenarios to VM  https://review.openstack.org/32670119:22
KevinErook: 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 do19:23
*** yamamoto has joined #openstack-rally19:24
*** pvaneck has joined #openstack-rally19:28
*** yamamoto has quit IRC19:34
openstackgerritMerged openstack/rally: [ci] rename rally-gate.py to wip-rally-gate.py  https://review.openstack.org/33287320:00
*** mvk_ has joined #openstack-rally20:01
openstackgerritMerged openstack/rally: Stopping server is redundant before cold-migrating server  https://review.openstack.org/33202120:02
*** mvk has quit IRC20:05
openstackgerritMerged openstack/rally: [Core] Introduce class-based scenarios  https://review.openstack.org/32282520:12
openstackgerritMerged openstack/rally: [spec] Making DB scalable and flexible enough  https://review.openstack.org/16706020:12
*** ushkalim has quit IRC20:20
*** rook has quit IRC20:51
*** rook has joined #openstack-rally20:51
andreykurilin__KevinE: sorry for not-reviewing. will try to look at your patch tomorrow20:56
KevinEandreykurilin__: 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 others20:57
KevinEandreykurilin__: Hopefully I'll have it pretty cleaned up tonight so it makes sense :)20:57
openstackgerritMerged openstack/rally: [Plugins] Improve scenario output from boot_runcommand_delete  https://review.openstack.org/32836920:59
*** rook has quit IRC21:01
*** rook has joined #openstack-rally21:01
*** abalutoiu has joined #openstack-rally21:39
openstackgerritBoris Pavlovic proposed openstack/rally: Use pytest instead of testr  https://review.openstack.org/32867221:39
openstackgerritKevin Esensoy proposed openstack/rally: Add scenarios to VM  https://review.openstack.org/32670121:44
*** zackf has quit IRC21:44
openstackgerritKevin Esensoy proposed openstack/rally: Add scenarios to VM  https://review.openstack.org/32670121:46
openstackgerritBoris Pavlovic proposed openstack/rally: [Plugins] Class-based scenario DummyFailure  https://review.openstack.org/32282921:56
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 week22:04
KevinEandreykurilin__: uhh how, didn't you say you had stuff done on it too?22:04
andreykurilin__I know how to fix it:)22:05
KevinEandreykurilin__: well I don't :P you kind of ran off and did your own thing22:05
andreykurilin__KevinE: heh. Yeah, I tried several changes before I got the final idea of fix.22:06
KevinEandreykurilin__: 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?" :p22:08
*** zackf has joined #openstack-rally22:10
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" lol22:11
*** zackf has quit IRC22:15
andreykurilin__KevinE: okay, I'll try to find someone else22:18
KevinEandreykurilin__: 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
KevinEandreykurilin__: I would absolutely love to do it. Am I ABLE to do it? I have no clue tell me what your plan is!22:18
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:21
*** arnoldje has quit IRC22:22
KevinEandreykurilin__: I don't even know what to say lol22: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 rally22:24
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 change22:25
andreykurilin__KevinE: the first part of change- extend config schema like https://github.com/openstack/rally/blob/master/rally/deployment/engines/existing.py#L99-10222:28
andreykurilin__and transmit new variable into Credentials object at L12422:28
andreykurilin__Also you need to extend Credentials object with new variable22:29
andreykurilin__And, of course, you need to transmit this new variable into keystoneclient like in your previous change22:29
andreykurilin__Btw, you will need to fix unit tests, but I'll help you with suggestions22:30
andreykurilin__KevinE: how does it sound for you?:)22:31
KevinEandreykurilin__: I understood like half of that :/22:34
*** pvaneck has quit IRC22:34
KevinEso 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:34
KevinEandreykurilin__: That'll take like 2 minutes so I must be missing something :)22:35
andreykurilin__KevinE: almost. You shout not change Credentials's endpoint_type, just add new variable there22:36
andreykurilin__*should22:36
andreykurilin__Yeah, this change is not so big, but you will need to fix existing tests22:37
*** adiantum has joined #openstack-rally22:37
andreykurilin__Btw, sorry for my English:)22:38
KevinEandreykurilin__: pshh, better than my *insert your native language*22:38
andreykurilin__Heh22:39
andreykurilin__KevinE: so can I hope to see a fix from you?)22:40
KevinEandreykurilin__: http://pastebin.com/reQb3FaG22:43
KevinEandreykurilin__: lines 77 and 109?22:43
andreykurilin__This part is correct22:44
andreykurilin__Also, credentials object should accept this new variable22:45
KevinEandreykurilin__:  Then "kwargs["interface"] = kwargs["endpoint_type"]" --> "kwargs["interface"] = kwargs["keystone_endpoint_type"]" ?22:45
andreykurilin__Yes22:46
KevinEandreykurilin__: I'm not sure what you mean by the credentials object accepting the new variable :)22:46
KevinEandreykurilin__: I'm learning I promise, I'm just learning from 0.00022:46
andreykurilin__https://github.com/openstack/rally/blob/master/rally/common/objects/credential.py#L2122:47
andreykurilin__You should add new argument in init method22:47
andreykurilin__Also, to_dict method of Credential object should be extended22:48
KevinEandreykurilin__: http://pastebin.com/Abq5cxXE lines 6 and 32?22:49
andreykurilin__Credential instance should include this variable too, so look at L1622:51
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 gerrit22:52
andreykurilin__And now I need to go sleep, it is too late22:52
KevinEso insert a "self.keystone_endpoint_type = keystone_endpoint_type"22:52
KevinEandreykurilin__: okay I'll work on it, and bug you tomorrow! hehe22:53
*** rcernin has quit IRC23:00
*** KevinE_ has joined #openstack-rally23:16
*** KevinE_ has quit IRC23:18
*** KevinE has quit IRC23:19
*** catintheroof has quit IRC23:35
« Tuesday, 2016-06-21 Index Thursday, 2016-06-23 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!