Add fibre channel volume explanation #3967
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
k8s-ci-robot
added
the
cncf-cla: yes
rootfs
reviewed
Jun 2, 2017
docs/concepts/storage/volumes.md
Outdated
| @@ -74,6 +74,7 @@ Kubernetes supports several types of Volumes: | |||
| * `awsElasticBlockStore` | |||
| * `nfs` | |||
| * `iscsi` | |||
| * `fc(fibre_channel)` | |||
rootfs
reviewed
Jun 2, 2017
docs/concepts/storage/volumes.md
Outdated
| targetWWNs in your volume configuration. If multiple WWNs are specified, | ||
| targetWWNs expects that those WWNs form multipath connection. | ||
|
|
||
| __Important: You must have your own FC Storage setup running with the volume |
There was a problem hiding this comment.
FC SAN Zone manager must allocate and mask those LUNs to the target WWNs so Kubernetes hosts can access them.
|
/assign |
|
Thanks. |
chenopis
suggested changes
Jun 2, 2017
docs/concepts/storage/volumes.md
Outdated
| @@ -74,6 +74,7 @@ Kubernetes supports several types of Volumes: | |||
| * `awsElasticBlockStore` | |||
| * `nfs` | |||
| * `iscsi` | |||
| * `fc(fibre channel)` | |||
There was a problem hiding this comment.
add a space between 'fc' and '(fibre channel)'
docs/concepts/storage/volumes.md
Outdated
| targetWWNs expects that those WWNs form multipath connection. | ||
|
|
||
| __Important: You must configure FC SAN Zoning to allocate and mask those | ||
| LUNs(volumues) to the target WWNs beforehand so that Kubernetes hosts |
There was a problem hiding this comment.
- add space after "LUNs"
- "volumues" => "volumes"
This patch adds explanation of fibre channel volume including usage of multiple WWNs. Fix kubernetes#33602
chenopis
suggested changes
Jun 2, 2017
docs/concepts/storage/volumes.md
Outdated
| @@ -320,6 +321,19 @@ simultaneous writers allowed. | |||
|
|
|||
| See the [iSCSI example](https://github.com/kubernetes/kubernetes/tree/{{page.githubbranch}}/examples/volumes/iscsi) for more details. | |||
|
|
|||
| ### fc(fibre channel) | |||
There was a problem hiding this comment.
add a space between "fc" and "(fibre channel)"
docs/concepts/storage/volumes.md
Outdated
| @@ -320,6 +321,19 @@ simultaneous writers allowed. | |||
|
|
|||
| See the [iSCSI example](https://github.com/kubernetes/kubernetes/tree/{{page.githubbranch}}/examples/volumes/iscsi) for more details. | |||
|
|
|||
| ### fc(fibre channel) | |||
|
|
|||
| A `fc` volume allows an existing fibre channel volume to be mounted into your pod. | |||
There was a problem hiding this comment.
An fc? What is this rule?
A glusterfs
A flocker
An rbd
There was a problem hiding this comment.
The rule is to use "an" for words that sound like they start with a vowel even if the first letter is a consonant. In this case, since 'fc' is an acronym, it would be pronounced EFF-CEE. Consequently, the leading sound is a vowel, so "an" should be used.
chenopis
approved these changes
Jun 2, 2017
|
Admittedly, the rules of English are confusing. =) |
|
Now that rule is make sense to me :) Thanks. |
chenopis
added a commit
that referenced
this pull request
Jun 5, 2017
…hub.io into release-1.7 * 'master' of https://github.com/kubernetes/kubernetes.github.io: Remove self-referential link Add fibre channel volume explanation (#3967)
chenopis
added a commit
to iphutch/kubernetes.github.io
that referenced
this pull request
Jun 26, 2017
…hub.io * 'master' of https://github.com/kubernetes/kubernetes.github.io: (859 commits) fixed the line giving the template for creating configmap, should include the word configmap as part of the creation process. Fixes issue kubernetes#3939 Add CRI runtime support Updaated install-kubeadm.md, removed redundant kubectl installation (kubernetes#4006) Add Chocolatey Package Manager for Windows (kubernetes#4000) Make issue title automatically a link Update directions for macOS bash completion Add TCP socket health check example (kubernetes#3977) Update networking.md (kubernetes#3950) [authorization] fix invalid href for webhook Remove self-referential link Add fibre channel volume explanation (kubernetes#3967) Improve RBAC doc (kubernetes#3951) Clarify CNI acronym Fix 404 to kubectl install link install-kubectl.md: Fix Windows texts, link to latest macOS binaries Create Installing kubeadm topic. (kubernetes#3709) Fix example in the "Adding ConfigMap data to a Volume" section Update ExternalName example Completes PR kubernetes#3325 ... # Conflicts: # _data/tasks.yml # docs/tasks/administer-cluster/namespaces-walkthrough.md # docs/tasks/administer-cluster/namespaces.md
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch adds explanation of fibre channel volume including usage of multiple WWNs.
Fix #33602
kubernetes/kubernetes#33602
This change is