Skip to content

Comments

Remove helm charts. They moved to wireapp/wire-server#401

Merged
jschaul merged 1 commit intodevelopfrom
delete-helm-charts
Jan 11, 2021
Merged

Remove helm charts. They moved to wireapp/wire-server#401
jschaul merged 1 commit intodevelopfrom
delete-helm-charts

Conversation

@jschaul
Copy link
Member

@jschaul jschaul commented Dec 29, 2020

Helm charts were added to that repository in wireapp/wire-server#1293

Scope: https://wearezeta.atlassian.net/browse/SQPIT-124

@jmatsushita
Copy link
Contributor

Hey @jschaul does this require updating docs? Seems like README in both repos and even the docs.wire.com would be affected, and probably a bunch of internal docs too?

@jschaul
Copy link
Member Author

jschaul commented Jan 7, 2021

Hey @jschaul does this require updating docs? Seems like README in both repos and even the docs.wire.com would be affected, and probably a bunch of internal docs too?

This particular PR doesn't need much updating docs. In general, the related issue, https://wearezeta.atlassian.net/browse/SQPIT-124 does implicitly require updating docs a little bit. Some of that has already happened, but I agree it might be good to go over various documentation places and see what could be updated.

docs.wire.com documentation will probably not need to change - CI keeps publishing helm charts to an S3 mirror, and that S3 mirror is what the docs on docs.wire.com make use of, so whether they are on wire-server or wire-server-deploy doesn't make a difference. Local documentation wire-server is partially there, and there's an open question here: wireapp/wire-server#1307 on what people think would make sense to still document.

Does that answer your question?

@jschaul jschaul merged commit a4ecbe4 into develop Jan 11, 2021
@jschaul jschaul deleted the delete-helm-charts branch January 11, 2021 16:23
jmatsushita pushed a commit that referenced this pull request Mar 3, 2021
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.

3 participants