Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
  • Loading branch information
abicky and coderabbitai[bot] committed Oct 28, 2024
1 parent c5fb1bc commit f04c95d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion cmd/recreateservice.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ func recreateService(cmd *cobra.Command, args []string) error {

cfg, err := newConfig(cmd.Context())
if err != nil {
return newRuntimeError("failed to initialize a session: %w", err)
return newRuntimeError("failed to initialize configuration: %w", err)
}

if err := service.NewService(ecs.NewFromConfig(cfg)).Recreate(cmd.Context(), cluster, serviceName, overrideDef); err != nil {
Expand Down
4 changes: 2 additions & 2 deletions internal/service/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ func (s *Service) Recreate(ctx context.Context, cluster string, serviceName stri
func (s *Service) copy(ctx context.Context, cluster string, serviceName string, overrides Definition) error {
resp, err := s.ecsSvc.DescribeServices(ctx, &ecs.DescribeServicesInput{
Cluster: aws.String(cluster),
Include: []ecstypes.ServiceField{"TAGS"},
Include: []ecstypes.ServiceField{ecstypes.ServiceFieldTags},
Services: []string{serviceName},
})
if err != nil {
Expand Down Expand Up @@ -122,7 +122,7 @@ func (s *Service) stopAndWaitUntilStopped(ctx context.Context, cluster string, s
taskArns := make([]string, 0)
params := &ecs.ListTasksInput{
Cluster: aws.String(cluster),
DesiredStatus: "RUNNING",
DesiredStatus: ecstypes.DesiredStatusRunning,
ServiceName: aws.String(serviceName),
}

Expand Down

0 comments on commit f04c95d

Please sign in to comment.