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

ci: bump k8s matrix #3267

Merged
merged 4 commits into from Feb 15, 2024
Merged

Conversation

sozercan
Copy link
Member

What this PR does / why we need it:

drops eol 1.25, adds 1.29

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #

Special notes for your reviewer:

Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
@sozercan sozercan requested a review from a team as a code owner February 14, 2024 00:41
@codecov-commenter
Copy link

codecov-commenter commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9ecdd9e) 54.46% compared to head (6543b3a) 54.50%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3267      +/-   ##
==========================================
+ Coverage   54.46%   54.50%   +0.04%     
==========================================
  Files         134      134              
  Lines       12318    12318              
==========================================
+ Hits         6709     6714       +5     
+ Misses       5113     5109       -4     
+ Partials      496      495       -1     
Flag Coverage Δ
unittests 54.50% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@maxsmythe maxsmythe left a comment

Choose a reason for hiding this comment

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

LGTM, assuming flaky test passes?

@maxsmythe
Copy link
Contributor

Looks like the breakage may not be a flake:

# The Service "mutate-svc" is invalid: spec.externalTrafficPolicy: Invalid value: "Cluster": may only be set for externally-accessible services

Copy link
Contributor

@maxsmythe maxsmythe left a comment

Choose a reason for hiding this comment

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

un-LGTMing until we know what's needed to fix the test

@sozercan
Copy link
Member Author

yea this needs further investigation

@sozercan sozercan self-assigned this Feb 14, 2024
@sozercan sozercan marked this pull request as draft February 14, 2024 21:20
@sozercan sozercan force-pushed the bump-k8s-matrix-129 branch 2 times, most recently from d9137d7 to 3b1cc4a Compare February 14, 2024 21:35
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
sozercan and others added 2 commits February 14, 2024 13:39
Signed-off-by: Sertac Ozercan <sozercan@gmail.com>
@sozercan sozercan marked this pull request as ready for review February 14, 2024 22:53
@sozercan
Copy link
Member Author

@maxsmythe updated. there was a change in k8s 1.29 to require certain types for externalIPs kubernetes/kubernetes#119150

@maxsmythe
Copy link
Contributor

Good find! Kinda weird that not defining that results in a stack trace.

Copy link
Contributor

@maxsmythe maxsmythe left a comment

Choose a reason for hiding this comment

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

LGTM

@sozercan sozercan merged commit b758571 into open-policy-agent:master Feb 15, 2024
16 checks passed
@sozercan sozercan deleted the bump-k8s-matrix-129 branch February 15, 2024 05:10
leewoobin789 pushed a commit to softlee-io/gatekeeper that referenced this pull request Apr 1, 2024
Signed-off-by: Sertac Ozercan <sozercan@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.

None yet

3 participants