Skip to content

fix: Configure priorityClassName for Cilium Hubble #1045

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

What problem does this PR solve?:
Assigns the system-cluster-critical PriorityClassName to Hubble. This is the same class name assigned to Cilium itself.

Which issue(s) this PR fixes:
Fixes https://jira.nutanix.com/browse/NCN-105637

How Has This Been Tested?:

Special notes for your reviewer:

@github-actions github-actions bot added the fix label Feb 11, 2025
@dlipovetsky dlipovetsky force-pushed the dlipovetsky/cilium-hubble-priorityclassname branch from 2a1abc6 to 19af8c3 Compare February 11, 2025 21:11
@jimmidyson jimmidyson enabled auto-merge (squash) February 11, 2025 21:49
@jimmidyson jimmidyson merged commit 506323a into main Feb 11, 2025
21 checks passed
@jimmidyson jimmidyson deleted the dlipovetsky/cilium-hubble-priorityclassname branch February 11, 2025 22:49
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.

3 participants