Skip to content

Conversation

@rix0rrr
Copy link
Contributor

@rix0rrr rix0rrr commented Jul 2, 2025

aws-cdk-lib 2.203.0 gives an incorrect error message when used with AWS CLI 2.2017.0 and 2.2018.0. The error message should have been:

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

Instead, the error message is:

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
...

This reverts the change that introduces that new artifact type: #34798

@rix0rrr rix0rrr requested a review from a team as a code owner July 2, 2025 08:09
@aws-cdk-automation aws-cdk-automation requested a review from a team July 2, 2025 08:09
@github-actions github-actions bot added the p2 label Jul 2, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Jul 2, 2025
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

rix0rrr added a commit to aws/aws-cdk-cli that referenced this pull request 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
@rix0rrr rix0rrr added the pr-linter/exempt-integ-test The PR linter will not require integ test changes label Jul 2, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review July 2, 2025 08:19

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 82579ff
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify
Copy link
Contributor

mergify bot commented Jul 2, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 741b0a7 into main Jul 2, 2025
42 checks passed
@mergify mergify bot deleted the revert-34798-feature-flag-report branch July 2, 2025 08:40
@github-actions
Copy link
Contributor

github-actions bot commented Jul 2, 2025

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2 pr-linter/exempt-integ-test The PR linter will not require integ test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants