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.

Sign up for GitHub

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

Jump to bottom

Bump github.com/aws/aws-sdk-go-v2/service/ec2 to v1.13.0 #17113

Merged
merged 1 commit into from Aug 16, 2021
Merged

Bump github.com/aws/aws-sdk-go-v2/service/ec2 to v1.13.0 #17113

merged 1 commit into from Aug 16, 2021

Conversation

ungureanuvladvictor
Copy link
Member

Signed-off-by: Vlad Ungureanu vladu@palantir.com

Bump github.com/aws/aws-sdk-go-v2/service/ec2 to v1.13.0

@ungureanuvladvictor ungureanuvladvictor added area/operator Impacts the cilium-operator component area/eni Impacts ENI based IPAM. integration/cloud Related to integration with cloud environments such as AKS, EKS, GKE, etc. labels Aug 9, 2021
@ungureanuvladvictor ungureanuvladvictor requested a review from a team as a code owner August 9, 2021 22:41
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Aug 9, 2021
@ungureanuvladvictor ungureanuvladvictor added the release-note/misc This PR makes changes that have no direct user impact. label Aug 9, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 9, 2021
@ungureanuvladvictor
Copy link
Member Author

ungureanuvladvictor commented Aug 9, 2021 •
edited by maintainer-s-little-helper bot

test-me-please

Job 'Cilium-PR-K8s-1.19-kernel-5.4' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDatapathConfig AutoDirectNodeRoutes Check connectivity with sockops and direct routing

Failure Output

FAIL: Error creating resource /home/jenkins/workspace/Cilium-PR-K8s-1.19-kernel-5.4/src/github.com/cilium/cilium/test/k8sT/manifests/l3-policy-demo.yaml: Cannot retrieve cilium pod cilium-598fl policy revision: cannot get revision from json output '': could not parse JSON from command "kubectl exec -n kube-system cilium-598fl -- cilium policy get -o json"

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.19-kernel-5.4 so I can create a new GitHub issue to track it.

Job 'Cilium-PR-K8s-1.20-kernel-4.19' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sServicesTest Checks service across nodes Tests NodePort BPF Tests with direct routing With host policy Tests NodePort

Failure Output

FAIL: Request from testclient-w7vnf pod to service tftp://[fd03::c070]:10069/hello failed

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.20-kernel-4.19 so I can create a new GitHub issue to track it.

@ungureanuvladvictor
Copy link
Member Author

test-1.19-5.4

@ungureanuvladvictor
Copy link
Member Author

test-1.20-4.19

@ungureanuvladvictor
Bump github.com/aws/aws-sdk-go-v2/service/ec2 to v1.13.0
b84d60c 
Signed-off-by: Vlad Ungureanu <vladu@palantir.com>
@ungureanuvladvictor
Copy link
Member Author

test-me-please

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 14, 2021
@ti-mo ti-mo merged commit 0a58ea7 into cilium:master Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rolinh rolinh rolinh approved these changes

Assignees
No one assigned
Labels
area/eni Impacts ENI based IPAM. area/operator Impacts the cilium-operator component integration/cloud Related to integration with cloud environments such as AKS, EKS, GKE, etc. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ungureanuvladvictor @rolinh @ti-mo