Skip to content

Conversation

@Eijebong
Copy link
Contributor

This fixes a regression from d2dee6a which added the graph config arg but then got the order wrong in these calls which lead to TypeError: unhashable type: '_Environ' in image_path when calling taskgraph build-image

Traceback (most recent call last):
  File "src/taskgraph/main.py", line 1020, in main
    return args.command(vars(args))
           ~~~~~~~~~~~~^^^^^^^^^^^^
  File "src/taskgraph/main.py", line 593, in build_image
    build_image(args["image_name"], args["tag"], os.environ, graph_config)
    ~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "src/taskgraph/docker.py", line 122, in build_image
    image_dir = docker.image_path(name, graph_config)
  File "src/taskgraph/util/docker.py", line 225, in image_path
    paths = image_paths(graph_config)
TypeError: unhashable type: '_Environ'

This fixes a regression from d2dee6a
which added the graph config arg but then got the order wrong in these
calls which lead to `TypeError: unhashable type: '_Environ'` in
image_path when calling `taskgraph build-image`

```
Traceback (most recent call last):
  File "src/taskgraph/main.py", line 1020, in main
    return args.command(vars(args))
           ~~~~~~~~~~~~^^^^^^^^^^^^
  File "src/taskgraph/main.py", line 593, in build_image
    build_image(args["image_name"], args["tag"], os.environ, graph_config)
    ~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "src/taskgraph/docker.py", line 122, in build_image
    image_dir = docker.image_path(name, graph_config)
  File "src/taskgraph/util/docker.py", line 225, in image_path
    paths = image_paths(graph_config)
TypeError: unhashable type: '_Environ'
```
@Eijebong Eijebong requested a review from a team as a code owner August 20, 2025 09:00
@Eijebong Eijebong requested a review from bhearsum August 20, 2025 09:00
@Eijebong Eijebong merged commit 6317207 into taskcluster:main Aug 20, 2025
16 of 17 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