-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update manager field #160
Update manager field #160
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request contains a valid label.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request contains a valid label.
Codecov Report
@@ Coverage Diff @@
## main #160 +/- ##
=======================================
Coverage ? 63.22%
=======================================
Files ? 41
Lines ? 3073
Branches ? 0
=======================================
Hits ? 1943
Misses ? 1014
Partials ? 116
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request contains a valid label.
0f65096
to
60175b7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request contains a valid label.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request contains a valid label.
What does this PR do?
Configure a controller manager name,
eds-controller
for the manager managedField.Motivation
Previously, the
ExtendedDaemonSet
resource created by the ExtendedDaemonset controller had the managed field,Manager: manager
, which was ambiguous and confusing.This change customizes the Manager name to a more meaningful name,
eds-controller
.Additional Notes
This change also updates the
Manager
field in the ExtendedReplicaSet toeds-controller
.Describe your test plan
ExtendedDaemonSet
manifest fileMetadata.Managed Fields.Manager
iseds-controller