Closed Bug 1484739 Opened 6 years ago Closed 6 years ago

Use tc-lib-pulse in tc-web-server

Categories

(Taskcluster :: UI, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dustin, Assigned: dustin)

References

Details

From Eli: @djmitche in 65a68ac[1] I attempted to switch out the PulseListener for the new implementation from taskcluster-lib-pulse, but I couldn't seem to get it to work right, so I think I'm doing something wrong. Is there any chance you could take a look and see what could be wrong here? [1] https://github.com/taskcluster/taskcluster-web-server/commit/65a68ac369938c051ebe05eb29b50f75d3aedf37 --- Hassan, any idea where this is at? And priority? I've not worked with tc-web yet, but I should. I also have about 200 tasks in front of this one in Taskwarrior.. but if this can wait, maybe it's a good way for me to get started?
Flags: needinfo?(helfi92)
This is definitely a good way for you to get started. The priority for this one is low but would be awesome if we can have it done by the end of the quarter. The README should cover everything to get you started and also includes a sample subscription query[1]. Once you have the web server running, you can access the playground at http://localhost:3050/playground. Let me know if there is anything else I can help with. [1] https://github.com/taskcluster/taskcluster-web-server/blob/master/README.md#sample-subscriptions
Flags: needinfo?(helfi92)
Assignee: nobody → dustin
I made a bit of progress on writing a good solution here, but getting it to pass lint is proving difficult, so I haven't been able to run it yet.
Some progress here.. https://github.com/taskcluster/taskcluster-web-server/compare/master...djmitche:bug1484739?expand=1 It seems that the "Pulse Inspector" tool doesn't work at all yet? It doesn't appear to use any GraphQL. Also, the PulseEngine is not providing the routes, which it needs to display. We can likely deal with all of that when it comes time to update that tool.
Depends on: 1488526
Right, the basic UI skeleton for the pulse messages view is there but none of the GraphQL stuff is wired. I can take care of that once the backend aspect of it has been merged.
Depends on: 1489305
Depends on: 1489330
Blocks: 1436456
Blocks: 1490723
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Component: Tools → UI and Tools
You need to log in before you can comment on or make changes to this bug.