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

Upgrade upstream to v4.57.0 #2408

Merged
merged 6 commits into from
Mar 10, 2023
Merged

Conversation

danielrbradley
Copy link
Member

@danielrbradley danielrbradley commented Mar 9, 2023

Revert MaxItems changes in EC2 Fleet launchTemplateConfig and ECS clientAlias to maintain SDK backward compatibility as per our backward compatibility guidelines:

https://github.com/pulumi/platform-providers-team/blob/main/playbooks/maxitemsone.md

[...] For non-major version upgrades of Tier 1 providers, we will typically err on the side of keeping MaxItems: 1 to avoid breaking our users' code even if it is an unnecessary restriction [...]

@danielrbradley danielrbradley requested a review from a team March 9, 2023 17:12
@danielrbradley danielrbradley self-assigned this Mar 9, 2023
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions
Copy link

github-actions bot commented Mar 9, 2023

Does the PR have any schema changes?

Found 6 breaking changes:
Resource "aws:ec2/fleet:Fleet" missing input "launchTemplateConfig"
Resource "aws:ec2/fleet:Fleet" missing output "launchTemplateConfig"
Type "aws:ecs/ServiceServiceConnectConfigurationService:ServiceServiceConnectConfigurationService" missing property "clientAliases"
Type "aws:ecs/ServiceServiceConnectConfiguration:ServiceServiceConnectConfiguration" missing property "service"
Type "aws:autoscaling/PolicyPredictiveScalingConfigurationMetricSpecification:PolicyPredictiveScalingConfigurationMetricSpecification" input "targetValue" type changed from "integer" to "number"
Type "aws:appflow/FlowSourceFlowConfigSourceConnectorPropertiesSapoData:FlowSourceFlowConfigSourceConnectorPropertiesSapoData" missing property "object"

New resources:

  • ec2/vpcIpamResourceDiscovery.VpcIpamResourceDiscovery
  • ec2/vpcIpamResourceDiscoveryAssociation.VpcIpamResourceDiscoveryAssociation
  • lightsail/bucketResourceAccess.BucketResourceAccess
  • networkmanager/connectPeer.ConnectPeer

New functions:

  • organizations/getOrganizationalUnitChildAccounts.getOrganizationalUnitChildAccounts
  • organizations/getOrganizationalUnitDescendantAccounts.getOrganizationalUnitDescendantAccounts

Copy link
Contributor

@squaremo squaremo left a 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!).

.github/workflows/run-acceptance-tests.yml Outdated Show resolved Hide resolved
upstream-tools/replacements.json Show resolved Hide resolved
upstream-tools/replacements.json Show resolved Hide resolved
@@ -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"
},
{
Copy link
Contributor

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?

@danielrbradley
Copy link
Member Author

danielrbradley commented Mar 10, 2023

Issue investigation:

  • EC2 Fleet launchTemplateConfig has changed from being exactly 1 element to being between 0 and 50 elements. This causes our property to be renamed to a plural. Resolution: reinstateMaxItemsOne: true, so the type and field name don't change and break user code.
  • ECS ServiceServiceConnectConfigurationService clientAlias has now added maxItems 1, but optional - so no more pluralisation. Resolution: remove MaxItemsOne: true, so the type and field name don't change
  • Autoscaling PolicyPredictiveScalingConfigurationMetricSpecification targetValue has had its type changed upstream from int to float. Resolution: leave it, on the basis that it's low impact.
  • AppFlow FlowSourceFlowConfigSourceConnectorPropertiesSapoData object property has been renamed to AttrObjectPath. ResolutionL leave it, on the basis that it's low impact.

Investigating if there's a fix for the first two.
(EDIT: made this into a checklist)

@danielrbradley danielrbradley force-pushed the danielrbradley/upstream-v4.57.0 branch from 0d847ce to 9ef9e01 Compare March 10, 2023 11:56
@squaremo
Copy link
Contributor

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.

@danielrbradley danielrbradley enabled auto-merge (squash) March 10, 2023 12:22
@github-actions
Copy link

Does the PR have any schema changes?

Found 4 breaking changes:
Type "aws:appflow/FlowSourceFlowConfigSourceConnectorPropertiesSapoData:FlowSourceFlowConfigSourceConnectorPropertiesSapoData" missing property "object"
Type "aws:ecs/ServiceServiceConnectConfigurationService:ServiceServiceConnectConfigurationService" missing property "clientAliases"
Type "aws:ecs/ServiceServiceConnectConfiguration:ServiceServiceConnectConfiguration" missing property "service"
Type "aws:autoscaling/PolicyPredictiveScalingConfigurationMetricSpecification:PolicyPredictiveScalingConfigurationMetricSpecification" input "targetValue" type changed from "integer" to "number"

New resources:

  • ec2/vpcIpamResourceDiscovery.VpcIpamResourceDiscovery
  • ec2/vpcIpamResourceDiscoveryAssociation.VpcIpamResourceDiscoveryAssociation
  • lightsail/bucketResourceAccess.BucketResourceAccess
  • networkmanager/connectPeer.ConnectPeer

New functions:

  • organizations/getOrganizationalUnitChildAccounts.getOrganizationalUnitChildAccounts
  • organizations/getOrganizationalUnitDescendantAccounts.getOrganizationalUnitDescendantAccounts

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.

None yet

3 participants