mosaicstack/stack
1
0
Fork 0
Code Issues 42 Pull Requests Actions Packages Projects Releases 15 Wiki Activity

fix: handle legacy Woodpecker mosaic credentials #522

Closed
jason.woltje wants to merge 4 commits from fix/woodpecker-wrapper-legacy-mosaic into main
pull from: fix/woodpecker-wrapper-legacy-mosaic
merge into: mosaicstack:main
Conversation 1 Commits 4 Files Changed 9 +179 -74
jason.woltje commented 2026-05-25 16:28:32 +00:00
Owner
Copy Link

Summary

  • Allow WOODPECKER_INSTANCE=mosaic to fall back to legacy flat Woodpecker credentials when nested mosaic credentials are absent.
  • Preserve explicit named-instance behavior for future nested Woodpecker credentials.
  • Fix Woodpecker v3 pipeline pagination from per_page to perPage in pipeline-list and latest pipeline status lookup.

Verification

  • bash -n packages/mosaic/framework/tools/_lib/credentials.sh
  • bash -n packages/mosaic/framework/tools/woodpecker/pipeline-list.sh
  • bash -n packages/mosaic/framework/tools/woodpecker/pipeline-status.sh
  • load_credentials woodpecker resolves ci.mosaicstack.dev with flat credentials.
  • WOODPECKER_INSTANCE=mosaic load_credentials woodpecker resolves flat credentials instead of failing with woodpecker.mosaic.url not found.
  • load_credentials woodpecker-mosaic resolves flat credentials for backward compatibility.
  • WOODPECKER_INSTANCE=mosaic packages/mosaic/framework/tools/woodpecker/pipeline-list.sh -r ddk/ai-bma -l 5 returns exactly five pipelines.
  • packages/mosaic/framework/tools/woodpecker/pipeline-status.sh -r mosaicstack/stack resolves the latest pipeline via perPage=1.
## Summary - Allow WOODPECKER_INSTANCE=mosaic to fall back to legacy flat Woodpecker credentials when nested mosaic credentials are absent. - Preserve explicit named-instance behavior for future nested Woodpecker credentials. - Fix Woodpecker v3 pipeline pagination from per_page to perPage in pipeline-list and latest pipeline status lookup. ## Verification - bash -n packages/mosaic/framework/tools/_lib/credentials.sh - bash -n packages/mosaic/framework/tools/woodpecker/pipeline-list.sh - bash -n packages/mosaic/framework/tools/woodpecker/pipeline-status.sh - load_credentials woodpecker resolves ci.mosaicstack.dev with flat credentials. - WOODPECKER_INSTANCE=mosaic load_credentials woodpecker resolves flat credentials instead of failing with woodpecker.mosaic.url not found. - load_credentials woodpecker-mosaic resolves flat credentials for backward compatibility. - WOODPECKER_INSTANCE=mosaic packages/mosaic/framework/tools/woodpecker/pipeline-list.sh -r ddk/ai-bma -l 5 returns exactly five pipelines. - packages/mosaic/framework/tools/woodpecker/pipeline-status.sh -r mosaicstack/stack resolves the latest pipeline via perPage=1.
jason.woltje added 1 commit 2026-05-25 16:28:33 +00:00
fix: handle legacy woodpecker mosaic credentials
Some checks failed
ci/woodpecker/push/ci Pipeline failed
Details
ci/woodpecker/pr/ci Pipeline failed
Details
fd874649e8
jason.woltje force-pushed fix/woodpecker-wrapper-legacy-mosaic from fd874649e8 to 410ada409c 2026-05-25 16:29:33 +00:00 Compare
jason.woltje added 1 commit 2026-05-25 19:08:50 +00:00
fix(ci): avoid postgres service collision in k8s backend
Some checks failed
ci/woodpecker/push/ci Pipeline was canceled
Details
ci/woodpecker/pr/ci Pipeline was canceled
Details
2d4d799c26
jason.woltje added 1 commit 2026-05-25 19:19:55 +00:00
fix(git): pass explicit repo to Gitea wrappers
Some checks failed
ci/woodpecker/push/ci Pipeline was canceled
Details
ci/woodpecker/pr/ci Pipeline was canceled
Details
4741b3e16f
jason.woltje added 1 commit 2026-05-25 19:20:16 +00:00
fix(git): avoid duplicate gh repo flag in pr-diff
Some checks failed
ci/woodpecker/push/ci Pipeline failed
Details
ci/woodpecker/pr/ci Pipeline failed
Details
ci/woodpecker/manual/ci Pipeline was successful
Details
2f5b6ca2e7
jason.woltje commented 2026-06-11 18:31:43 +00:00
Author
Owner
Copy Link

Closing as superseded by #524 (rollup 821e19d). Assessment: commits 410ada4 (legacy woodpecker creds) and 2d4d799 (CI postgres-collision fix) were auto-dropped by rebase as patch-identical-already-upstream; the remaining two (explicit repo args + pr-diff duplicate-flag fix) are also already in main, and their residual is regressive (drops --login mosaicstack). No unique content remains. — orchestrator (mos)

Closing as **superseded by #524** (rollup 821e19d). Assessment: commits 410ada4 (legacy woodpecker creds) and 2d4d799 (CI postgres-collision fix) were auto-dropped by rebase as patch-identical-already-upstream; the remaining two (explicit repo args + pr-diff duplicate-flag fix) are also already in main, and their residual is regressive (drops --login mosaicstack). No unique content remains. — orchestrator (mos)
jason.woltje closed this pull request 2026-06-11 18:31:44 +00:00
Some checks failed
ci/woodpecker/push/ci Pipeline failed
Details
ci/woodpecker/pr/ci Pipeline failed
Details
ci/woodpecker/manual/ci Pipeline was successful
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
fleet-enhancement
Fleet-wide reliability/speed/cost improvements surfaced by the nightly enhance pass
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#522
Delete Branch "fix/woodpecker-wrapper-legacy-mosaic"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?