Skip to content

Change grafana port name to http#735

Merged
rshriram merged 1 commit intomasterfrom
costin-raw
Sep 13, 2017
Merged

Change grafana port name to http#735
rshriram merged 1 commit intomasterfrom
costin-raw

Conversation

@costinm
Copy link
Copy Markdown
Contributor

@costinm costinm commented Sep 11, 2017

Istio conventions are to use 'http' for http ports.

This allows adding ingress rules for grafana.

NONE

@istio-merge-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
We suggest the following additional approver: douglas-reid

Assign the PR to them by writing /assign @douglas-reid in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@douglas-reid
Copy link
Copy Markdown
Contributor

Does this imply that we want istio monitoring and features for grafana itself?

Copy link
Copy Markdown
Member

@rshriram rshriram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

http-grafana

@rshriram rshriram merged commit 1038720 into master Sep 13, 2017
@ldemailly ldemailly deleted the costin-raw branch September 15, 2017 21:29
rshriram pushed a commit that referenced this pull request Oct 30, 2017
vbatts pushed a commit to vbatts/istio that referenced this pull request Oct 31, 2017
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
kyessenov pushed a commit to kyessenov/istio that referenced this pull request Aug 13, 2018
Automatic merge from submit-queue.

[DO NOT MERGE] Auto PR to update dependencies of proxy

This PR will be merged automatically once checks are successful.
```release-note
none
```
howardjohn pushed a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
* add issue link to the ObjectMeta marshalling.

* fix test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants