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

Render image trigger From field for deploy configs #875

Closed

Conversation

ironcladlou
Copy link
Contributor

Render deployment image trigger From field when present.

Closes #874.

Render deployment image trigger From field when present.

Closes openshift#874.
@ironcladlou
Copy link
Contributor Author

@pmorie PTAL.

@pmorie
Copy link
Contributor

pmorie commented Feb 4, 2015

LGTM, but travis failed @ironcladlou

@pmorie
Copy link
Contributor

pmorie commented Feb 4, 2015

[test]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_openshift3/958/)

@openshift-bot
Copy link
Contributor

Evaluated for origin up to e191f28

@ironcladlou
Copy link
Contributor Author

Looks like something's up with Jenkins. Also, I now want to change this slightly in response to #858, so let's hold off for a bit.

@smarterclayton smarterclayton modified the milestone: 0.3.0 (beta1) Feb 4, 2015
@ironcladlou
Copy link
Contributor Author

Replaced by #910

@ironcladlou ironcladlou closed this Feb 5, 2015
@ironcladlou ironcladlou deleted the deploy-describer-trigger-fix branch February 6, 2015 19:11
jpeeler pushed a commit to jpeeler/origin that referenced this pull request Feb 1, 2018
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.

The deploymentConfig describer isn't aware of From field of image change triggers
4 participants