Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates in the Disk v1beta3 API #131

Closed
mauriciopoppe opened this issue May 4, 2021 · 1 comment · Fixed by #140
Closed

Updates in the Disk v1beta3 API #131

mauriciopoppe opened this issue May 4, 2021 · 1 comment · Fixed by #140
Assignees

Comments

@mauriciopoppe
Copy link
Member

mauriciopoppe commented May 4, 2021

API Changes:

  • Change all parameters from camelCase to underscores e.g. diskNumber -> disk_number
  • Use disk_number instead of ID, also change from string to int64
  • PartitionDisk: is disk partitioned formatted? (TODO: ask michelle or deep about this one)
  • ListDiskLocation TODO(mauricio): look for a better way to represent DiskLocation fields, it’s a map of Location to string but an empty location should be a nil pointer, check how this should be done in protobuf
  • ListDIskIDsResponse, change diskIDs to disk_ids
  • DiskIDs: change identifiers to map<DiskType, string>, DiskType is a new enum with only one value (page83), instead change it to a message with these known types: page83 and serial_number
  • PartitionDisk add comment how is partitioned: “PartitionDisk initializes and partitions a disk device with the GPT partition style (if the disk has not been partitioned already) and returns the resulting volume device ID“
  • Rescan: (typically part of nodeStageVolume, maybe move it to sytem API)? this is just a suggestion
  • Change DiskStats to GetDiskStat, also change diskSize to total_bytes
  • Change SetAttachState to SetDiskState also change request and response
  • Change GetAttachState to GetDiskState, also change request and response
  • Make sure that the comments match what the api does (all methods)

Also:

pohly added a commit to pohly/csi-proxy that referenced this issue May 5, 2021
1d60e77 Merge pull request kubernetes-csi#131 from pohly/kubernetes-1.20-tag
9f10459 prow.sh: support building Kubernetes for a specific version
fe1f284 Merge pull request kubernetes-csi#121 from kvaps/namespace-check
8fdf0f7 Merge pull request kubernetes-csi#128 from fengzixu/master
1c94220 fix: fix a bug of csi-sanity
a4c41e6 Merge pull request kubernetes-csi#127 from pohly/fix-boilerplate
ece0f50 check namespace for snapshot-controller
dbd8967 verify-boilerplate.sh: fix path to script
9289fd1 Merge pull request kubernetes-csi#125 from sachinkumarsingh092/optional-spelling-boilerplate-checks
ad29307 Make the spelling and boilerplate checks optional
5f06d02 Merge pull request kubernetes-csi#124 from sachinkumarsingh092/fix-spellcheck-boilerplate-tests
48186eb Fix spelling and boilerplate errors
71690af Merge pull request kubernetes-csi#122 from sachinkumarsingh092/include-spellcheck-boilerplate-tests
981be3f Adding spelling and boilerplate checks.
2bb7525 Merge pull request kubernetes-csi#117 from fengzixu/master
3b6d17b Merge pull request kubernetes-csi#118 from pohly/cloud-build-timeout
9318c6c cloud build: double the timeout, now 1 hour
4ab8b15 use the tag to replace commit of csi-test
5d74e45 change the csi-test import path to v4
7dcd0a9 upgrade csi-test to v4.0.2
86ff580 Merge pull request kubernetes-csi#116 from andyzhangx/export-image-name
c3a9662 allow export image name and registry name
c6a88c6 Merge pull request kubernetes-csi#113 from xing-yang/install_snapshot_controller
45ec4c6 Fix the install of snapshot CRDs and controller
5d874cc Merge pull request kubernetes-csi#112 from xing-yang/cleanup
79bbca7 Cleanup
d437673 Merge pull request kubernetes-csi#111 from xing-yang/update_snapshot_v1_rc
57718f8 Update snapshot CRD version

git-subtree-dir: release-tools
git-subtree-split: 1d60e7792624a9938c0bd1b045211fbb89e513d6
@mauriciopoppe
Copy link
Member Author

/assign

@mauriciopoppe mauriciopoppe changed the title Updates in the Disk API Updates in the Disk v1beta3 API May 26, 2021
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 a pull request may close this issue.

1 participant