Skip to content
This repository was archived by the owner on Feb 20, 2020. It is now read-only.

Commit 56e23d0

Browse files
committed
Merge branch 'master' of https://github.com/taskcluster/generic-worker into bug1534223
2 parents 64c162f + f444c25 commit 56e23d0

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

README.md

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -459,17 +459,15 @@ In v14.1.0 since v14.0.2
459459

460460
To enable websocktunnel, include configuration options `wstAudience` and `wstServerURL` to correspond to the websocktunnel server the worker should use.
461461
For the current production environment that would be:
462-
462+
463463
```json
464464
{
465-
...
466465
"wstAudience": "taskcluster-net",
467-
"wstServerURL": "https://websocktunnel.tasks.build",
468-
...
466+
"wstServerURL": "https://websocktunnel.tasks.build"
469467
}
470468
```
471-
472-
Omit the configuration items `livelogCertificate`, `livelogKey`, and `livelogSecret`.
469+
470+
Omit the configuration items `subdomain`, `livelogGETPort`, `livelogCertificate`, `livelogKey`, and `livelogSecret`.
473471
These options will be removed in a future release.
474472

475473
Ensure that the worker's credentials have scope `auth:websocktunnel-token:<wstAudience>/<workerGroup>.<workerId>.*`.

main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -916,7 +916,7 @@ func (task *TaskRun) logHeader() {
916916
if err != nil {
917917
panic(err)
918918
}
919-
task.Info("Worker Type (" + config.WorkerType + ") settings:")
919+
task.Infof("Worker Type (%v/%v) settings:", config.ProvisionerID, config.WorkerType)
920920
task.Info(" " + string(jsonBytes))
921921
task.Info("Task ID: " + task.TaskID)
922922
task.Info("=== Task Starting ===")

plat_all-unix-style.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,9 @@ func purgeOldTasks() error {
8585
log.Printf("WARNING: Not purging previous task directories/users since config setting cleanUpTaskDirs is false")
8686
return nil
8787
}
88-
return deleteTaskDirs(config.TasksDir, taskContext.User.Name)
88+
// Use filepath.Base(taskContext.TaskDir) rather than taskContext.User.Name
89+
// since taskContext.User is nil if running tasks as current user.
90+
return deleteTaskDirs(config.TasksDir, filepath.Base(taskContext.TaskDir))
8991
}
9092

9193
func install(arguments map[string]interface{}) (err error) {

0 commit comments

Comments
 (0)