Skip to content

Added information about device id 0 for root volume while attaching to VM#6518

Merged
yadvr merged 1 commit into
4.17from
AddInfoAboutDeviceID0
Jun 30, 2022
Merged

Added information about device id 0 for root volume while attaching to VM#6518
yadvr merged 1 commit into
4.17from
AddInfoAboutDeviceID0

Conversation

@harikrishna-patnala

@harikrishna-patnala harikrishna-patnala commented Jun 30, 2022

Copy link
Copy Markdown
Member

Description

This is in the workflow of attaching and detaching a root volume of a VM. We can detach the root volume from a VM and Cloudstack marks it as datadisk. We can reattach the volume to make it root volume again. During attach Device ID 0 has to be passed so that the volume will be converted back to ROOT again

Added a small information in the DeviceID section in Attach volume form.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@GabrielBrascher GabrielBrascher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@yadvr yadvr added this to the 4.17.1.0 milestone Jun 30, 2022
@yadvr

yadvr commented Jun 30, 2022

Copy link
Copy Markdown
Member

Merging this is simply doc change, no need to build pkgs/smoketests.

@yadvr
yadvr merged commit 2326164 into 4.17 Jun 30, 2022
@yadvr
yadvr deleted the AddInfoAboutDeviceID0 branch June 30, 2022 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants