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

Please call out .exe filename changes in changelogs #253

Open
SimonSapin opened this issue Nov 3, 2019 · 2 comments
Open

Please call out .exe filename changes in changelogs #253

SimonSapin opened this issue Nov 3, 2019 · 2 comments
Assignees

Comments

@SimonSapin
Copy link

For Servo CI, we install generic-worker in a Windows AMI for EC2. This works by having a PowerShell script that runs at first boot. However this script runs non-interactively, and its output is not easy to get at.

Twice now I’ve bumped the version number in the script like this:

--- windows/first-boot.ps1
+++ windows/first-boot.ps1
@@ -36,7 +36,7 @@ New-NetFirewallRule -DisplayName "Allow livelog GET requests" `
 # Install generic-worker and dependencies
 md C:\generic-worker
 $client.DownloadFile("https://github.com/taskcluster/generic-worker/releases/download" +
-    "/v14.1.0/generic-worker-nativeEngine-windows-amd64.exe", "C:\generic-worker\generic-worker.exe")
+    "/v16.5.2/generic-worker-nativeEngine-windows-amd64.exe", "C:\generic-worker\generic-worker.exe")
 $client.DownloadFile("https://github.com/taskcluster/livelog/releases/download" +
     "/v1.1.0/livelog-windows-amd64.exe", "C:\generic-worker\livelog.exe")
 $client.DownloadFile("https://github.com/taskcluster/taskcluster-proxy/releases/download" +

Only to realize after after lengthy debugging that the filename has changed to generic-worker-multiuser-windows-amd64.exe, so the download fails with a 404 error.

When doing such upgrades I do read the change logs at https://github.com/taskcluster/generic-worker/releases. It would be nice if future such filename changes were called out in the change logs. Thanks!

@petemoore petemoore self-assigned this Nov 13, 2019
@petemoore
Copy link
Member

Sorry about this Simon! This is a very fair point, apologies that got missed. I'll update the historic release notes and make sure not to make the same mistake again.

@SimonSapin
Copy link
Author

I wouldn’t really call it a mistake, having this kind of reminder is nice to have :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants