search

amazon-ion / ion-java

Java streaming parser/serializer for Ion.
https://amazon-ion.github.io/ion-docs/
Apache License 2.0
864 stars 110 forks source link

Ion-test-driver issue. #795

Open github-actions[bot] opened 5 months ago

github-actions[bot] commented 5 months ago

Ion-test-driver complained that behavior changed for the commit 5367d124acdb7ff2efb7f448b2ce61f4740da133 created by tgregg. Refer to the workflow for more details.

nirosys commented 5 months ago

Assert thrown in ion-test-driver's pip install step.

Successfully built amazon.ion docopt
ERROR: Exception:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/pip/_internal/cli/base_command.py", line 165, in exc_logging_wrapper
    status = run_func(*args)
  File "/usr/lib/python3/dist-packages/pip/_internal/cli/req_command.py", line 205, in wrapper
    return func(self, options, args)
  File "/usr/lib/python3/dist-packages/pip/_internal/commands/install.py", line 389, in run
    to_install = resolver.get_installation_order(requirement_set)
  File "/usr/lib/python3/dist-packages/pip/_internal/resolution/resolvelib/resolver.py", line 188, in get_installation_order
    weights = get_topological_weights(
  File "/usr/lib/python3/dist-packages/pip/_internal/resolution/resolvelib/resolver.py", line 276, in get_topological_weights
    assert len(weights) == expected_node_count
AssertionError

There are multiple issues for what looks like the same error, going to use this one to track the investigation.

nirosys commented 5 months ago

This error appears to be an issue that occurs in pip 22.0, ~which was added to the the GHA ubuntu image ~a week ago~. Looks like 22.0 has been included in some image for quite a while, I'm not sure exactly what changed yet.

I'm testing out an update to ion-java's workflow to use setup-python to use a fixed python version, which also forces a pip update, installing 24.0 (at the time of writing).