Skip to content

Conversation

danielblando
Copy link
Contributor

@danielblando danielblando commented Sep 4, 2024

Please merge this PR with merge commit, do not squash

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@danielblando danielblando force-pushed the merge-release-1.18-to-master branch from 6d93dbe to 35b2dfa Compare September 4, 2024 02:37
@danielblando danielblando force-pushed the merge-release-1.18-to-master branch from 35b2dfa to 4619723 Compare September 4, 2024 02:41
@danielblando danielblando marked this pull request as ready for review September 4, 2024 02:41
* [CHANGE] QueryFrontend/QueryScheduler: Deprecate `-querier.max-outstanding-requests-per-tenant` and `-query-scheduler.max-outstanding-requests-per-tenant` flags. Use frontend.max-outstanding-requests-per-tenant instead. #6146
* [CHANGE] Ingesters: Enable 'snappy-block' compression on ingester clients by default. #6148
* [CHANGE] Ruler: Scheduling `ruler.evaluation-delay-duration` to be deprecated. Use the highest value between `ruler.evaluation-delay-duration` and `ruler.query-offset` #6149
* [CHANGE] Ruler: Scheduling `ruler.evaluation-delay-duration` to be deprecated. Ruler will use the highest value between `ruler.evaluation-delay-duration` and `ruler.query-offset` #6149
Copy link
Contributor

Choose a reason for hiding this comment

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

If we need to modify changelog content let's update in the release branch first then merge back to master?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These changes are already on the release branch. We are just bringing it to master now
https://github.com/cortexproject/cortex/blob/release-1.18/CHANGELOG.md?plain=1#L15

Copy link
Contributor

Choose a reason for hiding this comment

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

I see. Thanks for the clarification

@danielblando danielblando merged commit 4fc1828 into cortexproject:master Sep 4, 2024
16 checks passed
@danielblando danielblando deleted the merge-release-1.18-to-master branch September 4, 2024 03:05
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.

2 participants