chains icon indicating copy to clipboard operation
chains copied to clipboard

converting a few info into debug message

Open pritidesai opened this issue 2 years ago • 4 comments

Changes

A taskRun/pipelineRun is watched as soon as created. Until the run is complete and ready to proceed further, the chains controller is logging that the taskRun/pipelineRun is still running at the info level. This looks good but generates a lot of info in the controller logs and exponentially increases with the number of taskRuns and pipelineRuns. Converting such message to be reported at the debug log level.

Formatting a few error message such that the namespace/name is enclosed with parenthesis so that its easy to read.

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • [ ] Has Docs included if any changes are user facing
  • [x] Has Tests included if any functionality added or changed
  • [x] Follows the commit message standard
  • [x] Meets the Tekton contributor standards (including functionality, content, code)
  • [x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
  • [ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes

NONE

pritidesai avatar Sep 07 '23 05:09 pritidesai

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign wlynch after the PR has been reviewed. You can assign the PR to them by writing /assign @wlynch 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

tekton-robot avatar Sep 07 '23 05:09 tekton-robot

/retest

pritidesai avatar Sep 12 '23 16:09 pritidesai

@pritidesai: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

tekton-robot avatar Jan 17 '24 22:01 tekton-robot

@pritidesai are you still interested in driving this PR forward?

lcarva avatar Oct 15 '24 13:10 lcarva

Let's close this out due to lack of activity. Feel free to reopen!

lcarva avatar Jan 20 '25 13:01 lcarva