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

Disabling mysterious CI failures #974

Merged
merged 13 commits into from
Oct 22, 2024
Merged

Disabling mysterious CI failures #974

merged 13 commits into from
Oct 22, 2024

Conversation

therealryan
Copy link
Collaborator

@therealryan therealryan commented Oct 21, 2024

Our selenium-based tests started throwing failures today. We've made no changes to the project and can't replicate the failure locally.

We've tried:

  • Testing on a windows-os CI machine. This had no effect - the windows testing shows the same failure as linux
  • Downgrading the chrome version to what should have existed for previous successful runs. This had no effect.

It really sucks, but this PR disables the tests in question in CI runs. We can still run the tests locally.

I've raised #975 to fix this in the future, but right now:

  • these failures are blocking PRs that we probably want
  • local testing shows that the functionality being exercised does still work.

Copy link

sonarcloud bot commented Oct 21, 2024

@therealryan therealryan changed the title Adding windows-os testing Disabling mysterious CI failures Oct 21, 2024
@vchaitanya vchaitanya merged commit 514d8b1 into main Oct 22, 2024
7 checks passed
@vchaitanya vchaitanya deleted the multi-os branch October 22, 2024 04:44
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.

2 participants