Skip to content

ci: Remove redis and scheduler log gathering#338

Merged
dsavineau merged 1 commit intoansible:mainfrom
dsavineau:ci_remove_redis_scheduler_logs
Apr 13, 2026
Merged

ci: Remove redis and scheduler log gathering#338
dsavineau merged 1 commit intoansible:mainfrom
dsavineau:ci_remove_redis_scheduler_logs

Conversation

@dsavineau
Copy link
Copy Markdown
Contributor

@dsavineau dsavineau commented Apr 13, 2026

Both redis and scheduler deployments were removed not so long ago so we don't need to gather logs for those.

Summary by CodeRabbit

  • Chores
    • Simplified CI/CD workflow logging by removing certain log collection steps from automated processes.

Both redis and scheduler deployments were removed not so long ago
so we don't need to gather logs for those.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
@sonarqubecloud
Copy link
Copy Markdown

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a72a385c-05d1-47bc-a03f-996c00962351

📥 Commits

Reviewing files that changed from the base of the PR and between d55e0ef and 21c95bc.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • .github/workflows/pr.yml
💤 Files with no reviewable changes (2)
  • .github/workflows/pr.yml
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

Two GitHub Actions workflow files have identical modifications removing log collection blocks for Redis cache and EDA scheduler components from their "Get logs" steps. No other workflow logic or structure was altered.

Changes

Cohort / File(s) Summary
CI/CD Workflow Log Removal
.github/workflows/ci.yml, .github/workflows/pr.yml
Removed REDIS_LOGS and EDA_SCHEDULER_LOGS kubectl log-capture blocks from the "Get logs" step in both workflows (6 lines removed per file).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately summarizes the main change: removing redis and scheduler log gathering from CI workflows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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 and usage tips.

@dsavineau dsavineau merged commit 95c0eea into ansible:main Apr 13, 2026
7 checks passed
@dsavineau dsavineau deleted the ci_remove_redis_scheduler_logs branch April 13, 2026 19:00
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