saltyorg / Saltbox

Ansible-based solution for rapidly deploying a Docker containerized cloud media server.
https://docs.saltbox.dev
GNU General Public License v3.0
571 stars 65 forks source link

jaeger: map a volume for `/tmp` #125

Closed powerdude closed 1 year ago

owine commented 1 year ago

What's the purpose of adding this? I did not see any mentions in the project documentation.

powerdude commented 1 year ago

It may not be mentioned, but without it, portainer shows it as unmapped volume and they get left around after the container updates.

Thanks


From: owine @.> Sent: Friday, December 16, 2022 6:57:11 PM To: saltyorg/Saltbox @.> Cc: powerdude @.>; Author @.> Subject: Re: [saltyorg/Saltbox] jaeger: map a volume for /tmp (PR #125)

What's the purpose of adding this? I did not see any mentions in the project documentation.

— Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsaltyorg%2FSaltbox%2Fpull%2F125%23issuecomment-1355845567&data=05%7C01%7C%7C82dfd4d2955a4fde5e9508dadfc13fd0%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638068318331615510%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Qm6rPAQm6g9mAk7692Up5oyBIJxKSDa%2BXe5KnvWBBIo%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAF6UUWBTWRSS3S34UMEN3DWNT6VPANCNFSM6AAAAAATBRTWQE&data=05%7C01%7C%7C82dfd4d2955a4fde5e9508dadfc13fd0%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638068318331615510%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=%2FN6ZPxvN9sOgp5lfHPgXyohByrTHT5sAGglCJZO7Wa8%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

owine commented 1 year ago
[yaml[trailing-spaces]: roles/jaeger/defaults/main.yml#L94](https://github.com/saltyorg/Saltbox/pull/125/files#annotation_7222156260)
Trailing spaces

@saltydk would you prefer a tmpfs for this?