Skip to content

Commit

Permalink
Ecs provider fix (#2056)
Browse files Browse the repository at this point in the history
* Include cluster name in describe tasks call

* Fix poll interval to be based on seconds
  • Loading branch information
dariussantaras authored Sep 22, 2023
1 parent 39a8044 commit 48b2218
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/Proto.Cluster.AmazonECS/AmazonEcsProvider.cs
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ private void StartClusterMonitor() =>
Logger.LogError(x, "Failed to get members from ECS");
}

await Task.Delay(_config.PollIntervalSeconds).ConfigureAwait(false);
await Task.Delay(TimeSpan.FromSeconds(_config.PollIntervalSeconds)).ConfigureAwait(false);
}
}
);
Expand Down
1 change: 1 addition & 0 deletions src/Proto.Cluster.AmazonECS/EcsUtils.cs
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ public static async Task<Member[]> GetMembers(this AmazonECSClient c, string ecs
var describedTasks = await c.DescribeTasksAsync(new DescribeTasksRequest
{
Include = { "TAGS" },
Cluster = ecsClusterName,
Tasks = instanceArns
}
).ConfigureAwait(false);
Expand Down

0 comments on commit 48b2218

Please sign in to comment.