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

add tmp dir mount everywhere #204

Closed
wants to merge 3 commits into from
Closed

Conversation

monotek
Copy link
Member

@monotek monotek commented Jul 26, 2023

What this PR does / why we need it

  • add tmp dir mount everywhere
  • update zammad to 6.0.0-76

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer

  • .

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Upgrading instructions are documented in the README.md

@mgruner
Copy link
Collaborator

mgruner commented Jul 26, 2023

@monotek not sure if I fully understand this correctly. But why do you think that tmp/ needs to be shared in the cluster?

@monotek
Copy link
Member Author

monotek commented Jul 26, 2023

i saw some tmp related logs.
not sure if it helps.
just testing.

@mgruner
Copy link
Collaborator

mgruner commented Jul 26, 2023

If you find something in tmp/ that needs to be shared in the cluster, we need to look at it. Only var/ should need to be shared at this point.
Only local stuff should get written in tmp/.

@monotek
Copy link
Member Author

monotek commented Jul 26, 2023

I'll check for the error messages later.
Have to work now...

monotek and others added 2 commits August 8, 2023 12:59
Signed-off-by: André Bauer <andre.bauer@staffbase.com>
@monotek
Copy link
Member Author

monotek commented Sep 25, 2023

superseeded by: #215

@monotek monotek closed this Sep 25, 2023
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