Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding /debug url to echosrv +... #536

Merged
merged 7 commits into from
Aug 3, 2017
Merged

Adding /debug url to echosrv +... #536

merged 7 commits into from
Aug 3, 2017

Conversation

ldemailly
Copy link
Contributor

@ldemailly ldemailly commented Aug 3, 2017

Adding /debug url to echosrv
 that echoes back request and optionally dumps the env vars
( /debug?env=dump )

Adding (rfc/please tell me the right way) a docker image so I can run echorv on k8s

prints interesting stuff like:

Φορτίο version 0.2.0 echo debug server on echosrv-3493911808-z85j9 - request from 127.0.0.1:60762

GET /debug?env=dump HTTP/1.1 

headers:

User-Agent: curl/7.54.0
Accept: */*

body:



environment:

PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
HOSTNAME=echosrv-3493911808-z85j9
KUBERNETES_PORT=tcp://10.3.240.1:443
KUBERNETES_PORT_443_TCP=tcp://10.3.240.1:443
KUBERNETES_PORT_443_TCP_PROTO=tcp
KUBERNETES_PORT_443_TCP_PORT=443
KUBERNETES_PORT_443_TCP_ADDR=10.3.240.1
KUBERNETES_SERVICE_HOST=10.3.240.1
KUBERNETES_SERVICE_PORT=443
KUBERNETES_SERVICE_PORT_HTTPS=443
HOME=/

That echoes back request and optionally dumps the env vars
(Dog science… my first docker… probably all wrong…)
@istio-merge-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ldemailly

No associated issue. Update pull-request body to add a reference to an issue, or get approval with /approve no-issue

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

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

@ldemailly
Copy link
Contributor Author

/retest

@istio istio deleted a comment from istio-testing Aug 3, 2017
Copy link
Contributor

@douglas-reid douglas-reid left a comment

Choose a reason for hiding this comment

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

Seems fine.

@ldemailly ldemailly merged commit 0f6af17 into master Aug 3, 2017
@ldemailly ldemailly deleted the rawvm-register-2 branch August 3, 2017 22:55
rshriram pushed a commit that referenced this pull request Oct 30, 2017
* Adding /debug url to echosrv

That echoes back request and optionally dumps the env vars

* Adding rudimentary docker image

(Dog science… my first docker… probably all wrong…)

* Switch to layered builds

So it can auto build from docker swarm (testing that)


Former-commit-id: 0f6af17
vbatts pushed a commit to vbatts/istio that referenced this pull request Oct 31, 2017
* Adding /debug url to echosrv

That echoes back request and optionally dumps the env vars

* Adding rudimentary docker image

(Dog science… my first docker… probably all wrong…)

* Switch to layered builds

So it can auto build from docker swarm (testing that)


Former-commit-id: 0f6af17
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
* Adding /debug url to echosrv

That echoes back request and optionally dumps the env vars

* Adding rudimentary docker image

(Dog science… my first docker… probably all wrong…)

* Switch to layered builds

So it can auto build from docker swarm (testing that)


Former-commit-id: 0f6af17
rshriram pushed a commit to rshriram/istio that referenced this pull request Jul 31, 2018
howardjohn pushed a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
* Add health check to Citadel.

* Small fix.

* Small fix.
luksa pushed a commit to luksa/istio that referenced this pull request Sep 20, 2022
Co-authored-by: maistra-bot <null>
antonioberben pushed a commit to antonioberben/istio that referenced this pull request Jan 29, 2024
Signed-off-by: Grace Do <xgrace@gmail.com>
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.

None yet

4 participants