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

Add support for envFrom on container definitions #82

Merged
merged 6 commits into from
Oct 17, 2018

Conversation

pdecat
Copy link
Contributor

@pdecat pdecat commented Oct 26, 2017

Relates to #78.

  • Rebase on master
  • Update schema
  • Update documentation
  • Implement mapping
  • Add acceptance test cases

@pdecat
Copy link
Contributor Author

pdecat commented Oct 26, 2017

Test results

# make testacc TEST=./kubernetes/ TESTARGS='-run=TestAccKubernetesPod_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./kubernetes/ -v -run=TestAccKubernetesPod_ -timeout 120m
=== RUN   TestAccKubernetesPod_basic
--- PASS: TestAccKubernetesPod_basic (9.43s)
=== RUN   TestAccKubernetesPod_updateForceNew
--- PASS: TestAccKubernetesPod_updateForceNew (100.17s)
=== RUN   TestAccKubernetesPod_importBasic
--- PASS: TestAccKubernetesPod_importBasic (5.06s)
=== RUN   TestAccKubernetesPod_with_pod_security_context
--- PASS: TestAccKubernetesPod_with_pod_security_context (3.06s)
=== RUN   TestAccKubernetesPod_with_container_liveness_probe_using_exec
--- PASS: TestAccKubernetesPod_with_container_liveness_probe_using_exec (41.88s)
=== RUN   TestAccKubernetesPod_with_container_liveness_probe_using_http_get
--- PASS: TestAccKubernetesPod_with_container_liveness_probe_using_http_get (9.86s)
=== RUN   TestAccKubernetesPod_with_container_liveness_probe_using_tcp
--- PASS: TestAccKubernetesPod_with_container_liveness_probe_using_tcp (10.64s)
=== RUN   TestAccKubernetesPod_with_container_lifecycle
--- PASS: TestAccKubernetesPod_with_container_lifecycle (6.67s)
=== RUN   TestAccKubernetesPod_with_container_security_context
--- PASS: TestAccKubernetesPod_with_container_security_context (6.66s)
=== RUN   TestAccKubernetesPod_with_volume_mount
--- PASS: TestAccKubernetesPod_with_volume_mount (17.11s)
=== RUN   TestAccKubernetesPod_with_cfg_map_volume_mount
--- PASS: TestAccKubernetesPod_with_cfg_map_volume_mount (9.07s)
=== RUN   TestAccKubernetesPod_with_resource_requirements
--- PASS: TestAccKubernetesPod_with_resource_requirements (6.64s)
=== RUN   TestAccKubernetesPod_with_empty_dir_volume
--- PASS: TestAccKubernetesPod_with_empty_dir_volume (9.06s)
=== RUN   TestAccKubernetesPod_with_secret_vol_items
--- PASS: TestAccKubernetesPod_with_secret_vol_items (17.08s)
=== RUN   TestAccKubernetesPod_gke_with_nodeSelector
--- SKIP: TestAccKubernetesPod_gke_with_nodeSelector (0.01s)
        provider_test.go:206: The Kubernetes endpoint must come from GKE for this test to run - skipping
PASS
ok      github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 252.543s

@pdecat pdecat changed the title [WIP] Add support for envFrom on container definitions Add support for envFrom on container definitions Oct 26, 2017
@ayansen
Copy link

ayansen commented Dec 19, 2017

is there a timeline as to when this pr is going to get merged ?

@rawkode
Copy link

rawkode commented May 14, 2018

Is there anything blocking this from being merged?

@seivan
Copy link

seivan commented May 17, 2018

Does this mean I can define ConfigMap from Terraform output values (e.g creating Route53 subdomains and RDS resources) while still defining the rest of the infrastructure in regular YAML manifest files?
I don't want to use the K8 provider fully, but just for defining configmaps

@nblakely731
Copy link

Any timeline around when this will be merged? Looks like everything is passing unit tests.

@pdecat pdecat force-pushed the f-container-env_from branch from c2311e3 to 3b2900c Compare October 5, 2018 09:16
@ghost ghost added the size/L label Oct 5, 2018
@pdecat
Copy link
Contributor Author

pdecat commented Oct 5, 2018

Acceptance tests results

New run after rebase with local minikube 0.29:

make testacc TEST=./kubernetes/ TESTARGS='-run=TestAccKubernetesPod_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./kubernetes/ -v -run=TestAccKubernetesPod_ -timeout 120m
=== RUN   TestAccKubernetesPod_basic
--- PASS: TestAccKubernetesPod_basic (108.69s)
=== RUN   TestAccKubernetesPod_initContainer_updateForcesNew
--- PASS: TestAccKubernetesPod_initContainer_updateForcesNew (186.64s)
=== RUN   TestAccKubernetesPod_updateArgsForceNew
--- PASS: TestAccKubernetesPod_updateArgsForceNew (100.54s)
=== RUN   TestAccKubernetesPod_updateEnvForceNew
--- PASS: TestAccKubernetesPod_updateEnvForceNew (67.37s)
=== RUN   TestAccKubernetesPod_importBasic
--- PASS: TestAccKubernetesPod_importBasic (68.72s)
=== RUN   TestAccKubernetesPod_with_pod_security_context
--- PASS: TestAccKubernetesPod_with_pod_security_context (5.05s)
=== RUN   TestAccKubernetesPod_with_container_liveness_probe_using_exec
--- PASS: TestAccKubernetesPod_with_container_liveness_probe_using_exec (51.89s)
=== RUN   TestAccKubernetesPod_with_container_liveness_probe_using_http_get
--- PASS: TestAccKubernetesPod_with_container_liveness_probe_using_http_get (9.87s)
=== RUN   TestAccKubernetesPod_with_container_liveness_probe_using_tcp
--- PASS: TestAccKubernetesPod_with_container_liveness_probe_using_tcp (18.67s)
=== RUN   TestAccKubernetesPod_with_container_lifecycle
--- PASS: TestAccKubernetesPod_with_container_lifecycle (18.68s)
=== RUN   TestAccKubernetesPod_with_container_security_context
--- PASS: TestAccKubernetesPod_with_container_security_context (18.69s)
=== RUN   TestAccKubernetesPod_with_volume_mount
--- PASS: TestAccKubernetesPod_with_volume_mount (17.14s)
=== RUN   TestAccKubernetesPod_with_cfg_map_volume_mount
--- PASS: TestAccKubernetesPod_with_cfg_map_volume_mount (5.14s)
=== RUN   TestAccKubernetesPod_with_resource_requirements
--- PASS: TestAccKubernetesPod_with_resource_requirements (6.67s)
=== RUN   TestAccKubernetesPod_with_empty_dir_volume
--- PASS: TestAccKubernetesPod_with_empty_dir_volume (18.70s)
=== RUN   TestAccKubernetesPod_with_secret_vol_items
--- PASS: TestAccKubernetesPod_with_secret_vol_items (6.73s)
=== RUN   TestAccKubernetesPod_gke_with_nodeSelector
--- SKIP: TestAccKubernetesPod_gke_with_nodeSelector (0.01s)
        provider_test.go:215: The Kubernetes endpoint must come from GKE for this test to run - skipping
PASS
ok      github.com/terraform-providers/terraform-provider-kubernetes/kubernetes 709.256s

@pdecat
Copy link
Contributor Author

pdecat commented Oct 5, 2018

Hi @alexsomesan, would you mind having a look at this one?

@pdecat pdecat force-pushed the f-container-env_from branch from 3b2900c to db080b2 Compare October 16, 2018 06:11
Copy link
Member

@alexsomesan alexsomesan left a comment

Choose a reason for hiding this comment

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

Looks good.

@alexsomesan alexsomesan merged commit 284b00c into hashicorp:master Oct 17, 2018
@pdecat pdecat deleted the f-container-env_from branch October 17, 2018 19:30
@ghost ghost locked and limited conversation to collaborators Apr 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants