Skip to content
This repository was archived by the owner on Mar 6, 2023. It is now read-only.

Commit 5b73b9c

Browse files
authored
Merge pull request #207 from nop33/fix-typos
Fix typos in README
2 parents 3906d68 + d08eb7e commit 5b73b9c

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

README.md

+8-8
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
## Warning
1111

12-
Due to limitations of galaxy.ansible.com we had to move role to https://galaxy.ansible.com/cloudalchemy/node_exporter and use `_` instead of `-` in role name. This is a breaking change and unfortunatelly it affects all versions of node_exporter role as ansible galaxy doesn't offer any form of redirection. We are sorry for the inconvenience.
12+
Due to limitations of galaxy.ansible.com we had to move the role to https://galaxy.ansible.com/cloudalchemy/node_exporter and use `_` instead of `-` in role name. This is a breaking change and unfortunately, it affects all versions of node_exporter role as ansible galaxy doesn't offer any form of redirection. We are sorry for the inconvenience.
1313

1414
## Description
1515

@@ -19,7 +19,7 @@ Deploy prometheus [node exporter](https://github.com/prometheus/node_exporter) u
1919

2020
- Ansible >= 2.9 (It might work on previous versions, but we cannot guarantee it)
2121
- gnu-tar on Mac deployer host (`brew install gnu-tar`)
22-
- Passlib is required when using the basic authentatication feature (`pip install passlib[bcrypt]`)
22+
- Passlib is required when using the basic authentication feature (`pip install passlib[bcrypt]`)
2323

2424
## Role Variables
2525

@@ -51,7 +51,7 @@ Use it in a playbook as follows:
5151
5252
### TLS config
5353
54-
Before running node_exporter role, user needs to provision their own certificate and key.
54+
Before running node_exporter role, the user needs to provision their own certificate and key.
5555
```yaml
5656
- hosts: all
5757
pre_tasks:
@@ -85,26 +85,26 @@ We provide demo site for full monitoring solution based on prometheus and grafan
8585
8686
## Local Testing
8787
88-
The preferred way of locally testing the role is to use Docker and [molecule](https://github.com/metacloud/molecule) (v2.x). You will have to install Docker on your system. See "Get started" for a Docker package suitable to for your system.
89-
We are using tox to simplify process of testing on multiple ansible versions. To install tox execute:
88+
The preferred way of locally testing the role is to use Docker and [molecule](https://github.com/metacloud/molecule) (v2.x). You will have to install Docker on your system. See "Get started" for a Docker package suitable for your system.
89+
We are using tox to simplify the process of testing on multiple ansible versions. To install tox execute:
9090
```sh
9191
pip3 install tox
9292
```
9393
To run tests on all ansible versions (WARNING: this can take some time)
9494
```sh
9595
tox
9696
```
97-
To run a custom molecule command on custom environment with only default test scenario:
97+
To run a custom molecule command on a custom environment with only default test scenario:
9898
```sh
9999
tox -e py35-ansible28 -- molecule test -s default
100100
```
101101
For more information about molecule go to their [docs](http://molecule.readthedocs.io/en/latest/).
102102

103-
If you would like to run tests on remote docker host just specify `DOCKER_HOST` variable before running tox tests.
103+
If you would like to run tests on a remote docker host just specify `DOCKER_HOST` variable before running tox tests.
104104

105105
## Travis CI
106106

107-
Combining molecule and travis CI allows us to test how new PRs will behave when used with multiple ansible versions and multiple operating systems. This also allows use to create test scenarios for different role configurations. As a result we have a quite large test matrix which will take more time than local testing, so please be patient.
107+
Combining molecule and travis CI allows us to test how new PRs will behave when used with multiple ansible versions and multiple operating systems. This also allows creating test scenarios for different role configurations. As a result, we have a quite large test matrix which will take more time than local testing, so please be patient.
108108

109109
## Contributing
110110

0 commit comments

Comments
 (0)