Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade-2.x: Block draft version upgrades using the release_tag version #388

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

thgreasi
Copy link
Contributor

@thgreasi thgreasi commented Jan 2, 2024

Atm given a draft OS release 3.2.1-2345, users would be able to HUP to it via both

  • device os-update 3.2.1-2345, in which case the script finds the version via the raw_version field
  • and device os-update 3.2.1, in which case the script find the version via the release_tag

After this, HUPing to draft-OS releases will only work using the raw_version which removes the ambiguity and block users from accidentally HUPing to a draft release (eg they misstype the patch version or on purpose try to increase it to check whether a new version is out).

Partially reverts commit ce70f31.

Change-type: patch

See: https://balena.fibery.io/Work/Improvement/Remove-hostApp-version-release-tag-dependency-from-API-1412

After this, HUPing to draft-OS releases will
only work using the raw_version. The
reasoning is to blocks users from accidentally
HUPing to a draft release. This also removes
the ambiguity that we atm have, where for
a draft release 3.2.1-2345, users can perform
the same HUP using either 3.2.1-2345 (found
via the raw_version field) and also
with 3.2.1 (found via the release_tag).
Partially reverts commit ce70f31.

Change-type: patch
@thgreasi thgreasi force-pushed the block-tag-based-draft-os-hups branch from 0719f7f to 8ff13de Compare January 2, 2024 15:09
@flowzone-app flowzone-app bot merged commit 2160066 into master Jan 2, 2024
50 checks passed
@flowzone-app flowzone-app bot deleted the block-tag-based-draft-os-hups branch January 2, 2024 18:01
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.

2 participants