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

Automated cherry pick of #8467: Stop logging to /var/log/kops-controller.log #8494

Conversation

johngmyers
Copy link
Member

Cherry pick of #8467 on release-1.17.

#8467: Stop logging to /var/log/kops-controller.log

For details on the cherry pick process, see the cherry pick requests page.

Writing to a hostPath from a non-root container requires file
ownership changes, which is difficult to roll out today.  See
discussion in kubernetes#8454

We were primarily using the logfile for e2e diagnostics, so we're
going to look into collecting the information via other means instead.

We also haven't yet shipped this logfile in a released version (though
we have shipped it in beta releases)
@k8s-ci-robot k8s-ci-robot added this to the v1.17 milestone Feb 6, 2020
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 6, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: johngmyers
To complete the pull request process, please assign granular-ryanbonham
You can assign the PR to them by writing /assign @granular-ryanbonham in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@johngmyers johngmyers closed this Feb 6, 2020
@johngmyers johngmyers deleted the automated-cherry-pick-of-#8467-upstream-release-1.17 branch February 6, 2020 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. 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.

3 participants