Skip to content
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

Make kubeconfig attributes available as resource output #378

Merged
merged 4 commits into from
Feb 13, 2023

Conversation

SkYNewZ
Copy link
Contributor

@SkYNewZ SkYNewZ commented Feb 10, 2023

Fix #325

SkYNewZ and others added 2 commits February 10, 2023 17:13
if tf state has no kubeconfig kubeconfig_attributes then reload kubeconfig
Copy link
Collaborator

@scraly scraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add output in examples for kubeconfig_attributes.host attribute for resource and datasource.

And do we really need an array for this?

output "mycluster-host" {
    value = ovh_cloud_project_kube.mycluster.kubeconfig_attributes[0].host
}

output "mycluster-client_certificate" {
    value = ovh_cloud_project_kube.mycluster.kubeconfig_attributes[0].client_certificate
}

Copy link
Collaborator

@scraly scraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :)

@scraly scraly added the 0.28.0 label Feb 13, 2023
@scraly scraly merged commit dd0a7f0 into ovh:master Feb 13, 2023
@SkYNewZ SkYNewZ deleted the fix/325 branch February 13, 2023 14:53
@FlorentPoinsaut
Copy link

Thanks a lot @SkYNewZ!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add host, cluster_ca_certificate and token attributes of cloud_project_kube resource
3 participants