-
Notifications
You must be signed in to change notification settings - Fork 122
Add OCM Test Suite matrix workflow for CERNBox #5458
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
base: master
Are you sure you want to change the base?
Add OCM Test Suite matrix workflow for CERNBox #5458
Conversation
Update OCM Test Suite to use ocmstub version 1.1.0 for improved functionality add: OCM Test Suite workflow_dispatch for manual execution Enhance OCM Test Suite workflow Refactor OCM Test Suite workflow to directly pull images fix: job names add: better job names for summary and prepare Signed-off-by: Mahdi Baghbani <mahdi-baghbani@azadehafzar.io>
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
glpatcern
left a comment
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.
This is very good work, if you could also look into the comment below I think we can merge it right away (even if the test currently fails)
|
|
||
| - name: Extract gaia from DockyPody image | ||
| run: | | ||
| docker pull ghcr.io/mahdibaghbani/containers/gaia:latest |
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.
Could you please use some "upstream" references here and in the following, instead of /mahdibaghbani/...?
| ghcr.io/mahdibaghbani/containers/cernbox-revad:mahdi_fix_localhome-development | ||
| ghcr.io/mahdibaghbani/containers/cernbox-web:testing | ||
| - combo_id: cernbox-stub | ||
| display_name: "CERNBox to Stub" |
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.
Do we have an option to mark a failure as "warning" as opposed to full red cross? If not, it's fine to have it as is, definitely better than hiding it
This PR is a part of cs3org/OCM-STA#3
This PR is a part of cs3org/OCM-STA#4
The document for this workflow can be found on OCM Test Suite
A demo of the run can be found in the GitHub Actions, you can also download and review the videos generated by Cypress from artifact section.
This pull request adds a dedicated GitHub Actions workflow that runs the OCM Test Suite against CERNBox and Reva using a small, fixed invite link matrix.
This gives CERNBox and Reva a clear, repeatable invite link compatibility check between Stub and the current Reva build.
The OCM-Stub is subject to be developed in the next milestone, and this workflow would also be updated, as of now it is expected that the results for the CERNBox to Stub and Stub to CERNBox to be failure.