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

osbuild: ensure /var/tmp is a real directory #1673

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

mvo5
Copy link
Contributor

@mvo5 mvo5 commented Mar 20, 2024

This is a followup for #1649

Instead of symlinking /var/tmp to /tmp which may be on a tmpfs this commit puts it on a real filesystem.

This should fix:
osbuild/bootc-image-builder#285

achilleas-k
achilleas-k previously approved these changes Mar 20, 2024
Copy link
Member

@achilleas-k achilleas-k left a comment

Choose a reason for hiding this comment

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

Thanks! Let's get @henrywang to test this on his issue before merging?

ondrejbudai
ondrejbudai previously approved these changes Mar 20, 2024
@achilleas-k achilleas-k dismissed stale reviews from ondrejbudai and themself via e9d67c9 March 20, 2024 12:51
@achilleas-k achilleas-k force-pushed the fix-var-tmp-in-buildroot branch from e9d67c9 to c4e6651 Compare March 20, 2024 12:57
osbuild/buildroot.py Outdated Show resolved Hide resolved
@mvo5 mvo5 force-pushed the fix-var-tmp-in-buildroot branch 2 times, most recently from d476b72 to 2eded98 Compare March 20, 2024 13:31
kingsleyzissou
kingsleyzissou previously approved these changes Mar 20, 2024
@ondrejbudai ondrejbudai enabled auto-merge (rebase) March 20, 2024 14:26
cgwalters
cgwalters previously approved these changes Mar 20, 2024
This is a followup for osbuild#1649

Instead of symlinking /var/tmp to /tmp which may be on a tmpfs
this commit puts it on a real filesystem.

This should fix:
osbuild/bootc-image-builder#285
@mvo5 mvo5 dismissed stale reviews from cgwalters and kingsleyzissou via d419330 March 20, 2024 14:58
@mvo5 mvo5 force-pushed the fix-var-tmp-in-buildroot branch from 2eded98 to d419330 Compare March 20, 2024 14:58
@henrywang
Copy link
Member

Thanks! Let's get @henrywang to test this on his issue before merging?

Sure. Ping me here if this PR is ready to test.

@cgwalters
Copy link
Contributor

Sure. Ping me here if this PR is ready to test.

I believe the PR is ready to test

Copy link
Member

@supakeen supakeen left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this, it was breaking some things :)

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.

7 participants