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

Migrate to CodeBuild batch. #51

Merged
merged 28 commits into from
Oct 6, 2020
Merged

Migrate to CodeBuild batch. #51

merged 28 commits into from
Oct 6, 2020

Conversation

bryce-shang
Copy link
Contributor

Issues:

Resolves CryptoAlg-489

Description of changes:

This PR migrates CodeBuild build type from single to batch.
This migration gets below items involved:

Call-outs:

CryptoAlg-575: CFN CodeBuild is still releasing BUILD_BATCH type. Current workaround is to use AWS CLI to update webhook build type.

aws codebuild update-webhook --project-name aws-lc-ci --build-type BUILD_BATCH

Testing:

Open aws-lc AWS account and check CodeBuild aws-lc-ci, which has not webhook to this repo due to the limit.

Next Steps:

  • Clean up old code build.
  • Add webhook to CodeBuild aws-lc-ci.

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

@@ -27,9 +27,9 @@ fi

if [ $(dpkg --print-architecture) == "amd64" ]; then
# x86 TSAN runs get stuck on PoolTest.Threads for over an hour https://github.com/awslabs/aws-lc/issues/13
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this comment still relevant? The issue has been closed for a while now.

Copy link
Contributor

Choose a reason for hiding this comment

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

If we don't need it anymore, can we also remove the comment on line 20 as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The comment explained why test is not executed. run_build instead of build_and_test is used in this condition.

tests/ci/cdk/README.md Outdated Show resolved Hide resolved
tests/ci/cdk/README.md Show resolved Hide resolved
tests/ci/cdk/run-cdk.sh Outdated Show resolved Hide resolved
tests/ci/cdk/run-cdk.sh Show resolved Hide resolved
tests/ci/cdk/util/metadata.py Show resolved Hide resolved
@bryce-shang bryce-shang merged commit b8f5511 into aws:main Oct 6, 2020
@bryce-shang bryce-shang deleted the batch-ci branch October 19, 2020 23:07
bryce-shang added a commit to bryce-shang/aws-lc that referenced this pull request Oct 19, 2020
* Add batch build spec.

* Use batch-list.

* Remove invalid characters from identifier.

* Fix syntax warning.

* Upgrade cdk to latest version 1.64.

* Add github_codebuild_batch_stack.py.

* Move build specs.

* Refactor cdk code.

* Use aws-lc AWS account.

* Unify default values of env variable.

* Fix windows docker build issue.

* Add s3 bucket deletion when destroy resource.

* Add new variable github_source_version.

* Upgrade ec2 windows from 2016 to 2019.

* Add git to PATH.

* Revert "Upgrade ec2 windows from 2016 to 2019."

This reverts commit a72c367.

* Add 2019 windows.

* Move cyg and quilt installation to build spec.

* Add docker image build doc.

* Fix identifier name.

* Switch to 2016. CodeBuild does not support 2019.

* Clean up and add more docs.

* Fix typo.

* Modify README.md

* Remove '*'.

* Change privileged-mode to false.

* ARM sanitizer needs privileged-mode.
bryce-shang added a commit to bryce-shang/aws-lc that referenced this pull request Oct 20, 2020
* Add batch build spec.

* Use batch-list.

* Remove invalid characters from identifier.

* Fix syntax warning.

* Upgrade cdk to latest version 1.64.

* Add github_codebuild_batch_stack.py.

* Move build specs.

* Refactor cdk code.

* Use aws-lc AWS account.

* Unify default values of env variable.

* Fix windows docker build issue.

* Add s3 bucket deletion when destroy resource.

* Add new variable github_source_version.

* Upgrade ec2 windows from 2016 to 2019.

* Add git to PATH.

* Revert "Upgrade ec2 windows from 2016 to 2019."

This reverts commit a72c367.

* Add 2019 windows.

* Move cyg and quilt installation to build spec.

* Add docker image build doc.

* Fix identifier name.

* Switch to 2016. CodeBuild does not support 2019.

* Clean up and add more docs.

* Fix typo.

* Modify README.md

* Remove '*'.

* Change privileged-mode to false.

* ARM sanitizer needs privileged-mode.
bryce-shang added a commit to bryce-shang/aws-lc that referenced this pull request Oct 22, 2020
* Add batch build spec.

* Use batch-list.

* Remove invalid characters from identifier.

* Fix syntax warning.

* Upgrade cdk to latest version 1.64.

* Add github_codebuild_batch_stack.py.

* Move build specs.

* Refactor cdk code.

* Use aws-lc AWS account.

* Unify default values of env variable.

* Fix windows docker build issue.

* Add s3 bucket deletion when destroy resource.

* Add new variable github_source_version.

* Upgrade ec2 windows from 2016 to 2019.

* Add git to PATH.

* Revert "Upgrade ec2 windows from 2016 to 2019."

This reverts commit a72c367.

* Add 2019 windows.

* Move cyg and quilt installation to build spec.

* Add docker image build doc.

* Fix identifier name.

* Switch to 2016. CodeBuild does not support 2019.

* Clean up and add more docs.

* Fix typo.

* Modify README.md

* Remove '*'.

* Change privileged-mode to false.

* ARM sanitizer needs privileged-mode.
torben-hansen pushed a commit to torben-hansen/aws-lc that referenced this pull request Nov 15, 2023
Add support for 8 more NEON instructions
s2n-bignum original commit: awslabs/s2n-bignum@8337486
torben-hansen pushed a commit to torben-hansen/aws-lc that referenced this pull request Sep 18, 2024
Add support for 8 more NEON instructions
s2n-bignum original commit: awslabs/s2n-bignum@8337486

s2n-bignum original commit: awslabs/s2n-bignum@935b9dc
dkostic pushed a commit to dkostic/aws-lc that referenced this pull request Dec 5, 2024
Add support for 8 more NEON instructions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants