prefect 3.7.8.dev2__tar.gz → 3.7.8.dev4__tar.gz

{prefect-3.7.8.dev2 → prefect-3.7.8.dev4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: prefect
-Version: 3.7.8.dev2
+Version: 3.7.8.dev4
 Summary: Workflow orchestration and management.
 Project-URL: Changelog, https://github.com/PrefectHQ/prefect/releases
 Project-URL: Documentation, https://docs.prefect.io

prefect-3.7.8.dev4/src/prefect/_build_info.py

@@ -0,0 +1,5 @@
+# Generated by versioningit
+__version__ = "3.7.8.dev4"
+__build_date__ = "2026-07-09 09:24:59.772054+00:00"
+__git_commit__ = "467544da6cd2526a549c1003d8a991c95a514392"
+__dirty__ = False

{prefect-3.7.8.dev2 → prefect-3.7.8.dev4}/src/prefect/_internal/schemas/validators.py

@@ -321,6 +321,7 @@ def normalize_rrule_string(
     rrule_string: str,
     *,
     now: Optional[datetime.datetime] = None,
+    timezone: Optional[str] = None,
 ) -> str:
     """Inject a `DTSTART` line into an rrule string if it doesn't already have one.
 
@@ -350,8 +351,16 @@ def normalize_rrule_string(
       datetimes, ~3 MB).
 
     Strings that already contain a `DTSTART` are returned unchanged.
+
+    The injected anchor is written as a floating (offset-less) `DTSTART`,
+    which `RRuleSchedule.to_rrule` later relabels with the schedule's
+    timezone. `timezone` is the schedule's IANA timezone (defaulting to
+    UTC); the recent anchor's wall-clock digits are computed in it so the
+    floating anchor lands on the intended local instant rather than being
+    shifted by the timezone's UTC offset (PrefectHQ/prefect#22455).
     """
     import dateutil.rrule
+    import dateutil.tz
 
     # Already anchored — leave it alone.
     if "DTSTART" in rrule_string.upper():
@@ -400,10 +409,22 @@ def normalize_rrule_string(
     unit_seconds = 1 if freq == dateutil.rrule.SECONDLY else 60
     period_seconds = unit_seconds * interval
 
+    # The injected `DTSTART` is serialized as floating (no offset/`Z`) and
+    # `RRuleSchedule.to_rrule` later relabels it with the schedule's timezone
+    # via `.replace(tzinfo=...)`, which keeps the wall-clock digits without
+    # converting the offset. The anchor's wall-clock digits must therefore be
+    # computed in the schedule's own timezone; computing them in UTC shifts
+    # every occurrence forward by the timezone's offset, suppressing runs on
+    # the deploy day (PrefectHQ/prefect#22455).
+    anchor_tz = dateutil.tz.gettz(timezone) if timezone else datetime.timezone.utc
+    if anchor_tz is None:
+        anchor_tz = datetime.timezone.utc
+
     if now is None:
-        now = datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None)
-    elif now.tzinfo is not None:
-        now = now.astimezone(datetime.timezone.utc).replace(tzinfo=None)
+        now = datetime.datetime.now(datetime.timezone.utc)
+    if now.tzinfo is None:
+        now = now.replace(tzinfo=datetime.timezone.utc)
+    now = now.astimezone(anchor_tz).replace(tzinfo=None)
 
     delta_seconds = (now - DEFAULT_RRULE_ANCHOR).total_seconds()
     if delta_seconds <= 0:
@@ -443,7 +464,9 @@ def normalize_schedule_rrule(schedule: _T) -> _T:
     rrule = getattr(schedule, "rrule", None)
     if not isinstance(rrule, str):
         return schedule
-    normalized = normalize_rrule_string(rrule)
+    normalized = normalize_rrule_string(
+        rrule, timezone=getattr(schedule, "timezone", None)
+    )
     if normalized == rrule:
         return schedule
     return type(schedule)(rrule=normalized, timezone=schedule.timezone)  # type: ignore[call-arg]

{prefect-3.7.8.dev2 → prefect-3.7.8.dev4}/src/prefect/events/clients.py

@@ -436,8 +436,17 @@ class PrefectEventsClient(EventsClient):
         # Clear the unconfirmed events here, because they are going back through emit
         # and will be added again through the normal checkpointing process
         self._unconfirmed_events = []
-        for event in events_to_resend:
-            await self.emit(event)
+        try:
+            while events_to_resend:
+                event = events_to_resend.pop(0)
+                await self.emit(event)
+        except Exception:
+            # If a resend fails partway through (emit() has its own retry
+            # budget), restore the events that were never attempted so a later
+            # reconnect can retry them. The event whose emit() failed was
+            # already added back to the buffer by emit() itself.
+            self._unconfirmed_events.extend(events_to_resend)
+            raise
         logger.debug("Finished resending unconfirmed events.")
         self._start_checkpoint_task()
 
@@ -452,17 +461,44 @@ class PrefectEventsClient(EventsClient):
 
     async def _checkpoint_loop(self) -> None:
         """Periodically checkpoint unconfirmed events on a time interval,
-        independent of the count-based checkpoint in _checkpoint."""
+        independent of the count-based checkpoint in _checkpoint.
+
+        If the connection has been lost (e.g. a server restart closed it),
+        reconnects and resends the unconfirmed events rather than leaving them
+        stranded until the next emit."""
         while True:
             await asyncio.sleep(self._checkpoint_interval)
-            if self._websocket and self._unconfirmed_events:
-                try:
+            if not self._unconfirmed_events:
+                continue
+            try:
+                if self._websocket:
                     await self._force_checkpoint()
+                    continue
+                # The connection was lost while events were still unconfirmed;
+                # reconnect to resend them rather than waiting for the next emit.
+                await self._reconnect()
+            except ConnectionClosed:
+                # The connection died since the last checkpoint; reconnect and
+                # resend the unconfirmed events. _reconnect() tears down the
+                # dead connection before connecting again.
+                try:
+                    await self._reconnect()
                 except Exception:
                     logger.debug(
-                        "Time-based checkpoint failed, will retry next interval.",
+                        "Reconnecting during time-based checkpoint failed, "
+                        "will retry next interval.",
                         exc_info=True,
                     )
+                    continue
+            except Exception:
+                logger.debug(
+                    "Time-based checkpoint failed, will retry next interval.",
+                    exc_info=True,
+                )
+                continue
+            # A successful _reconnect() resent the unconfirmed events and
+            # started a replacement checkpoint task, so this task is done.
+            return
 
     async def _force_checkpoint(self) -> None:
         """Checkpoint all unconfirmed events unconditionally."""

{prefect-3.7.8.dev2 → prefect-3.7.8.dev4}/src/prefect/runner/storage.py

@@ -1,8 +1,11 @@
 from __future__ import annotations
 
+import os
 import re
 import shutil
+import stat
 import subprocess
+import sys
 import warnings
 from copy import deepcopy
 from pathlib import Path
@@ -31,6 +34,41 @@ from prefect.logging.loggers import get_logger
 from prefect.utilities.collections import visit_collection
 
 
+def _clear_read_only_attributes(path: Path) -> None:
+    """
+    Recursively clear the read-only attribute on files within a directory.
+
+    On Windows, git writes pack and loose object files as read-only (mode
+    `0o444`). A subsequent `git pull` that needs to repack or replace those
+    files then fails with `[WinError 5] Access is denied`. Marking the files
+    as writable beforehand lets git manage them. This is a no-op on
+    non-Windows platforms, where read-only object files do not block git.
+    """
+    if sys.platform != "win32":
+        return
+    if not path.exists():
+        return
+    for entry in path.rglob("*"):
+        try:
+            if entry.is_file() and not entry.is_symlink():
+                os.chmod(entry, stat.S_IWRITE | stat.S_IREAD)
+        except OSError:
+            # Best effort: if a single file can't be made writable, let git
+            # surface the underlying error rather than masking it here.
+            continue
+
+
+def _rmtree_including_read_only(path: Path) -> None:
+    """
+    Remove a directory tree, first clearing read-only attributes so that
+    read-only git object files (mode `0o444` on Windows) do not raise
+    `[WinError 5] Access is denied` during deletion. This is a plain
+    `shutil.rmtree` on non-Windows platforms.
+    """
+    _clear_read_only_attributes(path)
+    shutil.rmtree(path)
+
+
 @runtime_checkable
 class RunnerStorage(Protocol):
     """
@@ -393,6 +431,11 @@ class GitRepository:
                     f"does not match the configured repository {configured_repo_url}"
                 )
 
+            # On Windows, git object files are read-only, which prevents git
+            # from repacking or replacing them during a pull. Clear the
+            # read-only attribute first to avoid `[WinError 5] Access is denied`.
+            _clear_read_only_attributes(git_dir)
+
             # Sparsely checkout the repository if directories are specified and the repo is not in sparse-checkout mode already
             if self._directories and not await self.is_sparsely_checked_out():
                 await run_process(
@@ -423,7 +466,7 @@ class GitRepository:
                     self._logger.error(
                         f"Failed to fetch latest changes with exit code {exc}"
                     )
-                    shutil.rmtree(self.destination)
+                    _rmtree_including_read_only(self.destination)
                     await self._clone_repo()
 
                 await run_process(
@@ -449,7 +492,7 @@ class GitRepository:
                     self._logger.error(
                         f"Failed to pull latest changes with exit code {exc}"
                     )
-                    shutil.rmtree(self.destination)
+                    _rmtree_including_read_only(self.destination)
                     await self._clone_repo()
 
         else:

prefect-3.7.8.dev2/src/prefect/_build_info.py

@@ -1,5 +0,0 @@
-# Generated by versioningit
-__version__ = "3.7.8.dev2"
-__build_date__ = "2026-07-07 09:13:51.928208+00:00"
-__git_commit__ = "3f47d765fea655c92f2d700fedee8635ea2d4d2f"
-__dirty__ = False