Skip to content

Conversation

@pvital
Copy link
Member

@pvital pvital commented May 7, 2025

This PR proposes to fix the CircleCI failure when running the tests on top of Python 3.14.0.

It uses a GitHub Actions workflow to build a container image with all test dependencies and publish it in the GitHub Container Registry every Monday and Wednesday at 00:01. This was pre-tested in a personal repository here.

In addition, this PR modified the CircleCI configuration file to use matrix parameters and point to the new container image as the base image to test Python 3.14.0.

@pvital pvital requested a review from a team as a code owner May 7, 2025 22:01
@pvital pvital self-assigned this May 7, 2025
@pvital pvital requested review from a team and removed request for a team May 7, 2025 22:02
@pvital pvital added currency Keeping supported versions updated. ci/cd labels May 7, 2025
@pvital pvital force-pushed the fix_py3140_ci_failure branch from da0615f to 950e7ae Compare May 7, 2025 22:27
Copy link
Contributor

@GSVarsha GSVarsha left a comment

Choose a reason for hiding this comment

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

Unrelated to this PR, but I guess these lines were copied from ruby-sensor when this file was created. Can we remove them in this PR?

@pvital pvital force-pushed the fix_py3140_ci_failure branch 2 times, most recently from 97801a6 to 4b39ab7 Compare May 8, 2025 08:08
@pvital pvital requested a review from GSVarsha May 8, 2025 08:08
@pvital
Copy link
Member Author

pvital commented May 8, 2025

Hello @instana/eng-python. I have adapted the CircleCI configuration file to pull the Python 3.14.0 image from my personal project.

As soon as we have this reviewed, I will point to the correct image before merging. The image used now is the same as the one we will build in the GH Action.

Copy link
Contributor

@CagriYonca CagriYonca left a comment

Choose a reason for hiding this comment

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

Looks good to me, good job!

@pvital pvital force-pushed the fix_py3140_ci_failure branch 3 times, most recently from 7fafb72 to 7f5233d Compare May 9, 2025 10:16
@pvital pvital requested review from CagriYonca and GSVarsha May 9, 2025 11:09
pvital added 3 commits May 12, 2025 11:35
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Type annotation can’t be interpreted for Annotated Declarative Table form.

https://docs.sqlalchemy.org/en/20/errors.html#type-annotation-can-t-be-interpreted-for-annotated-declarative-table-form
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@GSVarsha GSVarsha force-pushed the fix_py3140_ci_failure branch from 00ad4df to 4fbafe8 Compare May 12, 2025 06:05
Copy link
Contributor

@GSVarsha GSVarsha left a comment

Choose a reason for hiding this comment

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

Looks PERFECT to me :)

@pvital pvital merged commit 57d98fa into main May 12, 2025
16 of 17 checks passed
@pvital pvital deleted the fix_py3140_ci_failure branch May 12, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd currency Keeping supported versions updated.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants