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

Automatic common/ update #321

Merged

Conversation

mbaldessari
Copy link
Contributor

This is part of an automatic process run by mbaldessari on Wed Jan 3 10:05:31 AM CET 2024

…ttern.sh

This allows us to drop the /root bind mount and it will also show any
errors related to paths in the proper folder. E.g. any permission
problem of KUBECONFIG files won't be shown as /root/kubeconfig (inside
the container) but as the proper path inside the /home folder.

Tested on F38, F39, RHEL8.9 and RHEL9.2
If podman is not installed we get the following unfriendly output:
[michele@rhel1]~/multicloud-gitops% ./pattern.sh
./pattern.sh: line 10: podman: command not found
./pattern.sh: line 10: podman: command not found
./pattern.sh: line 32: podman: command not found

Let's bail out and have a generic function to check for that in case
we need to add other requirements
@mbaldessari mbaldessari self-assigned this Jan 3, 2024
@mbaldessari mbaldessari merged commit fa6bf5c into validatedpatterns:main Jan 3, 2024
4 checks passed
tim-stasse pushed a commit to tim-stasse/multicloud-gitops that referenced this pull request Mar 18, 2024
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