Merge pull request #3 from 1Password/ignore_restart_annotation

Ignore restart annotation when looking for 1Password annotations
This commit is contained in:
Jillian W
2021-01-13 17:24:29 -04:00
committed by GitHub
2 changed files with 2 additions and 2 deletions

View File

@@ -99,7 +99,7 @@ func (r *ReconcileDeployment) Reconcile(request reconcile.Request) (reconcile.Re
annotations, annotationsFound := op.GetAnnotationsForDeployment(deployment, r.opAnnotationRegExp)
if !annotationsFound {
reqLogger.Info("No One Password Annotations found")
reqLogger.Info("No 1Password Annotations found")
return reconcile.Result{}, nil
}

View File

@@ -34,7 +34,7 @@ func GetAnnotationsForDeployment(deployment *appsv1.Deployment, regex *regexp.Re
func FilterAnnotations(annotations map[string]string, regex *regexp.Regexp) map[string]string {
filteredAnnotations := make(map[string]string)
for key, value := range annotations {
if regex.MatchString(key) {
if regex.MatchString(key) && key != RestartAnnotation {
filteredAnnotations[key] = value
}
}