Skip to content
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

[WIP] Configure datachallenge/quicklook #11

Merged
merged 12 commits into from
May 20, 2019
Merged

Conversation

weaverba137
Copy link
Member

@weaverba137 weaverba137 commented May 18, 2019

This PR:

I'm still doing tests on this, don't merge yet.

Items for discussion:

  • While running tests, I've found some data sets that have files that have changed subsequent to backup. What policy should we apply to these?

@weaverba137 weaverba137 requested a review from sbailey May 18, 2019 03:39
@weaverba137 weaverba137 self-assigned this May 18, 2019
@weaverba137 weaverba137 added the WIP Work in progress. label May 18, 2019
@weaverba137
Copy link
Member Author

@sbailey and I discussed this in person, and this is ready to merge. We still need to do the work of cleaning up problematic backups, but that is more of a sociological than a technical challenge.

In the cases of data that has appeared in areas that are already backed up, we need to look at these on a case-by case basis.

@weaverba137 weaverba137 merged commit d610008 into master May 20, 2019
@weaverba137 weaverba137 deleted the configure-quicklook branch May 20, 2019 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

backup datachallenge/quicklook/review-19.1
1 participant