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

chore(release): 1.128.0 #16969

Merged
merged 38 commits into from
Oct 14, 2021
Merged

chore(release): 1.128.0 #16969

merged 38 commits into from
Oct 14, 2021

Conversation

aws-cdk-automation
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation commented Oct 14, 2021

See CHANGELOG

jogold and others added 30 commits October 8, 2021 18:00
Add support for Step Functions' AWS SDK integrations to call
any of the over two hundred AWS services directly from a state
machine.

See https://docs.aws.amazon.com/step-functions/latest/dg/supported-services-awssdk.html
See https://aws.amazon.com/blogs/aws/now-aws-step-functions-supports-200-aws-services-to-enable-easier-workflow-automation/

Closes #16780 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…6.23 (#16879)

Add new PostgreSQL RDS versions.

Announcement: https://aws.amazon.com/about-aws/whats-new/2021/10/amazon-rds-postgresql-new-minor-versions-amazon-rds-outposts-postgresql-minor-versions/

Features `s3Export` and `s3Import` checked with command `aws rds describe-db-engine-versions --region us-east-1 --engine postgres --engine-version xx.x`.

I changed the end of life date for version 9.6.22 from 2022 to 2021. According to the [PostgreSQL news](https://www.postgresql.org/about/news/postgresql-134-128-1113-1018-9623-and-14-beta-3-released-2277/) and the [Versioning Policy](https://www.postgresql.org/support/versioning/), version 9.6 will reach end of life on November 11, 2021.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Add a `dockerPlatform` property in `Architecture` and use it to pass the
correct `platform` when bundling in a container in `aws-lambda-nodejs`,
`aws-lambda-go` and `aws-lambda-python`.

Note that the SAM build images (`public.ecr.aws/sam/build-<runtime>`) are now
multi-arch.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Update doc to reflect deprecation from #16849

----

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

Rosetta automatic snippet translation is doing a poor
job of snippet translation. This is mainly because the
snippets are not compilable.

Update snippets to make them compilable and translate
correctly.

Additionally, make the rosetta compilation strict and run
during the build step.

There are still a few more translation errors, but these are
arising from bugs in rosetta.
aws/jsii#3026
aws/jsii#3029

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The Amazon Cognito service demands that the user invitation SMS message
must contain both the placeholders '{username}' and '{####}'.

The code example in the README does not and hence is erroneous.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…thentication (#16712)

closes #16710


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
With the commitment to not support breaking changes in stable modules (and all
of aws-cdk-lib), we need to document the standard for introducing APIs we know
to be potentially experimental or unstable.

This standard was originally proposed in https://github.com/aws/aws-cdk-rfcs/blob/master/text/0249-v2-experiments.md, and
was first actually used in #13780.

This task documents the standard in the CONTRIBUTING guide so we can use it as a
reference going forward.

closes #16434


----

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

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
This extends the `cdk deploy --hotswap` command to support ECS Services,
in addition to Lambda Functions and StepFunctions State Machines.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
resoruce -> resource
…#16692)

This change bundles together two changes for how ubergen handles package
READMEs. The first is to generate a base README for experimental/alpha modules
where only the L1s are included. The second is to strip out the stability
banners from *all* READMEs; this is because, by definition, everything included
in aws-cdk-lib is stable, so the banners are now redundant/misleading.

fixes #16567
The maven artifact id is configured to 'cdk-assertions'.

This makes the assertions module naming different from the other package
managers, and hence harder to discover.

Change the artifact id to 'assertions' and make this consistent.

BREAKING CHANGE: Starting this release, the `assertions` module will be
published to Maven with the name 'assertions' instead of
'cdk-assertions'.
…ined (#16907)

The error message is inaccurate and results in a message like -

```
AWS::Foo::Bar.Properties.Baz has known type: {"Documentation":"...","Required":false,"UpdateType":"Immutable"}
```

Improve the message.


----

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

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…g machineImageType (#16361)

fix(aws-ecs): make `Cluster.addAsgCapacityProvider()` not need specify `machineImageType`

close #16360 
----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…ons from assets. (#16914)

This small PR should fix an issue brought up that we used a static `Template` as the resource
name for assets which causes a collision if you have multiple versions.  The correct configuration
should be that the asset name is unique for each unique template file uploaded for a product version.

Fixes: #16892


----

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

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
In CDK v2, the `aws-cdk` package no longer contains an `index.ts` file that exports members on the package level.
Because of that, stop relying on package-level imports in the unit tests of hotswapping,
and instead import the needed classes directly.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
To recreate this issue, create a blank CDK App and add a single Lambda Function to it. Synthesis will fail with the message "Class extends value undefined is not a constructor or null". This is caused by `AnyPrincipal` being imported from `'./'` and other classes from the same file as `AnyPrincipal` (`principals.ts`) being imported from `'./principals'`.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
npm treats registry.npmjs.org as a special value that means 'the current configured package' in package-lock and npm-shrinkwrap. npm will request aws-cdk's dependencies from yarnpkg instead of from the installers configured registry because aws-cdk's shrinkwrap uses yarnpkg. This behavior seems new to [npm v7]. It causes issues for us because we run our builds with a isolated network and a private registry.

[npm v7]: npm/cli#3783

This commit changes the registry from yarnpkg to npmjs. I updated yarn.lock with sed.

```
sed 's|https://registry.yarnpkg.com|https://registry.npmjs.org|' yarn.lock -i
```
Alternatively we could modify the yarn-cling tool to replace the registry. [registry.yarnpkg.com is a cname for registry.npmjs.org](https://yarnpkg.com/getting-started/qa#why-registryyarnpkgcom-does-facebook-track-us) so changing to registry.npmjs.org shouldn't affect available packages or performance.

```
dig registry.yarnpkg.com | rg CNAME
registry.yarnpkg.com.	300	IN	CNAME	yarn.npmjs.org
```

----

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

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Ran npm-check-updates and yarn upgrade to keep the `yarn.lock` file up-to-date.
I just took a cursory glance at some READMEs in my ownership areas and fixed discrepancies in the indentation and some commas at the end of the line. Very minor README changes to `stepfunctions`, `stepfunctions-tasks`, and `synthetics`.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
On occasion, dependabot will create PR's that fix security vulnerabilities. This is regardless of whether we have a dependabot configuration in the repo.

> See "Dependabot security updates" in https://github.com/aws/aws-cdk/settings/security_analysis

Right now, these PR's are not getting auto-approved because they don't container the `auto-approve` label. Unfortunately these PR are not configurable and we cannot add this label. 

As discussed in the team, we decided to forgo the label requirement for these PR and allow dependabot to be merged given all checks pass.

This PR changes the auto approve conditions to allow either one of:

- PR comes from `dependabot[bot]` user.
- PR comes from the `dependabot-preview[bot]` user.
- PR comes from the `aws-cdk-automation` user **AND** PR has the `auto-approve` label.

----

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

Currently, our bump process creates `CHANGELOG.md` for the `master` branch, and
`CHANGELOG.v2.md` and `CHANGELOG.v2.alpha.md` on the `v2-main` branch. For the
latter, we need to combine the two changelogs into one for the GitHub release,
where all of the artifacts of the release (both aws-cdk-lib and all the alpha
modules) are bundled together.

As proposed in the [RFC], this combined format shows the stable/aws-cdk-lib
changes first, and then the alpha module changes under their own heading.  This
generated -- but not saved -- RELEASE_NOTES.md will be used by our publishing
pipeline to generate the GitHub release notes.

[RFC]: https://github.com/aws/aws-cdk-rfcs/blob/master/text/0249-v2-experiments.md#changelog--release-notes

See companion PR - cdklabs/aws-delivlib#1044

fixes #16802

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
The hash for a specific file in a directory include its relative path.
This gives different results on Linux vs Windows because of the
different path separator. The solution is to normalize the relative path
using forward slashes.

Affects directory assets with subdirectories.

Closes #14555
Closes #16928


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…eter mapping (#15630)

----
This is an initial PR as discussed with @nija-at in an attempt to describe the user experience for supporting parameter mapping.
This PR will only support parameter mappings for HTTP APIs _without_ an integration subtypes, but it will provide interfaces that can (and probably should) be reused when adding support for integration subtypes as well. Since it also provides the possibility to provide custom key/value pairs for maximum flexibility, it can support and integration subtype although it requires a bit more work on the user side.

closes #15293 

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
kaizencc and others added 7 commits October 13, 2021 15:50
…ype (#16884)

Fixes #16806. Now, setting `type: ssm.ParameterType.AWS_EC2_IMAGE_ID` throws an error and instead, you can set `dataType: 'aws:ec2:image'`. Specifically, the `ssm.ParameterType.AWS_EC2_IMAGE_ID` value is used internally (original [PR](#4161)) in a few places and really shouldn't be exposed to the customer. But I'm not sure what else we can do, especially since this is a stable module.

Original code using `CfnParameter`: 
```ts
const parameter = ssm.CfnParameter(this, "ImageBuilderAMI", {
  type: "String",
  dataType: "aws:ec2:image",
  name: "/ec2-imagebuilder/latest",
  description: "Latest AMI Image",
  value: self.node.try_get_context(env_context)["LinuxAmi"] 
});
```

Can now use `StringParameter`: 
```ts
const parameter = ssm.StringParameter(this, 'ImageBuilderAMI', {
  dataType: 'aws:ec2:image',
  parameterName: '/ec2-imagebuilder/latest',
  description: "Latest AMI Image",
  stringValue: self.node.try_get_context(env_context)["LinuxAmi"]
});
```

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Ran npm-check-updates and yarn upgrade to keep the `yarn.lock` file up-to-date.
The release-notes script generates a RELEASE_NOTES file based on the current
version and current changelog. During a BUMP_CANDIDATE build, a new version
number has been generated, but no corresponding changelog has been
created/updated. This causes the release notes to fail (and currently breaks the
pipeline).

Simplest fix is to skip release notes generation for these builds.


----

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

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
fixes: #15632
fixes: #14336

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…bda invoke action (#16946)

feat(codepipeline-actions): Add support for string user parameters

closes #16776 

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@gitpod-io
Copy link

gitpod-io bot commented Oct 14, 2021

@aws-cdk-automation aws-cdk-automation added the pr/no-squash This PR should be merged instead of squash-merging it label Oct 14, 2021
CHANGELOG.md Outdated Show resolved Hide resolved
@aws-cdk-automation
Copy link
Collaborator Author

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 5724dce
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

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

@mergify mergify bot merged commit 1d3883a into release Oct 14, 2021
@mergify mergify bot deleted the bump/1.128.0 branch October 14, 2021 04:55
@mergify
Copy link
Contributor

mergify bot commented Oct 14, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-squash This PR should be merged instead of squash-merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.