Skip to content

Commit

Permalink
Fix config parser for appendOnly (#111)
Browse files Browse the repository at this point in the history
The struct needed an explicit `yaml:"appendOnly"` as camel case is not
the default.

Signed-off-by: Galo Navarro <anglorvaroa@gmail.com>
  • Loading branch information
srvaroa authored Nov 11, 2023
1 parent 4889f18 commit 632f5f8
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion pkg/labeler.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ type LabelerConfigV1 struct {
// When set to true, we will only add labels when they match a rule
// but it will NOT remove labels that were previously set and stop
// matching a rule
AppendOnly bool
AppendOnly bool `yaml:"appendOnly"`
Labels []LabelMatcher
}

Expand Down Expand Up @@ -169,6 +169,11 @@ func (l *Labeler) ExecuteOn(target *Target) error {
intentions[label] = true
}

log.Printf("Current labels: `%v`", intentions)
if config.AppendOnly {
log.Printf("AppendOnly is active, removals are forbidden")
}
log.Printf("Preliminary label updates: `%v`", labelUpdates)
// update, adding new ones and unflagging those to remove if
// necessary
for label, isDesired := range labelUpdates.set {
Expand Down

0 comments on commit 632f5f8

Please sign in to comment.