pulp-ansible 0.29.1__tar.gz → 0.29.3__tar.gz

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/CHANGES.md

@@ -8,6 +8,20 @@
 
 [//]: # (towncrier release notes start)
 
+## 0.29.3 (2026-02-06) {: #0.29.3 }
+
+No significant changes.
+
+---
+
+## 0.29.2 (2026-01-27) {: #0.29.2 }
+
+#### Bugfixes {: #0.29.2-bugfix }
+
+- Exclude some bloated fields by default from the collection version views.
+
+---
+
 ## 0.29.1 (2026-01-26) {: #0.29.1 }
 
 #### Bugfixes {: #0.29.1-bugfix }

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pulp-ansible
-Version: 0.29.1
+Version: 0.29.3
 Summary: Pulp plugin to manage Ansible content, e.g. roles
 Author-email: Pulp Ansible Plugin Project Developers <pulp-dev@redhat.com>
 Project-URL: Homepage, https://pulpproject.org

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible/app/__init__.py

@@ -8,6 +8,6 @@ class PulpAnsiblePluginAppConfig(PulpPluginAppConfig):
 
     name = "pulp_ansible.app"
     label = "ansible"
-    version = "0.29.1"
+    version = "0.29.3"
     python_package_name = "pulp-ansible"
     domain_compatible = True

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible/app/serializers.py

@@ -665,6 +665,22 @@ class CollectionVersionSerializer(ContentChecksumSerializer, CollectionVersionUp
 
     creating = True
 
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        try:
+            request = self.context["request"]
+            if request.method != "GET":
+                return
+        except (AttributeError, TypeError, KeyError):
+            # The serializer was not initialized with request context.
+            return
+        query_params = request.query_params
+        if self.include_arg_name in query_params or self.exclude_arg_name in query_params:
+            return
+        for fieldname in ["files", "manifest", "docs_blob", "contents"]:
+            # Redact these fields by default as they have no size limit.
+            self.fields.pop(fieldname)
+
     def validate(self, data):
         """Run super() validate if creating, else return data."""
         # This validation is for creating CollectionVersions

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible/app/tasks/collections.py

@@ -57,6 +57,7 @@ from semantic_version import SimpleSpec, Version
 from semantic_version.base import Always
 
 from pulp_ansible.app.constants import PAGE_SIZE
+from pulp_ansible.exceptions import CollectionNotFound
 from pulp_ansible.app.models import (
     AnsibleCollectionDeprecated,
     AnsibleNamespace,
@@ -841,9 +842,7 @@ class CollectionSyncFirstStage(Stage):
             namespace not in self._unpaginated_collection_metadata
             or name not in self._unpaginated_collection_metadata[namespace]
         ):
-            raise FileNotFoundError(
-                f"Collection {namespace}.{name} does not exist on {self.remote.url}"
-            )
+            raise CollectionNotFound(namespace, name, self.remote.url)
 
         if self._unpaginated_collection_metadata[namespace][name]["deprecated"]:
             d_content = DeclarativeContent(

pulp_ansible-0.29.3/pulp_ansible/exceptions.py

@@ -0,0 +1,26 @@
+from gettext import gettext as _
+
+from pulpcore.plugin.exceptions import PulpException
+
+
+class CollectionNotFound(PulpException):
+    error_code = "PLPAN01"
+
+    def __init__(self, namespace, name, url):
+        """
+        :param url: The full URL that failed validation.
+        :type url: str
+        """
+        # Work around a sudden api change in pulpcore 3.103.
+        try:
+            super().__init__()
+        except BaseException:
+            super().__init__(self.error_code)
+        self.namespace = namespace
+        self.name = name
+        self.url = url
+
+    def __str__(self):
+        return f"[{self.error_code}] " + _(
+            "Collection {namespace}.{name} does not exist on {url}"
+        ).format(namespace=self.namespace, name=self.name, url=self.url)

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible/tests/functional/api/collection/v3/test_proxy.py

@@ -77,6 +77,7 @@ def test_sync_through_http_proxy_with_auth(
 
 @pytest.mark.parallel
 def test_sync_through_http_proxy_with_auth_but_auth_not_configured(
+    has_pulp_plugin,
     ansible_bindings,
     ansible_repo,
     ansible_collection_remote_factory,
@@ -101,4 +102,7 @@ def test_sync_through_http_proxy_with_auth_but_auth_not_configured(
             monitor_task,
         )
     except PulpTaskError as exc:
-        assert "407, message='Proxy Authentication Required'" in exc.task.error["description"]
+        if has_pulp_plugin("core", min="3.102"):
+            assert "[PLP0010]" in exc.task.error["description"]
+        else:
+            assert "407, message='Proxy Authentication Required'" in exc.task.error["description"]

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible/tests/functional/api/collection/v3/test_sync.py

@@ -72,6 +72,7 @@ class TestSync:
     @pytest.mark.parametrize("mirror", (True, False))
     def test_sync_absent_collection_from_pulp_fails(
         self,
+        has_pulp_plugin,
         ansible_bindings,
         monitor_task,
         ansible_repository_factory,
@@ -89,10 +90,11 @@ class TestSync:
         result = ansible_bindings.RepositoriesAnsibleApi.sync(
             repository.pulp_href, {"mirror": mirror}
         )
-        message = "Collection absent.not_present does not exist on"
         with pytest.raises(Exception) as exc_info:
             monitor_task(result.task)
-        assert message in exc_info.value.task.error["description"]
+        if has_pulp_plugin("core", min="3.102"):
+            assert "[PLPAN01]" in exc_info.value.task.error["description"]
+        assert "absent.not_present" in exc_info.value.task.error["description"]
 
     @pytest.mark.parametrize("mirror", (True, False))
     def test_resync_is_noop(

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible/tests/functional/api/role/test_sync.py

@@ -30,12 +30,23 @@ def test_role_sync(
 
 
 @pytest.mark.parallel
-def test_role_sync_invalid(ansible_repo_factory, ansible_role_remote_factory, ansible_sync_factory):
+def test_role_sync_invalid(
+    has_pulp_plugin,
+    ansible_repo_factory,
+    ansible_role_remote_factory,
+    ansible_sync_factory,
+):
+    # Note this is not a DNS lookup error.
+    # This url fails a check before even reaching out overthe network.
     remote = ansible_role_remote_factory(url="http://i-am-an-invalid-url.com/invalid/")
 
     with pytest.raises(PulpTaskError) as exc_info:
         ansible_sync_factory(remote=remote.pulp_href)
-    assert (
-        exc_info.value.task.error["description"]
-        == "Could not determine API version for: http://i-am-an-invalid-url.com/invalid/"
-    )
+    if has_pulp_plugin("core", min="3.102"):
+        # This should actually have been caught by validation of the remote.
+        assert "[PLP0000]" in exc_info.value.task.error["description"]
+    else:
+        assert (
+            exc_info.value.task.error["description"]
+            == "Could not determine API version for: http://i-am-an-invalid-url.com/invalid/"
+        )

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pulp-ansible
-Version: 0.29.1
+Version: 0.29.3
 Summary: Pulp plugin to manage Ansible content, e.g. roles
 Author-email: Pulp Ansible Plugin Project Developers <pulp-dev@redhat.com>
 Project-URL: Homepage, https://pulpproject.org

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pulp_ansible.egg-info/SOURCES.txt

@@ -9,6 +9,7 @@ pyproject.toml
 test_requirements.txt
 unittest_requirements.txt
 pulp_ansible/__init__.py
+pulp_ansible/exceptions.py
 pulp_ansible/pytest_plugin.py
 pulp_ansible.egg-info/PKG-INFO
 pulp_ansible.egg-info/SOURCES.txt

{pulp_ansible-0.29.1 → pulp_ansible-0.29.3}/pyproject.toml

@@ -7,7 +7,7 @@ build-backend = 'setuptools.build_meta'
 
 [project]
 name = "pulp-ansible"
-version = "0.29.1"
+version = "0.29.3"
 description = "Pulp plugin to manage Ansible content, e.g. roles"
 readme = "README.rst"
 authors = [
@@ -99,7 +99,7 @@ ignore = [
 [tool.bumpversion]
 # This section is managed by the plugin template. Do not edit manually.
 
-current_version = "0.29.1"
+current_version = "0.29.3"
 commit = false
 tag = false
 parse = "(?P<major>\\d+)\\.(?P<minor>\\d+)\\.(?P<alpha>0a)?(?P<patch>\\d+)(\\.(?P<release>[a-z]+))?"