-
Notifications
You must be signed in to change notification settings - Fork 33
Get credential from Cattle's service account #77
base: master
Are you sure you want to change the base?
Conversation
@raphink @cloudnautique ready to review. |
@raphink could you please review this? The idea is to be able to deploy stack from a CI that would run on an environment without needing to pass credentials. For example we could have a small DroneCI container in every Rancher environment that could be used by our developers to deploy their stacks. |
// Override with new environment variables | ||
apiURL = d.Get("cattle_url").(string) | ||
accessKey = d.Get("cattle_access_key").(string) | ||
secretKey = d.Get("cattle_secret_key").(string) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why duplicate the keys if they have exactly the same role?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So that we can use the environment variables provided by cattle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I see…
@mcanevet do you still want to merge this? |
If it does not break what exists, yes. |
Implement #76