-
Notifications
You must be signed in to change notification settings - Fork 138
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
BCR presubmit: Support testing test module #1303
Conversation
buildkite/bazelci.py
Outdated
With matrix = {'a': [1, 2], 'b': [1], 'c': [2]}, | ||
if attributes = ['a', 'b'], then returns [[('a', 1), ('b', 1)], [('a', 2), ('b', 1)]] | ||
if attributes = ['b', 'c'], then returns [[('b', 1), ('c', 1)]] | ||
if attributes = ['c'], then returns [[('c', 1)]] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"then returns [[('c', 2)]]"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch, but I'll fix the matrix instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also rebase with master branch after merging #1300
BCR modules can now specify task configurations for a test module under `bcr_test_module`. Closes: #1302
688ac93
to
aa57834
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
output_dir = root.joinpath("output") | ||
bazelci.eprint("* Download and unpack %s\n" % archive_url) | ||
download(archive_url, archive_file) | ||
shutil.unpack_archive(str(archive_file), output_dir) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At present, we lose the permissions on the extracted files, per python/cpython#59999.
BCR modules can now specify task configurations for a test module under `bcr_test_module`. Closes: bazelbuild#1302
Implementing #1302