search

nebtex / vault-migrator

migrate vault data between different physical backends
Apache License 2.0
53 stars 11 forks source link

Update to Vault 0.8.1 #3

Closed willglynn closed 6 years ago

willglynn commented 6 years ago

I wanted to use vault-migrator with etcd but ran into trouble using the prebuilt binaries. This changeset fixes it.

I updated vault-migrator to track upstream via glide up github.com/hashicorp/vault, which brings it to ~0.8.1. This updated all the backends but broke vault-migrator because the internal APIs changed. Somehow they refactored it such that the complete list of physical backends exists only in vault/cli/commands.go, so this PR fishes it back out again in func init() and replaces physical.NewBackend() with a replacement newBackend().

Also, I guess this PR gofmt + goimports main.go because my editor does that on save.

willglynn commented 6 years ago

The CLA for this project is a little weird:

Thank you for your interest in menshend

I guess it's copy/pasted from this repo?

You accept and agree to the following terms and conditions for Your present and future Contributions submitted to the Company. In return, the Company shall not use Your Contributions in a way that is contrary to the public benefit or inconsistent with its bylaws in effect at the time of the Contribution.

Which Company? It's not defined in this document. Who's the counterparty to this agreement?

criloz commented 6 years ago

@willglynn I just delete that cla, is not make much sense in this repo, also I will add mit license soon. thanks for your contribution.

at the moment the docker build and binary release are broken, I need to pass them to circle-ci. while you need to compile the repo, I will release this with the tag 0.2.0

Thanks again