Skip to content
This repository was archived by the owner on Jan 7, 2026. It is now read-only.

Conversation

@emalinowski
Copy link
Contributor

…to match cloud-automation

Description about what this pull request does.

Please make sure to follow the DEV guidelines before asking for review.

New Features

  • Implemented XXX

Breaking Changes

Bug Fixes

Updated fence-config to disable migration by default

Improvements

Dependency updates

Deployment changes

@emalinowski emalinowski requested a review from vzpgb January 30, 2020 22:02
Copy link
Contributor

@vzpgb vzpgb left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

Copy link
Contributor

@paulineribeyre paulineribeyre left a comment

Choose a reason for hiding this comment

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

this file seems pretty out of date, maybe we should update it with all the latest config instead of just one. and i'm not sure we want to set it to false by default, because users will not be aware they need to manually run migrations when they update the service

@emalinowski
Copy link
Contributor Author

this file seems pretty out of date, maybe we should update it with all the latest config instead of just one. and i'm not sure we want to set it to false by default, because users will not be aware they need to manually run migrations when they update the service

@paulineribeyre I set it to false because I thought a couple weeks ago we set cloud automation to not migrate by default. It looks like it may default to true now though 🤷‍♂. I can update the rest of the config to be more up to date though.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants