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

[tests-only][full-ci]Bump latest-ocis commit to reva master #3776

Merged
merged 5 commits into from
Apr 26, 2023

Conversation

SagarGi
Copy link
Contributor

@SagarGi SagarGi commented Apr 6, 2023

Part of: owncloud/QA#805
This PR bumps the ocis commit id for tests

@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch 5 times, most recently from 8ecb34b to 2d764e6 Compare April 13, 2023 04:18
@phil-davis phil-davis self-requested a review April 13, 2023 04:20
@SagarGi
Copy link
Contributor Author

SagarGi commented Apr 13, 2023

When bumping the latest commit, the pipeline gets killed. But when i use the older commits the pipeline get passed. May be a debug is required for which commit the pipeline gets killed.

@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch 4 times, most recently from 8f51488 to 46b1dca Compare April 18, 2023 05:36
@SagarGi
Copy link
Contributor Author

SagarGi commented Apr 19, 2023

checking in edge if it also has the same problem: #3803 with this PR

@SagarGi SagarGi changed the title Bump latest-ocis commit to reva master [tests-only][full-ci]Bump latest-ocis commit to reva master Apr 20, 2023
@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch 6 times, most recently from 4723435 to 7e93ec6 Compare April 20, 2023 05:03
@SagarGi
Copy link
Contributor Author

SagarGi commented Apr 20, 2023

The bad commit changes that has stuck the pipeline is because of this PR changes in PR owncloud/ocis@1a31aba.

@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch 2 times, most recently from c3c34b6 to 03d2270 Compare April 20, 2023 08:23
@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch 5 times, most recently from a26f372 to 3e0729d Compare April 20, 2023 09:27
@phil-davis phil-davis reopened this Apr 20, 2023
@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch from 824e2c3 to 1c48163 Compare April 21, 2023 03:31
@cs3org cs3org deleted a comment from update-docs bot Apr 21, 2023
@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch 6 times, most recently from 125505f to f8761f9 Compare April 23, 2023 02:31
@SagarGi SagarGi force-pushed the bump-ocis-commit-reva-master branch from 049e1c3 to a72b06f Compare April 26, 2023 03:28
@phil-davis
Copy link
Contributor

https://github.com/cs3org/reva/actions/runs/4804580793/jobs/8550227128?pr=3776

failed to remove network docker_default: Error response from daemon: error while removing network: network docker_default id ab95f490ea59f25884bbfcda14fc61c2bda8ba7bb1818dcbeb1cb14b824aa25b has active endpoints
make: *** [Makefile:130: docker-clean] Error 1

failure seems unrelated to the actual acceptance tests.

@phil-davis phil-davis marked this pull request as ready for review April 26, 2023 04:04
@phil-davis
Copy link
Contributor

@labkode @glpatcern @gmgigi96 please review.
We needed to touch .drone.star so GitHub wants a code-owner review.

@phil-davis phil-davis merged commit f2fca5a into cs3org:master Apr 26, 2023
@phil-davis phil-davis deleted the bump-ocis-commit-reva-master branch April 26, 2023 08:23
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Jun 5, 2023
)

* Bump latest commit of ocis on reva

* update expexted to failure lines

* Add skip on master on compose file

* Take latest ocis

* Took ocis commit 25 apirl
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Jun 28, 2023
)

* Bump latest commit of ocis on reva

* update expexted to failure lines

* Add skip on master on compose file

* Take latest ocis

* Took ocis commit 25 apirl
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.

4 participants