Skip to content

[full-ci] remove collaboration service. run e2e tests against euro-office - #2739

Merged
v-scharf merged 2 commits into
mainfrom
remove-collaboration
Jun 30, 2026
Merged

[full-ci] remove collaboration service. run e2e tests against euro-office#2739
v-scharf merged 2 commits into
mainfrom
remove-collaboration

Conversation

@v-scharf

@v-scharf v-scharf commented Jun 24, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • removed collaboration service from docker-compose.yml and .woodpecker.star
  • replace onlyoffice to euro-office in CI

@v-scharf v-scharf added the Type:DevOps Related to CI/CD label Jun 24, 2026
@v-scharf

v-scharf commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

@kulmann before I adjust the onlyoffice setup in CI to run without the collaboration service, may I replace onlyoffice with euroOffice?

v-scharf
v-scharf commented Jun 24, 2026
View reviewed changes
Comment thread docker-compose.yml
@JammingBen

JammingBen commented Jun 25, 2026

Copy link
Copy Markdown
Member

I don't think we should run OnlyOffice tests in CI, given our repeated statements that we don't want to promote using it. It should be easy to just replace it with EuroOffice though, no? IMO it would't hurt to have that.

@v-scharf

v-scharf commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

I don't think we should run OnlyOffice tests in CI, given our repeated statements that we don't want to promote using it. It should be easy to just replace it with EuroOffice though, no? IMO it would't hurt to have that.

It shouldn’t be difficult. I’ll switch our CI setup to eurooffice

@v-scharf v-scharf mentioned this pull request Jun 26, 2026
Closed
@v-scharf
v-scharf marked this pull request as draft June 29, 2026 07:14
@v-scharf
v-scharf force-pushed the remove-collaboration branch from e5173c8 to e2789e9 Compare June 29, 2026 13:27
@v-scharf

v-scharf commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

It shouldn’t be difficult. I’ll switch our CI setup to eurooffice

works 🚀
see here https://ci.opencloud.rocks/repos/6/pipeline/304/199

@v-scharf
v-scharf marked this pull request as ready for review June 29, 2026 13:44
@v-scharf v-scharf changed the title [full-ci] remove collaboration service [full-ci] remove collaboration service. run e2e tests against euro-office Jun 29, 2026
JammingBen
JammingBen reviewed Jun 29, 2026
View reviewed changes

@JammingBen JammingBen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! The only thing I don't quite like is that you now need to run Collabora. I usually started OpenCloud via docker compose up opencloud -d, which then only ran OpenCloud, tika and traefik. Now it also starts Collabora, which takes quite some time. Is there no way around that?

Comment thread .woodpecker.star
@v-scharf

v-scharf commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

Nice! The only thing I don't quite like is that you now need to run Collabora. I usually started OpenCloud via docker compose up opencloud -d, which then only ran OpenCloud, tika and traefik. Now it also starts Collabora, which takes quite some time. Is there no way around that?

I didn't find good way except of moving collabora to compose.collabora.yml and override opencloud service there to using depends_on of collabora

@v-scharf
v-scharf requested a review from JammingBen June 30, 2026 07:26
@v-scharf v-scharf mentioned this pull request Jun 30, 2026
Closed
@v-scharf
v-scharf force-pushed the remove-collaboration branch from 1f7c7e2 to affecc2 Compare June 30, 2026 08:21
@v-scharf
v-scharf merged commit ec1ea09 into main Jun 30, 2026
31 checks passed
@v-scharf
v-scharf deleted the remove-collaboration branch June 30, 2026 08:51
AlexAndBear pushed a commit that referenced this pull request Jun 30, 2026
@v-scharf @AlexAndBear
[full-ci] remove collaboration service. run e2e tests against euro-of…
c4599b0 
…fice (#2739)

* remove collaboration service

* remove onlyoffice dependancy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JammingBen JammingBen JammingBen approved these changes
@AlexAndBear AlexAndBear Awaiting requested review from AlexAndBear
@kulmann kulmann Awaiting requested review from kulmann

Assignees

No one assigned

Labels

Type:DevOps Related to CI/CD

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@v-scharf @JammingBen