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

openshift.ks - add "EXTRA" repo vars for testing #423

Merged
merged 1 commit into from
Aug 5, 2014

Conversation

dobbymoodge
Copy link
Contributor

enterprise bug 1119419
https://bugzilla.redhat.com/show_bug.cgi?id=1119419

This adds options for setting up additional repos for jbosseap,
jbossews, rhscl and rhel so we can test alpha/beta packages for channels
we depend on.

# If set, will define a yum repo under the yum,rhsm,rhn install
# methods. This will parallel the regular RHEL channels/repos at the
# same priority. The value of this option sets the "baseurl" setting
# for the defined repo. Useful for testing prerelease content
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing period.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a sentence fragment, not a sentence. No full-stop required. :)

@Miciah
Copy link
Contributor

Miciah commented Jul 30, 2014

Looks fine notwithstanding my nitpicks that don't really matter. 👍

@brenton
Copy link
Contributor

brenton commented Jul 30, 2014

@dobbymoodge, Feel free to put the bug ON_QA with the link to this pull request.

enterprise bug 1119419
https://bugzilla.redhat.com/show_bug.cgi?id=1119419

This adds options for setting up additional repos for jbosseap,
jbossews, rhscl and rhel so we can test alpha/beta packages for channels
we depend on.
jianlinliu added a commit that referenced this pull request Aug 5, 2014
openshift.ks - add "EXTRA" repo vars for testing
@jianlinliu jianlinliu merged commit bcb1d3c into openshift:enterprise-2.1 Aug 5, 2014
{
extra_repo_file=/etc/yum.repos.d/ose_extra.repo
if [ -e "${extra_repo_file}" ]; then
echo > "${extra_repo_file}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can drop the echo and just use > "${extra_repo_file}" unless there is some reason to start with a blank line (in which case a comment would be helpful).

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.

4 participants