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: filter ordinals #1386

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

reegnz
Copy link
Contributor

@reegnz reegnz commented Oct 25, 2024

Implements #1384

What this PR does / why we need it

See #1384 for explanation.

Which issue(s) this PR fixes

Fixes #1384

Does this PR introduced a user-facing change?

Filter and ClusterFilter ordering can now be influenced with the `.spec.ordinal` field.

Additional documentation, usage docs, etc


@reegnz
Copy link
Contributor Author

reegnz commented Oct 25, 2024

I'm not sure about the usage of positive/negative ordinals, so maybe the runs after namespaced filters could be a separate field in the ClusterFilter CRD instead.

@reegnz
Copy link
Contributor Author

reegnz commented Oct 25, 2024

Actually I think I need to separate the ordinal-based sorting from the before/after namespace filters part.

Implements fluent#1384

Signed-off-by: Zoltán Reegn <[email protected]>
@benjaminhuo
Copy link
Member

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.

Feature request: proper control over filter order with ordinals
2 participants