G1: Display compute requirements in form submission results - #8
Merged
Merged
Conversation
"Start the virtual machines small, and back them up" was an action in the client's list that restated server figures, which now contradicted the sized compute requirements. It is guidance about those figures, so it belongs under them: requirement hints can now be gated on the answers, and the hint appears only when the servers are virtual machines. It no longer restates the figures it sits beneath, which is what made the two disagree.
"Provision the physical servers generously up front" had the same problem as its virtual-machine counterpart: an action restating figures that contradicted the sized rows. Moved to a hint gated on physical servers, keeping the headroom judgement and the RAID and multiple-disk guidance while dropping the figures. A deployment running both kinds now gets both hints, which is what "Both" means.
"It is fine to start smaller and grow into them" left it unclear whether the figures were the target or the starting point. Say it outright: the figures are a target rather than a day-one commitment, and name storage as the part that can start smaller. Read against the bare-metal hint, the pair now contrasts cleanly: grow into these figures on a VM, buy above them on hardware.
Both hints were wordier than a hint should be, and the virtual-machine one did not say that the figures are the recommendation. Lead with aiming for them, and allow a VM to sit slightly under only where its host has room to grow. The pair now reads as one contrast: aim for these on a VM, buy above them on hardware.
"Buy above these figures" read as an instruction without saying why. State what the figures are on hardware, and leave the buying decision to the reader: a minimum, with more being wise to allow for growth.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #7