Skip to content

Commit 9bd1cce

Browse files
cstyanIvan Kiselev
authored andcommitted
Fix typos, including 'fomrat' -> 'format' in tracing.config-file help text. (thanos-io#1552)
Signed-off-by: Callum Styan <callumstyan@gmail.com>
1 parent bd16a35 commit 9bd1cce

File tree

9 files changed

+18
-18
lines changed

9 files changed

+18
-18
lines changed

cmd/thanos/flags.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ func regCommonTracingFlags(app *kingpin.Application) *pathOrContent {
118118
fileFlagName := fmt.Sprintf("tracing.config-file")
119119
contentFlagName := fmt.Sprintf("tracing.config")
120120

121-
help := fmt.Sprintf("Path to YAML file that contains tracing configuration. See fomrat details: https://thanos.io/tracing.md/#configuration ")
121+
help := fmt.Sprintf("Path to YAML file that contains tracing configuration. See format details: https://thanos.io/tracing.md/#configuration ")
122122
tracingConfFile := app.Flag(fileFlagName, help).PlaceHolder("<tracing.config-yaml-path>").String()
123123

124124
help = fmt.Sprintf("Alternative to '%s' flag. Tracing configuration in YAML. See format details: https://thanos.io/tracing.md/#configuration", fileFlagName)

docs/components/bucket.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ config:
2424
```
2525
2626
Bucket can be extended to add more subcommands that will be helpful when working with object storage buckets
27-
by adding a new command within `/cmd/thanos/bucket.go`
27+
by adding a new command within `/cmd/thanos/bucket.go`.
2828

2929

3030
## Deployment
@@ -44,7 +44,7 @@ Flags:
4444
--log.format=logfmt Log format to use.
4545
--tracing.config-file=<tracing.config-yaml-path>
4646
Path to YAML file that contains tracing
47-
configuration. See fomrat details:
47+
configuration. See format details:
4848
https://thanos.io/tracing.md/#configuration
4949
--tracing.config=<tracing.config-yaml>
5050
Alternative to 'tracing.config-file' flag. Tracing
@@ -103,7 +103,7 @@ Flags:
103103
--log.format=logfmt Log format to use.
104104
--tracing.config-file=<tracing.config-yaml-path>
105105
Path to YAML file that contains tracing
106-
configuration. See fomrat details:
106+
configuration. See format details:
107107
https://thanos.io/tracing.md/#configuration
108108
--tracing.config=<tracing.config-yaml>
109109
Alternative to 'tracing.config-file' flag.
@@ -151,7 +151,7 @@ Flags:
151151
--log.format=logfmt Log format to use.
152152
--tracing.config-file=<tracing.config-yaml-path>
153153
Path to YAML file that contains tracing
154-
configuration. See fomrat details:
154+
configuration. See format details:
155155
https://thanos.io/tracing.md/#configuration
156156
--tracing.config=<tracing.config-yaml>
157157
Alternative to 'tracing.config-file' flag. Tracing
@@ -213,7 +213,7 @@ Flags:
213213
--log.format=logfmt Log format to use.
214214
--tracing.config-file=<tracing.config-yaml-path>
215215
Path to YAML file that contains tracing
216-
configuration. See fomrat details:
216+
configuration. See format details:
217217
https://thanos.io/tracing.md/#configuration
218218
--tracing.config=<tracing.config-yaml>
219219
Alternative to 'tracing.config-file' flag. Tracing
@@ -256,7 +256,7 @@ Flags:
256256
--log.format=logfmt Log format to use.
257257
--tracing.config-file=<tracing.config-yaml-path>
258258
Path to YAML file that contains tracing
259-
configuration. See fomrat details:
259+
configuration. See format details:
260260
https://thanos.io/tracing.md/#configuration
261261
--tracing.config=<tracing.config-yaml>
262262
Alternative to 'tracing.config-file' flag. Tracing

docs/components/check.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ Flags:
2525
--log.format=logfmt Log format to use.
2626
--tracing.config-file=<tracing.config-yaml-path>
2727
Path to YAML file that contains tracing
28-
configuration. See fomrat details:
28+
configuration. See format details:
2929
https://thanos.io/tracing.md/#configuration
3030
--tracing.config=<tracing.config-yaml>
3131
Alternative to 'tracing.config-file' flag. Tracing
@@ -69,7 +69,7 @@ Flags:
6969
--log.format=logfmt Log format to use.
7070
--tracing.config-file=<tracing.config-yaml-path>
7171
Path to YAML file that contains tracing
72-
configuration. See fomrat details:
72+
configuration. See format details:
7373
https://thanos.io/tracing.md/#configuration
7474
--tracing.config=<tracing.config-yaml>
7575
Alternative to 'tracing.config-file' flag. Tracing

docs/components/compact.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ Flags:
6565
--log.format=logfmt Log format to use.
6666
--tracing.config-file=<tracing.config-yaml-path>
6767
Path to YAML file that contains tracing
68-
configuration. See fomrat details:
68+
configuration. See format details:
6969
https://thanos.io/tracing.md/#configuration
7070
--tracing.config=<tracing.config-yaml>
7171
Alternative to 'tracing.config-file' flag.

docs/components/query.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ Flags:
206206
--log.format=logfmt Log format to use.
207207
--tracing.config-file=<tracing.config-yaml-path>
208208
Path to YAML file that contains tracing
209-
configuration. See fomrat details:
209+
configuration. See format details:
210210
https://thanos.io/tracing.md/#configuration
211211
--tracing.config=<tracing.config-yaml>
212212
Alternative to 'tracing.config-file' flag.

docs/components/rule.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ Flags:
159159
--log.format=logfmt Log format to use.
160160
--tracing.config-file=<tracing.config-yaml-path>
161161
Path to YAML file that contains tracing
162-
configuration. See fomrat details:
162+
configuration. See format details:
163163
https://thanos.io/tracing.md/#configuration
164164
--tracing.config=<tracing.config-yaml>
165165
Alternative to 'tracing.config-file' flag.

docs/components/sidecar.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ In details:
2121
Prometheus servers connected to the Thanos cluster via the sidecar are subject to a few limitations and recommendations for safe operations:
2222

2323
* The recommended Prometheus version is 2.2.1 or greater (including newest releases). This is due to Prometheus instability in previous versions as well as lack of `flags` endpoint.
24-
* (!) The Prometheus `external_labels` section of the Prometheus configuration file has unique labels in the overall Thanos system. Those external labels will be used by sidecar and then Thanos in many places:
24+
* (!) The Prometheus `external_labels` section of the Prometheus configuration file has unique labels in the overall Thanos system. Those external labels will be used by the sidecar and then Thanos in many places:
2525

2626
* [Querier](./query.md) to filter out store APIs to touch during query requests.
2727
* Many object storage readers like [compactor](./compact.md) and [store gateway](./store.md) which groups the blocks by Prometheus source. Each produced TSDB block by Prometheus is labelled with external label by sidecar before upload to object storage.
@@ -31,7 +31,7 @@ Prometheus servers connected to the Thanos cluster via the sidecar are subject t
3131
If you choose to use the sidecar to also upload to object storage:
3232

3333
* The `--storage.tsdb.min-block-duration` and `--storage.tsdb.max-block-duration` must be set to equal values to disable local compaction on order to use Thanos sidecar upload, otherwise leave local compaction on if sidecar just exposes StoreAPI and your retention is normal. The default of `2h` is recommended.
34-
Mentioned parameters set to equal values disable the internal Prometheus compaction, which is needed to avoid the uploaded data corruption when Thanos compactor does its job, this is critical for data consistency and should not be ignored if you plan to use Thanos compactor. Even though you set mentioned parameters equal, you might observe Prometheus internal metric `prometheus_tsdb_compactions_total` being incremented, don't be confused by that: Prometheus writes initial head block to filesytem via internal compaction mechanism, but if you have followed recommendations - data won't be modified by Prometheus before sidecar uploads it. Thanos sidecar will also check sanity of the flags set to Prometheus on the startup and log errors or warning if they have been configured improperly (#838).
34+
Mentioned parameters set to equal values disable the internal Prometheus compaction, which is needed to avoid the uploaded data corruption when Thanos compactor does its job, this is critical for data consistency and should not be ignored if you plan to use Thanos compactor. Even though you set mentioned parameters equal, you might observe Prometheus internal metric `prometheus_tsdb_compactions_total` being incremented, don't be confused by that: Prometheus writes initial head block to filesytem via its internal compaction mechanism, but if you have followed recommendations - data won't be modified by Prometheus before the sidecar uploads it. Thanos sidecar will also check sanity of the flags set to Prometheus on the startup and log errors or warning if they have been configured improperly (#838).
3535
* The retention is recommended to not be lower than three times the min block duration, so 6 hours. This achieves resilience in the face of connectivity issues to the object storage since all local data will remain available within the Thanos cluster. If connectivity gets restored the backlog of blocks gets uploaded to the object storage.
3636

3737
## Reloader Configuration
@@ -40,7 +40,7 @@ Thanos can watch changes in Prometheus configuration and refresh Prometheus conf
4040

4141
You can configure watching for changes in directory via `--reloader.rule-dir=DIR_NAME` flag.
4242

43-
Thanos sidecar can watch `--reloader.config-file=CONFIG_FILE` configuration file, evaluate environment variables found in there and produce generated config in `--reloader.config-envsubst-file=OUT_CONFIG_FILE` file.
43+
Thanos sidecar can watch `--reloader.config-file=CONFIG_FILE` configuration file, evaluate environment variables found in there, and produce generated config in `--reloader.config-envsubst-file=OUT_CONFIG_FILE` file.
4444

4545

4646
## Example basic deployment
@@ -83,7 +83,7 @@ Flags:
8383
--log.format=logfmt Log format to use.
8484
--tracing.config-file=<tracing.config-yaml-path>
8585
Path to YAML file that contains tracing
86-
configuration. See fomrat details:
86+
configuration. See format details:
8787
https://thanos.io/tracing.md/#configuration
8888
--tracing.config=<tracing.config-yaml>
8989
Alternative to 'tracing.config-file' flag.

docs/components/store.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ Flags:
4242
--log.format=logfmt Log format to use.
4343
--tracing.config-file=<tracing.config-yaml-path>
4444
Path to YAML file that contains tracing
45-
configuration. See fomrat details:
45+
configuration. See format details:
4646
https://thanos.io/tracing.md/#configuration
4747
--tracing.config=<tracing.config-yaml>
4848
Alternative to 'tracing.config-file' flag.

pkg/shipper/shipper.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ func (c *lazyOverlapChecker) IsOverlapping(ctx context.Context, newMeta tsdb.Blo
247247
// Sync performs a single synchronization, which ensures all non-compacted local blocks have been uploaded
248248
// to the object bucket once.
249249
//
250-
// If updload
250+
// If uploaded.
251251
//
252252
// It is not concurrency-safe, however it is compactor-safe (running concurrently with compactor is ok)
253253
func (s *Shipper) Sync(ctx context.Context) (uploaded int, err error) {

0 commit comments

Comments
 (0)