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

Merge release 1.15 into master #1499

Merged
merged 14 commits into from
Mar 14, 2025

Conversation

antontroshin
Copy link
Contributor

Description

merge release-1.15 work into master

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

yaron2 and others added 14 commits January 21, 2025 22:52
…pr#1480)

* Scheduler: set broadcast address to localhost:50006 in selfhosted

Signed-off-by: joshvanl <[email protected]>

* Set schedulder override flag for edge and dev

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
* remove deprecated flags

Signed-off-by: yaron2 <[email protected]>

* update Dapr version in tests

Signed-off-by: yaron2 <[email protected]>

---------

Signed-off-by: yaron2 <[email protected]>
…pr#1486)

The issue found when similar resource were installed in k8s that use the name "configurations".
In this case the knative's "configurations.serving.knative.dev/v1" was the last in the list and the command returned the error
`Error from server (NotFound): configurations.serving.knative.dev "daprsystem" not found`

Signed-off-by: Anton Troshin <[email protected]>
* fix: arguments accept units
`max-body-size` and `read-buffer-size` now accept units as defined in the docs.

Fixes dapr#1489

Signed-off-by: Mike Nguyen <[email protected]>

* chore: gofumpt

Signed-off-by: Mike Nguyen <[email protected]>

* refactor: modify logic to comply with vetting

Signed-off-by: Mike Nguyen <[email protected]>

* chore: gofumpt -w .

Signed-off-by: Mike Nguyen <[email protected]>

* refactor: set defaults
`max-body-size` is defaulted to 4Mi
`request-buffer-size` is defaulted to 4Ki

This is inline with the runtime.

Signed-off-by: Mike Nguyen <[email protected]>

* fix: set defaults in run and annotate

Signed-off-by: Mike Nguyen <[email protected]>

* chore: gofumpt

Signed-off-by: Mike Nguyen <[email protected]>

* refactor: exit with error rather than panic

Co-authored-by: Anton Troshin <[email protected]>
Signed-off-by: Mike Nguyen <[email protected]>

---------

Signed-off-by: Mike Nguyen <[email protected]>
Co-authored-by: Anton Troshin <[email protected]>
* Fix podman CI
Update to podman 5.4.0

Signed-off-by: Anton Troshin <[email protected]>

* fix --cpus flag

Signed-off-by: Anton Troshin <[email protected]>

---------

Signed-off-by: Anton Troshin <[email protected]>
…n 1.15 (dapr#1494)

* Fix dapr upgrade command detecting HA mode for new version 1.15
The issue is that the scheduler by default uses 3 replicas, which incorrectly identified non-HA install as HA.

Signed-off-by: Anton Troshin <[email protected]>

* Fix e2e

Signed-off-by: Anton Troshin <[email protected]>

---------

Signed-off-by: Anton Troshin <[email protected]>
* release: test upgrade/downgrade for 1.13/1.14/1.15 + mariner

Signed-off-by: Mike Nguyen <[email protected]>

* fix: version skews

Co-authored-by: Anton Troshin <[email protected]>
Signed-off-by: Mike Nguyen <[email protected]>

* Update tests/e2e/upgrade/upgrade_test.go

Accepted

Co-authored-by: Anton Troshin <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>

* Update tests/e2e/upgrade/upgrade_test.go

Co-authored-by: Anton Troshin <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>

* Fix downgrade issue from 1.15 by deleting previous version scheduler pods
Update 1.15 RC to latest RC.18

Signed-off-by: Anton Troshin <[email protected]>

* Fix downgrade 1.15 to 1.13 scenario with 0 scheduler pods

Signed-off-by: Anton Troshin <[email protected]>

* increase update test timeout to 60m and update latest version to 1.15

Signed-off-by: Anton Troshin <[email protected]>

* fix httpendpoint tests cleanup and checks

Signed-off-by: Anton Troshin <[email protected]>

* make sure matrix runs appropriate tests, every matrix ran the same tests

Signed-off-by: Anton Troshin <[email protected]>

* skip TestKubernetesRunFile on HA

Signed-off-by: Anton Troshin <[email protected]>

* fix skip TestKubernetesRunFile on HA

Signed-off-by: Anton Troshin <[email protected]>

* update to latest dapr 1.15.2

Signed-off-by: Anton Troshin <[email protected]>

* add logs when waiting for pod deletion

Signed-off-by: Anton Troshin <[email protected]>

---------

Signed-off-by: Mike Nguyen <[email protected]>
Signed-off-by: Yaron Schneider <[email protected]>
Signed-off-by: Anton Troshin <[email protected]>
Co-authored-by: Anton Troshin <[email protected]>
Co-authored-by: Yaron Schneider <[email protected]>
Co-authored-by: Anton Troshin <[email protected]>
* Fix goroutine channel leaks and ensure proper cleanup in tests

Signed-off-by: Anton Troshin <[email protected]>

* Add artificial delay before deleting scheduler pods during downgrade

Signed-off-by: Anton Troshin <[email protected]>

* Add timeout to helm upgrade tests, they are being stuck sometime for 5+ minutes

Signed-off-by: Anton Troshin <[email protected]>

* bump helm.sh/helm/v3 to v3.17.1

Signed-off-by: Anton Troshin <[email protected]>

---------

Signed-off-by: Anton Troshin <[email protected]>
@antontroshin antontroshin marked this pull request as ready for review March 14, 2025 01:10
@antontroshin antontroshin requested review from a team as code owners March 14, 2025 01:10
@yaron2 yaron2 merged commit 29f8962 into dapr:master Mar 14, 2025
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants