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

Support enableServiceLinks in PodSpec #693

Closed
JPMoresmau opened this issue Nov 15, 2019 · 6 comments
Closed

Support enableServiceLinks in PodSpec #693

JPMoresmau opened this issue Nov 15, 2019 · 6 comments
Labels
acknowledged Issue has undergone initial review and is in our work queue. enhancement theme/coverage issues that involve improving API coverage for the provider

Comments

@JPMoresmau
Copy link

Terraform Version

Terraform v0.12.7

  • provider.kubernetes v1.10.0

Affected Resource(s)

  • kubernetes_deployment
  • kubernetes_pod

Expected Behavior

Have a way to use enable_service_links in a deployment/pod spec

Actual Behavior

An argument named "enable_service_links" is not expected here.

References

kubernetes/kubernetes#68754

Since environment variables can wreak havoc with some programs (most notably Apache Java apps), turning them off was requested.
Note that I can see enableServiceLinks in https://github.com/terraform-providers/terraform-provider-kubernetes/blob/master/vendor/k8s.io/api/core/v1/types_swagger_doc_generated.go

@abrzozowski-ITM
Copy link

StatefulSet is also affected.

@MathyV
Copy link
Contributor

MathyV commented Jul 17, 2020

@aareet any chance of getting that PR reviewed?

@dak1n1 dak1n1 added theme/coverage issues that involve improving API coverage for the provider acknowledged Issue has undergone initial review and is in our work queue. labels Jul 20, 2020
@shd4
Copy link

shd4 commented Aug 19, 2020

Any updates on this? enableServiceLinks not being disabled produces a lot of pain...

@aareet
Copy link
Contributor

aareet commented Aug 19, 2020

The PR is waiting on an acceptance test being added.

@dak1n1
Copy link
Contributor

dak1n1 commented Nov 12, 2020

It looks like we implemented this a while back but didn't update this issue. The enable_service_links option is present in the PodSpec now: https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/pod#enable_service_links

@dak1n1 dak1n1 closed this as completed Nov 12, 2020
@ghost
Copy link

ghost commented Dec 13, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!

@ghost ghost locked as resolved and limited conversation to collaborators Dec 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
acknowledged Issue has undergone initial review and is in our work queue. enhancement theme/coverage issues that involve improving API coverage for the provider
Projects
None yet
Development

No branches or pull requests

6 participants