Skip to content

chore: sync from internal (rie-private-to-public) - #189

Merged
roger-zhangg merged 1 commit into
developfrom
rie-private-to-public
Sep 21, 2026
Merged

roger-zhangg merged 1 commit into
developfrom
rie-private-to-public

Conversation

@aws-sam-tooling-bot

Copy link
Copy Markdown
Contributor

Automated sync from internal LambdaRIEGithubSource.

The sync branch is reset to develop on every run, so this PR is always a single-commit diff against the current develop with no accumulated divergence. Force-pushed by the pipeline — do not commit to this branch by hand.

Comment thread CONTRIBUTING.md
* **If we adopt your change, we credit you in the release notes** for the version that ships it.

To send us a pull request, please:
So that we can act on a pull request, please:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This sentence opening seems abrupt in continuous reading. Let's have something that is direct sentence. For example:
"To help us act on your pull request, please:"
or
"When submitting a pull request, please:"

Comment thread CONTRIBUTING.md
4. Commit to your fork using clear commit messages.
5. Send us a pull request, answering any default questions in the pull request interface.
1. Open an issue first to discuss any significant work -- we would hate for your time to be wasted on something we
cannot take.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Let's use "dislike" instead of "hate"

@roger-zhangg
roger-zhangg merged commit 3ccddd7 into develop Sep 21, 2026
6 checks passed
carole-lavillonniere added a commit to localstack/lambda-runtime-init that referenced this pull request Sep 22, 2026
Brings the fork up to aws/aws-lambda-runtime-interface-emulator@3ccddd7
(Update from upstream - 2026-09-17), covering four upstream syncs:

  3ccddd7 Update from upstream - 2026-09-17 (aws#189)
  c622c06 Update from upstream - 2026-08-25 (aws#187)
  ddf2bc4 Update from upstream - 2026-07-31 (aws#182)
  a0f264a chore: sync from internal (rie-private-to-public) (aws#181)

The fork had been on develop@391c3f1 since April, and a0f264a is why this
matters now: it makes raptor.Server.Shutdown drain in-flight responses via
httpServer.Shutdown instead of calling httpServer.Close(), which severed
connections whose handler was still writing. That race is what failed the
weekly release twice -- as `Post ...: EOF` in TestRie_InvokeFatalError, and as
a 10m package timeout in TestRie_InvokeWaitingForInitError, where the invoke
goroutine's require.NoError skipped its un-deferred wg.Done() and deadlocked
the package. Both tests are green on this merge: 0 failures in 40 runs of the
package under -race, against 10/10 hangs before it.

Conflicts, all in files we customize:

  go.mod                Kept our go 1.27.1 directive over upstream's 1.26.6.
  Makefile              Kept our GO_VERSION derived from go.mod, and the
                        nojsonv2 GOEXPERIMENT export.
  check-binaries.yml    Took upstream's switch from robinraju/release-downloader
  release.yml           to `gh release download`, and from softprops/action-gh-release
                        to `gh release create`; kept our Renovate action bumps
                        (checkout/setup-python v7). release.yml's trailing
                        context had already moved to upstream's shape, so
                        keeping ours there would not have parsed.
  integ-tests.yml       Version bumps only; kept ours.

Note for the next integration: the previous one (#92) landed as a single
squashed commit, so its upstream parent was lost and git computed the merge
base as the Go 1.24 bump from 2025 -- 48 conflicts, 36 of them spurious
add/add on files that were already identical. This was resolved against the
true base (391c3f1) using a local `git replace --graft`, giving the 5 real
conflicts above. Merging this one with --ff, per README-LOCALSTACK step 7,
records the upstream parent and stops the next integration needing that.
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.

2 participants