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

Revert: Pass the container name to deployment config log options #10609

Merged
merged 2 commits into from
Aug 26, 2016

Conversation

mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Aug 24, 2016

Fixes: #10598

I will open a follow-up issue to discuss how to fix the original issue without breaking the backward compatibility.

@mfojtik
Copy link
Contributor Author

mfojtik commented Aug 24, 2016

@Kargakis @jwforres @deads2k PTAL

@deads2k
Copy link
Contributor

deads2k commented Aug 24, 2016

You had a separate commit for tests?

[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 b5fac0b

@openshift-bot
Copy link
Contributor

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

@0xmichalis
Copy link
Contributor

#10002 [merge]

Looks like the replication controller and the deployer pod both exist and run but for some reason the watch for the running deployment expires, and we end up reporting that the replication controller wasn't found.

@mfojtik
Copy link
Contributor Author

mfojtik commented Aug 25, 2016

#9355 [merge]

@0xmichalis
Copy link
Contributor

#10635 [merge]

@mfojtik
Copy link
Contributor Author

mfojtik commented Aug 26, 2016

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 26, 2016

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

@0xmichalis
Copy link
Contributor

#10663 [merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b5fac0b

@openshift-bot openshift-bot merged commit de02920 into openshift:master Aug 26, 2016
@mfojtik mfojtik deleted the revert-logging branch September 5, 2018 21:08
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.

Deployment config log validation is backwards incompatible and too strict
4 participants