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.

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

Docs: Add changelog entry for configuration parameter name change #6340

Merged

Conversation

KMiller-Grafana
Copy link
Contributor

PR #6120 renamed a configuration parameter. I forgot to include a changelog entry with that PR. This PR adds that changelog entry.

@KMiller-Grafana KMiller-Grafana requested a review from a team as a code owner June 8, 2022 16:22
Copy link
Contributor

@DylanGuedes DylanGuedes left a comment

Choose a reason for hiding this comment

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

small nit

CHANGELOG.md Outdated
@@ -81,6 +81,7 @@
* [5685](https://github.com/grafana/loki/pull/5685) **chaudum**: Assert that push values tuples consist of string values
##### Changes
* [6042](https://github.com/grafana/loki/pull/6042) **slim-bean**: Add a new configuration to allow fudging of ingested timestamps to guarantee sort order of duplicate timestamps at query time.
* [6120](https://github.com/grafana/loki/pull/6120) **KMiller-Grafana**: Rename the fudging of ingested timestamps configuration parameter introduced in PR 6042 to avoid using the slang term "fudge."
Copy link
Contributor

@DylanGuedes DylanGuedes Jun 8, 2022

Choose a reason for hiding this comment

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

I think just saying the change will be more pleasing to end-users. In case they'd like to understand the reasoning behind the change they can always look for the PR description.

Suggested change
* [6120](https://github.com/grafana/loki/pull/6120) **KMiller-Grafana**: Rename the fudging of ingested timestamps configuration parameter introduced in PR 6042 to avoid using the slang term "fudge."
* [6120](https://github.com/grafana/loki/pull/6120) **KMiller-Grafana**: Rename fudge_duplicate_timestamp parameter to increment_duplicate_timestamp"

@grafanabot
Copy link
Collaborator

./tools/diff_coverage.sh ../loki-main/test_results.txt test_results.txt ingester,distributor,querier,querier/queryrange,iter,storage,chunkenc,logql,loki

Change in test coverage per package. Green indicates 0 or positive change, red indicates that test coverage for a package fell.

+           ingester	0%
+        distributor	0%
+            querier	0%
+ querier/queryrange	0.1%
+               iter	0%
+            storage	0%
+           chunkenc	0%
+              logql	0%
+               loki	0%

@grafanabot
Copy link
Collaborator

./tools/diff_coverage.sh ../loki-main/test_results.txt test_results.txt ingester,distributor,querier,querier/queryrange,iter,storage,chunkenc,logql,loki

Change in test coverage per package. Green indicates 0 or positive change, red indicates that test coverage for a package fell.

+           ingester	0%
-        distributor	-0.3%
+            querier	0%
+ querier/queryrange	0%
+               iter	0%
+            storage	0%
+           chunkenc	0%
+              logql	0%
+               loki	0%

@KMiller-Grafana KMiller-Grafana merged commit e725a21 into grafana:main Jun 8, 2022
@KMiller-Grafana KMiller-Grafana deleted the docs/add-changelog-entry branch June 8, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants