Skip to content

Add end to end CI#121

Merged
batmendbar merged 6 commits into
mainfrom
add-end-to-end-CI
Jun 5, 2025
Merged

Add end to end CI#121
batmendbar merged 6 commits into
mainfrom
add-end-to-end-CI

Conversation

@batmendbar
Copy link
Copy Markdown
Collaborator

No description provided.

@batmendbar batmendbar requested review from Dzejkop, kustosz and recmo June 2, 2025 07:51
Comment thread .github/workflows/ci.yml Outdated
@Dzejkop
Copy link
Copy Markdown
Collaborator

Dzejkop commented Jun 3, 2025

@batmendbar
Copy link
Copy Markdown
Collaborator Author

The gnark verifier is currently failing in CI: https://github.com/worldfnd/ProveKit/actions/runs/15407265349/job/43352493286?pr=121#step:10:31

I might be wrong, but it doesn't seem like the gnark verifier threw an error? Do you have more details on what an error 143 means?

@Dzejkop
Copy link
Copy Markdown
Collaborator

Dzejkop commented Jun 3, 2025

Could be OOM or maybe some timeout on GH side? Is this just spinning up the server or is it some one-shot job?

@batmendbar
Copy link
Copy Markdown
Collaborator Author

We are running a recursive verifier on the output created from the rust code. The action was completed in 28 minutes without throwing any error when I tested it on a disposable repo https://github.com/batmendbar/ProveKit/actions/runs/15385512934/job/43284876707. Not sure what happened here.

@Dzejkop
Copy link
Copy Markdown
Collaborator

Dzejkop commented Jun 4, 2025

Looks like it's working!

@Dzejkop
Copy link
Copy Markdown
Collaborator

Dzejkop commented Jun 4, 2025

Actually before you merge, can you discard the changes on the ci.yml file?

@batmendbar
Copy link
Copy Markdown
Collaborator Author

Actually before you merge, can you discard the changes on the ci.yml file?

Done

@batmendbar batmendbar merged commit 895fbbc into main Jun 5, 2025
1 check passed
@batmendbar batmendbar deleted the add-end-to-end-CI branch June 23, 2025 08:23
dcbuild3r pushed a commit that referenced this pull request May 16, 2026
Add end-to-end CI test to test the outputs on recursive verifier
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