Skip to content

IGN-14665: Update for Ignition 8.3.3#4

Merged
thirdgen88 merged 7 commits into
inductiveautomation:mainfrom
thirdgen88:IGN-14665-update-for-8.3.2
Jan 30, 2026
Merged

IGN-14665: Update for Ignition 8.3.3#4
thirdgen88 merged 7 commits into
inductiveautomation:mainfrom
thirdgen88:IGN-14665-update-for-8.3.2

Conversation

@thirdgen88
Copy link
Copy Markdown
Member

@thirdgen88 thirdgen88 commented Jan 12, 2026

📖 Background

Upgrading from one version to the next in a redundant configuration had previously required manual intervention to fail-over the active Primary to the Backup in order for the Backup to become ready and allow the rollout of the StatefulSet to continue. This PR will set the default AppVersion for the chart to 8.3.3 and incorporate an adjustment to this behavior to allow for automatic upgrades of a redundant pair.

⚙️ Changes

9944e22 chore: bump appVersion to 8.3.3
8f98b8f chore: bump helm-unittest plugin to 1.0.3
d46e269 feat: add PR workflow artifacting
75ca5f6 fix: correct usage info on redundant health check
b301d85 fix: Update redundant health check accepted states
cadde71 chore: bump appVersion to 8.3.2
b713746 fix: correct application of supplemental pod labels

For the redundant health check, we're now accepting Good, OutOfDate and Incompatible as valid values for the readiness probe (when in a redundant configuration via gateway.redundancy.enabled=true). This should help both situations where the backup gateway becomes temporarily out-of-sync or during upgrades where the "Incompatible (Platform)" status is observed on the Backup gateway.

📝 Reviewer Notes

I've got the commits broken out, best to review them individually first.

☑️ QA Notes

We'll want to test a redundant base 8.3.0 configuration using chart version 0.2.0. We should setup some tags and stateful data that is sync'd over redundancy and then test an upgrade to the new chart version. Since the changes are only in the redundant-health-check.sh script that is applied when gateway.redundancy.enabled=true, the test scope shouldn't have to worry much about a standalone upgrade.

@thirdgen88 thirdgen88 requested a review from ia-eknorr January 12, 2026 21:51
Copy link
Copy Markdown
Collaborator

@ia-eknorr ia-eknorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all looks good. I verified each commit as requested where applicable. I did a full test of upgrading the helm release from 8.3.0 (using chart version 0.2.0) to 8.3.2 (using chart version 0.2.1), which spun up successfully.

@thirdgen88 thirdgen88 closed this Jan 29, 2026
@thirdgen88 thirdgen88 deleted the IGN-14665-update-for-8.3.2 branch January 29, 2026 19:42
@thirdgen88 thirdgen88 restored the IGN-14665-update-for-8.3.2 branch January 29, 2026 19:43
@thirdgen88 thirdgen88 reopened this Jan 29, 2026
@thirdgen88 thirdgen88 changed the title IGN-14665: Update for Ignition 8.3.2 IGN-14665: Update for Ignition 8.3.3 Jan 29, 2026
@ia-jdahlinger
Copy link
Copy Markdown

This looks good in QA as well. We've verified upgrades of redundant 0.2.0 (8.3.0) deployments to the latest 0.2.1 (8.3.3) chart with active Perspective and Vision clients (referencing tags, active alarms, etc). Failover occurs as expected and data can be seen to be in sync between primary and backup, before and after failover.

@thirdgen88 thirdgen88 merged commit 2fccfe8 into inductiveautomation:main Jan 30, 2026
1 check 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.

3 participants