Skip to content

Add detection that we are running inside a podman container#226

Closed
SvenVD wants to merge 1 commit intoCheckmk:masterfrom
SvenVD:podman-fix
Closed

Add detection that we are running inside a podman container#226
SvenVD wants to merge 1 commit intoCheckmk:masterfrom
SvenVD:podman-fix

Conversation

@SvenVD
Copy link
Contributor

@SvenVD SvenVD commented Jul 29, 2020

Not detecting that we are running in a podman container with --tmpfs makes the ./docker-entrypoint.sh and especially the 'omd create --no-tmpfs -u 1000 -g 1000 --admin-password "$CMK_PASSWORD" "$CMK_SITE_ID"' will fail because it uses the wrong path to check if the cmk site is already existing. Result is that it does not create a new site because it thinks it already exists ( because --tmpfs created the path)

Not detecting that we are running in a podman container with --tmpfs makes the ./docker-entrypoint.sh and especially the  'omd create --no-tmpfs -u 1000 -g 1000 --admin-password "$CMK_PASSWORD" "$CMK_SITE_ID"' will fail because it uses the wrong path to check if the cmk site is already existing. Result is that it does not create a new site because it thinks it already exists ( because --tmpfs created the path)
@LarsMichelsen
Copy link
Member

Thanks for your PR. It has passed review and will be merged soon (werk 11257).

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