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 #18035

Merged
merged 4 commits into from
Apr 25, 2024

Conversation

galaxybot
Copy link
Contributor

Run make update-dependencies.

@github-actions github-actions bot added this to the 24.1 milestone Apr 20, 2024
@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch from 339021f to ab3a84c Compare April 23, 2024 16:28
nsoranzo and others added 3 commits April 25, 2024 14:33
With the merge of galaxyproject#17156 ,
`rucio-clients` was listed as a conditional requirement, but it was actually
installed almost everywhere (the only condition was
`sys.version_info >= (3, 9)`), independently of its use for an object store.

This also installs `rucio-clients` as a dev dependency, otherwise the
`test/integration/objectstore/test_rucio_objectstore.py` integration tests
would fail. Pinned dependencies are updated in the next commit.

Fix galaxyproject#17695 .
@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch from a98c5ec to 501c9e3 Compare April 25, 2024 13:33
Fix the following error:

```
____ TestCeleryUserRateLimitIntegrationNoLimit.test_mock_pass_user_id_task _____

self = <integration.test_celery_user_rate_limit.TestCeleryUserRateLimitIntegrationNoLimit object at 0x7f978fc0e400>

    def test_mock_pass_user_id_task(self):
>       self._test_mock_pass_user_id_task(2, 3, self._user_rate_limit)

test/integration/test_celery_user_rate_limit.py:161:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <integration.test_celery_user_rate_limit.TestCeleryUserRateLimitIntegrationNoLimit object at 0x7f978fc0e400>
num_users = 2, num_calls = 3, tasks_per_user_per_sec = 0.0

        for user_results in results.values():
            last_task_end_time = start_time
            for result in user_results:
>               if result.date_done > last_task_end_time:
E               TypeError: can't compare offset-naive and offset-aware datetimes

test/integration/test_celery_user_rate_limit.py:110: TypeError
```

Also https://docs.python.org/3/library/datetime.html#datetime.datetime.utcnow
is deprecated since Python 3.12 .
@martenson martenson merged commit 4d6c53e into galaxyproject:dev Apr 25, 2024
56 checks passed
@martenson
Copy link
Member

thanks @nsoranzo !

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