Skip to content

Commit f3401b3

Browse files
committed
Add endpointContainerID to tasks and tasksStats endpoint.
1 parent 5de8ba6 commit f3401b3

File tree

6 files changed

+25
-21
lines changed

6 files changed

+25
-21
lines changed

agent/handlers/v4/tmdsstate.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -249,15 +249,15 @@ func (s *TMDSAgentState) GetTaskStats(v3EndpointID string) (map[string]*tmdsv4.S
249249
return taskStatsResponse, nil
250250
}
251251

252-
func (s *TMDSAgentState) GetTasksMetadata() ([]tmdsv4.TaskResponse, error) {
252+
func (s *TMDSAgentState) GetTasksMetadata(endpointContainerID string) ([]tmdsv4.TaskResponse, error) {
253253
return nil, tmdsv4.NewErrorMetadataFetchFailure("tasks metadata endpoint not supported")
254254
}
255255

256-
func (s *TMDSAgentState) GetTasksMetadataWithTags() ([]tmdsv4.TaskResponse, error) {
256+
func (s *TMDSAgentState) GetTasksMetadataWithTags(endpointContainerID string) ([]tmdsv4.TaskResponse, error) {
257257
return nil, tmdsv4.NewErrorMetadataFetchFailure("tasks metadata with tags endpoint not supported")
258258
}
259259

260-
func (s *TMDSAgentState) GetTasksStats(v3EndpointID string) ([]map[string]*tmdsv4.StatsResponse, error) {
260+
func (s *TMDSAgentState) GetTasksStats(endpointContainerID string) ([]map[string]*tmdsv4.StatsResponse, error) {
261261
return nil, tmdsv4.NewErrorStatsFetchFailure("tasks stats endpoint not supported", nil)
262262
}
263263

agent/vendor/github.com/aws/amazon-ecs-agent/ecs-agent/tmds/handlers/v4/handlers.go

Lines changed: 5 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

agent/vendor/github.com/aws/amazon-ecs-agent/ecs-agent/tmds/handlers/v4/state/state.go

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ecs-agent/tmds/handlers/v4/handlers.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -204,13 +204,14 @@ func tasksMetadataHandler(
204204
includeTags bool,
205205
) func(http.ResponseWriter, *http.Request) {
206206
return func(w http.ResponseWriter, r *http.Request) {
207+
endpointContainerID := mux.Vars(r)[EndpointContainerIDMuxName]
207208
var tasksMetadata []state.TaskResponse
208209
var err error
209210

210211
if includeTags {
211-
tasksMetadata, err = agentState.GetTasksMetadataWithTags()
212+
tasksMetadata, err = agentState.GetTasksMetadataWithTags(endpointContainerID)
212213
} else {
213-
tasksMetadata, err = agentState.GetTasksMetadata()
214+
tasksMetadata, err = agentState.GetTasksMetadata(endpointContainerID)
214215
}
215216

216217
if err != nil {
@@ -274,8 +275,9 @@ func TasksStatsHandler(
274275
metricsFactory metrics.EntryFactory,
275276
) func(http.ResponseWriter, *http.Request) {
276277
return func(w http.ResponseWriter, r *http.Request) {
278+
endpointContainerID := mux.Vars(r)[EndpointContainerIDMuxName]
277279
// Get stats for all tasks
278-
stats, err := agentState.GetTasksStats("")
280+
stats, err := agentState.GetTasksStats(endpointContainerID)
279281
if err != nil {
280282
logger.Error("Failed to get v4 tasks stats", logger.Fields{
281283
field.Error: err,

ecs-agent/tmds/handlers/v4/state/mocks/state_mock.go

Lines changed: 8 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ecs-agent/tmds/handlers/v4/state/state.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,12 +158,12 @@ type AgentState interface {
158158
// Returns metadata for all tasks on the container instance in v4 format.
159159
// Returns ErrorTaskLookupFailed if task lookup fails.
160160
// Returns ErrorMetadataFetchFailure if something else goes wrong.
161-
GetTasksMetadata() ([]TaskResponse, error)
161+
GetTasksMetadata(endpointContainerID string) ([]TaskResponse, error)
162162

163163
// Returns metadata for all tasks on the container instance including task and container instance tags (if applicable) in v4 format.
164164
// Returns ErrorTaskLookupFailed if task lookup fails.
165165
// Returns ErrorMetadataFetchFailure if something else goes wrong.
166-
GetTasksMetadataWithTags() ([]TaskResponse, error)
166+
GetTasksMetadataWithTags(endpointContainerID string) ([]TaskResponse, error)
167167

168168
// Returns task metadata including task and container instance tags (if applicable) in v4 format
169169
// for the task identified by the provided endpointContainerID.

0 commit comments

Comments
 (0)