Skip to content

Commit

Permalink
Rename to use DeploymentID
Browse files Browse the repository at this point in the history
  • Loading branch information
norkans7 committed Dec 13, 2024
1 parent 775ee9f commit c59d04f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions cmd/rp-indexer/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,8 @@ func main() {
logger.Error("unable to connect to database")
}

if rt.Config.Deployment != "dev" {
rt.CW, err = cwatch.NewService(rt.Config.AWSAccessKeyID, rt.Config.AWSSecretAccessKey, rt.Config.AWSRegion, rt.Config.CloudwatchNamespace, rt.Config.Deployment)
if rt.Config.DeploymentID != "dev" {
rt.CW, err = cwatch.NewService(rt.Config.AWSAccessKeyID, rt.Config.AWSSecretAccessKey, rt.Config.AWSRegion, rt.Config.CloudwatchNamespace, rt.Config.DeploymentID)

Check warning on line 79 in cmd/rp-indexer/main.go

View check run for this annotation

Codecov / codecov/patch

cmd/rp-indexer/main.go#L78-L79

Added lines #L78 - L79 were not covered by tests
if err != nil {
logger.Error("unable to create cloudwatch service")
}
Expand Down
4 changes: 2 additions & 2 deletions runtime/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ type Config struct {
AWSRegion string `help:"region to use for AWS services, e.g. us-east-1"`

CloudwatchNamespace string `help:"the namespace to use for cloudwatch metrics"`
Deployment string `help:"the deployment identifier to use for metrics"`
DeploymentID string `help:"the deployment identifier to use for metrics"`

LibratoUsername string `help:"the username that will be used to authenticate to Librato"`
LibratoToken string `help:"the token that will be used to authenticate to Librato"`
Expand All @@ -43,7 +43,7 @@ func NewDefaultConfig() *Config {
AWSRegion: "us-east-1",

CloudwatchNamespace: "Temba",
Deployment: "dev",
DeploymentID: "dev",

InstanceName: hostname,

Expand Down

0 comments on commit c59d04f

Please sign in to comment.