Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

fix(monitoring) use default params to sort services #5744

Merged
merged 3 commits into from
Oct 10, 2017
Merged

fix(monitoring) use default params to sort services #5744

merged 3 commits into from
Oct 10, 2017

Conversation

loiclau
Copy link
Contributor

@loiclau loiclau commented Oct 5, 2017 •
edited
Loading

@kduret kduret self-requested a review October 5, 2017 14:57
@ganoze
Copy link
Contributor

ganoze commented Oct 5, 2017

@loiclau @kduret @Dacchi Are the changes compatible with the master branch ?

@loiclau
Copy link
Contributor Author

loiclau commented Oct 5, 2017

I think

@Dacchi
Copy link
Contributor

Dacchi commented Oct 5, 2017

Sounds good for master to me.
But i will re-check to be sure of it.

@ganoze ganoze merged commit 0501531 into 2.8.x Oct 10, 2017
@ganoze ganoze deleted the fix-monitoring-sort branch October 10, 2017 08:15
@ganoze ganoze added this to the 2.8.15 milestone Oct 11, 2017
@chgautier
Copy link
Contributor

refer to #5695

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kduret kduret kduret approved these changes

@Dacchi Dacchi Dacchi approved these changes

Assignees
No one assigned
Labels
area/monitoring kind/bug priority/major
Projects
None yet
Milestone
2.8.15
Development

Successfully merging this pull request may close these issues.
6 participants
@loiclau @ganoze @Dacchi @chgautier @kduret @lpinsivy