Skip to content

Wrongly detected path - #241

Merged
phracek merged 1 commit into
masterfrom
another_fix_for_upstream
Oct 22, 2026
Merged

phracek merged 1 commit into
masterfrom
another_fix_for_upstream

Conversation

@phracek

@phracek phracek commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Updates
    • Updated the daily test and end-of-life checker images to version 0.10.9.
  • Bug Fixes
    • Improved nightly test reports by checking that container log files exist before sending them to Pastebin. This helps avoid errors when a log file is missing.

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@coderabbitai

coderabbitai Bot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 03f0886e-3a36-4c50-979c-9a252c01bf5c
📥 Commits

Reviewing files that changed from the base of the PR and between 66abb93 and 832a81b.

📒 Files selected for processing: 4
  • .github/workflows/build-and-push.yml
  • Dockerfile.eol-checker
  • Makefile
  • daily-tests/daily_tests/daily_nightly_tests_report.py
+-----------------------------------------------------------------+
| This PR is a classic: 'small change' with 'large consequences'. |
+-----------------------------------------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 62.67%. Comparing base (66abb93) to head (832a81b).

Files with missing lines Patch % Lines
...ly-tests/daily_tests/daily_nightly_tests_report.py 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #241   +/-   ##
=======================================
  Coverage   62.67%   62.67%           
=======================================
  Files          13       13           
  Lines        1353     1353           
=======================================
  Hits          848      848           
  Misses        505      505           
Flag Coverage Δ
daily-tests-unit 44.78% <0.00%> (ø)
eol-checker-unit 97.34% <ø> (ø)
ocp-stream-generator-unit 84.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ly-tests/daily_tests/daily_nightly_tests_report.py 55.98% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@phracek
phracek merged commit 2006d88 into master Oct 22, 2026
13 of 15 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.

1 participant