| *** andreykurilin__ has quit IRC | 00:17 | |
| kun_huang | ping BaQs | 00:30 |
|---|---|---|
| *** boris-42 has joined #openstack-rally | 00:31 | |
| kun_huang | ping boris-42 | 00:31 |
| boris-42 | kun_huang: pong | 00:31 |
| kun_huang | boris-42: https://review.openstack.org/#/c/196855/ | 00:32 |
| kun_huang | this one nearly done | 00:32 |
| *** dmorita has joined #openstack-rally | 00:33 | |
| boris-42 | kun_huang: yep going to update it | 01:03 |
| boris-42 | kun_huang: we decide just to remove those benchmarks | 01:03 |
| boris-42 | tests** | 01:03 |
| kun_huang | boris-42: his patch need update a little | 01:06 |
| boris-42 | kun_huang: yep going to do that now | 01:20 |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Restore coverage test functionality https://review.openstack.org/196799 | 01:26 |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Remove extraneous failed server/image tests https://review.openstack.org/196855 | 01:26 |
| boris-42 | kun_huang: ^ okay I hope we will be able to merge this | 01:27 |
| *** echoingumesh has joined #openstack-rally | 01:27 | |
| *** echoingu_ has joined #openstack-rally | 01:29 | |
| *** echoingumesh has quit IRC | 01:32 | |
| kun_huang | boris-42: btw, the old "self.get_fm.cleanUp()" seems being used in a wrong way | 01:34 |
| boris-42 | kun_huang: ya it looks quite strange | 01:35 |
| kun_huang | it looks good, just looks good | 01:36 |
| kun_huang | cleanup() seems to clear fixture instance | 01:37 |
| kun_huang | before 1.2.0, it did, but now, not | 01:37 |
| boris-42 | kun_huang: so I agree with stpierre | 01:38 |
| kun_huang | that test case need run with real object and isntances, but all other need complicated mock and fixture environment | 01:38 |
| boris-42 | let's just get rid of wrong tests | 01:38 |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time https://review.openstack.org/196493 | 01:49 |
| boris-42 | kun_huang: ^ less then 300ms to start rally | 01:52 |
| boris-42 | kun_huang: -80ms from booting start | 01:52 |
| boris-42 | kun_huang: need to tune mreeeee | 01:52 |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time https://review.openstack.org/196493 | 02:04 |
| *** echoingu_ has quit IRC | 02:06 | |
| *** echoingumesh has joined #openstack-rally | 02:07 | |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time https://review.openstack.org/196493 | 02:08 |
| kun_huang | boris-42: have you maintained a etherpad to maintain some review work? | 02:09 |
| boris-42 | kun_huang: ? | 02:09 |
| *** echoingumesh has quit IRC | 02:11 | |
| kun_huang | boris-42: to track that some patches are close to merge, some need argue further and so on | 02:12 |
| boris-42 | kun_huang: so we have | 02:15 |
| boris-42 | http://goo.gl/04aT6p | 02:16 |
| boris-42 | kun_huang: ^ rally dashboard | 02:16 |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time https://review.openstack.org/196493 | 02:19 |
| kun_huang | boris-42: that is good but not enough for me | 02:33 |
| kun_huang | boris-42: I tried this https://etherpad.openstack.org/p/rally_review_work | 02:33 |
| boris-42 | kun_huang: so if you would like to maintain such list | 02:37 |
| boris-42 | kun_huang: I think everybody will be happy | 02:37 |
| boris-42 | kun_huang: to use it) | 02:37 |
| boris-42 | rally boot time real0m0.267s | 02:37 |
| boris-42 | 02:37 | |
| boris-42 | WOOOT | 02:37 |
| kun_huang | you could add in channel topic :) | 02:40 |
| kun_huang | btw I'm building openstack in my temporary office | 02:40 |
| kun_huang | really wanna have a test on your patch boris-42 | 02:41 |
| *** echoingumesh has joined #openstack-rally | 02:42 | |
| openstackgerrit | Merged openstack/rally: Remove extraneous failed server/image tests https://review.openstack.org/196855 | 02:42 |
| *** echoingumesh has quit IRC | 02:43 | |
| *** echoingumesh has joined #openstack-rally | 02:43 | |
| boris-42 | kun_huang: what test/ | 02:46 |
| kun_huang | boris-42: your patch on rally boot | 02:46 |
| boris-42 | kun_huang: did you see this lib https://github.com/boris-42/profimp ? | 02:46 |
| boris-42 | kun_huang: I will attach to reports to it | 02:46 |
| *** echoingumesh has quit IRC | 02:48 | |
| openstackgerrit | Boris Pavlovic proposed openstack/rally: Optimize Rally imports to reduce start time https://review.openstack.org/196493 | 02:50 |
| *** oanufriev__ has quit IRC | 02:56 | |
| boris-42 | kun_huang: and actually I will add test | 02:56 |
| *** kevinbenton has joined #openstack-rally | 03:08 | |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: VMTasks: Make waiting for ping timing configurable https://review.openstack.org/192776 | 03:33 |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: Fix EC2 utils and tests https://review.openstack.org/192775 | 03:33 |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: Better Scenario test case fixtures https://review.openstack.org/192774 | 03:33 |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: Add Heat scenario to test scaling policies https://review.openstack.org/178436 | 03:33 |
| openstackgerrit | Merged openstack/rally: Restore coverage test functionality https://review.openstack.org/196799 | 03:41 |
| *** coolsvap|away is now known as coolsvap | 04:08 | |
| openstackgerrit | Kiran proposed openstack/rally: Add Neutron LoadBalancer v1 create and list vips https://review.openstack.org/193527 | 04:46 |
| *** yfried|afk has quit IRC | 04:48 | |
| *** kiran-r has joined #openstack-rally | 04:52 | |
| *** tfreger has joined #openstack-rally | 04:54 | |
| *** kiran-r has quit IRC | 05:00 | |
| *** kiran-r has joined #openstack-rally | 05:03 | |
| openstackgerrit | Merged openstack/rally: Handle NoneType error from pool_create_args https://review.openstack.org/192222 | 05:04 |
| *** rdas has joined #openstack-rally | 05:12 | |
| *** kiran-r has quit IRC | 05:17 | |
| *** kiran-r has joined #openstack-rally | 05:18 | |
| openstackgerrit | Merged openstack/rally: unit-tests: fix mock check errors messages https://review.openstack.org/194403 | 05:19 |
| openstackgerrit | Merged openstack/rally: Support validate parameters list for restricted_parameters https://review.openstack.org/194552 | 05:19 |
| openstackgerrit | Merged openstack/rally: Support validate parameters list for restricted_parameters https://review.openstack.org/194552 | 05:19 |
| *** _kiran_ has joined #openstack-rally | 05:23 | |
| *** kiran-r has quit IRC | 05:24 | |
| *** nihilifer has joined #openstack-rally | 05:25 | |
| *** _kiran_ is now known as kiran-r | 05:27 | |
| kiran-r | kun_huang: http://185.8.56.88/gqYeldQhUx/mgsenjaTDd/ | 05:27 |
| kiran-r | kun_huang: https://review.openstack.org/#/c/193432/ https://review.openstack.org/#/c/193527/ | 05:30 |
| *** kiran-r has quit IRC | 05:46 | |
| *** oanufriev__ has joined #openstack-rally | 06:08 | |
| *** echoingumesh has joined #openstack-rally | 06:15 | |
| *** yfried|afk has joined #openstack-rally | 06:24 | |
| *** yfried has joined #openstack-rally | 06:29 | |
| *** yfried|afk has quit IRC | 06:29 | |
| *** nkhare has joined #openstack-rally | 06:40 | |
| *** oanufriev__ has quit IRC | 06:42 | |
| *** neeti has joined #openstack-rally | 06:58 | |
| *** oanufriev__ has joined #openstack-rally | 07:00 | |
| *** boris-42 has quit IRC | 07:02 | |
| *** anshul has joined #openstack-rally | 07:05 | |
| *** oanufriev__ has quit IRC | 07:06 | |
| *** kiran-r has joined #openstack-rally | 07:09 | |
| *** tfreger has quit IRC | 07:10 | |
| *** tfreger has joined #openstack-rally | 07:10 | |
| kiran-r | yfried: Hi! | 07:15 |
| *** fhubik has joined #openstack-rally | 07:19 | |
| *** temujin has quit IRC | 07:26 | |
| *** temujin has joined #openstack-rally | 07:27 | |
| *** albertw1 has joined #openstack-rally | 07:31 | |
| *** echoingumesh has quit IRC | 07:32 | |
| *** echoingumesh has joined #openstack-rally | 07:32 | |
| *** e0ne has joined #openstack-rally | 07:33 | |
| *** echoingumesh has quit IRC | 07:37 | |
| *** e0ne has quit IRC | 07:37 | |
| yfried | kiran-r: hi | 07:38 |
| kiran-r | yfried: Can you review this patch? https://review.openstack.org/#/c/193432/17 | 07:40 |
| yfried | kiran-r: looking | 07:41 |
| kiran-r | yfried: Thanks.! =) | 07:42 |
| *** fhubik is now known as fhubik_afk | 07:44 | |
| kiran-r | yfried: Sorry, I forgot to add a dependent patch which used lbaas_v1 context. https://review.openstack.org/#/c/193527/ | 07:45 |
| *** tfreger has quit IRC | 07:45 | |
| kiran-r | yfried: Please review this one too | 07:45 |
| *** tfreger has joined #openstack-rally | 07:45 | |
| *** lutostag has quit IRC | 07:46 | |
| *** amaretskiy has joined #openstack-rally | 07:54 | |
| *** fhubik_lunch has joined #openstack-rally | 08:02 | |
| *** yfried is now known as yfried|afk | 08:03 | |
| *** temujin has quit IRC | 08:03 | |
| *** temujin has joined #openstack-rally | 08:03 | |
| *** fhubik_afk has quit IRC | 08:05 | |
| *** fhubik_lunch has quit IRC | 08:07 | |
| *** fhubik_lunch has joined #openstack-rally | 08:07 | |
| *** fhubik_lunch is now known as fhubik_afk | 08:07 | |
| *** acruz has joined #openstack-rally | 08:12 | |
| *** yfried|afk is now known as yfried | 08:22 | |
| *** andreykurilin__ has joined #openstack-rally | 08:25 | |
| *** amalinow has joined #openstack-rally | 08:26 | |
| *** fhubik_afk is now known as fhubik_lunch | 08:33 | |
| *** tfreger has quit IRC | 08:35 | |
| *** tfreger has joined #openstack-rally | 08:36 | |
| openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Reports] Add class for merging data stream https://review.openstack.org/194745 | 08:47 |
| *** temujin has quit IRC | 08:48 | |
| -openstackstatus- NOTICE: OpenStack CI is down due to hard drive failures | 08:49 | |
| *** ChanServ changes topic to "OpenStack CI is down due to hard drive failures" | 08:49 | |
| openstackgerrit | Alexander Gubanov proposed openstack/rally: Add Nova network scenarios https://review.openstack.org/178139 | 08:51 |
| *** fabian4 has joined #openstack-rally | 08:56 | |
| *** redixin has joined #openstack-rally | 08:58 | |
| *** lutostag has joined #openstack-rally | 08:59 | |
| *** e0ne has joined #openstack-rally | 09:02 | |
| *** albertw1 has quit IRC | 09:08 | |
| *** karimb has joined #openstack-rally | 09:12 | |
| *** e0ne is now known as e0ne_ | 09:15 | |
| *** e0ne_ has quit IRC | 09:26 | |
| *** fhubik_lunch is now known as fhubik_afk | 09:28 | |
| *** e0ne has joined #openstack-rally | 09:29 | |
| *** tosky has joined #openstack-rally | 09:32 | |
| andreykurilin__ | gates are broken... again | 09:34 |
| *** fhubik_afk is now known as fhubik_lunch | 09:36 | |
| *** tfreger has quit IRC | 09:40 | |
| *** tfreger has joined #openstack-rally | 09:41 | |
| *** tfreger has quit IRC | 09:45 | |
| *** tfreger has joined #openstack-rally | 09:46 | |
| *** aix has quit IRC | 09:46 | |
| *** yfried is now known as yfried|afk | 09:52 | |
| *** karimb has quit IRC | 09:53 | |
| *** karimb has joined #openstack-rally | 09:56 | |
| *** oanufriev_ has joined #openstack-rally | 09:57 | |
| *** yfried|afk is now known as yfried | 10:00 | |
| *** fhubik_lunch is now known as fhubik_afk | 10:13 | |
| *** coolsvap is now known as coolsvap|away | 10:14 | |
| *** fhubik_afk is now known as fhubik_lunch | 10:17 | |
| *** aix has joined #openstack-rally | 10:17 | |
| *** paboldin has joined #openstack-rally | 10:22 | |
| paboldin | Review please: https://review.openstack.org/194754/ | 10:22 |
| *** e0ne is now known as e0ne_ | 10:23 | |
| paboldin | Yo, rally. We heard you like unit tests so we put unittests in your unittests so you can unittest while you unittest. | 10:24 |
| openstackgerrit | Kiran proposed openstack/rally: Adds LBaaS-V1 context https://review.openstack.org/193432 | 10:24 |
| *** e0ne_ is now known as e0ne | 10:25 | |
| *** kairat_kushaev has left #openstack-rally | 10:26 | |
| *** fhubik_lunch is now known as fhubik_afk | 10:29 | |
| *** fhubik_afk is now known as fhubik_lunch | 10:33 | |
| *** tfreger has quit IRC | 10:40 | |
| *** tfreger1 has joined #openstack-rally | 10:40 | |
| *** tfreger1 has quit IRC | 10:41 | |
| *** tfreger has joined #openstack-rally | 10:41 | |
| *** karmatronic has joined #openstack-rally | 10:43 | |
| *** karimb has quit IRC | 10:44 | |
| openstackgerrit | Kiran proposed openstack/rally: Adds LBaaS-V1 context https://review.openstack.org/193432 | 10:47 |
| *** karmatronic has quit IRC | 10:48 | |
| *** fhubik_lunch is now known as fhubik_afk | 10:52 | |
| *** dmorita has quit IRC | 10:54 | |
| *** oanufrie1 has joined #openstack-rally | 10:57 | |
| *** karmatronic has joined #openstack-rally | 11:00 | |
| openstackgerrit | Kiran proposed openstack/rally: Add Neutron LoadBalancer v1 create and list vips https://review.openstack.org/193527 | 11:04 |
| kiran-r | andreykurilin__: Not again! :( | 11:05 |
| *** karmatronic has quit IRC | 11:06 | |
| *** karimb has joined #openstack-rally | 11:06 | |
| *** anshul has quit IRC | 11:08 | |
| *** exploreshaifali has joined #openstack-rally | 11:13 | |
| openstackgerrit | Pavel Boldin proposed openstack/rally: Add command-dict option to specify command args https://review.openstack.org/177016 | 11:15 |
| openstackgerrit | Pavel Boldin proposed openstack/rally: Add command_customizer context https://review.openstack.org/183227 | 11:15 |
| openstackgerrit | Pavel Boldin proposed openstack/rally: Add boot_runcommand_delete_custom_image scenario https://review.openstack.org/183228 | 11:15 |
| openstackgerrit | Pavel Boldin proposed openstack/rally: Add command-dict option to upload a local command https://review.openstack.org/177015 | 11:15 |
| *** oanufriev_ has quit IRC | 11:17 | |
| *** oanufriev_ has joined #openstack-rally | 11:18 | |
| *** anshul has joined #openstack-rally | 11:22 | |
| *** neeti has quit IRC | 11:26 | |
| openstackgerrit | Roman Vasilets proposed openstack/rally: Add ability to specify directory with murano package files https://review.openstack.org/195677 | 11:32 |
| *** anshul has quit IRC | 11:49 | |
| *** karimb has quit IRC | 11:53 | |
| *** karimb has joined #openstack-rally | 11:53 | |
| *** MaxPC has joined #openstack-rally | 11:54 | |
| *** mwagner_lap has joined #openstack-rally | 11:54 | |
| *** fhubik_afk is now known as fhubik_lunch | 11:58 | |
| *** fhubik_lunch is now known as fhubik_afk | 11:59 | |
| *** yfried is now known as yfried|afk | 11:59 | |
| *** anshul has joined #openstack-rally | 12:01 | |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: VMTasks: Make waiting for ping timing configurable https://review.openstack.org/192776 | 12:07 |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: Fix EC2 utils and tests https://review.openstack.org/192775 | 12:07 |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: Better Scenario test case fixtures https://review.openstack.org/192774 | 12:07 |
| *** stpierre has joined #openstack-rally | 12:14 | |
| openstackgerrit | Kiran proposed openstack/rally: Adds LBaaS-V1 context https://review.openstack.org/193432 | 12:18 |
| *** yfried|afk is now known as yfried | 12:19 | |
| *** oanufrie1 has quit IRC | 12:25 | |
| *** nkhare has quit IRC | 12:25 | |
| openstackgerrit | Kiran proposed openstack/rally: Add Neutron LoadBalancer v1 create and list vips https://review.openstack.org/193527 | 12:31 |
| *** frayedknot has joined #openstack-rally | 12:42 | |
| openstackgerrit | Kiran proposed openstack/rally: Add Neutron LoadBalancer v1 create and delete vips https://review.openstack.org/193818 | 12:49 |
| *** kbaikov has quit IRC | 12:52 | |
| *** gubouvier has joined #openstack-rally | 13:01 | |
| yfried | redixin: I'm having trouble installing rally on a rhel machine | 13:05 |
| yfried | http://paste.openstack.org/show/328437/ | 13:05 |
| yfried | redixin: IIRC you wrote the last version of the intall script | 13:06 |
| yfried | andreykurilin__: ^ | 13:09 |
| yfried | meteorfox: ping | 13:09 |
| redixin | yfried, it failed while trying to download https://pypi.python.org/simple/setuptools/../../packages/source/s/setuptools/setuptools-9.1.tar.gz | 13:09 |
| redixin | with 404 | 13:09 |
| yfried | redixin: yeah. But I have no idea why it built this url to DL | 13:09 |
| paboldin | btw it works for wget | 13:14 |
| yfried | paboldin: my error? | 13:14 |
| paboldin | yfried: probably some temporary failure | 13:14 |
| openstackgerrit | Kiran proposed openstack/rally: Add Neutron Loadbalancer v1 create and update vips https://review.openstack.org/194106 | 13:18 |
| *** tfreger has quit IRC | 13:20 | |
| *** nihilifer has quit IRC | 13:21 | |
| stpierre | i keep dreaming of a day when gates aren't broken | 13:23 |
| *** e0ne is now known as e0ne_ | 13:29 | |
| *** boris-42 has joined #openstack-rally | 13:31 | |
| *** oanufriev_ has quit IRC | 13:33 | |
| yfried | paboldin: redixin: https://bugs.launchpad.net/rally/+bug/1470110 | 13:37 |
| openstack | Launchpad bug 1470110 in Rally "Installation script fails with "curl"" [Medium,New] | 13:37 |
| yfried | boris-42: ^ | 13:37 |
| boris-42 | redixin: stpierre ^ | 13:38 |
| boris-42 | yfried: uh oh | 13:38 |
| paboldin | curl versoin please | 13:38 |
| *** e0ne_ has quit IRC | 13:39 | |
| stpierre | looks like wget resolves the ../'s client side | 13:40 |
| stpierre | while curl does this: | 13:40 |
| stpierre | > GET /simple/setuptools/../../packages/source/s/setuptools/setuptools-9.1.tar.gz HTTP/1.1 | 13:40 |
| stpierre | the easy fix is to just require wget and forget about curl | 13:40 |
| yfried | paboldin: boris-42: redixin: curl 7.29.0 | 13:41 |
| paboldin | stpierre: we should not pass an invalid url to the CURL at the first place | 13:42 |
| stpierre | sure, but that's not the easy fix :) | 13:42 |
| stpierre | that's the URL we get from pypi | 13:42 |
| redixin | +1 for using wget | 13:42 |
| *** e0ne has joined #openstack-rally | 13:46 | |
| yfried | paboldin: boris-42: redixin: http://paste.openstack.org/show/328596/ problem with jsonschema requirments | 13:48 |
| yfried | paboldin: boris-42: redixin: http://paste.openstack.org/show/328611/ | 13:48 |
| yfried | seems like jsonschema requires functools32 but it wasn't installed. I wonder if the bug is in rally or in jsonschema | 13:49 |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: install_rally.sh: always use wget, not cURL https://review.openstack.org/197079 | 13:49 |
| stpierre | yfried: i've seen that with several openstack components, so i suspect jsonschema | 13:50 |
| stpierre | seems like there should be an upstream global-requirements fix for it by now | 13:50 |
| stpierre | ...but there isn't, afaict | 13:51 |
| boris-42 | yfried: recently I saw that bug | 13:53 |
| boris-42 | yfried: with functools | 13:53 |
| yfried | boris-42: https://bugs.launchpad.net/rally/+bug/1466055 | 13:53 |
| openstack | Launchpad bug 1466055 in Rally "functools32 should be added to requirements" [Undecided,New] | 13:53 |
| boris-42 | yfried: stpierre redixin let's get rid of curl | 13:54 |
| stpierre | boris-42: https://review.openstack.org/197079 | 13:54 |
| yfried | boris-42: stpierre: where do I post bugs against req | 13:54 |
| boris-42 | yfried: stpierre redixin it will make installation script simpler | 13:54 |
| kun_huang | kiran-r: ping | 13:54 |
| *** fhubik_afk is now known as fhubik_lunch | 13:54 | |
| stpierre | yfried: i'm not actually sure. looking through commits to the requirements repo i don't see many that reference open bugs. you may just want to put in a review | 13:57 |
| yfried | stpierre: ^ re jsonschema and functools32 ? | 13:57 |
| yfried | stpierre: where do I submit bugs against requirements? | 13:58 |
| stpierre | yeah, i don't know where. i don't see a relevant project in launchpad | 13:59 |
| stpierre | the reviews against requirements with bugs associated have the bugs in different projects | 14:01 |
| stpierre | e.g., the bug closed by https://review.openstack.org/#/c/196867/ is against nova | 14:01 |
| *** albertw has quit IRC | 14:01 | |
| stpierre | so i'm ugessing the bug you entered against rally is sufficient | 14:01 |
| *** albertw has joined #openstack-rally | 14:01 | |
| yfried | stpierre: boris-42: https://bugs.launchpad.net/openstack-gate/+bug/1462421 | 14:02 |
| openstack | Launchpad bug 1462421 in OpenStack-Gate "jsonschema 2.5.0 pulling in functools32 which breaks py26 and py34 jobs" [Undecided,Confirmed] | 14:02 |
| *** kiran-r has quit IRC | 14:02 | |
| stpierre | i was just looking at that :) | 14:02 |
| yfried | stpierre: do you have a fix? | 14:02 |
| yfried | stpierre: because this breaks my automation | 14:02 |
| yfried | boris-42: stpierre: can we use a lower version of jsonschema in rally? | 14:03 |
| *** albertw1 has joined #openstack-rally | 14:03 | |
| yfried | like <2.5.0 ? | 14:04 |
| *** rdas has quit IRC | 14:05 | |
| stpierre | we're not allowed to diverge from global requirements | 14:05 |
| stpierre | which has: jsonschema>=2.0.0,<3.0.0,!=2.5.0 | 14:05 |
| yfried | stpierre: but that's a range | 14:07 |
| yfried | stpierre: jsonschema>=2.0.0,<2.5.0 | 14:08 |
| stpierre | yeah, but we can't diverge from that | 14:08 |
| yfried | stpierre: ^ doesn't contradict | 14:08 |
| stpierre | we can't specify our own range | 14:08 |
| yfried | stpierre: so what can we do? | 14:09 |
| stpierre | i think we need to add to global requirements: functools32;python_version=='2.7' | 14:09 |
| stpierre | but i've never used that syntax before so i'm not entirely sure | 14:09 |
| *** fhubik_lunch is now known as fhubik | 14:10 | |
| *** nihilifer has joined #openstack-rally | 14:17 | |
| yfried | stpierre: boris-42: https://github.com/Julian/jsonschema/issues/233 | 14:19 |
| yfried | so, must use newer version of pip | 14:19 |
| stpierre | that's odd, global requirements specifies pip>=6.0 | 14:21 |
| yfried | stpierre: no "pip" in rally/requirements.txt | 14:23 |
| *** karmatronic has joined #openstack-rally | 14:23 | |
| stpierre | oh, that could sure be the issue then. we need to add pip to our requirements | 14:23 |
| yfried | stpierre: I'm on it | 14:24 |
| *** karimb has quit IRC | 14:24 | |
| *** fhubik is now known as fhubik_afk | 14:25 | |
| openstackgerrit | Yair Fried proposed openstack/rally: Adds pip to requirements https://review.openstack.org/197109 | 14:28 |
| yfried | boris-42: stpierre: ^ | 14:28 |
| *** karmatronic has quit IRC | 14:29 | |
| *** neeti has joined #openstack-rally | 14:30 | |
| *** karimb has joined #openstack-rally | 14:36 | |
| boris-42 | yfried: this seems a bit strange... | 14:39 |
| yfried | boris-42: ? | 14:39 |
| boris-42 | yfried: adding to requriments something that is not used in runtime | 14:40 |
| yfried | boris-42: what do you mean? | 14:40 |
| yfried | boris-42: since we are creating a venv, we need to make sure pip is up to date | 14:41 |
| *** rushiagr_away is now known as rushiagr | 14:45 | |
| *** oanufrie1 has joined #openstack-rally | 14:52 | |
| *** ChanServ changes topic to "☁ 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" | 14:54 | |
| -openstackstatus- NOTICE: The log volume was repaired and brought back online at 14:00 UTC. Log links today from before that time may be missing, and changes should be rechecked if fresh job logs are desired for them. | 14:54 | |
| yfried | boris-42: looks like latest pip doesn't fix the issue | 14:54 |
| *** e0ne is now known as e0ne_ | 14:55 | |
| *** frayedknot has quit IRC | 14:57 | |
| *** jaypipes has quit IRC | 14:58 | |
| boris-42 | yfried: .... | 14:58 |
| *** e0ne_ is now known as e0ne | 14:58 | |
| boris-42 | yfried: we have so many issues latest 2 week... | 14:58 |
| *** e0ne is now known as e0ne_ | 14:59 | |
| *** e0ne_ is now known as e0ne | 14:59 | |
| *** exploreshaifali has quit IRC | 15:01 | |
| openstackgerrit | Yair Fried proposed openstack/rally: Adds pip to requirements https://review.openstack.org/197109 | 15:02 |
| yfried | boris-42: pip has to come first on the list | 15:02 |
| yfried | boris-42: testing now | 15:02 |
| *** frayedknot has joined #openstack-rally | 15:02 | |
| *** aix has quit IRC | 15:09 | |
| *** temujin has joined #openstack-rally | 15:10 | |
| *** oanufrie1 has left #openstack-rally | 15:10 | |
| *** kiran-r has joined #openstack-rally | 15:16 | |
| *** andreykurilin__ has quit IRC | 15:17 | |
| yfried | looks like pip is updated before jsonschema is installed, and yet - jsonschema is installed without functools32 | 15:17 |
| yfried | however, when I install jsonschema again, it works as expected | 15:17 |
| yfried | seems like an issue with pip were it's still using the old version in the same operations | 15:17 |
| yfried | boris-42: stpierre: ^ http://paste.openstack.org/show/328808/ search for "@@" | 15:18 |
| yfried | boris-42: stpierre: seems like the best solution is to update pip as part of installation script | 15:18 |
| stpierre | ugh | 15:19 |
| yfried | [18:21] <fungi> yfried: this is why we never bothered to include pip in our requirements lists in the past | 15:22 |
| yfried | [18:21] <yfried> fungi: ack | 15:22 |
| yfried | [18:21] <fungi> you need a separate bootstrapping step to upgrade pip before you use it to install other things | 15:22 |
| yfried | [18:21] <yfried> fungi: so, I'm still at loss for the bug | 15:22 |
| yfried | stpierre: ^ any ideas? | 15:22 |
| stpierre | uggggggggggh | 15:23 |
| stpierre | this is terrible | 15:23 |
| stpierre | it looks like we don't actually use pip until after we've set up the virtualenv | 15:23 |
| stpierre | so i guess we could remove python-pip from the required package list, and then install it with get-pip.py after the venv has been activated | 15:24 |
| *** aix has joined #openstack-rally | 15:24 | |
| yfried | stpierre: and make sure it's latest | 15:24 |
| stpierre | e.g., lines 9-11 here: https://review.openstack.org/#/c/191194/7/Dockerfile | 15:24 |
| stpierre | i *think* get-pip.py does that, but yeah | 15:25 |
| yfried | stpierre: that's in docker. what about install script? | 15:25 |
| yfried | paboldin: ^ | 15:25 |
| stpierre | i'm just offering that up as an example of what to do in install_rally.sh | 15:25 |
| yfried | stpierre: also - what about no venv (system mode)? | 15:26 |
| stpierre | i think we could just add the get-pip.py step after the virtualenv is (optionally) installed, regardless of whether or not it's used. so remove python-pip from required packages, remove the 'pip wheel' check from install_virtualenv, and then add get-pip.py on line 651 | 15:27 |
| stpierre | although i'm not sure if we'd need to add the 'pip wheel' check back in, or if get-pip.py can be counted on to take care of that | 15:28 |
| yfried | stpierre: removing https://github.com/openstack/rally/blob/master/install_rally.sh#L427-L442 | 15:33 |
| stpierre | yeah, since pip would get installed after that | 15:33 |
| yfried | stpierre: where's "remove python-pip from required packages"? | 15:34 |
| stpierre | https://github.com/openstack/rally/blob/master/install_rally.sh#L308-L310 and https://github.com/openstack/rally/blob/master/install_rally.sh#L331-L346 | 15:34 |
| *** kiran-r has quit IRC | 15:35 | |
| stpierre | hrm, i'm not sure that get-pip.py respects virtualenvs :( | 15:36 |
| yfried | stpierre: who would know? | 15:37 |
| stpierre | well, i tried it and it didn't; i'm guessing you could get it to with the right combination of CLI options, but i can't find much on the googles | 15:38 |
| stpierre | i might just not be searching for the right thing, though | 15:38 |
| stpierre | oh, wait | 15:39 |
| stpierre | virtualenvs include pip | 15:39 |
| stpierre | so all we need to do is make sure that our venv has a new enough pip, *or* use get-pip.py if we're installing to the system | 15:39 |
| stpierre | so we don't need get-pip.py to respect venvs | 15:39 |
| *** PrashantS has joined #openstack-rally | 15:42 | |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: WIP: install newer pip in install_rally.sh https://review.openstack.org/197142 | 15:44 |
| stpierre | yfried: there's a starting point | 15:44 |
| yfried | stpierre: can't we just run "pip install -U pip" before doing anything else? | 15:46 |
| yfried | stpierre: this would upgrade pip prior to isntalling rally | 15:47 |
| *** neeti has quit IRC | 15:47 | |
| stpierre | yeah, i guess all the other changes i made in there aren't really necessary, are they? | 15:48 |
| *** aix has quit IRC | 15:48 | |
| *** kiran-r has joined #openstack-rally | 15:49 | |
| yfried | stpierre: I don't really know. I get lost in bash... | 15:50 |
| yfried | stpierre: so can we merge this? | 15:50 |
| *** kiran-r has quit IRC | 15:51 | |
| openstackgerrit | Chris St. Pierre proposed openstack/rally: Install newer pip in install_rally.sh https://review.openstack.org/197142 | 15:53 |
| stpierre | i want to get redixin's feedback on it first | 15:54 |
| yfried | stpierre: I think you must use -U when package exists | 15:55 |
| stpierre | if you specify a version it will respect that | 15:56 |
| stpierre | so if you run 'pip install pip>=6.0' and pip 6.0 is installed, it won't upgrade it | 15:56 |
| redixin | stpierre, we have line 337 download "$getpip" https://raw.github.com/pypa/pip/master/contrib/get-pip.py | 15:56 |
| stpierre | yeah, but that only runs if installing pip via system packages fails | 15:57 |
| stpierre | and get-pip.py doesn't actually install the latest pip | 15:57 |
| *** tosky has quit IRC | 15:58 | |
| *** frayedknot has quit IRC | 15:59 | |
| kun_huang | anyone knows the status of cloud-certification task? | 16:00 |
| *** aix has joined #openstack-rally | 16:00 | |
| *** aix has quit IRC | 16:01 | |
| *** tosky has joined #openstack-rally | 16:01 | |
| *** frayedknot has joined #openstack-rally | 16:02 | |
| boris-42 | kun_huang: there is patch | 16:02 |
| *** yfried has quit IRC | 16:02 | |
| *** kiran-r has joined #openstack-rally | 16:02 | |
| boris-42 | kun_huang: https://review.openstack.org/#/c/193521/ | 16:02 |
| *** acruz has quit IRC | 16:02 | |
| boris-42 | kun_huang: I belive I will have to continue work on this patch | 16:02 |
| kun_huang | about this work, we will develop another level engine which interacts with scenarios and sla functionality? | 16:05 |
| kun_huang | boris-42: | 16:05 |
| boris-42 | kun_huang: hm not sure that I understand you... | 16:05 |
| kun_huang | or just design some json/yaml task files? | 16:06 |
| boris-42 | kun_huang: it will be just a paramtrized task.. | 16:06 |
| boris-42 | kun_huang: it will be single task | 16:06 |
| redixin | stpierre, what will happen if we run "pip install 'pip>=6.0'" while pip==5.0 is installed? | 16:06 |
| stpierre | it will upgrade pip to >=6.0 | 16:06 |
| boris-42 | pip install -U pip | 16:06 |
| boris-42 | stpierre: ^ | 16:06 |
| kun_huang | it seems not need develop another new module | 16:06 |
| boris-42 | kun_huang: ? | 16:07 |
| redixin | stpierre, so there is no need to add -U, right? | 16:07 |
| stpierre | redixin: not if we specify a version | 16:07 |
| *** fhubik_afk is now known as fhubik | 16:07 | |
| stpierre | boris-42: the only reason i'm concerned about that is if someone is installing rally alongside other openstack stuff without a virtualenv, it seems like that could (theoretically) cause version conflicts | 16:07 |
| kun_huang | https://www.irccloud.com/pastebin/IJA8Ftb2/ | 16:07 |
| stpierre | if we use the version pinning as it is in global requirements, that can't happen | 16:08 |
| kun_huang | boris-42: this looks interesting | 16:08 |
| stpierre | but that's really, really tenuous | 16:08 |
| boris-42 | stpierre: yep that can be terrible.. | 16:08 |
| boris-42 | kun_huang: yep that is the idea | 16:08 |
| stpierre | so i *slightly* lean towards 'pip install pip>=6.0', but it's a very slight lean | 16:08 |
| boris-42 | kun_huang: https://rally.readthedocs.org/en/latest/tutorial/step_5_task_templates.html | 16:08 |
| redixin | imo script should not touch anything if pip>=6.0 is good enough | 16:09 |
| *** anshul has quit IRC | 16:09 | |
| redixin | latest version of pip is not absolutely required | 16:09 |
| redixin | better do not upgrade if it is possible | 16:10 |
| *** karimb has quit IRC | 16:10 | |
| kun_huang | redixin: why? does "pip install -U pip" bother people in some cases? | 16:12 |
| redixin | kun_huang, it will upgrade pip, but this is not necessary if there is already good version of pip | 16:12 |
| kun_huang | got it redixin | 16:13 |
| kun_huang | I think we both agree that pip is unstable usually | 16:14 |
| *** kiran-r has quit IRC | 16:15 | |
| *** e0ne is now known as e0ne_ | 16:23 | |
| *** fhubik has quit IRC | 16:24 | |
| boris-42 | stpierre: btw what do you think https://review.openstack.org/#/c/195966/ ?) | 16:26 |
| *** jaypipes has joined #openstack-rally | 16:26 | |
| *** amalinow has quit IRC | 16:27 | |
| boris-42 | stpierre: not so clear but works on more platforms | 16:27 |
| stpierre | are there platforms that we care about that lack sed -i? | 16:27 |
| *** andreykurilin__ has joined #openstack-rally | 16:28 | |
| *** dontalton has joined #openstack-rally | 16:29 | |
| stpierre | bash portability is *hard*, and it can deeply, deeply compromise readability. i really don't want to just start adding portability for portability's sake. if we honestly want to target running rally on solaris or OS X, then i guess let's do that (and get the CI resources necessary to do so), but i don't think a piecemeal approach to maybe making install_rally.sh kinda work on Solaris or OS X is a good thing. | 16:31 |
| *** e0ne_ has quit IRC | 16:34 | |
| boris-42 | stpierre: I am using Mac OS | 16:34 |
| boris-42 | stpierre: so I would be happy if rally will be simple to install on it | 16:34 |
| *** kiran-r has joined #openstack-rally | 16:34 | |
| stpierre | if we want to target OS X i'm down with that. i just want to make sure we know what our targeted platforms are, because targeting * is a losing plan. | 16:35 |
| albertw1 | any reason that install_rally wasn't written in python? | 16:35 |
| *** e0ne has joined #openstack-rally | 16:36 | |
| stpierre | presumably because it'd be a python script that just made a bunch of subprocess calls to shell commands :) | 16:37 |
| paboldin | albertw1: every language has it's design goals. python is a clue language, not shell language. | 16:38 |
| *** anshul has joined #openstack-rally | 16:42 | |
| *** acruz has joined #openstack-rally | 16:46 | |
| boris-42 | albertw: albertw1 ping | 16:47 |
| albertw1 | well yea there is that in some cases alright! but perhaps some of the shell commands could be better handled for portability by leaving it to python. | 16:47 |
| boris-42 | albertw1: so let's do the next | 16:48 |
| boris-42 | albertw1: we will create CI that tests installation on Solaris | 16:48 |
| boris-42 | albertw1: and make one patch that fixes all issues and merge it | 16:48 |
| boris-42 | albertw1: thougths? | 16:48 |
| *** openstackgerrit has quit IRC | 16:50 | |
| *** openstackgerrit has joined #openstack-rally | 16:51 | |
| stpierre | +1, then we'll be able to see how much complexity the whole thing adds (and what features, like long options, we might lose) | 16:54 |
| stpierre | and make an informed decision | 16:54 |
| stpierre | i like informed decisions :) | 16:54 |
| openstackgerrit | Merged openstack/rally: install_rally.sh: always use wget, not cURL https://review.openstack.org/197079 | 16:56 |
| albertw1 | sounds good. | 16:56 |
| *** kiran-r has quit IRC | 17:02 | |
| *** kiran-r has joined #openstack-rally | 17:03 | |
| *** e0ne has quit IRC | 17:05 | |
| openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Bash-Completion] Fix bash completon for files paths https://review.openstack.org/197186 | 17:14 |
| *** klindgren_ is now known as klindgren | 17:14 | |
| *** agarciam has quit IRC | 17:18 | |
| *** amaretskiy has quit IRC | 17:21 | |
| *** kiran-r has quit IRC | 17:22 | |
| *** oanufriev_ has joined #openstack-rally | 17:27 | |
| *** kiran-r has joined #openstack-rally | 17:37 | |
| *** yfried has joined #openstack-rally | 17:38 | |
| *** kiran-r has quit IRC | 17:45 | |
| openstackgerrit | Oleh Anufriiev proposed openstack/rally: Remove task validation from api.Task.start https://review.openstack.org/158899 | 17:51 |
| *** e0ne has joined #openstack-rally | 17:58 | |
| *** e0ne is now known as e0ne_ | 18:04 | |
| *** temujin_ has joined #openstack-rally | 18:06 | |
| *** oanufriev_ has quit IRC | 18:06 | |
| *** e0ne_ is now known as e0ne | 18:07 | |
| *** temujin_ has quit IRC | 18:10 | |
| *** temujin_ has joined #openstack-rally | 18:12 | |
| *** tosky has quit IRC | 18:14 | |
| *** temujin_ has quit IRC | 18:17 | |
| *** e0ne is now known as e0ne_ | 18:18 | |
| *** rushiagr is now known as rushiagr_away | 18:18 | |
| *** e0ne_ is now known as e0ne | 18:23 | |
| *** temujin_ has joined #openstack-rally | 18:34 | |
| *** davideagnello has quit IRC | 18:39 | |
| *** davideagnello has joined #openstack-rally | 18:44 | |
| *** anshul has quit IRC | 19:01 | |
| *** e0ne is now known as e0ne_ | 19:14 | |
| *** e0ne_ is now known as e0ne | 19:15 | |
| *** echoingumesh has joined #openstack-rally | 19:22 | |
| *** Prashant_ has joined #openstack-rally | 19:23 | |
| *** PrashantS has quit IRC | 19:23 | |
| *** echoingumesh has quit IRC | 19:26 | |
| *** tosky has joined #openstack-rally | 19:29 | |
| *** nihilifer has quit IRC | 19:44 | |
| openstackgerrit | Merged openstack/rally: find should use portable options https://review.openstack.org/196086 | 19:47 |
| *** adiantum has joined #openstack-rally | 19:56 | |
| *** temujin_ has quit IRC | 20:02 | |
| *** temujin_ has joined #openstack-rally | 20:02 | |
| *** rook has joined #openstack-rally | 20:05 | |
| *** frobware_ has joined #openstack-rally | 20:13 | |
| *** nihilifer has joined #openstack-rally | 20:14 | |
| *** echoingumesh has joined #openstack-rally | 20:14 | |
| *** nihilifer has quit IRC | 20:28 | |
| *** gubouvier has quit IRC | 20:34 | |
| *** e0ne has quit IRC | 20:35 | |
| *** amit213 has quit IRC | 20:44 | |
| *** amit213 has joined #openstack-rally | 20:44 | |
| *** amit213 has quit IRC | 20:44 | |
| *** andreykurilin__ has quit IRC | 20:47 | |
| *** karimb has joined #openstack-rally | 20:50 | |
| *** andreykurilin__ has joined #openstack-rally | 21:00 | |
| *** Prashant_ has quit IRC | 21:04 | |
| *** PrashantS has joined #openstack-rally | 21:04 | |
| *** PrashantS has quit IRC | 21:08 | |
| *** temujin_ has quit IRC | 21:41 | |
| *** echoingumesh has quit IRC | 21:44 | |
| *** stpierre has quit IRC | 21:57 | |
| *** frobware_ has quit IRC | 21:59 | |
| *** davideag_ has joined #openstack-rally | 22:09 | |
| *** davideagnello has quit IRC | 22:13 | |
| *** dontalton has quit IRC | 22:37 | |
| *** MaxPC has quit IRC | 22:57 | |
| *** karimb has quit IRC | 22:59 | |
| *** oanufriev__ has joined #openstack-rally | 23:13 | |
| *** mwagner_lap has quit IRC | 23:15 | |
| *** tosky has quit IRC | 23:45 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!