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

fix: Remove duplicated AuthorizationPolicy #75

Merged
merged 1 commit into from
Sep 12, 2024
Merged

fix: Remove duplicated AuthorizationPolicy #75

merged 1 commit into from
Sep 12, 2024

Conversation

ciiiii
Copy link
Member

@ciiiii ciiiii commented Aug 22, 2024 •
edited
Loading

@ciiiii ciiiii requested a review from maxsxu August 22, 2024 13:20
@ciiiii ciiiii requested a review from a team as a code owner August 22, 2024 13:20
@ciiiii ciiiii changed the title Remove duplicated AuthorizationPolicy fix: Remove duplicated AuthorizationPolicy Aug 22, 2024
@ignacioli
Copy link
Contributor

@ciiiii What's the next action after the PR is merged, when will this fix be released to argoCD and infra-app repo?

@ciiiii
Copy link
Member Author

ciiiii commented Sep 12, 2024

infra-app already includes the fix, this for legacy envs

@ciiiii ciiiii merged commit 587c786 into master Sep 12, 2024
4 of 5 checks passed
@ciiiii ciiiii deleted the yscai/remove-duplicated branch September 12, 2024 22:40
@github-actions github-actions bot mentioned this pull request Sep 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ignacioli ignacioli ignacioli approved these changes

@maxsxu maxsxu maxsxu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ciiiii @ignacioli @maxsxu