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

Release opentelemetry version contraint (#2192) #2199

Merged
merged 3 commits into from
Feb 20, 2025

Conversation

ghjklw
Copy link
Contributor

@ghjklw ghjklw commented Jan 22, 2025

Release soda-core version contraint for opentelemetry as there is no reason to support Python 3.7 and this is creating dependency conflicts with many other libraries.

Closes #2192

Release soda-core version contraint for
opentelemetry as there is no reason to support
Python 3.7 and this is creating dependency
conflicts with many other libraries.
@CLAassistant
Copy link

CLAassistant commented Jan 22, 2025

CLA assistant check
All committers have signed the CLA.

@ghjklw
Copy link
Contributor Author

ghjklw commented Feb 6, 2025

@m1n0 I'm quite confused by the error messages in the CI pipeline. They seem totally unrelated to this chnage and look to me rather like a pipeline configuration issue:

  • Snowflake test data source configuration issue:

    ERROR    soda.scan:log.py:101 [13:55:37] Could not connect to data source "snowflake": 251005: 251005: User is empty
    ERROR    soda.scan:log.py:105   | 251005: 251005: User is empty
    ERROR    soda.scan:log.py:112   | Stacktrace:
    ERROR    soda.scan:log.py:113   | Traceback (most recent call last):
    
  • Bigquery authentication issue: google.auth.exceptions.DefaultCredentialsError: Your default credentials were not found. To set up Application Default Credentials, see https://cloud.google.com/docs/authentication/external/set-up-adc for more information.

Do you have any insight?

@titowoche30
Copy link

It would be really great to see this getting merged

@m1n0
Copy link
Contributor

m1n0 commented Feb 20, 2025

@ghjklw that is because a pr opened based on a fork does not inherit the CI variables, making it impossible for bigquery and snowflake to connect. This is fine as I can see that all other suites pass and this change does not affect the cloud warehouses in any way. Thanks for the contrib, merging.

@m1n0 m1n0 merged commit 17e2c38 into sodadata:main Feb 20, 2025
13 of 15 checks passed
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.

Outdated opentelemetry dependency creating conflicts
4 participants