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

[v2] location of SKUPPER_HOME is unconventional #1689

Open
hash-d opened this issue Sep 27, 2024 · 1 comment
Open

[v2] location of SKUPPER_HOME is unconventional #1689

hash-d opened this issue Sep 27, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@hash-d
Copy link
Member

hash-d commented Sep 27, 2024

Describe the bug

SKUPPER_HOME, which houses non-static configuration and other non-static content, is rooted on /usr/local/share/skupper/namespaces/ for the root user.

That location may not be adequate given the characteristics of the files that reside there.

According to hier(7):

/usr This directory is usually mounted from a separate partition. It should hold only shareable, read-only data, so that it can be mounted by various machines running Linux.
(...)
/usr/share
This directory contains subdirectories with specific application data, that can be shared among different architectures of the same OS. Often one finds stuff here that used to live in /usr/doc or /usr/lib or /usr/man.
(...)
/usr/local/share
Local application data that can be shared among different architectures of the same OS.

The main point here is the data not being read-only. Also, the examples of data that is expected to reside there.

How To Reproduce

N/A

Expected behavior

/etc might be a good candidate to host the files, but as some of them are created and updated by the running application, perhaps /var/lib may be a better location for them. Again hier(7):

/var/lib
Variable state information for programs.

Examples of similar uses of that are /var/lib/containers/storage/ and /var/lib/libvirt/dnsmasq

Environment details

N/A

Additional context

See discussion on refdog for start of conversation: skupperproject/refdog#3

@fgiorgetti fgiorgetti self-assigned this Sep 27, 2024
@fgiorgetti fgiorgetti added the enhancement New feature or request label Sep 27, 2024
fgiorgetti added a commit to fgiorgetti/skupper that referenced this issue Oct 25, 2024
* Only rootful namespace home has been updated to /var/lib
* Rootless remain as is assuming that XDG_DATA_HOME seems to be appropriate
  (see: https://specifications.freedesktop.org/basedir-spec/latest/)

Fixes skupperproject#1689
@c-kruse
Copy link
Contributor

c-kruse commented Oct 25, 2024

@hash-d +100 for man hier. I did not know that existed until now, but have always wanted something like that that wasn't some decades old blog post.

fgiorgetti added a commit to fgiorgetti/skupper that referenced this issue Nov 5, 2024
* Only rootful namespace home has been updated to /var/lib
* Rootless remain as is assuming that XDG_DATA_HOME seems to be appropriate
  (see: https://specifications.freedesktop.org/basedir-spec/latest/)

Fixes skupperproject#1689
fgiorgetti added a commit that referenced this issue Nov 5, 2024
* Updated rootful namespace home to var lib

* Only rootful namespace home has been updated to /var/lib
* Rootless remain as is assuming that XDG_DATA_HOME seems to be appropriate
  (see: https://specifications.freedesktop.org/basedir-spec/latest/)

Fixes #1689

* Fixed namespace home path on bootstrap documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants