Skip to content
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

Feature/log section processor #3272

Closed
wants to merge 14 commits into from
Closed

Conversation

bomoko
Copy link
Contributor

@bomoko bomoko commented Aug 28, 2022

This PR introduces a new way of Parsing and displaying the build logs, it introduces sections that, in combination with the changes in uselagoon/build-deploy-tool#98 will make the build logs easier to navigate.

It splits the build log where it finds a section header of the form

/##############################################\n(BEGIN) (.+)\n##############################################/;

and uses whatever is in the section capture group as the section heading. Anything falling between the section heading and the next section heading is considered to be the log text for that section.

We fall back to displaying the entire log in a single section if there are no sections found.

Below is an example of the output

image

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for inclusion in changelog

Explain the details for making this change. What existing problem does the pull request solve?

@tobybellwood
Copy link
Member

transferred to uselagoon/lagoon-ui#23

@tobybellwood tobybellwood deleted the feature/log_section_processor branch July 24, 2023 01:20
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