Skip to content

Conversation

@sicoyle
Copy link
Contributor

@sicoyle sicoyle commented Sep 9, 2025

Description

TLDR this PR fixes the first part of the commit history corruption going on in contrib. I need a follow up PR in the release branch after this gets merged.

Context:

We have the release-1.16 branch and main. Every PR update grabbing latest from main to the release branch reverted any PR that had only been opened to the release branch. From what I see, at no point did we have a PR from release-1.16 -> main. What this means is that all of the 1.16 release branch specific fixes got reverted on every merge that we did to update the release branch with the latest from main. This PR brings all of the PRs opened / commits only to the release branch back to main.

PRs to release branch ONLY that I am cherry-picking in this PR back into main (with context on how I did it):

TODO: PRs updating release branch to what is sitting in main that I still need to cross check all of the changes are correct and more things are not missing:

Since main is missing PRs, then commits/PRs missing in main must be added first, and then we can have an official PR from main to update release-1.16:

  • TODO I will do this when we merge this PR

Note: there were attempted fixes which is super helpful, but they were missing the full story / all changes to correct this situation

To prevent this in future, one option we could do is to to have a PR open during the release back to main, so that way we capture if a PR is only merged into release branch, then it will for sure be captured back in main.

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
    • Created the dapr/docs PR:

Note: We expect contributors to open a corresponding documentation PR in the dapr/docs repository. As the implementer, you are the best person to document your work! Implementation PRs will not be merged until the documentation PR is opened and ready for review.

@sicoyle sicoyle requested review from a team as code owners September 9, 2025 18:28
filintod and others added 6 commits September 9, 2025 13:33
Signed-off-by: Filinto Duran <1373693+filintod@users.noreply.github.com>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: joshvanl <me@joshvanl.dev>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
Signed-off-by: Mike Nguyen <hey@mike.ee>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
…dapr#4006)

Signed-off-by: Filinto Duran <1373693+filintod@users.noreply.github.com>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: Samantha Coyle <sam@diagrid.io>
@sicoyle
Copy link
Contributor Author

sicoyle commented Sep 9, 2025

working on dco pls give me a sec 🙏

@sicoyle sicoyle force-pushed the update-main-to-release-branch branch from bfa6d92 to be80fe5 Compare September 9, 2025 18:34
@sicoyle
Copy link
Contributor Author

sicoyle commented Sep 9, 2025

did a

git rebase --signoff main
git push --force-with-lease

to make dco build step happy and append my sign-off everywhere

@JoshVanL JoshVanL enabled auto-merge September 9, 2025 18:56
@JoshVanL JoshVanL disabled auto-merge September 9, 2025 18:58
@JoshVanL JoshVanL enabled auto-merge September 9, 2025 18:59
@JoshVanL JoshVanL disabled auto-merge September 9, 2025 19:03
@mikeee mikeee merged commit 0c9dd49 into dapr:main Sep 9, 2025
90 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.

5 participants