Skip to content

Commit

Permalink
fix(HMS-3731): workaround to fix type for cloudwatch
Browse files Browse the repository at this point in the history
Temporary workaround that fill type with "cloudwatch"
string when we fill the cloudwatch configuration. Once
the referenced PR is merged, we can remove

See: RedHatInsights/clowder#627

Signed-off-by: Alejandro Visiedo <[email protected]>
  • Loading branch information
avisiedo committed Mar 20, 2024
1 parent 5d403ae commit 2a6ea02
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions internal/config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -290,6 +290,11 @@ func setClowderConfiguration(v *viper.Viper, clowderConfig *clowder.AppConfig) {
v.Set("logging.cloudwatch.group", clowderConfig.Logging.Cloudwatch.LogGroup)
v.Set("logging.cloudwatch.region", clowderConfig.Logging.Cloudwatch.Region)
v.Set("logging.cloudwatch.secret", clowderConfig.Logging.Cloudwatch.SecretAccessKey)
// TODO Delete the block below when the below PR is merged
// See: https://github.com/RedHatInsights/clowder/pull/627
if clowderConfig.Logging.Type == "" {
v.Set("logging.type", "cloudwatch")
}
}

// Metrics configuration
Expand Down

0 comments on commit 2a6ea02

Please sign in to comment.