*** yangyapeng has quit IRC | 00:01 | |
*** ddieterly has joined #openstack-freezer | 00:22 | |
*** yangyapeng has joined #openstack-freezer | 01:11 | |
*** yangyapeng has quit IRC | 01:12 | |
*** EinstCrazy has joined #openstack-freezer | 01:24 | |
*** EinstCrazy has quit IRC | 01:28 | |
*** yangyapeng has joined #openstack-freezer | 01:28 | |
*** EinstCrazy has joined #openstack-freezer | 01:29 | |
*** EinstCrazy has quit IRC | 01:29 | |
*** EinstCrazy has joined #openstack-freezer | 01:30 | |
*** EinstCra_ has joined #openstack-freezer | 01:32 | |
*** EinstCrazy has quit IRC | 01:33 | |
*** EinstCrazy has joined #openstack-freezer | 01:33 | |
*** EinstCra_ has quit IRC | 01:37 | |
*** ddieterly is now known as ddieterly[away] | 01:38 | |
*** EinstCrazy has quit IRC | 01:39 | |
*** EinstCrazy has joined #openstack-freezer | 01:41 | |
*** EinstCrazy has quit IRC | 01:59 | |
*** EinstCrazy has joined #openstack-freezer | 02:00 | |
*** EinstCra_ has joined #openstack-freezer | 02:04 | |
*** EinstCrazy has quit IRC | 02:08 | |
*** ddieterly[away] has quit IRC | 02:11 | |
*** EinstCra_ has quit IRC | 02:22 | |
*** EinstCrazy has joined #openstack-freezer | 02:44 | |
*** EinstCrazy has joined #openstack-freezer | 02:57 | |
*** EinstCrazy has quit IRC | 02:58 | |
*** EinstCrazy has joined #openstack-freezer | 03:00 | |
*** EinstCrazy has quit IRC | 03:01 | |
*** EinstCrazy has joined #openstack-freezer | 03:03 | |
*** EinstCrazy has quit IRC | 03:07 | |
*** EinstCrazy has joined #openstack-freezer | 03:09 | |
*** EinstCra_ has joined #openstack-freezer | 03:12 | |
*** EinstCra_ has quit IRC | 03:15 | |
*** EinstCrazy has quit IRC | 03:15 | |
*** EinstCrazy has joined #openstack-freezer | 03:18 | |
*** EinstCra_ has joined #openstack-freezer | 03:23 | |
*** EinstCrazy has quit IRC | 03:23 | |
*** sc has quit IRC | 03:24 | |
*** sc has joined #openstack-freezer | 03:24 | |
*** sc is now known as Guest40083 | 03:24 | |
*** EinstCra_ has quit IRC | 04:03 | |
*** EinstCrazy has joined #openstack-freezer | 04:04 | |
*** EinstCrazy has quit IRC | 04:05 | |
*** EinstCrazy has joined #openstack-freezer | 04:08 | |
*** EinstCrazy has quit IRC | 04:09 | |
*** EinstCrazy has joined #openstack-freezer | 04:10 | |
*** EinstCrazy has joined #openstack-freezer | 04:12 | |
*** EinstCrazy has joined #openstack-freezer | 04:14 | |
*** EinstCrazy has quit IRC | 04:14 | |
*** EinstCrazy has joined #openstack-freezer | 04:15 | |
*** EinstCrazy has joined #openstack-freezer | 04:16 | |
*** EinstCrazy has joined #openstack-freezer | 04:17 | |
*** EinstCrazy has quit IRC | 04:19 | |
*** EinstCrazy has joined #openstack-freezer | 04:21 | |
*** EinstCra_ has joined #openstack-freezer | 04:22 | |
*** EinstCrazy has quit IRC | 04:22 | |
*** EinstCra_ has quit IRC | 04:22 | |
*** EinstCrazy has joined #openstack-freezer | 04:24 | |
*** EinstCrazy has quit IRC | 04:24 | |
*** EinstCra_ has joined #openstack-freezer | 04:27 | |
*** EinstCra_ has quit IRC | 04:28 | |
*** EinstCrazy has joined #openstack-freezer | 04:32 | |
*** EinstCra_ has joined #openstack-freezer | 04:37 | |
*** EinstCra_ has quit IRC | 04:38 | |
*** EinstCra_ has joined #openstack-freezer | 04:38 | |
*** EinstCra_ has quit IRC | 04:38 | |
*** EinstCrazy has joined #openstack-freezer | 04:43 | |
*** EinstCrazy has joined #openstack-freezer | 04:45 | |
*** EinstCrazy has quit IRC | 04:45 | |
*** EinstCrazy has joined #openstack-freezer | 04:46 | |
*** EinstCrazy has joined #openstack-freezer | 04:46 | |
*** EinstCra_ has joined #openstack-freezer | 04:49 | |
*** EinstCrazy has quit IRC | 04:50 | |
*** EinstCra_ has quit IRC | 04:53 | |
*** EinstCrazy has joined #openstack-freezer | 04:58 | |
*** EinstCrazy has quit IRC | 04:58 | |
*** EinstCrazy has joined #openstack-freezer | 04:58 | |
*** EinstCrazy has quit IRC | 04:59 | |
*** EinstCrazy has joined #openstack-freezer | 05:05 | |
*** EinstCrazy has quit IRC | 05:06 | |
*** EinstCrazy has joined #openstack-freezer | 05:07 | |
*** EinstCrazy has quit IRC | 05:07 | |
*** EinstCrazy has joined #openstack-freezer | 05:07 | |
*** openstackgerrit has quit IRC | 06:17 | |
*** openstackgerrit has joined #openstack-freezer | 06:18 | |
*** lpetrut has joined #openstack-freezer | 06:40 | |
*** reldan has joined #openstack-freezer | 07:21 | |
*** lpetrut has quit IRC | 07:40 | |
*** lpetrut has joined #openstack-freezer | 07:54 | |
*** EinstCra_ has joined #openstack-freezer | 07:57 | |
*** lpetrut has quit IRC | 07:58 | |
*** EinstCrazy has quit IRC | 08:00 | |
*** reldan has quit IRC | 08:11 | |
*** reldan has joined #openstack-freezer | 08:48 | |
*** electrofelix has joined #openstack-freezer | 09:01 | |
*** yangyape_ has joined #openstack-freezer | 09:02 | |
*** yangyap__ has joined #openstack-freezer | 09:04 | |
*** yangyapeng has quit IRC | 09:06 | |
*** yangyape_ has quit IRC | 09:07 | |
reldan | Please review https://review.openstack.org/#/c/311073/ | 09:30 |
---|---|---|
*** daemontool has joined #openstack-freezer | 09:35 | |
*** lpetrut has joined #openstack-freezer | 09:38 | |
openstackgerrit | Cynthia Lopes do Sacramento proposed openstack/freezer-api: Add consistency_checksum parameter on backup metadata https://review.openstack.org/300070 | 10:14 |
openstackgerrit | yapeng Yang proposed openstack/freezer: Fix restore nova for multiple networks https://review.openstack.org/312430 | 10:27 |
*** yangyap__ has quit IRC | 10:29 | |
*** yangyapeng has joined #openstack-freezer | 10:30 | |
*** EinstCra_ has quit IRC | 10:30 | |
*** yangyapeng_ has joined #openstack-freezer | 10:33 | |
*** yangyapeng_ has quit IRC | 10:33 | |
openstackgerrit | Saad Zaher proposed openstack/freezer-api: Updating freezer-api mappings adding no_incremental and overwite https://review.openstack.org/312436 | 10:34 |
openstackgerrit | Merged openstack/freezer: fix some rst syntax error of code block https://review.openstack.org/266813 | 10:34 |
*** yangyapeng has quit IRC | 10:35 | |
openstackgerrit | Merged openstack/freezer: Fix typos https://review.openstack.org/297565 | 10:41 |
*** reldan has quit IRC | 10:47 | |
*** reldan has joined #openstack-freezer | 11:01 | |
*** daemontool has quit IRC | 11:27 | |
*** yangyapeng has joined #openstack-freezer | 11:37 | |
yangyapeng | reldan: hello | 11:55 |
reldan | Hello yangyapeng | 11:55 |
yangyapeng | reldan: https://review.openstack.org/#/c/311073/ why to remove it auto_snap ? | 11:55 |
yangyapeng | reldan: I review it . but I don't know it about the story | 11:56 |
reldan | yangyapeng: Why do you want to have auto_snap? | 11:56 |
reldan | How do you use auto_snap? | 11:56 |
yangyapeng | reldan: no ,I don't have the demand。 | 11:57 |
yangyapeng | reldan: I have not a try to auto_snap. | 11:58 |
reldan | yangyapeng: Exactly. See usually we provide path_to_backup - as path to data you want to backup | 11:58 |
reldan | reldan: and if you want to have lvm snapshot you can provide —snapshot parameter | 11:58 |
reldan | yangyapeng: But we also have this strange mode called auto_snap | 11:59 |
yangyapeng | reldan: oh It is not useful about auto_snap | 11:59 |
yangyapeng | reldan: I see | 11:59 |
reldan | in this case you should provide instead of path_to_backup = %mount_path% + path_to_backup | 11:59 |
reldan | and auto_snap should be equal to path_to_backup without mount_path | 12:00 |
yangyapeng | yes | 12:00 |
reldan | in our code we compare %mount_path% + auto_snap with path_to_backup and if is different we throw an exception | 12:00 |
reldan | but it doesn’t have any sense for me. If I just can provide path to my data and other may be calculated automatically | 12:01 |
reldan | but because we support auto_snap, you can see it in our code, we have all these strange manipulations with path_to_backup and autosnap | 12:02 |
reldan | and many different conditions | 12:02 |
reldan | And I don’t see any pros to keep auto_snap | 12:02 |
szaher | slashme | 12:36 |
slashme | szaher: yup ? | 12:38 |
szaher | check redlan reply about why should we remove auto_snap | 12:39 |
slashme | It makes sense to me. I'd like vannif opinion as well. | 12:41 |
szaher | slashme: 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-freezer | 13:01 | |
openstackgerrit | Merged openstack/freezer-api: Add consistency_checksum parameter on backup metadata https://review.openstack.org/300070 | 13:03 |
reldan | +A here? https://review.openstack.org/#/c/310733/ | 13:03 |
*** ddieterly has joined #openstack-freezer | 13:04 | |
slashme | reldan: that change is waiting for a few last checks | 13:08 |
reldan | good | 13:08 |
reldan | thank you | 13:08 |
slashme | regarding autosnap : | 13:08 |
slashme | Just 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 |
slashme | Second, LVM options may need a bigger rework. | 13:09 |
reldan | slashme: Do we have any user who uses auto_snap? | 13:10 |
reldan | slashme: Sure, it is first essential step in this direction | 13:10 |
reldan | slashme: If we even don’t know what path_to_backup means | 13:10 |
reldan | slashme: sometimes it is path to our data, sometimes it is path to mount directory | 13:10 |
reldan | https://review.openstack.org/#/c/311073/3/freezer/snapshot/lvm.py | 13:11 |
reldan | if backup_opt_dict.snapshot: | 13:11 |
reldan | if not backup_opt_dict.lvm_auto_snap: | 13:11 |
reldan | backup_opt_dict.lvm_auto_snap =\ | 13:11 |
reldan | backup_opt_dict.path_to_backup | 13:11 |
reldan | backup_opt_dict.path_to_backup = '' | 13:11 |
reldan | It is like WHAT? | 13:12 |
slashme | ping daemontool | 13:21 |
*** electrofelix has quit IRC | 13:28 | |
*** electrofelix has joined #openstack-freezer | 13:31 | |
openstackgerrit | Saad Zaher proposed openstack/freezer: Adds checksum before backup and after restore for consistency check. https://review.openstack.org/300080 | 13:38 |
*** electrofelix has quit IRC | 13:52 | |
*** electrofelix has joined #openstack-freezer | 13:52 | |
*** electrofelix has quit IRC | 13:57 | |
*** electrofelix has joined #openstack-freezer | 13:57 | |
szaher | ddieterly: 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 |
ddieterly | something must have changed | 14:13 |
szaher | guys I think we have some problems with gates | 14:14 |
ddieterly | that test does a backup and restore | 14:14 |
szaher | ddieterly: 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 |
szaher | even the same patch you will find me, slashme doing some recheck, reverify | 14:15 |
ddieterly | then there is either a race condition and/or bug | 14:15 |
yangyapeng | I met this situation aslo | 14:16 |
szaher | ddieterly: can we add debug statements like if the backup didn't match the restore we list what inside this directory | 14:17 |
ddieterly | yes, you can | 14:18 |
*** daemontool has joined #openstack-freezer | 14:19 | |
openstackgerrit | Clenimar Filemon proposed openstack/freezer-api: Request service token using identity v3 https://review.openstack.org/304742 | 14:21 |
dmellado | Hi guys, back online after the looong flight | 14:22 |
dmellado | it was my pleasure seeing you around again! ;) | 14:22 |
daemontool | dmellado, likewise :) | 14:23 |
daemontool | slashme, ping | 14:23 |
*** ddieterly is now known as ddieterly[away] | 14:31 | |
*** kelepirci has quit IRC | 14:36 | |
*** ddieterly[away] is now known as ddieterly | 14:39 | |
slashme | Hey daemontool | 14:43 |
slashme | Any idea why the pep8 CI did not catch the line lenght and trailing whitespaces on https://www.youtube.com/watch?v=PivpCKEiQOQ&app=desktop | 14:44 |
slashme | Not the right link \ | 14:45 |
slashme | sorry | 14:45 |
slashme | this: https://review.openstack.org/#/c/300080 | 14:45 |
*** htruta has joined #openstack-freezer | 14:50 | |
*** lpetrut has quit IRC | 15:18 | |
*** lpetrut has joined #openstack-freezer | 15:23 | |
reldan | daemontool: Hi | 15:49 |
daemontool | reldan, hi :) | 15:49 |
daemontool | how are you doing | 15:49 |
daemontool | :) | 15:50 |
reldan | daemontool: I’m great! Thank you. How are you? | 15:50 |
daemontool | we've been talking about you at the Summit :) | 15:50 |
daemontool | all good here | 15:50 |
reldan | daemontool: I hope you talked good thing about me :) | 15:50 |
reldan | daemontool: Could you take a look https://review.openstack.org/#/c/311073/ | 15:50 |
daemontool | reldan, wonderful things | 15:51 |
reldan | daemontool: because you should know it very good | 15:51 |
reldan | daemontool: Thank you :) | 15:51 |
openstackgerrit | Saad Zaher proposed openstack/freezer: Adds checksum before backup and after restore for consistency check. https://review.openstack.org/300080 | 15:52 |
daemontool | reldan, what about if that option is used, the agent exit with a deprecated message, saying what should be used instead? | 15:53 |
daemontool | the rest is good | 15:53 |
reldan | daemontool: Great idea. Do you thin auto_snap can be replaced by path_to_backup and —snapshot without any limitations? | 15:54 |
daemontool | well, if the snapshot underneath can be always guessed... the answer is yes | 15:55 |
daemontool | the day will support more snapshot technologies | 15:55 |
daemontool | we need to make sure the guessing works well | 15:56 |
daemontool | otherwise users have no other option to workaround | 15:56 |
daemontool | other than that... yes | 15:56 |
reldan | daemontool: Just because here with auto_snap we have a problem. Because sometimes snapshot modifies path_to_backup | 15:57 |
reldan | and it means we cannot separate snapshot from modification of config params | 15:57 |
daemontool | I think now the one you are proposing is the best solution | 15:58 |
daemontool | because we support only lvm anyway | 15:58 |
daemontool | so no issues | 15:58 |
daemontool | I think is good | 15:59 |
daemontool | only add a message&log | 15:59 |
daemontool | then that option is called | 15:59 |
reldan | Ok, great. Let’me log a message about deprecation. Thank you! | 16:00 |
daemontool | also the option shouldn't be removed | 16:00 |
daemontool | I-m going to add that | 16:00 |
daemontool | as a comment | 16:00 |
daemontool | ok? | 16:00 |
reldan | Yes, sure. Thank you for the review! | 16:00 |
daemontool | reldan, slashme do you mind if anyone can hep me with the restore side of rsync? | 16:01 |
daemontool | I-m struggling with time | 16:01 |
daemontool | like creating a new commit from the current backup ( I can rebase) | 16:02 |
daemontool | and focus on restore | 16:02 |
daemontool | as the backup works | 16:02 |
*** lpetrut has quit IRC | 16:02 | |
reldan | daemontool: I can try to help you with rsync | 16:03 |
reldan | daemontool: We have a major problem with tar incremental. Did you hear about that? | 16:04 |
daemontool | yes | 16:05 |
daemontool | I think is --incremental is removed when restoring | 16:05 |
daemontool | the issue disappear | 16:05 |
daemontool | not sure | 16:05 |
slashme | daemontool: I'll test that today. | 16:05 |
daemontool | but in that case, the removed files between levels | 16:05 |
daemontool | will be restored and not be removed | 16:06 |
daemontool | also I-m not sure about that either | 16:06 |
daemontool | if that would be the case | 16:06 |
daemontool | we should start the restore for --incremental | 16:06 |
daemontool | and if it fails for some level it is done without --incremental | 16:06 |
daemontool | and so on | 16:06 |
slashme | That's what I want to check | 16:07 |
reldan | daemontool: But it tries to rename a folder before removing | 16:09 |
reldan | daemontool: And cannot rename to already existing folder | 16:09 |
reldan | daemontool: So even if we forbid removing, we still should have the same problem | 16:10 |
daemontool | without incremental I think it just create the new folder | 16:10 |
daemontool | without rename | 16:10 |
daemontool | if the folder does not exists | 16:10 |
daemontool | and untar the content | 16:10 |
daemontool | the only way to see if it works | 16:10 |
daemontool | would be to tried it | 16:10 |
reldan | daemontool: If I understand it correct the tar archive doesn’t contain informaton about data in the directory, only information about renaming | 16:12 |
daemontool | ok | 16:13 |
daemontool | we need to test it | 16:13 |
daemontool | so we'll be sure | 16:13 |
clenimar | hi everyone :) | 16:19 |
clenimar | any hint on how to run freezer's tempest plugin? | 16:19 |
clenimar | i'm working on a patch to make freezer-api request its service token from identity v3 | 16:20 |
clenimar | but those tests keep failing and i couldnt reproduce yet :( | 16:20 |
daemontool | clenimar, :( I think slashme and probably more specifically vannif_ can help you out? | 16:23 |
daemontool | m3m0, maybe? | 16:23 |
daemontool | brb | 16:26 |
clenimar | daemontool: hi. how you're doing? i'll stay around and wait. thanks | 16:27 |
*** daemontool_ has joined #openstack-freezer | 16:27 | |
*** daemontool_ has quit IRC | 16:28 | |
*** daemontool has quit IRC | 16:28 | |
*** daemontool has joined #openstack-freezer | 16:29 | |
slashme | Hi 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 |
ddieterly | clenimar running tempest tests on a development machine is well documented in the readme | 16:35 |
openstackgerrit | Saad Zaher proposed openstack/freezer: Adds checksum before backup and after restore for consistency check. https://review.openstack.org/300080 | 16:43 |
openstackgerrit | Merged openstack/freezer: Allow freezer to use trickle installed system wide https://review.openstack.org/304047 | 16:46 |
clenimar | slashme: thank you :) | 16:48 |
clenimar | ddieterly: 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 IRC | 17:03 | |
*** ddieterly[away] is now known as ddieterly | 17:03 | |
*** lpetrut has joined #openstack-freezer | 17:07 | |
openstackgerrit | Merged openstack/freezer-api: Updating freezer-api mappings adding no_incremental and overwite https://review.openstack.org/312436 | 17:17 |
szaher | ddieterly: 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 |
szaher | I 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 directory | 17:25 |
szaher | 2016-05-04 17:10:02.272 | 2016-05-04 17:10:02.238 | Running freezer integration tests | 17:25 |
szaher | 2016-05-04 17:10:02.273 | + return 0 | 17:25 |
szaher | ./post_test_hook.sh: line 21: cd: /opt/stack/new/freezer-api/tests: No such file or directory | 17:26 |
*** ddieterly is now known as ddieterly[away] | 17:29 | |
*** reldan has joined #openstack-freezer | 17:30 | |
*** electrofelix has quit IRC | 17:36 | |
*** reldan has quit IRC | 17:54 | |
*** reldan has joined #openstack-freezer | 18:05 | |
*** ddieterly[away] is now known as ddieterly | 18:38 | |
m3m0 | ddieterly: which readme? for tempest docs? | 18:40 |
ddieterly | m3m0 yes, in the tempest dir | 18:40 |
ddieterly | in the freezer api repo | 18:41 |
m3m0 | nice :) thanks, do you mind if we move this to wiki.openstack.org? | 18:41 |
openstackgerrit | Memo Garcia proposed openstack/python-freezerclient: Move docs from readme to wiki.openstack.org https://review.openstack.org/312691 | 18:56 |
openstackgerrit | Memo Garcia proposed openstack/freezer-web-ui: Move docs from readme to wiki.openstack.org https://review.openstack.org/312693 | 18:58 |
openstackgerrit | Memo Garcia proposed openstack/freezer-web-ui: Move docs from readme to wiki.openstack.org https://review.openstack.org/312693 | 19:00 |
openstackgerrit | Memo Garcia proposed openstack/freezer: Move docs from readme to wiki.openstack.org https://review.openstack.org/312694 | 19:02 |
openstackgerrit | Memo Garcia proposed openstack/freezer-api: Move docs from readme to wiki.openstack.org https://review.openstack.org/312700 | 19:07 |
openstackgerrit | Memo Garcia proposed openstack/freezer-api: Move docs from readme to wiki.openstack.org https://review.openstack.org/312702 | 19:08 |
*** ddieterly is now known as ddieterly[away] | 19:40 | |
*** openstackgerrit has quit IRC | 19:48 | |
*** openstackgerrit has joined #openstack-freezer | 19:48 | |
*** ddieterly[away] is now known as ddieterly | 20:03 | |
*** reldan has quit IRC | 20:24 | |
*** fawadkhaliq has joined #openstack-freezer | 20:27 | |
*** reldan has joined #openstack-freezer | 20:34 | |
*** fawadk has joined #openstack-freezer | 20:47 | |
*** fawadkhaliq has quit IRC | 20:50 | |
*** ddieterly is now known as ddieterly[away] | 21:12 | |
*** ddieterly[away] is now known as ddieterly | 21:14 | |
*** ddieterly is now known as ddieterly[away] | 21:27 | |
*** ddieterly[away] is now known as ddieterly | 21:50 | |
*** reldan has quit IRC | 21:58 | |
*** reldan has joined #openstack-freezer | 22:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/freezer-api: Updated from global requirements https://review.openstack.org/288699 | 22:07 |
*** ddieterly is now known as ddieterly[away] | 22:07 | |
*** reldan_ has joined #openstack-freezer | 22:17 | |
*** reldan has quit IRC | 22:20 | |
*** reldan_ is now known as reldan | 22:20 | |
*** ddieterly[away] is now known as ddieterly | 22:34 | |
*** ddieterly has quit IRC | 22:35 | |
*** lpetrut has quit IRC | 22:36 | |
*** ddieterly has joined #openstack-freezer | 23:10 | |
*** ddieterly is now known as ddieterly[away] | 23:40 | |
*** ddieterly[away] is now known as ddieterly | 23:42 | |
*** fawadkhaliq has joined #openstack-freezer | 23:50 | |
*** fawadk has quit IRC | 23:52 | |
*** reldan has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!