Problem/Motivation

In #3357941: Write helpful message to FailureMarker if script execution time was exceeded during stage life cycle the FailureMarker was going to be used if there were time outs must of which would not leave your code in an indeterminate state which is the purpose the FailureMarker file.

We document in the class doc but we should probably document that elsewhere.

If the PACKAGE_MANAGER_FAILURE.yml exists probably the only thing you can do is restore your code from a backup. We intentionally made this a file instead of using Drupal state service because depending which files did not get copied over you site might not even load. In this case the docs should let the user know that they should check for this existence of this file.

Proposed resolution

Lets document this in several places

  1. package_manager.api.php
  2. In https://www.drupal.org/docs/8/update/automatic-updates
  3. Project Browser docs?
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

tedbow created an issue. See original summary.

tedbow’s picture

Issue tags: +core-mvp, +beta target, +sprint
kunal.sachdev’s picture

Assigned: Unassigned » kunal.sachdev

kunal.sachdev’s picture

Assigned: kunal.sachdev » Unassigned
Status: Active » Needs review
omkar.podey’s picture

Assigned: Unassigned » omkar.podey
omkar.podey’s picture

Assigned: omkar.podey » kunal.sachdev
Status: Needs review » Needs work

3 nits and still need to address the remaining 2 points in proposed resolution.

kunal.sachdev’s picture

Title: Document what the existence of the PACKAGE_MANAGER_FAILURE.json means » Document what the existence of the PACKAGE_MANAGER_FAILURE.yml means
Issue summary: View changes
kunal.sachdev’s picture

Assigned: kunal.sachdev » tedbow
Status: Needs work » Needs review
wim leers’s picture

Assigned: tedbow » kunal.sachdev
Status: Needs review » Needs work

99% ready!

kunal.sachdev’s picture

Assigned: kunal.sachdev » Unassigned
Status: Needs work » Needs review
wim leers’s picture

Status: Needs review » Reviewed & tested by the community

LGTM!

phenaproxima made their first commit to this issue’s fork.

phenaproxima’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.