-
Notifications
You must be signed in to change notification settings - Fork 73
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
Doc - Follow file name conventions of main docs, move to Tutorials #108
Conversation
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.
1 thing I would like to revert, and few typo's.
@@ -102,7 +102,7 @@ This setting specifies the type of storage engine used for all servers | |||
in the cluster. | |||
Possible values are: | |||
|
|||
- `MMFiles` To use the MMfiles storage engine. | |||
- `MMFiles` To use the MMFiles storage engine. |
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.
I don't like this change. IMO it becomes confusing, since the possible value is case sensitive.
So even if the rest of the documentation uses MMfiles
, I'll still prefer to keep MMFiles
here.
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.
I changed it from MMfiles to MMFiles, isn't that what you prefer? In the API, the storage engine names are all lower case IIRC, but we use MMFiles and RocksDB in the docs in most places if it doesn't have to be verbatim.
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.
Sorry, my mistake
- [Upgrading](./upgrading.md) | ||
The ArangoDB Kubernetes Operator (`kube-arangodb`) is a set of two operators | ||
that you deploy in your Kubernetes cluster to manage deployments of the | ||
ArangoDB database and provide `PersistenVolumes` on local storage of your |
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.
PersistentVolumes
(looks like we have that wrong in the entire document)
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.
Good catch, I didn't see the missing T.
@@ -1,7 +1,7 @@ | |||
# Storage | |||
|
|||
An ArangoDB cluster relies heavily on fast persistent storage. | |||
The ArangoDB operator uses `PersistenVolumeClaims` to deliver | |||
The ArangoDB Kubernetes Operator uses `PersistenVolumeClaims` to deliver |
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.
PersistentVolumeClaims
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.
LGTM
Some changes as discussed with Santo.
Also see arangodb/arangodb#5014
@ewoutp Is this a good general description of what kube-arangodb is?
https://github.com/arangodb/kube-arangodb/compare/docs-integration?expand=1#diff-f7213dcd3b2930325722b1d1627b0152
Should the file names of the design docs be changed as well to follow CamelCase convention as used by our main documentation?