14:05:17 #startmeeting Rally 14:05:17 Meeting started Mon Apr 10 14:05:17 2017 UTC and is due to finish in 60 minutes. The chair is andreykurilin. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:05:18 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:05:20 The meeting name has been set to 'rally' 14:05:37 [From Gitter] andreykurilin : @tohin @astarove @shihai1991 14:05:47 hi all! 14:05:50 Hi 14:05:52 [From Gitter] astarove : рш 14:05:55 [From Gitter] astarove : hi 14:05:59 hi 14:06:44 chenhb are you here?) 14:07:06 ok, let's start 14:07:17 #topic Status of Rally 0.9.1 14:07:35 Last week we decided to make a release of 0.9.1 14:07:50 which includes a bunch of bug-fixes 14:08:06 BUT, setuptools-scm release before that :) 14:08:12 it broke our CI 14:08:24 and we were blocked with that 14:08:57 I pushed a change to rally master with a workaround and our master branch is unblocked now 14:09:20 as for stable/0.9 branch, we need to wait for https://review.openstack.org/#/c/455234/ 14:10:19 After that we will be able to deliver 0.9.1 14:10:20 hi 14:10:25 hi 14:10:53 Also, today one more bug-fix was proposed to rally master - https://review.openstack.org/#/c/455173/ 14:10:57 i am late again 14:11:12 chenhb: np 14:12:01 If comments will be addressed and 455173 change will be merged before 0.9.1, I'm for backporting it to 0.9 branch too 14:12:26 but it is not so critical bug-fix to block releasing 0.9.1 14:12:43 that is all about 0.9.1 release 14:12:48 Any questions? 14:13:28 nothing from me 14:13:41 no question from my side 14:13:56 ok 14:14:16 # topic Free discussion 14:14:26 Actually I do not have other topic to discuss 14:14:37 so "free-discussion" section is open 14:14:38 :) 14:15:49 pls help review this patch https://review.openstack.org/#/c/431624/ ;) 14:16:20 andreykurilin: hi 14:17:31 recently, rcherrueau from inria asked you about Rally + OSprofiler 14:17:39 Do you remember that? :D 14:17:47 hai_shi: yeah... I remember about it. It is in my opened tabs. Actually, I dislike how is validation in embedded in CLI and not sure that we should save in database errors like "unable to open file". Today I want to experiment with validation and then will return to you patch 14:17:57 tovin07: yes, I remember about it 14:17:58 :) 14:18:36 so, do you guys have any plan to make it work? 14:19:20 tovin07, my reply to inria folks maybe was not accurate, so they thought that it will take too much human-resources which they do not have :( 14:19:27 actually, it should not be a hard task 14:20:01 currently, I see that there is no way to benchmark OpenStack with OSprofiler enabled for services with current initialization of OpenStack clients in Rally 14:20:21 it is true 14:20:42 but as I said, it should not be hard to implement it 14:20:51 for example, we init nova client with something like this: nova.Client(something but **kwargs) 14:20:58 yup 14:22:04 so, I think it’s good to do that soon ==> we can measure overhead of using OSprofiler all-the-time in production 14:23:16 I agree that it is not a hard to implement :D 14:23:53 tovin07: [implementation details] as far as I understand how osprofiler works, while osprofiler initialization it uses global variable, so initialization of one openstack client means initialzation fro all clients. That is why there is no need to transmit profiler_enable to nova client itself. 14:24:22 If you are interested in implementation, I'm ok to extend rally with osprofiler feauture 14:24:25 we can start from the spec 14:25:04 oh yes 14:25:10 I will follow it if I can 14:25:50 thank you andreykurilin 14:25:57 that’s all from me 14:26:33 Ok, I'll start a spec about it. Can you share your email(you can do it in direct messages to avoid logging)? I'll add you to thread with inrea folks when the spec will ready. 14:27:05 okay, I’ll send a dirrect message 14:27:09 Anything else to discuss? 14:28:14 https://review.openstack.org/#/c/432138/6/rally/plugins/openstack/services/storage/block.py 14:28:21 a small question 14:29:11 if cinder_v1 have not update_volumes() function, it looks we can not add this update_volues() in BlockStorage 14:29:11 we call osprofile.trace while calling client? 14:30:18 update_volume_type? 14:30:26 yep 14:31:18 If update_volume_type is a specific for Cinder V2, only Cinder V2 should include that method and scenario should have proper validator that checks that that cinder v2 is enabled. 14:32:13 chenhb: no we will just initialize osprofiler before each iteration and save trace id into iteration result then it will be displayed in reports as text area. 14:32:20 it is a first step for integration 14:32:47 then, it will be possible to embed osprofiler report into rally's report 14:33:10 got it 14:33:34 fine 14:33:53 andreykurilin: (y) 14:34:00 so we can get a unify report 14:34:14 *we will able to get 14:34:19 now we cannot do it :) 14:36:18 anything else? 14:36:46 no question 14:37:03 nothing from me 14:38:15 ok, let's finish 14:38:27 #endmeeting