-
Notifications
You must be signed in to change notification settings - Fork 92
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] CI/CD testing on local servers #424
Conversation
* update rules * update compile and test rules
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should test csim for all Xilinx backend. At least for code generation. But we can add those incrementally. I would suggest you create a separate test case just for testing the CI flow. We'll create a separate PR for modifying the test flow of existing tests.
@seanlatias Finally fixed. It turned out the previous seg fault error was caused by a small error in host code generation (i.e., I believe it is ready for review. |
Can we just revert |
For adding new features
Add feature: add CD/CI testing on our local server. Right now, some test cases requiring EDA tools are skipped in CD/CI testing on CircleCI servers. This PR will add CI/CD testing on our local server, which enables us to verify our design with EDA tools like Vivado HLS or AOC.
How to use the new feature: The local CI/CD testing process is completely automatic.
Detailed description:. For each commit, the webhook on github will automatically inform the local CI/CD runner installed on our local server to pull back the latest changes and run the test cases. The test result will be sent back to github repo and be displayed as the status (i.e., passed or failed) of the commit.
Link to the tests: test_runtime_build.py