Skip to content
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.

Sign up for GitHub

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

Update install.md #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update install.md #15

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions doc/install.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Installation guide
Installation Guide
==================

ClickHouse
Expand Down Expand Up @@ -159,29 +159,29 @@ CREATE TABLE graphite.metrics ( date Date DEFAULT toDate(0), name String, leve
CREATE TABLE graphite.data ( metric String, value Float64, timestamp UInt32, date Date, updated UInt32) ENGINE = GraphiteMergeTree(date, (metric, timestamp), 8192, 'graphite_rollup');
```

**Notice**: If you don't want ClickHouse to rollup data, you can use ReplacingMergeTree instead of GraphiteMergeTree.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use GraphiteMergeTree instead of GraphiteMergeTree, if Clickhouse data roll up is unnecessary?

**Notice**: If you don't want ClickHouse to roll up data, you can use ReplacingMergeTree instead of GraphiteMergeTree.
```sql
CREATE TABLE graphite.data ( metric String, value Float64, timestamp UInt32, date Date, updated UInt32) ENGINE = ReplacingMergeTree(date, (metric, timestamp), 8192, updated)
```
But you still need to describe the rules for the rotation, so that Graphouse knows its metrics retention.
Though it's better to describe the rotation rules, so that Graphouse knows its metrics retention.


Graphouse
---------
- Add ClickHouse debian repo. [See doc.](https://clickhouse.yandex/reference_en.html#Installing)
- [Install JDK8.](https://tecadmin.net/install-oracle-java-8-ubuntu-via-ppa/)
- Install Graphouse `sudo agt-get install graphouse`
- Set `graphouse.clickhouse.retention-config` property in graphouse config /etc/graphouse/graphouse.properties. You can skip this step, then [default config](../src/main/java/ru/yandex/market/graphouse/retention/DefaultRetentionProvider.java#L29) will be used
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Set... otherwise ... will be used?

- Start graphouse `sudo /etc/init.d/graphouse start`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sudo service graphouse start is also valid command.

- Set `graphouse.clickhouse.retention-config` property in Graphouse config `/etc/graphouse/graphouse.properties`. You can skip this step. If so, [default config](../src/main/java/ru/yandex/market/graphouse/retention/DefaultRetentionProvider.java#L29) will be used
- Start Graphouse `sudo /etc/init.d/graphouse start`

If you have any problems check graphouse log dir for details `/var/log/graphouse`.
If you have any problems check Graphouse log dir for details `/var/log/graphouse/`.
See [Configuration](config.md) for more details.


Graphite-web
------------
- Install [graphite-web](http://graphite.readthedocs.io/en/latest/), if you don't have it already. You don't need carbon or whisper, Graphouse and ClickHouse completely replace them.
- Add graphouse plugin `/opt/graphouse/bin/graphouse.py` to your graphite webapp root dir.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy into?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я же дальше предлагаю симлинком это сделать

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ок, тогда 2 вопроса:

  1. ИМХО формулировка "добавь плагин в папку" немного странная. Как тебе такая формулировка: Install Graphouse module into your Graphite directory)?
  2. И про симлинк - для создания симлинка sudo не всегда нужен. Думаю, более корректно, указывать команду без него

- Add Graphouse plugin `/opt/graphouse/bin/graphouse.py` to your Graphite webapp root dir.
For example, if you dir is `/opt/graphite/webapp/graphite/` use command below
```bash
sudo ln -fs /opt/graphouse/bin/graphouse.py /opt/graphite/webapp/graphite/graphouse.py
Expand Down