Skip to content

Fix documentation for single node walkthrough#795

Merged
robskillington merged 2 commits into
masterfrom
r/fix-documentation
Jul 16, 2018
Merged

Fix documentation for single node walkthrough#795
robskillington merged 2 commits into
masterfrom
r/fix-documentation

Conversation

@robskillington

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov

codecov Bot commented Jul 16, 2018

Copy link
Copy Markdown

Codecov Report

Merging #795 into master will decrease coverage by 0.05%.
The diff coverage is 45.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #795      +/-   ##
==========================================
- Coverage   78.11%   78.05%   -0.06%     
==========================================
  Files         364      364              
  Lines       31052    31058       +6     
==========================================
- Hits        24255    24243      -12     
- Misses       5180     5190      +10     
- Partials     1617     1625       +8
Flag Coverage Δ
#coordinator 60.04% <45.45%> (-0.07%) ⬇️
#dbnode 81.68% <ø> (-0.06%) ⬇️
#m3ninx 72.7% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b32472...2718c75. Read the comment docs.

Comment thread docs/integrations/prometheus.md Outdated
m3coordinator -f <config-name.yml>
```

Or use the docker container:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: Or, ..

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing.

instances = []*placementpb.Instance{
&placementpb.Instance{
Id: "localhost",
Id: "m3db_local",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Worth pulling this out into configs and/or explicitly mentioning that this is a 'reserved id' or something similar?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing, done.

Comment thread docs/how_to/cluster_hard_way.md Outdated
## Introduction
This document lists the manual steps involved in deploying a M3DB cluster. In practice, you wouldn’t be doing this by hand, you’d be automating it using Terraform or Kubernetes. If you’re interested in how you could do so, we have some other guides you might be interested under the How-To section on https://m3db.github.io/m3db/.

This document lists the manual steps involved in deploying a M3DB cluster. In practice, you wouldn’t be doing this by hand, you’d be automating it using Terraform or Kubernetes. If you’re interested in how you could do so, we have some other guides you might be interested under the How-To section.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Might read better as:

This document lists the manual steps involved in deploying a M3DB cluster. In practice, you'd be automating this using Terraform or Kubernetes rather than doing this by hand; guides for doing so are available under the How-To section.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing, thanks.

Comment thread docs/how_to/single_node.md Outdated
}
```

## Use integrations

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Integrations might be a better header than Use integrations

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing, done.

Comment thread docs/how_to/single_node.md Outdated

## Use integrations

Checkout the integrations documentation to integrate with our software, such as [Prometheus as a long term storage remote read/write endpoint](../integrations/prometheus.md).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe just this?
[Prometheus as a long term storage remote read/write endpoint](../integrations/prometheus.md)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing, done.

Comment thread docs/integrations/prometheus.md Outdated
...
Start by downloading the [config template](https://github.com/m3db/m3db/blob/master/src/coordinator/config/m3coordinator-cluster-template.yml). Update the `namespaces` and the `client` section for a new cluster to match your cluster's configuration.

At the very least you'll need to specify the name of your namespace that you setup, the retention that it has. You can leave the metrics type as `unaggregated` since it's required by default to have a cluster that receives all Prometheus metrics unaggregated. In the future you might also want to aggregate and downsample metrics for longer retention, and you can come back and update the config once you've setup those clusters.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The first line of this and the next line can be combined into something like You'll need to specify the static IPs or hostnames of your M3DB seed nodes, and the name and retention values of the namespace you set up.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing, done.

@robskillington robskillington merged commit f94b19a into master Jul 16, 2018
@robskillington robskillington deleted the r/fix-documentation branch July 16, 2018 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants