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

/var/qm test #552

Merged
merged 8 commits into from
Sep 11, 2024
Merged

/var/qm test #552

merged 8 commits into from
Sep 11, 2024

Conversation

Yarboa
Copy link
Collaborator

@Yarboa Yarboa commented Sep 11, 2024

PR based on this #538
Tested on soc c9s manually

dougsland and others added 8 commits September 9, 2024 16:56
Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
The shebang (#!) at the beginning of the line indicates that the script should be executed using the specified interpreter, which in this case is /bin/bash. This means that the script will be run using the Bash shell.

Signed-off-by: nsednev <nsednev@redhat.com>
Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
No need to be specific for disk, lets to the whole thing

Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
on SoC stop qm mounting
check if parted is installed

Signed-off-by: Yariv Rachmani <yrachman@redhat.com>
Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
Signed-off-by: Douglas Landgraf <dlandgra@redhat.com>
@Yarboa Yarboa marked this pull request as draft September 11, 2024 03:34
@Yarboa
Copy link
Collaborator Author

Yarboa commented Sep 11, 2024

@dougsland please review
this one was tested on:
gate PASS
c9s manual PASS
SoC, <testcase name="/external/tests/ffi/disk" time="134.000000"> is FAIL,
junit.md

So script is good

/dev/sde38       16G  3.4G   12G  23% /
/dev/sde41       12G  1.2G   10G  11% /var/qm

@dougsland
Copy link
Collaborator

I'd merge what we have now working, tag, release and that's it.

@Yarboa
Copy link
Collaborator Author

Yarboa commented Sep 11, 2024

I'd merge what we have now working, tag, release and that's it.

@dougsland please review, rebase and comment

@Yarboa Yarboa marked this pull request as ready for review September 11, 2024 14:58
@dougsland
Copy link
Collaborator

if works for SoC and upstream, let's merge now. We can always improve later. I will tag it and make a release.

@dougsland
Copy link
Collaborator

Thanks for your efforts downstream @Yarboa !

@dougsland dougsland merged commit 49da5e2 into containers:main Sep 11, 2024
10 checks passed
This was referenced Sep 11, 2024
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