-
Notifications
You must be signed in to change notification settings - Fork 196
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
Updates dapr/dapr to v1.12 #1342
Updates dapr/dapr to v1.12 #1342
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## release-1.12 #1342 +/- ##
================================================
- Coverage 22.95% 22.82% -0.13%
================================================
Files 40 40
Lines 4697 4701 +4
================================================
- Hits 1078 1073 -5
- Misses 3542 3550 +8
- Partials 77 78 +1
☔ View full report in Codecov by Sentry. |
911c2fd
to
4c18002
Compare
go.mod
Outdated
k8s.io/api v0.26.3 | ||
k8s.io/apiextensions-apiserver v0.26.3 | ||
k8s.io/apimachinery v0.26.3 | ||
k8s.io/cli-runtime v0.26.3 | ||
k8s.io/client-go v0.26.3 |
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.
We still want to support 26.2 for this release can be updated in the next release . There is an issue tracking it.
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.
I have pushed a commit to update the runtime to rc.2 and have reverted the k8s package update in this commit - a139d39
let me know what you think ? @JoshVanL @mukundansundar
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.
@pravinpushkar please try not to push to authors branches, and instead favor code review or PRing to their branch 🙂
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.
@JoshVanL Sure, thought you will not be able to look into it until my night time. So, was only trying to fast pace the process for RC cut.
Should we include the upgrade path changes to this PR as well ? We also need to point it to runtime RC.2. |
Signed-off-by: joshvanl <me@joshvanl.dev>
360c8cf
to
f6c8778
Compare
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
* Updates the Dapr version to v1.12.0-rc.1 Signed-off-by: joshvanl <me@joshvanl.dev> * Use correct rc tag name Signed-off-by: joshvanl <me@joshvanl.dev> * Fix string match on `HTTP server is running on port x` Signed-off-by: joshvanl <me@joshvanl.dev> * Fix error string check for `internal gRPC server` Signed-off-by: joshvanl <me@joshvanl.dev> * Update dapr/dapr to 1.12.0-rc.3 Signed-off-by: joshvanl <me@joshvanl.dev> * Rolling restart the sidecar injector during a restart Signed-off-by: joshvanl <me@joshvanl.dev> * Linting Signed-off-by: joshvanl <me@joshvanl.dev> * Fix string matching on dapr app output logs Signed-off-by: joshvanl <me@joshvanl.dev> * Increase e2e tests `20m` -> `25m` Signed-off-by: joshvanl <me@joshvanl.dev> --------- Signed-off-by: joshvanl <me@joshvanl.dev> Signed-off-by: Nikita Sarawgi <sarawgi.nikita@gmail.com>
Updates dapr/dapr to
1.12
PR currently points to the unreleased tag v1.12.0-rc2
/cc @mukundansundar