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

Let integration tests pass again #2141

Merged
merged 2 commits into from
Apr 19, 2022
Merged

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Apr 11, 2022

Adapt to changed wopi allow list configuration using alias groups: https://sdk.collaboraonline.com/docs/installation/CODE_Docker_image.html#setting-the-application-configuration-dynamically-via-environment-variables

Makes them pass (except for the federation ones which are being looked into by @Raudius separately)

@juliushaertl juliushaertl added the 2. developing Work in progress label Apr 11, 2022
@juliushaertl
Copy link
Member Author

As a side-note on the federation tests: Still no luck with nextcloud/server#31896 it seems even after nextcloud/server#31901 was merged

@juliushaertl juliushaertl added 3. to review Ready to be reviewed and removed 2. developing Work in progress labels Apr 11, 2022
@juliushaertl juliushaertl added this to the Nextcloud 24 milestone Apr 11, 2022
@juliushaertl
Copy link
Member Author

/backport to stable23

@juliushaertl
Copy link
Member Author

/backport to stable4

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl
Copy link
Member Author

Rebased to trigger it again with nextcloud/server#31934

@juliushaertl juliushaertl merged commit 4a70e23 into master Apr 19, 2022
@delete-merged-branch delete-merged-branch bot deleted the ci/collabora-aliasgroup branch April 19, 2022 07:48
@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable4 failed. Please do this backport manually.

@juliushaertl
Copy link
Member Author

/backport 1260f4d to stable4

@juliushaertl
Copy link
Member Author

/backport 1260f4d to stable24

@backportbot-nextcloud
Copy link

The backport to stable4 failed. Please do this backport manually.

@juliushaertl
Copy link
Member Author

/backport 1260f4d to stable23

@backportbot-nextcloud
Copy link

The backport to stable24 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

@juliushaertl
Copy link
Member Author

All right, then manually 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants