-
-
Notifications
You must be signed in to change notification settings - Fork 411
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
release: v2023.3 #1744
release: v2023.3 #1744
Conversation
Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
WalkthroughThis update introduces enhanced Docker image tagging in CI/CD, expands configuration options for various services in Changes
TipsChat with CodeRabbit Bot (
|
Qodana for GoIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 1
Files ignored due to filter (3)
- go.mod
- go.sum
- schemas/config/3.0.schema.json
Files selected for processing (4)
- .github/workflows/release.yml (1 hunks)
- .rr.yaml (7 hunks)
- CHANGELOG.md (5 hunks)
- container/plugins.go (1 hunks)
Files skipped from review due to trivial changes (2)
- .github/workflows/release.yml
- container/plugins.go
Additional comments (Suppressed): 5
.rr.yaml (2)
- 573-600: The new hunk introduces OpenTelemetry resource attributes for service name, version, namespace, and instance ID. Ensure that these values are correctly set as per your application's requirements. The default value for
service_instance_id
is a generated UUID, but it's currently set to "UUID" which might not be unique.# Default: Generated UUID service_instance_id: "UUID" service_instance_id: "<Generated or Unique UUID>"
- 2164-2167: These lines introduce a new configuration option
stream_timeout
for stream cancellation. The comment says "Zero means 60s", but the default value is also set to 60s. If zero is intended to represent a different behavior (like no timeout), this could lead to confusion. Clarify the purpose of zero and ensure the default value aligns with the intended behavior.CHANGELOG.md (3)
3-18: The new hunk introduces the features and improvements in version v2023.3.0 of the RoadRunner server. The changes are well-documented with links to the relevant pull requests or issues, and contributors are acknowledged. No issues found.
22-35: > Note: This review was outside of the patch, so it was mapped to the patch with the greatest overlap. Original lines [22-46]
This hunk continues the list of features and improvements in different plugins like JOBS plugin, Temporal plugin, Service plugin, and HTTP plugin. Each feature is linked to its corresponding feature request (FR) or issue on GitHub, providing traceability. No issues found.
- 154-165: This hunk lists the bug fixes and chores done in this release. Each fix is linked to its corresponding pull request (PR) on GitHub, providing traceability. The chores section mentions the update of Golang version and project dependencies. No issues found.
Codecov Report
@@ Coverage Diff @@
## master #1744 +/- ##
=======================================
Coverage 37.12% 37.12%
=======================================
Files 16 16
Lines 862 862
=======================================
Hits 320 320
Misses 516 516
Partials 26 26
|
Reason for This PR
Description of Changes
🚀 v2023.3.0 🚀
🔥 Features:
RR Core:
sdnotify
support: FR, (thanks @Kaspiman), Docs: link👀 JOBS plugin:
routing_key
in the JOBS payload: FR, (thanks @rauanmayemir)this feature should significantly reduce RR startup/shutdown time: FR, (thanks @Kaspiman)
👀 KV drivers (all):
👀 App-Logger plugin:
👀 Temporal plugin:
👀 Service plugin:
Configuration example:
👀 HTTP plugin:
Worker example:
103
Early Hints via streamed response: FR, (thanks @azjezz)Worker example:
👀 Server plugin:
.rr.yaml
configuration: FR, (thanks @nunomaduro)First argument should be a command (executable) and the rest of the arguments are passed to the command as arguments.
🩹 Fixes:
./rr jobs list/stop/resume
commands: PR, (thanks @gam6itko).🧹 Chore:
Summary by CodeRabbit
Release Notes:
routing_key
in AMQP driver.sdnotify
support in RR Core.