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

traceback on sync with new namespace "user" field #1598

Closed
jctanner opened this issue Sep 28, 2023 · 0 comments
Closed

traceback on sync with new namespace "user" field #1598

jctanner opened this issue Sep 28, 2023 · 0 comments

Comments

@jctanner
Copy link
Contributor

jctanner commented Sep 28, 2023

pulp-ansible==0.19.0 and main ...

https://issues.redhat.com/browse/AAH-2721

We're adding a new field for "users" to the namespace serializer in galaxy_ng. When more recent pulp_ansible versions sync from such a system, they traceback on an unexpected field when creating a new namespace...

pulp [9ac390ffb6a7466faeb16dc66bc7e7db]: pulpcore.tasking.tasks:INFO: Task 018ace1c-2c92-7788-9072-8b8611c5ac5e failed (AnsibleNamespaceMetadata() got unexpected keyword arguments: 'users')
pulp [9ac390ffb6a7466faeb16dc66bc7e7db]: pulpcore.tasking.tasks:INFO:   File "/usr/local/lib/python3.11/site-packages/pulpcore/tasking/tasks.py", line 66, in _execute_task
    result = func(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 191, in sync
    repo_version = d_version.create()
                   ^^^^^^^^^^^^^^^^^^

  File "/usr/local/lib/python3.11/site-packages/pulpcore/plugin/stages/declarative_version.py", line 161, in create
    loop.run_until_complete(pipeline)

  File "/usr/lib64/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^

  File "/usr/local/lib/python3.11/site-packages/pulpcore/plugin/stages/api.py", line 220, in create_pipeline
    await asyncio.gather(*futures)

  File "/usr/local/lib/python3.11/site-packages/pulpcore/plugin/stages/api.py", line 41, in __call__
    await self.run()

  File "/usr/local/lib/python3.11/site-packages/asgiref/sync.py", line 534, in thread_handler
    raise exc_info[1]

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 1072, in run
    await asyncio.gather(*tasks)

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 1012, in _find_all_collections
    await asyncio.gather(*tasks)

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 866, in _fetch_collection_metadata
    await self._fetch_paginated_collection_metadata(

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 809, in _fetch_paginated_collection_metadata
    await asyncio.gather(*tasks)

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 598, in _fetch_collection_version_metadata
    await self._add_collection_version(api_version, collection_version_url, metadata)

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 688, in _add_collection_version
    if await self._add_namespace(namespace, namespace_sha):
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "/usr/local/lib/python3.11/site-packages/pulp_ansible/app/tasks/collections.py", line 741, in _add_namespace
    namespace = AnsibleNamespaceMetadata(**namespace)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  File "/usr/local/lib/python3.11/site-packages/django_lifecycle/mixins.py", line 61, in __init__
    super().__init__(*args, **kwargs)

  File "/usr/local/lib/python3.11/site-packages/django/db/models/base.py", line 567, in __init__
    raise TypeError(
jctanner added a commit to jctanner/pulp_ansible that referenced this issue Sep 28, 2023
fixes: pulp#1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
jctanner added a commit to jctanner/pulp_ansible that referenced this issue Sep 29, 2023
fixes: pulp#1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
jctanner added a commit to jctanner/pulp_ansible that referenced this issue Sep 29, 2023
fixes: pulp#1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
patchback bot pushed a commit that referenced this issue Sep 29, 2023
fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)
patchback bot pushed a commit that referenced this issue Sep 29, 2023
fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)
patchback bot pushed a commit that referenced this issue Sep 29, 2023
fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)
mdellweg pushed a commit that referenced this issue Sep 29, 2023
… (#1600)

fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)

Co-authored-by: jctanner <tanner.jc@gmail.com>
mdellweg pushed a commit that referenced this issue Sep 29, 2023
fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)
mdellweg pushed a commit that referenced this issue Sep 29, 2023
… (#1601)

fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)

Co-authored-by: jctanner <tanner.jc@gmail.com>
patchback bot pushed a commit that referenced this issue Sep 29, 2023
fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)
mdellweg pushed a commit that referenced this issue Sep 29, 2023
… (#1605)

fixes: #1598

Signed-off-by: James Tanner <tanner.jc@gmail.com>
(cherry picked from commit 4a7c5b0)

Co-authored-by: jctanner <tanner.jc@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

No branches or pull requests

1 participant