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

Fixes #24665: Number of nodes in rule badge is not correct with tenants #5572

Conversation

fanf
Copy link
Member

@fanf fanf commented Apr 3, 2024

https://issues.rudder.io/issues/24665

It's likely an older bug than that, I need to check it better.
EDIT: it's from 7.3.
But the only usage of that method in all rudder seems to be for badge, so not too important.

@fanf fanf requested a review from VinceMacBuche April 3, 2024 21:22
@fanf fanf marked this pull request as draft April 3, 2024 21:22
@fanf fanf changed the base branch from branches/rudder/8.1 to branches/rudder/7.3 April 4, 2024 14:43
@fanf fanf force-pushed the bug_24665/number_of_nodes_in_rule_badge_is_not_correct_with_tenants branch from e988d7e to 7d0d06a Compare April 4, 2024 15:06
@fanf fanf marked this pull request as ready for review April 4, 2024 15:07
@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder/pull/5572
-- Your faithful QA
Kant merge: "Science is organized knowledge. Wisdom is organized life."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/83105/console)

@fanf
Copy link
Member Author

fanf commented Apr 17, 2024

OK, merging this PR

@fanf fanf merged commit b608506 into Normation:branches/rudder/7.3 Apr 17, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants