Skip to content

Conversation

@rix0rrr
Copy link
Contributor

@rix0rrr rix0rrr commented Jul 2, 2025

I did not want to bump the version number previously because the CLI did not yet act on the new definitions found inside the schema, and users would get the error message:

This CDK CLI is not compatible with the CDK library used by your application. Please upgrade the CLI to the latest version.

For no benefit.

This was a mistake, because we use jsonschema to validate the contents of the manifest file and now old versions of the CLI try and validate new files using the old schema, and instead they get the error:

Invalid assembly manifest:
instance.artifacts.aws-cdk-lib/feature-flag-report.type is not one of enum values: none,aws:cloudformation:stack,cdk:tree,cdk:asset-manifest,cdk:cloud-assembly
instance.artifacts.aws-cdk-lib/feature-flag-report.properties requires property "templateFile"
instance.artifacts.aws-cdk-lib/feature-flag-report.properties requires property "file"

We will need another mechanism to achieve the goal of not requiring people to upgrade their CLI unnecessarily; holding back the version number is just exchanging one error message for a worse one.

Belatedly bump the version number here, so that we can release this and reapply this change: aws/aws-cdk#34880


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

I did not want to bump the version number previously because the CLI did
not yet *act* on the new definitions found inside the schema, and users
would get the error message:

```
This CDK CLI is not compatible with the CDK library used by your application. Please upgrade the CLI to the latest version.
```

For no benefit.

This was a mistake, because we use `jsonschema` to validate the contents
of the manifest file and now old versions of the CLI try and validate
new files using the old schema, and instead they get the error:

```
Invalid assembly manifest:
instance.artifacts.aws-cdk-lib/feature-flag-report.type is not one of enum values: none,aws:cloudformation:stack,cdk:tree,cdk:asset-manifest,cdk:cloud-assembly
instance.artifacts.aws-cdk-lib/feature-flag-report.properties requires property "templateFile"
instance.artifacts.aws-cdk-lib/feature-flag-report.properties requires property "file"
```

We will need another mechanism to achieve the goal of not requiring
people to upgrade their CLI unnecessarily; holding back the version
number is just exchanging one error message for a worse one.

Belatedly bump the version number here, so that we can release this
and reapply this change: aws/aws-cdk#34880
@rix0rrr rix0rrr requested a review from a team July 2, 2025 08:16
@rix0rrr rix0rrr temporarily deployed to integ-approval July 2, 2025 08:16 — with GitHub Actions Inactive
@aws-cdk-automation aws-cdk-automation requested a review from a team July 2, 2025 08:17
@github-actions github-actions bot added the p2 label Jul 2, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jul 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.94%. Comparing base (3b4d18e) to head (61c5f61).
⚠️ Report is 64 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #677   +/-   ##
=======================================
  Coverage   78.94%   78.94%           
=======================================
  Files          46       46           
  Lines        7102     7102           
  Branches      794      794           
=======================================
  Hits         5607     5607           
  Misses       1476     1476           
  Partials       19       19           
Flag Coverage Δ
suite.unit 78.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aws-cdk-automation aws-cdk-automation added this pull request to the merge queue Jul 2, 2025
Merged via the queue into main with commit 6735323 Jul 2, 2025
49 of 51 checks passed
@aws-cdk-automation aws-cdk-automation deleted the huijbers/schema-force-bump branch July 2, 2025 09:04
@rubenfonseca
Copy link

Can you please release cdk-assets? Right now with the latest 4.0.1 version we can't deploy schema 45.0.0

@dil-rahul
Copy link

Can you please release cdk-assets? Right now with the latest 4.0.1 version we can't deploy schema 45.0.0

same issue i am facing while deploying pipeline getting error like : Error: Cannot read asset manifest 'assembly-ckeditorPipeline-Deploy-dev-us-west-2/ckeditorPipelineDeploydevuswest2CKeditorStack72A7D191.assets.json': Cloud assembly schema version mismatch: Maximum schema version supported is 44.x.x, but found 45.0.0

@dil-rahul
Copy link

error with cdk assembly versions

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants