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

ci: pro: test update #871

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

detule
Copy link
Collaborator

@detule detule commented Dec 15, 2024

Skip SQL server encoding test since OEM driver does not provide for a way to disable "AutoTranslate" feature.

Fixes #869

@detule detule requested a review from simonpcouch December 15, 2024 18:39
@detule
Copy link
Collaborator Author

detule commented Dec 15, 2024

Hey Simon:

This should hopefully fix the issue error with PRO CI workflow. I don't think I have the ability to run that pipeline - if you get a chance to run it manually, we can verify if the test is skipped appropriately.

@detule
Copy link
Collaborator Author

detule commented Dec 15, 2024

Looks like either I broke something unintentionally, or there is an issue with the runner. I'll look into it and ping back when ready.

@detule
Copy link
Collaborator Author

detule commented Dec 15, 2024

Just adding some running commentary: I think our runner started using 24.04 ( was using Ubuntu 22.04 ) - i suspect this is the culprit of the new issue.

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.

failures re: independent column name encodings
1 participant