فهرست منبع

Merge branch 'master' into develop

Matt Bolt 5 سال پیش
والد
کامیت
24d821c57c
2فایلهای تغییر یافته به همراه11 افزوده شده و 2 حذف شده
  1. 3 2
      pkg/costmodel/router.go
  2. 8 0
      pkg/env/costmodelenv.go

+ 3 - 2
pkg/costmodel/router.go

@@ -688,6 +688,7 @@ type ConfigWatchers struct {
 // captures the panic event in sentry
 // captures the panic event in sentry
 func capturePanicEvent(err string, stack string) {
 func capturePanicEvent(err string, stack string) {
 	msg := fmt.Sprintf("Panic: %s\nStackTrace: %s\n", err, stack)
 	msg := fmt.Sprintf("Panic: %s\nStackTrace: %s\n", err, stack)
+	klog.V(1).Infoln(msg)
 	sentry.CurrentHub().CaptureEvent(&sentry.Event{
 	sentry.CurrentHub().CaptureEvent(&sentry.Event{
 		Level:   sentry.LevelError,
 		Level:   sentry.LevelError,
 		Message: msg,
 		Message: msg,
@@ -718,11 +719,11 @@ func Initialize(additionalConfigWatchers ...ConfigWatchers) {
 	klog.InitFlags(nil)
 	klog.InitFlags(nil)
 	flag.Set("v", "3")
 	flag.Set("v", "3")
 	flag.Parse()
 	flag.Parse()
-	klog.V(1).Infof("Starting cost-model (git commit \"%s\")", gitCommit)
+	klog.V(1).Infof("Starting cost-model (git commit \"%s\")", env.GetAppVersion())
 
 
 	var err error
 	var err error
 	if errorReportingEnabled {
 	if errorReportingEnabled {
-		err = sentry.Init(sentry.ClientOptions{Release: gitCommit})
+		err = sentry.Init(sentry.ClientOptions{Release: env.GetAppVersion()})
 		if err != nil {
 		if err != nil {
 			klog.Infof("Failed to initialize sentry for error reporting")
 			klog.Infof("Failed to initialize sentry for error reporting")
 		} else {
 		} else {

+ 8 - 0
pkg/env/costmodelenv.go

@@ -1,6 +1,8 @@
 package env
 package env
 
 
 const (
 const (
+	AppVersionEnvVar = "APP_VERSION"
+
 	AWSAccessKeyIDEnvVar     = "AWS_ACCESS_KEY_ID"
 	AWSAccessKeyIDEnvVar     = "AWS_ACCESS_KEY_ID"
 	AWSAccessKeySecretEnvVar = "AWS_SECRET_ACCESS_KEY"
 	AWSAccessKeySecretEnvVar = "AWS_SECRET_ACCESS_KEY"
 	AWSClusterIDEnvVar       = "AWS_CLUSTER_ID"
 	AWSClusterIDEnvVar       = "AWS_CLUSTER_ID"
@@ -42,6 +44,12 @@ const (
 	KubeConfigPathEnvVar = "KUBECONFIG_PATH"
 	KubeConfigPathEnvVar = "KUBECONFIG_PATH"
 )
 )
 
 
+// GetAWSAccessKeyID returns the environment variable value for AWSAccessKeyIDEnvVar which represents
+// the AWS access key for authentication
+func GetAppVersion() string {
+	return Get(AppVersionEnvVar, "Pre-1.68.0")
+}
+
 // GetAWSAccessKeyID returns the environment variable value for AWSAccessKeyIDEnvVar which represents
 // GetAWSAccessKeyID returns the environment variable value for AWSAccessKeyIDEnvVar which represents
 // the AWS access key for authentication
 // the AWS access key for authentication
 func GetAWSAccessKeyID() string {
 func GetAWSAccessKeyID() string {