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

Bug 1495460 - Fix build config source namespace #2169

Merged
merged 1 commit into from
Sep 26, 2017

Conversation

spadgett
Copy link
Member

Correctly set the build config source image namespace when adding a builder using advanced options.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1495460

Correctly set the build config source image namespace when adding a
builder using advanced options.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1495460
@spadgett
Copy link
Member Author

@jeff-phillips-18 After looking at this more, I think it's better to undo the ApplicationGenerator change in #2068 and update the test. input.namespace should always be defined. Do you mind taking another look?

@spadgett
Copy link
Member Author

Thanks @jeff-phillips-18

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to e7f6d90

@openshift-bot
Copy link

openshift-bot commented Sep 26, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/270/) (Base Commit: ef5746b) (PR Branch Commit: e7f6d90)

@openshift-bot openshift-bot merged commit 9a50f87 into openshift:master Sep 26, 2017
@spadgett spadgett deleted the build-advanced-options branch September 26, 2017 14:13
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.

None yet

3 participants