Skip to content

Automated SDK generation @ aws-cloudformation-user-guide 19dc52cd3f2007d6d268b65b739ffb5ebf8c1e76 - #3131

Merged
pulumi-bot merged 1 commit into
masterfrom
create-pull-request/generate-sdk
Aug 24, 2026
Merged

Automated SDK generation @ aws-cloudformation-user-guide 19dc52cd3f2007d6d268b65b739ffb5ebf8c1e76#3131
pulumi-bot merged 1 commit into
masterfrom
create-pull-request/generate-sdk

Conversation

@pulumi-bot

Copy link
Copy Markdown
Contributor

Automated PR

@pulumi-bot pulumi-bot added the needs-release/minor When a PR with this label merges, it initiates a release of vX.Y+1.0 label Aug 24, 2026
@pulumi-bot
pulumi-bot enabled auto-merge August 24, 2026 04:56
@eon-pulumi

eon-pulumi Bot commented Aug 24, 2026

Copy link
Copy Markdown

Review Verdict: Approved

Mechanical, uniform SDK regeneration confined entirely to generated/vendored paths (sdk/, schema.json, metadata.json, aws-cloudformation-schema/); confirmed the two flagged breaking changes trace to legitimate upstream CFN schema updates rather than a codegen defect, and independently verified the Go/.NET/Node.js/Python SDKs all build cleanly. CI is green on every completed check with no failures. Automated low-risk assessment, not a human review.


View session · Was this review helpful? Yes · No

@github-actions

Copy link
Copy Markdown
Contributor

Does the PR have any schema changes?

Generated by schema-tools v0.8.1.

Found 2 breaking changes:

Resources

  • 🔴 "aws-native:bedrockagentcore:PaymentConnector":
    • required:
      • 🔴 "credentialProviderConfigurations" property is no longer Required
  • 🔴 "aws-native:bedrockagentcore:Runtime":
    • required:
      • 🔴 "networkConfiguration" property is no longer Required

New resources:

  • agentregistry.Registry
  • agentregistry.RegistryRecord
  • amplify.Webhook
  • appconfig.ExperimentDefinition
  • appsync.Type
  • cases.Case
  • cloudformation.ChangeSet
  • codebuild.SourceCredential
  • codecommit.Repository
  • codeconnections.Host
  • connect.Metric
  • dataexchange.EventAction
  • dax.ParameterGroup
  • deadline.Budget
  • deadline.Worker
  • devicefarm.Upload
  • drs.SourceNetwork
  • dynamodb.Backup
  • eks.CertificateAuthority
  • glue.Session
  • guardduty.CustomDetectionRuleAssociation
  • iot.Stream
  • iotsecuretunneling.Tunnel
  • kendra.QuerySuggestionsBlockList
  • kendra.Thesaurus
  • lambda.ResourcePolicy
  • mediatailor.PrefetchSchedule
  • mgn.NetworkMigrationDefinition
  • omics.RunCache
  • personalize.MetricAttribution
  • quicksight.LimitsProfile
  • rds.ClusterSnapshot
  • rds.DbSnapshot
  • route53.RecordSet
  • sdb.Domain
  • ses.ReceiptFilter
  • textract.Adapter
  • translate.ParallelData

New functions:

  • agentregistry.getRegistry
  • agentregistry.getRegistryRecord
  • amplify.getWebhook
  • appconfig.getExperimentDefinition
  • appsync.getType
  • cases.getCase
  • cloudformation.getChangeSet
  • codebuild.getSourceCredential
  • codecommit.getRepository
  • codeconnections.getHost
  • connect.getMetric
  • dataexchange.getEventAction
  • dax.getParameterGroup
  • deadline.getBudget
  • deadline.getWorker
  • devicefarm.getUpload
  • drs.getSourceNetwork
  • dynamodb.getBackup
  • eks.getCertificateAuthority
  • glue.getSession
  • guardduty.getCustomDetectionRuleAssociation
  • iot.getStream
  • iotsecuretunneling.getTunnel
  • kendra.getQuerySuggestionsBlockList
  • kendra.getThesaurus
  • lambda.getResourcePolicy
  • mediatailor.getPrefetchSchedule
  • mgn.getNetworkMigrationDefinition
  • omics.getRunCache
  • personalize.getMetricAttribution
  • quicksight.getLimitsProfile
  • rds.getClusterSnapshot
  • rds.getDbSnapshot
  • route53.getRecordSet
  • sdb.getDomain
  • ses.getReceiptFilter
  • textract.getAdapter
  • translate.getParallelData

New types: 184

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.42%. Comparing base (9edee96) to head (aa4623b).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3131   +/-   ##
=======================================
  Coverage   60.42%   60.42%           
=======================================
  Files          58       58           
  Lines        8157     8157           
=======================================
  Hits         4929     4929           
  Misses       2870     2870           
  Partials      358      358           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eon-pulumi-agent eon-pulumi-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed this automated SDK regeneration PR (bump to aws-cloudformation-user-guide @ 19dc52cd3f2007d6d268b65b739ffb5ebf8c1e76).

Scope verification: confirmed the entire 1028-file diff is confined to generated/vendored artifacts per this repo's AGENTS.md — sdk/{nodejs,python,dotnet,go}/**, provider/cmd/pulumi-resource-aws-native/{schema,metadata}.json, provider/cmd/pulumi-gen-aws-native/supported-types.txt, aws-cloudformation-schema/**, meta/.botocore.version, and reports/missedAutonaming.json. No hand-written files under provider/pkg/** or examples/** were touched, and no forbidden files (.github/workflows/**, .config/mise.toml) were modified.

Breaking changes: the schema-diff bot flagged two required-property relaxations (bedrockagentcore:PaymentConnector.credentialProviderConfigurations, bedrockagentcore:Runtime.networkConfiguration). Traced both directly to the upstream CloudFormation resource schema changes in this diff (new ProvisionMode/QUICK_CREATE flow for PaymentConnector, and NetworkConfiguration removed from the upstream required list for Runtime) — these are legitimate upstream schema changes carried through by the regeneration, not a codegen defect.

Specialist passes (correctness, security, AGENTS.md compliance, tests) all returned no findings:

  • Correctness: validated schema.json/metadata.json integrity (no new dangling refs), and confirmed the Go, .NET, Node.js, and Python SDKs all build/typecheck/import cleanly including the newly added resource packages (agentregistry, bedrockagentcore, deadline, personalize, translate, etc.).
  • Security: no hardcoded secrets, no new unmarked-secret regressions beyond this repo's existing (pre-existing) convention for credential-like resource properties.
  • AGENTS.md compliance: no hand-edits to generated/forbidden paths; all changed files match expected generation-pipeline outputs.
  • Tests: no gap relative to this repo's established pattern for this PR category — prior "Automated SDK generation" PRs (#3122, #3107) followed the same pure-regeneration shape and relied on the same CI build/typecheck jobs plus generic schema-driven checks (e.g. TestNoUnexpectedTagsShapes) rather than per-resource example tests.

CI: all checks that have completed are green (lint, prerequisites, build_sdks for java/nodejs/python/dotnet, codecov, warn_codegen); build_sdks (go) was still in progress at review time with no failures observed anywhere.

This matches the established, repeatedly-merged pattern for this PR class (precedent: #3122, #3114, #3107, #3084, #3073), so I'm treating it as a low-risk, uniform, mechanical regeneration.

@pulumi-bot
pulumi-bot merged commit d729621 into master Aug 24, 2026
21 checks passed
@pulumi-bot
pulumi-bot deleted the create-pull-request/generate-sdk branch August 24, 2026 05:23
@pulumi-bot

Copy link
Copy Markdown
Contributor Author

This PR has been shipped in release v1.76.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-release/minor When a PR with this label merges, it initiates a release of vX.Y+1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant