Remove or document the backendStatus() method

RESOLVED FIXED

Status

defect
RESOLVED FIXED
3 years ago
3 months ago

People

(Reporter: dustin, Assigned: jhford)

Tracking

Details

Attachments

(1 attachment)

This method has no documentation -- I don't know if it does anything useful, or even whether it's used.

It should be documented or deleted.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1251911
Duplicate of bug: 1261911
Per John:

Please do not delete this. It is supposed to be used to get information into status.taskcluster.net. If that's not where the data for the aws-provisioner graph is coming from, then the graph is measuring the wrong thing.

----

So this should be documented, including response schema, and added to https://tools.taskcluster.net/status/
Assignee: nobody → jhford
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Posted file patch
Attachment #8741428 - Flags: review?(jopsen)
Comment on attachment 8741428 [details] [review]
patch

Feedback already on the PR:
https://github.com/taskcluster/aws-provisioner/pull/69

I guess it's not critical, but for consistency I suggest naming of the `checked_in_at` is changed, and that "format: date-time" is used to enforce formatting of timestamps.
Attachment #8741428 - Flags: review?(jopsen)
Status: REOPENED → RESOLVED
Last Resolved: 3 years agoa year ago
Resolution: --- → FIXED
Component: AWS-Provisioner → Services
Product: Taskcluster → Taskcluster
You need to log in before you can comment on or make changes to this bug.