paketo-buildpacks / nodejs

A Cloud Native Buildpack for Node.JS
Apache License 2.0
57 stars 20 forks source link

Unified log output for all buildpacks used (ideally even between language families) #766

Closed c0d1ngm0nk3y closed 2 months ago

c0d1ngm0nk3y commented 1 year ago

When using the nodejs, you get a different log output compared to java. It varies even between buildpacks since the ca-certificates buildpack is used in both meta buildpacks.

Describe the Enhancement

All buildpacks should make use of the nicer log output of libpak/bard.

Possible Solution

How about moving libpak/bard to a separate repository to be used in all buildpacks and then bit by bit use this instead until we can remove libpak/bard and 'packit/scribe' altogether?

Motivation

If you try the npm sample, you will get somewhat confusing log output which is caused by the first buildpack to use a different logger. In addition, if you use java and nodejs, the log output is not as unified than it could be. That would make a better user experience.

loewenstein commented 1 year ago

@c0d1ngm0nk3y I brought this up in the working group meeting and @sophiewigmore's suggestion was to propose this as an RFC (that would likely need a longer period of discussion on convergence).

mhdawson commented 2 months ago

@c0d1ngm0nk3y is this still something you plan on pursuing?

sophiewigmore commented 2 months ago

Personally I had wanted to propose an RFC and then got really busy with other initiatives and never got to it. I'd love to see someone in the community push this forward if they're so inclined

c0d1ngm0nk3y commented 2 months ago

@loewenstein-sap @sophiewigmore I opened https://github.com/paketo-buildpacks/rfcs/pull/308, so this can be probably closed.