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

Fix mock-ip-count test #131

Merged
merged 3 commits into from
Aug 19, 2021
Merged

Fix mock-ip-count test #131

merged 3 commits into from
Aug 19, 2021

Conversation

brycahta
Copy link
Contributor

@brycahta brycahta commented Aug 19, 2021

Issue #, if available: N/A

Description of changes:

Testing:

  • make helm-mock-ip-count-test
======================================================================================================
🥑 Starting AEMM mock-ip-count test
======================================================================================================
executing mock-ip-count test with mockIPCount=1
Release "kind-ct-aemm" does not exist. Installing it now.
NAME: kind-ct-aemm
LAST DEPLOYED: Thu Aug 19 11:43:56 2021
NAMESPACE: default
STATUS: deployed
REVISION: 1
NOTES:
kind-ct-aemm has been installed 

Some useful commands:
kubectl get pods --namespace default
kubectl port-forward service/amazon-ec2-metadata-mock 1338
pod/test-pod created
pod/test-pod-404 created
test-pod-404 still provisioning with status: Pending . status check 1/30, sleeping for 2 seconds
✅ Verified test-pod-404 is created and running
copying 404_response.golden to 404-pod
Pods are still Running. test-pod status: Running    test-pod-404 status: Running
status check 1/30, sleeping for 2 seconds
Pods are still Running. test-pod status: Succeeded    test-pod-404 status: Running
status check 2/30, sleeping for 2 seconds
✅ Verified both pods are no longer Running
✅ Verified test-pod Succeeded w/mockIPCount=1
✅ Verified test-pod-404 Succeeded w/mockIPCount=1
pod "test-pod" deleted
pod "test-pod-404" deleted
executing mock-ip-count test with mockIPCount=2
Release "kind-ct-aemm" has been upgraded. Happy Helming!
NAME: kind-ct-aemm
LAST DEPLOYED: Thu Aug 19 11:44:10 2021
NAMESPACE: default
STATUS: deployed
REVISION: 2
NOTES:
kind-ct-aemm has been updated.

Some useful commands:
kubectl get pods --namespace default
kubectl port-forward service/amazon-ec2-metadata-mock 1338
pod/test-pod created
pod/test-pod-404 created
test-pod-404 still provisioning with status: Pending . status check 1/30, sleeping for 2 seconds
✅ Verified test-pod-404 is created and running
copying 404_response.golden to 404-pod
Pods are still Running. test-pod status: Running    test-pod-404 status: Running
status check 1/30, sleeping for 2 seconds
Pods are still Running. test-pod status: Succeeded    test-pod-404 status: Running
status check 2/30, sleeping for 2 seconds
✅ Verified both pods are no longer Running
✅ Verified test-pod Succeeded w/mockIPCount=2
✅ Verified test-pod-404 Failed w/mockIPCount=2
pod "test-pod" deleted
pod "test-pod-404" deleted

🥑 Cleaning up resources...
-> Deleting kind cluster kind-ct...
-> Deleting tmp dir '/Users/crtbry/workplace/go/src/aemm_brycahta_fork_ws/amazon-ec2-metadata-mock/build/tmp-kind-ct'

✅✅ All tests passed and cleaned up ✅✅

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@brycahta brycahta requested a review from bwagner5 August 19, 2021 16:47
@brycahta brycahta changed the title update image for mock-ip-count test pods Fix mock-ip-count test Aug 19, 2021
Copy link
Contributor

@bwagner5 bwagner5 left a comment

Choose a reason for hiding this comment

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

amazing LGTM

@bwagner5 bwagner5 merged commit 6ba0db5 into aws:main Aug 19, 2021
@brycahta brycahta deleted the mock-ip-count-sux branch August 19, 2021 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants