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: Updated checking psp only for versions below 1.25 #68

Merged
merged 2 commits into from
Feb 23, 2023
Merged

fix: Updated checking psp only for versions below 1.25 #68

merged 2 commits into from
Feb 23, 2023

Conversation

quixoticmonk
Copy link
Contributor

Summary

Resolves: #66

Changes

Updated the preflight check on gathering pod security policies for upgrade versions above 1.25. The updated Policy API has removed this attribute leading to failures on pre-flight checks.

Tested changes against running a pre-flight check and upgrading the cluster to 1.25.

User experience

No changes to user experience. This should continue to behave as expected for cluster upgrades till 1.25 and from 1.25.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@quixoticmonk quixoticmonk requested review from mbeacom and a team February 23, 2023 18:04
@quixoticmonk quixoticmonk marked this pull request as draft February 23, 2023 19:12
@mbeacom mbeacom marked this pull request as ready for review February 23, 2023 21:44
@mbeacom mbeacom merged commit d41305f into aws-samples:main Feb 23, 2023
@quixoticmonk quixoticmonk deleted the fix/uncheck-psp-post1.25 branch March 22, 2023 02:18
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.

Bug: Pre-flight check fails in v0.6.0 for a 1.25 version upgrade
3 participants