Skip to content

Conversation

@iltyty
Copy link
Owner

@iltyty iltyty commented Jun 23, 2025

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce an API-breaking change?:

none

@iltyty iltyty changed the title Allow modifying volume topology [New GRPC] Allow modifying volume topology Jun 23, 2025
@iltyty
Copy link
Owner Author

iltyty commented Jun 23, 2025

详细描述下Attach、ModifyTopology不同阶段,ModifyTopology的表现
两个的区别:

  1. 是否需要allow_modify_topology字段
  2. 返回的topology是否为required

@iltyty
Copy link
Owner Author

iltyty commented Jun 23, 2025

不要手动输入topology,作为ControllerModifyVolume的superset。
新增capability,CONTROLLER_MODIFY_VOLUME_TOPOLOGY。

@iltyty iltyty force-pushed the kep-5381-new-grpc branch from 6d9cf20 to 7984786 Compare June 23, 2025 12:33
spec.md Outdated
**ALPHA FEATURE**

A Controller plugin MUST implement this RPC call if the plugin has the MODIFY_VOLUME_TOPOLOGY controller capability.
This RPC allows the CO to change mutable key attributes and the accessible topology of a volume.

Choose a reason for hiding this comment

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

Do you mean Allows the CO change the accessible topology of a volume by mutable key attributes?

@iltyty iltyty force-pushed the kep-5381-new-grpc branch 2 times, most recently from 6bc8ad2 to cf0b28d Compare July 1, 2025 14:48
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.

3 participants