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

The file structures in the OL image bulit by paketo and ci.docker are different #184

Open
gkwan-ibm opened this issue Jun 23, 2022 · 3 comments
Labels
type:enhancement A general enhancement

Comments

@gkwan-ibm
Copy link

gkwan-ibm commented Jun 23, 2022

The OL image built by ci.docker has following file structures

  • liberty runtime is located at /opt/ol/wlp
  • server configuration is also located at /config
  • logs files are located at /logs
  • ifixes should be put at /opt/ol/fixes/

The OL image built by Paketo

  • liberty runtime /layers/paketo-buildpacks_liberty/open-liberty-runtime-kernel
  • server configuration and logs is located at /layers/paketo-buildpacks_liberty/open-liberty-runtime-kernel/usr/servers/defaultServer

For user experience reason, it is better for the Paketo OL image to have same file structures as ci.docker does.

@dmikusa dmikusa added the type:enhancement A general enhancement label Jun 27, 2022
@dmikusa
Copy link
Contributor

dmikusa commented Jun 27, 2022

That is not possible at the moment, sorry.

The buildpack spec requires files to be added under /layers. It is required for reasons related to buildpack rebase that I won't go into but also for security reasons. Buildpacks do not run as root so they cannot put files into the locations you referenced. /layers is a special location that is owned by the cnb user/group so buildpacks can write to it.

It might be possible to do this in the future with a builder extension, which can have elevated permissions. That functionality hasn't landed in buildpacks yet though so there's nothing Paketo could do about it now.

I agree with what you're saying, so I'll leave this open as a potential future enhancement. I can't guarantee it'll be implemented or provide any timeframe though, due to the reasons above.

@gkwan-ibm
Copy link
Author

hi @dmikusa-pivotal

is there a doc to describe the file structure?

@dmikusa
Copy link
Contributor

dmikusa commented Jun 29, 2022

The buildpacks API spec dictates where most things end up, https://github.com/buildpacks/spec/blob/main/buildpack.md.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

2 participants