-
Notifications
You must be signed in to change notification settings - Fork 39
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
Rebrand rename driver #57
Conversation
@jmarhee I pushed up fixes for my last bit of feedback. If we merge this we will want to get the repo rename in immediately afterward. Now that I've looked at the code a little closer, I'm wondering if Users can alias drivers, so if I'm ok with I'll approve this PR since it is passing tests and I've taken a closer look. I'll leave it to you to offer a final review and merge. |
…cessfully provisioned a machine Signed-off-by: Joseph D. Marhee <jmarhee@interiorae.com>
Signed-off-by: Joseph D. Marhee <jmarhee@interiorae.com>
Signed-off-by: Joseph D. Marhee <jmarhee@interiorae.com>
Signed-off-by: Marques Johansson <mjohansson@equinix.com> Signed-off-by: Joseph D. Marhee <jmarhee@interiorae.com>
Signed-off-by: Marques Johansson <mjohansson@equinix.com> Signed-off-by: Joseph D. Marhee <jmarhee@interiorae.com>
…e would expect from an environment variable-- this isn't breaking example, but as a debug step is clearer/more consistent. Signed-off-by: Joseph D. Marhee <jmarhee@interiorae.com>
I began updating this package in conjunction with the Rancher UI driver, which for whatever reason, was assuming the driver name would be one word rather than two, so I elected to use
metal
rather thanequinix-metal
-- this would be a trivial thing to revert, if we prefer the full name, but opening the PR using what I know to be a working use of the rebranded name and providing the context.Fixes #56