15:02:25 #startmeeting Freezer_weekly_01/10/2015 15:02:25 Meeting started Thu Oct 1 15:02:24 2015 UTC and is due to finish in 60 minutes. The chair is daemontool_. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:02:25 Meeting started Thu Oct 1 15:02:25 2015 UTC and is due to finish in 60 minutes. The chair is daemontool_. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:02:25 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:02:25 The meeting name has been set to 'freezer_weekly_01_10_2015' 15:02:26 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:02:29 The meeting name has been set to 'freezer_weekly_01_10_2015' 15:02:36 Meetings notes available at: https://etherpad.openstack.org/p/freezer_meetings 15:03:18 hello everyone 15:03:23 hi 15:03:57 Hi 15:04:27 So, welcome back daemontool_ 15:04:32 do you want to start ? 15:04:37 yes, thanks 15:04:52 I've been working mainly on the integration gate tests 15:05:25 the objectie of this would be to have a dsvm gate job to automatically execute integration tests on freezer-* 15:05:54 also I've been working on code review 15:05:58 and fixed this bug https://review.openstack.org/#/c/226435/ 15:06:42 ont he next week I'm planning to finish the integrate-gate jobs with vannif and improve the freezer documentation related to the scheduler with example, config json files ,etc 15:07:19 another very important think I'd like to do, is to use oslo.log instead of the logging std lib module we are currently suing, that would align us more to openstackish 15:07:42 yes, I definitely want to help on that, since I've worked on the integration tests in the source code tree 15:07:49 for the integrated-data we are more or less taking barbican as example for the devstack plugins and integration 15:08:22 we need also to release next week the release 1.1.3 on pypi 15:08:42 and create a new branch proposed/kilo as our effort are focused to be kilo compliance 15:08:52 after that we'll switch to be liberty compliance and so on 15:08:58 that's all from me 15:09:27 any question please let me know 15:09:46 regarding the gate job, do you plan to have the in openstack CI ? 15:10:25 I think before we'll set it up in our internal zuul deployment, and afterwards we'll do for the public openstack ci 15:10:41 using our internal ci toolchain will be probably fast in terms of learning experience 15:10:59 so when we'll set it up publicly we'll have less obstacles 15:11:28 I agree 15:13:01 how much will be the effort to switch to oslo.log ? 15:14:10 I think 5 full days 15:14:20 including testing 15:14:37 I'd say 3-5 days 15:15:41 if you need a hand I can help after the integration gate jobs are walking 15:16:05 yes we can do that together, thanks a lot vannif : ) 15:17:05 no worries :) 15:17:21 if that's all I can start 15:17:31 sure, that's all from me 15:17:40 ok, thanks deamontool_ 15:18:01 so, I've been working lvm code 15:18:49 that was needed to fix a bug when the lvm volume to be snapshotted was not mounted on filesystem root 15:19:42 the patch basically reworkad a little how the paths are managed for the lvm snapshots, and simplifies the usage 15:20:49 the cli logic is still compatible with the former, but now the user can just request a snapshot without having to provide mount points and backup paths 15:21:00 freezer will figure out and use some defautls 15:21:46 vannif, so that improved the usability as now much less options needs be used, is that right? 15:22:04 yes. that's right 15:23:35 instead of providing parameters such as lvm_auto_snap, backup_path, lvm_snapname 15:23:52 the user can provide the path to the resource she wants to backup 15:24:06 great, lvm_dirmount is still needed? 15:24:07 and request a lvm snapshot using the cli flag --snapshot 15:24:58 no. when lvm_dir_mount is not provided, freezer uses a default value of /var/lib/freezer 15:25:35 BTW, I'd like to discuss further use cases and checks to this default behavior 15:25:55 ah ok, wouldn't be a reasonable idea to use /var/lib/freezer-{$backup-name} ? so that would avoid conflicts for multiple backup execution? 15:25:59 ok 15:26:18 for example in case the user triggers two concurrent freezer runs which require snapshots 15:26:50 ok 15:27:16 one more question: do we have couple of examples in the README/doc for the simplified execution? 15:28:27 not yet. I'll do that. 15:28:42 ok 15:30:09 besides that I've improved the integration tests while supporting Eldar in testing the parallel storage code 15:32:12 I'm also working on a procedure to set up the testing environment on a clean devstack machine. the purpose is to use it in integration gat jobs 15:32:27 s/gat/gate 15:33:16 vannif, just an idea what do you think if we set a default also for the container? 15:34:29 well, the temporary mount point is just that: temporary. it will be used for the snapshot, chdir into it and backup the filesystem tree 15:34:37 no particular record of it 15:34:46 the user does not need to remember it 15:34:55 +1 15:35:04 the container is actually a resource the user must be aware of 15:35:16 since it's the location of his data :) 15:35:32 well yes, but we can use a default 15:35:58 it's worth having a discussion about it 15:36:19 yes, sure. the default value must be reasonable, well advertised, and possibly searchable easily 15:36:37 yes something like freezer_backup-name 15:36:58 somethign like that 15:37:32 this issue is somewhat linked with the api and the possibility for the user to browse and search through is backups 15:38:35 before taking in consideration the api, though, we can think of a request to the scheduler to go into the (default) container and gather a list of backups 15:38:41 with relative metadata 15:39:19 ok 15:39:22 more evoluted than a simple list of object contained in the object server 15:39:28 yep 15:40:57 I think we can probably move forward 15:41:03 ok, I'll work on the idea to improve the lvm_dirmount default position 15:41:08 yes 15:41:15 any question ? 15:41:28 ok. Eldar 15:41:36 Thank you. 15:42:14 I’m working on peformance improvement for ssh connection, I was trying to increase transport window, but it didn’t help. Looking for new solution. 15:42:42 After that I’m going to iprove overall test coverage 15:42:58 it’s all from my side 15:43:12 yes, we need to work on coverage each patch we release 15:43:25 otherwise untested lines stack up :) 15:43:55 any idea about when it will be in review ? 15:43:57 Agree, agree - I recently have did a lot of changes without proper coverage, my bad 15:44:54 regarding the parallel storage 15:44:59 I hope I will show it tomorrow - because I have no many ideas how to improve it. Or I will postpond it, if no on works 15:45:38 is there a way to use such feature ? 15:46:06 starting with using it for testing purposes 15:46:31 Sorry, I don’t quite understand - what the feature we are talking about? 15:46:39 parallel storage, sorry 15:47:28 Oh, we need have a new mechanism of processing config files / cli. It requires not a major, but big enought refactoring 15:47:49 So we decided to do it in different branch and merge after release 15:48:14 I suppose I will start it next week 15:49:27 ok. you need any help. we were talking about the definition of the configuration options to pass to the freezer agent. 15:49:44 Sure, we can prepare the blueprint together 15:50:59 ok, I'd say let's be clear upon the format of the config options and share them as first step 15:51:08 I'll be glad to help on that 15:51:23 anything else to say ? 15:51:37 Deal - let’s do it together. Because it really requres some refactoring. We are getting the dict from 15:51:46 argparse and use it as a core object 15:51:50 in our architecture 15:52:44 so it's not a trivial refactoring. it's a new object with some responsibilities 15:53:36 good. we need to consider also the switch to cliff then. so that all the code will not need to be thrown away 15:53:39 I believe so, we also can have a lot of checking of input - like validator now 15:54:01 We can do it in two steps, but it is a good idea as well 15:54:26 First step - extract configuration from dictionary and work inside of our code with configuration 15:54:39 And second - replace mechanism of getting arguments from CLI 15:55:40 so that will be one of the major refactoring in the branch we are going to create, isn't it ? 15:56:27 I have no idea ) We can do it in two steps - it will be better. But if we cannot pull small updates before the release - we can do a lot of changes in the branch 15:57:25 ok. when we have a clearer idea we will discuss, as usual :) 15:57:33 Agree ) 15:57:38 anything else ? 15:57:49 Nope, it is all ) 15:58:00 thank you reldan 15:58:05 Thank you 15:58:07 szaher 15:58:15 you turn :) 15:58:26 Hi all 15:58:37 actually I was working on some bugs in freezer 15:58:58 trying to tune some error messages and enhancing freezer 15:59:33 Also there is some bugs related to lvm, I am not reporting and not working on them because vannif is work on lvm staff 15:59:52 vannif, you are going to rewrite it ? 16:00:41 it's merged, feel free to submit any bug you find 16:01:38 Okay, so If i found something I will report it and work on it 16:01:52 yes please 16:01:56 :) 16:02:10 Sure :) 16:03:37 anything else to say ? 16:04:00 Thanks 16:04:29 if you have any doubt, feel free to ask. freezer code base has moved quite a lot recently :) 16:05:19 Yea I can see 16:05:31 I think slashme is not available atm 16:05:49 if you don't mind we can suspend for a few minutes 16:15:39 Hi, I still didn't do anything on the public side I think 16:16:05 ok. you did great on the private though :) 16:16:37 sir m3m0 is on holidays 16:16:51 so I think that is pretty much all 16:17:01 anyone has anything else to say ? 16:19:10 thanks 16:20:25 ok, thank you all. 16:20:27 #endmeeting