Skip to content

fix: make sure we set debug on all components#24

Merged
chmouel merged 1 commit intopipelines-as-code:mainfrom
chmouel:fix-make-sure-we-set-debug-on-all-components
Apr 16, 2026
Merged

fix: make sure we set debug on all components#24
chmouel merged 1 commit intopipelines-as-code:mainfrom
chmouel:fix-make-sure-we-set-debug-on-all-components

Conversation

@chmouel
Copy link
Copy Markdown
Collaborator

@chmouel chmouel commented Apr 16, 2026

Updated the logging configuration to include additional pipeline components. This change ensured that debug information is captured consistently across all relevant services during troubleshooting.

Updated the logging configuration to include additional pipeline
components. This change ensured that debug information is captured
consistently across all relevant services during troubleshooting.

Signed-off-by: Chmouel Boudjnah <chmouel@redhat.com>
@chmouel chmouel merged commit f5c7b02 into pipelines-as-code:main Apr 16, 2026
2 of 3 checks passed
@chmouel chmouel deleted the fix-make-sure-we-set-debug-on-all-components branch April 16, 2026 06:45
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the patch_configmap function in the startpaac script to configure multiple log levels for the pac-config-logging ConfigMap. The patch method was changed from a JSON replace operation to a merge patch to simultaneously set loglevel.pac-watcher, loglevel.pipelines-as-code-webhook, and loglevel.pipelinesascode to debug. I have no feedback to provide as there were no review comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant