Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Detailed NGINX-Ingress-Controller status page config #2204

Closed
wants to merge 1 commit into from
Closed

Detailed NGINX-Ingress-Controller status page config #2204

wants to merge 1 commit into from

Conversation

ysolt
Copy link
Contributor

@ysolt ysolt commented Dec 22, 2016

Add detailed example for NGINX-Ingress-Controller status page configuration

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 22, 2016
@k8s-reviewable
Copy link

This change is Reviewable

@bprashanth
Copy link

Thanks for the pr, can you please move it to https://github.com/kubernetes/ingress/tree/master/examples (in examples/status-page/nginx/, we would presumably add a example/status-page/{aws,gce..etc} as well, though I'm not sure it's worth it since it's pretty obvious how to visit the status page on a cloudprovider)

@ysolt
Copy link
Contributor Author

ysolt commented Dec 23, 2016

@bprashanth alright! Already opened a new PR, please find it here kubernetes/ingress-nginx#84

@k8s-github-robot
Copy link

[APPROVALNOTIFIER] Needs approval from an approver in each of these OWNERS Files:

We suggest the following people:
cc @bprashanth
You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-github-robot k8s-github-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 7, 2017
@k8s-github-robot
Copy link

@ysolt PR needs rebase

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants