Merge branch 'master' into dashboard-fetch-rate-limit

api/integreatly/v1alpha1/grafana_types.go

@@ -130,6 +130,7 @@ type GrafanaHttpProxy struct {
 	Enabled   bool   `json:"enabled"`
 	URL       string `json:"url,omitempty"`
 	SecureURL string `json:"secureUrl,omitempty"`
+	NoProxy   string `json:"noProxy,omitempty"`
 }
 
 // GrafanaIngress provides a means to configure the ingress created

config/crd/bases/integreatly.org_grafanas.yaml

@@ -4415,6 +4415,8 @@ spec:
                     properties:
                       enabled:
                         type: boolean
+                      noProxy:
+                        type: string
                       secureUrl:
                         type: string
                       url:

controllers/grafanadashboard/dashboard_pipeline.go

@@ -54,6 +54,9 @@ type DashboardPipelineImpl struct {
 }
 
 func NewDashboardPipeline(client client.Client, dashboard *v1alpha1.GrafanaDashboard, ctx context.Context) DashboardPipeline {
+	if dashboard.Spec.ContentCacheDuration == nil {
+		dashboard.Spec.ContentCacheDuration = &metav1.Duration{Duration: 24 * time.Hour}
+	}
 	return &DashboardPipelineImpl{
 		Client:    client,
 		Dashboard: dashboard,

controllers/model/grafanaDeployment.go

@@ -578,6 +578,12 @@ func getContainers(cr *v1alpha1.Grafana, configHash, dsHash string) []v13.Contai
 				Value: cr.Spec.Deployment.HttpProxy.SecureURL,
 			})
 		}
+		if cr.Spec.Deployment.HttpProxy.NoProxy != "" {
+			envVars = append(envVars, v13.EnvVar{
+				Name:  "NO_PROXY",
+				Value: cr.Spec.Deployment.HttpProxy.NoProxy,
+			})
+		}
 	}
 
 	if cr.Spec.Deployment != nil && cr.Spec.Deployment.Env != nil {

controllers/model/grafanaDeployment_test.go

@@ -0,0 +1,69 @@
+package model
+
+import (
+	"testing"
+
+	grafanav1alpha1 "github.com/grafana-operator/grafana-operator/v4/api/integreatly/v1alpha1"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestGrafanaDeployment_httpProxy(t *testing.T) {
+	t.Run("noProxy is setting", func(t *testing.T) {
+		cr := &grafanav1alpha1.Grafana{
+			Spec: grafanav1alpha1.GrafanaSpec{
+				Deployment: &grafanav1alpha1.GrafanaDeployment{
+					HttpProxy: &grafanav1alpha1.GrafanaHttpProxy{
+						Enabled:   true,
+						URL:       "http://1.2.3.4",
+						SecureURL: "http://1.2.3.4",
+						NoProxy:   ".svc.cluster.local,.svc",
+					},
+				},
+			},
+		}
+		deployment := GrafanaDeployment(cr, "", "")
+		for _, container := range deployment.Spec.Template.Spec.Containers {
+			if container.Name != "grafana" {
+				continue
+			}
+
+			noProxyExist := false
+			for _, env := range container.Env {
+				if env.Name == "NO_PROXY" {
+					noProxyExist = true
+					assert.Equal(t, ".svc.cluster.local,.svc", env.Value)
+				}
+			}
+			assert.True(t, noProxyExist)
+		}
+	})
+
+	t.Run("noProxy is not setting", func(t *testing.T) {
+		cr := &grafanav1alpha1.Grafana{
+			Spec: grafanav1alpha1.GrafanaSpec{
+				Deployment: &grafanav1alpha1.GrafanaDeployment{
+					HttpProxy: &grafanav1alpha1.GrafanaHttpProxy{
+						Enabled:   true,
+						URL:       "http://1.2.3.4",
+						SecureURL: "http://1.2.3.4",
+					},
+				},
+			},
+		}
+		deployment := GrafanaDeployment(cr, "", "")
+		for _, container := range deployment.Spec.Template.Spec.Containers {
+			if container.Name != "grafana" {
+				continue
+			}
+
+			noProxyExist := false
+			for _, env := range container.Env {
+				if env.Name == "NO_PROXY" {
+					noProxyExist = true
+				}
+			}
+			assert.False(t, noProxyExist)
+		}
+	})
+}

deploy/manifests/latest/crds.yaml

@@ -5038,6 +5038,8 @@ spec:
                     properties:
                       enabled:
                         type: boolean
+                      noProxy:
+                        type: string
                       secureUrl:
                         type: string
                       url:

documentation/api.md

@@ -11868,6 +11868,13 @@ GrafanaHttpProxy provides a means to configure the Grafana deployment to use an
           <br/>
         </td>
         <td>true</td>
+      </tr><tr>
+        <td><b>noProxy</b></td>
+        <td>string</td>
+        <td>
+          <br/>
+        </td>
+        <td>false</td>
       </tr><tr>
         <td><b>secureUrl</b></td>
         <td>string</td>