Skip to content

Conversation

@georgi-l95
Copy link
Member

@georgi-l95 georgi-l95 commented Oct 14, 2025

Summary by CodeRabbit

  • Chores
    • Release workflow now standardizes Docker image names to lowercase, ensuring consistent builds and pulls.
    • Multi-architecture images (amd64 and arm64) are built and pushed using the standardized image name.
    • Both specific tag and latest images are pulled using the standardized name for consistency across platforms.
    • Existing image name configuration remains supported, but operational steps now derive the image name from the normalized repository name for reliability.

@georgi-l95 georgi-l95 self-assigned this Oct 14, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 14, 2025

Walkthrough

Adds a step in the release workflow to derive a lowercase image name from the repository and expose it as an output. Replaces references to env.IMAGE_NAME with the new step output in Docker pull/build/push commands for multi-arch images, while keeping the original env variable declared.

Changes

Cohort / File(s) Summary
Release workflow updates
\.github/workflows/release.yml
Added step repo_lowercase to compute lowercase repo name and expose image_name via GITHUB_OUTPUT. Updated Docker pull/buildx/build/push commands to use ${{ steps.repo_lowercase.outputs.image_name }} instead of env.IMAGE_NAME. Retained env.IMAGE_NAME declaration but no longer used in affected steps.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Dev as Developer
    participant GH as GitHub Actions Runner
    participant RL as repo_lowercase Step
    participant DK as Docker CLI
    participant REG as Docker Registry

    Dev->>GH: Trigger release workflow
    GH->>RL: Run step to compute lowercase repo name
    RL-->>GH: Output image_name (lowercase)
    note over RL,GH: New output: steps.repo_lowercase.outputs.image_name

    GH->>DK: docker pull ${image_name}:<tag>
    DK-->>REG: Request image
    REG-->>DK: Image layers (if any)
    DK-->>GH: Pull result

    GH->>DK: docker buildx build --platform amd64,arm64 -t ${image_name}:<tag> --push
    DK->>REG: Push multi-arch image manifests/layers
    REG-->>DK: Acknowledge push
    DK-->>GH: Build & push complete

    GH->>DK: docker pull ${image_name}:latest (optional/if used)
    DK-->>GH: Pull result
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • improve release workflow #124 — Also modifies .github/workflows/release.yml to adjust Docker image handling; likely touches similar pull/build steps and naming.

Poem

I nibble on names, make lowercase cheer,
Hopping through workflows, the path is clear.
Pull, build, and push—two arches aligned,
A tidy tag trail the carrots defined.
Docker dreams hum, releases take flight—
Bunny approved, it’s shipshape tonight! 🥕🚀

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The pull request description is missing entirely and does not include any of the required sections from the repository’s template, leaving reviewers without context on the change goals, related issues, or detailed file modifications. Please add a detailed pull request description following the repository template, including a concise Description of the change objectives, linked Related Issues, a summary of Changes by file, and a completed Reviewer Checklist to ensure all review requirements are addressed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title “chore: improve release workflow” accurately reflects the primary update to the CI/CD pipeline by improving the repository’s release workflow and follows conventional commit styling, making it clear and concise for future reference.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch improve-relese-workflow

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1d6f8ef and 551edca.

📒 Files selected for processing (1)
  • .github/workflows/release.yml (3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Security Audit
  • GitHub Check: Build and Basic Quality Checks

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

@georgi-l95 georgi-l95 merged commit 84b60c8 into main Oct 14, 2025
7 checks passed
@georgi-l95 georgi-l95 deleted the improve-relese-workflow branch October 14, 2025 15:05
@codecov
Copy link

codecov bot commented Oct 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.82%. Comparing base (cdf2fd2) to head (551edca).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
- Coverage   79.90%   79.82%   -0.08%     
==========================================
  Files          56       56              
  Lines       10493    10504      +11     
==========================================
+ Hits         8384     8385       +1     
- Misses       2109     2119      +10     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coderabbitai coderabbitai bot mentioned this pull request Oct 15, 2025
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