Merge pull request #95 from fahlmant/console-error-fix

openshift-merge-robot · web-flow · commit bf5c45fcff6f · 2021-05-21T19:50:20.000+02:00
Add error message for failure to generate console
diff --git a/cmd/account/console.go b/cmd/account/console.go
@@ -83,6 +83,7 @@ func (o *consoleOptions) run() error {
 		aws.String(o.k8sclusterresourcefactory.Awscloudfactory.SessionName), aws.String(fmt.Sprintf("arn:aws:iam::%s:role/%s",
 			o.k8sclusterresourcefactory.AccountID, o.k8sclusterresourcefactory.Awscloudfactory.RoleName)))
 	if err != nil {
+		fmt.Fprintf(o.IOStreams.Out, "Generating console failed. If CCS cluster, customer removed or denied access to the BYOC role.")
 		return err
 	}
 	fmt.Fprintf(o.IOStreams.Out, "The AWS Console URL is:\n%s\n", consoleURL)

Original file line number	Diff line number	Diff line change
`@@ -83,6 +83,7 @@ func (o *consoleOptions) run() error {`
`83`	`83`	`aws.String(o.k8sclusterresourcefactory.Awscloudfactory.SessionName), aws.String(fmt.Sprintf("arn:aws:iam::%s:role/%s",`
`84`	`84`	`o.k8sclusterresourcefactory.AccountID, o.k8sclusterresourcefactory.Awscloudfactory.RoleName)))`
`85`	`85`	`if err != nil {`
	`86`	`+ fmt.Fprintf(o.IOStreams.Out, "Generating console failed. If CCS cluster, customer removed or denied access to the BYOC role.")`
`86`	`87`	`return err`
`87`	`88`	`}`
`88`	`89`	`fmt.Fprintf(o.IOStreams.Out, "The AWS Console URL is:\n%s\n", consoleURL)`