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

Allow env placeholders for the podman pilot #57

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Conversation

schaefi
Copy link
Contributor

@schaefi schaefi commented Dec 17, 2024

The podman runtime arguments allows to set environment variable placeholders starting with '%' and followed by the name of the environment variable. For example %HOME will be replaced to the value of $HOME of the calling user. If the given placeholder cannot be translated into an existing environment variable it will be turned into the variable name, $HOME in the above example.

@schaefi schaefi requested a review from rjschwei December 17, 2024 09:21
@schaefi schaefi self-assigned this Dec 17, 2024
@schaefi
Copy link
Contributor Author

schaefi commented Dec 17, 2024

@rjschwei The reason why I'm adding this feature is the idea to set the following in our aws flake

podman:
  - "--volume %HOME:/root"

This maps the user home to the home of the only available user inside of the container and should allow to access the calling user .aws setup

@schaefi schaefi force-pushed the allow_env_placeholders branch 3 times, most recently from 56dc9f1 to 46e459c Compare December 17, 2024 10:41
The podman runtime arguments allows to set environment
variable placeholders starting with '%' and followed by
the name of the environment variable. For example %HOME
will be replaced to the value of $HOME of the calling user.
If the given placeholder cannot be translated into an
existing environment variable it will be turned into the
variable name, $HOME in the above example.
@schaefi schaefi force-pushed the allow_env_placeholders branch from 46e459c to 707023d Compare December 17, 2024 11:06
@schaefi schaefi merged commit 9ae82b4 into main Dec 17, 2024
7 checks passed
@schaefi schaefi deleted the allow_env_placeholders branch December 17, 2024 13:45
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