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

ecs-settings-applier: remove bottlerocket.version attribute #1395

Merged

Conversation

jpculp
Copy link
Member

@jpculp jpculp commented Mar 16, 2021

Issue number:

#1292

Description of changes:

This commit will remove the somewhat deceptive bottlerocket.version attribute. The attributes described in ecs.config.json are set when the instance first joins an ECS cluster, but are not updated in subsequent runs of the ECS agent. This leads to updated Bottlerocket instances continuing to show the lower version they were at first initialization within the ECS API.

Testing done:

  • Built aws-ecs-1 ami and launched instance.
  • Instance connected to ecs cluster.
  • Test task deployed successfully.
  • Connected to control container via ssm session.
  • Enabled and launched admin container.
  • Connected to admin container via ssh.
  • Ran sudo sheltie to verify root shell was still available.
  • Checked for failed systemd units.
  • Verified that /etc/ecs/ecs.config.json did not contain bottlerocket.version
  • Used aws ecs describe-container-instances to verify the bottlerocket.version attribute was absent on the ECS side.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

This commit will remove the somewhat deceptive bottlerocket.version
attribute. The attributes described in ecs.config.json are set when the
instance first joins an ECS cluster, but are not updated in subsequent
runs of the ECS agent. This leads to updated Bottlerocket instances
continuing to show the lower version they were at first initialization
within the ECS API.
@jpculp jpculp requested a review from samuelkarp March 16, 2021 00:00
@jpculp jpculp merged commit c66251e into bottlerocket-os:develop Mar 16, 2021
@jpculp jpculp deleted the remove-bottlerocket-version-attribute branch March 16, 2021 16:29
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