Skip to content

Snapshot feature#26

Merged
ambiknai merged 15 commits intomasterfrom
snapshotmerge
Sep 14, 2022
Merged

Snapshot feature#26
ambiknai merged 15 commits intomasterfrom
snapshotmerge

Conversation

@ambiknai
Copy link
Copy Markdown
Member

@ambiknai ambiknai commented Aug 8, 2022

Orginal PR by @meghnasiingh : #16

Copy link
Copy Markdown
Member

@prankulmahajan prankulmahajan left a comment

Choose a reason for hiding this comment

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

lgtm

//SnapshotedVolume *Volume `json:"SnapshotedVolume"`
// The size of the snapshot, in bytes
SnapshotSize int64 `json:"snapshotSize"`

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Check int64

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

size* | integerminimum: 1The size of this snapshot rounded up to the next gigabyte

So it is int64

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread lib/provider/data_types.go Outdated

// SnapshotList ...
type SnapshotList struct {
Next string `json:"next,omitempty"`
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Remove omitempty. Check.

Comment thread lib/provider/default_volume_provider_test.go
Comment thread lib/provider/default_volume_provider_test.go
Comment thread lib/provider/snapshot_manager.go
Copy link
Copy Markdown
Member

@prankulmahajan prankulmahajan left a comment

Choose a reason for hiding this comment

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

lgtm

Meghna Singh and others added 15 commits September 13, 2022 21:38
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <meghna@Meghnas-MacBook-Pro.local>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <Meghna.Singh1@ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <Meghna.Singh1@ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <Meghna.Singh1@ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <Meghna.Singh1@ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Meghna Singh <Meghna.Singh1@ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
Signed-off-by: Ambika Nair <ambiknai@in.ibm.com>
@ambiknai ambiknai merged commit acf5e53 into master Sep 14, 2022
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.

5 participants