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

contrib/kind: Log DNS queries in CoreDNS pods #27874

Merged
merged 1 commit into from
Sep 4, 2023

Conversation

pchaigno
Copy link
Member

@pchaigno pchaigno commented Sep 1, 2023

In CI, we currently collect CoreDNS logs as part of the sysdump. Those logs are however mostly useless after startup because we don't log the DNS queries. This commit fixes it. That provides for another data point when debugging failing DNS queries.

@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. sig/contributing Impacts contribution workflow, guidelines, and tools. labels Sep 1, 2023
@pchaigno pchaigno requested a review from a team as a code owner September 1, 2023 08:44
In CI, we currently collect CoreDNS logs as part of the sysdump. Those
logs are however mostly useless after startup because we don't log the
DNS queries. This commit fixes it. That provides for another data point
when debugging failing DNS queries.

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>
@pchaigno
Copy link
Member Author

pchaigno commented Sep 1, 2023

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 4, 2023
@pchaigno pchaigno merged commit 2f63ba9 into cilium:main Sep 4, 2023
61 checks passed
@pchaigno pchaigno deleted the kind-log-dns-queries branch September 4, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI. sig/contributing Impacts contribution workflow, guidelines, and tools.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants