Import vhost configs #108
Merged
Import vhost configs #108
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jvperrin
added a commit
to jvperrin/utils
that referenced
this pull request
May 3, 2020
jvperrin
added a commit
to jvperrin/ocflib
that referenced
this pull request
May 3, 2020
This was added in ocf/etc#108 and should be present on every host, unlike the NFS-backed files previously. This should remove the need for the web fallback path.
jvperrin
added a commit
to jvperrin/ocfweb
that referenced
this pull request
May 3, 2020
Since ocf/etc#108 these configs are in the ocf/etc and we're switching that to be the source of truth. I also updated some of the language here around how quickly updates for new vhosts/apphosts/mailhosts go through the pipeline.
jvperrin
added a commit
to ocf/utils
that referenced
this pull request
May 3, 2020
check: Update vhost config paths (after ocf/etc#108)
Kalissaac
pushed a commit
to Kalissaac/newocfdocs
that referenced
this pull request
Mar 6, 2022
Since ocf/etc#108 these configs are in the ocf/etc and we're switching that to be the source of truth. I also updated some of the language here around how quickly updates for new vhosts/apphosts/mailhosts go through the pipeline.
jyxzhang
pushed a commit
to ocf/utils
that referenced
this pull request
Oct 23, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This imports the vhost configs from
~staff/vhostinto this repo and adds some tests for the config files to make sure they are still formatted and loading correctly when any updates are made. Unfortunately I couldn't make a schema for it since it's a custom config file format currently, but maybe in the future we can make the file a standard YAML file or something, idk.Next steps:
/etc/ocf) instead of the current location (NFS or~staffuserdir via the web)