FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

write non-timestamped tag to GitOps repo by monotek · Pull Request #153 · Staffbase/gitops-github-action · GitHub

write non-timestamped tag to GitOps repo - #153

Merged
monotek merged 1 commit into
mainfrom
timestamp-mismatch-fix
Jun 30, 2026
Merged

write non-timestamped tag to GitOps repo#153
monotek merged 1 commit into
mainfrom
timestamp-mismatch-fix

Conversation

monotek commented Jun 30, 2026

Copy link
Copy Markdown
Member

Type of Change

  • Bugfix
  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

Branch builds push a timestamped image tag generated with date -u, so
when the action runs across separate invocations (e.g. build and push as
different steps in Staffbase/backend) each invocation recomputes a fresh
timestamp and the tag committed to mops no longer matches the one pushed,
leaving a dangling reference.

Emit a new gitops_tag output (${ALIAS_TAG:-$TAG}) and have the Update
Docker Image step consume it instead of the timestamped tag. The
<prefix>-<short-sha> alias is already pushed alongside and is
deterministic across invocations, so the GitOps reference stays
consistent and resolvable. The image keeps its timestamped tag for Flux
sorting. Works with docker-tag-timestamp true and false; not
configurable.

Checklist

  • Write tests
  • Make sure all tests pass
  • Update documentation
  • Review the Contributing Guideline and sign CLA
  • Reference relevant issue(s) and close them after merging

Signed-off-by: André Bauer <andre.bauer@staffbase.com>
monotek marked this pull request as ready for review June 30, 2026 13:44
monotek requested review from a team as code owners June 30, 2026 13:44
monotek merged commit e525ee7 into main Jun 30, 2026
10 checks passed
monotek deleted the timestamp-mismatch-fix branch June 30, 2026 14:03
github-actions Bot locked and limited conversation to collaborators Jun 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL