alibaba/sentinel-golang

Rule Manager: LoadRules API returns failed Rule list

Aperta

#236 aperta il 9 set 2020

 (4 commenti) (0 reazioni) (1 assegnatario)Go (453 fork)github user discovery
area/rule-managergood first issuekind/enhancement

Metriche repository

Star
 (2957 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor