Skip to content

fix(ci): Docker Hub repo names and workflow improvements#32

Merged
ian-flores merged 3 commits intomainfrom
fix-dockerhub-repo-names
Jan 13, 2026
Merged

fix(ci): Docker Hub repo names and workflow improvements#32
ian-flores merged 3 commits intomainfrom
fix-dockerhub-repo-names

Conversation

@ian-flores
Copy link
Contributor

Summary

This PR includes several CI/CD fixes and improvements:

  • Docker Hub repository names: Use correct ptd- prefix for Docker Hub images (posit/ptd-team-operator, posit/ptd-flightdeck)
  • Adhoc image cleanup timing: Wait for main branch build to succeed before cleaning up merged PR images (ensures stable fallback exists)
  • Release workflow: Skip unnecessary releases for docs, markdown, and workflow-only changes
  • Cleanup safety: Use commit SHA to identify correct PR for cleanup (race-condition safe when multiple PRs merge quickly)

Changes

cleanup-adhoc-images.yml

  • Added workflow_run trigger to run after main build succeeds
  • Non-merged PRs still cleanup immediately on close
  • Uses GitHub API to correlate commit SHA with specific PR

release.yml

  • Added paths-ignore for *.md, docs/**, .github/workflows/**

build.yml & flightdeck.yml

  • Fixed Docker Hub repository names to use ptd- prefix

Test plan

  • Verify adhoc cleanup skips merged PRs on PR close event
  • Verify cleanup runs after successful main build
  • Verify release workflow doesn't trigger for docs-only changes
  • Verify Docker Hub images use correct ptd- prefix

- Adhoc image cleanup now waits for main build to succeed before
  deleting merged PR images (ensures stable fallback exists)
- Non-merged PRs still cleanup immediately on close
- Uses commit SHA to identify correct PR (race-condition safe)
- Release workflow skips docs, markdown, and workflow-only changes
@ian-flores ian-flores marked this pull request as ready for review January 13, 2026 15:18
The Docker buildx GHA cache (type=gha) requires actions: write
permission to save cache entries. Without it, the cache API
returns 502 errors when attempting to write.
Cache operations can fail transiently. Adding ignore-error=true
ensures builds complete even when cache read/write fails.
@ian-flores ian-flores merged commit 8529117 into main Jan 13, 2026
4 checks passed
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.

1 participant