| *** adiantum has quit IRC | 00:07 | |
| *** pvaneck has quit IRC | 00:11 | |
| *** yanyanhu has joined #openstack-rally | 00:23 | |
| *** rook has quit IRC | 00:30 | |
| *** rook has joined #openstack-rally | 00:32 | |
| *** madhu_ak has quit IRC | 00:56 | |
| *** kjorgensen has quit IRC | 01:02 | |
| *** kjorgensen has joined #openstack-rally | 01:04 | |
| *** kjorgens_ has joined #openstack-rally | 01:08 | |
| openstackgerrit | Merged openstack/rally: [Common] Move code from utils.percentile to PercentileComputation https://review.openstack.org/304595 | 01:09 |
|---|---|---|
| *** kjorgensen has quit IRC | 01:09 | |
| *** kjorgens_ has quit IRC | 01:12 | |
| *** vkmc is now known as Guest9874 | 01:20 | |
| *** soumitm has quit IRC | 01:21 | |
| *** Guest9874 is now known as cmkv | 01:22 | |
| *** cmkv is now known as violenciarivas | 01:22 | |
| *** violenciarivas is now known as vkmc | 01:23 | |
| *** psuriset has quit IRC | 01:39 | |
| *** ilbot3 has quit IRC | 01:45 | |
| *** ilbot3 has joined #openstack-rally | 01:48 | |
| *** anshul has joined #openstack-rally | 01:58 | |
| *** anshul is now known as Guest32715 | 01:59 | |
| *** Guest32715 has quit IRC | 02:03 | |
| *** akrzos has quit IRC | 02:06 | |
| *** arnoldje has joined #openstack-rally | 02:16 | |
| *** abehl has joined #openstack-rally | 02:19 | |
| openstackgerrit | Merged openstack/rally: Fix load duration calulation in case of 0 iterations were run https://review.openstack.org/303560 | 02:40 |
| *** akrzos has joined #openstack-rally | 03:06 | |
| *** kjorgensen has joined #openstack-rally | 03:07 | |
| *** kjorgensen has quit IRC | 03:12 | |
| *** akrzos has quit IRC | 03:12 | |
| *** abehl has quit IRC | 03:16 | |
| *** Alex_Stef has quit IRC | 03:23 | |
| *** psuriset has joined #openstack-rally | 03:25 | |
| *** Alex_Stef has joined #openstack-rally | 03:35 | |
| *** psuriset has quit IRC | 03:41 | |
| *** psuriset has joined #openstack-rally | 03:41 | |
| *** e0ne has joined #openstack-rally | 03:59 | |
| *** e0ne has quit IRC | 04:01 | |
| *** e0ne has joined #openstack-rally | 04:02 | |
| *** e0ne has quit IRC | 04:05 | |
| *** akrzos has joined #openstack-rally | 04:08 | |
| *** sverma has joined #openstack-rally | 04:11 | |
| *** akrzos has quit IRC | 04:13 | |
| *** arnoldje has quit IRC | 04:40 | |
| *** amit213 has quit IRC | 04:44 | |
| *** shausy has joined #openstack-rally | 05:11 | |
| *** rdas has joined #openstack-rally | 05:13 | |
| *** kjorgensen has joined #openstack-rally | 06:09 | |
| *** akrzos has joined #openstack-rally | 06:10 | |
| *** kjorgensen has quit IRC | 06:14 | |
| *** akrzos has quit IRC | 06:14 | |
| *** openstackgerrit has quit IRC | 06:17 | |
| *** openstackgerrit has joined #openstack-rally | 06:18 | |
| *** goutham has joined #openstack-rally | 06:25 | |
| *** Alex_Stef has quit IRC | 06:28 | |
| *** e0ne has joined #openstack-rally | 06:41 | |
| *** goutham has quit IRC | 06:57 | |
| *** soumitm has joined #openstack-rally | 06:57 | |
| *** tesseract has joined #openstack-rally | 07:01 | |
| *** tesseract is now known as Guest23319 | 07:01 | |
| *** ushkalim has joined #openstack-rally | 07:03 | |
| *** arxcruz has joined #openstack-rally | 07:09 | |
| *** amaretskiy has joined #openstack-rally | 07:32 | |
| *** mvk_ has joined #openstack-rally | 08:23 | |
| *** mvk has quit IRC | 08:26 | |
| *** Alex_Stef has joined #openstack-rally | 08:34 | |
| *** abehl has joined #openstack-rally | 08:36 | |
| *** Alex_Stef has quit IRC | 08:39 | |
| *** adiantum has joined #openstack-rally | 08:41 | |
| *** yanyanhu has quit IRC | 08:49 | |
| *** Alex_Stef has joined #openstack-rally | 08:55 | |
| *** abalutoiu has joined #openstack-rally | 09:08 | |
| *** kbaikov has quit IRC | 09:12 | |
| *** adiantum has quit IRC | 09:13 | |
| *** soumitm has quit IRC | 09:23 | |
| *** prithiv has joined #openstack-rally | 09:28 | |
| prithiv | Hi | 09:28 |
| prithiv | i tried creating my environment with existing users but am getting error. anyone facing the same issue | 09:30 |
| *** kbaikov has joined #openstack-rally | 09:33 | |
| *** kbaikov has quit IRC | 09:42 | |
| *** kbaikov has joined #openstack-rally | 09:42 | |
| *** prithiv has quit IRC | 09:45 | |
| *** zackf has quit IRC | 09:47 | |
| *** tosky has joined #openstack-rally | 09:50 | |
| *** yingjun has quit IRC | 09:57 | |
| *** Alex_Stef has quit IRC | 09:59 | |
| *** prithiv has joined #openstack-rally | 10:06 | |
| *** e0ne has quit IRC | 10:06 | |
| *** e0ne has joined #openstack-rally | 10:06 | |
| *** prithiv has quit IRC | 10:08 | |
| *** prithiv has joined #openstack-rally | 10:09 | |
| *** prithiv has quit IRC | 10:10 | |
| *** prithiv has joined #openstack-rally | 10:10 | |
| *** prithiv has quit IRC | 10:11 | |
| *** Alex_Stef has joined #openstack-rally | 10:12 | |
| *** prithiv has joined #openstack-rally | 10:12 | |
| *** ylobankov has joined #openstack-rally | 10:40 | |
| openstackgerrit | Yaroslav Lobankov proposed openstack/rally: [Verify] Add 'version' arg to install/reinstall cmd https://review.openstack.org/310503 | 10:42 |
| *** adiantum has joined #openstack-rally | 10:45 | |
| *** mvk_ has quit IRC | 10:46 | |
| *** prithiv has quit IRC | 10:47 | |
| *** akrzos has joined #openstack-rally | 10:57 | |
| *** prithiv has joined #openstack-rally | 10:58 | |
| *** prithiv has quit IRC | 10:59 | |
| *** abalutoiu has quit IRC | 11:03 | |
| *** prithiv has joined #openstack-rally | 11:04 | |
| *** akrzos has quit IRC | 11:05 | |
| *** prithiv has quit IRC | 11:05 | |
| *** stpierre has joined #openstack-rally | 11:20 | |
| *** MaxPC has joined #openstack-rally | 11:28 | |
| *** mvk_ has joined #openstack-rally | 11:54 | |
| *** akrzos has joined #openstack-rally | 11:58 | |
| *** e0ne has quit IRC | 12:05 | |
| *** abehl has quit IRC | 12:09 | |
| *** prithiv has joined #openstack-rally | 12:12 | |
| *** davidlenwell has quit IRC | 12:13 | |
| *** davidlenwell has joined #openstack-rally | 12:15 | |
| *** e0ne has joined #openstack-rally | 12:22 | |
| *** asti has joined #openstack-rally | 12:22 | |
| *** shausy has quit IRC | 12:26 | |
| *** prithiv has quit IRC | 12:28 | |
| *** prithiv has joined #openstack-rally | 12:30 | |
| *** abalutoiu has joined #openstack-rally | 12:38 | |
| *** rook has quit IRC | 12:42 | |
| *** prithiv has quit IRC | 13:18 | |
| *** abalutoiu has quit IRC | 13:24 | |
| *** diogogmt has quit IRC | 13:28 | |
| *** diogogmt has joined #openstack-rally | 13:32 | |
| *** arnoldje has joined #openstack-rally | 13:41 | |
| *** abalutoiu_ has joined #openstack-rally | 13:42 | |
| *** diogogmt has quit IRC | 13:43 | |
| *** arnoldje has quit IRC | 13:45 | |
| *** rdas has quit IRC | 13:49 | |
| *** rook has joined #openstack-rally | 13:55 | |
| *** prithiv has joined #openstack-rally | 14:01 | |
| *** diogogmt has joined #openstack-rally | 14:03 | |
| *** prithiv has quit IRC | 14:09 | |
| *** arnoldje has joined #openstack-rally | 14:11 | |
| openstackgerrit | Andrey Kurilin proposed openstack/rally: Remove unused requirements https://review.openstack.org/312528 | 14:14 |
| *** yingjun has joined #openstack-rally | 14:15 | |
| *** absubram has joined #openstack-rally | 14:16 | |
| *** tosky has quit IRC | 14:21 | |
| *** tosky has joined #openstack-rally | 14:24 | |
| *** prithiv has joined #openstack-rally | 14:32 | |
| *** Alex_Stef has quit IRC | 14:34 | |
| *** e0ne has quit IRC | 14:50 | |
| *** yingjun has quit IRC | 15:14 | |
| *** yingjun has joined #openstack-rally | 15:14 | |
| *** sverma has quit IRC | 15:15 | |
| *** sverma has joined #openstack-rally | 15:15 | |
| *** absubram has quit IRC | 15:17 | |
| *** absubram has joined #openstack-rally | 15:18 | |
| *** yingjun has quit IRC | 15:18 | |
| *** absubram has quit IRC | 15:19 | |
| *** lakerzhou has joined #openstack-rally | 15:27 | |
| lakerzhou | hello, I am new to rally. after install/config, I had no issues with "rally show images", but always got this error "ERROR rally.task.engine [-] Unable to setup context 'users': 'Failed to create the requested number of tenants.". | 15:30 |
| lakerzhou | can any one help me here please? | 15:31 |
| *** absubram has joined #openstack-rally | 15:34 | |
| lakerzhou | Hi Absubram | 15:38 |
| lakerzhou | I am new to rally | 15:38 |
| *** asti has quit IRC | 15:42 | |
| stpierre | lakerzhou: there should be a more detailed error message a few lines before that | 15:43 |
| *** lakerzhou has quit IRC | 15:45 | |
| *** davideagnello has joined #openstack-rally | 15:46 | |
| *** Guest23319 has quit IRC | 15:46 | |
| *** asti has joined #openstack-rally | 15:47 | |
| *** rook has quit IRC | 15:48 | |
| *** e0ne has joined #openstack-rally | 15:49 | |
| *** prithiv has quit IRC | 15:58 | |
| *** davideagnello has quit IRC | 15:59 | |
| *** prithiv has joined #openstack-rally | 16:00 | |
| prithiv | hi | 16:01 |
| prithiv | does rally create a user every time i run a task | 16:01 |
| prithiv | like boot_and_delete_server | 16:01 |
| *** davideagnello has joined #openstack-rally | 16:01 | |
| openstackgerrit | Merged openstack/rally: Remove unused requirements https://review.openstack.org/312528 | 16:02 |
| prithiv | can i make it use just the existing admin user ? | 16:02 |
| stpierre | yes, it does. you can run it with existing users if you really want to, but if you have admin then letting it create its own ephemeral users and tenants is unquestionably better -- it's more isolated, and better tested | 16:03 |
| stpierre | http://rally.readthedocs.io/en/latest/tutorial/step_3_benchmarking_with_existing_users.html | 16:04 |
| prithiv | i saw the link. when i try the same way, its throwing error about having no user section in the context | 16:04 |
| prithiv | @stpierre have you tried it before | 16:04 |
| stpierre | nope | 16:04 |
| stpierre | i think you need to provide an empty users context | 16:05 |
| prithiv | yes i did. still same error | 16:05 |
| *** amaretskiy has quit IRC | 16:05 | |
| stpierre | that makes no sense | 16:05 |
| stpierre | can you pastebin your task file, and the exact error you're getting from rally? | 16:05 |
| prithiv | please give me five minutes. i will run it now again and will paste it | 16:05 |
| *** kjorgensen has joined #openstack-rally | 16:14 | |
| prithiv | Reason: | 16:19 |
| prithiv | Invalid scenario argument: 'You should specify 'users' context'` | 16:19 |
| prithiv | this is the error | 16:19 |
| prithiv | i followed the same link you provided.. | 16:20 |
| stpierre | okay, now please pastebin your task file | 16:21 |
| *** Serlex has joined #openstack-rally | 16:21 | |
| openstackgerrit | Zack Feldstein proposed openstack/rally: Allow non-admin users to run tasks https://review.openstack.org/312625 | 16:25 |
| Serlex | hi, does one need to run any cleanup post running "rally verify start..."? | 16:26 |
| *** madhu_ak has joined #openstack-rally | 16:26 | |
| *** Jeffrey4l has joined #openstack-rally | 16:28 | |
| *** sverma has quit IRC | 16:30 | |
| *** arxcruz has quit IRC | 16:30 | |
| *** abalutoiu has joined #openstack-rally | 16:33 | |
| *** adiantum has quit IRC | 16:34 | |
| *** abalutoiu_ has quit IRC | 16:35 | |
| prithiv | http://paste.openstack.org/show/496103/ | 16:36 |
| prithiv | my task file | 16:36 |
| prithiv | apologies if i have missed anything | 16:36 |
| *** psuriset has quit IRC | 16:37 | |
| prithiv | any update ? | 16:43 |
| smurke_ | prithiv: just out of curiosity is your keystone running on port 35357 ? can you also paste your scenario config ? | 16:43 |
| stpierre | that's not your task file | 16:43 |
| stpierre | that's your deployment file | 16:43 |
| stpierre | paste the file that you give to rally with --task | 16:43 |
| prithiv | i am sorry i am new to rally. what do you mean by scenario file. i created the env using —fromenv option | 16:44 |
| stpierre | when you ran 'rally task start', you gave it a task file | 16:44 |
| stpierre | 'rally task start --task <some filename here>' | 16:44 |
| stpierre | that file | 16:44 |
| stpierre | pastebin it | 16:44 |
| prithiv | that is the one i [pasted above | 16:44 |
| stpierre | uh | 16:44 |
| stpierre | well then we've found your problem | 16:44 |
| prithiv | rally task start existing.json | 16:44 |
| prithiv | thank you so mucn... | 16:44 |
| stpierre | right | 16:44 |
| prithiv | what was it... | 16:45 |
| stpierre | you need to pass it a task file, something like these: https://github.com/openstack/rally/tree/master/samples/tasks | 16:45 |
| *** e0ne has quit IRC | 16:46 | |
| smurke_ | like you want to boot_delete_server so you want to pass https://github.com/openstack/rally/blob/master/samples/tasks/scenarios/nova/boot-and-delete.json but keep your context empty | 16:46 |
| *** bkopilov has quit IRC | 16:46 | |
| *** e0ne has joined #openstack-rally | 16:47 | |
| prithiv | if you dont mind can you send few examples | 16:47 |
| prithiv | this link https://github.com/openstack/rally/tree/master/samples/tasks/scenarios/nova | 16:47 |
| prithiv | i am using boot.json | 16:47 |
| prithiv | i ran | 16:47 |
| prithiv | rally task start boot.json | 16:48 |
| prithiv | i was getting the above error at that time only. in that file only i have given empty context (hope that was the mistake) | 16:48 |
| stpierre | yes, you need to pass a users context. it can be empty if you're using existing users, but it needs to be there | 16:49 |
| prithiv | i have an existing user admin (which is openstack user) | 16:49 |
| prithiv | if am passing empty context, it should be using my admin user rite… if am not mistaken | 16:49 |
| smurke_ | yes, probably you want to do is http://paste.openstack.org/show/496107/ | 16:50 |
| smurke_ | if you are using boot.json | 16:51 |
| stpierre | you cannot pass a completely empty context | 16:51 |
| stpierre | rally requires you to pass some sort of users context | 16:51 |
| prithiv | okay.. can you also give me your existing.json. just to see my mistakes... | 16:52 |
| prithiv | an example of existing.json | 16:52 |
| *** zackf has joined #openstack-rally | 16:52 | |
| stpierre | your existing.json is fine | 16:52 |
| stpierre | you can either a) use boot.json as-is, in which case rally will create three ephemeral tenants and two users in each tenant, and use those users and tenants to boot the servers; or b) something like http://paste.openstack.org/show/496108/ | 16:53 |
| stpierre | but the users context is required | 16:53 |
| stpierre | at least, i think your existing.json is fine. i' | 16:53 |
| stpierre | i'm not sure what rally will do if you provide both existing users and admin creds | 16:53 |
| stpierre | i guess that's in the example, though | 16:53 |
| prithiv | can i use admin as my existing user ? | 16:53 |
| stpierre | so it should be fine | 16:53 |
| stpierre | yes, but why? | 16:54 |
| stpierre | you're going to a lot of work here when you don't need to. just let rally create ephemeral tenants and users | 16:54 |
| stpierre | unless your keystone is backed by LDAP, or you have some technical reason that you cannot create users, just let rally create users | 16:54 |
| prithiv | yes but when rally creates a user, there will be a default quota limit of 10 or 40 instances per user/tenant | 16:54 |
| prithiv | i want to see scalability since am using a fake driver | 16:55 |
| stpierre | you can change quotas on the fly with rally | 16:55 |
| stpierre | there's a quotas context | 16:55 |
| prithiv | oh is it.. i did not see that. | 16:55 |
| prithiv | Quotas.nova_update ? | 16:56 |
| stpierre | e.g., https://github.com/openstack/rally/blob/master/samples/tasks/contexts/large-ops/create-volume-large-scale.yaml#L14-L16 | 16:56 |
| stpierre | no, that's a scenario that tests updating quotas | 16:56 |
| prithiv | thank you. this is the one i was exactly looking for. also, if i have more than 2 or 3 networks, while creating a server, is it possible to specify the network i want to use | 16:57 |
| stpierre | or: https://github.com/openstack/rally/blob/master/rally-jobs/rally.yaml#L508-L510 | 16:57 |
| stpierre | yes, you can pass it the network id | 16:57 |
| stpierre | let me see if i can find an example | 16:57 |
| prithiv | thanks | 16:57 |
| stpierre | doesn't look like we have one | 16:58 |
| stpierre | i always just use auto_assign_nic: True, which just picks a network | 16:58 |
| stpierre | ultimately, every scenario that boots a server has a way to pass arbitrary keyword arguments to the underlying nova.servers.create() call | 16:59 |
| stpierre | i can't remember the exact format, but i think it's something like: "nics": [{"net-id": "some-network-uuid"}] | 16:59 |
| prithiv | okay. i will see if there is any way… but i have seen example of auto_assign_nic… | 16:59 |
| stpierre | where you need to pass that will depend on the scenario | 16:59 |
| prithiv | thanks :) | 16:59 |
| prithiv | that was much helpful | 16:59 |
| *** e0ne has quit IRC | 17:01 | |
| *** e0ne has joined #openstack-rally | 17:02 | |
| *** prithiv has quit IRC | 17:07 | |
| *** tosky_ has joined #openstack-rally | 17:08 | |
| *** tosky has quit IRC | 17:09 | |
| *** bigdogstl has joined #openstack-rally | 17:09 | |
| *** bigdogstl has quit IRC | 17:15 | |
| *** prithiv has joined #openstack-rally | 17:17 | |
| *** prithiv has quit IRC | 17:19 | |
| *** prithiv has joined #openstack-rally | 17:20 | |
| *** soumitm has joined #openstack-rally | 17:20 | |
| *** prithiv has quit IRC | 17:20 | |
| *** diogogmt has quit IRC | 17:22 | |
| *** ushkalim has quit IRC | 17:23 | |
| *** diogogmt has joined #openstack-rally | 17:26 | |
| *** bigdogstl has joined #openstack-rally | 17:35 | |
| *** Serlex has quit IRC | 17:40 | |
| zackf | andreykurilin: you around? | 17:43 |
| *** pvaneck has joined #openstack-rally | 17:44 | |
| *** bigdogstl has quit IRC | 17:45 | |
| *** mvk_ has quit IRC | 17:53 | |
| *** tosky_ has quit IRC | 17:57 | |
| openstackgerrit | Piyush proposed openstack/rally: [WIP]Add create-flavor and flavor-access-list scenario https://review.openstack.org/302754 | 18:01 |
| *** psuriset has joined #openstack-rally | 18:08 | |
| *** diogogmt has quit IRC | 18:28 | |
| *** bigdogstl has joined #openstack-rally | 18:32 | |
| *** abalutoiu has quit IRC | 18:33 | |
| *** Jeffrey4l has quit IRC | 18:34 | |
| *** bigdogstl has quit IRC | 18:39 | |
| *** bigdogstl has joined #openstack-rally | 18:43 | |
| *** rook has joined #openstack-rally | 18:48 | |
| *** bigdogstl has quit IRC | 18:48 | |
| *** abehl has joined #openstack-rally | 18:57 | |
| *** mvk_ has joined #openstack-rally | 19:00 | |
| *** e0ne has quit IRC | 19:23 | |
| *** smurke is now known as Guest90714 | 19:27 | |
| *** smurke_ is now known as smurke | 19:27 | |
| *** bigdogstl has joined #openstack-rally | 19:28 | |
| *** bigdogstl has quit IRC | 19:33 | |
| *** mkrcmari__ has joined #openstack-rally | 19:40 | |
| *** mvk_ has quit IRC | 19:44 | |
| *** e0ne has joined #openstack-rally | 19:46 | |
| *** openstackgerrit has quit IRC | 19:48 | |
| *** openstackgerrit has joined #openstack-rally | 19:48 | |
| *** e0ne has quit IRC | 19:48 | |
| *** Alex_Stef has joined #openstack-rally | 19:53 | |
| *** abehl has quit IRC | 19:56 | |
| *** e0ne has joined #openstack-rally | 20:02 | |
| *** e0ne has quit IRC | 20:07 | |
| *** harlowja has quit IRC | 20:09 | |
| *** e0ne has joined #openstack-rally | 20:10 | |
| *** mvk_ has joined #openstack-rally | 20:16 | |
| *** mvk has joined #openstack-rally | 20:18 | |
| *** mkrcmari__ has quit IRC | 20:19 | |
| *** mvk_ has quit IRC | 20:21 | |
| *** mkrcmari__ has joined #openstack-rally | 20:21 | |
| *** bigdogstl has joined #openstack-rally | 20:22 | |
| *** mvk has quit IRC | 20:25 | |
| *** bigdogstl has quit IRC | 20:27 | |
| *** Alex_Stef has quit IRC | 20:28 | |
| *** akrzos has quit IRC | 20:34 | |
| *** e0ne has quit IRC | 20:37 | |
| *** bigdogstl has joined #openstack-rally | 20:38 | |
| *** psuriset has quit IRC | 20:40 | |
| *** mvk has joined #openstack-rally | 20:41 | |
| *** bigdogstl has quit IRC | 20:43 | |
| *** mkrcmari__ has quit IRC | 20:44 | |
| *** mvk_ has joined #openstack-rally | 20:44 | |
| *** mkrcmari__ has joined #openstack-rally | 20:47 | |
| *** mvk has quit IRC | 20:48 | |
| *** mvk_ has quit IRC | 20:50 | |
| *** mvk has joined #openstack-rally | 20:51 | |
| andreykurilin__ | zackf: hi | 20:51 |
| *** mkrcmari__ has quit IRC | 20:53 | |
| *** mvk_ has joined #openstack-rally | 20:53 | |
| *** mkrcmari__ has joined #openstack-rally | 20:56 | |
| *** mvk has quit IRC | 20:56 | |
| *** madhu_ak has quit IRC | 20:57 | |
| *** mvk_ has quit IRC | 20:59 | |
| *** dharinic_ has joined #openstack-rally | 21:00 | |
| *** mvk_ has joined #openstack-rally | 21:00 | |
| *** mkrcmari__ has quit IRC | 21:04 | |
| *** harlowja has joined #openstack-rally | 21:10 | |
| andreykurilin__ | zackf: I have not a lot of time to check existing users, but it looks like you need to specify some user(can be fake) as admin in rally deployment and it will not be used for non-admin scenarios if you specify a list of users in deployemnt | 21:11 |
| zackf | andreykurilin__: I figured out why i couldn't run as admin | 21:12 |
| zackf | There is a check in osclients that mandates your creds be apart of the admin role | 21:13 |
| zackf | I put a patch up: https://review.openstack.org/#/c/312625/ | 21:13 |
| zackf | There is some discussion on it currently | 21:13 |
| *** MaxPC has quit IRC | 21:13 | |
| stpierre | i was under the impression that there were still validation bits that required admin | 21:13 |
| andreykurilin__ | stpierre: we need to find and remove them all:) | 21:15 |
| andreykurilin__ | zackf: ok, will look | 21:15 |
| stpierre | easier said than done, AIUI :) | 21:15 |
| zackf | Thanks andreykurilin__ | 21:15 |
| andreykurilin__ | stpierre: will see | 21:18 |
| andreykurilin__ | :) | 21:18 |
| *** akrzos has joined #openstack-rally | 21:22 | |
| *** psuriset has joined #openstack-rally | 21:29 | |
| *** stpierre has quit IRC | 21:30 | |
| *** bigdogstl has joined #openstack-rally | 21:34 | |
| *** bigdogstl has quit IRC | 21:39 | |
| *** asti has quit IRC | 22:00 | |
| *** psuriset has quit IRC | 22:02 | |
| zackf | fwiw with my patch i was able to run tasks with non-admin. So i'm not sure if there are alot of other places that admin is required. | 22:02 |
| zackf | @stpierrer/ @andreykurilin__ | 22:03 |
| andreykurilin__ | zackf: do you specify "admin" field in deployment or not? If yes, is it a simple user? | 22:04 |
| zackf | yeah, i had to specify admin field, but it was a non-admin user | 22:05 |
| zackf | It belonged to demo tenant | 22:05 |
| andreykurilin__ | So, I think that your patch is doing the wrong thing | 22:06 |
| andreykurilin__ | We should not remove this check | 22:07 |
| andreykurilin__ | Admin user should be admin | 22:07 |
| andreykurilin__ | So first of all, we need to call this line https://github.com/openstack/rally/blob/master/rally/deployment/engines/existing.py#L146 only if admin field was specified | 22:08 |
| andreykurilin__ | The second change: move check from verified_keystone to somewhere else and check admin field with it | 22:10 |
| zackf | ah, ok, i see a bug that was opened a couple of days ago. | 22:10 |
| zackf | https://bugs.launchpad.net/rally/+bug/1577431 | 22:10 |
| openstack | Launchpad bug 1577431 in Rally "Rally existing cloud requires admin entry" [Wishlist,In progress] - Assigned to Boris Pavlovic (boris-42) | 22:10 |
| zackf | When i tried to remove admin entry, it wouldn't even create the deployment. | 22:10 |
| andreykurilin__ | Hm... | 22:10 |
| zackf | Probably due to that bug, which is why i thought i had to include the admin field in my deployment json | 22:11 |
| andreykurilin__ | Your solution looks like workaround:) | 22:11 |
| zackf | quite possible | 22:11 |
| zackf | :) | 22:11 |
| zackf | however | 22:12 |
| zackf | If you're just checking that a user belongs to the admin role | 22:12 |
| zackf | That seems like it might not be sufficient 100% of the time. | 22:12 |
| andreykurilin__ | Yeah, we not need to do it there | 22:12 |
| zackf | You can really define any name for an "admin" type role | 22:12 |
| zackf | and that check would cause issues, if someone had their policy files using something like "cloud_admin" for the true admin role | 22:13 |
| andreykurilin__ | It is true | 22:13 |
| andreykurilin__ | I think the better check is call something admin specific(for example list services via keystone, not via service catalog) and catch exception | 22:14 |
| *** absubram has quit IRC | 22:15 | |
| *** psuriset has joined #openstack-rally | 22:17 | |
| *** arnoldje has quit IRC | 22:19 | |
| zackf | For that check, what is the goal? If you're an admin then return the keystone client? | 22:19 |
| zackf | It looks to me, this check is just limiting using rally in general to running as admin | 22:20 |
| zackf | It doesn't seem to take into account the task you're running might not require admin. | 22:20 |
| andreykurilin__ | zackf: first of all, admin field should be optional. In this case, such check will test that admin user is a cloud admin and rally task will not fail with unexpected error( 403 error). | 22:28 |
| *** harlowja has quit IRC | 22:52 | |
| *** harlowja has joined #openstack-rally | 22:53 | |
| *** bapalm_ has quit IRC | 23:30 | |
| *** yingjun has joined #openstack-rally | 23:39 | |
| *** amit213 has joined #openstack-rally | 23:51 | |
| *** davideagnello has quit IRC | 23:51 | |
| *** davideagnello has joined #openstack-rally | 23:52 | |
| *** davideagnello has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!