-
Notifications
You must be signed in to change notification settings - Fork 151
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
Upgrade upstream to v4.57.0 #2408
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Does the PR have any schema changes?Found 6 breaking changes: New resources:
New functions:
|
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.
Looks sound; I will have a look at the rebased upstream/, too. The breaking changes reported will need chasing down, to make sure they are expected.
My comments here are mainly queries about what's going on -- it's hard to understand the changes to replacements.json (though I prefer it to the prior state of the art!).
@@ -4893,35 +4865,8 @@ | |||
{ | |||
"old": "-> **NOTE:** The parameter `server_side_encryption_configuration` is deprecated.\nUse the resource [`aws_s3_bucket_server_side_encryption_configuration`](s3_bucket_server_side_encryption_configuration.html) instead.\n" | |||
}, | |||
{ |
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.
This looks really similar (and in the same place, I think, under the key "website/docs/r/s3_bucket.html.markdown"
) to a replacement that was added in the previous commit. I'm not clear what this means -- does the text get changed now, and not removed?
Issue investigation:
Investigating if there's a fix for the first two. |
0d847ce
to
9ef9e01
Compare
Could you explain the justification for the MaxItemsOne reversion in the commit message, please. (I'd include a link to the breaking changes policy, too.) As recently as this particular update, a (rebased) commit that didn't come with an explanation held us up until we could chase down why it was done. This change is less likely to cause difficulties during a rebase specifically, but there are other situations in which having the explanation with the change avoids hold-ups. |
Does the PR have any schema changes?Found 4 breaking changes: New resources:
New functions:
|
Revert MaxItems changes in EC2 Fleet
launchTemplateConfig
and ECSclientAlias
to maintain SDK backward compatibility as per our backward compatibility guidelines: