Tommy McNeely

I was actually going to suggest that *most* of the time is spent on the "repo add" portion, so we could try to detect whether the repo was already added...

NOTE: This would replace PR #65

Fixes #67

I will need to re-work this after the other changes

wow, blast from the past. I worked on this while I was staying in my dad's hospital room after a back surgery... don't have any time to dive into it...

Right, I am not sure if it is even possible to do. I am not sure if we should even provide the endpoint. We should probably look at other plugins?...

I think that any token it creates should be cleaned up, before deleting the admin config, there could maybe be an option to not do that?

We hit this issue too, this is not a node or node-vault issue. The latest version of OpenSSL, which apparently the latest node container is using requires the destination to...

We also see this for elasticsearch_license, even on 5.6.14... ``` Warning: Found multiple default providers for elasticsearch_license: shield, xpack; using shield ``` Probably shouldn't have multiple "default" providers :)

This would be nice in the Linux world too, to reboot _if_ there was a kernel update applied. It would be nice if it was conditional.