BUG/MINOR: prevents unecessary restart when using multiple syslog servers

When using more than one syslog server, the algorithm for comparing log targets
never succeeds, because the conversion of each `syslog-server` annotation's
line into a `models.LogTarget` will always set the `.Index` field equal to 0.

In this scenario, any change causes a restart because `lg` and `newLg` will
never be equal (either because of the slice order, or because of the different
`.Index` fields)

Author: fabianonunes

pkg/annotations/global/syslogServer.go

@@ -35,7 +35,7 @@ func (a *SyslogServers) GetName() string {
 func (a *SyslogServers) Process(k store.K8s, annotations ...map[string]string) error {
 	input := common.GetValue(a.GetName(), annotations...)
 	a.stdout = false
-	for _, syslogLine := range strings.Split(input, "\n") {
+	for i, syslogLine := range strings.Split(input, "\n") {
 		if syslogLine == "" {
 			continue
 		}
@@ -56,7 +56,7 @@ func (a *SyslogServers) Process(k store.K8s, annotations ...map[string]string) e
 			}
 		}
 		// populate annotation data
-		logTarget := models.LogTarget{Index: utils.PtrInt64(0)}
+		logTarget := models.LogTarget{Index: utils.PtrInt64(int64(i))}
 		address, ok := logParams["address"]
 		if !ok {
 			return fmt.Errorf("incorrect syslog Line: no address param in '%s'", syslogLine)