Closed Bug 1453649 Opened 7 years ago Closed 6 years ago

Missing required properties in /v1/state/<worker-type>

Categories

(Taskcluster :: Services, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: hassan, Assigned: jhford)

References

Details

Attachments

(4 files)

There may be a bug with the /v1/state/<worker-type GET endpoint. It is not returning the `type` and `ami` property inside the `instances` property: https://github.com/taskcluster/aws-provisioner/blob/cb1aed6de14b8617400d9f03810f98b2a3963c6c/schemas/state-definition.json#L28-L37 As a result, because the `type` is not returned, the worker-type status tab shows 0's in the UI for the running and pending capacity columns (see screenshot).
John, any idea why this is happening?
Flags: needinfo?(jhford)
(In reply to Hassan Ali (:hassan) from comment #2) > John, any idea why this is happening? I'm not sure why. The EC2-Manager endpoints are returning this information, but the provisioner is not. I'm not entirely sure why, but the problem is in the /state methods of the provisioner.
Flags: needinfo?(jhford)
Can we get this prioritized, please? During outages (like today), it's hard to pinpoint bad regions/AZs without this data.
Attached image resources tab
The instance type and AMI columns in the EC2 Resources tab are also broken in production.
Attached patch patchSplinter Review
This was broken when case started to be preserved in the ec2-manager database.
Assignee: nobody → jhford
Attachment #9009939 - Flags: review?(dustin)
Attached file GitHub Pull Request
This also adds much more detail to the state schema so that we can catch something like this in the future.
Attachment #9009943 - Flags: review?(dustin)
Attachment #9009939 - Flags: review?(dustin) → review+
Attachment #9009943 - Flags: review?(dustin) → review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Component: AWS-Provisioner → Services
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: