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: Enable open telemetry tracing #1189

Merged

Conversation

harshanarayana
Copy link
Contributor

@harshanarayana harshanarayana commented Jul 16, 2023

Reworking the changes from #958 as is got too mess to cleanup and rework the changes easily since a lot of things changed during the framework rebuild.

Thanks and Credits to @Dentrax for the help and contribution in the original PR under #958

Screenshot 2023-07-17 at 12 46 40 AM

@k8s-ci-robot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 16, 2023
@harshanarayana
Copy link
Contributor Author

@a7i @damemi I have ported the changes from #958 into this and addressed most of the comments that were left on that thread in terms of reusing the context and making it simple in terms of invoking the tracers and getting rid of the un-necessary helper functions. Please take a look when time permits.

@harshanarayana
Copy link
Contributor Author

I am still having issue with make gen command like I used to have before.

@damemi
Copy link
Contributor

damemi commented Jul 17, 2023

@harshanarayana thanks for keeping this alive! are you still having trouble with make gen? I ran it on your branch and it seemed to work. if not, please let me know when this is ready for review

@harshanarayana
Copy link
Contributor Author

harshanarayana commented Jul 17, 2023

@damemi I've managed to get make gen working inside a container for now.. I've pushed the changes to this branch already.. Please go ahead with review

@damemi damemi marked this pull request as ready for review July 17, 2023 13:33
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 17, 2023
@k8s-ci-robot k8s-ci-robot requested review from a7i and damemi July 17, 2023 13:33
@harshanarayana
Copy link
Contributor Author

@damemi I am working on getting the tests fixed as we speak.

@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 17, 2023
Copy link
Contributor

@damemi damemi left a comment

Choose a reason for hiding this comment

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

just minor nits at this point but looking good

pkg/descheduler/descheduler.go Outdated Show resolved Hide resolved
pkg/descheduler/descheduler.go Outdated Show resolved Hide resolved
pkg/descheduler/evictions/evictions.go Outdated Show resolved Hide resolved
pkg/descheduler/evictions/evictions.go Outdated Show resolved Hide resolved
pkg/descheduler/evictions/evictions.go Outdated Show resolved Hide resolved
pkg/tracing/tracing.go Outdated Show resolved Hide resolved
pkg/tracing/tracing.go Outdated Show resolved Hide resolved
pkg/tracing/tracing.go Outdated Show resolved Hide resolved
@harshanarayana harshanarayana force-pushed the feature/enable-otel branch 2 times, most recently from 866e00f to 284f8b7 Compare July 17, 2023 15:04
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Jul 17, 2023
Copy link
Contributor

@damemi damemi left a comment

Choose a reason for hiding this comment

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

one nit, otherwise lgtm. Thanks @harshanarayana, this is really cool!
/approve

pkg/tracing/tracing.go Outdated Show resolved Hide resolved
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: damemi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 17, 2023
@damemi damemi changed the title feat: Enable open telemetry feature feat: Enable open telemetry tracing Jul 17, 2023
@harshanarayana
Copy link
Contributor Author

@damemi @a7i PTAL. I have pushed the updated changes based on Mike's suggestion

Copy link
Contributor

@a7i a7i left a comment

Choose a reason for hiding this comment

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

one last nit, and ready to sign LGTM

pkg/tracing/tracing.go Outdated Show resolved Hide resolved
1. Enable OTEL configuration and base framework
2. update generated conversion spec
3. enable docker based conversion and deep copy generate
4. fix broken unit tests
@harshanarayana
Copy link
Contributor Author

@a7i Commit pushed with requested changes. PTAL when you can

@a7i
Copy link
Contributor

a7i commented Jul 18, 2023

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 18, 2023
@k8s-ci-robot k8s-ci-robot merged commit 06dc26a into kubernetes-sigs:master Jul 18, 2023
3 checks passed
szinn referenced this pull request in szinn/k8s-homelab Aug 25, 2023
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [descheduler](https://togithub.com/kubernetes-sigs/descheduler) |
minor | `0.27.1` -> `0.28.0` |

---

### Release Notes

<details>
<summary>kubernetes-sigs/descheduler (descheduler)</summary>

###
[`v0.28.0`](https://togithub.com/kubernetes-sigs/descheduler/releases/tag/v0.28.0):
Descheduler v0.28.0

[Compare
Source](https://togithub.com/kubernetes-sigs/descheduler/compare/v0.27.1...v0.28.0)

#### What's Changed

- add unit tests for version compatibility check by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1096](https://togithub.com/kubernetes-sigs/descheduler/pull/1096)
- fix plugin arg conversion when using multiple profiles with same
plugin by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1143](https://togithub.com/kubernetes-sigs/descheduler/pull/1143)
- Use dl.k8s.io instead of kubernetes-release bucket by
[@&#8203;ratnopamc](https://togithub.com/ratnopamc) in
[https://github.com/kubernetes-sigs/descheduler/pull/1145](https://togithub.com/kubernetes-sigs/descheduler/pull/1145)
- update deprecated sets.String to generic sets by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1146](https://togithub.com/kubernetes-sigs/descheduler/pull/1146)
- helm: ability to override command and args. set args inline by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1151](https://togithub.com/kubernetes-sigs/descheduler/pull/1151)
- fix: imagepullsecrets indentation for kind: Deployment by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1150](https://togithub.com/kubernetes-sigs/descheduler/pull/1150)
- docs: supplement missing link by
[@&#8203;mikutas](https://togithub.com/mikutas) in
[https://github.com/kubernetes-sigs/descheduler/pull/1159](https://togithub.com/kubernetes-sigs/descheduler/pull/1159)
- bump chart to v0.27.1 by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1160](https://togithub.com/kubernetes-sigs/descheduler/pull/1160)
- Custom labels for ServiceMonitor resource by
[@&#8203;nlamirault](https://togithub.com/nlamirault) in
[https://github.com/kubernetes-sigs/descheduler/pull/1147](https://togithub.com/kubernetes-sigs/descheduler/pull/1147)
- pod anti-affinity check among nodes by
[@&#8203;10hin](https://togithub.com/10hin) in
[https://github.com/kubernetes-sigs/descheduler/pull/1033](https://togithub.com/kubernetes-sigs/descheduler/pull/1033)
- PodLifeTime: support CrashLoopBackOff container state by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1164](https://togithub.com/kubernetes-sigs/descheduler/pull/1164)
- update pause image from 'kubernetes/pause' to 'registry.k8s.io/pause'
by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1166](https://togithub.com/kubernetes-sigs/descheduler/pull/1166)
- TooManyRestart: state filter for CrashLoopBackOff by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1165](https://togithub.com/kubernetes-sigs/descheduler/pull/1165)
- Replace deprecated command with environment file by
[@&#8203;jongwooo](https://togithub.com/jongwooo) in
[https://github.com/kubernetes-sigs/descheduler/pull/1173](https://togithub.com/kubernetes-sigs/descheduler/pull/1173)
- use pod informers for listing pods in
removepodsviolatingtopologyspreadconstraint and
removepodsviolatinginterpodantiaffinity by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1163](https://togithub.com/kubernetes-sigs/descheduler/pull/1163)
- removepodsviolatingtopologyspreadconstraint: implement explicit
constraints by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1148](https://togithub.com/kubernetes-sigs/descheduler/pull/1148)
- FakeClientset: bump watch channel size by
[@&#8203;ingvagabund](https://togithub.com/ingvagabund) in
[https://github.com/kubernetes-sigs/descheduler/pull/1174](https://togithub.com/kubernetes-sigs/descheduler/pull/1174)
- deschedule/balance order (continuation) by
[@&#8203;ingvagabund](https://togithub.com/ingvagabund) in
[https://github.com/kubernetes-sigs/descheduler/pull/1177](https://togithub.com/kubernetes-sigs/descheduler/pull/1177)
- bump log level for processing info by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1141](https://togithub.com/kubernetes-sigs/descheduler/pull/1141)
- fix priority threshold by name alone by
[@&#8203;knelasevero](https://togithub.com/knelasevero) in
[https://github.com/kubernetes-sigs/descheduler/pull/1186](https://togithub.com/kubernetes-sigs/descheduler/pull/1186)
- feat: Enable open telemetry tracing by
[@&#8203;harshanarayana](https://togithub.com/harshanarayana) in
[https://github.com/kubernetes-sigs/descheduler/pull/1189](https://togithub.com/kubernetes-sigs/descheduler/pull/1189)
- bump to k8s 1.28 beta.0 by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1201](https://togithub.com/kubernetes-sigs/descheduler/pull/1201)
- nodefit: aggregate errors by
[@&#8203;lucming](https://togithub.com/lucming) in
[https://github.com/kubernetes-sigs/descheduler/pull/1203](https://togithub.com/kubernetes-sigs/descheduler/pull/1203)
- fix: base configmap missing plugin RemoveDuplicates by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1207](https://togithub.com/kubernetes-sigs/descheduler/pull/1207)
- feat: Implement preferredDuringSchedulingIgnoredDuringExecution for
RemovePodsViolatingNodeAffinity by
[@&#8203;jordipiqueselles](https://togithub.com/jordipiqueselles) in
[https://github.com/kubernetes-sigs/descheduler/pull/1210](https://togithub.com/kubernetes-sigs/descheduler/pull/1210)
- fix: descheduler_loop_duration_seconds has wrong value by
[@&#8203;Abirdcfly](https://togithub.com/Abirdcfly) in
[https://github.com/kubernetes-sigs/descheduler/pull/1215](https://togithub.com/kubernetes-sigs/descheduler/pull/1215)
- .gitattribute to not pollute PRs or stats by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1202](https://togithub.com/kubernetes-sigs/descheduler/pull/1202)
- Bump Kubernetes dependencies to v1.28.0 by
[@&#8203;JaneLiuL](https://togithub.com/JaneLiuL) in
[https://github.com/kubernetes-sigs/descheduler/pull/1216](https://togithub.com/kubernetes-sigs/descheduler/pull/1216)
- profile: fix span attribute typo by
[@&#8203;antoinedeschenes](https://togithub.com/antoinedeschenes) in
[https://github.com/kubernetes-sigs/descheduler/pull/1221](https://togithub.com/kubernetes-sigs/descheduler/pull/1221)
- k8s 1.28: update docs and go-version by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1224](https://togithub.com/kubernetes-sigs/descheduler/pull/1224)

#### New Contributors

- [@&#8203;ratnopamc](https://togithub.com/ratnopamc) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1145](https://togithub.com/kubernetes-sigs/descheduler/pull/1145)
- [@&#8203;mikutas](https://togithub.com/mikutas) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1159](https://togithub.com/kubernetes-sigs/descheduler/pull/1159)
- [@&#8203;nlamirault](https://togithub.com/nlamirault) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1147](https://togithub.com/kubernetes-sigs/descheduler/pull/1147)
- [@&#8203;10hin](https://togithub.com/10hin) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1033](https://togithub.com/kubernetes-sigs/descheduler/pull/1033)
- [@&#8203;jongwooo](https://togithub.com/jongwooo) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1173](https://togithub.com/kubernetes-sigs/descheduler/pull/1173)
- [@&#8203;jordipiqueselles](https://togithub.com/jordipiqueselles) made
their first contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1210](https://togithub.com/kubernetes-sigs/descheduler/pull/1210)
- [@&#8203;Abirdcfly](https://togithub.com/Abirdcfly) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1215](https://togithub.com/kubernetes-sigs/descheduler/pull/1215)
- [@&#8203;antoinedeschenes](https://togithub.com/antoinedeschenes) made
their first contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1221](https://togithub.com/kubernetes-sigs/descheduler/pull/1221)

**Full Changelog**:
kubernetes-sigs/descheduler@v0.27.0...v0.28.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Ny44IiwidXBkYXRlZEluVmVyIjoiMzYuNTcuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: repo-jeeves <106431701+repo-jeeves[bot]@users.noreply.github.com>
spirkaa added a commit to spirkaa/infra that referenced this pull request Aug 27, 2023
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [descheduler](https://github.com/kubernetes-sigs/descheduler) | minor | `0.27.1` -> `0.28.0` |

---

### Release Notes

<details>
<summary>kubernetes-sigs/descheduler (descheduler)</summary>

### [`v0.28.0`](https://github.com/kubernetes-sigs/descheduler/releases/tag/v0.28.0): Descheduler v0.28.0

[Compare Source](kubernetes-sigs/descheduler@v0.27.1...v0.28.0)

#### What's Changed

-   add unit tests for version compatibility check by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1096
-   fix plugin arg conversion when using multiple profiles with same plugin by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1143
-   Use dl.k8s.io instead of kubernetes-release bucket by [@&#8203;ratnopamc](https://github.com/ratnopamc) in kubernetes-sigs/descheduler#1145
-   update deprecated sets.String to generic sets by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1146
-   helm: ability to override command and args. set args inline by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1151
-   fix: imagepullsecrets indentation for kind: Deployment by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1150
-   docs: supplement missing link by [@&#8203;mikutas](https://github.com/mikutas) in kubernetes-sigs/descheduler#1159
-   bump chart to v0.27.1 by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1160
-   Custom labels for ServiceMonitor resource by [@&#8203;nlamirault](https://github.com/nlamirault) in kubernetes-sigs/descheduler#1147
-   pod anti-affinity check among nodes by [@&#8203;10hin](https://github.com/10hin) in kubernetes-sigs/descheduler#1033
-   PodLifeTime: support CrashLoopBackOff container state by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1164
-   update pause image from 'kubernetes/pause' to 'registry.k8s.io/pause' by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1166
-   TooManyRestart: state filter for CrashLoopBackOff by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1165
-   Replace deprecated command with environment file by [@&#8203;jongwooo](https://github.com/jongwooo) in kubernetes-sigs/descheduler#1173
-   use pod informers for listing pods in removepodsviolatingtopologyspreadconstraint and removepodsviolatinginterpodantiaffinity by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1163
-   removepodsviolatingtopologyspreadconstraint: implement explicit constraints by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1148
-   FakeClientset: bump watch channel size by [@&#8203;ingvagabund](https://github.com/ingvagabund) in kubernetes-sigs/descheduler#1174
-   deschedule/balance order (continuation) by [@&#8203;ingvagabund](https://github.com/ingvagabund) in kubernetes-sigs/descheduler#1177
-   bump log level for processing info by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1141
-   fix priority threshold by name alone by [@&#8203;knelasevero](https://github.com/knelasevero) in kubernetes-sigs/descheduler#1186
-   feat: Enable open telemetry tracing by [@&#8203;harshanarayana](https://github.com/harshanarayana) in kubernetes-sigs/descheduler#1189
-   bump to k8s 1.28 beta.0 by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1201
-   nodefit: aggregate errors by [@&#8203;lucming](https://github.com/lucming) in kubernetes-sigs/descheduler#1203
-   fix: base configmap missing plugin RemoveDuplicates by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1207
-   feat: Implement preferredDuringSchedulingIgnoredDuringExecution for RemovePodsViolatingNodeAffinity by [@&#8203;jordipiqueselles](https://github.com/jordipiqueselles) in kubernetes-sigs/descheduler#1210
-   fix: descheduler_loop_duration_seconds has wrong value by [@&#8203;Abirdcfly](https://github.com/Abirdcfly) in kubernetes-sigs/descheduler#1215
-   .gitattribute to not pollute PRs or stats by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1202
-   Bump Kubernetes dependencies to v1.28.0 by [@&#8203;JaneLiuL](https://github.com/JaneLiuL) in kubernetes-sigs/descheduler#1216
-   profile: fix span attribute typo by [@&#8203;antoinedeschenes](https://github.com/antoinedeschenes) in kubernetes-sigs/descheduler#1221
-   k8s 1.28: update docs and go-version by [@&#8203;a7i](https://github.com/a7i) in kubernetes-sigs/descheduler#1224

#### New Contributors

-   [@&#8203;ratnopamc](https://github.com/ratnopamc) made their first contribution in kubernetes-sigs/descheduler#1145
-   [@&#8203;mikutas](https://github.com/mikutas) made their first contribution in kubernetes-sigs/descheduler#1159
-   [@&#8203;nlamirault](https://github.com/nlamirault) made their first contribution in kubernetes-sigs/descheduler#1147
-   [@&#8203;10hin](https://github.com/10hin) made their first contribution in kubernetes-sigs/descheduler#1033
-   [@&#8203;jongwooo](https://github.com/jongwooo) made their first contribution in kubernetes-sigs/descheduler#1173
-   [@&#8203;jordipiqueselles](https://github.com/jordipiqueselles) made their first contribution in kubernetes-sigs/descheduler#1210
-   [@&#8203;Abirdcfly](https://github.com/Abirdcfly) made their first contribution in kubernetes-sigs/descheduler#1215
-   [@&#8203;antoinedeschenes](https://github.com/antoinedeschenes) made their first contribution in kubernetes-sigs/descheduler#1221

**Full Changelog**: kubernetes-sigs/descheduler@v0.27.0...v0.28.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41Mi4yIiwidXBkYXRlZEluVmVyIjoiMzYuNTIuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Reviewed-on: https://git.devmem.ru/projects/infra/pulls/949
Co-authored-by: Renovate Bot <renovate@devmem.ru>
Co-committed-by: Renovate Bot <renovate@devmem.ru>
renovate bot referenced this pull request in teutonet/teutonet-helm-charts Sep 11, 2023
…0.28.x (#538)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [descheduler](https://togithub.com/kubernetes-sigs/descheduler) |
minor | `0.27.x` -> `0.28.x` |

---

### Release Notes

<details>
<summary>kubernetes-sigs/descheduler (descheduler)</summary>

###
[`v0.28.0`](https://togithub.com/kubernetes-sigs/descheduler/releases/tag/v0.28.0):
Descheduler v0.28.0

[Compare
Source](https://togithub.com/kubernetes-sigs/descheduler/compare/v0.27.1...v0.28.0)

#### What's Changed

- add unit tests for version compatibility check by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1096](https://togithub.com/kubernetes-sigs/descheduler/pull/1096)
- fix plugin arg conversion when using multiple profiles with same
plugin by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1143](https://togithub.com/kubernetes-sigs/descheduler/pull/1143)
- Use dl.k8s.io instead of kubernetes-release bucket by
[@&#8203;ratnopamc](https://togithub.com/ratnopamc) in
[https://github.com/kubernetes-sigs/descheduler/pull/1145](https://togithub.com/kubernetes-sigs/descheduler/pull/1145)
- update deprecated sets.String to generic sets by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1146](https://togithub.com/kubernetes-sigs/descheduler/pull/1146)
- helm: ability to override command and args. set args inline by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1151](https://togithub.com/kubernetes-sigs/descheduler/pull/1151)
- fix: imagepullsecrets indentation for kind: Deployment by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1150](https://togithub.com/kubernetes-sigs/descheduler/pull/1150)
- docs: supplement missing link by
[@&#8203;mikutas](https://togithub.com/mikutas) in
[https://github.com/kubernetes-sigs/descheduler/pull/1159](https://togithub.com/kubernetes-sigs/descheduler/pull/1159)
- bump chart to v0.27.1 by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1160](https://togithub.com/kubernetes-sigs/descheduler/pull/1160)
- Custom labels for ServiceMonitor resource by
[@&#8203;nlamirault](https://togithub.com/nlamirault) in
[https://github.com/kubernetes-sigs/descheduler/pull/1147](https://togithub.com/kubernetes-sigs/descheduler/pull/1147)
- pod anti-affinity check among nodes by
[@&#8203;10hin](https://togithub.com/10hin) in
[https://github.com/kubernetes-sigs/descheduler/pull/1033](https://togithub.com/kubernetes-sigs/descheduler/pull/1033)
- PodLifeTime: support CrashLoopBackOff container state by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1164](https://togithub.com/kubernetes-sigs/descheduler/pull/1164)
- update pause image from 'kubernetes/pause' to 'registry.k8s.io/pause'
by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1166](https://togithub.com/kubernetes-sigs/descheduler/pull/1166)
- TooManyRestart: state filter for CrashLoopBackOff by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1165](https://togithub.com/kubernetes-sigs/descheduler/pull/1165)
- Replace deprecated command with environment file by
[@&#8203;jongwooo](https://togithub.com/jongwooo) in
[https://github.com/kubernetes-sigs/descheduler/pull/1173](https://togithub.com/kubernetes-sigs/descheduler/pull/1173)
- use pod informers for listing pods in
removepodsviolatingtopologyspreadconstraint and
removepodsviolatinginterpodantiaffinity by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1163](https://togithub.com/kubernetes-sigs/descheduler/pull/1163)
- removepodsviolatingtopologyspreadconstraint: implement explicit
constraints by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1148](https://togithub.com/kubernetes-sigs/descheduler/pull/1148)
- FakeClientset: bump watch channel size by
[@&#8203;ingvagabund](https://togithub.com/ingvagabund) in
[https://github.com/kubernetes-sigs/descheduler/pull/1174](https://togithub.com/kubernetes-sigs/descheduler/pull/1174)
- deschedule/balance order (continuation) by
[@&#8203;ingvagabund](https://togithub.com/ingvagabund) in
[https://github.com/kubernetes-sigs/descheduler/pull/1177](https://togithub.com/kubernetes-sigs/descheduler/pull/1177)
- bump log level for processing info by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1141](https://togithub.com/kubernetes-sigs/descheduler/pull/1141)
- fix priority threshold by name alone by
[@&#8203;knelasevero](https://togithub.com/knelasevero) in
[https://github.com/kubernetes-sigs/descheduler/pull/1186](https://togithub.com/kubernetes-sigs/descheduler/pull/1186)
- feat: Enable open telemetry tracing by
[@&#8203;harshanarayana](https://togithub.com/harshanarayana) in
[https://github.com/kubernetes-sigs/descheduler/pull/1189](https://togithub.com/kubernetes-sigs/descheduler/pull/1189)
- bump to k8s 1.28 beta.0 by [@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1201](https://togithub.com/kubernetes-sigs/descheduler/pull/1201)
- nodefit: aggregate errors by
[@&#8203;lucming](https://togithub.com/lucming) in
[https://github.com/kubernetes-sigs/descheduler/pull/1203](https://togithub.com/kubernetes-sigs/descheduler/pull/1203)
- fix: base configmap missing plugin RemoveDuplicates by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1207](https://togithub.com/kubernetes-sigs/descheduler/pull/1207)
- feat: Implement preferredDuringSchedulingIgnoredDuringExecution for
RemovePodsViolatingNodeAffinity by
[@&#8203;jordipiqueselles](https://togithub.com/jordipiqueselles) in
[https://github.com/kubernetes-sigs/descheduler/pull/1210](https://togithub.com/kubernetes-sigs/descheduler/pull/1210)
- fix: descheduler_loop_duration_seconds has wrong value by
[@&#8203;Abirdcfly](https://togithub.com/Abirdcfly) in
[https://github.com/kubernetes-sigs/descheduler/pull/1215](https://togithub.com/kubernetes-sigs/descheduler/pull/1215)
- .gitattribute to not pollute PRs or stats by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1202](https://togithub.com/kubernetes-sigs/descheduler/pull/1202)
- Bump Kubernetes dependencies to v1.28.0 by
[@&#8203;JaneLiuL](https://togithub.com/JaneLiuL) in
[https://github.com/kubernetes-sigs/descheduler/pull/1216](https://togithub.com/kubernetes-sigs/descheduler/pull/1216)
- profile: fix span attribute typo by
[@&#8203;antoinedeschenes](https://togithub.com/antoinedeschenes) in
[https://github.com/kubernetes-sigs/descheduler/pull/1221](https://togithub.com/kubernetes-sigs/descheduler/pull/1221)
- k8s 1.28: update docs and go-version by
[@&#8203;a7i](https://togithub.com/a7i) in
[https://github.com/kubernetes-sigs/descheduler/pull/1224](https://togithub.com/kubernetes-sigs/descheduler/pull/1224)

#### New Contributors

- [@&#8203;ratnopamc](https://togithub.com/ratnopamc) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1145](https://togithub.com/kubernetes-sigs/descheduler/pull/1145)
- [@&#8203;mikutas](https://togithub.com/mikutas) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1159](https://togithub.com/kubernetes-sigs/descheduler/pull/1159)
- [@&#8203;nlamirault](https://togithub.com/nlamirault) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1147](https://togithub.com/kubernetes-sigs/descheduler/pull/1147)
- [@&#8203;10hin](https://togithub.com/10hin) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1033](https://togithub.com/kubernetes-sigs/descheduler/pull/1033)
- [@&#8203;jongwooo](https://togithub.com/jongwooo) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1173](https://togithub.com/kubernetes-sigs/descheduler/pull/1173)
- [@&#8203;jordipiqueselles](https://togithub.com/jordipiqueselles) made
their first contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1210](https://togithub.com/kubernetes-sigs/descheduler/pull/1210)
- [@&#8203;Abirdcfly](https://togithub.com/Abirdcfly) made their first
contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1215](https://togithub.com/kubernetes-sigs/descheduler/pull/1215)
- [@&#8203;antoinedeschenes](https://togithub.com/antoinedeschenes) made
their first contribution in
[https://github.com/kubernetes-sigs/descheduler/pull/1221](https://togithub.com/kubernetes-sigs/descheduler/pull/1221)

**Full Changelog**:
kubernetes-sigs/descheduler@v0.27.0...v0.28.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/teutonet/teutonet-helm-charts).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi42OC4xIiwidXBkYXRlZEluVmVyIjoiMzYuNjguMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Chris Werner Rau <cwr@teuto.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants