Skip to content

Conversation

@rohansonecha
Copy link
Collaborator

@rohansonecha rohansonecha commented Nov 4, 2025

Logged the entire response except for the response.items in case this fails for some reason. Then we grab pods = response.items, log a summary indicating how many pods were returned and then log detailed information about each pod. We log every field that the k8s Pod object has except for PodSpec which would not be very helpful for debugging here and is extremely verbose.

Restarted the api server, observed the logs, and verified that they look as expected/useful.

Tested (run the relevant ones):

  • Code formatting: install pre-commit (auto-check on commit) or bash format.sh
  • Any manual or new tests for this PR (please specify below)
  • All smoke tests: /smoke-test (CI) or pytest tests/test_smoke.py (local)
  • Relevant individual tests: /smoke-test -k test_name (CI) or pytest tests/test_smoke.py::test_name (local)
  • Backward compatibility: /quicktest-core (CI) or pytest tests/smoke_tests/test_backward_compat.py (local)

@rohansonecha rohansonecha changed the title gitQuery logs [k8s] add even more debug logs around k8s list_namespaced_pod Nov 4, 2025
@rohansonecha rohansonecha self-assigned this Nov 4, 2025
@rohansonecha rohansonecha merged commit f6c9429 into master Nov 4, 2025
20 checks passed
@rohansonecha rohansonecha deleted the query-logs branch November 4, 2025 23:33
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.

3 participants