-
Notifications
You must be signed in to change notification settings - Fork 36
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
[terraformer-v2] Add Pod e2e test #51
Merged
timebertt
merged 2 commits into
gardener:terraformer-v2
from
timebertt:enh/pod-e2e-test
Nov 20, 2020
Merged
[terraformer-v2] Add Pod e2e test #51
timebertt
merged 2 commits into
gardener:terraformer-v2
from
timebertt:enh/pod-e2e-test
Nov 20, 2020
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gardener-robot-ci-1
added
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
gardener-robot
added
area/quality
Output qualification (tests, checks, scans, automation in general, etc.) related
kind/test
Test
needs/review
Needs review
labels
Nov 12, 2020
gardener-robot-ci-1
added
needs/ok-to-test
Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
and removed
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
labels
Nov 12, 2020
gardener-robot
added
size/xl
Size of pull request is huge (see gardener-robot robot/bots/size.py)
needs/second-opinion
Needs second review by someone else
labels
Nov 12, 2020
timebertt
force-pushed
the
enh/pod-e2e-test
branch
from
November 12, 2020 15:13
7d1c77c
to
699f766
Compare
gardener-robot-ci-3
added
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
/ready |
gardener-robot-ci-2
removed
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
timebertt
force-pushed
the
enh/pod-e2e-test
branch
from
November 12, 2020 15:46
699f766
to
c1814eb
Compare
gardener-robot-ci-3
added
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
gardener-robot-ci-1
removed
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
/test-single |
Testrun: e2e-6g7kp +--------------+--------------+--------+----------+ | NAME | STEP | PHASE | DURATION | +--------------+--------------+--------+----------+ | pod-e2e-test | pod-e2e-test | Failed | 1m55s | +--------------+--------------+--------+----------+ |
schrodit
suggested changes
Nov 12, 2020
gardener-robot
added
needs/changes
Needs (more) changes
and removed
needs/review
Needs review
labels
Nov 12, 2020
timebertt
force-pushed
the
enh/pod-e2e-test
branch
from
November 12, 2020 22:05
c1814eb
to
d1977a1
Compare
gardener-robot-ci-1
added
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
gardener-robot
added
needs/review
Needs review
and removed
needs/review
Needs review
labels
Nov 12, 2020
gardener-robot-ci-2
removed
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 12, 2020
timebertt
force-pushed
the
enh/pod-e2e-test
branch
from
November 12, 2020 22:08
d1977a1
to
47c7172
Compare
Testrun: e2e-rjzfg +--------------+--------------+--------+----------+ | NAME | STEP | PHASE | DURATION | +--------------+--------------+--------+----------+ | pod-e2e-test | pod-e2e-test | Failed | 21m44s | +--------------+--------------+--------+----------+ |
/test |
Testrun: e2e-xqsbk +--------------+--------------+-----------+----------+ | NAME | STEP | PHASE | DURATION | +--------------+--------------+-----------+----------+ | pod-e2e-test | pod-e2e-test | Succeeded | 2m58s | +--------------+--------------+-----------+----------+ |
Thank you very much, @schrodit! 🎉 |
schrodit
reviewed
Nov 19, 2020
timebertt
force-pushed
the
enh/pod-e2e-test
branch
from
November 19, 2020 15:36
72244cd
to
f85186d
Compare
gardener-robot
added
the
needs/second-opinion
Needs second review by someone else
label
Nov 19, 2020
gardener-robot-ci-1
added
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 19, 2020
gardener-robot-ci-3
removed
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 19, 2020
/test |
Testrun: e2e-xxv6r +--------------+--------------+-----------+----------+ | NAME | STEP | PHASE | DURATION | +--------------+--------------+-----------+----------+ | pod-e2e-test | pod-e2e-test | Succeeded | 2m29s | +--------------+--------------+-----------+----------+ |
rfranzke
approved these changes
Nov 19, 2020
dguendisch
requested changes
Nov 19, 2020
timebertt
force-pushed
the
enh/pod-e2e-test
branch
from
November 19, 2020 17:50
f85186d
to
06ebb7a
Compare
gardener-robot-ci-2
added
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 19, 2020
gardener-robot-ci-1
removed
the
reviewed/ok-to-test
Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
label
Nov 19, 2020
/test |
Testrun: e2e-ftrbd +--------------+--------------+-----------+----------+ | NAME | STEP | PHASE | DURATION | +--------------+--------------+-----------+----------+ | pod-e2e-test | pod-e2e-test | Succeeded | 3m48s | +--------------+--------------+-----------+----------+ |
dguendisch
approved these changes
Nov 19, 2020
schrodit
approved these changes
Nov 19, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/quality
Output qualification (tests, checks, scans, automation in general, etc.) related
kind/test
Test
needs/changes
Needs (more) changes
needs/ok-to-test
Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
needs/second-opinion
Needs second review by someone else
size/xl
Size of pull request is huge (see gardener-robot robot/bots/size.py)
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
/area quality
/kind test
What this PR does / why we need it:
This PR adds an e2e test, that is executed by the test-machinery. It functions similarly to some infrastructure integration tests in our provider extensions.
It uses an existing cluster (pointed to by the
KUBECONFIG
env var) and deploys a terraformerapply
Pod, that will create some lightweight resource (ec2 keypair) on AWS.The test validates, that the resource was created on AWS using the AWS go-sdk and that the state ConfigMap has been updated accordingly.
After that, the test deploys a terraformer
destroy
Pod and validates again the changes on AWS and the state ConfigMap.Which issue(s) this PR fixes:
Fixes #10
Special notes for your reviewer:
/invite @gardener/test-infra-maintainers
Release note: