alibaba/sentinel-golang

Rule Manager: LoadRules API returns failed Rule list

Ouverte

#236 ouverte le 9 sept. 2020

 (4 commentaires) (0 réaction) (1 personne assignée)Go (453 forks)github user discovery
area/rule-managergood first issuekind/enhancement

Métriques du dépôt

Stars
 (2 957 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Issue Description

Type: feature request

Describe what feature you want

func LoadRules(rules []*Rule) (bool, error)

Currently, LoadRule function will hide the failed rules(not valid or other reasons), maybe it doesn't make sense.

We might need to return failed rules.

The func signature is as below:

// LoadRules loads the given flow rules to the rule manager, while all previous rules will be replaced.
// First returned value(bool): whether occurs loading, if the rules is the same with current effective rules, return false, else return true.
// Second returned value(error): error when do load
// Third returned value([]*Rule): failed rule list
// If error isn't nil, the third returned value will be same with input params rules
func LoadRules(rules []*Rule) (bool, error, []*Rule) {
//......
}

Additional context

Add any other context or screenshots about the feature request here.

Guide contributeur