From bfff7104aa513d6f76b56b9894ee3de884d73abc Mon Sep 17 00:00:00 2001 From: irti Date: Thu, 25 Apr 2019 14:11:31 +0500 Subject: [PATCH] [fix-config-change-logs-issue] logs issue fixed --- internal/pkg/handler/update.go | 3 +-- internal/pkg/handler/upgrade.go | 3 ++- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/internal/pkg/handler/update.go b/internal/pkg/handler/update.go index 5a51ad4..543f0ae 100644 --- a/internal/pkg/handler/update.go +++ b/internal/pkg/handler/update.go @@ -3,7 +3,7 @@ package handler import ( "github.com/sirupsen/logrus" "github.com/stakater/Reloader/internal/pkg/util" - "k8s.io/api/core/v1" + "k8s.io/api/core/v1" ) // ResourceUpdatedHandler contains updated objects @@ -19,7 +19,6 @@ func (r ResourceUpdatedHandler) Handle() error { } else { config, oldSHAData := r.GetConfig() if config.SHAValue != oldSHAData { - logrus.Infof("Changes detected in '%s' of type '%s' in namespace '%s'", config.ResourceName, config.Type, config.Namespace) // process resource based on its type doRollingUpgrade(config) } diff --git a/internal/pkg/handler/upgrade.go b/internal/pkg/handler/upgrade.go index 8c3fb9f..2333f5d 100644 --- a/internal/pkg/handler/upgrade.go +++ b/internal/pkg/handler/upgrade.go @@ -98,6 +98,7 @@ func PerformRollingUpgrade(client kubernetes.Interface, config util.Config, upgr logrus.Errorf("Update for '%s' of type '%s' in namespace '%s' failed with error %v", resourceName, upgradeFuncs.ResourceType, config.Namespace, err) } else { logrus.Infof("Updated '%s' of type '%s' in namespace '%s'", resourceName, upgradeFuncs.ResourceType, config.Namespace) + logrus.Infof("Changes detected in '%s' of type '%s' in namespace '%s'", config.ResourceName, config.Type, config.Namespace) } } } @@ -177,7 +178,7 @@ func getContainerToUpdate(upgradeFuncs callbacks.RollingUpgradeFuncs, item inter // Get the container with referenced secret or configmap as env var container = getContainerWithEnvReference(containers, config.ResourceName, config.Type) - if container == nil && len(initContainers) > 0 { + if container == nil && len(initContainers) > 0 { container = getContainerWithEnvReference(initContainers, config.ResourceName, config.Type) if container != nil { // if configmap/secret is being used in init container then return the first Pod container to save reloader env