Skip to content

fix(ci): Consolidate staging deployments and prevent duplicate builds#617

Open
richarddushime wants to merge 7 commits intomasterfrom
ci-fix
Open

fix(ci): Consolidate staging deployments and prevent duplicate builds#617
richarddushime wants to merge 7 commits intomasterfrom
ci-fix

Conversation

@richarddushime
Copy link
Contributor

@richarddushime richarddushime commented Feb 4, 2026 •
edited
Loading

Description

@LukasWallrich

This Removes duplicate staging deployment from deploy.yaml (now handled only by staging-aggregate.yaml),
Also adds draft PR filtering to avoid wasted runs, and implements workflow queuing with job timeouts to ensure all open PRs deploy to staging sequentially without blocking each other.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Tested Locally
  • Manual review / previewed on staging.forrt.org content/webpage changes
  • Not Tested yet

Checklist for Developers:

  • I have attempted to stay aligned to related code in this repository rather than reinventing the wheel.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

Additional Notes

@richarddushime richarddushime requested a review from a team as a code owner February 4, 2026 19:54
@github-actions github-actions bot added the cicd Relevant to GitHub workflows label Feb 4, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

👍 All image files/references (if any) are in webp format, in line with our policy.

@richarddushime richarddushime changed the title fix(ci): Consolidate staging deployments and prevent duplicate builds fix(ci): Consolidate staging deployments and prevent duplicate builds [In Testing Phase] Feb 4, 2026
@LukasWallrich
Copy link
Contributor

LukasWallrich commented Feb 4, 2026 •
edited
Loading

✅ Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-02-06 13:44:09 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

@richarddushime
Copy link
Contributor Author

richarddushime commented Feb 4, 2026

@LukasWallrich Good to go but i will keep monitoring it

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Feb 5, 2026

@copilot please review and document the approach to staging in the README

Copy link
Contributor

Copilot AI commented Feb 5, 2026

@LukasWallrich I've opened a new pull request, #621, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI mentioned this pull request Feb 5, 2026
Merged
@LukasWallrich
docs: Document staging deployment aggregation strategy (#621)
05cdb54 
* Initial plan

* docs: Document staging deployment approach in README

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Feb 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@LukasWallrich
Copy link
Contributor

LukasWallrich commented Feb 5, 2026

@copilot there should be an option to manually deploy a single PR to staging (which seems to be implemented in deploy.yml). Is that just missing from README, or is there an issue with that?

Copilot AI mentioned this pull request Feb 5, 2026
Draft
Copy link
Contributor

Copilot AI commented Feb 5, 2026

@LukasWallrich I've opened a new pull request, #622, to work on those changes. Once the pull request is ready, I'll request review from you.

@richarddushime richarddushime changed the title fix(ci): Consolidate staging deployments and prevent duplicate builds [In Testing Phase] fix(ci): Consolidate staging deployments and prevent duplicate builds Feb 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@richarddushime richarddushime mentioned this pull request Feb 5, 2026
Draft
9 tasks
Copilot AI mentioned this pull request Feb 5, 2026
Closed
9 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

✅ Spell Check Passed

No spelling issues found in this PR! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cicd Relevant to GitHub workflows documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@richarddushime @LukasWallrich