Added code for CSCwo96334:/tmp directory disk space for snapshot storage#368
Open
Harinadh-Saladi wants to merge 1 commit intodatacenter:masterfrom
Open
Added code for CSCwo96334:/tmp directory disk space for snapshot storage#368Harinadh-Saladi wants to merge 1 commit intodatacenter:masterfrom
Harinadh-Saladi wants to merge 1 commit intodatacenter:masterfrom
Conversation
dhaselva
reviewed
Mar 18, 2026
| if dn: | ||
| mountpoint = dn.group('mountpoint') | ||
| # CSCwo96334: Skip /tmp faults if tversion >= 6.1(4a) (snapshots use /data instead) | ||
| if mountpoint == '/tmp' and tversion and not tversion.older_than("6.1(4a)"): |
There was a problem hiding this comment.
Check and update the target version.
Issue fixed in 6.0(9f) and I can see the bug in the resolved issues section. Issue also got fixed in 6.1(4h) as per bug but I do not see the bug id under resolved issues section. Validate and include the right target versions.
dhaselva
reviewed
Mar 18, 2026
| if dn: | ||
| mountpoint = dn.group('mountpoint') | ||
| # CSCwo96334: Skip /tmp faults if tversion >= 6.1(4a) (snapshots use /data instead) | ||
| if mountpoint == '/tmp' and tversion and not tversion.older_than("6.1(4a)"): |
There was a problem hiding this comment.
if mountpoint == '/tmp' and tversion and not tversion.older_than("6.1(4a)"):
Remove 'and tversion' .
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added tmp_dir_snapshot_storage_check to validate /tmp space before upgrades < 6.1(4a)
Check monitors F1527/F1528/F1529 faults on /tmp mount point only
Returns FAIL_UF if /tmp disk space issues found, NA for versions >= 6.1(4a)
Added comprehensive test cases covering affected and fixed versions
Added test fixtures for positive, negative, and mixed fault scenarios.
Have tested in the lab and enclosed comprehensive logs.
CSCwo96334_comprehensive_Logs.txt
CSCwo96334_Pytest_Logs.txt