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

Jump to bottom

ovirt: move /var/tmp and /var/crash to own partition #3931

Merged
merged 1 commit into from Mar 9, 2022
Merged

ovirt: move /var/tmp and /var/crash to own partition #3931

merged 1 commit into from Mar 9, 2022

Conversation

poncovka
Copy link
Contributor

@poncovka poncovka commented Mar 8, 2022

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=2057475
Signed-off-by: Sandro Bonazzola sbonazzo@redhat.com

Resolves: rhbz#2057475
Ported from: #3888

@sandrobonazzola @poncovka
ovirt: move /var/tmp and /var/crash to own partition
f4cbe44 
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=2057475
Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>

Resolves: rhbz#2057475
@poncovka
Copy link
Contributor Author

poncovka commented Mar 8, 2022

/kickstart-test --testtype smoke

rvykydal
rvykydal approved these changes Mar 9, 2022
View reviewed changes
Copy link
Contributor

@rvykydal rvykydal left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@VladimirSlavik
Copy link
Contributor

We won't be getting kickstart tests here, because "Branch rhel-8.6 is not supported by kickstart tests yet!" - see https://github.com/rhinstaller/anaconda/runs/5466138667?check_suite_focus=true

VladimirSlavik
VladimirSlavik approved these changes Mar 9, 2022
View reviewed changes
Copy link
Contributor

@VladimirSlavik VladimirSlavik left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for porting this!

@rvykydal
Copy link
Contributor

rvykydal commented Mar 9, 2022 •
edited

We won't be getting kickstart tests here, because "Branch rhel-8.6 is not supported by kickstart tests yet!" - see https://github.com/rhinstaller/anaconda/runs/5466138667?check_suite_focus=true

Yes, the tests passed on rhel-8 branch (#3888) so believe we should be ok for this PR.

M4rtinK
M4rtinK approved these changes Mar 9, 2022
View reviewed changes
Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

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

Looks good to me as well, thanks! :)

@poncovka poncovka added the ready to merge The PR can be merged. It should have all BZ flags required for releasing set (usually release+). label Mar 9, 2022
@rvykydal rvykydal merged commit 81bc6ad into rhinstaller:rhel-8.6 Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@M4rtinK M4rtinK M4rtinK approved these changes

@rvykydal rvykydal rvykydal approved these changes

@jkonecny12 jkonecny12 jkonecny12 approved these changes

@VladimirSlavik VladimirSlavik VladimirSlavik approved these changes

Assignees
No one assigned
Labels
ready to merge The PR can be merged. It should have all BZ flags required for releasing set (usually release+). rhel8-branch
Milestone
No milestone
6 participants
@poncovka @VladimirSlavik @rvykydal @M4rtinK @jkonecny12 @sandrobonazzola