Skip to content

chore: Set version of 'next' bundle to always update and add build info #815

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

Merged
merged 1 commit into from
Apr 13, 2022

Conversation

amisevsk
Copy link
Collaborator

What does this PR do?

Configure the 'next' index image build to add incrementing prerelease identifiers and include the git-sha as the build identifier. E.g. for version v0.15.0-dev, build index images with versions

v0.15.0-dev.0+59b2f40e
v0.15.0-dev.1+ca15cf09
v0.15.0-dev.2+bd17527a

To ensure clusters deploying the 'next' catalog will always update to
new builds.

Note the semantics for version comparison in SemVer are:

  • Build identifiers are ignored (e.g. +59b2f40e)
  • Each dot-separated segment of a prerelease identifier is compared:
    • if alphanumeric, lexically in ASCII sort order
    • if only digits, numerically
    • numeric identifiers < alphanumeric identifiers

What issues does this PR fix or reference?

Closes #814

Is it tested? How?

Tested in my fork, see recent runs of the test.ymll workflow: https://github.com/amisevsk/devworkspace-operator/actions/workflows/test.yml

  • logs from run #13

    Updating version for this build to 0.15.0-dev.0+7c43dcc

  • logs from run #14

    Updating version for this build to 0.15.0-dev.1+ba74740

I've also tested

  • OLM behavior on crc: with the versioning scheme used here, new changes are automatically rolled out and new DWO images are pulled (technically each one is a new version of DWO)
  • The script to determine the next version to ensure it correctly handles the cases where 1) the current index does not have a build number (i.e. the first run after merging this PR), 2) the actual version is incremented (i.e. a minor release) to ensure the counter resets to 0.

PR Checklist

  • E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)
    • v8-devworkspace-operator-e2e: DevWorkspace e2e test
    • v8-che-happy-path: Happy path for verification integration with Che

Configure the 'next' index image build to add incrementing prerelease
identifiers and include the git-sha as the build identifier. E.g. for
version v0.15.0-dev, build index images with versions

  v0.15.0-dev.0+59b2f40e
  v0.15.0-dev.1+ca15cf09
  v0.15.0-dev.2+bd17527a

To ensure clusters deploying the 'next' catalog will always update to
new builds.

Note the semantics for version comparison in SemVer are:
* Build identifiers are ignored (e.g. +59b2f40e)
* Each dot-separated segment of a prerelease identifier is compared:
  * if alphanumeric, lexically in ASCII sort order
  * if only digits, numerically
  * numeric identifiers < alphanumeric identifiers

Signed-off-by: Angel Misevski <[email protected]>
@openshift-ci
Copy link

openshift-ci bot commented Apr 13, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: amisevsk, ibuziuk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ibuziuk ibuziuk merged commit 1e8f41e into devfile:main Apr 13, 2022
@amisevsk amisevsk deleted the next-catalog-buildnums branch April 13, 2022 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upstream 'next' catalog does not update automatically.
2 participants