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

CBMC Batch scripts seem to need buckets and packages specified #550

Closed
cmangla opened this issue Nov 18, 2019 · 1 comment
Closed

CBMC Batch scripts seem to need buckets and packages specified #550

cmangla opened this issue Nov 18, 2019 · 1 comment
Labels
bug This issue is a bug. needs-reproduction This issue needs reproduction. p2 This is a standard priority issue

Comments

@cmangla
Copy link
Contributor

cmangla commented Nov 18, 2019

I built the latest aws-batch-cbmc, deployed on a new AWS account, and ran the aws-c-common CBMC tests. I needed to modify cbmc-batch.sh to get the tests to work correctly.

At


I had to add the following flags:

--srcbucket s3://.../cbmc/srcbucket \
--wsbucket s3://.../cbmc/wsbucket/$job \
--outbucket s3://.../cbmc/outbucket/$job \
--pkgbucket s3://.../cbmc/package \
--cbmcpkg cbmc-ubuntu16.tar.gz \
--batchpkg cbmc-batch.tar.gz \
--viewerpkg cbmc-viewer.tar.gz \

The outbucket needed the $job subdirectory, without which the different tests were overwriting each other's files (I think).

Steps to reproduce:

  1. Build and deploy aws-batch-cbmc
  2. Set $PATH to cbmc-batch of aws-batch-cbmc.
  3. Clone aws-c-common and run cbmc-batch.sh --start in .cbmc-batch/.
  4. Notice complains about missing *bucket specification.
  5. Add the above flags but without the $job subdirectory.
  6. Run cbmc-batch.sh and see tests failing and getting utterly confused.
  7. Add the $job subdirectories as shown above and see the jobs all pass.
@jmklix jmklix added bug This issue is a bug. needs-reproduction This issue needs reproduction. p2 This is a standard priority issue labels Sep 6, 2023
@jmklix
Copy link
Member

jmklix commented Dec 13, 2023

Thanks for opening this, but we don't use this script anymore. Closing this issue

@jmklix jmklix closed this as completed Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. needs-reproduction This issue needs reproduction. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

2 participants