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 <avisiedo@redhat.com>
  • Loading branch information
avisiedo committed Apr 2, 2024
1 parent 3aeb94e commit ae5a792
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 == "" && len(clowderConfig.Logging.Cloudwatch.SecretAccessKey) > 0 {
v.Set("logging.type", "cloudwatch")
}
}

// Metrics configuration
Expand Down

0 comments on commit ae5a792

Please sign in to comment.