-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Cleaning the build docs #492
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/a//
|
@soltysh comments complete. |
creating_images/custom.adoc
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copy/paste leftover?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of course...
550be6f to
8237d28
Compare
|
@adellape addressed your comments, except for one, reasoning inline. |
|
@soltysh Sorry, got distracted and wasn't done. :( At least one more comment incoming. |
dev_guide/builds.adoc
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this ==== broke the rest of the headings/TOC.
|
@soltysh Done commenting, thanks! |
|
@adellape I'm done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this isn't really how i'd describe custom builder... the role it fills is basically anything we may not have anticipated in our STI and Docker builders. In addition the most likely use case is not to produce a new docker image (use Docker or STI builds for that), it's to produce individual artifacts like jars, wars, installable zips, etc. The point is that it can perform any steps you want to produce some output artfact, and then is responsible for pushing that artifact to some location (eg scp'ing it to a server).
It also provides a backdoor way of introducing a CI/CD flow since you could write a custom builder that runs unit or integration tests, in theory. (sort of like the Jenkins custom job type)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll update this in a follow up PR.
On Jun 15, 2015 6:58 PM, "Ben Parees" [email protected] wrote:
In creating_images/custom.adoc
#492 (comment)
:+{product-author}
+{product-version}
+:data-uri:
+:icons:
+:experimental:
+:toc: macro
+:toc-title:
+
+toc::[]
+
+== Overview
+link:../architecture/core_objects/builds.html#custom-build[Custom build] is designed
+to fill the gap that was created when everybody jumped into creating docker images.
+Still there is a requirement to produce packages with all the software being installed
+and most importantly one needs to be able to build base docker images. This is
+where Custom build is the perfect match to fill in that gap.this isn't really how i'd describe custom builder... the role it fills is
basically anything we may not have anticipated in our STI and Docker
builders. In addition the most likely use case is not to produce a new
docker image (use Docker or STI builds for that), it's to produce
individual artifacts like jars, wars, installable zips, etc. The point is
that it can perform any steps you want to produce some output artfact, and
then is responsible for pushing that artifact to some location (eg scp'ing
it to a server).It also provides a backdoor way of introducing a CI/CD flow since you
could write a custom builder that runs unit or integration tests, in
theory. (sort of like the Jenkins custom job type)—
Reply to this email directly or view it on GitHub
https://github.com/openshift/openshift-docs/pull/492/files#r32441856.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I'll wait for #490 to rebase on top of it. I'll be doing 2nd pass through dev_guide/builds.adoc on Mon.
@bparees @adellape fyi and have a good weekend!