-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Windows] Unpin Postgres allowing latest v14 version #11047
base: main
Are you sure you want to change the base?
[Windows] Unpin Postgres allowing latest v14 version #11047
Conversation
removed these by accident, postgres releases are signed with same key so keeping this allows us to validate it
Hitting this when running the CI. Having a look at debugging now.
|
With a local Windows box I can reproduce this error. I get:
Searching this maps to the issue seen here Running Testing locally, with that param removed the installation goes ahead I think this worked with previous versions as they used As that that provisioner is invoked before Now that postgres needs MSVC 17 that is no longer the case. I've removed the |
Newer postgres installes require MSVC 17 which isn't on the box by default. Previously this worked as they required MSVC 16 which was installed by Visual Studio actions#11047 (comment)
Install-Binary ` | ||
-Url $installerUrl ` | ||
-InstallArgs $installerArgs ` | ||
-ExpectedSignature (Get-ToolsetContent).postgresql.signature | ||
-ExpectedSignature (Get-ToolsetContent).postgresql.signature ` | ||
-InstallerLogPath "$env:TEMP\**\install-postgresql.log" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using this I was able to find the issue, the installer errors with:
azure-arm.image: The database cluster will be initialized with locale "English_United States.1252".
azure-arm.image: The default text search configuration will be set to "english".
azure-arm.image:
azure-arm.image: Data page checksums are disabled.
azure-arm.image: initdb: error: could not open file "C:\Users\packer\AppData\Local\Temp/postgresql_installer_b876155f3b\rad3B194.tmp" for reading: Permission denied
azure-arm.image:
azure-arm.image: Called Die(Failed to initialise the database cluster with initdb)...
azure-arm.image: Failed to initialise the database cluster with initdb
azure-arm.image:
azure-arm.image: Script stderr:
azure-arm.image: Program ended with an error exit code
azure-arm.image:
azure-arm.image: Error running C:\Windows\System32\cscript //NoLogo "C:\Program Files\PostgreSQL\14/installer/server/initcluster.vbs" "NT AUTHORITY\NetworkService" "postgres" "****" "C:\Users\packer\AppData\Local\Temp/postgresql_installer_b876155f3b" "C:\Program Files\PostgreSQL\14" "C:\Program Files\PostgreSQL\14\data" 5432 "DEFAULT" 1: Program ended with an error exit code
https://github.com/actions/runner-images-ci/actions/runs/12159209946/job/33908835054#step:8:51246
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like others are hitting this issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EBD have a test installer which may resolve, looks like it's being actively worked on at least 🤞
e08af0a
to
9f19530
Compare
Description
New tool, Bug fixing, or Improvement?
Please include a summary of the change and which issue is fixed. Also include relevant motivation and context.
For new tools, please provide total size and installation time.
Related issue:
Check list