Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 15, 2025

Problem

Result uploading was failing in the Helix CI environment with the following error:

Traceback (most recent call last):
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/performance/scripts/benchmarks_ci.py", line 327, in main
    import upload
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/performance/scripts/upload.py", line 4, in <module>
    from azure.storage.blob import BlobClient, ContentSettings
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/.venv/lib/python3.10/site-packages/azure/storage/blob/__init__.py", line 10, in <module>
    from ._blob_client import BlobClient
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/.venv/lib/python3.10/site-packages/azure/storage/blob/_blob_client.py", line 17, in <module>
    import six
ModuleNotFoundError: No module named 'six'

Root Cause

The azure.storage.blob package (version 12.13.0) has an internal dependency on the six module (a Python 2 and 3 compatibility library). While six is imported throughout the azure.storage.blob codebase (_blob_client.py, _container_client.py, _upload_helpers.py, etc.), it was not explicitly listed in our project dependencies.

Solution

Added six==1.17.0 to the project dependencies in two locations:

  1. requirements.txt - For documentation and local development
  2. scripts/run_performance_job.py - Added to the pip install commands that execute in the Helix CI environment (where the actual failure was occurring)

This ensures the six module is available when azure.storage.blob attempts to import it during result uploading.

Testing

Verified that the six module can be imported successfully, resolving the import chain: benchmarks_ci.pyupload.pyazure.storage.blobsix

Fixes the issue reported where uploads were failing due to the missing module.

Original prompt

This section details on the original issue you should resolve

<issue_title>Upload is failing due to missing module six</issue_title>
<issue_description>Result uploading is failing due to the following error:

Traceback (most recent call last):
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/performance/scripts/benchmarks_ci.py", line 339, in <module>
    main(sys.argv[1:])
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/performance/scripts/performance/tracer.py", line 100, in wrapped
    return func(*args, **kwargs)
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/performance/scripts/benchmarks_ci.py", line 327, in main
    import upload
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/performance/scripts/upload.py", line 4, in <module>
    from azure.storage.blob import BlobClient, ContentSettings
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/.venv/lib/python3.10/site-packages/azure/storage/blob/__init__.py", line 10, in <module>
    from ._blob_client import BlobClient
  File "/home/helixbot/work/A0B5090B/w/ADE60940/e/.venv/lib/python3.10/site-packages/azure/storage/blob/_blob_client.py", line 17, in <module>
    import six
ModuleNotFoundError: No module named 'six'
```</issue_description>

## Comments on the Issue (you are @copilot in this section)

<comments>
</comments>

Fixes #5016


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: LoopedBard3 <15679138+LoopedBard3@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix upload failure due to missing module six Fix upload failure due to missing 'six' module dependency Oct 15, 2025
Copilot AI requested a review from LoopedBard3 October 15, 2025 18:46
@LoopedBard3 LoopedBard3 marked this pull request as ready for review October 15, 2025 18:52
@LoopedBard3 LoopedBard3 enabled auto-merge (squash) October 15, 2025 18:56
DrewScoggins
DrewScoggins previously approved these changes Oct 15, 2025
Copy link
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

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

LGTM

@LoopedBard3 LoopedBard3 disabled auto-merge October 15, 2025 19:40
@DrewScoggins DrewScoggins merged commit 2c0bd74 into main Oct 15, 2025
11 of 81 checks passed
@LoopedBard3 LoopedBard3 deleted the copilot/fix-upload-failure-missing-module branch October 15, 2025 19:41
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.

Upload is failing due to missing module six

3 participants