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

Being able to manually set a ODO_BOOTSTAPPER_IMAGE via odo config #2424

Closed
cdrage opened this issue Dec 3, 2019 · 4 comments
Closed

Being able to manually set a ODO_BOOTSTAPPER_IMAGE via odo config #2424

cdrage opened this issue Dec 3, 2019 · 4 comments
Labels
kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation

Comments

@cdrage
Copy link
Member

cdrage commented Dec 3, 2019

/kind enhancement

Which functionality do you think we should update/improve?

When we release a new update to the bootstrapper image, the only way to update is:

export ODO_BOOTSTRAPPER_IMAGE=registry.access.redhat.com/openshiftdo/odo-init-image-rhel7:1.0.1
odo delete
odo push

Alternatively, we should be able to set it via: odo config and NOT have to recreate the image.

For example:

odo config set BootstrapperImage registry.access.redhat.com/openshiftdo/odo-init-image-rhel7:1.0.1
odo push --config # like it's said in the output whenever you do an update

Why is this needed?

Need this for backwards compatibility when releasing a new image.

@cdrage cdrage closed this as completed Dec 3, 2019
@cdrage cdrage reopened this Dec 3, 2019
@openshift-ci-robot openshift-ci-robot added kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation and removed kind/enhancement labels Dec 4, 2019
@kadel
Copy link
Member

kadel commented Dec 5, 2019

Changing the bootstrapper image is something that the user should never do.
ODO_BOOTSTRAPPER_IMAGE is there only for use in testing, to make it easier for us to test an image before we do a release. Users should never change the image.

If you put it into the config as you propose it will never get updated with a new odo release, and the user will have to update it in the config.

The initImage is tied to given odo cli release, and outside of us testing it should be never changed to what is specified in odo. odo doesn't guarantee compatibility of you use different initImage from what is specified for given odo release.

@cdrage
Copy link
Member Author

cdrage commented Dec 6, 2019

@kadel My reasoning is backwards compatibility. If there was a bug in the odo init image, but not in the CLI, I'd make sense to update the init image. As of right now, if Red Hat were to release an errata, we cannot provide a solution to the current bug other than: "Download the new release". Do you think we should keep it that way?

@kadel
Copy link
Member

kadel commented Dec 11, 2019

Not sure if I understand what you mean.

So if there is bug in init image and we need to release a new version you would like to just build the image, and then tell users to do odo config set BootstrapperImage ... instead of downloading new odo binary?
How users will know when they need to do it? If someone configures a new image like this how we will make sure then when we release a new complete odo version they will start using new image?

I think that this creates a lot more problems than it solves. The mean tool that users are using is CLI. The initImage is just an implementation detail. If there is a bug in the image I think that it makes complete sense to release a new odo binary as this is the only way how we can ensure that everything was properly tested.

As of right now, if Red Hat were to release an errata, we cannot provide a solution to the current bug other than: "Download the new release". Do you think we should keep it that way?

I really think that this is the correct and right way to do it.

@cdrage
Copy link
Member Author

cdrage commented Dec 12, 2019

Going to close this as @kadel 's points are clear, this discussion helped clarify a bunch of things!

@cdrage cdrage closed this as completed Dec 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation
Projects
None yet
Development

No branches or pull requests

3 participants