Skip to content

fix: self-update operator early to ensure consistency#3108

Merged
merll merged 32 commits intomainfrom
APL-1716
Apr 14, 2026
Merged

fix: self-update operator early to ensure consistency#3108
merll merged 32 commits intomainfrom
APL-1716

Conversation

@merll
Copy link
Copy Markdown
Contributor

@merll merll commented Apr 2, 2026

📌 Summary

Updating the platform currently works in the following steps:

  1. Update values otomi.version to new image tag
  2. The operator starts to reconcile
  • Team apps are deployed
  • Pre-upgrade runs
  • Operator detects it is out of sync with the version and updates itself (ArgoCD application)
  1. The operator restarts and eventually completes the upgrade

This PR moves the version check above all other actions, making sure the operator image tag and ArgoCD application revision match the values. If an update has been made, it does not proceed but wait for its restart.

🔍 Reviewer Notes

Since the operator handles its own upgrade, only following changes benefit from this. For testing one needs to upgrade to this feature branch first, then on a following branch change the effect can be observed.

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

Comment thread src/cmd/apply.ts Outdated
Comment thread src/cmd/apply.ts Outdated
Co-authored-by: Ferruh <63190600+ferruhcihan@users.noreply.github.com>
@merll merll merged commit ad73861 into main Apr 14, 2026
13 checks passed
@merll merll deleted the APL-1716 branch April 14, 2026 07:08
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.

4 participants