Skip to content

Comments

Conditionally include vSphere cloud provider secret config#2128

Closed
nikkelma wants to merge 1 commit intorancher:masterfrom
nikkelma:vsphere-cloud-provider-secret
Closed

Conditionally include vSphere cloud provider secret config#2128
nikkelma wants to merge 1 commit intorancher:masterfrom
nikkelma:vsphere-cloud-provider-secret

Conversation

@nikkelma
Copy link

Include vSphere cloud provider secret config instead of literal credentials if provided. Depends on rancher/types#1165.

@nikkelma
Copy link
Author

Is this the entire set of changes needed to support these two fields introduced in v1.11? Unsure if the in-tree or out-of-tree cloud provider is used.

@nikkelma nikkelma changed the title WIP: Conditionally include vSphere cloud provider secret config Conditionally include vSphere cloud provider secret config Jun 18, 2020
@nikkelma nikkelma force-pushed the vsphere-cloud-provider-secret branch from 7d1f8ab to a1bf4ab Compare September 4, 2020 14:57
@stale
Copy link

stale bot commented Nov 3, 2020

This issue/PR has been automatically marked as stale because it has not had activity (commit/comment/label) for 60 days. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/stale label Nov 3, 2020
@nikkelma nikkelma marked this pull request as ready for review November 3, 2020 16:43
@stale stale bot removed the status/stale label Nov 3, 2020
@stale
Copy link

stale bot commented Jan 2, 2021

This issue/PR has been automatically marked as stale because it has not had activity (commit/comment/label) for 60 days. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/stale label Jan 2, 2021
@stale stale bot closed this Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant