*** pradeep has joined #openstack-rally | 00:37 | |
openstackgerrit | Li Yingjun proposed openstack/rally: Set default value for updated_at https://review.openstack.org/176583 | 00:46 |
---|---|---|
*** yingjun has joined #openstack-rally | 00:48 | |
boris-42 | yingjun: hi there | 00:53 |
boris-42 | Wooot Rally is availbe in debian https://twitter.com/PavlovicBoris/status/595384972250370048 | 00:55 |
boris-42 | jlk: ^ | 00:56 |
boris-42 | jaypipes: ^ =) | 00:56 |
*** oanufriev has quit IRC | 01:00 | |
pradeep | boris-42: debain seems to be too fast | 01:01 |
pradeep | :) | 01:01 |
boris-42 | pradeep: ?) | 01:01 |
openstackgerrit | Merged openstack/rally: Add Nova scenario to boot and associate a floating IP https://review.openstack.org/177968 | 01:02 |
pradeep | boris-42: as soon as its moved from stackforge to openstack, they picked up immediatly | 01:02 |
boris-42 | pradeep: so actually Thomas is working on Mirantis and we talk about this before=) | 01:03 |
boris-42 | pradeep: even when rally was on stackforge=) | 01:03 |
pradeep | boris-42: ahh ok :) | 01:03 |
openstackgerrit | Merged openstack/rally: Updated from global requirements https://review.openstack.org/179939 | 01:03 |
boris-42 | pradeep: but in any case now need to do the same in centos=) | 01:04 |
pradeep | boris-42: True :) | 01:04 |
boris-42 | pradeep: heh | 01:07 |
boris-42 | pradeep: 4 morning need to go and sleep | 01:07 |
pradeep | boris-42: GN :) | 01:07 |
*** pradeep has quit IRC | 01:50 | |
openstackgerrit | Merged openstack/rally: Remove _delete_all_servers() nova utils function https://review.openstack.org/179795 | 02:09 |
*** softCloud has quit IRC | 02:10 | |
*** panbalag has quit IRC | 02:21 | |
*** pradeep has joined #openstack-rally | 03:12 | |
*** coolsvap|afk is now known as coolsvap | 03:23 | |
vaidy | boris-42, ping | 03:29 |
*** tfreger has joined #openstack-rally | 03:31 | |
*** anshul has joined #openstack-rally | 03:51 | |
*** yingjun has quit IRC | 03:51 | |
*** nkhare has joined #openstack-rally | 03:53 | |
*** yfried|afk is now known as yfried | 04:01 | |
*** yingjun has joined #openstack-rally | 04:06 | |
*** yingjun has quit IRC | 04:06 | |
*** rdas has joined #openstack-rally | 04:22 | |
boris-42 | vaidy: hi | 04:28 |
vaidy | hey | 04:29 |
vaidy | boris-42, i am almost done with the comments provided | 04:30 |
vaidy | boris-42, i will resend it for review today | 04:30 |
*** klindgren_ has joined #openstack-rally | 04:30 | |
vaidy | https://review.openstack.org/#/c/179739/ | 04:30 |
*** klindgren has quit IRC | 04:31 | |
boris-42 | vaidy: ok great | 04:31 |
vaidy | boris-42, we alredy have the unit test for admin users do you want me to create method for non admin users? | 04:35 |
vaidy | boris-42, or use the same method and call it with different credentials? | 04:36 |
boris-42 | vaidy: not sure what you are asking | 04:36 |
boris-42 | vaidy: in case of writting unit tests it will be better to have 2 methods (one tests admin credintials) another one (admin + users) | 04:36 |
vaidy | boris-42, unit test for validating deployment check command | 04:37 |
boris-42 | vaidy: so 2 methods | 04:37 |
vaidy | boris-42, currently we have it only for admin users | 04:37 |
boris-42 | vaidy: so let me | 04:39 |
boris-42 | vaidy: left some comments | 04:39 |
vaidy | https://github.com/openstack/rally/blob/master/tests/unit/cmd/commands/test_deployment.py#L278-L286 | 04:40 |
boris-42 | vaidy: I left comment | 04:40 |
boris-42 | vaidy: so code should be fixed a bit | 04:40 |
vaidy | boris-42, yeadh | 04:43 |
vaidy | yeah | 04:43 |
vaidy | boris-42, got your point for admin we just validate from deployment and for users we validate from keystone | 04:44 |
vaidy | boris-42, hope my understanding is right | 04:44 |
boris-42 | vaidy: what do you mean validate from deployment? | 04:44 |
boris-42 | vaidy: for admin user are checking roles and listing service (for users you are just trying to authenticate) | 04:45 |
vaidy | boris-42, get it from deployment as i am currently doing | 04:45 |
boris-42 | vaidy: you are getting users as well from deployment | 04:45 |
vaidy | boris-42, client = clients.keystone()?? | 04:47 |
vaidy | boris-42, may be i need more under standing on this one. | 04:47 |
vaidy | boris-42, ok | 04:48 |
boris-42 | clients = osclients.Clients( | 04:48 |
boris-42 | vaidy: ^ | 04:48 |
boris-42 | after that | 04:48 |
vaidy | boris-42, we are authenticating it from keystone | 04:48 |
boris-42 | clients.keystone() will return intialized python keystone client | 04:48 |
boris-42 | with credentials that you pass to oscleints.Cleints() | 04:49 |
vaidy | boris-42, whatever i have coded still holds good | 04:52 |
vaidy | boris-42, I will have to authenticate the users which i got from deployment | 04:53 |
vaidy | boris-42, hope my understanding is right | 04:54 |
boris-42 | vaidy: nope what you did doens't work actually | 04:54 |
*** yfried is now known as yfried|afk | 04:54 | |
boris-42 | vaidy: we are not able to test it in gates unforutantelly | 04:54 |
vaidy | boris-42, it will not because it is not complete | 04:55 |
vaidy | boris-42, its half complete.. :) | 04:55 |
boris-42 | vaidy: nope it just won't work * | 04:56 |
vaidy | boris-42, the later part of your comment is not incorporated yet | 04:56 |
boris-42 | vaidy: you can't call verifed_keystone from not admin user | 04:56 |
vaidy | boris-42, yeah i agree i will have to modify that code | 04:57 |
*** yfried|afk is now known as yfried | 04:58 | |
*** yingjun has joined #openstack-rally | 05:10 | |
*** neeti has joined #openstack-rally | 05:27 | |
*** neeti has quit IRC | 05:28 | |
*** neeti has joined #openstack-rally | 05:30 | |
*** yingjun has quit IRC | 06:02 | |
*** yingjun has joined #openstack-rally | 06:04 | |
*** openstackgerrit has quit IRC | 06:23 | |
*** openstackgerrit has joined #openstack-rally | 06:23 | |
*** svasheka has quit IRC | 06:25 | |
openstackgerrit | Jens Rosenboom proposed openstack/rally: Fix security groups cleanup https://review.openstack.org/179428 | 06:34 |
*** yingjun has quit IRC | 06:36 | |
*** svasheka has joined #openstack-rally | 06:37 | |
*** yingjun has joined #openstack-rally | 06:38 | |
*** yingjun_ has joined #openstack-rally | 06:40 | |
*** yingjun has quit IRC | 06:43 | |
*** e0ne has joined #openstack-rally | 06:51 | |
*** yfried has quit IRC | 06:56 | |
*** pbandzi has joined #openstack-rally | 06:56 | |
*** amalinow has joined #openstack-rally | 07:05 | |
*** e0ne has quit IRC | 07:06 | |
kairat_kushaev | boris-42: Boris, hello | 07:08 |
kairat_kushaev | boris-42: Do you have an example of patch where the heat test is failing with this issue https://bugs.launchpad.net/rally/+bug/1450997? | 07:08 |
openstack | Launchpad bug 1450997 in Rally "HeatStacks are not properly deleted " [High,Triaged] | 07:08 |
kairat_kushaev | boris-42: because it is not clear from logs which test is failing | 07:09 |
kairat_kushaev | boris-42: Oops, found the test definition | 07:09 |
boris-42 | kairat_kushaev: yep | 07:09 |
boris-42 | kairat_kushaev: btw it is not failing 100% of runs | 07:10 |
boris-42 | kairat_kushaev: but it is failing often | 07:10 |
amalinow | While preparing rally tests for my company, I had to add some code for test runners. What are possibilities for upstreaming this code? Should I prepare blueprint? | 07:10 |
*** rdas has quit IRC | 07:10 | |
boris-42 | amalinow: so usually we don't use blueprints | 07:11 |
boris-42 | amalinow: if you have something to discuss (or it affects a lot of code) than you should write a spec | 07:12 |
boris-42 | amalinow: if you already have some code you can just sent it on review | 07:12 |
boris-42 | amalinow: and we will decide does it require spec or not | 07:13 |
*** amalinow_ has joined #openstack-rally | 07:13 | |
boris-42 | amalinow: works for you? | 07:13 |
boris-42 | amalinow: btw what company?) | 07:13 |
amalinow_ | I had to reload, I only saw: "we usually don't use blueprints'" | 07:13 |
boris-42 | amalinow_: ah | 07:13 |
boris-42 | https://www.irccloud.com/pastebin/AAZyXOum | 07:14 |
boris-42 | amalinow_: ^ | 07:14 |
*** amalinow has quit IRC | 07:14 | |
*** agarciam has joined #openstack-rally | 07:16 | |
boris-42 | amalinow_: so | 07:16 |
boris-42 | amalinow_: does that work for you? | 07:16 |
*** amalinow_ has quit IRC | 07:18 | |
*** rdas has joined #openstack-rally | 07:22 | |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 07:29 |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 07:29 |
*** rdas has quit IRC | 07:32 | |
*** karimb has joined #openstack-rally | 07:38 | |
*** rdas has joined #openstack-rally | 07:41 | |
*** amalinow has joined #openstack-rally | 07:42 | |
boris-42 | amalinow: https://www.irccloud.com/pastebin/AAZyXOum | 07:44 |
amalinow | boris-42: soory, I had problems with connection, I changed client | 07:44 |
boris-42 | amalinow: no worries that paste has all my messages | 07:45 |
amalinow | boris-42: ok, looks good to me | 07:45 |
amalinow | boris-42: Intel | 07:45 |
boris-42 | amalinow: ok I will wait for your patches | 07:48 |
amalinow | boris-42: great | 07:51 |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 07:52 |
*** aarefiev has joined #openstack-rally | 07:54 | |
*** redixin has joined #openstack-rally | 07:59 | |
*** jose_lausuch has joined #openstack-rally | 08:00 | |
*** arxcruz has joined #openstack-rally | 08:03 | |
*** yingjun_ has quit IRC | 08:10 | |
*** amaretskiy has joined #openstack-rally | 08:12 | |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 08:14 |
*** fhubik has joined #openstack-rally | 08:14 | |
*** rdas has quit IRC | 08:18 | |
*** fhubik is now known as fhubik_afk | 08:24 | |
andreykurilin | yfried: pong | 08:25 |
*** yingjun has joined #openstack-rally | 08:27 | |
*** e0ne has joined #openstack-rally | 08:28 | |
*** e0ne is now known as e0ne_ | 08:28 | |
*** fhubik_afk is now known as fhubik | 08:28 | |
*** rdas has joined #openstack-rally | 08:39 | |
*** e0ne_ is now known as e0ne | 08:44 | |
*** e0ne is now known as e0ne_ | 08:45 | |
*** e0ne_ is now known as e0ne | 08:46 | |
*** aix has quit IRC | 08:59 | |
openstackgerrit | Sergey Skripnick proposed openstack/rally: Add new script for rally-gate jobs https://review.openstack.org/175549 | 09:01 |
*** fhubik is now known as fhubik_afk | 09:02 | |
openstackgerrit | Kairat Kushaev proposed openstack/rally: Cleanup heat stacks 1 time per tenant https://review.openstack.org/180068 | 09:04 |
*** tfreger has quit IRC | 09:11 | |
*** agarciam1 has joined #openstack-rally | 09:14 | |
*** agarciam has quit IRC | 09:14 | |
*** cdent has joined #openstack-rally | 09:17 | |
*** fhubik_afk is now known as fhubik | 09:18 | |
*** aix has joined #openstack-rally | 09:26 | |
*** msdubov_ has joined #openstack-rally | 09:27 | |
openstackgerrit | Merged openstack/rally: Set default value for updated_at https://review.openstack.org/176583 | 09:31 |
openstackgerrit | Merged openstack/rally: Add Nova scenario to list hypervisors https://review.openstack.org/178805 | 09:32 |
*** yingjun has quit IRC | 09:33 | |
*** igormarnat is now known as imarnat | 09:41 | |
*** fhubik is now known as fhubik_afk | 09:52 | |
*** imarnat is now known as igormarnat | 09:53 | |
*** tfreger has joined #openstack-rally | 09:56 | |
*** panbalag has joined #openstack-rally | 10:04 | |
*** yfried has joined #openstack-rally | 10:11 | |
*** aix has quit IRC | 10:12 | |
*** fhubik_afk is now known as fhubik | 10:12 | |
boris-42 | kairat_kushaev: okay I tested locally your patch | 10:15 |
boris-42 | kairat_kushaev: yep heat stacks are per tenant | 10:16 |
kairat_kushaev | boris-42: Ok, thanx for review | 10:16 |
boris-42 | kairat_kushaev: sorry for not accepting this patch before | 10:17 |
kairat_kushaev | boris-42: I remember there were some troubles with the whole OS | 10:17 |
boris-42 | kairat_kushaev: they are always | 10:17 |
boris-42 | kairat_kushaev: problmes with OS=) | 10:17 |
kairat_kushaev | boris-42: :) | 10:18 |
*** pbandzi has quit IRC | 10:22 | |
*** pbandzi has joined #openstack-rally | 10:22 | |
*** kiran-r has joined #openstack-rally | 10:24 | |
*** aix has joined #openstack-rally | 10:25 | |
*** msdubov_ has quit IRC | 10:30 | |
openstackgerrit | Merged openstack/rally: Shifting _generate_random_name() call to utils https://review.openstack.org/159719 | 10:34 |
*** e0ne is now known as e0ne_ | 10:35 | |
*** e0ne_ is now known as e0ne | 10:36 | |
*** igormarnat is now known as imarnat | 10:37 | |
*** msdubov_ has joined #openstack-rally | 10:49 | |
rvasilets_ | vaidy, hi im here | 10:52 |
*** msdubov_ has quit IRC | 10:53 | |
*** e0ne is now known as e0ne_ | 10:56 | |
*** imarnat is now known as igormarnat | 10:56 | |
*** yfried is now known as yfried|afk | 10:58 | |
openstackgerrit | Roman Vasilets proposed openstack/rally: Add deploy environment murano benchmark https://review.openstack.org/171625 | 10:59 |
openstackgerrit | Roman Vasilets proposed openstack/rally: Add MuranoPackage context https://review.openstack.org/137650 | 10:59 |
*** fhubik has quit IRC | 11:01 | |
*** fhubik has joined #openstack-rally | 11:02 | |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 11:03 |
*** yfried|afk is now known as yfried | 11:04 | |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 11:07 |
openstackgerrit | Evgeny Sikachev proposed openstack/rally: [Sahara] Split Data Sources context https://review.openstack.org/177732 | 11:09 |
vaidy | rvasilets_, hi | 11:12 |
vaidy | rvasilets_, got it clarified from boris-42 | 11:12 |
vaidy | rvasilets_, thanks | 11:12 |
rvasilets_ | ) | 11:13 |
rvasilets_ | sorry for delay | 11:13 |
vaidy | no problem | 11:13 |
*** igormarnat is now known as imarnat | 11:43 | |
*** agarciam1 has quit IRC | 11:44 | |
*** oanufriev has joined #openstack-rally | 11:50 | |
*** e0ne_ is now known as e0ne | 11:51 | |
*** nkhare has quit IRC | 11:52 | |
*** fhubik has quit IRC | 12:05 | |
*** fhubik has joined #openstack-rally | 12:06 | |
*** agarciam has joined #openstack-rally | 12:07 | |
*** agarciam has quit IRC | 12:08 | |
*** softCloud has joined #openstack-rally | 12:09 | |
*** mwagner_lap has quit IRC | 12:16 | |
openstackgerrit | Oleh Anufriiev proposed openstack/rally: Removed task validation from api.Task.start https://review.openstack.org/158899 | 12:16 |
openstackgerrit | Evgeny Sikachev proposed openstack/rally: [Sahara] Split Data Sources context https://review.openstack.org/177732 | 12:17 |
*** fhubik is now known as fhubik_afk | 12:25 | |
openstackgerrit | Evgeny Sikachev proposed openstack/rally: [Sahara] Add Swift to input data sources https://review.openstack.org/178677 | 12:26 |
*** fhubik_afk is now known as fhubik | 12:27 | |
*** mwagner_lap has joined #openstack-rally | 12:33 | |
openstackgerrit | Vitaly Gusev proposed openstack/rally: [Ceilometer] Add context and scenario for resources https://review.openstack.org/165092 | 12:36 |
openstackgerrit | Sergey Skripnick proposed openstack/rally: Improve installation script https://review.openstack.org/162418 | 12:44 |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Clients] Add classmethod osclients.Clients.register https://review.openstack.org/177884 | 12:45 |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Clients] Add classmethod osclients.Clients.register https://review.openstack.org/177884 | 12:47 |
*** msdubov_ has joined #openstack-rally | 12:52 | |
openstackgerrit | Oleh Anufriiev proposed openstack/rally: Removed task validation from api.Task.start https://review.openstack.org/158899 | 12:53 |
*** agarciam has joined #openstack-rally | 12:55 | |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Fuel] (WIP) Add Fuel scenario `list_environments' https://review.openstack.org/178731 | 12:55 |
*** msdubov_ has quit IRC | 12:56 | |
openstackgerrit | Nikita Konovalov proposed openstack/rally: Wait for status https://review.openstack.org/172466 | 13:00 |
*** stpierre has joined #openstack-rally | 13:01 | |
openstackgerrit | Nikita Konovalov proposed openstack/rally: [Sahara] More debug logging https://review.openstack.org/170491 | 13:01 |
openstackgerrit | Nikita Konovalov proposed openstack/rally: [Sahara] More debug logging https://review.openstack.org/170491 | 13:03 |
*** msdubov_ has joined #openstack-rally | 13:10 | |
openstackgerrit | Ilya Tyaptin proposed openstack/rally: [Ceilometer] Improve create_meter function https://review.openstack.org/180142 | 13:13 |
openstackgerrit | Andrey Kurilin proposed openstack/rally: WIP: implement subunit_python_parser https://review.openstack.org/180144 | 13:18 |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Add Nova lock and unlock scenario https://review.openstack.org/175912 | 13:18 |
*** rdas has quit IRC | 13:19 | |
openstackgerrit | Ilya Tyaptin proposed openstack/rally: [Ceilometer] Improve create_meter function https://review.openstack.org/180142 | 13:22 |
*** rook has joined #openstack-rally | 13:28 | |
*** tosky has joined #openstack-rally | 13:37 | |
*** yfried is now known as yfried|afk | 13:52 | |
openstackgerrit | Chris St. Pierre proposed openstack/rally: Add Nova scenario to boot multiple servers in one request https://review.openstack.org/176858 | 13:55 |
*** e0ne is now known as e0ne_ | 13:56 | |
*** e0ne_ is now known as e0ne | 13:59 | |
*** kkuznetsova has joined #openstack-rally | 13:59 | |
*** e0ne is now known as e0ne_ | 13:59 | |
*** e0ne_ is now known as e0ne | 14:00 | |
*** e0ne is now known as e0ne_ | 14:00 | |
*** e0ne_ is now known as e0ne | 14:00 | |
*** imarnat is now known as igormarnat | 14:01 | |
*** igormarnat is now known as imarnat | 14:02 | |
*** nkhare has joined #openstack-rally | 14:05 | |
*** exploreshaifali has joined #openstack-rally | 14:11 | |
*** yfried|afk is now known as yfried | 14:12 | |
openstackgerrit | Ilya Tyaptin proposed openstack/rally: [Ceilometer] Improve create_meter function https://review.openstack.org/180142 | 14:13 |
*** agarciam has quit IRC | 14:14 | |
*** fhubik is now known as fhubik_afk | 14:16 | |
*** pradeep has quit IRC | 14:24 | |
*** fhubik_afk is now known as fhubik | 14:35 | |
*** kiran-r has quit IRC | 14:36 | |
openstackgerrit | Roman Vasilets proposed openstack/rally: Add functional test for task samples https://review.openstack.org/163870 | 14:37 |
openstackgerrit | Roman Vasilets proposed openstack/rally: Add functional test for task samples https://review.openstack.org/163870 | 14:39 |
*** coolsvap is now known as coolsvap|afk | 14:48 | |
*** agarciam has joined #openstack-rally | 14:53 | |
*** amalinow has quit IRC | 14:54 | |
*** fhubik has quit IRC | 14:56 | |
*** nkhare has quit IRC | 15:06 | |
*** jaypipes has quit IRC | 15:13 | |
*** pradeep has joined #openstack-rally | 15:15 | |
*** anshul has quit IRC | 15:16 | |
*** yfried is now known as yfried|afk | 15:18 | |
*** neeti has quit IRC | 15:18 | |
openstackgerrit | Alex Drahon proposed openstack/rally: Add Dockerfile for CentOS-based Rally image https://review.openstack.org/180191 | 15:19 |
*** neeti has joined #openstack-rally | 15:20 | |
openstackgerrit | Sergey Skripnick proposed openstack/rally: DO-NOT-MERGE: add fuel scenario https://review.openstack.org/180195 | 15:20 |
boris-42 | ^ oh so active day | 15:20 |
*** arxcruz has quit IRC | 15:21 | |
*** pradeep has quit IRC | 15:27 | |
*** yfried|afk is now known as yfried | 15:29 | |
*** msdubov_ has quit IRC | 15:35 | |
*** amalinow has joined #openstack-rally | 15:39 | |
*** yfried is now known as yfried|afk | 15:39 | |
*** mwagner_lap has quit IRC | 15:40 | |
openstackgerrit | Roman Vasilets proposed openstack/rally: Fix broken sample scenario for CinderVolumes.create-and-attach-volume https://review.openstack.org/180205 | 15:43 |
openstackgerrit | Vadim Rovachev proposed openstack/rally: Fix validation for rps parameter https://review.openstack.org/170160 | 15:43 |
openstackgerrit | Roman Vasilets proposed openstack/rally: Fix broken sample scenario for CinderVolumes.create-and-attach-volume https://review.openstack.org/180205 | 15:45 |
*** anshul has joined #openstack-rally | 15:50 | |
*** tosky has quit IRC | 15:55 | |
*** neeti has quit IRC | 15:57 | |
openstackgerrit | Alexander Maretskiy proposed openstack/rally: [Fuel] Add Fuel scenario `list_environments' https://review.openstack.org/178731 | 15:58 |
*** yfried|afk is now known as yfried | 15:59 | |
*** agarciam has quit IRC | 16:00 | |
*** tosky has joined #openstack-rally | 16:00 | |
openstackgerrit | Nikita Konovalov proposed openstack/rally: [Sahara] Improve Image context https://review.openstack.org/166859 | 16:03 |
boris-42 | yfried: I bet you will like new meeting times=) | 16:04 |
yfried | boris-42: didn't see that | 16:11 |
boris-42 | yfried: take a look at mailing list | 16:12 |
yfried | boris-42: I'm confused do we have a mtg today? | 16:14 |
yfried | boris-42: or will it be tomorrow at 14:00 or at 13:00? | 16:16 |
*** pradeep has joined #openstack-rally | 16:18 | |
boris-42 | yfried: we will have on 13:30 (release managment meeting( | 16:19 |
boris-42 | yfried: and on 14 regular meeting | 16:19 |
yfried | boris-42: so no mtg in 40 min? | 16:19 |
boris-42 | yfried: no meeting | 16:20 |
boris-42 | yfried: i hope now you will attend more often in meetings | 16:20 |
yfried | boris-42: me too | 16:21 |
yfried | boris-42: these are terrific news | 16:21 |
boris-42 | yfried: ?) | 16:21 |
yfried | boris-42: just saying I love the change. | 16:22 |
boris-42 | yfried: =) | 16:25 |
boris-42 | yfried: say that in mailing list=) | 16:26 |
*** amaretskiy has quit IRC | 16:37 | |
*** amaretskiy1 has joined #openstack-rally | 16:37 | |
*** jose_lausuch has quit IRC | 16:37 | |
*** jaypipes has joined #openstack-rally | 16:38 | |
*** amalinow has quit IRC | 16:38 | |
*** amaretskiy1 has quit IRC | 16:41 | |
yfried | boris-42: btw, where you guys yesterday? | 16:46 |
yfried | boris-42: is StarWars day an offical holiday at Mirantis? | 16:46 |
*** exploreshaifali has quit IRC | 16:53 | |
*** tfreger has quit IRC | 16:53 | |
*** imarnat is now known as igormarnat | 17:00 | |
andreykurilin | yfried: heh | 17:03 |
andreykurilin | yfried: almost | 17:03 |
andreykurilin | yfried: StarWars day was a day off in whole Russia and Ukraine this year;) | 17:04 |
*** e0ne has quit IRC | 17:05 | |
*** harlowja has quit IRC | 17:08 | |
*** harlowja has joined #openstack-rally | 17:09 | |
*** yfried is now known as yfried|afk | 17:14 | |
*** igormarnat is now known as imarnat | 17:29 | |
*** yfried|afk is now known as yfried | 17:30 | |
*** mwagner_lap has joined #openstack-rally | 17:33 | |
*** imarnat is now known as igormarnat | 17:35 | |
boris-42 | yfried: hm I was working LOL | 17:38 |
boris-42 | yfried: whole night lol | 17:39 |
yfried | boris-42: just remiding you I'm blocked on your plugins patches | 17:41 |
boris-42 | yfried: sure but I had to reivew a lot | 17:42 |
boris-42 | yfried: 3 days ago we had 20 OPEN without any comments ready for review patches | 17:42 |
boris-42 | yfried: so you can speed up the process | 17:44 |
boris-42 | yfried: if you help with reviews more | 17:44 |
*** e0ne has joined #openstack-rally | 17:50 | |
*** e0ne is now known as e0ne_ | 17:50 | |
*** e0ne_ is now known as e0ne | 17:51 | |
*** karimb is now known as karimb|champions | 17:54 | |
*** yfried is now known as yfried|afk | 17:55 | |
*** yfried|afk is now known as yfried | 18:02 | |
yfried | boris-42: I know. I was working from home today and usually I do a lot of reviews when I'm at home, but my internet was down for the whole day, so... | 18:03 |
yfried | boris-42: I appologize for that | 18:03 |
yfried | boris-42: still managed to log 2-3 reviews, so ... *success baby!* | 18:04 |
boris-42 | yfried: no worries=) | 18:04 |
boris-42 | yfried: thank you for your work I really appreciate | 18:04 |
yfried | boris-42: tnx | 18:06 |
meteorfox | boris-42: yfried: hey, sorry I missed the meetings. | 18:07 |
boris-42 | meteorfox: there is no meeting today | 18:08 |
yfried | meteorfox: yeah. mtg tomorrow | 18:09 |
boris-42 | meteorfox: take a look at mailing list | 18:10 |
meteorfox | heh | 18:10 |
yfried | boris-42: meteorfox: the ML isn't clear at all | 18:10 |
pradeep | boris-42: Now every reviewer reviwing all of the openstack components right. instead if each reviewer can take primary responsibility of one or 2 components, do you thing, things will speed up? | 18:10 |
yfried | pradeep: ^ what? | 18:11 |
meteorfox | yfried: ok, 14:00 UTC every Wednesday ? | 18:12 |
meteorfox | boris-42: ^^ | 18:12 |
pradeep | s/ Now every reviewer reviewing various rally scenarios & bugs right. instead if each reviewer can take primary responsibility of one or 2 ( nova & cinder scenarios) components, do you thing, things will speed up? | 18:12 |
yfried | meteorfox: 13:30 | 18:12 |
pradeep | yfried: ^ | 18:12 |
meteorfox | yfried: ok, damn, I'm going to be late most of the time | 18:13 |
yfried | pradeep: where have you seen that? what do you meen "reviewer"? do you mean core members? because people can review what they want when they want. I'm also unaware of any sort of reviewing distributions | 18:14 |
meteorfox | yfried: that's 8:30 am here, and when daylight saving ends, it will be at 7:30 am | 18:14 |
pradeep | yfried: yes | 18:14 |
meteorfox | I guess I'll attend meetings during the summers only | 18:14 |
pradeep | core members | 18:14 |
meteorfox | lol | 18:15 |
yfried | meteorfox: well, 13:30 - 14:00 should be release related, and the main mtg starts at 1400 | 18:15 |
*** oanufriev has quit IRC | 18:16 | |
meteorfox | yfried: yeah, I saw that in the chat log. That still will be too early for me, I usually get in between 9:30-10:00 am, so I'm going to run late most of the time | 18:17 |
yfried | meteorfox: can't you pick it up at home and then head on to the office? | 18:19 |
meteorfox | yfried: that's a good idea | 18:19 |
yfried | meteorfox: sorry. current (old) time was 20:00 for me | 18:19 |
meteorfox | yfried: If you don't mind I ask, where do you live? | 18:20 |
yfried | meteorfox: Israel. don't mind at all :) | 18:20 |
yfried | meteorfox: you? | 18:20 |
yfried | meteorfox: also, no idea how you travel, but I have a friend that has a 90min commute and he starts working on the bus via tethering. | 18:20 |
meteorfox | yfried: I'm at Texas, US. heh | 18:20 |
meteorfox | yfried: I drive, is only 15 min drive | 18:21 |
meteorfox | yfried: I live in the US, but I'm actually from Puerto Rico (still is US) | 18:22 |
meteorfox | yfried: anyway, but that's a good suggestion, I think I can attend the meeting at home, and then just head down to the office after that | 18:23 |
*** kiran-r has joined #openstack-rally | 18:24 | |
yfried | meteorfox: glad we can have you on-board | 18:25 |
*** tosky has quit IRC | 18:27 | |
*** logan2 has quit IRC | 18:28 | |
yfried | meteorfox: isn't there some app for IRC to voice that you could use while driving? | 18:30 |
openstackgerrit | Sergey Skripnick proposed openstack/rally: Improve installation script https://review.openstack.org/162418 | 18:30 |
*** logan2 has joined #openstack-rally | 18:31 | |
*** dpaterson has joined #openstack-rally | 18:41 | |
boris-42 | meteorfox: great that time works for you | 18:59 |
boris-42 | meteorfox: I think I will have soon issues with it | 18:59 |
*** kiran-r has quit IRC | 19:00 | |
boris-42 | pradeep: so not sure about reviewing | 19:01 |
boris-42 | pradeep: but I belive the key secret of scaling team is based on next two things | 19:01 |
boris-42 | pradeep: be a team (and work on common rules) | 19:01 |
boris-42 | pradeep: and automate everything that is possible to automate | 19:02 |
boris-42 | if CI says +1 it should mean only one patch is safe for merge | 19:02 |
boris-42 | after that we should just check that code is good written and aligned with road map | 19:02 |
pradeep | boris-42: ok | 19:03 |
boris-42 | pradeep: so that is our goal | 19:04 |
pradeep | boris-42: got it. | 19:04 |
boris-42 | pradeep: for exmple I am going someday to start this https://github.com/stackforge/nerd-reviewer | 19:04 |
boris-42 | pradeep: that will do a part of dummy reviews (write unit tests / do better commit / fill bug report / to big patch / to much files changed / ....) | 19:05 |
boris-42 | pradeep: will be automatically passed | 19:05 |
pradeep | boris-42: i was not aware of this nerd-reviewer. | 19:05 |
* pradeep checking | 19:05 | |
boris-42 | pradeep: so it's not done yet | 19:06 |
boris-42 | need to find the time to work on it | 19:06 |
yfried | pradeep: boris-42: where did the idea for devision of reivews come from? | 19:06 |
boris-42 | yfried: not sure maybe tempest? | 19:07 |
pradeep | yfried: hehe. :) just a thought. | 19:07 |
boris-42 | pradeep: for example | 19:07 |
yfried | pradeep: as boris-42 and I discussed, we need to decouple the code as much as we can so openstack code is not tied up with rally engine code | 19:07 |
boris-42 | yfried: yep | 19:07 |
boris-42 | yfried: and we can have nerd reviewer rule | 19:07 |
boris-42 | yfried: that will check that engine and plugins are not changed in single patch | 19:08 |
yfried | pradeep: that way, reviewers can focus on their strong zones | 19:08 |
boris-42 | yfried: as well if we make 2 repos one with plugins for openstack another with rally core | 19:08 |
yfried | boris-42: only once the split is clear | 19:08 |
boris-42 | yfried: split is VERY VERY OBVIOUS | 19:08 |
boris-42 | yfried: all SLA pulugins => are common | 19:08 |
boris-42 | yfried: all Runners plugins => are common | 19:08 |
boris-42 | yfried: all context (except dummy) => are openstack | 19:09 |
yfried | boris-42: https://review.openstack.org/#/c/179644/ this has to pass | 19:09 |
boris-42 | yfried: all scenarios (except request and dummy) => are openstack | 19:09 |
boris-42 | yfried: ^ could you add that to spec please | 19:09 |
yfried | boris-42: good idea | 19:09 |
boris-42 | yfried: in workitems | 19:09 |
boris-42 | yfried: because really task is clear and easy | 19:10 |
boris-42 | yfried: just pass it instead of your two step solution | 19:10 |
*** anshul has quit IRC | 19:10 | |
boris-42 | yfried: and I will be happy to put +2 on it | 19:10 |
boris-42 | so sorry guys I have to go | 19:10 |
yfried | boris-42: though this is just an observation of current status. no one says we won't have some new ctx that wouldn't be openstack related | 19:10 |
yfried | boris-42: will do | 19:10 |
boris-42 | yfried: and they will go to common/* or something_else/* | 19:11 |
boris-42 | yfried: but current (including on review) are all mostly openstack/* | 19:11 |
boris-42 | yfried: so I belive that we should push some stronger effort | 19:11 |
yfried | boris-42: still, it is blocked by plugin patches | 19:11 |
boris-42 | yfried: nope it is not | 19:11 |
boris-42 | yfried: I can always rebase | 19:11 |
boris-42 | yfried: and I am mostly changing base.py that won't be affected by your refactoring | 19:12 |
yfried | boris-42: IMO it is, cause they all should have a single ancestor that defines them | 19:12 |
boris-42 | yfried: ? | 19:12 |
boris-42 | yfried: there is NO blockers now | 19:12 |
boris-42 | yfried: you will just cause maybe small simple to fix merge conflicts with plugin base | 19:12 |
boris-42 | yfried: everything will work no matter where they are | 19:13 |
boris-42 | yfried: could you just start moving stuff? | 19:13 |
boris-42 | yfried: like first step move all context to rally/plugins/openstack/context/* and rally/plguins/common/context/* | 19:14 |
yfried | boris-42: I just like your patch because it makes things simpler, and more organized. I really want it to land before I start moving things around | 19:14 |
yfried | boris-42: but I agree it's not a HARD blocker | 19:14 |
boris-42 | yfried: yep | 19:14 |
boris-42 | yfried: so you can start moving and we will merge part of your patches | 19:14 |
boris-42 | and when my will be finished we will land them | 19:14 |
boris-42 | Okay now really have to go | 19:15 |
yfried | boris-42: "see" you tomorrow. good night | 19:16 |
*** subscope_ has joined #openstack-rally | 19:27 | |
stpierre | anyone have any clever ideas what might be going on with the tests here? https://review.openstack.org/#/c/174894/ | 19:34 |
stpierre | i've rechecked, same failure | 19:34 |
*** yfried is now known as yfried|afk | 19:35 | |
*** pradeep has quit IRC | 19:40 | |
*** yfried|afk is now known as yfried | 19:55 | |
*** yfried is now known as yfried|afk | 20:05 | |
*** redixin has quit IRC | 20:26 | |
*** pbandzi has quit IRC | 20:33 | |
*** pbandzi has joined #openstack-rally | 20:35 | |
*** pbandzi has quit IRC | 20:36 | |
*** e0ne has quit IRC | 20:38 | |
*** aix has quit IRC | 20:38 | |
*** pbandzi has joined #openstack-rally | 20:38 | |
*** mwagner_lap has quit IRC | 20:42 | |
*** _dpaterson has joined #openstack-rally | 20:50 | |
*** karimb|champions is now known as karimb | 20:53 | |
*** dpaterson has quit IRC | 20:55 | |
*** frobware_ has joined #openstack-rally | 20:59 | |
*** anshul has joined #openstack-rally | 21:13 | |
*** boris-42 has quit IRC | 21:28 | |
*** _dpaterson has quit IRC | 21:37 | |
*** softCloud has quit IRC | 21:46 | |
*** ptoohill is now known as pothole | 21:55 | |
*** cdent has quit IRC | 21:57 | |
*** kun_huang has quit IRC | 22:30 | |
*** dmellado has quit IRC | 22:30 | |
*** kun_huang has joined #openstack-rally | 22:30 | |
*** dmellado has joined #openstack-rally | 22:31 | |
*** subscope_ has quit IRC | 22:34 | |
*** karimb has quit IRC | 23:25 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!