Thursday, 2017-04-06

*** zhurong has joined #openstack-freezer00:32
*** jiaopengju has joined #openstack-freezer01:02
*** jiaopeng_ has joined #openstack-freezer01:03
*** jiaopengju has quit IRC01:06
openstackgerritPengju Jiao proposed openstack/freezer master: Add backup arguments validation  https://review.openstack.org/45205803:04
openstackgerritLingyong Xu proposed openstack/freezer master: Use HostAddressOpt for opts that accept IP and hostnames  https://review.openstack.org/45399406:40
openstackgerritLingyong Xu proposed openstack/freezer-api master: Use HostAddressOpt for opts that accept IP and hostnames  https://review.openstack.org/45399506:44
openstackgerritLingyong Xu proposed openstack/freezer master: Use HostAddressOpt for opts that accept IP and hostnames  https://review.openstack.org/45399407:19
openstackgerritLingyong Xu proposed openstack/freezer-api master: Use HostAddressOpt for opts that accept IP and hostnames  https://review.openstack.org/45399507:27
*** c00281451 has joined #openstack-freezer08:13
*** c00281451 is now known as zengchen08:13
*** zengchen has quit IRC08:43
*** zengchen has joined #openstack-freezer08:44
*** vnogin has joined #openstack-freezer09:13
*** daemontool has joined #openstack-freezer09:15
*** electrofelix has joined #openstack-freezer09:53
*** lpetrut1 has joined #openstack-freezer10:16
*** lpetrut1 has quit IRC10:21
*** dstepanenko has joined #openstack-freezer10:27
*** jiaopeng_ has quit IRC10:32
openstackgerritMerged openstack/python-freezerclient master: Follow up to I87f0add4dc2bcece832197cfc88e2d531a80a326  https://review.openstack.org/45310110:35
*** deepak_jon has quit IRC10:38
*** deepak_jon has joined #openstack-freezer10:39
openstackgerritDeepak Jon proposed openstack/freezer-web-ui master: Removed py27dj18 from tox  https://review.openstack.org/45410011:00
openstackgerritMerged openstack/freezer-web-ui master: Replace python version for tox  https://review.openstack.org/45362111:02
*** deepak_jon has quit IRC11:19
*** deepak_jon has joined #openstack-freezer11:19
vnoginRemoved py27dj18 from tox11:25
vnogindeepak_jon: could you please fix merge conflict in "Removed py27dj18 from tox"?11:25
deepak_jonyes, i am looking on this11:25
deepak_joncan you help me out11:26
vnogindeepak_jon: actually my patch has been merged prior yours. You need to check current version of tox file and change your patch accordingly11:27
deepak_jonBut now only i have clone it11:28
vnogindeepak_jon: here is patch https://review.openstack.org/#/c/453621/11:28
deepak_jonanyway, i will abandon it if your patch get merged then11:28
vnogindeepak_jon: next time check proposed patches :) https://review.openstack.org/#/q/project:%22%5E.*freezer.*%22status:open11:30
deepak_jonsure, Thanks :)11:31
deepak_jonCan we have osc in freezerclient?11:31
daemontooldeepak_jon, yes, but does it provide any tangile advantage?11:38
deepak_jonAs Every component is having the openstackclient and user can have only once client to work on instead of seperate client11:40
dstepanenkoHi everyone11:54
openstackgerritDmitry Stepanenko proposed openstack/freezer-api master: Implement migrations and config opts for oslo.db  https://review.openstack.org/45211511:58
dstepanenkocould you please take a look into https://review.openstack.org/#/c/441207/ and https://review.openstack.org/#/c/452115/11:59
*** deepak_jon has quit IRC12:12
*** deepak_jon has joined #openstack-freezer12:13
daemontooldeepak_jon, ok, thanks12:21
daemontooldstepanenko, about https://review.openstack.org/#/c/441207/ you have a question from m3m0  there_12:21
daemontool?12:21
dstepanenkohey Fausto12:22
m3m0hello :)12:22
dstepanenkoyes, we're having conversation with m3mo :)12:22
dstepanenkoI've asked last question just a moment ago12:22
dstepanenkobtw, sorry for misspeling such a short nickname :) m3m0 is a right version of it12:23
daemontoollike what would be the actual content os src_file in the db12:23
daemontoolI think it can be a plaintext12:24
daemontoolin it12:24
dstepanenkobtw, I didn't look into this part of code. I was sure that src_file is jsut a path, not content. Am I wrong?12:24
dstepanenkoif yes, my  last answer isn't corrrect, sorry then12:25
daemontoolI don't know, lol :)12:26
dstepanenkoactually, I look into unit tests for freezer and figured out that for elastic search this is a path to file12:26
dstepanenkoabsolute path12:26
daemontoolyes12:26
dstepanenkoso, I think it would be file path, not content of file12:26
daemontoolyes, you are right12:27
dstepanenkojust a text containing absolute file path12:27
m3m0but we need to save that file in the database somehow, the path in the filesystem is when freeezer reads conf_file12:28
m3m0when running the actions12:28
m3m0but why not save them in a json format? key->value12:28
m3m0not so sure if I explained myself correctly :)12:30
dstepanenkodo you wanna say we need to freeze content of the file?12:31
dstepanenkoin elasticsearch this worked without freezing it12:31
*** zhurong has quit IRC12:42
m3m0sorry for the delay, mmm, as far as I remember we do save the content of those parameters in ES, as a nested json inside a job12:49
m3m0all the backup_name, container, max_level, etc. Are stored in ES12:49
*** jiaopengju has joined #openstack-freezer12:57
dstepanenkoaccording to this https://github.com/openstack/freezer-api/blob/master/freezer_api/tests/freezer_api_tempest_plugin/tests/api/test_api_actions.py#L74 you're wrong12:57
dstepanenkom3m0 ^^12:57
dstepanenkohmm, sorry, it seems that my logic isn't correct and maybe you're right, m3m0. Need to dive into code a bit13:01
vnogindaemontool: ping13:05
*** catintheroof has joined #openstack-freezer13:05
daemontoolvnogin,  yes13:06
vnogindaemontool: Fausto, what do you think about path in this patch https://review.openstack.org/#/c/452115/15/etc/freezer/freezer-api.conf.sample ?13:15
jiaopengjuAnyone help to review this patch please? https://review.openstack.org/#/c/451648/13:28
jiaopengjuand this. https://review.openstack.org/#/c/453547/13:28
daemontoolvnogin,  one sec13:33
szaherHello guys13:54
szahermeeting in 10 minutes ?13:54
dstepanenkoszaher: hey! I believe, yes13:54
szaherdstepanenko: Cool!13:54
szaherhttps://etherpad.openstack.org/p/freezer_meetings13:54
szaherGuys, plz add the topic that you want to discuss to this etherpad https://etherpad.openstack.org/p/freezer_meetings13:58
dstepanenkoszaher: I see you added link fitering out all the active reviews. Should I add link to my review to the list of topics in this case?14:02
openstackgerritRuslan Aliev proposed openstack/freezer master: Os-brick engine for backup volumes  https://review.openstack.org/44762914:04
szaherdaemontool raliev slashme m3m0 dstepanenko deepak_jon vnogin jiaopengju can you join #openstack-meeting-alt for freezer meeting ?14:13
*** jiaopengju02 has joined #openstack-freezer14:29
*** jiaopengju02 has left #openstack-freezer14:30
*** deepak_jon has quit IRC14:47
*** deepak_jon has joined #openstack-freezer14:48
*** dstepanenko has quit IRC14:52
*** jiaopengju02 has joined #openstack-freezer15:04
jiaopengjuIs the meeting over?15:05
*** dstepanenko has joined #openstack-freezer15:07
vnoginyep http://eavesdrop.openstack.org/meetings/freezer/2017/freezer.2017-04-06-14.02.log.html15:07
jiaopengjuthanks15:09
jiaopengjuI saw the message 'continue in freezer room' and think that it will continue right now.15:11
*** zengchen has quit IRC15:16
*** zengchen has joined #openstack-freezer15:16
*** jiaopengju has quit IRC15:19
*** jiaopengju02 has quit IRC15:20
*** jiaopengju has joined #openstack-freezer15:21
*** jiaopengju has quit IRC15:22
*** jiaopengju has joined #openstack-freezer15:27
*** jiaopengju has quit IRC15:27
szaherdstepanenko: I don't think so the filter will get it15:28
*** yangyapeng has joined #openstack-freezer15:28
vnoginszaher: daemontool m3m0 dstepanenko guys, let's resolve folder issue here https://review.openstack.org/#/c/452115/15/etc/freezer/freezer-api.conf.sample15:36
vnoginszaher: daemontool m3m0 dstepanenko if provided folder is ok for you  can revert my -115:37
openstackgerritRuslan Aliev proposed openstack/freezer master: Os-brick engine for backup volumes  https://review.openstack.org/44762915:43
daemontoolvnogin,  ok need 20 min to review it15:44
daemontoolwill try to do it by EOD15:44
vnogindaemontool: thx Fausto15:47
*** zhurong has joined #openstack-freezer15:47
*** zhurong has quit IRC15:49
vnoginm3m0: Guillermo, could you please drop your current email, I want to invite you to the meeting. I've tried this one guillermo.ramirez-garcia@hp.com  and it seems it's invalid15:50
openstackgerritRuslan Aliev proposed openstack/freezer master: Os-brick engine for backup volumes  https://review.openstack.org/44762916:01
szahervnogin: guillermo.ramirez-garcia@hpe.com16:03
vnoginszaher: thanks :)16:04
*** vnogin has quit IRC16:08
openstackgerritRuslan Aliev proposed openstack/freezer master: Os-brick engine for backup volumes  https://review.openstack.org/44762916:12
*** deepak_jon has quit IRC16:24
*** deepak_jon has joined #openstack-freezer16:24
*** yangyapeng has quit IRC16:25
*** vnogin has joined #openstack-freezer17:47
*** vnogin has quit IRC17:51
*** electrofelix has quit IRC18:07
*** clenimar has quit IRC18:18
*** zengchen has quit IRC18:19
*** daemontool has quit IRC18:48
*** daemontool has joined #openstack-freezer19:09
*** daemontool has quit IRC19:18
*** openstack has joined #openstack-freezer19:47
*** openstackgerrit has quit IRC21:07
*** catintheroof has quit IRC21:38
*** openstack has joined #openstack-freezer22:02

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