Update 1.8 release notes for sig-storage #433

saad-ali · 2017-09-18T18:44:02Z

CC /sig-storage for review

childsb · 2017-09-18T19:06:04Z

lgtm

radhikapc · 2017-09-18T19:41:45Z

release-1.8/release_notes_draft.md

+began to extend the Kubernetes storage API to do more then just make volumes available
+(for example, resize volumes and take volume Snapshots), and focused on giving users
+more control over their storage (ability to set requests & limits on ephemeral storage,
+ability to specify mount options, more metrics, and improvments to Flex driver deployments).


For the 1.8 release, SIG Storage extends the Kubernetes storage API to provide users with more control over their storage, beyond resizing volumes and taking volume Snapshots. The storage API provides the ability to set requests and limits on ephemeral storage, and specify mount options and metrics. Additionally, the API delivers improvement to the Flex driver deployments.

Wording sounds better, but isn't accurate (new features include resizing and snapshots, so we're not going beyond those). I'll reworded it based on your structure.

radhikapc · 2017-09-18T19:48:48Z

release-1.8/release_notes_draft.md

-* PV spec refactoring for plugins that reference namespaced resources: Azure File, CephFS, iSCSI, Glusterfs
+* [stable] Mount Options
+  * The ability to speciy mount options has been promoted to stable.
+  * A new `MountOptions` field in the `PersistentVolume` spec can be used to specify mount options (instead of an annotation).


Introduce a new MountOptions field in the PersistentVolume spec to specify mount options (instead of an annotation).

Also a MountOptions field has been added to StorageClasses to enable k8s admins to control mount options being used in their clusters

radhikapc · 2017-09-18T19:50:39Z

release-1.8/release_notes_draft.md

+* [stable] Expose Storage Usage Metrics
+  * Expose how much available capacity a given PV has through the Kubernetes metrics API.
+* [stable] Volume Plugin Metrics
+  * Expose success and latency metrics for all Kubernetes mount/unmount/attach/detach/provision/delete calls through the Kubernetes metrics API.


Expose success and latency metrics for all the Kubernetes ..

radhikapc · 2017-09-18T19:52:59Z

release-1.8/release_notes_draft.md

+* [stable] Modify PV spec for Azure File, CephFS, iSCSI, Glusterfs to allow referencing namespaced resources.
+* [beta] Reclaim policy in StorageClass
+  * Allow configuration of reclaim policy in StorageClass, instead of always defaulting to `delete` for dynamically provisioned volumes.
+* [alpha] Resizing of Volumes


Volume resizing

radhikapc · 2017-09-18T19:53:12Z

release-1.8/release_notes_draft.md

+* [beta] Reclaim policy in StorageClass
+  * Allow configuration of reclaim policy in StorageClass, instead of always defaulting to `delete` for dynamically provisioned volumes.
+* [alpha] Resizing of Volumes
+  * Enable increasing the size of Volumes through the Kubernetes API.


volumes? (... or "a volume")

"a volume" makes more sense

radhikapc · 2017-09-18T19:55:34Z

release-1.8/release_notes_draft.md

+  * Enable increasing the size of Volumes through the Kubernetes API.
+  * For alpha, this feature only increases the size of the underlying volume and does not do filesystem resizing.
+  * For alpha, this feature is only implmented for Gluster volumes.
+* [alpha] Capacity Isolation/Resource Management for Local Ephemeral Storage:


Provide Capacity...

radhikapc · 2017-09-18T20:00:42Z

release-1.8/release_notes_draft.md

+  * For alpha, this feature only increases the size of the underlying volume and does not do filesystem resizing.
+  * For alpha, this feature is only implmented for Gluster volumes.
+* [alpha] Capacity Isolation/Resource Management for Local Ephemeral Storage:
+  * Introduce ability to set container requests/limits and node Alloctable reservations for new `ephemeral-storage` resource.


Introduce ability to set container requests and limits, and node allocatable reservations for the new ephemeral-storage resource.

radhikapc · 2017-09-18T20:02:07Z

release-1.8/release_notes_draft.md

+  * Introduce new `VolumeMount.Propagation` field for `VolumeMount` in pod containers.
+  * This field may be set to `Bidirectional` to enable a particular mount for a container to be propagated from the container to the host or other containers.
+* [alpha] Improve Flexvolume Deployment
+  * Make it easier to deploy Flex volume drivers:


Provide easy deployment of Flex volume drivers:

Provide sounds weird in this context. Reworded to Simplify Flex volume driver deployment.

obnoxxx · 2017-09-18T20:54:32Z

release-1.8/release_notes_draft.md

-* Expose Storage Usage Metrics
-* PV spec refactoring for plugins that reference namespaced resources: Azure File, CephFS, iSCSI, Glusterfs
+* [stable] Mount Options
+  * The ability to speciy mount options has been promoted to stable.


typo: speciy ->specify

Provides the ability to... doesn't make sense with has been promoted to stable, which is the salient point. Reworded to emphasis that.

that's a mistake. @saad-ali

obnoxxx · 2017-09-18T20:57:22Z

release-1.8/release_notes_draft.md

+    * Provide a sample DaemonSet that can be used to deploy Flexvolume drivers.
+* [prototype] Volume Snapshots
+  * Enable triggering a volume snapshoting through the Kubernetes API.
+  * The prototype does not support quiescing before snapshot, so snapshots maybe inconssistent.


typo: inconsisstent -> inconsistent

obnoxxx · 2017-09-18T20:59:30Z

release-1.8/release_notes_draft.md


+[SIG Storage](https://github.com/kubernetes/community/tree/master/sig-storage) is
+responsible for storage and volume plugin components. For the 1.8 release, SIG Storage
+began to extend the Kubernetes storage API to do more then just make volumes available


then -> than

obnoxxx · 2017-09-18T21:01:00Z

release-1.8/release_notes_draft.md

+* [beta] Reclaim policy in StorageClass
+  * Allow configuration of reclaim policy in StorageClass, instead of always defaulting to `delete` for dynamically provisioned volumes.
+* [alpha] Resizing of Volumes
+  * Enable increasing the size of Volumes through the Kubernetes API.


volumes? (... or "a volume")

obnoxxx · 2017-09-18T21:03:05Z

release-1.8/release_notes_draft.md

+    * Automatically discover and initialize new driver files instead of requiring kubelet/controller-manager restart.
+    * Provide a sample DaemonSet that can be used to deploy Flexvolume drivers.
+* [prototype] Volume Snapshots
+  * Enable triggering a volume snapshoting through the Kubernetes API.


maybe " Enable triggering the creation of a volume snapshot through .." ?

Much nicer. Done.

obnoxxx · 2017-09-18T21:06:58Z

release-1.8/release_notes_draft.md

+  * Allow configuration of reclaim policy in StorageClass, instead of always defaulting to `delete` for dynamically provisioned volumes.
+* [alpha] Resizing of Volumes
+  * Enable increasing the size of Volumes through the Kubernetes API.
+  * For alpha, this feature only increases the size of the underlying volume and does not do filesystem resizing.


Out of curiosity, shouldn't it be transparent to kube, and up to the backend storage implementation what they do with a volume resize / expand request? (In the sense of: the backend storage implementation may just do the filesystem resizing for you if required, but it may not even have a classical filesystem underneath at all and may do something entirely different...) I.e. the notion of filesystem and filesystem resize seems misplaced at the kube-layer.

Unfortunately, for most volume providers resizing is a 2 step process. One that happens controller-manager's control plan and another that involves file system resizing on kubelet. kubernetes/community#657 may be right place to discuss it

@gnufied thanks!

Kubernetes exposes volumes as file or block. For filesystem volumes, in order for an end user to actually make use of the newly available extra space, a filesystem resize is required. If volume resizing is to be part of the k8s storage API, then we need to make sure we automate this as well.

saad-ali · 2017-09-18T21:46:13Z

Feedback addressed. PTAL

obnoxxx · 2017-09-18T21:49:30Z

@saad-ali thx, LGTM

radhikapc · 2017-09-18T21:55:13Z

release-1.8/release_notes_draft.md

+[SIG Storage](https://github.com/kubernetes/community/tree/master/sig-storage) is
+responsible for storage and volume plugin components. For the 1.8 release, SIG Storage extends
+the Kubernetes storage API beyond just making volumes available to enabling volume resizing
+and snapshotting. Beyond these alpha/prototype features, the SIG, focused on providing users


For the 1.8 release, SIG Storage extends the Kubernetes storage API beyond enabling volumes for resizing and snapshotting.

@radhikapc this doesn't read correctly to me. This release is not extending beyond enabling volumes for resizing and snapshotting.

Spoke with @radhikapc offline. We are not extending the api beyond enabling volume resizing. Volume resizing and snapshots are the new features that we are growing into. Meaning while making volumes available == enable volumes, to enabling volume resizing and snapshotting is the new feature set and can't be collapsed into the making volumes available or enable volumes.

So to be clear, we are saying previously the storage API only made volumes available to use, now with 1.8, it is growing to accommodate things like snapshotting and resizing.

radhikapc · 2017-09-18T21:56:27Z

release-1.8/release_notes_draft.md

+    * Provide a sample DaemonSet that can be used to deploy Flexvolume drivers.
+* [prototype] Volume Snapshots
+  * Enable triggering the creation of a volume snapshot through the Kubernetes API.
+  * The prototype does not support quiescing before snapshot, so snapshots maybe inconsistent.


..might be inconsistent

radhikapc

two minor comments. else lgtm

mtanino · 2017-09-19T00:56:53Z

@saad-ali @childsb

Do we need to add these two API support?

iSCSI volume plugin: iSCSI initiatorname support #48789
iSCSI volume plugin: iSCSI initiatorname support kubernetes#48789
FC volume plugin: Support WWID for volume identifier #48741
FC volume plugin: Support WWID for volume identifier kubernetes#48741

saad-ali · 2017-09-19T18:43:48Z

@mtanino @radhikapc feedback addressed. PTAL

radhikapc

Thank you. lgtm.

mtanino · 2017-09-19T18:45:52Z

@saad-ali
Thanks, I confirmed the update.

Update 1.8 release notes for sig-storage

b0b80c8

saad-ali assigned childsb Sep 18, 2017

k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Sep 18, 2017

Merge branch 'master' into release-notes-storage-1.8

65541d5

radhikapc suggested changes Sep 18, 2017

View reviewed changes

obnoxxx suggested changes Sep 18, 2017

View reviewed changes

obnoxxx reviewed Sep 18, 2017

View reviewed changes

Improve storage SIG release note wording

746b7da

radhikapc reviewed Sep 18, 2017

View reviewed changes

radhikapc suggested changes Sep 18, 2017

View reviewed changes

saad-ali added 3 commits September 19, 2017 11:37

Merge branch 'master' into release-notes-storage-1.8

ce2eae8

Reword and add a couple new storage features.

cb9c982

Put sig storage description on separate line

bf47bcc

radhikapc approved these changes Sep 19, 2017

View reviewed changes

radhikapc merged commit 6c71d27 into master Sep 19, 2017

idvoretskyi deleted the release-notes-storage-1.8 branch January 4, 2018 23:49

Update 1.8 release notes for sig-storage #433

Update 1.8 release notes for sig-storage #433

Uh oh!

Conversation

saad-ali commented Sep 18, 2017

Uh oh!

childsb commented Sep 18, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

saad-ali Sep 18, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

radhikapc Sep 18, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

radhikapc Sep 18, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

saad-ali commented Sep 18, 2017

Uh oh!

obnoxxx commented Sep 18, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

saad-ali Sep 18, 2017 •

edited

Loading

radhikapc Sep 18, 2017 •

edited

Loading

radhikapc Sep 18, 2017 •

edited

Loading

saad-ali Sep 19, 2017 •

edited

Loading

mtanino commented Sep 19, 2017 •

edited

Loading