Label selection for pods/processGroups

[nicmorales9]

Description

This PR adds the ability to select pods/processGroups by label selection. This still requires a cluster name to be provided as I felt that would reduce the risk of accidental selection.
It also improves the error message of the cordon command for when no pods are found to be running on a node.

This solves #597

Type of change

Please select one of the options below.

• New feature (non-breaking change which adds functionality)

Discussion

Are there any design details that you would like to discuss further?

Testing

t

Documentation

Did you update relevant documentation within this repository?

If this change is adding new functionality, do we need to describe it in our user manual?

If this change is adding or removing subreconcilers, have we updated the core technical design doc to reflect that?

If this change is adding new safety checks or new potential failure modes, have we documented and how to debug potential issues?

Follow-up

Are there any follow-up issues that we should pursue in the future?

Does this introduce new defaults that we should re-evaluate in the future?

[foundationdb-ci]

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

• Commit ID: 82caec8
• Duration 0:05:09
• Result: ❌ FAILED
• Error: reference not found for primary source and source version 82caec84ebc61755b1d99f3b128916fa6ad70426
• Build Log terminal output (available for 30 days)
• Build Workspace zip file of the working directory (available for 30 days)

[foundationdb-ci]

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

• Commit ID: 3d2ece0
• Duration 2:07:57
• Result: ❌ FAILED
• Error: Error while executing command: if $fail_test; then exit 1; fi. Reason: exit status 1
• Build Log terminal output (available for 30 days)
• Build Workspace zip file of the working directory (available for 30 days)

[nicmorales9]

Github doesn't want me to edit the comment today but for testing:
Unit tests were added, and I tested all the process selection commands manually. I also ensured that cordon is still working fine since it has some overlapping functions.

[foundationdb-ci]

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

• Commit ID: 3f7e304
• Duration 2:20:06
• Result: ✅ SUCCEEDED
• Error: N/A
• Build Log terminal output (available for 30 days)
• Build Workspace zip file of the working directory (available for 30 days)

[foundationdb-ci]

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

• Commit ID: 93750e7
• Duration 2:26:47
• Result: ✅ SUCCEEDED
• Error: N/A
• Build Log terminal output (available for 30 days)
• Build Workspace zip file of the working directory (available for 30 days)

[brownleej]

👍

[johscheuer]

LGTM 👍 Thanks! As a next step we should remove the useProcessGroupID option.

[foundationdb-ci]

Result of fdb-kubernetes-operator-pr on Linux CentOS 7

• Commit ID: 3812ae7
• Duration 2:18:37
• Result: ✅ SUCCEEDED
• Error: N/A
• Build Log terminal output (available for 30 days)
• Build Workspace zip file of the working directory (available for 30 days)