Skip to content

feature: filter out internal build suffix #585

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ekitagawa
Copy link
Contributor

Description

  • feature: filter out internal build suffix

Why make this change

Installation of an internal MCR build fails when I set it like this.

  "mcr":
    "channel": "test-25.0.12/fips"
    "installURLLinux": "https://get.mirantis.com/"
    "installURLWindows": "https://get.mirantis.com/install.ps1"
    "repoURL": "https://repos-internal.mirantis.com"
    "version": "25.0.12-tp1"

I get this output

▶ launchpad version
version: 1.5.13-tp2
commit: fd80698b975657d1a0729bedd07e6e8e7696d962

▶ launchpad apply
.
.
.
INFO ==> Running phase: Validate Facts
INFO See /Users/eiichi.kitagawa/.mirantis-launchpad/cluster/4O7MR4/apply.log for more logs
FATA failed to apply cluster: failed to apply MKE: phase failure: Validate Facts => validation failed
MCR version and channel don't match, which is required for versions >= 25.0.0; MCR version does not match channel-version '25.0.12' vs '25.0.12-tp1'

Related Links

PRODENG-3068

Testing

Test run with the proposed fix.

Unit tests show the newly added tests as well as preexisting tests works.

▶ go test -v pkg/product/common/api/mcr_config_validate_test.go
=== RUN   Test_ValidateNil
--- PASS: Test_ValidateNil (0.00s)
=== RUN   Test_ValidateNilFIPS
--- PASS: Test_ValidateNilFIPS (0.00s)
=== RUN   Test_ValidateEmptyVersion
--- PASS: Test_ValidateEmptyVersion (0.00s)
=== RUN   Test_ValidateInvalidVersion
--- PASS: Test_ValidateInvalidVersion (0.00s)
=== RUN   Test_ValidateMissingChannelVersion
--- PASS: Test_ValidateMissingChannelVersion (0.00s)
=== RUN   Test_ValidateWrongChannelVersion
--- PASS: Test_ValidateWrongChannelVersion (0.00s)
=== RUN   Test_ValidateWrongChannelVersionFIPS
--- PASS: Test_ValidateWrongChannelVersionFIPS (0.00s)
=== RUN   Test_ValidateIncompleteChannelVersion
--- PASS: Test_ValidateIncompleteChannelVersion (0.00s)
=== RUN   Test_ValidateIncompleteChannelVersionFIPS
--- PASS: Test_ValidateIncompleteChannelVersionFIPS (0.00s)
=== RUN   Test_ValidateWildcardChannelVersion
--- PASS: Test_ValidateWildcardChannelVersion (0.00s)
=== RUN   Test_ValidateWildcardChannelVersionFIPS
--- PASS: Test_ValidateWildcardChannelVersionFIPS (0.00s)
=== RUN   Test_ValidateInternalBuild
--- PASS: Test_ValidateInternalBuild (0.00s)
=== RUN   Test_ValidateInternalBuildFIPS
--- PASS: Test_ValidateInternalBuildFIPS (0.00s)
PASS
ok  	command-line-arguments	0.430s

@ekitagawa ekitagawa force-pushed the handle_internal branch 2 times, most recently from a99eade to f5078bc Compare July 8, 2025 23:16
Signed-off-by: Eiichi Kitagawa <eiichi.kitagawa@docker.com>
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.

1 participant