*** pvaneck has quit IRC | 00:26 | |
openstackgerrit | Ton Ngo proposed openstack/rally: [Magnum] Context and scenario for Magnum baymodel https://review.openstack.org/309546 | 00:30 |
---|---|---|
*** akrzos has quit IRC | 00:50 | |
*** yingjun has joined #openstack-rally | 00:51 | |
*** akrzos has joined #openstack-rally | 00:56 | |
*** lakerzhou has joined #openstack-rally | 01:11 | |
*** zackf has joined #openstack-rally | 01:11 | |
*** zackf has quit IRC | 01:11 | |
*** zackf has joined #openstack-rally | 01:12 | |
*** lakerzhou has quit IRC | 01:15 | |
*** zackf has quit IRC | 01:17 | |
*** akrzos has quit IRC | 01:19 | |
*** Deng has joined #openstack-rally | 01:19 | |
*** akrzos has joined #openstack-rally | 01:21 | |
*** lakerzhou has joined #openstack-rally | 01:37 | |
*** yanyanhu has joined #openstack-rally | 01:40 | |
*** zackf has joined #openstack-rally | 01:44 | |
*** ilbot3 has quit IRC | 01:45 | |
*** ilbot3 has joined #openstack-rally | 01:48 | |
*** zackf has quit IRC | 01:49 | |
*** akrzos has quit IRC | 02:10 | |
*** bkopilov has quit IRC | 02:13 | |
*** arnoldje has joined #openstack-rally | 02:16 | |
*** zackf has joined #openstack-rally | 02:17 | |
*** zackf has quit IRC | 02:25 | |
*** absubram has joined #openstack-rally | 03:06 | |
*** absubram_ has joined #openstack-rally | 03:10 | |
*** absubram has quit IRC | 03:11 | |
*** absubram_ is now known as absubram | 03:11 | |
*** bkopilov has joined #openstack-rally | 03:35 | |
*** lakerzhou has quit IRC | 03:58 | |
*** zackf has joined #openstack-rally | 04:32 | |
*** zackf has quit IRC | 04:33 | |
*** zackf has joined #openstack-rally | 04:44 | |
openstackgerrit | Dave McCowan proposed openstack/rally: Use correct parameter name for CA file with heat client https://review.openstack.org/320776 | 04:47 |
*** zackf has quit IRC | 04:49 | |
*** dave-mccowan has quit IRC | 05:13 | |
openstackgerrit | Ton Ngo proposed openstack/rally: [Magnum] Scenario to create a Magnum bay https://review.openstack.org/309620 | 05:26 |
*** psuriset has quit IRC | 05:37 | |
*** psuriset has joined #openstack-rally | 05:52 | |
*** arnoldje has quit IRC | 05:58 | |
*** ig0r_ has joined #openstack-rally | 06:00 | |
*** psuriset has quit IRC | 06:01 | |
*** njohnston has quit IRC | 06:12 | |
*** psuriset has joined #openstack-rally | 06:13 | |
*** njohnston has joined #openstack-rally | 06:14 | |
*** ig0r_ has quit IRC | 06:17 | |
*** rcernin has joined #openstack-rally | 06:21 | |
*** yanyanhu has quit IRC | 06:24 | |
*** yanyanhu has joined #openstack-rally | 06:24 | |
*** Alex_Stef has joined #openstack-rally | 06:30 | |
*** yanyanhu has quit IRC | 06:37 | |
*** tesseract has joined #openstack-rally | 06:47 | |
*** e0ne has joined #openstack-rally | 06:52 | |
*** rdas has joined #openstack-rally | 06:52 | |
*** yanyanhu has joined #openstack-rally | 07:13 | |
*** amaretskiy has joined #openstack-rally | 07:14 | |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Plugins] Fix custom output from boot_runcommand_delete() https://review.openstack.org/320460 | 07:19 |
*** arxcruz has joined #openstack-rally | 07:21 | |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Plugins] Add deprecation mark to dummy scenario https://review.openstack.org/320365 | 07:44 |
*** anshul has joined #openstack-rally | 07:46 | |
*** anshul is now known as Guest50431 | 07:47 | |
*** lakerzhou has joined #openstack-rally | 07:54 | |
*** lakerzhou has quit IRC | 07:59 | |
*** ushkalim has joined #openstack-rally | 08:10 | |
*** ig0r_ has joined #openstack-rally | 08:20 | |
*** tosky has joined #openstack-rally | 08:26 | |
*** _hanhart has quit IRC | 08:32 | |
openstackgerrit | Merged openstack/rally: [Reports] Add Jinja2 template for upcoming Trends report https://review.openstack.org/298680 | 08:47 |
openstackgerrit | Merged openstack/rally: [Reports][CLI] Introduce Trends report https://review.openstack.org/276721 | 08:48 |
openstackgerrit | Chandidas Gharami proposed openstack/rally: rally-ci recheck https://review.openstack.org/314070 | 08:53 |
*** ushkalim has quit IRC | 09:04 | |
*** ushkalim has joined #openstack-rally | 09:17 | |
*** mvk has quit IRC | 09:22 | |
strigazi | Hi rally team, I have a question about adding magnum | 09:28 |
strigazi | from the first comment from Roman in https://review.openstack.org/#/c/309566/ I understand that we should first add a scenario for listing baymodels. Is this correct? | 09:30 |
*** alex____ has joined #openstack-rally | 09:31 | |
amaretskiy | strigazi: hi | 09:33 |
*** alex____ has left #openstack-rally | 09:33 | |
strigazi | amaretskiy: hi | 09:35 |
amaretskiy | strigazi: I'm currently having a look at your patches.... | 09:36 |
strigazi | amaretskiy: I'm waiting. We have only the magnum client merged. As you told be before, we need a job and a scenario | 09:37 |
amaretskiy | strigazi: yes, we need a way to prove that context or scenario really works | 09:39 |
amaretskiy | it is highly desirable to have context/scenario verified by real run | 09:40 |
*** madhu_ak has quit IRC | 09:40 | |
strigazi | amaretskiy: is it ok to start with a scenario that just lists baymodels? | 09:40 |
amaretskiy | strigazi: absolutely, it is ok | 09:40 |
strigazi | then we add context for baymodels and eventually add a scenario to create and delete baymodels | 09:41 |
*** redixin has joined #openstack-rally | 09:42 | |
strigazi | At the same (with creating the baymodel list scenario) we should add a job on openstack-infra/project-config | 09:42 |
amaretskiy | strigazi: I see that there are both context and scenario "list_baymodels" within the same patch - we usually split contexts and scenarios to different (but related) patches, however this maybe not critical | 09:43 |
strigazi | I'm wokring on it, to split | 09:43 |
strigazi | adding tests too | 09:43 |
amaretskiy | having a job is really important for having changes merged | 09:44 |
amaretskiy | even simple scenario like list_baymodels should be verified on gates | 09:44 |
strigazi | Does it make sense to added to openstack/project-config/jenkins/jobs/magnum.yaml instead of rally.yaml ? | 09:44 |
strigazi | s/openstack/openstack-infra | 09:46 |
amaretskiy | magnum.yaml will configure jobs to magnum project, which is also useful, especially for magnum team | 09:49 |
amaretskiy | however we need to add a magnum job to our project as well | 09:50 |
strigazi | amaretskiy: ok | 09:50 |
amaretskiy | I'm currently asking redixin about that | 09:50 |
amaretskiy | he usually does this | 09:51 |
amaretskiy | redixin: ping | 09:51 |
strigazi | amaretskiy: ok, so gate jobs on rally and magnum (for magnum team) | 09:52 |
redixin | we usually use tempest for functional tests on gates | 09:52 |
redixin | but you can use rally if you want to | 09:52 |
amaretskiy | redixin: we need magnum job | 09:53 |
amaretskiy | redixin: to check https://review.openstack.org/#/c/309546/ and related patches | 09:53 |
*** mvk has joined #openstack-rally | 09:55 | |
*** lakerzhou has joined #openstack-rally | 09:55 | |
strigazi | redixin: I'm working on spliting the patch to have one just for listing baymodels | 09:55 |
strigazi | I'll sync later today with Winnie Tsang and Ton Ngo from my team | 09:56 |
redixin | strigazi: so if we want to test magnum scenarios in rally, we need to edit rally.yaml | 09:58 |
strigazi | redixin: ack | 09:58 |
*** lakerzhou has quit IRC | 10:00 | |
strigazi | Please confirm that we should proceed as following: baymodel_list scenario + rally gate job -> baymodel_context -> create and delete baymodel scenario | 10:02 |
strigazi | then: the same three steps for bays | 10:03 |
*** yingjun has quit IRC | 10:05 | |
redixin | strigazi: list scenario first? to list something we should create something right? | 10:15 |
redixin | create_and_delete looks like better choice for first scenario | 10:15 |
amaretskiy | redixin: there is a magnum context for that | 10:15 |
redixin | magnum context already merged? | 10:15 |
amaretskiy | redixin: no, it is proposed | 10:15 |
amaretskiy | redixin: as well as list_baymodels scenario | 10:16 |
amaretskiy | redixin: I think we need "experimental" job with magnum service | 10:16 |
amaretskiy | to check these changes | 10:16 |
*** luzC has quit IRC | 10:34 | |
*** akrzos has joined #openstack-rally | 10:37 | |
*** akrzos has quit IRC | 10:39 | |
*** rdas has quit IRC | 10:42 | |
*** yanyanhu has quit IRC | 10:46 | |
*** luzC has joined #openstack-rally | 11:01 | |
openstackgerrit | Sergey Skripnick proposed openstack/rally: Mock generate_cidr in unit test https://review.openstack.org/320898 | 11:05 |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Spec] Small fixes in introduce-class-based-scenario-implementation.rst https://review.openstack.org/320903 | 11:09 |
openstackgerrit | chenhb proposed openstack/rally: Fix CinderVolumes.create_volume_and_clone https://review.openstack.org/318555 | 11:12 |
*** bkopilov has quit IRC | 11:12 | |
*** abalutoiu has joined #openstack-rally | 11:13 | |
*** akrzos has joined #openstack-rally | 11:17 | |
*** openstackgerrit has quit IRC | 11:18 | |
*** openstackgerrit has joined #openstack-rally | 11:18 | |
*** rdas has joined #openstack-rally | 11:26 | |
openstackgerrit | Merged openstack/rally: Use correct parameter name for CA file with heat client https://review.openstack.org/320776 | 11:44 |
*** yingjun has joined #openstack-rally | 11:44 | |
*** yingjun has quit IRC | 11:54 | |
*** MaxPC has joined #openstack-rally | 11:56 | |
*** lakerzhou has joined #openstack-rally | 11:56 | |
openstackgerrit | Merged openstack/rally: [Spec] Small fixes in introduce-class-based-scenario-implementation.rst https://review.openstack.org/320903 | 11:58 |
*** lakerzhou has quit IRC | 12:00 | |
*** yingjun has joined #openstack-rally | 12:01 | |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Plugins] Fix custom output from boot_runcommand_delete() https://review.openstack.org/320460 | 12:06 |
*** yingjun_ has joined #openstack-rally | 12:11 | |
*** yingjun has quit IRC | 12:14 | |
*** asti has joined #openstack-rally | 12:17 | |
openstackgerrit | Chandidas Gharami proposed openstack/rally: rally-ci recheck https://review.openstack.org/314070 | 12:25 |
*** lakerzhou has joined #openstack-rally | 12:39 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/rally: [Verify] Get rid of hard-coded list of plugins in tempest.conf https://review.openstack.org/320969 | 12:45 |
*** absubram has quit IRC | 12:47 | |
*** stpierre has joined #openstack-rally | 12:49 | |
*** bkopilov has joined #openstack-rally | 12:50 | |
*** absubram has joined #openstack-rally | 12:50 | |
*** yingjun_ has quit IRC | 13:03 | |
*** yingjun has joined #openstack-rally | 13:03 | |
strigazi | amaretskiy: are you there? | 13:04 |
amaretskiy | strigazi: yes | 13:04 |
amaretskiy | redixin rvasilets ping | 13:05 |
*** ig0r__ has joined #openstack-rally | 13:05 | |
strigazi | amaretskiy: sorry I was away. To plan our action with my team. Should we pursue the create and delete scenario first or go with the simpler baymodel_list? I was confused by redixin answer | 13:06 |
rvasilets | pong | 13:06 |
amaretskiy | redixin rvasilets: lets finally get a decision about magnum gates job | 13:06 |
*** dave-mccowan has joined #openstack-rally | 13:07 | |
rvasilets | I think that its doesn't matter what scenario. They already created list scenario. So lets left it imho | 13:07 |
strigazi | Which scenario must be priotitzed to be added to rally gate? | 13:07 |
redixin | create_and_delete vs context + whatever_list | 13:08 |
amaretskiy | strigazi: as far as I understand, redixin suggests to have a "scenario create_and_list_baymodels" instead of "context+scenario list_baymodels" | 13:08 |
amaretskiy | strigazi: I agree with him | 13:08 |
redixin | 2nd option is more compex right? | 13:08 |
*** ig0r_ has quit IRC | 13:08 | |
rvasilets | yes 2nd is more complex | 13:09 |
strigazi | but for scenario create_and_list_baymodels we need the context as well, right? | 13:09 |
redixin | so we need one patch with one scenario and working gate job | 13:09 |
rvasilets | ofcause create_and_delete much simpler | 13:09 |
redixin | for create_and_list we dont need any contexts | 13:09 |
amaretskiy | redixin rvasilets: how about magum job? we should verify scenarios, shouldn't we? | 13:10 |
rvasilets | because it creates itself | 13:10 |
rvasilets | of cause we should | 13:10 |
rvasilets | I'd prefer to create simple job | 13:10 |
rvasilets | not experemental | 13:10 |
rvasilets | as we don't have limits abount number of jobs | 13:11 |
redixin | im ok with rally-magnum-dsvm job. i just wonder why you decided to start with context+scenario instead of starting with create_and_delete scenario | 13:12 |
strigazi | So, for create and list we can do it only with magnum/utils and a scenario? | 13:12 |
strigazi | redixin: I thought that conext was needed for create | 13:12 |
redixin | yep | 13:12 |
strigazi | create_and_delete makes more sense for us as well | 13:13 |
*** yingjun_ has joined #openstack-rally | 13:14 | |
*** rvasilets__ has joined #openstack-rally | 13:15 | |
*** yingjun has quit IRC | 13:15 | |
strigazi | One more question, the patch for the job in project-config must be merged first? | 13:16 |
redixin | strigazi: sure. we need to test code before it can be merged into rally | 13:17 |
redixin | nobody wants broken code in the master branch | 13:18 |
strigazi | redixin: ack | 13:18 |
openstackgerrit | Illia Khudoshyn proposed openstack/rally: Use existing users in 'create_and_list_networks' https://review.openstack.org/320989 | 13:31 |
strigazi | redixin: I'm following https://wiki.openstack.org/wiki/Rally/RallyGates and https://review.openstack.org/#/c/186702/ | 13:43 |
strigazi | mostly change https://review.openstack.org/#/c/186702/ | 13:45 |
redixin | strigazi: looks good. just use check pipeline instead of experimental | 13:45 |
strigazi | redixin: ack | 13:45 |
*** yingjun_ has quit IRC | 13:46 | |
*** yingjun has joined #openstack-rally | 13:46 | |
*** yingjun has quit IRC | 13:50 | |
*** yingjun has joined #openstack-rally | 13:59 | |
*** rdas has quit IRC | 14:01 | |
*** zackf has joined #openstack-rally | 14:09 | |
*** adiantum has joined #openstack-rally | 14:11 | |
strigazi | redixin: I'm defining a job template and a builder in jenkins/jobs/rally.yaml and adding a rally job in magnum repo. Correct? | 14:17 |
*** diogogmt has quit IRC | 14:20 | |
*** arnoldje has joined #openstack-rally | 14:21 | |
redixin | strigazi: why magnum repo? do you want to have rally charts on every patch in magnum? | 14:22 |
redixin | strigazi: this make sense but first we need to cover scenarios itself | 14:23 |
redixin | so we need to add a job in rally repo | 14:23 |
strigazi | redixin: ack, adding to magnum repo can be done later | 14:24 |
strigazi | redixin: In a job we can have more than one scenarios? | 14:25 |
redixin | strigazi: sure. we usually run one .yaml file with bunch of scenarios | 14:27 |
strigazi | redixin: ok | 14:28 |
strigazi | https://review.openstack.org/#/c/321026/ | 14:33 |
*** KevinE has joined #openstack-rally | 14:35 | |
strigazi | rvasilets amaretskiy redixin: I hope I got it right https://review.openstack.org/#/c/321026/ | 14:37 |
*** rvasilets__ has quit IRC | 14:39 | |
amaretskiy | strigazi: looks good for first sight | 14:41 |
strigazi | amaretskiy: nice | 14:42 |
*** jroll has joined #openstack-rally | 14:43 | |
*** ciprian-barbu has joined #openstack-rally | 14:44 | |
ciprian-barbu | ylobankov: ping | 14:45 |
ylobankov | ciprian-barbu: pong | 14:46 |
ciprian-barbu | ylobankov: hey, I'm contacting you about a change I proposed for rally | 14:46 |
ciprian-barbu | I only found out about #openstack-rally now, so sorry for the delay | 14:46 |
*** abalutoiu has quit IRC | 14:46 | |
jroll | amaretskiy: /b 37 | 14:47 |
ciprian-barbu | the gerrit change is at https://review.openstack.org/#/c/315171/ | 14:47 |
jroll | oops, sorry for that :( | 14:47 |
amaretskiy | :) | 14:47 |
rvasilets | strigazi, will review | 14:47 |
ylobankov | ciprian-barbu: I will take a look at it today :) | 14:47 |
ciprian-barbu | ylobankov: any time it fancies you, thank you! | 14:48 |
ciprian-barbu | I need to sort out the failed jenkins verification as well, anteaya suggested I look at tox.ini | 14:48 |
KevinE | Hello all, anyone familiar enough with keystoneclient to tell me how I can manipulate this variable in my deployments? https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/httpclient.py#L226-L227 | 15:01 |
KevinE | Also good Morning :) | 15:02 |
openstackgerrit | Yaroslav Lobankov proposed openstack/rally: [Verify] Fixing sahara plugin tempest tests https://review.openstack.org/320969 | 15:03 |
stpierre | KevinE: rally has no support for that -- you'll need to add support for it to the ExistingCloud deployment plugin, and figure out how to pass it to keystoneauth1 | 15:06 |
KevinE | anyone want to help me add rally support for something to the ExistingCloud deployment plugin, and figure out how to pass it to keystoneauth1? | 15:07 |
KevinE | stpierre: lol thanks, at least I have a path to go down again :) | 15:08 |
*** yingjun has quit IRC | 15:10 | |
*** yingjun has joined #openstack-rally | 15:10 | |
KevinE | Why does the Openstack CLI differentiate between endpoint_type and OS-interface, but Rally does not? Is an Openstack cli interface = to a Rally endpoint_type? | 15:11 |
*** yingjun has quit IRC | 15:15 | |
openstackgerrit | Illia Khudoshyn proposed openstack/rally: Use existing users in 'create_and_list_networks' https://review.openstack.org/320989 | 15:16 |
stpierre | KevinE: afaict it looks like you can pass endpoint_override to keystoneclient.client.Client at https://github.com/openstack/rally/blob/master/rally/osclients.py#L265 | 15:19 |
stpierre | i might suggest hardcoding it there as a PoC just to make sure it will actually work for you | 15:19 |
*** diogogmt has joined #openstack-rally | 15:19 | |
stpierre | then you can worry about adding it to https://github.com/openstack/rally/blob/master/rally/deployment/engines/existing.py | 15:19 |
ylobankov | Hi guys! could you please take a look at https://review.openstack.org/#/c/310232/ ? Thanks in advance. | 15:21 |
*** Guest50431 has quit IRC | 15:41 | |
KevinE | stpierre: hmm, no matter what I put in that string, it still fails to contact the admin endpoint :/ | 15:49 |
*** madhu_ak has joined #openstack-rally | 15:50 | |
*** arxcruz has quit IRC | 15:51 | |
KevinE | ks = client.Client(version=version, endpoint_override="https://publicURL:5000", **args) | 15:51 |
stpierre | no idea. i just read the source | 15:57 |
*** kbaikov has quit IRC | 16:00 | |
stpierre | can anyone put some eyes on https://review.openstack.org/#/c/311526/ ? that will unblock our gate jobs on https://review.openstack.org/#/c/300159/, which would be a really nice improvement to have | 16:01 |
*** rcernin has quit IRC | 16:04 | |
*** amaretskiy has quit IRC | 16:07 | |
*** ushkalim has quit IRC | 16:10 | |
*** tesseract has quit IRC | 16:13 | |
openstackgerrit | Roman Vasilets proposed openstack/rally: [WIP][Spec] Rally Task Validation refactoring https://review.openstack.org/283180 | 16:22 |
*** MaxPC has quit IRC | 16:24 | |
KevinE | How do I make the endpoint auth? | 16:29 |
KevinE | endpoint_type * | 16:29 |
stpierre | how do you make it do what? | 16:34 |
*** madhu_ak has quit IRC | 16:34 | |
KevinE | oops lol I mean interface. The interface can be public admin or internal, but also auth to force-use the auth_url? | 16:36 |
openstackgerrit | Roman Vasilets proposed openstack/rally: [WIP][Spec] Rally Task Validation refactoring https://review.openstack.org/283180 | 16:49 |
*** Deng has quit IRC | 16:57 | |
*** MaxPC has joined #openstack-rally | 16:59 | |
*** e0ne has quit IRC | 17:00 | |
*** wootehfoot has joined #openstack-rally | 17:09 | |
*** KevinE has quit IRC | 17:20 | |
*** KevinE has joined #openstack-rally | 17:22 | |
*** ig0r__ has quit IRC | 17:51 | |
*** adiantum has quit IRC | 17:52 | |
*** e0ne has joined #openstack-rally | 17:54 | |
*** pvaneck has joined #openstack-rally | 18:04 | |
*** e0ne has quit IRC | 18:06 | |
*** e0ne has joined #openstack-rally | 18:11 | |
*** e0ne has quit IRC | 18:14 | |
*** lakerzhou has quit IRC | 18:20 | |
*** mvk has quit IRC | 18:20 | |
*** KevinE has quit IRC | 18:21 | |
*** lakerzhou has joined #openstack-rally | 18:34 | |
*** ningy has joined #openstack-rally | 18:55 | |
*** ningy_ has joined #openstack-rally | 18:56 | |
*** ningy has left #openstack-rally | 18:56 | |
ningy_ | @here can someone help to review https://review.openstack.org/#/c/248298/ ? | 18:58 |
ningy_ | rally ci check and experimental job are all passed | 18:59 |
*** mvk has joined #openstack-rally | 19:04 | |
*** KevinE has joined #openstack-rally | 19:26 | |
*** KevinE has quit IRC | 19:28 | |
*** KevinE has joined #openstack-rally | 19:28 | |
*** akrzos has quit IRC | 20:25 | |
*** KevinE has quit IRC | 20:52 | |
*** akrzos has joined #openstack-rally | 21:21 | |
*** akrzos has quit IRC | 21:32 | |
*** stpierre has quit IRC | 21:35 | |
*** redixin has quit IRC | 21:36 | |
*** ningy_ has left #openstack-rally | 21:39 | |
*** wootehfoot has quit IRC | 21:49 | |
*** asti has quit IRC | 21:51 | |
*** arnoldje has quit IRC | 22:13 | |
*** diogogmt has quit IRC | 22:27 | |
*** akrzos has joined #openstack-rally | 22:33 | |
*** arnoldje has joined #openstack-rally | 22:50 | |
*** akrzos has quit IRC | 22:58 | |
*** akrzos has joined #openstack-rally | 23:05 | |
*** harlowja has quit IRC | 23:12 | |
*** MaxPC has quit IRC | 23:24 | |
*** wormwood has quit IRC | 23:30 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!