Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move TMDS task protection types to ecs-agent #3764

Merged
merged 2 commits into from
Jun 29, 2023

Conversation

amogh09
Copy link
Contributor

@amogh09 amogh09 commented Jun 26, 2023

Summary

Moving types related to TMDS task protection endpoint to ecs-agent module. Task protection endpoint handlers will be moved to ecs-agent module in an upcoming PR.

Implementation details

  • Move agent/handlers/agentapi/taskprotection/v1/types/types.go to ecs-agent/tmds/handlers/taskprotection/v1/types/types.go.
  • Move TaskProtectionClientFactoryInterface from agent/handlers/agentapi/taskprotection/v1/handlers/handlers.go to ecs-agent/tmds/handlers/taskprotection/v1/handlers/interface.go.
  • Move TaskProtectionClientFactory that implements TaskProtectionClientFactoryInterface from agent/handlers/agentapi/taskprotection/v1/handlers/handlers.goto a new file agent/handlers/agentapi/taskprotection/factory.go. Similarly, move its test TestGetECSClientHappyCase from agent/handlers/agentapi/taskprotection/v1/handlers/handlers_test.go to agent/handlers/agentapi/taskprotection/factory_test.go.
  • Update imports.

Testing

Automated tests only.

New tests cover the changes: NA

Description for the changelog

Move TMDS task protection types to ecs-agent (refactoring only).

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@amogh09 amogh09 changed the title Move task protection types Move task protection types to ecs-agent Jun 26, 2023
@amogh09 amogh09 force-pushed the move-task-protection-types branch from 1568feb to db1d52c Compare June 28, 2023 20:33
@amogh09 amogh09 changed the title Move task protection types to ecs-agent Move TMDS task protection types to ecs-agent Jun 28, 2023
@amogh09 amogh09 force-pushed the move-task-protection-types branch from 58635cc to 58318ea Compare June 28, 2023 21:20
@amogh09 amogh09 marked this pull request as ready for review June 28, 2023 21:20
@amogh09 amogh09 requested a review from a team as a code owner June 28, 2023 21:20
@amogh09 amogh09 merged commit 2499162 into aws:dev Jun 29, 2023
6 checks passed
xxx0624 pushed a commit to xxx0624/amazon-ecs-agent that referenced this pull request Jun 30, 2023
@mye956 mye956 mentioned this pull request Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants