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

gitserver: allow specifying build strategy #9031

Merged
merged 1 commit into from
May 31, 2016

Conversation

csrwng
Copy link
Contributor

@csrwng csrwng commented May 25, 2016

Adds environment variable to git server yaml to allow you to specify the strategy to use when invoking new-app.

Fixes BZ 1336318

@csrwng
Copy link
Contributor Author

csrwng commented May 25, 2016

@bparees ptal

@csrwng csrwng force-pushed the gitserver_strategy branch from dca53eb to 094cb96 Compare May 25, 2016 20:43
Setting the BUILD_STRATEGY to `docker` will force new builds to be created with the Docker strategy:

```sh
oc set dc/git BUILD_STRATEGY=docker
Copy link
Contributor

Choose a reason for hiding this comment

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

oc set env ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, that's what I meant, will fix

@csrwng csrwng force-pushed the gitserver_strategy branch from 094cb96 to dfa8a40 Compare May 25, 2016 20:48
@csrwng
Copy link
Contributor Author

csrwng commented May 25, 2016

updated

@bparees
Copy link
Contributor

bparees commented May 25, 2016

lgtm

@csrwng
Copy link
Contributor Author

csrwng commented May 25, 2016

[merge]

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to dfa8a40

@csrwng
Copy link
Contributor Author

csrwng commented May 27, 2016

re-[merge]

@csrwng
Copy link
Contributor Author

csrwng commented May 31, 2016

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to dfa8a40

@openshift-bot
Copy link
Contributor

openshift-bot commented May 31, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4236/) (Image: devenv-rhel7_4298)

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4236/)

@openshift-bot openshift-bot merged commit d740851 into openshift:master May 31, 2016
@csrwng csrwng deleted the gitserver_strategy branch July 19, 2016 15:34
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