-
Notifications
You must be signed in to change notification settings - Fork 113
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
Migrate ocisIntegrationTests and s3ngIntegrationTests to GitHub Actions #3640
Conversation
https://github.com/cs3org/reva/actions/runs/4085928053/jobs/7044608957 I guess that docker-compose is starting up the reva system-under-test and the test runner (behat...), but the system-under-test is spewing out large amounts of "log" output. Can that be cut down? Or the output redirected to Maybe then we will be able to see |
Sure, my latest commit should do it. |
e504ecf
to
26ace15
Compare
@phil-davis The output of the system-under-test has been suppressed and now we're able to see the output of |
ca5fbe2
to
ac84a6c
Compare
Now it needs a rebase and resolve conflicts in The CI runs from last week do not have many unexpected failures. (from comment #3640 (comment) ) But somehow the rendering in the GitHub workflow output is very hard to read. Do the rebase and I will look again. Maybe I will need to get rid of the fancy colour output strings, which GitHub log output is not interpreting. That will make things easier to read. |
See #3672 - that works with the acceptance tests and the current drone scripts. The acceptance test output comes as plain text, without any escape sequence to select text colors etc. If you rebase this PR, and set env variable |
Thanks Phil, PLAIN_OUTPUT is set now but errors on the run.sh script are causing the tests to fail. I see that you've fixed the variable assignments (owncloud/ocis#5624) and tried to bump ocis on reva (#3680 ) but it's failing Drone. I'll rebase and re-run GitHub Actions after it's fixed. |
0f44a4e
to
781f283
Compare
There is also #3679 waiting for approval and merge. After that is merged, I will stop changing test code! And we can concentrate on what failures remain. |
9d0a1c8
to
6a28e60
Compare
Thanks @phil-davis , #3679 is now merged. Me and @labkode found that some tests were failing on Alma8 self-hosted runners so we upgraded the cluster used by my fork (only https://github.com/vascoguita/reva so far) to Alma9 but there are other tests failing. |
2ab58a8
to
9168412
Compare
1a27b6a
to
a7fa855
Compare
…ns (cs3org#3640) * Migrate ocisIntegrationTests and s3ngIntegrationTests to GitHub Actions * Temporary fix
No description provided.