Skip to content

test: install latest versions of python/java in linux-binary tests#8559

Merged
valerena merged 1 commit intoaws:developfrom
valerena:update-versions-appveyor-binary
Jan 13, 2026
Merged

test: install latest versions of python/java in linux-binary tests#8559
valerena merged 1 commit intoaws:developfrom
valerena:update-versions-appveyor-binary

Conversation

@valerena
Copy link
Copy Markdown
Contributor

@valerena valerena commented Jan 9, 2026

Which issue(s) does this change fix?

In linux-binary integration tests, we were not installing Python 3.14 or java 25.

Why is this change necessary?

How does it address the issue?

What side effects does this change have?

Mandatory Checklist

PRs will only be reviewed after checklist is complete

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

@valerena valerena requested a review from a team as a code owner January 9, 2026 23:09
- sh: "./aws_cli/bin/python -m pip install awscli"
- sh: "PATH=$(echo $PWD'/aws_cli/bin'):$PATH"

# install python 3.14
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code itself seems fine, but I personally don't understand what we're adding here. Why was 3.13 not being installed before (or any other version for that matter)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good question. I mostly brought the code from the appveyor-ubuntu definition. My guess is that the AppVeyor host has some versions of Python installed, so we're just using those without having to install them ourselves, but it doesn't have Python3.14

@valerena valerena added this pull request to the merge queue Jan 13, 2026
Merged via the queue into aws:develop with commit ff66168 Jan 13, 2026
44 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.

3 participants