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: fix circom deps installation steps #751

Merged
merged 4 commits into from
Apr 24, 2024
Merged

Conversation

sripwoud
Copy link
Contributor

@sripwoud sripwoud commented Apr 24, 2024

The circom deps in the pull requests workflow can't be install because there is an issue with their ubuntu repository (probably a temporary issue on the microsoft side related to their GPG keys).

As a temporary fix we can simply delete the microsoft source file in the workflow environment.
Besides, as long as the integrity checks of the packages we get from the microsoft repo are broken, I think it is fine to not fetch packages from their repo at all.
The drawback of not getting their softwares updates does not matter so much for us, as this is a ci environment we just use for running tests.

https://github.com/orgs/community/discussions/120966
https://stackoverflow.com/a/78377916

@sripwoud sripwoud changed the title ci: debug pull-requests workflow ci: fix circom deps installation steps Apr 24, 2024
@sripwoud sripwoud self-assigned this Apr 24, 2024
@sripwoud sripwoud marked this pull request as ready for review April 24, 2024 12:00
@sripwoud
Copy link
Contributor Author

sripwoud commented Apr 24, 2024

or we can just wait for microsoft to fix it
probably not, this seems to be a recurring issue at microsoft.
https://stackoverflow.com/a/78377916

So we could consider always skipping their repo in the ci?

@sripwoud sripwoud requested a review from cedoor April 24, 2024 12:38
@sripwoud sripwoud added bug 🐛 Something isn't working devops 🔧 Operations management and dev tools labels Apr 24, 2024
@cedoor cedoor merged commit 1849055 into main Apr 24, 2024
5 checks passed
Copy link

gitpoap-bot bot commented Apr 24, 2024

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2024 Semaphore Contributor:

GitPOAP: 2024 Semaphore Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

@sripwoud sripwoud deleted the fix/install-circom-deps branch April 24, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working devops 🔧 Operations management and dev tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants