-
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
Stabilize example config #913
Conversation
Change-Id: I995c3d65b64eb3dd3cecef6ea1e90bf516679ced
@@ -60,10 +60,6 @@ services: | |||
retry_delay: 250 | |||
retry_limit: 10 | |||
show_sql: false | |||
citoid: |
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 this 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.
Because the example.yaml config don't specify citoid or mathoid modules, so the setting is useless.
CHANGELOG.md
Outdated
|
||
**Breaking changes** | ||
|
||
- `RESTBase` now depends on Cassandra 3.11. In case SQLite backen is used no changes us required. |
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.
s/backen/back-end/
CHANGELOG.md
Outdated
**Breaking changes** | ||
|
||
- `RESTBase` now depends on Cassandra 3.11. In case SQLite backen is used no changes us required. | ||
- After updating `RESTBase` new keyspaces will be created for everything, so it's effectively starting the storage from scratch. All the content stored in `RESTBase` can be rerendered, so no need to transfer the data, but old keyspaces can be deleted afterwards. New keyspace names have `_ng` suffix. |
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.
s/keyspace names have _ng/keyspace names have the _ng/
Also, let's limit the line lengths to 80 chars.
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "restbase", | |||
"version": "0.17.0", | |||
"version": "0.18.0", |
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.
It's not time to release the new version yet. We need to switch everything over before that can happen.
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 that case maybe let's not add the changelog now but just fix the example config and postpone adding the changelog until we actually make the release?
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.
Sure, +1
Change-Id: I9233ca69783214a0b1dfb553edb0616629c56ee8
Make the example configs actually work so that third party or dev installations can use it.
cc @wikimedia/services