Skip to content
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

fix: checksum search for Kotlin #8704

Merged
merged 2 commits into from
Nov 1, 2023

Conversation

frittsy
Copy link
Contributor

@frittsy frittsy commented Nov 1, 2023

The latest release notes for Kotlin v1.9.20 includes multiple lines that contain the search string "kotlin-compiler", one of which is not the line that contains the checksum. This tweaks the search string to only look for the filename ending in .zip, where the checksum should be present.

@frittsy
Copy link
Contributor Author

frittsy commented Nov 1, 2023

There appears to be a similar issue with Windows too, so looking at that as well and may update this PR with the fix for both.

edit: fixed in additional commit

@frittsy frittsy changed the title [Ubuntu] Fix checksum search for Kotlin fix: checksum search for Kotlin Nov 1, 2023
@mikhailkoliada mikhailkoliada merged commit cdbbd8a into actions:main Nov 1, 2023
8 checks passed
@frittsy frittsy deleted the fix/ubuntu-kotlin-checksum branch November 1, 2023 23:29
mikhailkoliada added a commit that referenced this pull request Nov 8, 2023
* Update folder structure

* Adjust packer templates and debug

* Fix issues in ubuntu2204 template

* Remove Debug steps

* Rename template, sync pipelines and adjust resource gen script

* fix post-gen path in ubuntu-minimal

* fix shebang linter, workflow paths and adjust readme

* update preimagedata

* [MacOS] Add xcode 15.0.1 (#8604)

* Updating readme file for macos-13 version 20231018.14 (#8601)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for macos-13-arm64 version 20231017.1 (#8586)

Co-authored-by: Image generation service account <no-reply@microsoft.com>

* [MacOS] improve errexit option handling (#8432)

* [MacOS] improve errexit option handling

* Prevent prompting for override when unzipping

* Explicitly allow failing when installing virtualbox

* Remove libtcl symlinks

* Add debug output

* Updating readme file for ubuntu20 version 20231022.1.1 (#8637)

Co-authored-by: Image generation service account <no-reply@microsoft.com>

* Updating readme file for ubuntu22 version 20231022.1.1 (#8636)

Co-authored-by: Image generation service account <no-reply@microsoft.com>

* Updating readme file for win19 version 20231023.1.1 (#8644)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for win22 version 20231023.1.1 (#8643)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* [macos] simplify "brew_smart_install" helper (#8639)

* [macos] homebrew: use hardcoded condition for jq installation

we cannot use "jq" if we are asked to install "jq"

* [macos] always use "brew install"

* [macos] add retries to "get_github_package_download_url" helper

* [macos] add retries to chrome install script

* [macos] add retries to OpenJDK install script

* [macos] add retries to miniconda installer

* Update images/macos/provision/core/openjdk.sh

Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>

* fix copy-paste error

* Update images/macos/provision/core/openjdk.sh

Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>

* Update images/macos/provision/core/openjdk.sh

Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>

---------

Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>

* Updating readme file for ubuntu22 version 20231025.1.1 (#8663)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for ubuntu20 version 20231025.1.1 (#8664)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for win22 version 20231024.1.1 (#8668)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Windows Server 2019 (20231025) Image Update (#8667)

* Updating readme file for win19 version 20231025.2.1

* Update Windows2019-Readme.md

---------

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Mikhail Koliada <88318005+mikhailkoliada@users.noreply.github.com>
Co-authored-by: Actions service account <no-reply@github.com>

* [Mac OS] Fix logic for retrying AppleScript execution (#8662)

* [Ubuntu] Add PGP validation for swift (#8652)

* [Windows] Remove out-of-support vs components (#8678)

* [MacOS] Unpin Virtualbox (#8653)

* [MacOS] Unpin Virtualbox

* Remove obsolete virtualbox kext pester test

* [Ubuntu] Update apt repos list in documentation (#8634)

* fix: bump libssl version to ubuntu2.20 (#8682)

Signed-off-by: Jakub Oskera <iam@jakuboskera.dev>
Co-authored-by: Jakub Oskera <jakub.oskera@rws.com>

* [macOS] increase timeout for the macOS ci (#8692)

* Updating readme file for macos-13-arm64 version 20231024.2 (#8673)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Escape quotes when running in PowerShell 7.0-7.2 (#8705)

* fix: checksum search for Kotlin (#8704)

* [Ubuntu] Fix checksum search for Kotlin

* [Windows] Fixed checksum search for Kotlin

* [Ubuntu] Pin bindgen cli version to avoid bug (#8714)

* [Ubuntu] Add more checksum validations (#8660)

* [Windows] Pin bindgen cli version to avoid bug (#8713)

* Updating readme file for ubuntu22 version 20231030.2.1 (#8691)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for ubuntu20 version 20231030.2.1 (#8690)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for win19 version 20231029.1.1 (#8687)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Revert "[Windows] Pin bindgen cli version to avoid bug (#8713)" (#8719)

This reverts commit 1dac9b9.

* Revert "[Ubuntu] Pin bindgen cli version to avoid bug (#8714)" (#8720)

This reverts commit 3146c70.

* [Mac OS] Temporary disable VNC for Mac OS 14 (#8717)

* Updating readme file for win22 version 20231029.1.1 (#8686)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* Updating readme file for macOS-11 version 20231030.1 (#8694)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* [Mac OS] Add template for Mac OS 14 image (#8733)

* Add gfortran-13 (#8702)

* Revert "[MacOS] Unpin Virtualbox" (#8731)

* Revert "[MacOS] Unpin Virtualbox (#8653)"

This reverts commit f3d011e.

* Add more info

* [ubuntu] split git and git-lfs into separate install scripts (#8711)

for better observability of integrity validation let us split install
script into two

* Updating readme file for macOS-12 version 20231029.1 (#8695)

Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>

* [Windows] Remove Android SDK Patch applier v4 (#8750)

* remove ndk 23 from macos images (#8747)

* [Mac OS] Remove 'LTS' suffix from OpenJDK version (#8751)

* [windows] split docker install into 3 scripts (#8688)

* [windows] split docker install into 3 scripts

the idea is to end with 1 script per 1 component for better
observability

* Update images/win/scripts/Installers/Install-Docker-Compose.ps1

Co-authored-by: Erik Bershel <110455084+erik-bershel@users.noreply.github.com>

* Update images/win/scripts/Installers/Install-Docker-WinCred.ps1

Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>

* fix test

---------

Co-authored-by: Erik Bershel <110455084+erik-bershel@users.noreply.github.com>
Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>

* remove ndk 23 from ubuntu images (#8745)

* [macos] add android SDK to SW report on Ventura ARM64 (#8715)

* [Mac OS] Update Anka Helpers to build Mac OS 14 ARM image (#8757)

* Update Anka Helpers to build Mac OS 14 ARM image

* Drop Powershell 5 support

* [Mac OS] Add missing macOS Sonoma selector (#8759)

* remove ndk23 from windows images (#8746)

* Updating readme file for macos-13 version 20231025.2 (#8672)

Co-authored-by: Image generation service account <no-reply@microsoft.com>

---------

Signed-off-by: Jakub Oskera <iam@jakuboskera.dev>
Co-authored-by: sergei-pyshnoi <121864472+sergei-pyshnoi@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Image generation service account <no-reply@microsoft.com>
Co-authored-by: Actions service account <no-reply@github.com>
Co-authored-by: Vasilii Polikarpov <126792224+vpolikarpov-akvelon@users.noreply.github.com>
Co-authored-by: ilia-shipitsin <125650415+ilia-shipitsin@users.noreply.github.com>
Co-authored-by: Mikhail Koliada <88318005+mikhailkoliada@users.noreply.github.com>
Co-authored-by: Jakub Oskera <iam@jakuboskera.dev>
Co-authored-by: Jakub Oskera <jakub.oskera@rws.com>
Co-authored-by: Luke Fritts <luketfritts@gmail.com>
Co-authored-by: Erik Bershel <110455084+erik-bershel@users.noreply.github.com>
Co-authored-by: Harmen Stoppels <harmenstoppels@gmail.com>
fffonion pushed a commit to Kong/runner-images-kvm that referenced this pull request Nov 14, 2023
* [Ubuntu] Fix checksum search for Kotlin

* [Windows] Fixed checksum search for Kotlin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants