-
Notifications
You must be signed in to change notification settings - Fork 795
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
fix(daemonset): Allow the definition a ServiceAccount in DaemonSets #3441
base: main
Are you sure you want to change the base?
Conversation
Hey @manics could you take a look at this? |
@manics check this again when you can, everything is working as expected |
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.
Other than a couple of queries this looks good to me, thanks for working on this.
@consideRatio would you mind reviewing this? I'm not fully confident about all the changes.
@consideRatio please take a look when you can :) |
@manics can you push this to have the second review and be merged? |
This pull request introduces the option to define a ServiceAccount in DaemonSets used in the prepuller.
Closes #3442