DevFW-CICD/ingress-nginx-helm
12
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #613 from aledbf/fix-multiple-status

Browse source 
Status leader election must consired the ingress class
This commit is contained in:
Manuel Alejandro de Brito Fontes 2017-04-16 17:10:54 -03:00 committed by GitHub
commit fc67b1d5e2
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
core/pkg/ingress/status/status.go
View file

@ -17,6 +17,7 @@ limitations under the License.
package status
import (
"fmt"
"net"
"sort"
"sync"
@ -187,7 +188,14 @@ func NewStatusSyncer(config Config) Sync {
}
st.syncQueue = task.NewCustomTaskQueue(st.sync, st.keyfunc)
le, err := NewElection(config.ElectionID,
// we need to use the defined ingress class to allow multiple leaders
// in order to update information about ingress status
id := fmt.Sprintf("%v-%v", config.ElectionID, config.DefaultIngressClass)
if config.IngressClass != "" {
id = fmt.Sprintf("%v-%v", config.ElectionID, config.IngressClass)
}
le, err := NewElection(id,
pod.Name, pod.Namespace, 30*time.Second,
st.callback, config.Client)
if err != nil {