Problem/Motivation

If _drush_node_access_rebuild_progressive_process() triggers an exception, the drush node-access-rebuild-progressive command still returns an exit code of 0.
This means our CI can not detect if the command failed.

Proposed resolution

NodeAccessRebuildProgressiveCommands::accessRebuildProgressive() should return a non-zero exit code if the command fails.

Issue fork node_access_rebuild_progressive-3374396

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:

  • 3374396- Comparecompare
    • 1 hidden branch
  • 3374396-2.x Comparechanges, plain diff MR !2
About issue forks

Comments

prudloff created an issue. See original summary.

prudloff opened merge request !2

prudloff’s picture

prudloff commented
Status: Active » Needs review

  • shelane committed f331deb7 on 2.x authored by prudloff 
    Issue #3374396 by prudloff, shelane: Wrong exit code when...
shelane’s picture

shelane
Primary language English
commented
Status: Needs review » Fixed

It needed only a couple of minor updates to be able to merge. I did the work locally to make the changes and commit.

shelane closed merge request !2

Status: Fixed » Closed (fixed)

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