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

Fix module path for bcr testing #3069

Merged
merged 3 commits into from
Dec 9, 2024
Merged

Conversation

UebelAndre
Copy link
Collaborator

This should hopefully address

ERROR: The test module directory does not exist in the source archive: /Users/buildkite/builds/bk-macos-arm64-bcva/bazel-org-repo-root/output/extensions/bindgen
Traceback (most recent call last):
  File "/Users/buildkite/builds/bk-macos-arm64-bcva/bazel/bcr-presubmit/bcr_presubmit.py", line 537, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/buildkite/builds/bk-macos-arm64-bcva/bazel/bcr-presubmit/bcr_presubmit.py", line 528, in main
    repo_location, config_file = prepare_test_module_repo(args.module_name, args.module_version, args.overwrite_bazel_version)
                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/buildkite/builds/bk-macos-arm64-bcva/bazel/bcr-presubmit/bcr_presubmit.py", line 269, in prepare_test_module_repo
    error("The test module directory does not exist in the source archive: %s" % test_module_root)
  File "/Users/buildkite/builds/bk-macos-arm64-bcva/bazel/bcr-presubmit/bcr_presubmit.py", line 58, in error
    raise BcrPipelineException("BCR Presubmit failed!")
BcrPipelineException: BCR Presubmit failed!
🚨 Error: The command exited with status 1

@UebelAndre UebelAndre marked this pull request as ready for review December 9, 2024 14:15
@UebelAndre UebelAndre enabled auto-merge December 9, 2024 14:20
@UebelAndre UebelAndre added this pull request to the merge queue Dec 9, 2024
@UebelAndre UebelAndre removed this pull request from the merge queue due to a manual request Dec 9, 2024
@UebelAndre UebelAndre enabled auto-merge December 9, 2024 14:29
@UebelAndre UebelAndre disabled auto-merge December 9, 2024 14:29
@UebelAndre UebelAndre enabled auto-merge December 9, 2024 14:30
@UebelAndre UebelAndre added this pull request to the merge queue Dec 9, 2024
Merged via the queue into bazelbuild:main with commit c07cb1b Dec 9, 2024
4 checks passed
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.

2 participants