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

General upgrade of dependencies #15705

Draft
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

AlanCoding
Copy link
Member

SUMMARY

General run of

./updater.sh upgrade

This requires pulling in yaml/pyyaml#808 for cython stuff, thank you @nitzmahone. One of our unrelated libraries put a >3 constraint for cython, so this affects pyyaml and other stuff.

Otherwise the libraries seem to upgrade without issue.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Dec 16, 2024
@AlanCoding
Copy link
Member Author

Failures seem to be on:

  File "/awx_devel/awx/main/models/ha.py", line 57, in save
    super(BaseModel, self).save(*args, **kwargs)
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/django/db/models/base.py", line 814, in save
    self.save_base(
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/django/db/models/base.py", line 892, in save_base
    post_save.send(
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/django/dispatch/dispatcher.py", line 176, in send
    return [
           ^
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/django/dispatch/dispatcher.py", line 177, in <listcomp>
    (receiver, receiver(signal=self, sender=sender, **named))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/awx_devel/awx/main/signals.py", line 435, in activity_stream_create
    changes = model_to_dict(instance, model_serializer_mapping())
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/awx_devel/awx/main/utils/common.py", line 440, in model_to_dict
    allowed_fields = get_allowed_fields(obj, serializer_mapping)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/awx_devel/awx/main/utils/common.py", line 359, in get_allowed_fields
    allowed_fields = [x for x in serializer_actual.fields if not serializer_actual.fields[x].read_only] + ['id']
                                 ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/django/utils/functional.py", line 57, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
                                         ^^^^^^^^^^^^^^^^^^^
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/rest_framework/serializers.py", line 372, in fields
    for key, value in self.get_fields().items():
                      ^^^^^^^^^^^^^^^^^
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/rest_framework/serializers.py", line 1113, in get_fields
    fields[field_name] = field_class(**field_kwargs)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/var/lib/awx/venv/awx/lib64/python3.11/site-packages/rest_framework/fields.py", line 992, in __init__
    warnings.warn("min_value should be a Decimal instance.")
UserWarning: min_value should be a Decimal instance.

This is documented somewhere...

@AlanCoding
Copy link
Member Author

Copy link

sonarcloud bot commented Dec 17, 2024

@AlanCoding
Copy link
Member Author

Now the remaining work to be done is clear.

Exception: Included licenses not consistent with requirements:
 license file annotated-types.txt does not correspond to an existing requirement; it should be removed.
 license file azure-common.txt does not correspond to an existing requirement; it should be removed.
 license file docutils.txt does not correspond to an existing requirement; it should be removed.
 license file inflect.txt does not correspond to an existing requirement; it should be removed.
 embedded source for jwcrypto is 1.5.4 instead of the required version 1.5.6
 embedded source for psycopg is 3.1.18 instead of the required version 3.2.3
 license file pydantic.txt does not correspond to an existing requirement; it should be removed.
 license file pydantic-core.txt does not correspond to an existing requirement; it should be removed.
 embedded source for uwsgi is 2.0.24 instead of the required version 2.0.28
 license for requirement aiohappyeyeballs is missing
 license for requirement backports-tarfile is missing
 license for requirement durationpy is missing
 license for requirement importlib-resources is missing
 license for requirement propcache is missing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant