Skip to content

feat: add diagnostic log collection on test failure#91

Merged
jrichter1 merged 1 commit intoredhat-developer:mainfrom
subhashkhileri:podlogscollections
Apr 27, 2026
Merged

feat: add diagnostic log collection on test failure#91
jrichter1 merged 1 commit intoredhat-developer:mainfrom
subhashkhileri:podlogscollections

Conversation

@subhashkhileri
Copy link
Copy Markdown
Member

@subhashkhileri subhashkhileri commented Apr 27, 2026

JIRA: https://redhat.atlassian.net/browse/RHDHBUGS-2874

Summary

  • Adds collectDiagnosticLogs(namespace, outputDir?) to KubernetesClientHelper — collects events, pod status, deployments, statefulsets, routes, configmaps, and per-container pod logs (init + app, current + previous) to files under node_modules/.cache/e2e-test-results/logs/<namespace>/
  • TeardownReporter now tracks projects with test failures and calls collectDiagnosticLogs before namespace deletion. Log collection runs on both CI and local; namespace deletion remains CI-only
  • Per-container log collection instead of --all-containers (which fails entirely if any container hasn't started)
  • Bumps version to 1.1.34 with changelog and documentation updates

@subhashkhileri subhashkhileri force-pushed the podlogscollections branch 2 times, most recently from 5db2f40 to 6c7ee78 Compare April 27, 2026 08:41
Adds collectDiagnosticLogs() to KubernetesClientHelper that captures
cluster state (events, pods, deployments, statefulsets, routes,
per-container pod logs) to files on test failure.

TeardownReporter now tracks failed projects and collects diagnostics
before namespace deletion. Log collection runs on both CI and local;
namespace deletion remains CI-only.

Bumps version to 1.1.34.
@jrichter1 jrichter1 merged commit f2defd3 into redhat-developer:main Apr 27, 2026
3 checks passed
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.

2 participants