Skip to content

Commit

Permalink
fix: don't use hardcoded cluster domain (#962)
Browse files Browse the repository at this point in the history
  • Loading branch information
weisdd committed Mar 25, 2023
1 parent ee946e8 commit df64e32
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Expand Up @@ -48,7 +48,7 @@ func (r *ServiceReconciler) Reconcile(ctx context.Context, cr *v1beta1.Grafana,

// try to assign the admin url
if !cr.PreferIngress() {
status.AdminUrl = fmt.Sprintf("%v://%v.%v.svc.cluster.local:%d", getGrafanaServerProtocol(cr), service.Name, cr.Namespace,
status.AdminUrl = fmt.Sprintf("%v://%v.%v:%d", getGrafanaServerProtocol(cr), service.Name, cr.Namespace,
int32(GetGrafanaPort(cr)))
}

Expand Down
2 changes: 1 addition & 1 deletion docs/blog/v5-getting-started.md
Expand Up @@ -278,7 +278,7 @@ Should show you something like this

```yaml
status:
adminUrl: http://grafana-ingress-service.default.svc.cluster.local:3000
adminUrl: http://grafana-ingress-service.default:3000
dashboards:
- default/grafanadashboard-sample-ingress/6eaed1ab-0b0a-4d7f-bc46-0e3c1f58c8a8
stage: complete
Expand Down
2 changes: 1 addition & 1 deletion tests/e2e/example-test/00-assert.yaml
Expand Up @@ -3,7 +3,7 @@ kind: Grafana
metadata:
name: grafana
status:
adminUrl: http://grafana-service.grafana-operator-system.svc.cluster.local:3000
adminUrl: http://grafana-service.grafana-operator-system:3000
stage: complete
stageStatus: success
---
Expand Down

0 comments on commit df64e32

Please sign in to comment.