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

Admission controller example fixes #950

Merged
merged 3 commits into from
Jul 8, 2022

Conversation

Alibirb
Copy link
Contributor

@Alibirb Alibirb commented Jul 6, 2022

Motivation

Fixes #943 and #944

Solution

Added commented-out code to certificate setup script for adding DNS name to certificate.
Added check around adding /metadata/labels to only add the field if it doesn't already exist.

…on script showing necessary steps when running behind a service.

Fixes kube-rs#944

Signed-off-by: Alicrow <writerofalicrow@gmail.com>
…s field if it doesn't already exist.

Fixes kube-rs#943

Signed-off-by: Alicrow <writerofalicrow@gmail.com>
Copy link
Member

@clux clux left a comment

Choose a reason for hiding this comment

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

This looks great to me, thanks a lot!

@codecov-commenter
Copy link

codecov-commenter commented Jul 7, 2022

Codecov Report

Merging #950 (31aad12) into master (55bea0b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #950   +/-   ##
=======================================
  Coverage   72.43%   72.43%           
=======================================
  Files          64       64           
  Lines        4412     4412           
=======================================
  Hits         3196     3196           
  Misses       1216     1216           

Signed-off-by: Alicrow <writerofalicrow@gmail.com>
@clux clux added the changelog-fix changelog fix category for prs label Jul 8, 2022
@clux clux added this to the 0.74.0 milestone Jul 8, 2022
@clux clux merged commit 04c3578 into kube-rs:master Jul 8, 2022
@Alibirb Alibirb deleted the admission_controller_example_fixes branch July 8, 2022 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-fix changelog fix category for prs
Projects
None yet
3 participants