fix(proxy): don't use proxy for internal Grafana instances

controllers/client/grafana_client.go

@@ -143,7 +143,7 @@ func NewGrafanaClient(ctx context.Context, c client.Client, grafana *v1beta1.Gra
 	clientConfig := grapi.Config{
 		HTTPHeaders: nil,
 		Client: &http.Client{
-			Transport: NewInstrumentedRoundTripper(grafana.Name, metrics.GrafanaApiRequests),
+			Transport: NewInstrumentedRoundTripper(grafana.Name, metrics.GrafanaApiRequests, grafana.IsExternal()),
 			Timeout:   time.Second * timeout,
 		},
 		// TODO populate me

controllers/client/round_tripper.go

@@ -13,8 +13,14 @@ type instrumentedRoundTripper struct {
 	metric          *prometheus.CounterVec
 }
 
-func NewInstrumentedRoundTripper(relatedResource string, metric *prometheus.CounterVec) http.RoundTripper {
-	transport := http.DefaultTransport.(*http.Transport).Clone()
+func NewInstrumentedRoundTripper(relatedResource string, metric *prometheus.CounterVec, useProxy bool) http.RoundTripper {
+	transport := &http.Transport{}
+
+	// Default transport respects proxy settings
+	if useProxy {
+		transport = http.DefaultTransport.(*http.Transport).Clone()
+	}
+
 	transport.DisableKeepAlives = true
 	transport.MaxIdleConnsPerHost = -1
 	transport.TLSClientConfig.InsecureSkipVerify = true //nolint

controllers/fetchers/grafana_com_fetcher.go

@@ -42,7 +42,7 @@ func getLatestGrafanaComRevision(dashboard *v1beta1.GrafanaDashboard) (int, erro
 		return -1, err
 	}
 
-	client := client2.NewInstrumentedRoundTripper(fmt.Sprintf("%v/%v", dashboard.Namespace, dashboard.Name), metrics.GrafanaComApiRevisionRequests)
+	client := client2.NewInstrumentedRoundTripper(fmt.Sprintf("%v/%v", dashboard.Namespace, dashboard.Name), metrics.GrafanaComApiRevisionRequests, true)
 	response, err := client.RoundTrip(request)
 	if err != nil {
 		return -1, err

controllers/fetchers/url_fetcher.go

@@ -30,7 +30,7 @@ func FetchDashboardFromUrl(dashboard *v1beta1.GrafanaDashboard) ([]byte, error)
 		return nil, err
 	}
 
-	client := client2.NewInstrumentedRoundTripper(fmt.Sprintf("%v/%v", dashboard.Namespace, dashboard.Name), metrics.DashboardUrlRequests)
+	client := client2.NewInstrumentedRoundTripper(fmt.Sprintf("%v/%v", dashboard.Namespace, dashboard.Name), metrics.DashboardUrlRequests, true)
 	response, err := client.RoundTrip(request)
 	if err != nil {
 		return nil, err

docs/docs/_index.md

@@ -87,5 +87,5 @@ This is because especially the data sources contain secret information and we do
 
 ## Using a proxy server
 
-The Operator can use a proxy server when making requests to Grafana.
+The Operator can use a proxy server when fetching URL-based / Grafana.com dashboards or making requests to external Grafana instances.
 The proxy settings can be controlled through environment variables as documented [here](https://pkg.go.dev/golang.org/x/net/http/httpproxy#FromEnvironment).