Bug 1690799
Summary: | Better remove spec.output field from the example bc of openshift webconsole | ||||||
---|---|---|---|---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | XiuJuan Wang <xiuwang> | ||||
Component: | Management Console | Assignee: | Samuel Padgett <spadgett> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Yadan Pei <yapei> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | 4.1.0 | CC: | aos-bugs, jokerman, mmccomas | ||||
Target Milestone: | --- | ||||||
Target Release: | 4.1.0 | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | If docs needed, set a value | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2019-03-28 17:04:53 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
XiuJuan Wang
2019-03-20 09:39:15 UTC
Created attachment 1546282 [details]
Need remove spec.output from bc
Samuel, Sorry, slow ydown your step, let me add more areas for improvement: 1. As the attachment comment #2, also need remove spec.output from the examples bc of `Build from Dockerfile`and `Source-to-Image (S2I) build`. 2. The `pipeline build` need have a description, such as "before pipeline build create, need a jenkins pod runs", orelse the pipeline build always keep 'new' status lack of jenkins running. 3. For the `Build from Dockerfile`, need modify spec.strategy.dockerStrategy.imagestreamtag to use ruby:2.2, since https://github.com/openshift/ruby-hello-world.git is little outdate, and ruby:latest point to ruby:2.5. The combine conduces docker build failed. (In reply to XiuJuan Wang from comment #3) > 1. As the attachment comment #2, also need remove spec.output from the > examples bc of `Build from Dockerfile`and `Source-to-Image (S2I) build`. I'm not sure I agree with this. We need some YAML examples with an output image stream, otherwise it will be hard for users to discover how to set that. I'm a little unsure if this change in general is right even for the default template. It's nice if the YAML works out of the box, but really it's meant to be a template for creating real applications. To use these for real, you need to modify the example YAML. The more we take out of the examples, the harder it is to know that those capabilities exist and the properties to set to enable them. We now have a build that runs, but it doesn't do anything with the image. I guess it's better, but it's still unclear to me. I'd rather keep the output image in the other examples. > 2. The `pipeline build` need have a description, such as "before pipeline > build create, need a jenkins pod runs", orelse the pipeline build always > keep 'new' status lack of jenkins running. I think this really needs to be a doc link to the pipelines documentation, but the 4.0 doc is still being written. This will need to be a follow-on task. > 3. For the `Build from Dockerfile`, need modify > spec.strategy.dockerStrategy.imagestreamtag to use ruby:2.2, since > https://github.com/openshift/ruby-hello-world.git is little outdate, and > ruby:latest point to ruby:2.5. The combine conduces docker build failed. I think the right thing to do is update the ruby-hello-world example itself. I would rather make the example work with the newer Ruby version than to use a backlevel Ruby. I'm guessing this is also a problem for the example git repository when created through the "Developer Catalog"? This should be a separate bug. Moving back to ON_QA. (In reply to Samuel Padgett from comment #4) > (In reply to XiuJuan Wang from comment #3) > > 1. As the attachment comment #2, also need remove spec.output from the > > examples bc of `Build from Dockerfile`and `Source-to-Image (S2I) build`. > > I'm not sure I agree with this. We need some YAML examples with an output > image stream, otherwise it will be hard for users to discover how to set > that. I'm a little unsure if this change in general is right even for the > default template. It's nice if the YAML works out of the box, but really > it's meant to be a template for creating real applications. To use these for > real, you need to modify the example YAML. The more we take out of the > examples, the harder it is to know that those capabilities exist and the > properties to set to enable them. We now have a build that runs, but it > doesn't do anything with the image. I guess it's better, but it's still > unclear to me. I'd rather keep the output image in the other examples. It will make costomers confused if they use these examples to trigger builds. But I agreed with you about "We need some YAML examples with an output image stream" > > 3. For the `Build from Dockerfile`, need modify > > spec.strategy.dockerStrategy.imagestreamtag to use ruby:2.2, since > > https://github.com/openshift/ruby-hello-world.git is little outdate, and > > ruby:latest point to ruby:2.5. The combine conduces docker build failed. > > I think the right thing to do is update the ruby-hello-world example itself. > I would rather make the example work with the newer Ruby version than to use > a backlevel Ruby. I'm guessing this is also a problem for the example git Agreed. > repository when created through the "Developer Catalog"? This should be a > separate bug. > > > Moving back to ON_QA. Verified this issue with 4.0.0-0.nightly-2019-03-22-002648 build. |