Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

Initial commit of labelSelector support for ClusterDecisionResource #272

Merged
merged 3 commits into from
Jun 22, 2021

Conversation

jnpacker
Copy link
Contributor

Signed-off-by: Joshua Packer jpacker@redhat.com

  • This adds labelSelector support to the ClusterDecisionResource generator
  • It also fixed an issue in the functional tests showing false positives
  • I also fixed cluster_test.go which had the same false positive check
  • Includes a documentation update as well to show how a labelSelector would be used. (pulls in standard Kubernetes labelSelector)

Signed-off-by: Joshua Packer <jpacker@redhat.com>
@jnpacker
Copy link
Contributor Author

Issue Ref: #271

Signed-off-by: Joshua Packer <jpacker@redhat.com>
Signed-off-by: Joshua Packer <jpacker@redhat.com>
@@ -232,7 +232,7 @@ func TestGenerateParams(t *testing.T) {
}, nil)

if testCase.expectedError != nil {
assert.Error(t, testCase.expectedError, err)
assert.EqualError(t, err, testCase.expectedError.Error())
Copy link
Member

Choose a reason for hiding this comment

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

Good catch 😅

Copy link
Member

@jgwest jgwest left a comment

Choose a reason for hiding this comment

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

Looks great @jnpacker! Anything more you were planning to add or good to merge?

@jnpacker
Copy link
Contributor Author

Good to merge.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants