pkg/cli/admin/upgrade/recommend: "accepted ... via ConditionalUpdateRisk"

Make it clearer how '--accept ConditionalUpdateRisk' maps to a risk
like NonZonalAzureMachineSetScaling getting accepted, by turning the
previous:

  Reason: accepted NonZonalAzureMachineSetScaling

into:

  Reason: accepted NonZonalAzureMachineSetScaling via ConditionalUpdateRisk

Eventually we'll have an API that allows us to use the
conditional-update risk name itself (e.g. '--accept
NonZonalAzureMachineSetScaling') [1,2], but this 'via...' context will
hopefully help avoid confusion in the meantime.

[1]: openshift/enhancements#1807
[2]: https://issues.redhat.com/browse/OTA-1543

Author: wking

pkg/cli/admin/upgrade/recommend/recommend.go

@@ -333,11 +333,11 @@ func (o *options) Run(ctx context.Context) error {
 						}
 					} else {
 						if !o.quiet {
-							acceptContext := ""
+							reason := c.Reason
 							if accept.Has("ConditionalUpdateRisk") {
-								acceptContext = "accepted "
+								reason = "accepted %s via ConditionalUpdateRisk"
 							}
-							fmt.Fprintf(o.Out, "Update to %s %s=%s:\nImage: %s\nRelease URL: %s\nReason: %s%s\nMessage: %s\n", update.Release.Version, c.Type, c.Status, update.Release.Image, update.Release.URL, acceptContext, c.Reason, strings.ReplaceAll(c.Message, "\n", "\n  "))
+							fmt.Fprintf(o.Out, "Update to %s %s=%s:\nImage: %s\nRelease URL: %s\nReason: %s\nMessage: %s\n", update.Release.Version, c.Type, c.Status, update.Release.Image, update.Release.URL, reason, strings.ReplaceAll(c.Message, "\n", "\n  "))
 						}
 						issues.Insert("ConditionalUpdateRisk")
 					}