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 - Custom Attributes not updated after version update #1293

Closed
umairishaq opened this issue Jan 27, 2021 · 3 comments
Closed

ECS - Custom Attributes not updated after version update #1293

umairishaq opened this issue Jan 27, 2021 · 3 comments
Labels
area/ecs ECS status/needs-triage Pending triage or re-evaluation type/bug Something isn't working

Comments

@umairishaq
Copy link

Image I'm using:
Tried with a couple of versions; 1.0.2 and 1.0.3 with the following names:
bottlerocket-aws-ecs-1-x86_64-v1.0.2-ddeb03c8
bottlerocket-aws-ecs-1-x86_64-v1.0.3-0c93e9a6

What I expected to happen:
After the instance was updated, the ECS custom attribute, bottlerocket.version, should have reflected the new version, 1.0.5 in my case.

What actually happened:
The attribute was not updated.

How to reproduce the problem:

  1. Add Bottlelrocket (1.0.2 instances) to a new or existing ECS cluster.
  2. Validate the custom attribute reflects the version in ECS console.
  3. Drain the instance.
  4. Update the instance using "System Manager - Run Command".
  5. Verify the update was successful in CW Logs.
  6. Reboot the instance
  7. Validate the custom attribute was not updated.
@zmrow
Copy link
Contributor

zmrow commented Jan 27, 2021

Thanks for opening this! We'll look into it.

@jhaynes jhaynes added area/ecs ECS status/needs-triage Pending triage or re-evaluation type/bug Something isn't working labels Jan 27, 2021
@gregdek gregdek added priority/p1 and removed status/needs-triage Pending triage or re-evaluation labels Jan 29, 2021
@jpculp
Copy link
Member

jpculp commented Mar 16, 2021

Thanks @umairishaq for pointing this out. Due to some limitations in the ECS API we've decided to remove this attribute. We'll leave the issue open for now so that we can explore other options at a later date.

@gregdek gregdek added this to the techdebt milestone Apr 1, 2021
@stmcginnis stmcginnis added status/needs-triage Pending triage or re-evaluation and removed priority/p2 labels Dec 1, 2022
@stmcginnis stmcginnis removed this from the techdebt milestone Feb 23, 2023
@stmcginnis
Copy link
Contributor

The origin issue has been addressed (by removing it), and there hasn't been any new ideas for doing things differently. Therefore going to close, but feel free to file a new issue if you have an idea for improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ecs ECS status/needs-triage Pending triage or re-evaluation type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants