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

PRODENG-2780 mke output handler #557

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

james-nesbitt
Copy link
Collaborator

@james-nesbitt james-nesbitt commented Feb 19, 2025

Refactor how we handle the output from the MKE and MSR docker run bootstrapping. Primarily focused on better consuming log messages from the bootstrappers natively into launchpad log messages (proper log level handling, better message handling,) but also de-duping the docker runs themselves

  • new buffer tooling that can act like an output logging buffer for rig exec
  • new buffer line handling for logrus output (interprets command output as txt logrus output so that it can be consumed as launchpad logrus output)
  • new MKE and MSR functions for running the respective bootstrapper on a cluster, with better output handling using the new buffer tool to prevent embedded logrus
  • install, upgrade, reset and image pull phases use the new bootstrapper functions (reduce redundancy).

The command buffer may actually be something that the k0sproject/rig project is interested in.

@james-nesbitt james-nesbitt requested a review from nekwar February 19, 2025 15:45
@james-nesbitt james-nesbitt force-pushed the PRODENG-2780-mke-output-handler branch 4 times, most recently from 2badd0b to c46dd6a Compare February 20, 2025 13:54
@james-nesbitt james-nesbitt force-pushed the PRODENG-2780-mke-output-handler branch 7 times, most recently from dfaa3f1 to f95c00c Compare March 12, 2025 12:59
@james-nesbitt
Copy link
Collaborator Author

This is ready for testing.

@james-nesbitt james-nesbitt marked this pull request as ready for review March 12, 2025 13:00
@james-nesbitt james-nesbitt force-pushed the PRODENG-2780-mke-output-handler branch from f95c00c to e856199 Compare March 12, 2025 13:01
@james-nesbitt
Copy link
Collaborator Author

I think that we may hold off on this effort for the current release.

@james-nesbitt james-nesbitt force-pushed the PRODENG-2780-mke-output-handler branch 5 times, most recently from e650d8c to 2ee2c71 Compare March 19, 2025 13:05
- new MKE function for running the bootstrapper on a cluster
- new buffer function that can be used to capture rig Exec command
  output for scanning without early EOFs
- some buffer helpers that can relay rig Exec output if it outputs
  logrus

Signed-off-by: James Nesbitt <[email protected]>

PRODENG-2780 MSR cmdbuffer usage

- MKE style cmdbuffer usage
- migrating the install and upgrade and pull images phases to use the new cmdbuffer
  based bootstrap functions

Signed-off-by: James Nesbitt <[email protected]>
@james-nesbitt james-nesbitt force-pushed the PRODENG-2780-mke-output-handler branch from 2ee2c71 to 15df06e Compare March 19, 2025 14:54
@james-nesbitt
Copy link
Collaborator Author

the k0s team is not interested in the cmdbuffer approach.

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