search

FairwindsOps / pentagon

A framework for building repeatable, containerized, cloud-based infrastructure as code with Kubernetes.
https://www.reactiveops.com
Apache License 2.0
183 stars 25 forks source link

Migration branch name is not unique #146

Closed sudermanjr closed 5 years ago

sudermanjr commented 6 years ago

If previous migrations have been run, there is already a migration branch. This causes an error.

Maybe we should change the name of the branch to be unique to the current version or migration that is being run.

ejether commented 6 years ago

That was intentional. I made the assumption was made that the user would clean up the leftover migration branch and that there wouldn't end up being multiple migrations in flight at a time. Was that a bad assumption? Either way, better messaging around it would probably be useful.

sudermanjr commented 6 years ago

That's fine either way. Better messaging might be good. I just had the migration branch locally and it threw an error. Took me a minute or two to realize.

ejether commented 5 years ago

resolved by #158