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

Streamline MaxPipeline check for LogPipeline #1604

Open
a-thaler opened this issue Nov 12, 2024 · 1 comment
Open

Streamline MaxPipeline check for LogPipeline #1604

a-thaler opened this issue Nov 12, 2024 · 1 comment
Labels
area/logs LogPipeline

Comments

@a-thaler
Copy link
Collaborator

Description
The max pipeline check for LogPipelines is executed as part of the validating webhook. For TracePipeline and MetricPipeline it is executed as a reconciler activity with proper status on the CR.

Goal: Use the same approach for all pipelines and remove unneeded code

Criteria:

  • If a pipeline gets added which exceeds the max pipeline threshold, the pipeline never gets active
  • The approach is streamlined across the pipeline types
@a-thaler a-thaler added the area/logs LogPipeline label Nov 12, 2024
Copy link

This issue has been automatically marked as stale due to the lack of recent activity. It will soon be closed if no further activity occurs.
Thank you for your contributions.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 12, 2025
@a-thaler a-thaler removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logs LogPipeline
Projects
None yet
Development

No branches or pull requests

1 participant