Browse code

warn on missing log and metric URLs for console

deads2k authored on 2015/11/10 04:11:26
Showing 1 changed files
... ...
@@ -317,12 +317,16 @@ func ValidateAssetConfig(config *api.AssetConfig) ValidationResults {
317 317
 		if _, loggingURLErrs := ValidateSecureURL(config.LoggingPublicURL, "loggingPublicURL"); len(loggingURLErrs) > 0 {
318 318
 			validationResults.AddErrors(loggingURLErrs...)
319 319
 		}
320
+	} else {
321
+		validationResults.AddWarnings(fielderrors.NewFieldInvalid("loggingPublicURL", "", "required to view aggregated container logs in the console"))
320 322
 	}
321 323
 
322 324
 	if len(config.MetricsPublicURL) > 0 {
323 325
 		if _, metricsURLErrs := ValidateSecureURL(config.MetricsPublicURL, "metricsPublicURL"); len(metricsURLErrs) > 0 {
324 326
 			validationResults.AddErrors(metricsURLErrs...)
325 327
 		}
328
+	} else {
329
+		validationResults.AddWarnings(fielderrors.NewFieldInvalid("metricsPublicURL", "", "required to view cluster metrics in the console"))
326 330
 	}
327 331
 
328 332
 	for i, scriptFile := range config.ExtensionScripts {