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

Remove static max check labels verification #659

Merged
merged 1 commit into from
Apr 10, 2024
Merged

Conversation

ka3de
Copy link
Contributor

@ka3de ka3de commented Mar 18, 2024

This is a cherry-pick from #606 in order to delay the static labels limit verification removal until this mechanism is in place in the API. Once this static verification is removed from the agent, the limit will be checked in the API in relation with the specific tenant limit.

This should now be done in the API taking into account the particular
tenant limits.
ka3de added a commit that referenced this pull request Mar 18, 2024
This reverts commit 5e12d74.
This modification is now moved to a separate PR (see #659).
@ka3de ka3de marked this pull request as ready for review April 8, 2024 10:43
@ka3de ka3de requested a review from a team as a code owner April 8, 2024 10:43
@ka3de ka3de merged commit c866dc0 into main Apr 10, 2024
4 checks passed
@ka3de ka3de deleted the remove/static-labels-limit branch April 10, 2024 07:47
ka3de added a commit that referenced this pull request Apr 10, 2024
* Log publisher errors (#661)
* Remove max check labels verification (#659)

Signed-off-by: ka3de <danijs12@hotmail.com>
@ka3de ka3de mentioned this pull request Apr 10, 2024
ka3de added a commit that referenced this pull request Apr 10, 2024
* Log publisher errors (#661)
* Remove max check labels verification (#659)

Signed-off-by: ka3de <danijs12@hotmail.com>
ka3de added a commit that referenced this pull request Apr 10, 2024
* Log publisher errors (#661)
* Remove max check labels verification (#659)

Signed-off-by: ka3de <danijs12@hotmail.com>
ka3de added a commit that referenced this pull request Apr 10, 2024
* Log publisher errors (#661)
* Remove max check labels verification (#659)

Signed-off-by: ka3de <danijs12@hotmail.com>
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