Skip to content

Conversation

@rix0rrr
Copy link
Contributor

@rix0rrr rix0rrr commented Mar 4, 2021

Use lerna to generate a topologically sorted list of .tgz files.

This list will be used during publishing to impose an order
on the package publishing actions.

Fixes #13221.


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

Use lerna to generate a topologically sorted list of `.tgz` files.

This list will be used during publishing to impose an order
on the package publishing actions.

Fixes #13221.
@rix0rrr rix0rrr requested a review from a team March 4, 2021 13:30
@rix0rrr rix0rrr self-assigned this Mar 4, 2021
@gitpod-io
Copy link

gitpod-io bot commented Mar 4, 2021

@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Mar 4, 2021
pack.sh Outdated
done

# Record the dependency order of NPM packages into a file
# (This file will be opportunistically used during publishing, it does not need to be complete)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do the ones not in this file go before or after or random? Nice to include that in the comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unspecified, do not rely on any behavior there.

The details of how everything works will be documented in the PR that introduces the feature in aws-delivlib.

pack.sh Outdated

# Record the dependency order of NPM packages into a file
# (This file will be opportunistically used during publishing, it does not need to be complete)
node_modules/.bin/lerna exec \
Copy link
Contributor

Choose a reason for hiding this comment

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

why not lerna ls --toposort? Isn't that what that's meant for?

@rix0rrr rix0rrr requested a review from nija-at March 4, 2021 14:47
@nija-at nija-at added the pr/do-not-merge This PR should not be merged at this time. label Mar 4, 2021
pack.sh Outdated
Comment on lines 39 to 40
if false; then

Copy link
Contributor

Choose a reason for hiding this comment

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

err.... from your local testing?

@rix0rrr rix0rrr removed the pr/do-not-merge This PR should not be merged at this time. label Mar 4, 2021
@mergify
Copy link
Contributor

mergify bot commented Mar 4, 2021

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

@mergify
Copy link
Contributor

mergify bot commented Mar 4, 2021

Thank you for contributing! Your pull request will be updated from master 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 20a22fe into master Mar 4, 2021
@mergify mergify bot deleted the huijbers/npm-publish-order branch March 4, 2021 17:16
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

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

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

cornerwings pushed a commit to cornerwings/aws-cdk that referenced this pull request Mar 8, 2021
Use lerna to generate a topologically sorted list of `.tgz` files.

This list will be used during publishing to impose an order
on the package publishing actions.

Fixes aws#13221.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(pipelines): CDK packages deployment race condition

3 participants