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

WOPI Bridge integration #17

Merged
merged 3 commits into from
Sep 1, 2020

Conversation

SamuAlfageme
Copy link
Contributor

@SamuAlfageme SamuAlfageme commented Aug 19, 2020

Enable a simple wopibridge sidecar service to connect the WOPI Server to CodiMD and other office-like applications.

Additional features:

Contributing a Chart / update to an existing Chart

  • Run helm lint on the chart dir.
  • (Update) Bump the Chart.yaml version before merging, to release it as a new version.
  • (Update) If the PR includes new configurable parameters in the chart's values.yaml. Add documentation in the appropiate README.

Includes a fallback to http://<chart-name>:<svc-port> for non-
defined ingress'.

Also, remove hardcoded loglevel and enable its override.
@SamuAlfageme SamuAlfageme requested a review from glpatcern August 19, 2020 16:52
@SamuAlfageme SamuAlfageme force-pushed the wopibridge_integration branch 4 times, most recently from 3f4b682 to 2652ea3 Compare August 20, 2020 14:56
- Fix an issue with whitespace stripping on the 'wopiserver.url'
@SamuAlfageme SamuAlfageme force-pushed the wopibridge_integration branch from 2652ea3 to f520052 Compare August 20, 2020 15:41
... based on the ingress/service provided values, which sometimes
is unaccurate (e.g. wildcard paths for the nginx ingress resources).

Or when the operator does not want to rely on the chart-provided
ingress but rather on its own.

Add the config table for the WOPI bridge and use its url for the
.Values.config.appProviders.codimdurl to prevent using CodiMD's

Bump the wopiserver version to v5.4
@SamuAlfageme SamuAlfageme force-pushed the wopibridge_integration branch from cae36a2 to 05eb576 Compare August 27, 2020 08:57
Copy link
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

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

Tested successfully, all fine with me. Further docs-level improvements to come as we move on.

@Daniel-WWU-IT Daniel-WWU-IT merged commit d8c1c33 into cs3org:master Sep 1, 2020
@SamuAlfageme SamuAlfageme deleted the wopibridge_integration branch September 28, 2020 10:28
@SamuAlfageme SamuAlfageme linked an issue Oct 30, 2020 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adjust hardcoded values on the wopiserver ConfigMap
3 participants