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

Create backup file during migration from ACME V1 to ACME V2 #3191

Merged
merged 5 commits into from Apr 16, 2018

Conversation

nmengin
Copy link
Contributor

@nmengin nmengin commented Apr 16, 2018

What does this PR do?

Creating a backup file which contains the ACME V1 data during migration to ACME V2 file format.

Motivation

Allow easier rollback to ACME V1.

More

  • Added/updated documentation

Additional Notes

Fix #3162 by checking ACME certificate content before set and get actions.

Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏


!!! note
When Træfik is launched in a container, do not forget to create a volume of the parent folder to get the backup file on the host.
Otherwise, the backup file will be delete when the conatiner will be stopeed and Træfik will not generate it again.
Copy link
Member

@juliens juliens Apr 16, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise, the backup file will be deleted when the container will be stopped and Træfik will not generate it again.

Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@mmatur mmatur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👏

@ldez ldez removed the bot/no-merge label Apr 16, 2018
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@traefiker traefiker force-pushed the feature/add-acme-file-backup branch from ef73c08 to a083f4b Compare April 16, 2018 17:14
@traefiker traefiker merged commit a2e03e3 into traefik:v1.6 Apr 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/acme kind/enhancement a new or improved feature. size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants