Skip to content

Commit

Permalink
chore: bump launcher version for apptainer synonym for singularity (#881
Browse files Browse the repository at this point in the history
)
  • Loading branch information
CanmingCobble authored and determined-ci committed Feb 2, 2024
1 parent 14af14d commit cfcd82c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1120,7 +1120,7 @@ func (m *dispatcherResourceManager) startLauncherJob(

if tempDispatchID != dispatchID {
incompMsg := "HPC Launcher version is below the minimum required. " +
"Update to version 3.2.9 or greater."
"Update to version 3.3.1 or greater."
ctx.Log().WithField("allocation-id", msg.AllocationID).
WithField("description", msg.Spec.Description).
Errorf("Launcher did not honor DispatchID assignment of %s. "+
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import (

const versionCheckPeriod = 60 * time.Second

var launcherMinimumVersion = semvar.MustParse("3.2.9")
var launcherMinimumVersion = semvar.MustParse("3.3.1")

// periodicallyCheckLauncherVersion checks the launcher version every 60s, logging warnings while
// it is out of date and exiting if it finds it is ok.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@ import (
func TestCheckLauncherVersion(t *testing.T) {
assert.Equal(t, checkLauncherVersion(semver.MustParse("4.1.0")), true)
assert.Equal(t, checkLauncherVersion(semver.MustParse("4.1.3-SNAPSHOT")), true)
assert.Equal(t, checkLauncherVersion(semver.MustParse("3.2.9")), true)
assert.Equal(t, checkLauncherVersion(semver.MustParse("3.3.1")), true)
assert.Equal(t, checkLauncherVersion(semver.MustParse("3.2.9")), false)
assert.Equal(t, checkLauncherVersion(semver.MustParse("3.2.3")), false)
assert.Equal(t, checkLauncherVersion(semver.MustParse("3.2.0")), false)
assert.Equal(t, checkLauncherVersion(semver.MustParse("3.1.3")), false)
Expand Down

0 comments on commit cfcd82c

Please sign in to comment.