-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support skipping directories in backup tool #1792
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1792 +/- ##
==========================================
- Coverage 95.03% 89.13% -5.91%
==========================================
Files 139 139
Lines 26649 26657 +8
==========================================
- Hits 25326 23760 -1566
- Misses 1323 2897 +1574
Continue to review full report at Codecov.
|
synapse/tools/backup.py
Outdated
continue | ||
|
||
logger.info(f'making dir:{dstpath}') | ||
s_common.gendir(dstpath) | ||
|
||
for name in fnames: | ||
if name in skipfns: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we make this a glob match so i could specify "layers/*" as a skip?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to do a glob match on the relative dir. I left the exclusion specifically for directories, but we could of course extend it to match filenames as well (either as separate skip option, or just make one generic skip option).
No description provided.