pulpcore 3.77.1__tar.gz → 3.79.0__tar.gz

{pulpcore-3.77.1 → pulpcore-3.79.0}/CHANGES.md

@@ -8,6 +8,68 @@
 
 [//]: # (towncrier release notes start)
 
+## 3.79.0 (2025-05-20) {: #3.79.0 }
+
+### REST API {: #3.79.0-rest-api }
+
+#### Features {: #3.79.0-rest-api-feature }
+
+- Added the `execution_time` as microseconds and other task related information on the finished task log entry.
+  [#6569](https://github.com/pulp/pulpcore/issues/6569)
+
+### Plugin API {: #3.79.0-plugin-api }
+
+No significant changes.
+
+### Pulp File {: #3.79.0-pulp-file }
+
+No significant changes.
+
+### Pulp Cert Guard {: #3.79.0-pulp-cert-guard }
+
+No significant changes.
+
+---
+
+## 3.78.0 (2025-05-13) {: #3.78.0 }
+
+### REST API {: #3.78.0-rest-api }
+
+#### Features {: #3.78.0-rest-api-feature }
+
+- Added the optional feature to enable uvloop library as a replacement of asyncio eventloop.
+  The feature is activated through the UVLOOP_ENABLED feature flag settings and the 'uvloop' optional dependency.
+  [#6021](https://github.com/pulp/pulpcore/issues/6021)
+
+#### Bugfixes {: #3.78.0-rest-api-bugfix }
+
+- Taught tasks to update the state of associated progress-reports when canceled/failed.
+
+  Added a management-task, `clean-up-progress-reports`, to clean up existing reports.
+  [#3609](https://github.com/pulp/pulpcore/issues/3609)
+- Fixed a server error when listing user roles.
+  [#6330](https://github.com/pulp/pulpcore/issues/6330)
+- On-demand or pull-through content is now immediately cached upon first request.
+  [#6540](https://github.com/pulp/pulpcore/issues/6540)
+
+#### Misc {: #3.78.0-rest-api-misc }
+
+- 
+
+### Plugin API {: #3.78.0-plugin-api }
+
+No significant changes.
+
+### Pulp File {: #3.78.0-pulp-file }
+
+No significant changes.
+
+### Pulp Cert Guard {: #3.78.0-pulp-cert-guard }
+
+No significant changes.
+
+---
+
 ## 3.77.1 (2025-05-08) {: #3.77.1 }
 
 ### REST API {: #3.77.1-rest-api }
@@ -370,6 +432,26 @@ No significant changes.
 
 ---
 
+## 3.73.9 (2025-05-06) {: #3.73.9 }
+
+### REST API {: #3.73.9-rest-api }
+
+No significant changes.
+
+### Plugin API {: #3.73.9-plugin-api }
+
+No significant changes.
+
+### Pulp File {: #3.73.9-pulp-file }
+
+No significant changes.
+
+### Pulp Cert Guard {: #3.73.9-pulp-cert-guard }
+
+No significant changes.
+
+---
+
 ## 3.73.8 (2025-04-24) {: #3.73.8 }
 
 ### REST API {: #3.73.8-rest-api }
@@ -1261,6 +1343,30 @@ No significant changes.
 
 ---
 
+## 3.63.18 (2025-05-13) {: #3.63.18 }
+
+### REST API {: #3.63.18-rest-api }
+
+#### Bugfixes {: #3.63.18-rest-api-bugfix }
+
+- Disable retry logic on the context of content-app on-demand streaming, as we can't recover
+  from any errors after starting the streaming process (chunked transfer).
+  [#5937](https://github.com/pulp/pulpcore/issues/5937)
+
+### Plugin API {: #3.63.18-plugin-api }
+
+No significant changes.
+
+### Pulp File {: #3.63.18-pulp-file }
+
+No significant changes.
+
+### Pulp Cert Guard {: #3.63.18-pulp-cert-guard }
+
+No significant changes.
+
+---
+
 ## 3.63.17 (2025-04-24) {: #3.63.17 }
 
 ### REST API {: #3.63.17-rest-api }
@@ -2501,6 +2607,53 @@ No significant changes.
 
 ---
 
+## 3.49.41 (2025-05-13) {: #3.49.41 }
+
+### REST API {: #3.49.41-rest-api }
+
+#### Bugfixes {: #3.49.41-rest-api-bugfix }
+
+- Disable retry logic on the context of content-app on-demand streaming, as we can't recover
+  from any errors after starting the streaming process (chunked transfer).
+  [#5937](https://github.com/pulp/pulpcore/issues/5937)
+
+### Plugin API {: #3.49.41-plugin-api }
+
+No significant changes.
+
+### Pulp File {: #3.49.41-pulp-file }
+
+No significant changes.
+
+### Pulp Cert Guard {: #3.49.41-pulp-cert-guard }
+
+No significant changes.
+
+---
+
+## 3.49.40 (2025-05-06) {: #3.49.40 }
+
+### REST API {: #3.49.40-rest-api }
+
+#### Bugfixes {: #3.49.40-rest-api-bugfix }
+
+- Fixed to not use the OpenTelemetryMiddleware when PULP_OTEL_ENABLED is not set.
+  [#6529](https://github.com/pulp/pulpcore/issues/6529)
+
+### Plugin API {: #3.49.40-plugin-api }
+
+No significant changes.
+
+### Pulp File {: #3.49.40-pulp-file }
+
+No significant changes.
+
+### Pulp Cert Guard {: #3.49.40-pulp-cert-guard }
+
+No significant changes.
+
+---
+
 ## 3.49.39 (2025-04-24) {: #3.49.39 }
 
 ### REST API {: #3.49.39-rest-api }

{pulpcore-3.77.1 → pulpcore-3.79.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pulpcore
-Version: 3.77.1
+Version: 3.79.0
 Summary: Pulp Django Application and Related Modules
 Author-email: Pulp Team <pulp-list@redhat.com>
 Project-URL: Homepage, https://pulpproject.org
@@ -21,14 +21,14 @@ Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: aiodns<=3.3.0,>=3.0
+Requires-Dist: aiodns<=3.4.0,>=3.0
 Requires-Dist: aiofiles<24.2.0,>=22.1
 Requires-Dist: aiohttp<3.12,>=3.8.1
 Requires-Dist: asyncio-throttle<=1.0.2,>=1.0
 Requires-Dist: async-timeout<4.0.4,>=4.0.3; python_version < "3.11"
 Requires-Dist: backoff<2.2.2,>=2.1.2
 Requires-Dist: click<=8.1.8,>=8.1.0
-Requires-Dist: cryptography<44.0.4,>=38.0.1
+Requires-Dist: cryptography<45.0.3,>=38.0.1
 Requires-Dist: Django~=4.2.0
 Requires-Dist: django-filter<=25.1,>=23.1
 Requires-Dist: django-guid<=3.5.1,>=3.3
@@ -37,7 +37,7 @@ Requires-Dist: django-lifecycle<=1.2.4,>=1.0
 Requires-Dist: djangorestframework<=3.16.0,>=3.14.0
 Requires-Dist: djangorestframework-queryfields<=1.1.0,>=1.0
 Requires-Dist: drf-access-policy<1.5.1,>=1.1.2
-Requires-Dist: drf-nested-routers<=0.94.1,>=0.93.4
+Requires-Dist: drf-nested-routers<=0.94.2,>=0.93.4
 Requires-Dist: drf-spectacular==0.27.2
 Requires-Dist: dynaconf<3.3.0,>=3.2.5
 Requires-Dist: gunicorn<23.1.0,>=20.1
@@ -46,13 +46,13 @@ Requires-Dist: jinja2<=3.1.6,>=3.1
 Requires-Dist: json_stream<2.4,>=2.3.2
 Requires-Dist: jq<1.9.0,>=1.6.0
 Requires-Dist: PyOpenSSL<26.0
-Requires-Dist: opentelemetry-api<1.33,>=1.27.0
-Requires-Dist: opentelemetry-sdk<1.33,>=1.27.0
-Requires-Dist: opentelemetry-exporter-otlp-proto-http<1.33,>=1.27.0
+Requires-Dist: opentelemetry-api<1.34,>=1.27.0
+Requires-Dist: opentelemetry-sdk<1.34,>=1.27.0
+Requires-Dist: opentelemetry-exporter-otlp-proto-http<1.34,>=1.27.0
 Requires-Dist: protobuf<6.0,>=4.21.1
 Requires-Dist: pulp-glue<0.33,>=0.18.0
 Requires-Dist: pygtrie<=2.5.0,>=2.5
-Requires-Dist: psycopg[binary]<=3.2.7,>=3.1.8
+Requires-Dist: psycopg[binary]<=3.2.9,>=3.1.8
 Requires-Dist: pyparsing<=3.2.3,>=3.1.0
 Requires-Dist: python-gnupg<=0.5.4,>=0.5
 Requires-Dist: PyYAML<=6.0.2,>=5.1.1
@@ -78,6 +78,8 @@ Requires-Dist: confluent-kafka<2.10.0,>=2.4.0; extra == "kafka"
 Provides-Extra: diagnostics
 Requires-Dist: pyinstrument~=5.0; extra == "diagnostics"
 Requires-Dist: memray~=1.17; extra == "diagnostics"
+Provides-Extra: uvloop
+Requires-Dist: uvloop<0.22,>=0.20; extra == "uvloop"
 Dynamic: license-file
 
 

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulp_certguard/app/__init__.py

@@ -6,6 +6,6 @@ class PulpCertGuardPluginAppConfig(PulpPluginAppConfig):
 
     name = "pulp_certguard.app"
     label = "certguard"
-    version = "3.77.1"
+    version = "3.79.0"
     python_package_name = "pulpcore"
     domain_compatible = True

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulp_file/app/__init__.py

@@ -8,6 +8,6 @@ class PulpFilePluginAppConfig(PulpPluginAppConfig):
 
     name = "pulp_file.app"
     label = "file"
-    version = "3.77.1"
+    version = "3.79.0"
     python_package_name = "pulpcore"
     domain_compatible = True

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/apps.py

@@ -247,7 +247,7 @@ class PulpAppConfig(PulpPluginAppConfig):
     label = "core"
 
     # The version of this app
-    version = "3.77.1"
+    version = "3.79.0"
 
     # The python package name providing this app
     python_package_name = "pulpcore"

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/entrypoint.py

@@ -48,7 +48,7 @@ class PulpApiWorker(SyncWorker):
         from pulpcore.app.models import ApiAppStatus
 
         if settings.API_APP_TTL < 2 * self.timeout:
-            logger.warn(
+            logger.warning(
                 "API_APP_TTL (%s) is smaller than double the gunicorn timeout (%s). "
                 "You may experience workers wrongly reporting as missing",
                 settings.API_APP_TTL,

pulpcore-3.79.0/pulpcore/app/management/commands/clean-up-progress-reports.py

@@ -0,0 +1,52 @@
+from gettext import gettext as _
+
+from django.core.management import BaseCommand
+from django.db.models import F
+
+from pulpcore.app.models import ProgressReport
+from pulpcore.constants import TASK_STATES
+
+
+class Command(BaseCommand):
+    """
+    Django management command for repairing progress-reports in inconsistent states.
+    """
+
+    help = (
+        "Repairs issue #3609. Long-running tasks that utilize ProgressReports, which "
+        "fail or are cancelled, can leave their associated reports in state 'running'. "
+        "This script finds the ProgressReports marked as 'running', whose owning task "
+        "is in either 'cancelled or 'failed', and moves the state of the ProgressReport "
+        "to match that of the task."
+    )
+
+    def add_arguments(self, parser):
+        """Set up arguments."""
+        parser.add_argument(
+            "--dry-run",
+            action="store_true",
+            help=_(
+                "Don't modify anything, just collect results on how many ProgressReports "
+                "are impacted."
+            ),
+        )
+
+    def handle(self, *args, **options):
+        dry_run = options["dry_run"]
+        for state in [TASK_STATES.CANCELED, TASK_STATES.FAILED]:
+            if dry_run:
+                to_be_updated = ProgressReport.objects.filter(
+                    task__state__ne=F("state"), state=TASK_STATES.RUNNING, task__state=state
+                ).count()
+                print(
+                    _("Number of ProgressReports in inconsistent state for {} tasks: {}").format(
+                        state, to_be_updated
+                    )
+                )
+            else:
+                updated = ProgressReport.objects.filter(
+                    task__state__ne=F("state"), state=TASK_STATES.RUNNING, task__state=state
+                ).update(state=state)
+                print(
+                    _("Number of ProgressReports updated for {} tasks: {}").format(state, updated)
+                )

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/management/commands/dump-publications-to-fs.py

@@ -109,7 +109,7 @@ class Command(BaseCommand):
                         repo_path = os.path.join(options["dest"], distribution.base_path)
                         to_export.append((repo_path, publication))
                     except ObjectDoesNotExist:
-                        logging.warn(
+                        logging.warning(
                             "No publication found for the repo published at '{}': skipping".format(
                                 distribution.base_path
                             )

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/models/task.py

@@ -4,7 +4,6 @@ Django models related to the Tasking system
 
 import logging
 import traceback
-from contextlib import suppress
 from datetime import timedelta
 from gettext import gettext as _
 
@@ -76,6 +75,21 @@ class Task(BaseModel, AutoAddObjPermsMixin):
     """
     Represents a task
 
+    The Tasks state machine works like a finite automaton without loops:
+    The initial state is WAITING.
+    Final states are COMPLETED, FAILED and CANCELED.
+    The possible transitions are:
+        WAITING -> RUNNING
+        WAITING ->* CANCELING
+        RUNNING -> COMPLETED
+        RUNNING -> FAILED
+        RUNNING ->* CANCELING
+        CANCELING -> CANCELED
+
+    The transitions to CANCELING (marked with *) are the only ones allowed to happen without
+    holding the tasks advisory lock. Canceling is meant to be initiated asyncronously by a sparate
+    process before signalling the worker via Postgres LISTEN.
+
     Fields:
 
         state (models.TextField): The state of the task
@@ -180,24 +194,26 @@ class Task(BaseModel, AutoAddObjPermsMixin):
         """Set the "core.task_user_dispatcher" role for the current user after creation."""
         self.add_roles_for_object_creator("core.task_user_dispatcher")
 
+    def _cleanup_progress_reports(self, state):
+        """Find any running progress-reports and set their states to the specified end-state."""
+        self.progress_reports.filter(state=TASK_STATES.RUNNING).update(state=state)
+
     def set_running(self):
         """
         Set this Task to the running state, save it, and log output in warning cases.
 
         This updates the :attr:`started_at` and sets the :attr:`state` to :attr:`RUNNING`.
         """
+        started_at = timezone.now()
         rows = Task.objects.filter(pk=self.pk, state=TASK_STATES.WAITING).update(
-            state=TASK_STATES.RUNNING, started_at=timezone.now()
+            state=TASK_STATES.RUNNING,
+            started_at=started_at,
         )
-        with suppress(AttributeError):
-            del self.state
-        with suppress(AttributeError):
-            del self.started_at
-        with suppress(AttributeError):
-            del self.finished_at
-        with suppress(AttributeError):
-            del self.error
-        if rows != 1:
+        if rows == 1:
+            self.state = TASK_STATES.RUNNING
+            self.started_at = started_at
+        else:
+            self.refresh_from_db()
             raise RuntimeError(
                 _("Attempt to set not waiting task {} to running from '{}'.").format(
                     self.pk, self.state
@@ -212,18 +228,16 @@ class Task(BaseModel, AutoAddObjPermsMixin):
         """
         # Only set the state to finished if it's running. This is important for when the task has
         # been canceled, so we don't move the task from canceled to finished.
+        finished_at = timezone.now()
         rows = Task.objects.filter(pk=self.pk, state=TASK_STATES.RUNNING).update(
-            state=TASK_STATES.COMPLETED, finished_at=timezone.now()
+            state=TASK_STATES.COMPLETED,
+            finished_at=finished_at,
         )
-        with suppress(AttributeError):
-            del self.state
-        with suppress(AttributeError):
-            del self.started_at
-        with suppress(AttributeError):
-            del self.finished_at
-        with suppress(AttributeError):
-            del self.error
-        if rows != 1:
+        if rows == 1:
+            self.state = TASK_STATES.COMPLETED
+            self.finished_at = finished_at
+        else:
+            self.refresh_from_db()
             # If the user requested to cancel this task while the worker finished it, we leave it
             # as it is, but accept this is not an error condition.
             if self.state != TASK_STATES.CANCELING:
@@ -232,6 +246,7 @@ class Task(BaseModel, AutoAddObjPermsMixin):
                         self.pk, self.state
                     )
                 )
+        self._cleanup_progress_reports(TASK_STATES.COMPLETED)
 
     def set_failed(self, exc, tb):
         """
@@ -244,26 +259,26 @@ class Task(BaseModel, AutoAddObjPermsMixin):
             exc (Exception): The exception raised by the task.
             tb (traceback): Traceback instance for the current exception.
         """
+        finished_at = timezone.now()
         tb_str = "".join(traceback.format_tb(tb))
+        error = exception_to_dict(exc, tb_str)
         rows = Task.objects.filter(pk=self.pk, state=TASK_STATES.RUNNING).update(
             state=TASK_STATES.FAILED,
-            finished_at=timezone.now(),
-            error=exception_to_dict(exc, tb_str),
+            finished_at=finished_at,
+            error=error,
         )
-        with suppress(AttributeError):
-            del self.state
-        with suppress(AttributeError):
-            del self.started_at
-        with suppress(AttributeError):
-            del self.finished_at
-        with suppress(AttributeError):
-            del self.error
-        if rows != 1:
+        if rows == 1:
+            self.state = TASK_STATES.FAILED
+            self.finished_at = finished_at
+            self.error = error
+        else:
+            self.refresh_from_db()
             raise RuntimeError(
                 _("Attempt to set not running task {} to failed from '{}'.").format(
                     self.pk, self.state
                 )
             )
+        self._cleanup_progress_reports(TASK_STATES.FAILED)
 
     def set_canceling(self):
         """
@@ -274,15 +289,10 @@ class Task(BaseModel, AutoAddObjPermsMixin):
         rows = Task.objects.filter(pk=self.pk, state__in=TASK_INCOMPLETE_STATES).update(
             state=TASK_STATES.CANCELING,
         )
-        with suppress(AttributeError):
-            del self.state
-        with suppress(AttributeError):
-            del self.started_at
-        with suppress(AttributeError):
-            del self.finished_at
-        with suppress(AttributeError):
-            del self.error
-        if rows != 1:
+        if rows == 1:
+            self.state = TASK_STATES.CANCELING
+        else:
+            self.refresh_from_db()
             raise RuntimeError(
                 _("Attempt to set not incomplete task {} to canceling from '{}'.").format(
                     self.pk, self.state
@@ -295,50 +305,40 @@ class Task(BaseModel, AutoAddObjPermsMixin):
         """
         # Make sure this function was called with a proper final state
         assert final_state in [TASK_STATES.CANCELED, TASK_STATES.FAILED]
+        finished_at = timezone.now()
         task_data = {}
         if reason:
             task_data["error"] = {"reason": reason}
         rows = Task.objects.filter(pk=self.pk, state=TASK_STATES.CANCELING).update(
             state=final_state,
-            finished_at=timezone.now(),
+            finished_at=finished_at,
             **task_data,
         )
-        with suppress(AttributeError):
-            del self.state
-        with suppress(AttributeError):
-            del self.started_at
-        with suppress(AttributeError):
-            del self.finished_at
-        with suppress(AttributeError):
-            del self.error
-        if rows != 1:
+        if rows == 1:
+            self.state = final_state
+            self.finished_at = finished_at
+            if reason:
+                self.error = task_data["error"]
+        else:
+            self.refresh_from_db()
             raise RuntimeError(
                 _("Attempt to set not canceling task {} to canceled from '{}'.").format(
                     self.pk, self.state
                 )
             )
+        self._cleanup_progress_reports(final_state)
 
     def unblock(self):
         # This should be safe to be called without holding the lock.
-        Task.objects.filter(pk=self.pk).update(unblocked_at=timezone.now())
-        with suppress(AttributeError):
-            del self.unblocked_at
-
-    # Example taken from here:
-    # https://docs.djangoproject.com/en/3.2/ref/models/instances/#refreshing-objects-from-database
-    def refresh_from_db(self, using=None, fields=None, **kwargs):
-        # fields contains the name of the deferred field to be
-        # loaded.
-        if fields is not None:
-            fields = set(fields)
-            deferred_fields = {
-                field for field in self.get_deferred_fields() if not field.startswith("enc_")
-            }
-            # If any state related deferred field is going to be loaded
-            if fields.intersection(deferred_fields):
-                # then load all of them
-                fields = fields.union(deferred_fields)
-        super().refresh_from_db(using, fields, **kwargs)
+        unblocked_at = timezone.now()
+        rows = Task.objects.filter(pk=self.pk).update(unblocked_at=unblocked_at)
+        if rows == 1:
+            self.unblocked_at = unblocked_at
+        else:
+            self.refresh_from_db()
+            raise RuntimeError(
+                _("Falied to set task {} unblocked in state '{}'.").format(self.pk, self.state)
+            )
 
     class Meta:
         indexes = [

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/openpgp.py

@@ -289,15 +289,15 @@ def analyze_signature(data, pubkey, signed_packet_type, signed_packet):
             if version == 4:
                 hash_payload = b"\x99" + len(signed_packet).to_bytes(2, "big") + signed_packet
             else:  # version == 5
-                hash_payload = b"\x9A" + len(signed_packet).to_bytes(4, "big") + signed_packet
+                hash_payload = b"\x9a" + len(signed_packet).to_bytes(4, "big") + signed_packet
         elif signature_type in [0x10, 0x11, 0x12, 0x13, 0x16, 0x30]:
             # 0x10 - 0x13 Certification of a user id or attribute
             # 0x16 Attested Key Signature
             # 0x30 Certification Revocation Signature
             if signed_packet_type == 13:
-                hash_payload = b"\xB4" + len(signed_packet).to_bytes(4, "big") + signed_packet
+                hash_payload = b"\xb4" + len(signed_packet).to_bytes(4, "big") + signed_packet
             elif signed_packet_type == 17:
-                hash_payload = b"\xD1" + len(signed_packet).to_bytes(4, "big") + signed_packet
+                hash_payload = b"\xd1" + len(signed_packet).to_bytes(4, "big") + signed_packet
             else:
                 raise ValueError("Out of band user ID or attribute signature.")
         elif signature_type in [0x1F, 0x20, 0x30]:
@@ -318,18 +318,18 @@ def analyze_signature(data, pubkey, signed_packet_type, signed_packet):
         if version == 4:
             h.update(b"\x99" + len(pubkey).to_bytes(2, "big") + pubkey)
         else:  # version == 5
-            h.update(b"\x9A" + len(pubkey).to_bytes(4, "big") + pubkey)
+            h.update(b"\x9a" + len(pubkey).to_bytes(4, "big") + pubkey)
         h.update(hash_payload)
         if version == 4:
             h.update(
                 data[: 6 + hashed_size]
-                + b"\x04\xFF"
+                + b"\x04\xff"
                 + ((6 + hashed_size) % (1 << 32)).to_bytes(4, "big")
             )
         else:  # version == 5
             h.update(
                 data[: 6 + hashed_size]
-                + b"\x05\xFF"
+                + b"\x05\xff"
                 + ((6 + hashed_size) % (1 << 64)).to_bytes(8, "big")
             )
         if not h.digest().startswith(canary):

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/replica.py

@@ -27,7 +27,7 @@ class ReplicaContext(PulpContext):
         if err:
             self.err_buf += message
             if nl:
-                _logger.warn("{}", self.err_buf)
+                _logger.warning("{}", self.err_buf)
                 self.err_buf = ""
         else:
             self.out_buf += message

{pulpcore-3.77.1 → pulpcore-3.79.0}/pulpcore/app/serializers/base.py

@@ -112,7 +112,7 @@ class _DetailFieldMixin(HrefPrnFieldMixin):
             if view_name_pattern:
                 view_name = _MatchingRegexViewName(view_name_pattern)
             else:
-                log.warn(
+                log.warning(
                     _(
                         "Please provide either 'view_name' or 'view_name_pattern' for {} on {}."
                     ).format(self.__class__.__name__, traceback.extract_stack()[-4][2])