-
Notifications
You must be signed in to change notification settings - Fork 39
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
omicron-ls-apis::test_dependencies
fails outside of CI
#6839
Comments
Additionally there's a test flake if the Omicron tests take more than an hour to get to https://github.com/oxidecomputer/omicron/runs/31428158524 Same root cause. (Well, sort of, Buildomat's API tokens expire after an hour.) |
I think I have a decent workaorund for the outside-of-CI issue that I'll get a PR up for soon. The workaround I have planned for the other issue is to move the separate |
@andrewjstone reported the following issue with the new test added in #6831 which I can trivially reproduce:
This test works okay if you have your system configured in a particular way (either a GitHub token configured or a way to rewrite the HTTPS URL to an SSH one). This is a new requirement to run tests locally that I'd like to avoid having.
The text was updated successfully, but these errors were encountered: