Skip to content

feat - Added K8s warnings capture for select operations. Resolves #647 #2319

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chriswhite199
Copy link

Explanation

Allows capture of K8s API server warnings for create / apply / update / patch and delete operations, which can then be used in expectations.

This is useful feature for testing Admission controllers that return warnings when resources are amended

Related issue

#647

Proposed Changes

  • Add a new warnings list for Cluster and update the k8s client to add a WarningHandler
  • Update create / apply / update / patch and delete operations to reset this warnings list and inject the list into the bindings as a new variable

Checklist

  • I have read the contributing guidelines.
  • I have read the PR documentation guide and followed the process including adding proof manifests to this PR.
  • This is a new feature and I have added e2e tests that prove my implementation is effective.

@chriswhite199
Copy link
Author

e2e Tests are failing due to docker ToS change and rate limiting on image downloads:

Failed to pull image "nginx": failed to pull and unpack image "docker.io/library/nginx:latest": failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/nginx/manifests/sha256:124b44bfc9ccd1f3cedf4b592d4d1e8bddb78b51ec2ed5056c52d3692baebc19: 429 Too Many Requests - Server message: toomanyrequests: You have reached your unauthenticated pull rate limit. https://www.docker.com/increase-rate-limit

@chriswhite199 chriswhite199 changed the title Added K8s warnings capture for select operations. Resolves #647 feat - Added K8s warnings capture for select operations. Resolves #647 Mar 26, 2025
Signed-off-by: Chris White <chriswhite199@gmail.com>
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.

1 participant