Wednesday, 2016-05-04

*** yangyapeng has quit IRC00:01
*** ddieterly has joined #openstack-freezer00:22
*** yangyapeng has joined #openstack-freezer01:11
*** yangyapeng has quit IRC01:12
*** EinstCrazy has joined #openstack-freezer01:24
*** EinstCrazy has quit IRC01:28
*** yangyapeng has joined #openstack-freezer01:28
*** EinstCrazy has joined #openstack-freezer01:29
*** EinstCrazy has quit IRC01:29
*** EinstCrazy has joined #openstack-freezer01:30
*** EinstCra_ has joined #openstack-freezer01:32
*** EinstCrazy has quit IRC01:33
*** EinstCrazy has joined #openstack-freezer01:33
*** EinstCra_ has quit IRC01:37
*** ddieterly is now known as ddieterly[away]01:38
*** EinstCrazy has quit IRC01:39
*** EinstCrazy has joined #openstack-freezer01:41
*** EinstCrazy has quit IRC01:59
*** EinstCrazy has joined #openstack-freezer02:00
*** EinstCra_ has joined #openstack-freezer02:04
*** EinstCrazy has quit IRC02:08
*** ddieterly[away] has quit IRC02:11
*** EinstCra_ has quit IRC02:22
*** EinstCrazy has joined #openstack-freezer02:44
*** EinstCrazy has joined #openstack-freezer02:57
*** EinstCrazy has quit IRC02:58
*** EinstCrazy has joined #openstack-freezer03:00
*** EinstCrazy has quit IRC03:01
*** EinstCrazy has joined #openstack-freezer03:03
*** EinstCrazy has quit IRC03:07
*** EinstCrazy has joined #openstack-freezer03:09
*** EinstCra_ has joined #openstack-freezer03:12
*** EinstCra_ has quit IRC03:15
*** EinstCrazy has quit IRC03:15
*** EinstCrazy has joined #openstack-freezer03:18
*** EinstCra_ has joined #openstack-freezer03:23
*** EinstCrazy has quit IRC03:23
*** sc has quit IRC03:24
*** sc has joined #openstack-freezer03:24
*** sc is now known as Guest4008303:24
*** EinstCra_ has quit IRC04:03
*** EinstCrazy has joined #openstack-freezer04:04
*** EinstCrazy has quit IRC04:05
*** EinstCrazy has joined #openstack-freezer04:08
*** EinstCrazy has quit IRC04:09
*** EinstCrazy has joined #openstack-freezer04:10
*** EinstCrazy has joined #openstack-freezer04:12
*** EinstCrazy has joined #openstack-freezer04:14
*** EinstCrazy has quit IRC04:14
*** EinstCrazy has joined #openstack-freezer04:15
*** EinstCrazy has joined #openstack-freezer04:16
*** EinstCrazy has joined #openstack-freezer04:17
*** EinstCrazy has quit IRC04:19
*** EinstCrazy has joined #openstack-freezer04:21
*** EinstCra_ has joined #openstack-freezer04:22
*** EinstCrazy has quit IRC04:22
*** EinstCra_ has quit IRC04:22
*** EinstCrazy has joined #openstack-freezer04:24
*** EinstCrazy has quit IRC04:24
*** EinstCra_ has joined #openstack-freezer04:27
*** EinstCra_ has quit IRC04:28
*** EinstCrazy has joined #openstack-freezer04:32
*** EinstCra_ has joined #openstack-freezer04:37
*** EinstCra_ has quit IRC04:38
*** EinstCra_ has joined #openstack-freezer04:38
*** EinstCra_ has quit IRC04:38
*** EinstCrazy has joined #openstack-freezer04:43
*** EinstCrazy has joined #openstack-freezer04:45
*** EinstCrazy has quit IRC04:45
*** EinstCrazy has joined #openstack-freezer04:46
*** EinstCrazy has joined #openstack-freezer04:46
*** EinstCra_ has joined #openstack-freezer04:49
*** EinstCrazy has quit IRC04:50
*** EinstCra_ has quit IRC04:53
*** EinstCrazy has joined #openstack-freezer04:58
*** EinstCrazy has quit IRC04:58
*** EinstCrazy has joined #openstack-freezer04:58
*** EinstCrazy has quit IRC04:59
*** EinstCrazy has joined #openstack-freezer05:05
*** EinstCrazy has quit IRC05:06
*** EinstCrazy has joined #openstack-freezer05:07
*** EinstCrazy has quit IRC05:07
*** EinstCrazy has joined #openstack-freezer05:07
*** openstackgerrit has quit IRC06:17
*** openstackgerrit has joined #openstack-freezer06:18
*** lpetrut has joined #openstack-freezer06:40
*** reldan has joined #openstack-freezer07:21
*** lpetrut has quit IRC07:40
*** lpetrut has joined #openstack-freezer07:54
*** EinstCra_ has joined #openstack-freezer07:57
*** lpetrut has quit IRC07:58
*** EinstCrazy has quit IRC08:00
*** reldan has quit IRC08:11
*** reldan has joined #openstack-freezer08:48
*** electrofelix has joined #openstack-freezer09:01
*** yangyape_ has joined #openstack-freezer09:02
*** yangyap__ has joined #openstack-freezer09:04
*** yangyapeng has quit IRC09:06
*** yangyape_ has quit IRC09:07
reldanPlease review https://review.openstack.org/#/c/311073/09:30
*** daemontool has joined #openstack-freezer09:35
*** lpetrut has joined #openstack-freezer09:38
openstackgerritCynthia Lopes do Sacramento proposed openstack/freezer-api: Add consistency_checksum parameter on backup metadata  https://review.openstack.org/30007010:14
openstackgerrityapeng Yang proposed openstack/freezer: Fix restore nova for multiple networks  https://review.openstack.org/31243010:27
*** yangyap__ has quit IRC10:29
*** yangyapeng has joined #openstack-freezer10:30
*** EinstCra_ has quit IRC10:30
*** yangyapeng_ has joined #openstack-freezer10:33
*** yangyapeng_ has quit IRC10:33
openstackgerritSaad Zaher proposed openstack/freezer-api: Updating freezer-api mappings adding no_incremental and overwite  https://review.openstack.org/31243610:34
openstackgerritMerged openstack/freezer: fix some rst syntax error of code block  https://review.openstack.org/26681310:34
*** yangyapeng has quit IRC10:35
openstackgerritMerged openstack/freezer: Fix typos  https://review.openstack.org/29756510:41
*** reldan has quit IRC10:47
*** reldan has joined #openstack-freezer11:01
*** daemontool has quit IRC11:27
*** yangyapeng has joined #openstack-freezer11:37
yangyapengreldan: hello11:55
reldanHello yangyapeng11:55
yangyapengreldan: https://review.openstack.org/#/c/311073/  why to remove it auto_snap ?11:55
yangyapengreldan: I review it . but I don't know it about the story11:56
reldanyangyapeng: Why do you want to have auto_snap?11:56
reldanHow do you use auto_snap?11:56
yangyapengreldan: no ,I don't have the demand。11:57
yangyapengreldan: I have not a try to auto_snap.11:58
reldanyangyapeng: Exactly. See usually we provide path_to_backup - as path to data you want to backup11:58
reldanreldan: and if you want to have lvm snapshot you can provide —snapshot parameter11:58
reldanyangyapeng: But we also have this strange mode called auto_snap11:59
yangyapengreldan: oh It is not useful about auto_snap11:59
yangyapengreldan: I see11:59
reldanin this case you should provide instead of path_to_backup =  %mount_path% + path_to_backup11:59
reldanand auto_snap should be equal to path_to_backup without mount_path12:00
yangyapengyes12:00
reldanin our code we compare %mount_path% + auto_snap with path_to_backup and if is different we throw an exception12:00
reldanbut it doesn’t have any sense for me. If I just can provide path to my data and other may be calculated automatically12:01
reldanbut because we support auto_snap, you can see it in our code, we have all these strange manipulations with path_to_backup and autosnap12:02
reldanand many different conditions12:02
reldanAnd I don’t see any pros to keep auto_snap12:02
szaherslashme12:36
slashmeszaher: yup ?12:38
szahercheck redlan reply about why should we remove auto_snap12:39
slashmeIt makes sense to me. I'd like vannif opinion as well.12:41
szaherslashme: I sent him the review and asked him about his opinion on hipchat and skype :) but he is not here I guess !12:56
*** vannif_ has joined #openstack-freezer13:01
openstackgerritMerged openstack/freezer-api: Add consistency_checksum parameter on backup metadata  https://review.openstack.org/30007013:03
reldan+A here? https://review.openstack.org/#/c/310733/13:03
*** ddieterly has joined #openstack-freezer13:04
slashmereldan: that change is waiting for a few last checks13:08
reldangood13:08
reldanthank you13:08
slashmeregarding autosnap :13:08
slashmeJust checked with vannif_ and I think he is right. First we should not remove options without deprecating them for at least one version.13:09
slashmeSecond, LVM options may need a bigger rework.13:09
reldanslashme: Do we have any user who uses auto_snap?13:10
reldanslashme: Sure, it is first essential step in this direction13:10
reldanslashme: If we even don’t know what path_to_backup means13:10
reldanslashme: sometimes it is path to our data, sometimes it is path to mount directory13:10
reldanhttps://review.openstack.org/#/c/311073/3/freezer/snapshot/lvm.py13:11
reldanif backup_opt_dict.snapshot:13:11
reldan        if not backup_opt_dict.lvm_auto_snap:13:11
reldanbackup_opt_dict.lvm_auto_snap =\13:11
reldan                backup_opt_dict.path_to_backup13:11
reldan            backup_opt_dict.path_to_backup = ''13:11
reldanIt is like WHAT?13:12
slashmeping daemontool13:21
*** electrofelix has quit IRC13:28
*** electrofelix has joined #openstack-freezer13:31
openstackgerritSaad Zaher proposed openstack/freezer: Adds checksum before backup and after restore for consistency check.  https://review.openstack.org/30008013:38
*** electrofelix has quit IRC13:52
*** electrofelix has joined #openstack-freezer13:52
*** electrofelix has quit IRC13:57
*** electrofelix has joined #openstack-freezer13:57
szaherddieterly: http://logs.openstack.org/80/300080/19/check/gate-freezer-devstack-dsvm/11ecd12/console.html There is something wrong with the tempest gate. This error always happens !14:11
ddieterlysomething must have changed14:13
szaherguys I think we have some problems with gates14:14
ddieterlythat test does a backup and restore14:14
szaherddieterly: I can send you some patches where I just did a simple recheck and second time the tests passed and everything was fine !\14:14
szahereven the same patch you will find me, slashme doing some recheck, reverify14:15
ddieterlythen there is either a race condition and/or bug14:15
yangyapengI met this situation aslo14:16
szaherddieterly: can we add debug statements like if the backup didn't match the restore we list what inside this directory14:17
ddieterlyyes, you can14:18
*** daemontool has joined #openstack-freezer14:19
openstackgerritClenimar Filemon proposed openstack/freezer-api: Request service token using identity v3  https://review.openstack.org/30474214:21
dmelladoHi guys, back online after the looong flight14:22
dmelladoit was my pleasure seeing you around again! ;)14:22
daemontooldmellado,  likewise :)14:23
daemontoolslashme, ping14:23
*** ddieterly is now known as ddieterly[away]14:31
*** kelepirci has quit IRC14:36
*** ddieterly[away] is now known as ddieterly14:39
slashmeHey daemontool14:43
slashmeAny idea why the pep8 CI did not catch the line lenght and trailing whitespaces on https://www.youtube.com/watch?v=PivpCKEiQOQ&app=desktop14:44
slashmeNot the right link \14:45
slashmesorry14:45
slashmethis: https://review.openstack.org/#/c/30008014:45
*** htruta has joined #openstack-freezer14:50
*** lpetrut has quit IRC15:18
*** lpetrut has joined #openstack-freezer15:23
reldandaemontool: Hi15:49
daemontoolreldan,  hi :)15:49
daemontoolhow are you doing15:49
daemontool:)15:50
reldandaemontool: I’m great! Thank you. How are you?15:50
daemontoolwe've been talking about you at the Summit :)15:50
daemontoolall good here15:50
reldandaemontool: I hope you talked good thing about me :)15:50
reldandaemontool: Could you take a look https://review.openstack.org/#/c/311073/15:50
daemontoolreldan,  wonderful things15:51
reldandaemontool: because you should know it very good15:51
reldandaemontool: Thank you :)15:51
openstackgerritSaad Zaher proposed openstack/freezer: Adds checksum before backup and after restore for consistency check.  https://review.openstack.org/30008015:52
daemontoolreldan,  what about if that option is used, the agent exit with a deprecated message, saying what should be used instead?15:53
daemontoolthe rest is good15:53
reldandaemontool: Great idea. Do you thin  auto_snap can be replaced by path_to_backup and —snapshot without any limitations?15:54
daemontoolwell, if the snapshot underneath can be always guessed... the answer is yes15:55
daemontoolthe day will support more snapshot technologies15:55
daemontoolwe need to make sure the guessing works well15:56
daemontoolotherwise users have no other option to workaround15:56
daemontoolother than that... yes15:56
reldandaemontool: Just because here with auto_snap we have a problem. Because sometimes snapshot modifies path_to_backup15:57
reldanand it means we cannot separate snapshot from modification of config params15:57
daemontoolI think now the one you are proposing is the best solution15:58
daemontoolbecause we support only lvm anyway15:58
daemontoolso no issues15:58
daemontoolI think is good15:59
daemontoolonly add a message&log15:59
daemontoolthen that option is called15:59
reldanOk, great. Let’me log a message about deprecation. Thank you!16:00
daemontoolalso the option shouldn't be removed16:00
daemontoolI-m going to add that16:00
daemontoolas a comment16:00
daemontoolok?16:00
reldanYes, sure. Thank you for the review!16:00
daemontoolreldan, slashme  do you mind if anyone can hep me with the restore side of  rsync?16:01
daemontoolI-m struggling with time16:01
daemontoollike creating a new commit from the current backup ( I can rebase)16:02
daemontooland focus on restore16:02
daemontoolas the backup works16:02
*** lpetrut has quit IRC16:02
reldandaemontool: I can try to help you with rsync16:03
reldandaemontool: We have a major problem with tar incremental. Did you hear about that?16:04
daemontoolyes16:05
daemontoolI think is --incremental is removed when restoring16:05
daemontoolthe issue disappear16:05
daemontoolnot sure16:05
slashmedaemontool: I'll test that today.16:05
daemontoolbut in that case, the removed files between levels16:05
daemontoolwill be restored and not be removed16:06
daemontoolalso I-m not sure about that either16:06
daemontoolif that would be the case16:06
daemontoolwe should start the restore for --incremental16:06
daemontooland if it fails for some level it is done without --incremental16:06
daemontooland so on16:06
slashmeThat's what I want to check16:07
reldandaemontool: But it tries to rename a folder before removing16:09
reldandaemontool: And cannot rename to already existing folder16:09
reldandaemontool: So even if we forbid removing, we still should have the same problem16:10
daemontoolwithout incremental I think it just create the new folder16:10
daemontoolwithout rename16:10
daemontoolif the folder does not exists16:10
daemontooland untar the content16:10
daemontoolthe only way to see if it works16:10
daemontoolwould be to tried it16:10
reldandaemontool: If I understand it correct the tar archive doesn’t contain informaton about data in the directory, only information about renaming16:12
daemontoolok16:13
daemontoolwe need to test it16:13
daemontoolso we'll be sure16:13
clenimarhi everyone :)16:19
clenimarany hint on how to run freezer's tempest plugin?16:19
clenimari'm working on a patch to make freezer-api request its service token from identity v316:20
clenimarbut those tests keep failing and i couldnt reproduce yet :(16:20
daemontoolclenimar,  :( I think slashme  and probably more specifically vannif_  can help you out?16:23
daemontoolm3m0, maybe?16:23
daemontoolbrb16:26
clenimardaemontool: hi. how you're doing? i'll stay around and wait. thanks16:27
*** daemontool_ has joined #openstack-freezer16:27
*** daemontool_ has quit IRC16:28
*** daemontool has quit IRC16:28
*** daemontool has joined #openstack-freezer16:29
slashmeHi clenimar. How are you ? Just issued a recheck on your change to see if the error was not caused by a faulty deployment. ddieterly is the best person to help you with tempest tests.16:33
ddieterlyclenimar running tempest tests on a development machine is well documented in the readme16:35
openstackgerritSaad Zaher proposed openstack/freezer: Adds checksum before backup and after restore for consistency check.  https://review.openstack.org/30008016:43
openstackgerritMerged openstack/freezer: Allow freezer to use trickle installed system wide  https://review.openstack.org/30404716:46
clenimarslashme: thank you :)16:48
clenimarddieterly: oh, you're right. i'll take a look... thank you :)16:48
*** ddieterly is now known as ddieterly[away]16:55
*** reldan has quit IRC17:03
*** ddieterly[away] is now known as ddieterly17:03
*** lpetrut has joined #openstack-freezer17:07
openstackgerritMerged openstack/freezer-api: Updating freezer-api mappings adding no_incremental and overwite  https://review.openstack.org/31243617:17
szaherddieterly: What do you think about this https://review.openstack.org/#/c/300080/21/freezer/tests/freezer_tempest_plugin/tests/api/test_swift_backup.py ?17:20
szaherI guess there is something wrong with devstack gate 2016-05-04 17:10:02.271 | 2016-05-04 17:10:02.236 | ./post_test_hook.sh: line 21: cd: /opt/stack/new/freezer-api/tests: No such file or directory17:25
szaher2016-05-04 17:10:02.272 | 2016-05-04 17:10:02.238 | Running freezer integration tests17:25
szaher2016-05-04 17:10:02.273 | + return 017:25
szaher ./post_test_hook.sh: line 21: cd: /opt/stack/new/freezer-api/tests: No such file or directory17:26
*** ddieterly is now known as ddieterly[away]17:29
*** reldan has joined #openstack-freezer17:30
*** electrofelix has quit IRC17:36
*** reldan has quit IRC17:54
*** reldan has joined #openstack-freezer18:05
*** ddieterly[away] is now known as ddieterly18:38
m3m0ddieterly: which readme? for tempest docs?18:40
ddieterlym3m0 yes, in the tempest dir18:40
ddieterlyin the freezer api repo18:41
m3m0nice :) thanks, do you mind if we move this to wiki.openstack.org?18:41
openstackgerritMemo Garcia proposed openstack/python-freezerclient: Move docs from readme to wiki.openstack.org  https://review.openstack.org/31269118:56
openstackgerritMemo Garcia proposed openstack/freezer-web-ui: Move docs from readme to wiki.openstack.org  https://review.openstack.org/31269318:58
openstackgerritMemo Garcia proposed openstack/freezer-web-ui: Move docs from readme to wiki.openstack.org  https://review.openstack.org/31269319:00
openstackgerritMemo Garcia proposed openstack/freezer: Move docs from readme to wiki.openstack.org  https://review.openstack.org/31269419:02
openstackgerritMemo Garcia proposed openstack/freezer-api: Move docs from readme to wiki.openstack.org  https://review.openstack.org/31270019:07
openstackgerritMemo Garcia proposed openstack/freezer-api: Move docs from readme to wiki.openstack.org  https://review.openstack.org/31270219:08
*** ddieterly is now known as ddieterly[away]19:40
*** openstackgerrit has quit IRC19:48
*** openstackgerrit has joined #openstack-freezer19:48
*** ddieterly[away] is now known as ddieterly20:03
*** reldan has quit IRC20:24
*** fawadkhaliq has joined #openstack-freezer20:27
*** reldan has joined #openstack-freezer20:34
*** fawadk has joined #openstack-freezer20:47
*** fawadkhaliq has quit IRC20:50
*** ddieterly is now known as ddieterly[away]21:12
*** ddieterly[away] is now known as ddieterly21:14
*** ddieterly is now known as ddieterly[away]21:27
*** ddieterly[away] is now known as ddieterly21:50
*** reldan has quit IRC21:58
*** reldan has joined #openstack-freezer22:05
openstackgerritOpenStack Proposal Bot proposed openstack/freezer-api: Updated from global requirements  https://review.openstack.org/28869922:07
*** ddieterly is now known as ddieterly[away]22:07
*** reldan_ has joined #openstack-freezer22:17
*** reldan has quit IRC22:20
*** reldan_ is now known as reldan22:20
*** ddieterly[away] is now known as ddieterly22:34
*** ddieterly has quit IRC22:35
*** lpetrut has quit IRC22:36
*** ddieterly has joined #openstack-freezer23:10
*** ddieterly is now known as ddieterly[away]23:40
*** ddieterly[away] is now known as ddieterly23:42
*** fawadkhaliq has joined #openstack-freezer23:50
*** fawadk has quit IRC23:52
*** reldan has quit IRC23:57

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