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

Adds the cifs-common package #45117

Merged
merged 1 commit into from Apr 29, 2017

Conversation

Projects
None yet
7 participants
@lazypower
Member

lazypower commented Apr 28, 2017

What this PR does / why we need it: Enables mounting of CIFS volumes. Required for Azure.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes juju-solutions/bundle-canonical-kubernetes#227

Release note:

Added CIFS PV support for Juju Charms
Adds the cifs-common package
Enable mounting of filesystems over the CIFS protocol. Closes
juju-solutions/bundle-canonical-kubernetes#227
@k8s-reviewable

This comment has been minimized.

Show comment
Hide comment
@k8s-reviewable

k8s-reviewable Apr 28, 2017

This change is Reviewable

k8s-reviewable commented Apr 28, 2017

This change is Reviewable

@lazypower

This comment has been minimized.

Show comment
Hide comment
@lazypower
Member

lazypower commented Apr 28, 2017

/assign @wwwtyro

@k8s-ci-robot

This comment has been minimized.

Show comment
Hide comment
@k8s-ci-robot

k8s-ci-robot Apr 28, 2017

Contributor

@chuckbutler: GitHub didn't allow me to assign the following users: wwwtyro.

Note that only kubernetes members can be assigned..

In response to this:

/assign @wwwtyro

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Contributor

k8s-ci-robot commented Apr 28, 2017

@chuckbutler: GitHub didn't allow me to assign the following users: wwwtyro.

Note that only kubernetes members can be assigned..

In response to this:

/assign @wwwtyro

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@lazypower

This comment has been minimized.

Show comment
Hide comment
@lazypower
Member

lazypower commented Apr 28, 2017

/assign @marcoceppi

@marcoceppi

This comment has been minimized.

Show comment
Hide comment
@marcoceppi

marcoceppi Apr 28, 2017

Member

/lgtm

Member

marcoceppi commented Apr 28, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm label Apr 28, 2017

@k8s-merge-robot

This comment has been minimized.

Show comment
Hide comment
@k8s-merge-robot

k8s-merge-robot Apr 28, 2017

Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chuckbutler, marcoceppi

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

Contributor

k8s-merge-robot commented Apr 28, 2017

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chuckbutler, marcoceppi

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-merge-robot

This comment has been minimized.

Show comment
Hide comment
@k8s-merge-robot

k8s-merge-robot Apr 29, 2017

Contributor

Automatic merge from submit-queue (batch tested with PRs 41583, 45117, 45123)

Contributor

k8s-merge-robot commented Apr 29, 2017

Automatic merge from submit-queue (batch tested with PRs 41583, 45117, 45123)

@k8s-merge-robot k8s-merge-robot merged commit 02f9d60 into kubernetes:master Apr 29, 2017

1 of 9 checks passed

Jenkins Bazel Build Job triggered.
Details
Jenkins GCE Node e2e Jenkins job triggered.
Details
Jenkins GCE etcd3 e2e Jenkins job triggered.
Details
Jenkins Kubemark GCE e2e Jenkins job triggered.
Details
Jenkins kops AWS e2e Jenkins job triggered.
Details
Jenkins unit/integration Jenkins job triggered.
Details
Jenkins verification Jenkins job triggered.
Details
Submit Queue Required Github CI test is not green: Jenkins Bazel Build
Details
cla/linuxfoundation chuckbutler authorized
Details

@lazypower lazypower deleted the lazypower:add-cifs branch May 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment