Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Remove unused GET in CreateOrUpdate #411

Merged
merged 1 commit into from
Oct 4, 2019

Conversation

kkmsft
Copy link
Contributor

@kkmsft kkmsft commented Oct 4, 2019

Reason for Change:

Issue Fixed:

Notes for Reviewers:

Copy link
Member

@aramase aramase left a comment

Choose a reason for hiding this comment

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

lgtm

@kkmsft kkmsft merged commit 53db68c into Azure:master Oct 4, 2019
@kkmsft kkmsft deleted the remove_result_vmss branch October 4, 2019 00:32
@kkmsft
Copy link
Contributor Author

kkmsft commented Oct 4, 2019

[Fail] Kubernetes cluster using aad-pod-identity [It] should pass the identity format validation with gatekeeper constraint
/Users/kkmsft/projects/aad-pod-identity/go/src/github.com/Azure/aad-pod-identity/test/e2e/aadpodidentity_test.go:1452

Ran 17 of 19 Specs in 4005.475 seconds
FAIL! -- 16 Passed | 1 Failed | 0 Pending | 2 Skipped
--- FAIL: TestAADPodIdentity (4005.48s)
FAIL
FAIL github.com/Azure/aad-pod-identity/test/e2e 4005.514s

• [SLOW TEST:164.023 seconds]
Kubernetes cluster using aad-pod-identity
/Users/kkmsft/projects/aad-pod-identity/go/src/github.com/Azure/aad-pod-identity/test/e2e/aadpodidentity_test.go:111
should pass the identity format validation with gatekeeper constraint
/Users/kkmsft/projects/aad-pod-identity/go/src/github.com/Azure/aad-pod-identity/test/e2e/aadpodidentity_test.go:923

Tearing down the test suite environment...
$ kubectl delete -f template/_output/default-deployment.yaml --now --ignore-not-found
$ kubectl delete -f template/busyboxds.yaml
$ kubectl delete deploy --all
$ kubectl get nodes -ojson

Ran 1 of 19 Specs in 296.889 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 18 Skipped
PASS

Ginkgo ran 1 suite in 5m15.726807729s
Test Suite Passed

statbit pushed a commit to adobe-platform/aad-pod-identity that referenced this pull request Sep 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants