Skip to content

Commit

Permalink
warn wicket users that rack reset doesn't work yet (#6493)
Browse files Browse the repository at this point in the history
  • Loading branch information
iliana authored Aug 30, 2024
1 parent 8dad6f0 commit 344a5ac
Showing 1 changed file with 20 additions and 1 deletion.
21 changes: 20 additions & 1 deletion wicket/src/ui/panes/rack_setup.rs
Original file line number Diff line number Diff line change
Expand Up @@ -285,10 +285,29 @@ fn draw_rack_reset_popup(
"Rack Reset (DESTRUCTIVE!)",
style::header(true),
)]);
let body = Text::from(vec![Line::from(vec![Span::styled(
let mut body = Text::from(vec![Line::from(vec![Span::styled(
"Would you like to reset the rack to an uninitialized state?",
style::plain_text(),
)])]);
// One might see this warning and ask "why is this feature even
// here, then?" We do eventually want "rack reset" to work as a
// sort of factory reset, and the current implementation is a good
// starting point, so there's no sense in removing it (this is
// certainly not the only feature currently in this state).
//
// The warning is intended to remove the speed bump where someone
// has to find out the hard way that this doesn't work, without
// removing the speed bump where we're reminded of the feature that
// doesn't work yet.
body.lines.push(Line::from(""));
body.lines.push(Line::from(vec![
Span::styled("WARNING: ", style::warning()),
Span::styled(
"This does not work yet and will leave the rack \
in an unknown state (see omicron#3820)",
style::plain_text(),
),
]));
let buttons =
vec![ButtonText::new("Yes", "Y"), ButtonText::new("No", "N")];

Expand Down

0 comments on commit 344a5ac

Please sign in to comment.