Skip to content

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

@monotek 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 monotek marked this pull request as ready for review June 30, 2026 13:44
@monotek monotek requested review from a team as code owners June 30, 2026 13:44
@monotek monotek merged commit e525ee7 into main Jun 30, 2026
10 checks passed
@monotek monotek deleted the timestamp-mismatch-fix branch June 30, 2026 14:03
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 30, 2026
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