Bug 1093226 - Allow reserving manual systems via "Reserve workflow"
Summary: Allow reserving manual systems via "Reserve workflow"
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: scheduler
Version: develop
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 19.0
Assignee: Amit Saha
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On: 851354 1014438
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-05-01 02:19 UTC by Amit Saha
Modified: 2018-02-06 00:41 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-11-25 07:18:03 UTC
Embargoed:


Attachments (Terms of Use)

Description Amit Saha 2014-05-01 02:19:22 UTC
Description of problem:

The "Reserve Workflow" (via the "Show Systems" button) allows selecting a system to reserve from the list of automated systems. However, it should also allow manual systems in this list.



Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:

See description


Additional info:

Comment 2 Amit Saha 2014-05-01 07:23:25 UTC
http://gerrit.beaker-project.org/#/c/3052/

Comment 3 Dan Callaghan 2014-05-06 00:37:45 UTC
Nick pointed out that we should bring this up on the beaker-devel list since it's a somewhat drastic change in behaviour -- people might not be expecting or wanting their Manual systems to be offered in the Reserve Workflow.

However my reasoning for allowing it is that people can already reserve those Manual systems by finding them in Systems -> Available, going to the System page, clicking Take and then provisioning. We are just making it easier to do that through the scheduler instead (which is something I want to encourage). The system access policies will of course continue to be respected, so if you don't want your system to be reserved by people then you should not grant them Reserve permission.

Comment 4 Dan Callaghan 2014-07-11 04:11:18 UTC
We haven't heard from anyone that this is a bad idea so let's go ahead with it in 0.18.

Comment 5 Nick Coghlan 2014-07-21 02:02:40 UTC
In the context of the new system page design, it occurs to me that with this change, Manual system should perhaps show a split Take/Shedule Reservation quick action button by default.

Comment 6 Nick Coghlan 2014-07-22 05:27:56 UTC
Alternatively, we could make the quick action button *just* Schedule Reservation, and require users to go down into the detail tabs to manage manual provisioning.

Comment 9 Dan Callaghan 2014-10-29 00:54:55 UTC
(In reply to Amit Saha from comment #8)
> In case it is not clear, loan a manual system to yourself and then you
> should be able to reserve it via the workflow.

The entire point of this patch is that you *don't* have to loan it to yourself first, any Manual system which you have 'reserve' permission on should be offered in Reserve Workflow when are picking a specific system.

Comment 10 Amit Saha 2014-10-29 01:01:18 UTC
(In reply to Dan Callaghan from comment #9)
> (In reply to Amit Saha from comment #8)
> > In case it is not clear, loan a manual system to yourself and then you
> > should be able to reserve it via the workflow.
> 
> The entire point of this patch is that you *don't* have to loan it to
> yourself first, any Manual system which you have 'reserve' permission on
> should be offered in Reserve Workflow when are picking a specific system.

of course, my mistake. Anyway, it works - which is important. :)

Comment 12 Dan Callaghan 2014-11-25 07:18:03 UTC
Beaker 19.0 has been released.


Note You need to log in before you can comment on or make changes to this bug.