Remove or document the backendStatus() method

RESOLVED FIXED

Status

Product:
Taskcluster
Component:
Services
Type:
defect
Status:
RESOLVED FIXED
Reported:
3 years ago
Modified:
3 months ago

People

(Reporter: dustin, Assigned: jhford)

Tracking

Details

Attachments

(1 attachment)

patch
54 bytes, text/x-github-pull-request
Details | Review
Reporter

Description

•
3 years ago 
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
 Reporter

Comment 3

•
3 years ago 
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 → ---
 Assignee

Comment 4

•
3 years ago
Posted file patch —
Attachment #8741428 - Flags: review?(jopsen)

Comment 5

•
3 years ago 
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 ago → a 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.