Skip to content

Conversation

FyreByrd
Copy link
Contributor

@FyreByrd FyreByrd commented Sep 15, 2025

I am opening this draft PR for us all to work on, since I don't know how to do all this, and the rest of y'all have bits of important knowledge.

Per the discussion in Discord:

  • Mock BuildEngine
  • Mock CodeBuild???

Feel free to add to this

Summary by CodeRabbit

  • Chores
    • Updated the continuous integration workflow by removing an obsolete SSH tunnel setup and replacing it with a non-operational placeholder step. This streamlines the pipeline and reduces reliance on environment secrets. No changes to application features or performance. Deployment behavior remains unchanged, with no runtime impact for end-users.

Copy link
Contributor

coderabbitai bot commented Sep 15, 2025

Walkthrough

The CI workflow step "Setup stg-tunnel" in .github/workflows/main.yml is renamed to "Setup Mock Build-Engine." The SSH tunnel setup, related commands, and environment secrets are removed and replaced with placeholder comments, eliminating any SSH/stg-tunnel operations from the workflow.

Changes

Cohort / File(s) Summary
CI Workflow
.github/workflows/main.yml
Renamed step from "Setup stg-tunnel" to "Setup Mock Build-Engine"; removed SSH key handling, SSH tunnel commands, and env secrets; added non-functional placeholder content.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant GH as GitHub Actions Runner
  participant Step as Workflow Step

  Dev->>GH: Push/PR triggers workflow
  GH->>Step: Execute "Setup Mock Build-Engine"
  Note over Step: Placeholder content only<br/>No SSH keys, no tunnel
  Step-->>GH: Complete without external connections
  GH-->>Dev: Continue remaining jobs
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I nibbled the tunnel—snip! it’s gone,
A mocky block I’m hopping on.
No keys to guard, no ports to mend,
Just placeholder winds that softly wend.
CI still hums, the carrots cheer—
Onward builds, the path is clear! 🥕

Pre-merge checks and finishing touches

✅ 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 "Add mock Build-Engine for CI" accurately and concisely reflects the primary change in .github/workflows/main.yml, where the previous "Setup stg-tunnel" CI step was renamed and replaced with a non-functional "Setup Mock Build-Engine" placeholder; it is specific and meaningful for reviewers scanning PR history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/replace-stg-tunnel-ci

📜 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 b16a902 and bdf8266.

📒 Files selected for processing (1)
  • .github/workflows/main.yml (1 hunks)
🔇 Additional comments (1)
.github/workflows/main.yml (1)

119-125: Gate placeholder step; add a clear no‑op and confirm tunnel/token usage

Apply this minimal scaffold:

-      - name: Setup Mock Build-Engine
-        run: |
-          # This step may need to be reordered.
-          # I have no idea how much actually needs to be done here or if this should be multiple steps.
-          # Just adding this here for now so there is a diff so I can open a draft PR for all to work on.
-          # - Aidan
+      - name: Setup Mock Build-Engine
+        if: ${{ vars.USE_MOCK_BUILDENGINE == 'true' }}
+        run: |
+          echo "Mock Build-Engine enabled (no-op placeholder)."
+          echo "MOCK_BUILDENGINE_ENABLED=true" >> "$GITHUB_ENV"
  • Repo scan found stg-tunnel references in deployment/ci/docker-compose.yml, deployment/development/docker-compose.yml and README.md (DEFAULT_BUILDENGINE_URL → http://stg-tunnel:8443); the stg-tunnel service runs an SSH tunnel (ssh -N -L …). Do not remove or change the tunnel without updating these files.
  • deployment/ci/docker-compose.yml references BUILDENGINE_TOKEN — pick one convention (prefer BUILD_ENGINE_TOKEN) and update all occurrences.

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.

@FyreByrd FyreByrd force-pushed the fix/replace-stg-tunnel-ci branch from 767e763 to bdf8266 Compare September 17, 2025 16:23
@FyreByrd FyreByrd marked this pull request as ready for review September 23, 2025 14:03
@FyreByrd FyreByrd closed this Sep 23, 2025
@FyreByrd FyreByrd reopened this Sep 23, 2025
@FyreByrd FyreByrd marked this pull request as draft September 23, 2025 14:05
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.

3 participants