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

functional tests - output directory has to be created by hand with correct rights before deploying the chart #28

Open
pmauduit opened this issue Dec 18, 2024 · 1 comment

Comments

@pmauduit
Copy link
Member

The report output is saved under a directory which is mounted into the cronjob pod. But the job will fail if:

  1. the directory does not exist
  2. the pod does not have rights to write into it (10000:10001)

In my humble opinion this manual step should be avoided and managed by the chart.

@edevosc2c
Copy link
Member

edevosc2c commented Dec 18, 2024

Good idea but

It's difficult to introduce such a mechanism without adding a root container for a one-off container that creates the directory + set the correct permissions. But I guess that shouldn't be an issue if this is configurable and can be deactivated on infrastructures that require no root containers.

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

No branches or pull requests

2 participants