Add timestaps in logs

[afrittoli]

Changes

The controller and webhook logs do not include timestamps, which
makes troubleshooting issues much harder.
Adding the timestamps into the default logging configuration.

Fixes: #800

Signed-off-by: Andrea Frittoli andrea.frittoli@gmail.com

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

• [n/a] Includes tests (if functionality changed/added)
• [n/a] Includes docs (if user facing)
• Commit messages follow commit message best practices
• Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.

Release Notes

Add timestamp in the logs of the tekton pipelines controller, webhook, and eventlisteners

/kind feature

[afrittoli]

/cc @mattmoor

[mattmoor]

/lgtm

[dibyom]

Can we also add this to the EventListener's logger as well? (That will fix #800)

triggers/pkg/reconciler/v1alpha1/eventlistener/eventlistener.go

Line 524 in 6fbcf89

Data: map[string]string{

[afrittoli]

triggers/pkg/reconciler/v1alpha1/eventlistener/eventlistener.go

So, the event listener seems to be using config-logging-triggers - which means that it could load the same config already.
I think the problem might be that the event listener will live in a different namespace and have no access to the logging configmap in the tekton-pipelines namespace.

What I can do is to update the default content of the map, which is used when the config map is not available.

[afrittoli]

Seems to be working fine:

$ k logs el-dashboard-nightly-release-cron-6bfc46d87f-s76th
{"level":"info","ts":"2020-10-15T20:23:59.199Z","caller":"logging/config.go:110","msg":"Successfully created the logger."}
{"level":"info","ts":"2020-10-15T20:23:59.199Z","caller":"logging/config.go:111","msg":"Logging level set to: info"}
{"level":"info","ts":"2020-10-15T20:23:59.200Z","caller":"logging/config.go:78","msg":"Fetch GitHub commit ID from kodata failed","error":"open /var/run/ko/HEAD: no such file or directory"}
{"level":"info","ts":"2020-10-15T20:23:59.200Z","logger":"eventlistener","caller":"logging/logging.go:46","msg":"Starting the Configuration eventlistener","knative.dev/controller":"eventlistener"}
{"level":"info","ts":"2020-10-15T20:23:59.300Z","logger":"eventlistener","caller":"eventlistenersink/main.go:72","msg":"EventListener pod started","knative.dev/controller":"eventlistener"}
{"level":"info","ts":"2020-10-15T20:23:59.304Z","logger":"eventlistener","caller":"eventlistenersink/main.go:98","msg":"Listen and serve on port 8080","knative.dev/controller":"eventlistener"}

[mattmoor]

/lgtm

[dibyom]

/approve

[tekton-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dibyom, mattmoor

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [dibyom]

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