Closed
Bug 1334484
Opened 7 years ago
Closed 6 years ago
[taskcluster-worker] Set up integration tests as part of CI
Categories
(Taskcluster :: Workers, defect)
Taskcluster
Workers
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pmoore, Unassigned)
References
Details
As much as possible, we want this to run as close to production-like as possible. We should have a top level integrationtest package in taskcluster-worker that contains subpackages of all the tests. These should test against the production queue. The integration tests should run in taskcluster, on workers that run tasks as an Administrator on Mac (and later Windows), so that the user creation process, and running tasks under a newly spawned user, is flexed. We should look to parallelise testing the subpackages as much as possible, to keep CI turnaround times short. Note, parallelising tests inside a `go test` execution may also be to some extent possible, although I believe on Mac we are not planning on having capacity > 1 in production, so maybe this is not helpful/feasible.
Reporter | ||
Comment 1•7 years ago
|
||
We have taskcluster-worker unit tests now running in TaskCluster in this PR: * https://github.com/taskcluster/taskcluster-worker/pull/155 See for example: * https://tools.taskcluster.net/task-group-inspector/#/M6ePoqkYStebD4-E-HYYFA The next step was extending this to include end-to-end integration tests, as a separate "integrationtest" subpackage. However, this required enabling the worker to start up, run a fixed number of tasks, and then shutdown, which has been done in this PR: * https://github.com/taskcluster/taskcluster-worker/pull/164 After that I made a small fix here: * https://github.com/taskcluster/taskcluster-worker/pull/166 And I'm currently creating the integration tests (and porting them over from generic-worker), which will also be pushed to PR 155 when running successfully locally.
Reporter | ||
Comment 2•6 years ago
|
||
Are you happy with the integration testing we have in taskcluster-worker now?
Flags: needinfo?(jopsen)
QA Contact: pmoore
Comment 3•6 years ago
|
||
Yeah, they are quite solid :)
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(jopsen)
Resolution: --- → FIXED
Assignee | ||
Updated•5 years ago
|
Component: Worker → Workers
You need to log in
before you can comment on or make changes to this bug.
Description
•