Skip to content

Use Azure DevOps logger formatting commands #1702

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

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

lbussell
Copy link
Contributor

@lbussell lbussell commented May 7, 2025

This is an attempt at making ImageBuilder's logging output easier to parse by using Azure Pipelines' built-in log formatting commands: https://learn.microsoft.com/en-us/azure/devops/pipelines/scripts/logging-commands?view=azure-devops&tabs=bash#logging-command-format

The big one is the collapsible groups. If this makes any individual commands harder to read, we can adjust the output of those commands as necessary in follow-ups.

@lbussell lbussell requested a review from a team as a code owner May 7, 2025 19:10
@lbussell lbussell marked this pull request as draft May 8, 2025 15:56
@lbussell lbussell force-pushed the azdo-logging branch 2 times, most recently from 7256d07 to b5b0030 Compare May 16, 2025 20:46
@lbussell lbussell marked this pull request as ready for review May 16, 2025 20:50
@lbussell lbussell requested a review from mthalman May 16, 2025 20:50
@lbussell lbussell requested a review from mthalman May 16, 2025 22:01
Comment on lines +17 to +18
/// Only one LoggingGroup can be open at a time. If a second group is created
/// without closing the first, then the first group will not be collapsible.
Copy link
Member

Choose a reason for hiding this comment

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

This may be tough to notice while writing code. Can we enforce this from the logging service?

@lbussell lbussell marked this pull request as draft May 28, 2025 21:37
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.

2 participants