-
Notifications
You must be signed in to change notification settings - Fork 243
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
[BUG] Bundle logic not working as expected for the OKD preset #3595
Comments
Issue is around https://github.com/crc-org/crc/blob/main/pkg/crc/machine/bundle/metadata.go#L188-L199 logic which return func (bundle *CrcBundleInfo) GetBundleType() crcPreset.Preset {
switch bundle.Type {
case "snc", "snc_custom":
return crcPreset.OpenShift
case "podman", "podman_custom":
return crcPreset.Podman
case "microshift", "microshift_custom":
return crcPreset.Microshift
default:
return crcPreset.OpenShift // <!--- drops here
}
} and consuming it https://github.com/crc-org/crc/blob/main/pkg/crc/machine/start.go#L915-L920 here so in func bundleMismatchWithPreset(preset crcPreset.Preset, bundleMetadata *bundle.CrcBundleInfo) error {
bundlePreset := bundleMetadata.GetBundleType()
if preset == crcPreset.OKD && bundlePreset == crcPreset.OpenShift {
return nil
}
if preset != bundleMetadata.GetBundleType() {
return errors.Errorf("Preset %s is used but bundle is provided for %s preset", preset, bundleMetadata.GetBundleType())
}
return nil
} A proper fix would be in the metadata the type should be changed to:
|
On windows to make OKD work i created my fork with proposed fix: |
@Maurycy-Krzeminski yes this is correct fix, can you put a PR for same? |
Sure: #3610 |
Since PR is merged, closing this issue. |
As reported in okd-project/okd#1556, we see errors like:
The text was updated successfully, but these errors were encountered: