-
Notifications
You must be signed in to change notification settings - Fork 79
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
Document origin configuration and improve user guide. #106
Merged
mikkokar
merged 14 commits into
ExpediaGroup:master
from
dvlato:document-origin-configuration
Mar 13, 2018
Merged
Changes from 7 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
cb109de
#100 Backend configuration was missing from the user guide.
a-dlatorre 356087b
#100 Changed references in the quickstart to existing files in the di…
a-dlatorre 623e62a
Standard codes moved out to their own page. Additional pages revised.
a-dlatorre 2c71302
#100 Rest of documents reviewed.
a-dlatorre 413d973
#100 Merged changes for the server error codes.
a-dlatorre fa4d62a
#100 Merged changes for the server error codes.
a-dlatorre 5e18c02
#100 Small typos.
a-dlatorre 2d6d908
Standard codes moved out to their own page. Additional pages revised.
a-dlatorre a350d45
#100 Documentation improvements.
a-dlatorre cfcedc4
#100 Documentation improvements.
a-dlatorre f475532
#100 Documentation improvements.
a-dlatorre ca86cc7
#100 Minor fixes to the documentation.
a-dlatorre e912e97
#100 Minor fixes to the documentation.
a-dlatorre 681cb38
#100 Minor fixes to the documentation.
a-dlatorre File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,34 +2,30 @@ | |
|
||
## System Requirements | ||
|
||
Building Styx requires Java 1.8. It can be built with 1.8.0_45. Earlier maintenance | ||
releases may work, but are not guaranteed to. | ||
|
||
The build system requires Apache Maven. The Styx team uses Maven version 3.2.1 | ||
for automated continuous integration builds. On Mac OSX, a version installed | ||
by HomeBrew is satisfactory. | ||
Running Styx requires Java 1.8. It can be run with 1.8.0_45 or later versions. Earlier maintenance | ||
releases or Java 1.9 may work, but are not guaranteed to. | ||
|
||
|
||
## Step 1: Installing Styx: | ||
|
||
Styx comes pre-packaged in a zip file. Extract the zip file to a directory of choice: | ||
|
||
$ unzip styx.zip | ||
$ unzip styx-<VERSION>-<OS>-<PLATFORM>.zip | ||
|
||
This creates a subdirectory called `styx` that contains Styx binaries and a set | ||
of configuration examples. Example configuration files can be found in `styx/conf` | ||
This creates a subdirectory called `styx-<VERSION>` that contains Styx binaries and a set | ||
of configuration examples. Example configuration files can be found in `styx-<VERSION>/conf` | ||
subdirectory. There are some files worth noticing: | ||
|
||
* `default.yml` - A Styx server configuration file. It specifies the Styx server | ||
port numbers and other application configuration. | ||
|
||
* `styx/conf/origins` subdirectory containing examples of Styx origins configuration | ||
* `conf/origins` subdirectory containing examples of Styx origins configuration | ||
files. A styx origins configuration file specifies the backend services for Styx. | ||
Especially, have a look at `origins-configuration-documentation.yml`. It explains | ||
all aspects of origins configuration settings. | ||
all aspects of origins configuration settings (you can see more details in [Backend services and origins](configure-origins.md)). | ||
|
||
|
||
* `logback.xml` - Logging configuration file. There are more examples in the `styx/conf/logback` | ||
* `logback.xml` - Logging configuration file. There are more examples in the `conf/logback` | ||
subdirectory. | ||
|
||
* `styx-env.sh` - JVM settings file. See <TODO> section below. | ||
|
@@ -41,17 +37,17 @@ This creates a subdirectory called `styx` that contains Styx binaries and a set | |
To start Styx, run the startup script from *bin/* directory, passing in the | ||
configuration file as an argument: | ||
|
||
$ ./styx/bin/startup ~/configs/staging.yml | ||
$ ./bin/startup conf/env-development/styx-config.yml | ||
|
||
This starts Styx according to a configuration specified in *staging.yml* file. | ||
This starts Styx according to the configuration specified in *staging.yml* file. | ||
|
||
You also can specify the configuration via *STYX_CONFIG* environment | ||
variable. For example: | ||
|
||
STYX_CONFIG=~/configs/production.yml ./styx/bin/startup | ||
STYX_CONFIG=conf/env-development/styx-config.yml ./bin/startup | ||
|
||
If the configuration file is not specified as a command line argument or as an environment | ||
variable, Styx by default attempts to load configuration from *$STYX_HOME/conf/default.yaml*. | ||
variable, Styx by default attempts to load its configuration from *$STYX_HOME/conf/default.yaml*. | ||
|
||
Once Styx has started up, it displays the Styx logo banner, followed by information | ||
about the port numbers it is listening on. | ||
|
@@ -77,7 +73,7 @@ necessary before startup. | |
The styx startup script accepts an {-e} or {--env} command line option for specifying an alternative | ||
Styx environment file: | ||
|
||
$ ./bin/startup -e ~/configs/custom-env.sh | ||
$ ./bin/startup -e conf/env-development/styx-env.sh | ||
|
||
The custom environment file can also be specified via *STYX_ENV_FILE* environment variable. | ||
|
||
|
@@ -90,7 +86,7 @@ configuration. | |
By default, the logback file is loaded from *$STYX_HOME/conf/logback.xml*. You can specify an alternative | ||
logging configuration file using {-l} or {--logback} command line options: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There are some old formatting isssues with the page, such as |
||
|
||
$ ./bin/startup --logback ~/configs/logback-custom.xml | ||
$ ./bin/startup --logback conf/env-development/logback.xml | ||
|
||
You can also specify this via environment: *STYX_LOG_CONFIG*. | ||
|
||
|
@@ -152,7 +148,7 @@ Command line options are: | |
Any environment variable starting with <JVM_> declares a command line | ||
option for the underlying Java Virtual Machine. | ||
The startup throws away the JVM_<SETTING> part, and passes the | ||
<VALUE>, verbatim, for the JVM as a command line option. | ||
<VALUE>, verbatim, as a command line option for the JVM. | ||
|
||
|
||
|
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is the remark about maven being removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The rest of the instructions were considering that we were downloading an already compiled binary, but here we were explaining how to build the app instead of how to run it.
Probably we can also add instructions to build from source code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. Makes sense.