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: Render Taskless Services #2085

Open
2opremio opened this issue Dec 12, 2016 · 2 comments
Open

ECS: Render Taskless Services #2085

2opremio opened this issue Dec 12, 2016 · 2 comments
Labels
bug Broken end user or developer functionality; not working as the developers intended it ecs Pertains to integration with Amazon Elastic Container Service

Comments

@2opremio
Copy link
Contributor

2opremio commented Dec 12, 2016

Right now (after #2026) we only "discover" ECS services through tasks (we obtain the task list from Docker labels and obtain services associated to those tasks, the process is well documented in #2065).

This means we don't obtain information nor render services without associated tasks (e.g. Services with replication factor 0, which is valid, and services whose tasks are not running).

This may or may not be a real problem for users.

@2opremio 2opremio added the bug Broken end user or developer functionality; not working as the developers intended it label Dec 12, 2016
@2opremio 2opremio added the ecs Pertains to integration with Amazon Elastic Container Service label Dec 12, 2016
@2opremio
Copy link
Contributor Author

I think that a regular call to ListServices wouldn't add a lot of overhead considering we regularly call DescribeService for every service.

@ekimekim
Copy link
Contributor

Well, as of #2065 we only describe services that have tasks running on our machine. I like this because it means the number of calls is bounded by the number of things running per machine, whereas describing every service on every machine would be problematic in very large clusters.
Sigh. These rate limits make everything difficult.

Given the current constraints we may need to accept not supporting showing non-running services for now.
Ideally we should document that somewhere.

ekimekim added a commit that referenced this issue Feb 16, 2017
as currently this would make it disappear (#2085).
See also #2197 (comment)
ekimekim added a commit that referenced this issue Feb 17, 2017
as currently this would make it disappear (#2085).
See also #2197 (comment)
@ekimekim ekimekim removed their assignment Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Broken end user or developer functionality; not working as the developers intended it ecs Pertains to integration with Amazon Elastic Container Service
Projects
None yet
Development

No branches or pull requests

2 participants