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

bundle: drop fixed :latest version from controller image-name #72

Merged
merged 2 commits into from
Jan 14, 2022

Conversation

nixpanic
Copy link
Collaborator

The controller image-name should use the TAG just as other images.
When this is not done, the bundle includes the following reference to
the image, and fails to deploy:

quay.io/csiaddons/k8s-controller:latest:latest`

By using TAG as the other container images do, the bundle will refer
to the image with :latest, or the TAG that was used for creating a
release.

The InstallMode OwnNamespace is required for deploying with
odf-operator. If this is not set, deploying fails with the following
error:

OwnNamespace InstallModeType not supported, cannot configure to watch own namespace

Tested-with: red-hat-storage/odf-operator#165

The InstallMode `OwnNamespace` is required for deploying with
odf-operator. If this is not set, deploying fails with the following
error:

    OwnNamespace InstallModeType not supported, cannot configure to watch own namespace

Tested-with: red-hat-storage/odf-operator#165
Signed-off-by: Niels de Vos <[email protected]>
The controller image-name should use the `TAG` just as other images.
When this is not done, the bundle includes the following reference to
the image, and fails to deploy:

    quay.io/csiaddons/k8s-controller:latest:latest`

By using `TAG` as the other container images do, the bundle will refer
to the image with `:latest`, or the `TAG` that was used for creating a
release.

Signed-off-by: Niels de Vos <[email protected]>
@mergify mergify bot merged commit 008a55e into csi-addons:main Jan 14, 2022
nixpanic pushed a commit to nixpanic/kubernetes-csi-addons that referenced this pull request Mar 28, 2023
sync downstream main with upstream main branch
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.

3 participants