Skip to content

Commit a13b346

Browse files
crawfordwking
authored andcommitted
asset/cluster: clean up logging messages
This removes explicit mention of Terraform in the log messages. We received feedback that this was confusing users and product requested that we remove it.
1 parent 863a1ec commit a13b346

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

pkg/asset/cluster/cluster.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,10 +111,10 @@ func (c *Cluster) Generate(parents asset.Parents) (err error) {
111111
return fmt.Errorf("no known platform")
112112
}
113113

114-
logrus.Infof("Using Terraform to create cluster...")
114+
logrus.Infof("Creating cluster...")
115115
stateFile, err := terraform.Apply(tmpDir, installConfig.Config.Platform.Name())
116116
if err != nil {
117-
err = errors.Wrap(err, "failed to run terraform")
117+
err = errors.Wrap(err, "failed to create cluster")
118118
}
119119

120120
data, err2 := ioutil.ReadFile(stateFile)

pkg/destroy/bootstrap/bootstrap.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import (
99
"github.com/openshift/installer/pkg/asset/cluster"
1010
"github.com/openshift/installer/pkg/terraform"
1111
"github.com/pkg/errors"
12-
"github.com/sirupsen/logrus"
1312
)
1413

1514
// Destroy uses Terraform to remove bootstrap resources.
@@ -50,7 +49,6 @@ func Destroy(dir string) (err error) {
5049
}
5150
}
5251

53-
logrus.Infof("Using Terraform to destroy bootstrap resources...")
5452
if platform == "libvirt" {
5553
_, err = terraform.Apply(tempDir, platform)
5654
if err != nil {

0 commit comments

Comments
 (0)