Skip to content

fix: Specify PriorityClass for Node Feature Discovery components #1041

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

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

dlipovetsky
Copy link
Contributor

@dlipovetsky dlipovetsky commented Feb 10, 2025

What problem does this PR solve?:
NFD components were missing a PriorityClass.

Which issue(s) this PR fixes:
Fixes

How Has This Been Tested?:

Special notes for your reviewer:

@dlipovetsky dlipovetsky requested a review from faiq February 10, 2025 23:27
@github-actions github-actions bot added the fix label Feb 10, 2025
@dlipovetsky dlipovetsky requested a review from msdolbey February 10, 2025 23:27
@supershal supershal self-requested a review February 10, 2025 23:35
@github-actions github-actions bot added fix and removed fix labels Feb 10, 2025
@dlipovetsky dlipovetsky merged commit b32e253 into main Feb 11, 2025
26 checks passed
@dlipovetsky dlipovetsky deleted the dlipovetsky/nfd-priorityclass branch February 11, 2025 00:24
@github-actions github-actions bot added fix and removed fix labels Feb 11, 2025
jimmidyson added a commit that referenced this pull request Feb 12, 2025
🤖 I have created a release *beep* *boop*
---


## 0.27.0 (2025-02-12)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

## What's Changed
### Exciting New Features 🎉
* feat: Update COSI controller Addon by @dlipovetsky in
#1043
* feat: Build with Go 1.24.0 by @jimmidyson in
#1047
### Fixes 🔧
* fix: Specify PriorityClass for Node Feature Discovery components by
@dlipovetsky in
#1041
* fix: Correctly configure non-mirror registry certificates by
@jimmidyson in
#1039
* fix: Configure priorityClassName for Cilium Hubble by @dlipovetsky in
#1045
* fix: don't generate empty _default containerd mirror file by @dkoshkin
in
#1042
* fix: set priority class name for metallb by @supershal in
#1046
* fix: Correctly configure dynamic credential provider by @jimmidyson in
#1040


**Full Changelog**:
v0.26.0...v0.27.0

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants