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

change event recorder cluster namespace to kube-system #871

Merged
merged 1 commit into from Apr 8, 2022

Conversation

clux
Copy link
Member

@clux clux commented Apr 8, 2022

because publishing to "default" is a fallback thing.
fixes #870

because publishing to "default" is a fallback thing.
fixes #870

Signed-off-by: clux <sszynrae@gmail.com>
@clux clux added the changelog-change changelog change category for prs label Apr 8, 2022
@clux clux added this to the 0.71.0 milestone Apr 8, 2022
@nightkr
Copy link
Member

nightkr commented Apr 8, 2022

Might want to add a highlight for this, since it might require RBAC changes.

@codecov-commenter
Copy link

Codecov Report

Merging #871 (a9315c9) into master (7b2d145) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #871      +/-   ##
==========================================
+ Coverage   70.46%   70.48%   +0.02%     
==========================================
  Files          61       61              
  Lines        4249     4249              
==========================================
+ Hits         2994     2995       +1     
+ Misses       1255     1254       -1     
Impacted Files Coverage Δ
kube-runtime/src/events.rs 94.44% <100.00%> (ø)
kube-runtime/src/wait.rs 70.00% <0.00%> (+2.00%) ⬆️

@clux
Copy link
Member Author

clux commented Apr 8, 2022

Agreed, will add a highlight in the next release!

@clux clux merged commit 05b8bf6 into master Apr 8, 2022
@clux clux deleted the update-default-ns-events branch April 8, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-change changelog change category for prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Publishing events for cluster scoped resources broken for Kube <1.23
3 participants