Skip to content

Ensure upgrades apply latest journald settings #5796

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

Conversation

michaelgugino
Copy link
Contributor

Currently, existing clusters might not have journald
configurations applied. This may result in a rate-
limiting of important log messages on openshift-masters.

This commit ensures that journald settings are applied
during the upgrade process openshif-masters.

Fixes: #5642

Currently, existing clusters might not have journald
configurations applied. This may result in a rate-
limiting of important log messages on openshift-masters.

This commit ensures that journald settings are applied
during the upgrade process openshif-masters.

Fixes: openshift#5642
@michaelgugino michaelgugino added kind/bug Categorizes issue or PR as related to a bug. priority/P1 labels Oct 18, 2017
@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 18, 2017
@michaelgugino
Copy link
Contributor Author

bot, retest this please

@sdodson
Copy link
Member

sdodson commented Oct 18, 2017

Thanks for picking this up!
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 18, 2017
@michaelgugino
Copy link
Contributor Author

@michaelgugino
Copy link
Contributor Author

/test install
/test upgrade

Copy link

@soltysh soltysh left a comment

Choose a reason for hiding this comment

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

lgtm

@michaelgugino
Copy link
Contributor Author

/test install
/test upgrade

@openshift-merge-robot
Copy link
Contributor

/test all [submit-queue is verifying that this PR is safe to merge]

@sdodson
Copy link
Member

sdodson commented Oct 23, 2017

/test all

@openshift-merge-robot
Copy link
Contributor

/test all [submit-queue is verifying that this PR is safe to merge]

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 2e751e6 into openshift:master Oct 24, 2017
openshift-merge-robot added a commit to openshift/origin that referenced this pull request Dec 6, 2017
Automatic merge from submit-queue (batch tested with PRs 17217, 17597, 17606).

Remove journald limits

@mfojtik this is dropping the hacks we had in place to tweak journald, now that ansible is doing it (openshift/openshift-ansible#3753 and openshift/openshift-ansible#5796). I'm additionally bringing back the deployments e2e's that were suffering from it. Let's see how far we can go with it. 

/cc @tnozicka 

Fixes #14785
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. priority/P1 size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants