Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

fix: update EncryptionConfiguration #2856

Merged
merged 1 commit into from
Mar 9, 2020

Conversation

ritazh
Copy link
Member

@ritazh ritazh commented Mar 6, 2020

Reason for Change:

Update EncryptionConfig resources to EncryptionConfiguration using the stable API server encryption configuration as EncryptionConfig is deprecated.
Ref: https://kubernetes.io/docs/tasks/administer-cluster/kms-provider/#encrypting-your-data-with-the-kms-provider

Issue Fixed:

fixes: #2777

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Mar 6, 2020

Codecov Report

Merging #2856 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2856      +/-   ##
==========================================
+ Coverage   72.47%   72.47%   +<.01%     
==========================================
  Files         140      140              
  Lines       25589    25612      +23     
==========================================
+ Hits        18545    18562      +17     
- Misses       5976     5981       +5     
- Partials     1068     1069       +1

Copy link
Member

@mboersma mboersma left a comment

Choose a reason for hiding this comment

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

/lgtm

@mboersma
Copy link
Member

mboersma commented Mar 9, 2020

See also #2727.

@mboersma mboersma merged commit b467e69 into Azure:master Mar 9, 2020
@ritazh
Copy link
Member Author

ritazh commented Mar 9, 2020

@mboersma yea I saw! Thanks for updating the api server flag!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to EncryptionConfiguration
2 participants