Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

[Tests-Only] Fix upload UI tests in CI #223

Merged
merged 1 commit into from
May 22, 2020
Merged

[Tests-Only] Fix upload UI tests in CI #223

merged 1 commit into from
May 22, 2020

Conversation

dpakach
Copy link
Contributor

@dpakach dpakach commented May 22, 2020

Mount upload directory for files upload and add missing configuration variables for reva services in Ui tests.

@update-docs
Copy link

update-docs bot commented May 22, 2020

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.

@dpakach dpakach force-pushed the ci-test branch 3 times, most recently from da07394 to 2d09a88 Compare May 22, 2020 11:46
To make sure the correct URL is returned for accessing the TUS service
@dpakach dpakach requested a review from PVince81 May 22, 2020 11:49
@dpakach dpakach changed the title Ci test [Tests-Only] Fix upload UI tests in CI May 22, 2020
@dpakach dpakach marked this pull request as ready for review May 22, 2020 11:50
@dpakach dpakach self-assigned this May 22, 2020
@dpakach
Copy link
Contributor Author

dpakach commented May 22, 2020

@PVince81, this should pass now.
But it takes long time to complete, maybe we will have to split tests into multiple pipelines to make it a bit faster.

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@PVince81 PVince81 merged commit 2b94f09 into master May 22, 2020
@delete-merged-branch delete-merged-branch bot deleted the ci-test branch May 22, 2020 12:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants