Closed
Bug 1447265
Opened 7 years ago
Closed 7 years ago
Use go 1.10 os/exec package for running processes with CreateProcessAsUser on Windows
Categories
(Taskcluster :: Workers, enhancement)
Taskcluster
Workers
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pmoore, Unassigned)
References
Details
Currently we use github.com/taskcluster/runlib under the hood to manage executing processes on Windows using CreateProcessAsUser syscall.
From go 1.10 onwards, it is now possible to do this via the go standard library.
This can be done by setting (syscall.SysProcAttr).Token inside (os/exec.Cmd).SysProcAttr.
At the same time it may make sense to start using https://golang.org/pkg/os/exec/#CommandContext and then cancel the context in order to terminate the command.
Reporter | ||
Comment 1•7 years ago
|
||
(In reply to Pete Moore [:pmoore][:pete] from comment #0)
> At the same time it may make sense to start using
> https://golang.org/pkg/os/exec/#CommandContext and then cancel the context
> in order to terminate the command.
(when aborting a task)
Comment 2•7 years ago
|
||
Commits pushed to master at https://github.com/taskcluster/generic-worker
https://github.com/taskcluster/generic-worker/commit/b277fa8a9409cd44206b90801646972ff35357bc
Bug 1447265 - use os/exec package on Windows too
https://github.com/taskcluster/generic-worker/commit/9529f9818ff1c7ace1fb427fe1fb2270d0f5fe5e
Merge pull request #80 from taskcluster/bug1447265
Bug 1447265 - use os/exec package on Windows too
Reporter | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 3•7 years ago
|
||
Released in generic-worker 10.7.7.
Assignee | ||
Updated•6 years ago
|
Component: Generic-Worker → Workers
You need to log in
before you can comment on or make changes to this bug.
Description
•