Skip to content

web: Prevent resizing or deleting re-used partitions#2027

Merged
ancorgs merged 1 commit intoagama-project:storage-config-uifrom
ancorgs:protect_reused
Feb 20, 2025
Merged

web: Prevent resizing or deleting re-used partitions#2027
ancorgs merged 1 commit intoagama-project:storage-config-uifrom
ancorgs:protect_reused

Conversation

@ancorgs
Copy link
Contributor

@ancorgs ancorgs commented Feb 19, 2025

Problem

When using a custom space policy, it is possible to mark a reused partition to be shrunk or deleted.

Solution

Prevent that at UI level

canttouchme

Testing

  • Tested manually
  • Extended the component unit tests.

@ancorgs ancorgs changed the title WIP: web: Prevent resizing or deleting re-used partitions web: Prevent resizing or deleting re-used partitions Feb 20, 2025
@ancorgs ancorgs marked this pull request as ready for review February 20, 2025 08:52
import { Table, Td, Th, Tr, Thead, Tbody } from "@patternfly/react-table";
import { useConfigModel } from "~/queries/storage/config-model";

const isUsedPartition = (partition: configModel.Partition): boolean => {
Copy link
Contributor

Choose a reason for hiding this comment

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

NP: these are the things I still envision in a storage utils module. But it's hard to convince you guys.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is not that we don't agree. There is also a comment right below that like saying tht we need to reconsider all that. It currently feels like dealing with the old YaST huge storage "maps" (pre storage-ng).

@ancorgs ancorgs merged commit 141dd15 into agama-project:storage-config-ui Feb 20, 2025
1 check passed
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
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