Skip to content

Tone down the logging when failing to retrieve scriptpodtemplates#1216

Merged
APErebus merged 1 commit intomainfrom
ap/tweak-warning-message
May 1, 2026
Merged

Tone down the logging when failing to retrieve scriptpodtemplates#1216
APErebus merged 1 commit intomainfrom
ap/tweak-warning-message

Conversation

@APErebus
Copy link
Copy Markdown
Contributor

@APErebus APErebus commented May 1, 2026

Background

Customer feedback has indicated they think this is an error state, not just a warning.

Results

Make the warning message less scary and put the exception as verbose logging (which is off by default)

How to review this PR

Quality ✔️

Pre-requisites

  • I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
  • I have considered informing or consulting the right people, according to the ownership map.
  • I have considered appropriate testing for my change.

@APErebus APErebus requested a review from a team as a code owner May 1, 2026 04:58
Copy link
Copy Markdown
Contributor

@liam-mackie liam-mackie left a comment

Choose a reason for hiding this comment

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

Nice - thanks!

Copy link
Copy Markdown
Contributor

@flin-8 flin-8 left a comment

Choose a reason for hiding this comment

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

LGTM

@APErebus APErebus merged commit 936b51d into main May 1, 2026
51 checks passed
@APErebus APErebus deleted the ap/tweak-warning-message branch May 1, 2026 06:02
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.

3 participants