Tuesday, 2015-05-05

*** pradeep has joined #openstack-rally00:37
openstackgerritLi Yingjun proposed openstack/rally: Set default value for updated_at  https://review.openstack.org/17658300:46
*** yingjun has joined #openstack-rally00:48
boris-42yingjun: hi there00:53
boris-42Wooot Rally is availbe in debian https://twitter.com/PavlovicBoris/status/59538497225037004800:55
boris-42jlk: ^00:56
boris-42jaypipes: ^ =)00:56
*** oanufriev has quit IRC01:00
pradeepboris-42: debain seems to be too fast01:01
pradeep:)01:01
boris-42pradeep: ?)01:01
openstackgerritMerged openstack/rally: Add Nova scenario to boot and associate a floating IP  https://review.openstack.org/17796801:02
pradeepboris-42: as soon as its moved from stackforge to openstack, they picked up immediatly01:02
boris-42pradeep: so actually Thomas is working on Mirantis and we talk about this before=)01:03
boris-42pradeep: even when rally was on stackforge=)01:03
pradeepboris-42: ahh ok :)01:03
openstackgerritMerged openstack/rally: Updated from global requirements  https://review.openstack.org/17993901:03
boris-42pradeep: but in any case now need to do the same in centos=)01:04
pradeepboris-42: True :)01:04
boris-42pradeep: heh01:07
boris-42pradeep: 4 morning need to go and sleep01:07
pradeepboris-42: GN :)01:07
*** pradeep has quit IRC01:50
openstackgerritMerged openstack/rally: Remove _delete_all_servers() nova utils function  https://review.openstack.org/17979502:09
*** softCloud has quit IRC02:10
*** panbalag has quit IRC02:21
*** pradeep has joined #openstack-rally03:12
*** coolsvap|afk is now known as coolsvap03:23
vaidyboris-42, ping03:29
*** tfreger has joined #openstack-rally03:31
*** anshul has joined #openstack-rally03:51
*** yingjun has quit IRC03:51
*** nkhare has joined #openstack-rally03:53
*** yfried|afk is now known as yfried04:01
*** yingjun has joined #openstack-rally04:06
*** yingjun has quit IRC04:06
*** rdas has joined #openstack-rally04:22
boris-42vaidy: hi04:28
vaidyhey04:29
vaidyboris-42, i am almost done with the comments provided04:30
vaidyboris-42, i will resend it for review today04:30
*** klindgren_ has joined #openstack-rally04:30
vaidyhttps://review.openstack.org/#/c/179739/04:30
*** klindgren has quit IRC04:31
boris-42vaidy: ok great04:31
vaidyboris-42, we alredy have the unit test for admin users do you want me to create method for non admin users?04:35
vaidyboris-42, or use the same method and call it with different credentials?04:36
boris-42vaidy: not sure what you are asking04:36
boris-42vaidy: in case of writting unit tests it will be better to have 2 methods (one tests admin credintials) another one (admin + users)04:36
vaidyboris-42, unit test for validating deployment check command04:37
boris-42vaidy: so 2 methods04:37
vaidyboris-42, currently we have it only for admin users04:37
boris-42vaidy: so let me04:39
boris-42vaidy: left some comments04:39
vaidyhttps://github.com/openstack/rally/blob/master/tests/unit/cmd/commands/test_deployment.py#L278-L28604:40
boris-42vaidy: I left comment04:40
boris-42vaidy: so code should be fixed a bit04:40
vaidyboris-42, yeadh04:43
vaidyyeah04:43
vaidyboris-42, got your point for admin we just validate from deployment and for users we validate from keystone04:44
vaidyboris-42, hope my understanding is right04:44
boris-42vaidy: what do you mean validate from deployment?04:44
boris-42vaidy: for admin user are checking roles and listing service (for users you are just trying to authenticate)04:45
vaidyboris-42, get it from deployment as i am currently doing04:45
boris-42vaidy: you are getting users as well from deployment04:45
vaidyboris-42, client = clients.keystone()??04:47
vaidyboris-42, may be i need more under standing on this one.04:47
vaidyboris-42, ok04:48
boris-42clients = osclients.Clients(04:48
boris-42vaidy: ^04:48
boris-42after that04:48
vaidyboris-42, we are authenticating it from keystone04:48
boris-42clients.keystone() will return intialized python keystone client04:48
boris-42with credentials that you pass to oscleints.Cleints()04:49
vaidyboris-42, whatever i have coded still holds good04:52
vaidyboris-42, I will have to authenticate the users which i got from deployment04:53
vaidyboris-42, hope my understanding is right04:54
boris-42vaidy: nope what you did doens't work actually04:54
*** yfried is now known as yfried|afk04:54
boris-42vaidy: we are not able to test it in gates unforutantelly04:54
vaidyboris-42, it will not because it is not complete04:55
vaidyboris-42, its half complete.. :)04:55
boris-42vaidy: nope it just won't work *04:56
vaidyboris-42, the later part of your comment is not incorporated yet04:56
boris-42vaidy: you can't call verifed_keystone from not admin user04:56
vaidyboris-42, yeah i agree i will have to modify that code04:57
*** yfried|afk is now known as yfried04:58
*** yingjun has joined #openstack-rally05:10
*** neeti has joined #openstack-rally05:27
*** neeti has quit IRC05:28
*** neeti has joined #openstack-rally05:30
*** yingjun has quit IRC06:02
*** yingjun has joined #openstack-rally06:04
*** openstackgerrit has quit IRC06:23
*** openstackgerrit has joined #openstack-rally06:23
*** svasheka has quit IRC06:25
openstackgerritJens Rosenboom proposed openstack/rally: Fix security groups cleanup  https://review.openstack.org/17942806:34
*** yingjun has quit IRC06:36
*** svasheka has joined #openstack-rally06:37
*** yingjun has joined #openstack-rally06:38
*** yingjun_ has joined #openstack-rally06:40
*** yingjun has quit IRC06:43
*** e0ne has joined #openstack-rally06:51
*** yfried has quit IRC06:56
*** pbandzi has joined #openstack-rally06:56
*** amalinow has joined #openstack-rally07:05
*** e0ne has quit IRC07:06
kairat_kushaevboris-42: Boris, hello07:08
kairat_kushaevboris-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
openstackLaunchpad bug 1450997 in Rally "HeatStacks are not properly deleted " [High,Triaged]07:08
kairat_kushaevboris-42: because it is not clear from logs which test is failing07:09
kairat_kushaevboris-42: Oops, found the test definition07:09
boris-42kairat_kushaev: yep07:09
boris-42kairat_kushaev: btw it is not failing 100% of runs07:10
boris-42kairat_kushaev: but it is failing often07:10
amalinowWhile 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 IRC07:10
boris-42amalinow: so usually we don't use blueprints07:11
boris-42amalinow: if you have something to discuss (or it affects a lot of code) than you should write a spec07:12
boris-42amalinow: if you already have some code you can just sent it on review07:12
boris-42amalinow: and we will decide does it require spec or not07:13
*** amalinow_ has joined #openstack-rally07:13
boris-42amalinow: works for you?07:13
boris-42amalinow: btw what company?)07:13
amalinow_I had to reload, I only saw: "we usually don't use blueprints'"07:13
boris-42amalinow_: ah07:13
boris-42https://www.irccloud.com/pastebin/AAZyXOum07:14
boris-42amalinow_: ^07:14
*** amalinow has quit IRC07:14
*** agarciam has joined #openstack-rally07:16
boris-42amalinow_: so07:16
boris-42amalinow_: does that work for you?07:16
*** amalinow_ has quit IRC07:18
*** rdas has joined #openstack-rally07:22
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016007:29
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016007:29
*** rdas has quit IRC07:32
*** karimb has joined #openstack-rally07:38
*** rdas has joined #openstack-rally07:41
*** amalinow has joined #openstack-rally07:42
boris-42amalinow: https://www.irccloud.com/pastebin/AAZyXOum07:44
amalinowboris-42: soory, I had problems with connection, I changed client07:44
boris-42amalinow: no worries that paste has all my messages07:45
amalinowboris-42: ok, looks good to me07:45
amalinowboris-42: Intel07:45
boris-42amalinow: ok I will wait for your patches07:48
amalinowboris-42: great07:51
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016007:52
*** aarefiev has joined #openstack-rally07:54
*** redixin has joined #openstack-rally07:59
*** jose_lausuch has joined #openstack-rally08:00
*** arxcruz has joined #openstack-rally08:03
*** yingjun_ has quit IRC08:10
*** amaretskiy has joined #openstack-rally08:12
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016008:14
*** fhubik has joined #openstack-rally08:14
*** rdas has quit IRC08:18
*** fhubik is now known as fhubik_afk08:24
andreykurilinyfried: pong08:25
*** yingjun has joined #openstack-rally08:27
*** e0ne has joined #openstack-rally08:28
*** e0ne is now known as e0ne_08:28
*** fhubik_afk is now known as fhubik08:28
*** rdas has joined #openstack-rally08:39
*** e0ne_ is now known as e0ne08:44
*** e0ne is now known as e0ne_08:45
*** e0ne_ is now known as e0ne08:46
*** aix has quit IRC08:59
openstackgerritSergey Skripnick proposed openstack/rally: Add new script for rally-gate jobs  https://review.openstack.org/17554909:01
*** fhubik is now known as fhubik_afk09:02
openstackgerritKairat Kushaev proposed openstack/rally: Cleanup heat stacks 1 time per tenant  https://review.openstack.org/18006809:04
*** tfreger has quit IRC09:11
*** agarciam1 has joined #openstack-rally09:14
*** agarciam has quit IRC09:14
*** cdent has joined #openstack-rally09:17
*** fhubik_afk is now known as fhubik09:18
*** aix has joined #openstack-rally09:26
*** msdubov_ has joined #openstack-rally09:27
openstackgerritMerged openstack/rally: Set default value for updated_at  https://review.openstack.org/17658309:31
openstackgerritMerged openstack/rally: Add Nova scenario to list hypervisors  https://review.openstack.org/17880509:32
*** yingjun has quit IRC09:33
*** igormarnat is now known as imarnat09:41
*** fhubik is now known as fhubik_afk09:52
*** imarnat is now known as igormarnat09:53
*** tfreger has joined #openstack-rally09:56
*** panbalag has joined #openstack-rally10:04
*** yfried has joined #openstack-rally10:11
*** aix has quit IRC10:12
*** fhubik_afk is now known as fhubik10:12
boris-42kairat_kushaev: okay I tested locally your patch10:15
boris-42kairat_kushaev: yep heat stacks are per tenant10:16
kairat_kushaevboris-42: Ok, thanx for review10:16
boris-42kairat_kushaev: sorry for not accepting this patch before10:17
kairat_kushaevboris-42: I remember there were some troubles with the whole OS10:17
boris-42kairat_kushaev: they are always10:17
boris-42kairat_kushaev: problmes with OS=)10:17
kairat_kushaevboris-42: :)10:18
*** pbandzi has quit IRC10:22
*** pbandzi has joined #openstack-rally10:22
*** kiran-r has joined #openstack-rally10:24
*** aix has joined #openstack-rally10:25
*** msdubov_ has quit IRC10:30
openstackgerritMerged openstack/rally: Shifting _generate_random_name() call to utils  https://review.openstack.org/15971910:34
*** e0ne is now known as e0ne_10:35
*** e0ne_ is now known as e0ne10:36
*** igormarnat is now known as imarnat10:37
*** msdubov_ has joined #openstack-rally10:49
rvasilets_vaidy, hi im here10:52
*** msdubov_ has quit IRC10:53
*** e0ne is now known as e0ne_10:56
*** imarnat is now known as igormarnat10:56
*** yfried is now known as yfried|afk10:58
openstackgerritRoman Vasilets proposed openstack/rally: Add deploy environment murano benchmark  https://review.openstack.org/17162510:59
openstackgerritRoman Vasilets proposed openstack/rally: Add MuranoPackage context  https://review.openstack.org/13765010:59
*** fhubik has quit IRC11:01
*** fhubik has joined #openstack-rally11:02
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016011:03
*** yfried|afk is now known as yfried11:04
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016011:07
openstackgerritEvgeny Sikachev proposed openstack/rally: [Sahara] Split Data Sources context  https://review.openstack.org/17773211:09
vaidyrvasilets_, hi11:12
vaidyrvasilets_, got it clarified from boris-4211:12
vaidyrvasilets_, thanks11:12
rvasilets_)11:13
rvasilets_sorry for delay11:13
vaidyno problem11:13
*** igormarnat is now known as imarnat11:43
*** agarciam1 has quit IRC11:44
*** oanufriev has joined #openstack-rally11:50
*** e0ne_ is now known as e0ne11:51
*** nkhare has quit IRC11:52
*** fhubik has quit IRC12:05
*** fhubik has joined #openstack-rally12:06
*** agarciam has joined #openstack-rally12:07
*** agarciam has quit IRC12:08
*** softCloud has joined #openstack-rally12:09
*** mwagner_lap has quit IRC12:16
openstackgerritOleh Anufriiev proposed openstack/rally: Removed task validation from api.Task.start  https://review.openstack.org/15889912:16
openstackgerritEvgeny Sikachev proposed openstack/rally: [Sahara] Split Data Sources context  https://review.openstack.org/17773212:17
*** fhubik is now known as fhubik_afk12:25
openstackgerritEvgeny Sikachev proposed openstack/rally: [Sahara] Add Swift to input data sources  https://review.openstack.org/17867712:26
*** fhubik_afk is now known as fhubik12:27
*** mwagner_lap has joined #openstack-rally12:33
openstackgerritVitaly Gusev proposed openstack/rally: [Ceilometer] Add context and scenario for resources  https://review.openstack.org/16509212:36
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241812:44
openstackgerritAlexander Maretskiy proposed openstack/rally: [Clients] Add classmethod osclients.Clients.register  https://review.openstack.org/17788412:45
openstackgerritAlexander Maretskiy proposed openstack/rally: [Clients] Add classmethod osclients.Clients.register  https://review.openstack.org/17788412:47
*** msdubov_ has joined #openstack-rally12:52
openstackgerritOleh Anufriiev proposed openstack/rally: Removed task validation from api.Task.start  https://review.openstack.org/15889912:53
*** agarciam has joined #openstack-rally12:55
openstackgerritAlexander Maretskiy proposed openstack/rally: [Fuel] (WIP) Add Fuel scenario `list_environments'  https://review.openstack.org/17873112:55
*** msdubov_ has quit IRC12:56
openstackgerritNikita Konovalov proposed openstack/rally: Wait for status  https://review.openstack.org/17246613:00
*** stpierre has joined #openstack-rally13:01
openstackgerritNikita Konovalov proposed openstack/rally: [Sahara] More debug logging  https://review.openstack.org/17049113:01
openstackgerritNikita Konovalov proposed openstack/rally: [Sahara] More debug logging  https://review.openstack.org/17049113:03
*** msdubov_ has joined #openstack-rally13:10
openstackgerritIlya Tyaptin proposed openstack/rally: [Ceilometer] Improve create_meter function  https://review.openstack.org/18014213:13
openstackgerritAndrey Kurilin proposed openstack/rally: WIP: implement subunit_python_parser  https://review.openstack.org/18014413:18
openstackgerritChris St. Pierre proposed openstack/rally: Add Nova lock and unlock scenario  https://review.openstack.org/17591213:18
*** rdas has quit IRC13:19
openstackgerritIlya Tyaptin proposed openstack/rally: [Ceilometer] Improve create_meter function  https://review.openstack.org/18014213:22
*** rook has joined #openstack-rally13:28
*** tosky has joined #openstack-rally13:37
*** yfried is now known as yfried|afk13:52
openstackgerritChris St. Pierre proposed openstack/rally: Add Nova scenario to boot multiple servers in one request  https://review.openstack.org/17685813:55
*** e0ne is now known as e0ne_13:56
*** e0ne_ is now known as e0ne13:59
*** kkuznetsova has joined #openstack-rally13:59
*** e0ne is now known as e0ne_13:59
*** e0ne_ is now known as e0ne14:00
*** e0ne is now known as e0ne_14:00
*** e0ne_ is now known as e0ne14:00
*** imarnat is now known as igormarnat14:01
*** igormarnat is now known as imarnat14:02
*** nkhare has joined #openstack-rally14:05
*** exploreshaifali has joined #openstack-rally14:11
*** yfried|afk is now known as yfried14:12
openstackgerritIlya Tyaptin proposed openstack/rally: [Ceilometer] Improve create_meter function  https://review.openstack.org/18014214:13
*** agarciam has quit IRC14:14
*** fhubik is now known as fhubik_afk14:16
*** pradeep has quit IRC14:24
*** fhubik_afk is now known as fhubik14:35
*** kiran-r has quit IRC14:36
openstackgerritRoman Vasilets proposed openstack/rally: Add functional test for task samples  https://review.openstack.org/16387014:37
openstackgerritRoman Vasilets proposed openstack/rally: Add functional test for task samples  https://review.openstack.org/16387014:39
*** coolsvap is now known as coolsvap|afk14:48
*** agarciam has joined #openstack-rally14:53
*** amalinow has quit IRC14:54
*** fhubik has quit IRC14:56
*** nkhare has quit IRC15:06
*** jaypipes has quit IRC15:13
*** pradeep has joined #openstack-rally15:15
*** anshul has quit IRC15:16
*** yfried is now known as yfried|afk15:18
*** neeti has quit IRC15:18
openstackgerritAlex Drahon proposed openstack/rally: Add Dockerfile for CentOS-based Rally image  https://review.openstack.org/18019115:19
*** neeti has joined #openstack-rally15:20
openstackgerritSergey Skripnick proposed openstack/rally: DO-NOT-MERGE: add fuel scenario  https://review.openstack.org/18019515:20
boris-42^ oh so active day15:20
*** arxcruz has quit IRC15:21
*** pradeep has quit IRC15:27
*** yfried|afk is now known as yfried15:29
*** msdubov_ has quit IRC15:35
*** amalinow has joined #openstack-rally15:39
*** yfried is now known as yfried|afk15:39
*** mwagner_lap has quit IRC15:40
openstackgerritRoman Vasilets proposed openstack/rally: Fix broken sample scenario for CinderVolumes.create-and-attach-volume  https://review.openstack.org/18020515:43
openstackgerritVadim Rovachev proposed openstack/rally: Fix validation for rps parameter  https://review.openstack.org/17016015:43
openstackgerritRoman Vasilets proposed openstack/rally: Fix broken sample scenario for CinderVolumes.create-and-attach-volume  https://review.openstack.org/18020515:45
*** anshul has joined #openstack-rally15:50
*** tosky has quit IRC15:55
*** neeti has quit IRC15:57
openstackgerritAlexander Maretskiy proposed openstack/rally: [Fuel] Add Fuel scenario `list_environments'  https://review.openstack.org/17873115:58
*** yfried|afk is now known as yfried15:59
*** agarciam has quit IRC16:00
*** tosky has joined #openstack-rally16:00
openstackgerritNikita Konovalov proposed openstack/rally: [Sahara] Improve Image context  https://review.openstack.org/16685916:03
boris-42yfried: I bet you will like new meeting times=)16:04
yfriedboris-42: didn't see that16:11
boris-42yfried: take a look at mailing list16:12
yfriedboris-42: I'm confused do we have a mtg today?16:14
yfriedboris-42: or will it be tomorrow at 14:00 or at 13:00?16:16
*** pradeep has joined #openstack-rally16:18
boris-42yfried: we will have on 13:30 (release managment meeting(16:19
boris-42yfried: and on 14 regular meeting16:19
yfriedboris-42: so no mtg in 40 min?16:19
boris-42yfried: no meeting16:20
boris-42yfried: i hope now you will attend more often in meetings16:20
yfriedboris-42: me too16:21
yfriedboris-42: these are terrific news16:21
boris-42yfried: ?)16:21
yfriedboris-42: just saying I love the change.16:22
boris-42yfried: =)16:25
boris-42yfried: say that in mailing list=)16:26
*** amaretskiy has quit IRC16:37
*** amaretskiy1 has joined #openstack-rally16:37
*** jose_lausuch has quit IRC16:37
*** jaypipes has joined #openstack-rally16:38
*** amalinow has quit IRC16:38
*** amaretskiy1 has quit IRC16:41
yfriedboris-42: btw, where you guys yesterday?16:46
yfriedboris-42: is StarWars day an offical holiday at Mirantis?16:46
*** exploreshaifali has quit IRC16:53
*** tfreger has quit IRC16:53
*** imarnat is now known as igormarnat17:00
andreykurilinyfried: heh17:03
andreykurilinyfried: almost17:03
andreykurilinyfried: StarWars day was a day off in whole Russia and Ukraine this year;)17:04
*** e0ne has quit IRC17:05
*** harlowja has quit IRC17:08
*** harlowja has joined #openstack-rally17:09
*** yfried is now known as yfried|afk17:14
*** igormarnat is now known as imarnat17:29
*** yfried|afk is now known as yfried17:30
*** mwagner_lap has joined #openstack-rally17:33
*** imarnat is now known as igormarnat17:35
boris-42yfried: hm I was working LOL17:38
boris-42yfried: whole night lol17:39
yfriedboris-42: just remiding you I'm blocked on your plugins patches17:41
boris-42yfried: sure but I had to reivew a lot17:42
boris-42yfried: 3 days ago we had 20 OPEN without any comments ready for review patches17:42
boris-42yfried: so you can speed up the process17:44
boris-42yfried: if you help with reviews more17:44
*** e0ne has joined #openstack-rally17:50
*** e0ne is now known as e0ne_17:50
*** e0ne_ is now known as e0ne17:51
*** karimb is now known as karimb|champions17:54
*** yfried is now known as yfried|afk17:55
*** yfried|afk is now known as yfried18:02
yfriedboris-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
yfriedboris-42: I appologize for that18:03
yfriedboris-42: still managed to log 2-3 reviews, so ... *success baby!*18:04
boris-42yfried: no worries=)18:04
boris-42yfried: thank you for your work I really appreciate18:04
yfriedboris-42: tnx18:06
meteorfoxboris-42: yfried: hey, sorry I missed the meetings.18:07
boris-42meteorfox: there is no meeting today18:08
yfriedmeteorfox: yeah. mtg tomorrow18:09
boris-42meteorfox: take a look at mailing list18:10
meteorfoxheh18:10
yfriedboris-42: meteorfox: the ML isn't clear at all18:10
pradeepboris-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
yfriedpradeep: ^ what?18:11
meteorfoxyfried: ok, 14:00 UTC every Wednesday ?18:12
meteorfoxboris-42: ^^18:12
pradeeps/ 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
yfriedmeteorfox: 13:3018:12
pradeepyfried: ^18:12
meteorfoxyfried: ok, damn, I'm going to be late most of the time18:13
yfriedpradeep: 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 distributions18:14
meteorfoxyfried: that's 8:30 am here, and when daylight saving ends, it will be at 7:30 am18:14
pradeepyfried: yes18:14
meteorfoxI guess I'll attend meetings during the summers only18:14
pradeepcore members18:14
meteorfoxlol18:15
yfriedmeteorfox: well, 13:30 - 14:00 should be release related, and the main mtg starts at 140018:15
*** oanufriev has quit IRC18:16
meteorfoxyfried: 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 time18:17
yfriedmeteorfox: can't you pick it up at home and then head on to the office?18:19
meteorfoxyfried: that's a good idea18:19
yfriedmeteorfox: sorry. current (old) time was 20:00 for me18:19
meteorfoxyfried: If you don't mind I ask, where do you live?18:20
yfriedmeteorfox: Israel. don't mind at all :)18:20
yfriedmeteorfox: you?18:20
yfriedmeteorfox: 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
meteorfoxyfried: I'm at Texas, US. heh18:20
meteorfoxyfried: I drive, is only 15 min drive18:21
meteorfoxyfried: I live in the US, but I'm actually from Puerto Rico (still is US)18:22
meteorfoxyfried: 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 that18:23
*** kiran-r has joined #openstack-rally18:24
yfriedmeteorfox: glad we can have you on-board18:25
*** tosky has quit IRC18:27
*** logan2 has quit IRC18:28
yfriedmeteorfox: isn't there some app for IRC to voice that you could use while driving?18:30
openstackgerritSergey Skripnick proposed openstack/rally: Improve installation script  https://review.openstack.org/16241818:30
*** logan2 has joined #openstack-rally18:31
*** dpaterson has joined #openstack-rally18:41
boris-42meteorfox: great that time works for you18:59
boris-42meteorfox: I think I will have soon issues with it18:59
*** kiran-r has quit IRC19:00
boris-42pradeep: so not sure about reviewing19:01
boris-42pradeep: but I belive the key secret of scaling team is based on next two things19:01
boris-42pradeep: be a team (and work on common rules)19:01
boris-42pradeep: and automate everything that is possible to automate19:02
boris-42if CI says +1 it should mean only one patch is safe for merge19:02
boris-42after that we should just check that code is good written and aligned with road map19:02
pradeepboris-42: ok19:03
boris-42pradeep:  so that is our goal19:04
pradeepboris-42: got it.19:04
boris-42pradeep: for exmple I am going someday to start this https://github.com/stackforge/nerd-reviewer19:04
boris-42pradeep: 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-42pradeep:  will be automatically passed19:05
pradeepboris-42: i was not aware of this nerd-reviewer.19:05
* pradeep checking 19:05
boris-42pradeep: so it's not done yet19:06
boris-42need to find the time to work on it19:06
yfriedpradeep: boris-42: where did the idea for devision of reivews come from?19:06
boris-42yfried: not sure maybe tempest?19:07
pradeepyfried:  hehe. :)  just a thought.19:07
boris-42pradeep: for example19:07
yfriedpradeep: 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 code19:07
boris-42yfried: yep19:07
boris-42yfried: and we can have nerd reviewer rule19:07
boris-42yfried: that will check that engine and plugins are not changed in single patch19:08
yfriedpradeep: that way, reviewers can focus on their strong zones19:08
boris-42yfried: as well if we make 2 repos one with plugins for openstack another with rally core19:08
yfriedboris-42: only once the split is clear19:08
boris-42yfried: split is VERY VERY OBVIOUS19:08
boris-42yfried: all SLA pulugins => are common19:08
boris-42yfried: all Runners plugins => are common19:08
boris-42yfried: all context (except dummy) => are openstack19:09
yfriedboris-42: https://review.openstack.org/#/c/179644/ this has to pass19:09
boris-42yfried:  all scenarios (except request and dummy) => are openstack19:09
boris-42yfried:  ^ could you add that to spec please19:09
yfriedboris-42: good idea19:09
boris-42yfried: in workitems19:09
boris-42yfried: because really task is clear and easy19:10
boris-42yfried: just pass it instead of your two step solution19:10
*** anshul has quit IRC19:10
boris-42yfried: and I will be happy to put +2 on it19:10
boris-42so sorry guys I have to go19:10
yfriedboris-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 related19:10
yfriedboris-42: will do19:10
boris-42yfried: and they will go to common/* or something_else/*19:11
boris-42yfried: but current (including on review) are all mostly openstack/*19:11
boris-42yfried: so I belive that we should push some stronger effort19:11
yfriedboris-42: still, it is blocked by plugin patches19:11
boris-42yfried: nope it is not19:11
boris-42yfried: I can always rebase19:11
boris-42yfried: and I am mostly changing base.py that won't be affected by your refactoring19:12
yfriedboris-42: IMO it is, cause they all should have a single ancestor that defines them19:12
boris-42yfried: ?19:12
boris-42yfried: there is NO blockers now19:12
boris-42yfried: you will just cause maybe small simple to fix merge conflicts with plugin base19:12
boris-42yfried: everything will work no matter where they are19:13
boris-42yfried: could you just start moving stuff?19:13
boris-42yfried: like first step move all context to rally/plugins/openstack/context/* and rally/plguins/common/context/*19:14
yfriedboris-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 around19:14
yfriedboris-42: but I agree it's not a HARD blocker19:14
boris-42yfried: yep19:14
boris-42yfried: so you can start moving and we will merge part of your patches19:14
boris-42and when my will be finished we will land them19:14
boris-42Okay now really have to go19:15
yfriedboris-42: "see" you tomorrow. good night19:16
*** subscope_ has joined #openstack-rally19:27
stpierreanyone have any clever ideas what might be going on with the tests here? https://review.openstack.org/#/c/174894/19:34
stpierrei've rechecked, same failure19:34
*** yfried is now known as yfried|afk19:35
*** pradeep has quit IRC19:40
*** yfried|afk is now known as yfried19:55
*** yfried is now known as yfried|afk20:05
*** redixin has quit IRC20:26
*** pbandzi has quit IRC20:33
*** pbandzi has joined #openstack-rally20:35
*** pbandzi has quit IRC20:36
*** e0ne has quit IRC20:38
*** aix has quit IRC20:38
*** pbandzi has joined #openstack-rally20:38
*** mwagner_lap has quit IRC20:42
*** _dpaterson has joined #openstack-rally20:50
*** karimb|champions is now known as karimb20:53
*** dpaterson has quit IRC20:55
*** frobware_ has joined #openstack-rally20:59
*** anshul has joined #openstack-rally21:13
*** boris-42 has quit IRC21:28
*** _dpaterson has quit IRC21:37
*** softCloud has quit IRC21:46
*** ptoohill is now known as pothole21:55
*** cdent has quit IRC21:57
*** kun_huang has quit IRC22:30
*** dmellado has quit IRC22:30
*** kun_huang has joined #openstack-rally22:30
*** dmellado has joined #openstack-rally22:31
*** subscope_ has quit IRC22:34
*** karimb has quit IRC23:25

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