Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GUI: TPM - Enable / Disable #590

Closed
derick-montague opened this issue May 1, 2019 · 11 comments
Closed

GUI: TPM - Enable / Disable #590

derick-montague opened this issue May 1, 2019 · 11 comments
Assignees
Labels
GUI Catch all tag for GUI stories UI Design Indicates the GUI story is for Design UI FED Indicates a GUI Front End Development story
Milestone

Comments

@derick-montague
Copy link
Contributor

derick-montague commented May 1, 2019

Stakeholders

SME: Chris Engle, Nick Bofferding
Design Researcher: @nicoleconser
UX Designer: @jandraa
FED:

Use Case

As a user I need a way to enable the 'TPM required policy' so that my system does not run if the TPM is faulty. I need a way to disable the policy so that my system does run regardless of if the TPM is functional or not.

Our proposal is to place this functionality in the Server Power Operations panel given that the required policy will really only matter and take effect during reboot/power on. Additionally, this is where users would be configuring 'how' the boot occurs.

InVision Prototype

https://ibm.invisionapp.com/share/8ENYRVXAPFD#/319150964_3-7-1A_Boot_Options_Empty (Navigate by clicking on any flashing blue rectangles or using the keyboard arrows).

Since the interaction is a simple on/off, we added the component in the 'on' state to the boot options flow. To navigate the prototype, click on flashing blue rectangles or use the right and left keyboard arrows.

Design Issue (phosphor-webui)

openbmc/phosphor-webui#90

Development Issue


References/Resources


@derick-montague derick-montague added UI Design Indicates the GUI story is for Design GUI Catch all tag for GUI stories labels May 1, 2019
@jandraa
Copy link

jandraa commented May 2, 2019

This story refers to the TPM policy, correct?

@derick-montague
Copy link
Contributor Author

@jandraa - are you doing research and discovery on this as part of #403?

@jandraa
Copy link

jandraa commented May 9, 2019

@jandraa - are you doing research and discovery on this as part of #403?

Yes, definitely!
I am looking at all of TPM (the sensor for the FRU, enable/disable for the policy, and virtual TPM configurations). I am doing it this way to understand the wider use case and be able to get a better idea of where these things should live and in proximity to what.

So far, I have not created any designs yet. Maybe at that point it would make sense for them to be separate issues, especially if this ends up in a different panel than the sensor, for example.

@derick-montague
Copy link
Contributor Author

Maybe at that point it would make sense for them to be separate issues, especially if this ends up in a different panel than the sensor, for example.

@gtmills and I discussed that today. That seems odd, but maybe with more discovery it makes sense. I will add this as a dependency.

@jandraa
Copy link

jandraa commented May 10, 2019

Different issues seems odd or different panels?

@derick-montague
Copy link
Contributor Author

That TPM is on its own panel, but I could be wrong.

@jandraa
Copy link

jandraa commented May 10, 2019

My thinking, as of now, is that the actual TPM sensor would be on the Hardware Status with the other FRUs, and the policy would be with security stuff (I'm still figuring out what all 'security stuff' would entail).

@gtmills
Copy link

gtmills commented May 11, 2019

@derick-montague Why the blocked by the #403 ?

@rfrandse rfrandse added this to the A.1.999 milestone Jun 6, 2019
@jandraa
Copy link

jandraa commented Jun 13, 2019

Will be placed with the Server Power operations (power on, orderly shutdown, reboot, etc.)

The logic is the required policy will really only matter and take effect during reboot/power on. Additionally, this is where users would be configuring 'how' the boot occurs.

Chris and Nick agree.

@derick-montague derick-montague changed the title TPM: Enable Disable GUI: TPM - Enable / Disable Jun 25, 2019
@jandraa jandraa self-assigned this Jun 26, 2019
@jandraa jandraa added the User Feedback Needed Indicates a design story is blocked waiting for a sponsor user / community feedback label Jul 2, 2019
@gtmills gtmills added Design Sign-off Needed UI FED Indicates a GUI Front End Development story and removed User Feedback Needed Indicates a design story is blocked waiting for a sponsor user / community feedback labels Jul 9, 2019
@derick-montague derick-montague assigned dixsie and unassigned jandraa Jul 9, 2019
@derick-montague derick-montague modified the milestones: A.1.999, A.1.193 Jul 16, 2019
@dixsie
Copy link
Contributor

dixsie commented Aug 12, 2019

@rfrandse
Copy link

@rfrandse rfrandse modified the milestones: A.1.193, Not Applicable Sep 24, 2019
@rfrandse rfrandse removed this from the Not Applicable milestone Apr 29, 2020
@rfrandse rfrandse added this to the A.1.211 milestone Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI Catch all tag for GUI stories UI Design Indicates the GUI story is for Design UI FED Indicates a GUI Front End Development story
Projects
None yet
Development

No branches or pull requests

5 participants