Skip to content

fix: update container registry and credentials in CI/CD pipeline conf… - #82

Merged
omarghatasheh merged 1 commit into
mainfrom
cicd-patches
Mar 31, 2026
Merged

fix: update container registry and credentials in CI/CD pipeline conf…#82
omarghatasheh merged 1 commit into
mainfrom
cicd-patches

Conversation

@MusaMisto

@MusaMisto MusaMisto commented Mar 31, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated CI/CD deployment configuration to use a different container registry and authentication credentials for improved deployment reliability.

Note: This release contains infrastructure maintenance updates with no user-facing changes.

@coderabbitai

coderabbitai Bot commented Mar 31, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The GitHub Actions workflow configuration updates the container registry target from ghcr.io to docker.io, modifies Helm chart values to specify the new registry repository, and replaces GitHub-provided authentication credentials with custom registry secrets.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/ui-cicd.yml
Updated container registry from ghcr.io to docker.io, added image.repo=docker.io/simplify9 to Helm values, and replaced GitHub token credentials with custom secrets S9_REGISTRY_USERNAME and S9_REGISTRY_TOKEN.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A shift in registries, credentials anew,
From GitHub's vault to secrets we pursue,
Docker.io awaits our images bright,
Simplify9's path shines ever right!
A workflow refined, authentication secure,
Our pipeline's journey, steadfast and sure. 🚀

🚥 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 clearly and specifically describes the main change: updating container registry and credentials in the CI/CD pipeline configuration, which directly matches the primary focus of the changeset.
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
  • Commit unit tests in branch cicd-patches

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.

@omarghatasheh
omarghatasheh merged commit 5836b19 into main Mar 31, 2026
2 of 3 checks passed
@omarghatasheh
omarghatasheh deleted the cicd-patches branch March 31, 2026 13:35
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