Page MenuHomePhabricator

Give Harbormaster Build Plans real policies
ClosedPublic

Authored by epriestley on Oct 23 2015, 9:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 12:10 AM
Unknown Object (File)
Wed, Apr 10, 4:48 PM
Unknown Object (File)
Fri, Apr 5, 6:17 PM
Unknown Object (File)
Tue, Apr 2, 12:55 AM
Unknown Object (File)
Mon, Mar 25, 11:48 PM
Unknown Object (File)
Sun, Mar 24, 7:55 AM
Unknown Object (File)
Mar 13 2024, 9:01 PM
Unknown Object (File)
Mar 11 2024, 6:18 PM
Subscribers
None

Details

Summary

Ref T9614. Currently, a lot of Build Plan behavior is covered by a global "can manage" policy.

One install in particular is experiencing difficulty with warring factions within engineering aborting one another's builds.

As a first step to remedy this, and also generally make Harbormaster more flexible and bring it in line with other applications in terms of policy power:

  • Give Build Plans normal view/edit policies.
  • Require "Can Edit" to run a plan manually.

Having "Can View" on plans may be a little weird in some cases (the status of a Buildable might be bad because of a build you can't see) but we can cross that bridge when we come to it.

Next change here will require "Can Edit" to abort a build. This will reasonably allow installs to reserve pause/abort for administrators/adults. (I might let anyone restart a plan, though?)

Test Plan
  • Created a new build plan.
  • Verified defaults were inherited from application defaults (swapped them around, too).
  • Saved build plan.
  • Edited policies.
  • Verified autoplans get the right policies.
  • Verified old plans got migrated properly.
  • Tried to run a plan I couldn't edit (denied).
  • Ran a plan from CLI with bin/harbormaster.
  • Tried to create a plan with an unprivileged user.

Diff Detail

Repository
rP Phabricator
Branch
harbor1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8381
Build 9623: Run Core Tests
Build 9622: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Give Harbormaster Build Plans real policies.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 26 2015, 7:25 PM
This revision was automatically updated to reflect the committed changes.