Closed Bug 1261910 Opened 9 years ago Closed 9 years ago

Switch tools to use new API methods for aws provisioner

Categories

(Taskcluster :: Services, defect)

Product:
Taskcluster
Component:
Services
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: dustin, Assigned: dustin)

References

Details

Attachments

(1 file, 1 obsolete file)

https://github.com/taskcluster/taskcluster-tools/pull/94
56 bytes, text/x-github-pull-request
Details | Review
The awsState() method is deprecated: --- This method is a left over and will be removed as soon as the tools.tc.net UI is updated to use the per-worker state DEPRECATED
Reworking this a little: use the new API methods from bug 1198391 to support anyone viewing the "bar graph" display.
Depends on: 1198391
Summary: Switch tools to use state(<workerType>) and remove the awsState() method → Switch tools to use new API methods for aws provisioner
Commits pushed to master at https://github.com/taskcluster/taskcluster-tools https://github.com/taskcluster/taskcluster-tools/commit/61cfcd308e4be09316b7663606826cf080970b44 Bug 1261910: use listWorkerTypeSummaries() and state() The `listWorkerTypeSummaries()` endpoint does not require authentication, so this means that the site is visible even to those who are not logged in. As added bonuses: * the table renders immediately when `listWorkerTypeSummaries()` returns, much sooner than in older versions; pending counts are filled in as they arrive * the table's summary is updated whenver a workerType's details are loaded. https://github.com/taskcluster/taskcluster-tools/commit/61dd2538904efab7261b2221aa719a1c3e37d229 Merge pull request #94 from djmitche/bug1261910 Bug 1261910: use listWorkerTypeSummaries() and state()
Commit pushed to master at https://github.com/taskcluster/taskcluster-tools https://github.com/taskcluster/taskcluster-tools/commit/908992c0ed8ba0ec2eaeb1b91d75d9950226ccd9 Revert "Merge pull request #94 from djmitche/bug1261910" This reverts commit 61dd2538904efab7261b2221aa719a1c3e37d229, reversing changes made to fde9c37c7bc622d99e90f33812cd90ab443d07e3.
Comment on attachment 8743072 [details] [review] https://github.com/taskcluster/aws-provisioner/pull/70 I merged this into the re-landing in bug 1198391
Attachment #8743072 - Attachment is obsolete: true
Comment on attachment 8743071 [details] [review] https://github.com/taskcluster/taskcluster-tools/pull/94 John -- I think this can be re-landed as-is when the backend changes (bug 1198391) land. Concur?
Attachment #8743071 - Flags: review- → review?(jhford)
Commit pushed to master at https://github.com/taskcluster/taskcluster-tools https://github.com/taskcluster/taskcluster-tools/commit/ea94be89dc8378d1286c2741de194a08e6718e88 Bug 1261910: use listWorkerTypeSummaries() and state() The `listWorkerTypeSummaries()` endpoint does not require authentication, so this means that the site is visible even to those who are not logged in. As added bonuses: * the table renders immediately when `listWorkerTypeSummaries()` returns, much sooner than in older versions; pending counts are filled in as they arrive * the table's summary is updated whenver a workerType's details are loaded.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #8743071 - Flags: review?(jhford)
Component: AWS-Provisioner → Services
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: