From f1f140caea7b562eb77d5a95058e60ae8491ada2 Mon Sep 17 00:00:00 2001 From: David Eliahu Date: Sun, 23 Aug 2020 22:43:27 -0700 Subject: [PATCH] Update error messages --- pkg/types/clusterconfig/errors.go | 4 ++-- pkg/types/spec/errors.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/types/clusterconfig/errors.go b/pkg/types/clusterconfig/errors.go index 90d8e21a60..866e17afb6 100644 --- a/pkg/types/clusterconfig/errors.go +++ b/pkg/types/clusterconfig/errors.go @@ -150,7 +150,7 @@ func ErrorOnDemandBaseCapacityGreaterThanMax(onDemandBaseCapacity int64, max int func ErrorConfigCannotBeChangedOnUpdate(configKey string, prevVal interface{}) error { return errors.WithStack(&errors.Error{ Kind: ErrConfigCannotBeChangedOnUpdate, - Message: fmt.Sprintf("modifying %s in a running cluster is not supported, please set %s to its previous value: %s", configKey, configKey, s.UserStr(prevVal)), + Message: fmt.Sprintf("modifying %s in a running cluster is not supported, please set %s to its previous value (%s)", configKey, configKey, s.UserStr(prevVal)), }) } @@ -246,6 +246,6 @@ func ErrorCantOverrideDefaultTag() error { func ErrorSSLCertificateARNNotFound(sslCertificateARN string, region string) error { return errors.WithStack(&errors.Error{ Kind: ErrSSLCertificateARNNotFound, - Message: fmt.Sprintf("unable to find the specified ssl certificate in region %s: %s", region, sslCertificateARN), + Message: fmt.Sprintf("unable to find the specified ssl certificate in %s: %s", region, sslCertificateARN), }) } diff --git a/pkg/types/spec/errors.go b/pkg/types/spec/errors.go index 59d9ce33b2..cd67eed473 100644 --- a/pkg/types/spec/errors.go +++ b/pkg/types/spec/errors.go @@ -267,7 +267,7 @@ func ErrorInvalidTensorFlowModelPath() error { func ErrorMissingModel(predictorType userconfig.PredictorType) error { return errors.WithStack(&errors.Error{ Kind: ErrMissingModel, - Message: fmt.Sprintf("at least one model must be specified for %s predictor type; use fields %s:%s or %s:%s to add model(s)", predictorType, userconfig.PredictorKey, userconfig.ModelPathKey, userconfig.PredictorKey, userconfig.ModelsKey), + Message: fmt.Sprintf("at least one model must be specified for the %s predictor type; use fields %s.%s or %s.%s to add model(s)", predictorType, userconfig.PredictorKey, userconfig.ModelPathKey, userconfig.PredictorKey, userconfig.ModelsKey), }) }