Skip to content

PWX-24576: Implement clouddrive Expand API in CloudOps#113

Merged
nikita-bhatia merged 4 commits intolibopenstorage:masterfrom
nikita-bhatia:PWX-24576
Sep 9, 2022
Merged

PWX-24576: Implement clouddrive Expand API in CloudOps#113
nikita-bhatia merged 4 commits intolibopenstorage:masterfrom
nikita-bhatia:PWX-24576

Conversation

@nikita-bhatia
Copy link
Copy Markdown
Contributor

Jira ticket : https://portworx.atlassian.net/browse/PWX-24576

Changes done :
Implemented Expand API for Oracle cloud

Tests done :
Verified Expansion of disk with and without attachment to instance.

Comment thread oracle/oracle.go Outdated
Comment thread oracle/oracle.go
return false
}

func (o *oracleOps) Expand(volumeID string, newSizeInGiB uint64) (uint64, error) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

First check the current size of the disk. If requested newSizeInGiB is less than or equal to current size then return an error.
Take a look at how are we currently doing in AWS: https://github.com/libopenstorage/cloudops/blob/master/aws/aws.go#L937

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 5, 2022

Codecov Report

Merging #113 (176320a) into master (45c0db2) will increase coverage by 0.06%.
The diff coverage is 0.00%.

❗ Current head 176320a differs from pull request most recent head 0d5bcba. Consider uploading reports for the commit 0d5bcba to get more accurate results

@@            Coverage Diff            @@
##           master    #113      +/-   ##
=========================================
+ Coverage    7.36%   7.43%   +0.06%     
=========================================
  Files          17      17              
  Lines        4383    4345      -38     
=========================================
  Hits          323     323              
+ Misses       4041    4003      -38     
  Partials       19      19              
Impacted Files Coverage Δ
oracle/oracle.go 8.94% <0.00%> (+0.64%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Contributor

@vinayakshnd vinayakshnd left a comment

Choose a reason for hiding this comment

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

LGTM

@nikita-bhatia nikita-bhatia merged commit 0cbeca2 into libopenstorage:master Sep 9, 2022
@nikita-bhatia nikita-bhatia deleted the PWX-24576 branch September 9, 2022 06:09
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