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

Add support for PodSelector in DrainSpec #231

Merged
merged 4 commits into from
Mar 24, 2023

Conversation

brandond
Copy link
Member

@brandond brandond commented Mar 15, 2023

The selected labels are added to the default LabelSelector that excludes the SUC pods from being drained.

Also bumps various go modules and the kubectl image up to latest for Kubernetes 1.21

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@Sahota1225 Sahota1225 requested a review from a team March 15, 2023 15:27
@rancher-max
Copy link
Member

The selected labels are added to the default LabelSelector that excludes the SUC pods from being drained.

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@brandond brandond force-pushed the configurable-pod-selector branch 3 times, most recently from b502ec7 to a619dac Compare March 22, 2023 23:30
@brandond brandond requested a review from snasovich March 22, 2023 23:39
Copy link
Collaborator

@snasovich snasovich left a comment

Choose a reason for hiding this comment

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

LGTM, did not expect addressing my comment to be somewhat involved. Thank you!

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
@brandond brandond force-pushed the configurable-pod-selector branch from a619dac to 111cc4c Compare March 24, 2023 00:57
@brandond brandond merged commit ac25caa into rancher:master Mar 24, 2023
Stackclash referenced this pull request in Stackclash/home-cluster May 25, 2023
….11.0 (#887)

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

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[rancher/system-upgrade-controller](https://togithub.com/rancher/system-upgrade-controller)
| Kustomization | minor | `v0.10.0` -> `v0.11.0` |

---

### Release Notes

<details>
<summary>rancher/system-upgrade-controller</summary>

###
[`v0.11.0`](https://togithub.com/rancher/system-upgrade-controller/releases/tag/v0.11.0)

[Compare
Source](https://togithub.com/rancher/system-upgrade-controller/compare/v0.10.0...v0.11.0)

##### What's Changed

- Add support for PodSelector in DrainSpec by
[@&#8203;brandond](https://togithub.com/brandond) in
[https://github.com/rancher/system-upgrade-controller/pull/231](https://togithub.com/rancher/system-upgrade-controller/pull/231)
- Set Kubernetes owner references on Job objects by
[@&#8203;brandond](https://togithub.com/brandond) in
[https://github.com/rancher/system-upgrade-controller/pull/235](https://togithub.com/rancher/system-upgrade-controller/pull/235)
- examples: add upgrade YAML for MicroOS by
[@&#8203;aplanas](https://togithub.com/aplanas) in
[https://github.com/rancher/system-upgrade-controller/pull/237](https://togithub.com/rancher/system-upgrade-controller/pull/237)
- Fix Spec.Drain.PodSelector by
[@&#8203;brandond](https://togithub.com/brandond) in
[https://github.com/rancher/system-upgrade-controller/pull/239](https://togithub.com/rancher/system-upgrade-controller/pull/239)
- Bump github.com/rancher/wrangler from
0.8.11-0.20230207214913-209077f2d98f to 0.8.11 in /pkg/apis by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/rancher/system-upgrade-controller/pull/233](https://togithub.com/rancher/system-upgrade-controller/pull/233)

##### New Contributors

- [@&#8203;aplanas](https://togithub.com/aplanas) made their first
contribution in
[https://github.com/rancher/system-upgrade-controller/pull/237](https://togithub.com/rancher/system-upgrade-controller/pull/237)

**Full Changelog**:
rancher/system-upgrade-controller@v0.10.0...v0.11.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on saturday" (UTC), 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 [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/RickCoxDev/home-cluster).

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

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

Successfully merging this pull request may close these issues.

5 participants