Skip to content

Commit

Permalink
temporary fix for deb dependency
Browse files Browse the repository at this point in the history
  • Loading branch information
Realmonia committed Aug 27, 2021
1 parent d4d1ae2 commit ac0520c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
12 changes: 6 additions & 6 deletions agent/engine/task_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -1454,12 +1454,6 @@ func (mtask *managedTask) cleanupTask(taskStoppedDuration time.Duration) {
return
}

// Remove TaskExecutionCredentials from credentialsManager
if taskExecutionCredentialsID != "" {
logger.Info("Cleaning up task's execution credentials")
mtask.credentialsManager.RemoveCredentials(taskExecutionCredentialsID)
}

logger.Info("Cleaning up task's containers and data", logger.Fields{
field.TaskARN: mtask.Arn,
})
Expand All @@ -1471,6 +1465,12 @@ func (mtask *managedTask) cleanupTask(taskStoppedDuration time.Duration) {
mtask.engine.sweepTask(mtask.Task)
mtask.engine.deleteTask(mtask.Task)

// Remove TaskExecutionCredentials from credentialsManager
if taskExecutionCredentialsID != "" {
logger.Info("Cleaning up task's execution credentials")
mtask.credentialsManager.RemoveCredentials(taskExecutionCredentialsID)
}

// The last thing to do here is to cancel the context, which should cancel
// all outstanding go routines associated with this managed task.
mtask.cancel()
Expand Down
2 changes: 1 addition & 1 deletion misc/certs/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM debian:stable-20201012
FROM debian:stable-20210816

RUN apt-get update && \
apt-get install -y ca-certificates && \
Expand Down
2 changes: 1 addition & 1 deletion scripts/ci-ecr
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ golang:$GO_VERSION
golang:$GO_VERSION_WINDOWS
gcc:7.3.0
busybox:1.32.0
debian:stable-20201012
debian:stable-20210816
fluent/fluentd:v1.11.5-1.0"

0 comments on commit ac0520c

Please sign in to comment.