Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

debug: tool to check for DNS resolution from meshed Pod #2729

Closed
draychev opened this issue Mar 3, 2021 · 1 comment
Closed

debug: tool to check for DNS resolution from meshed Pod #2729

draychev opened this issue Mar 3, 2021 · 1 comment
Labels
area/debugging Debug & Troubleshooting tooling priority/P2 P2 priority size/L 14 days (~2.5 weeks)

Comments

@draychev
Copy link
Contributor

draychev commented Mar 3, 2021

Augment tests to first perform a DNS lookup (nslookup, dig) for the service a client needs. For example dig bookstore.bookstore

This guarantees that required service has been registered with the Kubernetes DNS.

This check needs to happen after a Service has been created and before the e2e tests log-into the pod and run the test commands.
For example: https://github.com/openservicemesh/osm/blob/release-v0.8/tests/e2e/e2e_tcp_client_server_test.go#L71


Similar task: #2728


Update: let's introduce this new functionality in a new pkg/troubleshoot (name TBD). From that package we can use it in e2e or any tests. This new package should also eventually become a CLI for troubleshooting OSM.

@draychev draychev added the area/e2e-tests End-to-end tests label Mar 3, 2021
@draychev draychev added this to the v0.9.0 milestone Mar 3, 2021
@draychev draychev added the size/L 14 days (~2.5 weeks) label Mar 5, 2021
@michelleN michelleN changed the title e2e: Augment tests to first check for DNS resolution from meshed Pod tool to check for DNS resolution from meshed Pod Mar 9, 2021
@michelleN michelleN added the priority/P2 P2 priority label Mar 9, 2021
@draychev draychev modified the milestones: v0.9.0, v0.10.0 May 5, 2021
@draychev draychev added area/debugging Debug & Troubleshooting tooling and removed area/e2e-tests End-to-end tests labels Jun 22, 2021
@draychev draychev changed the title tool to check for DNS resolution from meshed Pod debug: tool to check for DNS resolution from meshed Pod Jun 22, 2021
@draychev draychev removed this from the v0.10.0 milestone Jun 28, 2021
@shashankram
Copy link
Member

We haven't seen a need to integrate DNS lookups in test. The existing logic has been working well lately. Closing this for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/debugging Debug & Troubleshooting tooling priority/P2 P2 priority size/L 14 days (~2.5 weeks)
Projects
None yet
Development

No branches or pull requests

3 participants