Skip to content

fix: mount /tmp as a tmpfs so that file names can be 255 in length#93

Open
dkuc wants to merge 1 commit into
theforeman:masterfrom
dkuc:tmpfs_max_file_name_fix
Open

fix: mount /tmp as a tmpfs so that file names can be 255 in length#93
dkuc wants to merge 1 commit into
theforeman:masterfrom
dkuc:tmpfs_max_file_name_fix

Conversation

@dkuc
Copy link
Copy Markdown
Contributor

@dkuc dkuc commented Feb 19, 2026

Some customers have file systems that do not allow filenames up 255 characters. This change mounts the /tmp directory as tmpfs. This decouples the container from the host file system. This allows insights-core to create files with length of 255.

Note: tmpfs only claims memory as it needs it so this change will not result in a static 6GB of memory being allocated.

This is one potential fix for
https://issues.redhat.com/browse/SAT-42272#

@dkuc dkuc changed the title fix: mount /tmp as a tmpfs directory so that file names can be 255 ch… fix: mount /tmp as a tmpfs so that file names can be 255 in length Feb 19, 2026
@dkuc dkuc force-pushed the tmpfs_max_file_name_fix branch from bb05225 to 832e3b6 Compare February 19, 2026 15:17
Comment thread manifests/core_yuptoo.pp
'ContainerName' => 'iop-core-yuptoo',
'Network' => 'iop-core-network',
'Exec' => 'python -m main',
'Tmpfs' => '/tmp:rw,size=2G,mode=1777',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am a bit concerned that having three tmpfs mounts, all having an upper limit of 2Gi, could have a negative impact on the stability of the hosting system.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It could also have positive impact - extracting and reading archives in memory instead of on disk should be faster and lessen disk I/O. Hard to say without measuring.

@ehelms
Copy link
Copy Markdown
Member

ehelms commented Feb 20, 2026

@pablomh @evgeni either of y'all have thoughts on this direction or alternatives?

@pablomh
Copy link
Copy Markdown
Contributor

pablomh commented Feb 20, 2026

My immediate concern would be environments where it wouldn't be allowed to mount /tmp as tmpfs.

@dkuc
Copy link
Copy Markdown
Contributor Author

dkuc commented Feb 21, 2026

The insights-core team did actually release an update that uses smaller file paths. However, the first upload you do uses the insights-core version that ships with insights-client. This first upload would fail - you would need to re-register and do a second upload to get this application side fix to work.

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.

4 participants