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.

Sign up for GitHub

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

Jump to bottom

Feature/dynamic host port assignment #3599

Closed
wants to merge 20 commits into from
Closed

Feature/dynamic host port assignment #3599

wants to merge 20 commits into from

Conversation

chienhanlin
Copy link
Contributor

@chienhanlin chienhanlin commented Mar 2, 2023 •
edited
Loading

Summary

This PR is mainly to rebase the feature branch against the dev branch.

Commits/PRs will be merged into the feature branch include

  1. Remove fallback to Docker for host port ranges assignment #3569
  2. fix ecs-init logging #3577
  3. Remove set-output GitHub action command #3487
  4. Update CNI plugin versions #3581
  5. periodically disconnect from acs #3586
  6. Release 1.69.0 #3591
  7. fix TestMetricsDisabled unit test #3588
  8. make TestGetHostPortRange unit test deterministic #3587
  9. update go module dependencies #3593
  10. Update GitHub Actions checkout package in CI #3595

Implementation details

Merge the dev branch into the feature branch, and resolve conflicts.

Testing

Ran make release-agent to build ECS Agent, and the build was created successfully.

New tests cover the changes: no

Description for the changelog

N/A

Licensing

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

chienhanlin and others added 20 commits February 14, 2023 13:16
@chienhanlin
Remove fallback to Docker for host port ranges assignment
7df78ff
@konoui @singholt
fix ecs-init logging
31116db
@austinvazquez @yinyic
Remove set-output GitHub action command
2174b0a 
Removes deprecated set-output GitHub action command.

Signed-off-by: Austin Vazquez <[email protected]>
@yinyic
Update CNI plugin versions
7d2b6da
@singholt
periodically disconnect from acs
6075bec
@singholt
send pending acks before closing ACS connection
0fe21da
@YashdalfTheGray
Release 1.69.0
b32ab07
@YashdalfTheGray
Merge pull request aws#3591 from YashdalfTheGray/v1.69.0-stage
8c65fde 
Release 1.69.0
@singholt
fix TestMetricsDisabled
7600ecf
@singholt
make TestGetHostPortRange unit test deterministic
cea7a56
@singholt
agent: update golang.org/x/net from v0.5.0 -> v0.7.0
095b117
@singholt
agent: update github.com/containerd/containerd from v1.4.13 -> v1.5.18
302d9b8
@singholt
init: update github.com/containerd/containerd from v1.6.8 -> v.16.18 …
af237d2 
…and go version in go.mod
@singholt
agent: update github.com/prometheus/client_golang from v1.7.1 -> v1.11.1
d1d92f9
@singholt
agent: update github.com/gorilla/websocket from v1.4.2 -> v1.5.0
ddf4702
@austinvazquez @danehlim
Update GitHub Actions checkout package in CI
adb7799 
Moves actions/checkout from v2 to v3 to resolve NodeJS 12 deprecation
warnings in CI workflows.

Signed-off-by: Austin Vazquez <[email protected]>
@chienhanlin
Add GetHostPort() and update unit tests
d59ac07
@chienhanlin
Upudate dockerPortMap() in task.go with dynamic host port range suppo…
9f33ae4 
…rt part 1
@chienhanlin
Upudate dockerPortMap() in task.go with dynamic host port range suppo…
643c11e 
…rt part 2
@chienhanlin
Validate the host port/host port range found by ECS Agent before retu…
8c66e5b 
…rning it
@chienhanlin chienhanlin closed this Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@chienhanlin @konoui @austinvazquez @yinyic @singholt @YashdalfTheGray