Paths

Table of Contentst

Differential D17837

Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun`
ClosedPublic
Actions

Authored by epriestley on May 6 2017, 2:56 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jan 15, 7:19 PM

	Unknown Object (File)
	Thu, Jan 9, 7:46 AM

	Unknown Object (File)
	Fri, Jan 3, 9:04 PM

	Unknown Object (File)
	Fri, Jan 3, 8:22 AM

	Unknown Object (File)
	Sat, Dec 21, 10:47 AM

	Unknown Object (File)
	Thu, Dec 19, 5:26 PM

	Unknown Object (File)
	Thu, Dec 19, 5:26 PM

	Unknown Object (File)
	Thu, Dec 19, 5:26 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12682: `storage --dry-run` actually looks rather wet when applying patches

Commits

rPb0df33e6ecd7: (stable) Don't apply patches or mark patches applied with `bin/storage upgrade…
rPbcd87e0e3f37: Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun`

Summary

Fixes T12682.

Test Plan

Ran bin/storage upgrade --dryrun repeatedly with un-applied patches, saw it not apply them and not mark them applied.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 6 2017, 2:56 AM

chad accepted this revision.May 6 2017, 2:56 AM

This revision is now accepted and ready to land.May 6 2017, 2:56 AM

Closed by commit rPbcd87e0e3f37: Don't apply patches or mark patches applied with `bin/storage upgrade --dryrun` (authored by epriestley, committed by epriestley). · Explain WhyMay 6 2017, 2:57 AM

This revision was automatically updated to reflect the committed changes.

Harbormaster completed remote builds in B16836: Diff 42910.May 6 2017, 2:58 AM

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

storage/

management/

workflow/

PhabricatorStorageManagementWorkflow.php

8 lines

Diff 42911

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php

Loading...