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

Update Python dependencies #17420

Merged
merged 5 commits into from
Feb 15, 2024

Conversation

galaxybot
Copy link
Contributor

Run make update-dependencies.

@nsoranzo

This comment was marked as resolved.

@galaxybot galaxybot force-pushed the dev_auto_update_dependencies branch from fb9f229 to 30251b4 Compare February 10, 2024 03:11
@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch from 30251b4 to ccddfa0 Compare February 12, 2024 10:38
@nsoranzo
Copy link
Member

All test suites pass now except integration tests which fail with errors like:

_____ ERROR at setup of TestConfigSchema.test_schema_path_resolution_graph _____

cls = <class 'integration.test_config_schema.TestConfigSchema'>

    @classmethod
    def setup_class(cls):
>       cls.setUpClass()

lib/galaxy/util/unittest.py:19: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
lib/galaxy_test/driver/integration_util.py:123: in setUpClass
    cls._test_driver.setup(config_object=cls)
lib/galaxy_test/driver/driver_util.py:914: in setup
    self._register_and_run_servers(config_object)
lib/galaxy_test/driver/driver_util.py:969: in _register_and_run_servers
    server_wrapper = launch_server(
lib/galaxy_test/driver/driver_util.py:781: in launch_server
    port = attempt_ports(port)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

port = '8148', set_galaxy_web_port = True

    def attempt_ports(port=None, set_galaxy_web_port=True) -> str:
        if port is not None:
            if not attempt_port(int(port)):
>               raise Exception(f"An existing process seems bound to specified test server port [{port}]")
E               Exception: An existing process seems bound to specified test server port [8148]

lib/galaxy_test/driver/driver_util.py:496: Exception

Any idea what could be causing this before I start testing manually each dependency update?

nsoranzo and others added 4 commits February 14, 2024 00:57
xref: pydantic/pydantic#8760

Fix the following error when testing with mypy:

```
$ tox -e mypy
Deferral trace:
    cwltool.utils:62
    cwltool.utils:-1
    ...
    galaxy.model:666
    galaxy.model:666
    galaxy.model:666
    galaxy.model:666
lib/galaxy/model/__init__.py: error: INTERNAL ERROR: maximum semantic analysis
iteration count reached
Found 1 error in 1 file (errors prevented further checking)
```
@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch from ccddfa0 to 6250c97 Compare February 14, 2024 00:58
@nsoranzo
Copy link
Member

To fix the issue above I had to regenerate the dependency updates with pytest = "<8" in pyproject.toml (which actually downgraded only pytest itself).
I'd rather not pin pytest, so I haven't added the pin to this PR, which means the issue will happen again in the next dependency update PRs while we find a fix.

Anyway, this is green and ready for review/merge.

@mvdbeek mvdbeek merged commit c949977 into galaxyproject:dev Feb 15, 2024
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants