Skip to content
This repository was archived by the owner on Nov 23, 2025. It is now read-only.

refactor: Clean up deploy.yaml by removing comments and adding debugg…#8

Merged
RandithaK merged 1 commit intomainfrom
devOps
Nov 7, 2025
Merged

refactor: Clean up deploy.yaml by removing comments and adding debugg…#8
RandithaK merged 1 commit intomainfrom
devOps

Conversation

@RandithaK
Copy link
Copy Markdown
Member

@RandithaK RandithaK commented Nov 7, 2025

…ing step to display file contents before deployment

Summary by CodeRabbit

Release Notes

  • Chores
    • Enhanced deployment pipeline with stricter validation and explicit rollout status verification.
    • Improved deployment reliability through automated manifest validation and branch management enforcement.

…ing step to display file contents before deployment
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 7, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The GitHub Actions deployment workflow is modified to enforce checkout of the k8s-config repository's main branch, add debugging visibility into the Kubernetes manifest, install the yq YAML processing tool, and validate successful pod rollout after deployment.

Changes

Cohort / File(s) Change Summary
GitHub Actions Workflow
\.github/workflows/deploy\.yaml
Adds explicit ref: 'main' to k8s-config repository checkout; introduces debug step printing gateway-deployment.yaml contents; installs yq tool; adds kubectl rollout status validation step for api-gateway-deployment; removes commented workflow name suggestion.

Sequence Diagram

sequenceDiagram
    participant Workflow
    participant K8sRepo as K8s Config Repo
    participant Manifest as Manifest File
    participant Tools as yq/kubectl
    participant Deployment as api-gateway-deployment

    Workflow->>K8sRepo: Checkout (ref: main)
    K8sRepo-->>Workflow: config-repo/
    Workflow->>Manifest: Display gateway-deployment.yaml
    Manifest-->>Workflow: Manifest contents (debug output)
    Workflow->>Tools: Install yq
    Tools-->>Workflow: Ready
    Workflow->>Deployment: Deploy
    Deployment-->>Workflow: Deploying...
    Workflow->>Deployment: Check rollout status
    Deployment-->>Workflow: ✓ Rollout complete
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Review the explicit branch reference to ensure main branch is the correct target
  • Verify debug output step serves troubleshooting purposes without security concerns
  • Confirm yq installation aligns with subsequent YAML processing needs
  • Validate kubectl rollout status command syntax and deployment name accuracy

Poem

🐰 A workflow now flows with purpose and grace,
Main branch enforced—no wrong place!
With debug output and rollout checks in sight,
Deployments validated, oh what a delight! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch devOps

📜 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 1d2a40f and e6d966e.

📒 Files selected for processing (1)
  • .github/workflows/deploy.yaml (2 hunks)

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.

@RandithaK RandithaK merged commit 8cfca0d into main Nov 7, 2025
4 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant