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

feat(helm): add support for priorityClassName #1008

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

iarlyy
Copy link
Contributor

@iarlyy iarlyy commented Nov 14, 2024

Description

Add support for configuring priorityClassName for the agent and operator pods.

Related Issue

If this pull request is related to any issue, please mention it here. Additionally, make sure that the issue is assigned to you before submitting this pull request.

Checklist

  • I have read the contributing documentation.
  • I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
  • I have correctly attributed the author(s) of the code.
  • I have tested the changes locally.
  • I have followed the project's style guidelines.
  • I have updated the documentation, if necessary.
  • I have added tests, if applicable.

Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.

@iarlyy iarlyy requested a review from a team as a code owner November 14, 2024 14:35
@iarlyy
Copy link
Contributor Author

iarlyy commented Nov 14, 2024

@microsoft-github-policy-service agree

@timraymond
Copy link
Member

I think this is probably a good addition given Retina's use cases. @anubhabMajumdar -- should we be making these changes in the legacy charts or the hubble ones?

@nddq
Copy link
Contributor

nddq commented Nov 20, 2024

@timraymond i don't see why we shouldn't make this change in both legacy and hubble

@iarlyy iarlyy force-pushed the add-priorityclassname branch 2 times, most recently from e7ea974 to d48ba10 Compare November 20, 2024 17:32
@rbtr rbtr added type/enhancement New feature or request scope/S Change is Small labels Nov 20, 2024
Copy link
Contributor

@nddq nddq left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@nddq nddq added this pull request to the merge queue Nov 20, 2024
Merged via the queue into microsoft:main with commit 2ce1fdc Nov 20, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/S Change is Small type/enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants