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)
	Sat, Apr 6, 4:45 AM

	Unknown Object (File)
	Fri, Apr 5, 1:06 PM

	Unknown Object (File)
	Fri, Apr 5, 11:31 AM

	Unknown Object (File)
	Fri, Apr 5, 6:24 AM

	Unknown Object (File)
	Fri, Apr 5, 5:13 AM

	Unknown Object (File)
	Fri, Apr 5, 4:04 AM

	Unknown Object (File)
	Fri, Apr 5, 3:03 AM

	Unknown Object (File)
	Tue, Apr 2, 9:22 PM

View All 87 Files

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...