Skip to content

Enhance volume group status checks and clarify function naming#985

Closed
cgalibern wants to merge 3 commits intoopensvc:mainfrom
cgalibern:dev
Closed

Enhance volume group status checks and clarify function naming#985
cgalibern wants to merge 3 commits intoopensvc:mainfrom
cgalibern:dev

Conversation

@cgalibern
Copy link
Contributor

@cgalibern cgalibern commented Feb 27, 2026 

### Description of Changes

- Updated the `Status` function to include logical volume (LV) status checks. This prevents an "UP" status for volume groups where no volumes are activated.
- Renamed `ActiveLVs` to `ActiveLVDevices` for improved clarity in the codebase.
- Updated the CHANGELOG with refined syntax information regarding key-related commands.

@cgalibern
Update CHANGELOG: Uqe om3 syntax for key-related commands
9e4da2b
@cgalibern
[resdiskvg/util] Rename ActiveLVs to ActiveLVDevices for clarity
d737c2a
@cgalibern
[resdiskvg] Enhanced Status function to integrate LV status checks
e690760 
This commit improves Status to fix unexpected UP status on volume group
where all volumes are not activated (this may happen on automatically
detected volume group)

We this commit, to have an UP status, the volume group with some volumes
needs to have at least one activated volume.
@cgalibern cgalibern marked this pull request as draft February 27, 2026 07:20
@cgalibern
Copy link
Contributor Author

cgalibern commented Feb 27, 2026

Not needed, has vg tagging

@cgalibern cgalibern closed this Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arnaudveron arnaudveron Awaiting requested review from arnaudveron

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cgalibern