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

[No click le green button] Attempt to cherry pick changes into main #4879

Merged
merged 3 commits into from
Oct 5, 2022

Conversation

c-kruse
Copy link
Contributor

@c-kruse c-kruse commented Sep 29, 2022

No description provided.

@jamesdphillips
Copy link
Contributor

This may negate the need for #4807

jamesdphillips and others added 3 commits October 5, 2022 09:03
* Expose `EntityConfig` & `EntityState` resources through GraphQL service
* Add `Fields()` method to `EntityConfig` & `EntityState` resources so that they implement the `Fielder` interface; allows users to filter with field selectors.
* Complication: The core/v3 package previous checked that each resource in the package had a unique `RBACName()`, however, since we don't want distinct names for the config & state resources this proved to be problematic. As such I've disabled the uniqueness tests for the time being. (Same as: #4807)

Signed-off-by: James Phillips <[email protected]>
(cherry picked from commit f3bc8f5)
Heavy merge guessing by CK.

Signed-off-by: James Phillips <[email protected]>
(cherry picked from commit 7a13587)
Original work (#4862) adapted for 7.x track

Signed-off-by: Christian Kruse <[email protected]>

go mod tidy

Signed-off-by: Christian Kruse <[email protected]>
@c-kruse c-kruse merged commit 04283ff into main Oct 5, 2022
@c-kruse c-kruse deleted the cherrypick-dev6 branch October 5, 2022 16:37
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.

2 participants