Skip to content
Snippets Groups Projects

Issue #3259814: If a Cron update fails after it has started the stage should be destroyed

Merged Issue #3259814: If a Cron update fails after it has started the stage should be destroyed
Merged Adam G-H requested to merge issue/automatic_updates-3259814:3259814-if-a-cron into 8.x-2.x

Merge request reports

Merged by Adam G-HAdam G-H 3 years ago (Feb 10, 2022 5:13pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
107 107 // @todo Use the queue to add update jobs allowing jobs to span multiple
108 108 // cron runs.
109 109 $recommended_version = $recommended_release->getVersion();
110 $is_applied = FALSE;
110 111 try {
111 112 $this->begin([
112 113 'drupal' => $recommended_version,
113 114 ]);
114 115 $this->stage();
115 116 $this->apply();
117 $is_applied = TRUE;
116 118 $this->destroy();
117 119 }
118 120 catch (StageValidationException $e) {
119 121 $this->logger->error(static::formatValidationException($e));
  • Ted Bowman
  • Kunal Sachdev added 3 commits

    added 3 commits

    Compare with previous version

  • Ted Bowman added 11 commits

    added 11 commits

    Compare with previous version

  • Ted Bowman added 1 commit

    added 1 commit

    • 66f86349 - change starting core version

    Compare with previous version

  • Ted Bowman
  • Ted Bowman
  • Ted Bowman
  • Ted Bowman
  • Kunal Sachdev added 1 commit

    added 1 commit

    Compare with previous version

  • 123 125 $this->logger->error($e->getMessage());
    124 126 return;
    125 127 }
    126
    128 finally {
    129 // Destroy the stage if it still remains, unless the error happened during
    130 // destroy() in which case it is not possible to destroy the stage.
  • Adam G-H
  • Adam G-H
  • Adam G-H
  • Adam G-H
  • Adam G-H
  • Adam G-H
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading