Skip to content

init backward comptablity for policies v1 #1348

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

Merged
merged 2 commits into from
Aug 31, 2022

Conversation

waleedhammam
Copy link
Contributor

Flow

1- Check for v2beta policies
2- If errors check for v1 policies too
3- return v2beta and/or v1 policies
4- If no policies don't return error just nil as it doesn't make sense to show an error in case of no policies

@waleedhammam waleedhammam added the exclude from release notes Use this label to exclude a PR from the release notes label Aug 23, 2022
@waleedhammam waleedhammam force-pushed the 1283-policies-backward-comptablity branch 2 times, most recently from acfd0c9 to f67475f Compare August 30, 2022 12:34
@waleedhammam waleedhammam marked this pull request as ready for review August 30, 2022 12:38
@waleedhammam waleedhammam force-pushed the 1283-policies-backward-comptablity branch from f67475f to c9298f5 Compare August 30, 2022 12:40
@waleedhammam waleedhammam force-pushed the 1283-policies-backward-comptablity branch 5 times, most recently from 7f5d9e5 to b2876c0 Compare August 31, 2022 12:12
@waleedhammam waleedhammam force-pushed the 1283-policies-backward-comptablity branch from b2876c0 to 8ad590d Compare August 31, 2022 12:13
@waleedhammam waleedhammam merged commit b7d3efe into main Aug 31, 2022
@waleedhammam waleedhammam deleted the 1283-policies-backward-comptablity branch August 31, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/policy exclude from release notes Use this label to exclude a PR from the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants