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 os and pyodbc versions for github action #909

Conversation

rishabhtanwar29
Copy link
Contributor

@rishabhtanwar29 rishabhtanwar29 commented Dec 6, 2022

Description

The Github Actions for Python Framework and Upgrade Validation framework started Failing Due to new release of Pyodbc. There is a diff in expected output file due to this version Change More details about the issue can be found here - mkleehammer/pyodbc#1082

We're running the github actions on ubuntu-latest. Previously, it was ubuntu-20.04, but in any recently created fork, it's pointing to ubuntu-22.04. This is causing dependency/build failure in the github actions running in the fork. So, let's fix the os version.

Task: BABEL-OSS
Signed-off-by: Rishabh Tanwar ritanwar@amazon.com
Co-authored-by: Nirmit Shah nirmisha@amazon.com
Co-authored-by: Kuntal Ghosh kuntalgh@amazon.com

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.

For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The Github Actions for Python Framework and Upgrade Validation
framework started Failing Due to new release of Pyodbc. There is
a diff in expected output file due to this version Change More
details about the issue can be found here - mkleehammer/pyodbc#1082

We're running the github actions on ubuntu-latest. Previously, it
was ubuntu-20.04, but in any recently created fork, it's pointing
to ubuntu-22.04. This is causing dependency/build failure in the
github actions running in the fork. So, let's fix the os version.

Task: BABEL-OSS
Signed-off-by: Rishabh Tanwar <ritanwar@amazon.com>
@kuntalghosh kuntalghosh merged commit e04d3df into babelfish-for-postgresql:BABEL_1_4_STABLE Dec 6, 2022
@kuntalghosh kuntalghosh deleted the fix-build-failures branch December 6, 2022 14:20
kuntalghosh pushed a commit to kuntalghosh/babelfish_extensions that referenced this pull request Dec 21, 2022
…l#909)

The Github Actions for Python Framework and Upgrade Validation
framework started Failing Due to new release of Pyodbc. There is
a diff in expected output file due to this version Change More
details about the issue can be found here - mkleehammer/pyodbc#1082

We're running the github actions on ubuntu-latest. Previously, it
was ubuntu-20.04, but in any recently created fork, it's pointing
to ubuntu-22.04. This is causing dependency/build failure in the
github actions running in the fork. So, let's fix the os version.

Task: BABEL-OSS
Authored-by: HarshLunagariya <40052763+HarshLunagariya@users.noreply.github.com>
Signed-off-by: Rishabh Tanwar <ritanwar@amazon.com>
kuntalghosh pushed a commit that referenced this pull request Dec 21, 2022
The Github Actions for Python Framework and Upgrade Validation
framework started Failing Due to new release of Pyodbc. There is
a diff in expected output file due to this version Change More
details about the issue can be found here - mkleehammer/pyodbc#1082

We're running the github actions on ubuntu-latest. Previously, it
was ubuntu-20.04, but in any recently created fork, it's pointing
to ubuntu-22.04. This is causing dependency/build failure in the
github actions running in the fork. So, let's fix the os version.

Task: BABEL-OSS
Authored-by: HarshLunagariya <40052763+HarshLunagariya@users.noreply.github.com>
Signed-off-by: Rishabh Tanwar <ritanwar@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants