-
Notifications
You must be signed in to change notification settings - Fork 695
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
Switch to controller-runtime v0.2 logging package #1708
Labels
good first issue
Good for newcomers
justdoit
Continuous improvement not related to a specific feature
Comments
charith-elastic
added
good first issue
Good for newcomers
justdoit
Continuous improvement not related to a specific feature
labels
Sep 11, 2019
@charith-elastic Can I take a look at this? I'd like to start contributing |
Sure; I don't think anyone else has started working on it. |
great! I'll ping when I have a PR ready |
racevedoo
added a commit
to racevedoo/cloud-on-k8s
that referenced
this issue
Sep 26, 2019
@charith-elastic just opened a PR (#1808) for the |
charith-elastic
pushed a commit
that referenced
this issue
Sep 27, 2019
* change controller-runtime pkg/runtime/log to pkg/log (#1708) * remove unnecessary log config from tests
racevedoo
added a commit
to racevedoo/cloud-on-k8s
that referenced
this issue
Oct 1, 2019
@charith-elastic Remaining PR opened! #1828 |
racevedoo
added a commit
to racevedoo/cloud-on-k8s
that referenced
this issue
Oct 1, 2019
charith-elastic
pushed a commit
that referenced
this issue
Oct 1, 2019
Fixed by #1808 and #1828 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
good first issue
Good for newcomers
justdoit
Continuous improvement not related to a specific feature
Depends on #1188 and relates to #1699
pkg/runtime/log
package is now deprecated and we should switch topkg/log
instead.pkg/log/zap
provides configuration functions that can be used to simplify our logging configuration atpkg/utils/log
The text was updated successfully, but these errors were encountered: