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

chore: update appveyor image #6278

Merged
merged 73 commits into from
Dec 5, 2023
Merged

chore: update appveyor image #6278

merged 73 commits into from
Dec 5, 2023

Conversation

mndeveci
Copy link
Contributor

@mndeveci mndeveci commented Nov 16, 2023

Updating Appveyor Linux image to Ubuntu2204

  • Install jdk21 as it is not available in Appveyor images yet
  • Update maven and gradle installations to comply with jdk21 update abpove
  • Adding python3.12 runtime information to PATH environment variable so that we can build lambda functions with that runtime.
  • Limit test cases when running ARM64 and building inside container for java runtimes. For gradlew and gradle-kotlin test cases, we only run them if java runtime is latest (java21 atm).
  • Remove provided (al1) test cases for Go and Rust since these runtimes are expected to run on provided.al2 and provided.al2023 runtimes.
  • Revert Makefile and bring back Makefile-container which uses available python executables for each environment.

Update Appveyor Windows image:

  • install Corretto JDK 21 and set corresponding
  • Upgrade gradle
  • set path for Python3.12
  • skip all build command tests for the new runtimes (provided.al2023, python3.12, nodejs20.x and java21) in AppVeyor Windows that require docker. This is due to the docker version in AppVeyor Windows are not new enough (>=20.10.10) to support AL2023-based containers. See: https://docs.docker.com/engine/release-notes/20.10/#201010

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

appveyor-windows.yml Outdated Show resolved Hide resolved
@hawflau hawflau marked this pull request as ready for review December 4, 2023 17:52
@hawflau hawflau requested a review from a team as a code owner December 4, 2023 17:52
@hawflau hawflau requested review from lucashuy and bentvelj December 4, 2023 17:52
@hawflau hawflau added this pull request to the merge queue Dec 4, 2023
@hawflau hawflau removed this pull request from the merge queue due to a manual request Dec 4, 2023
@mndeveci mndeveci enabled auto-merge December 4, 2023 22:47
@mndeveci mndeveci added this pull request to the merge queue Dec 5, 2023
Merged via the queue into develop with commit ca87c88 Dec 5, 2023
55 checks passed
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.

4 participants