Commit Graph
3135 Commits
Author SHA1 Message Date
Noah Watkins df20c7da17 ceph: shush device discovery log messages
fixes: #3483

Signed-off-by: Noah Watkins <noahwatkins@gmail.com>
2019-07-31 14:34:24 -07:00
Travis Nielsen 69a6fdf279 Merge pull request #3551 from AllenZMC/patch-1
fix word `ressource` to `resource`
2019-07-31 10:14:28 -06:00
AllenZMC 4616f17a26 fix word ressource to resource
Signed-off-by: czm <zhongming.chang@daocloud.io>
2019-07-31 20:53:33 +08:00
Blaine Gardner 1b6fe840f6 Merge pull request #3460 from umangachapagain/servicemonitor-fix
fix Prometheus permissions and servicemonitor configurations
2019-07-30 11:05:12 -06:00
Blaine Gardner 9f7dbb85d8 Merge pull request #3527 from SUSE/remove-osd-copybins
ceph: osd use copy-binaries top level cmd
2019-07-30 09:41:03 -06:00
Travis Nielsen e5779f79d5 Merge pull request #3533 from ethan-daocloud/patch-1
cleanup: fix log message mis-spelled words
2019-07-29 11:14:37 -06:00
Blaine Gardner 55a726fda9 ceph: osd use copy-binaries top level cmd
Replace usage of the custom osd copybins command with the new top-level
copy-binaries command introduced with cmd-reporter.

Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
2019-07-29 09:23:45 -06:00
Travis Nielsen 0acdf9a4f1 Merge pull request #3532 from zgfh/patch-1
fix misspelling in bug_report.md
2019-07-27 07:25:02 -06:00
Alan aeb47eaf52 fix misspelling in bug_report.md
Signed-off-by: alan <zg.zhu@daocloud.io>
2019-07-27 15:47:17 +08:00
Travis Nielsen d50c9ef3bf Merge pull request #3530 from renekalff/fix-k8s-ownerreferences
Ceph (and EdgeFS?) cluster is being deleted after K8S cluster restart or api/etcd issues
2019-07-26 18:33:16 -06:00
René Kalff 25780878eb Make the ownerreference name match the cluster name referenced
In order to cleanup all cluster resources in case of cluster deletion
all resources have an ownerreference to the cluster crd instance.
Because the name of the reference was incorrectly set to the name of the
namespace the cluster resources can unintentionally be deleted by the
kubernetes garbage collector after a kubernetes cluster anomaly.

Signed-off-by: René Kalff <r.kalff@intermax.nl>
2019-07-26 18:33:38 +02:00
Travis Nielsen 81dde4b381 Merge pull request #3528 from phlogistonjohn/jjm-csi-new-template-changes
ceph csi: add new required tempfs mounts to csi templates
2019-07-26 10:00:43 -06:00
Dmitry Yusupov dde1b9114c Merge pull request #3516 from NullOranje/edgefs-configurable-servicetypes
EdgeFS: Allow control over K8s service type via CRD
2019-07-26 07:58:12 -07:00
ethan 3a6bef33df cleanup: fix log message mis-spelled words
Signed-off-by: ethan <guangming.wang@daocloud.io>
2019-07-26 16:40:48 +08:00
Umanga Chapagain d30c06860d fixes permissions and servicemonitor configurations
this adds the access for rook to create monitoring resources
across namespaces and provides servicemonitor config to
monitor the current service being created

Signed-off-by: Umanga Chapagain <chapagainumanga@gmail.com>
2019-07-26 10:53:18 +05:30
John Mulligan 57c663c0c1 ceph csi: add new required tempfs mounts to csi templates
Ceph CSI now requires a tempfs mountpoint and will not function without
it. These changes are cloned from the ceph csi yaml.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
2019-07-25 16:43:08 -04:00
Nick McKinney d9aec6bad6 Allow control over K8s service type via CRD
Signed-off-by: Nick McKinney <nmckinney@breakpoint-labs.com>
2019-07-25 15:49:18 -04:00
Blaine Gardner 6d2ef93669 Merge pull request #3443 from SUSE/nfs-refactor
Ceph NFS: use new config
2019-07-25 09:39:50 -06:00
Travis Nielsen 1bf5118c7f Merge pull request #3514 from Madhu-1/fix-csi-deployment
update ceph-csi templates
2019-07-24 21:21:52 -06:00
Travis Nielsen 180b3db759 Merge pull request #3509 from travisn/simplify-cluster-status
Simplify CR status updates for Ceph and EdgeFS
2019-07-24 15:48:50 -06:00
Travis Nielsen 91d2f95f34 Merge pull request #3224 from mvollman/ceph-vol-md-fail
ceph: fail when metadataDevice not used by cephvol
2019-07-24 15:27:27 -06:00
travisn 3142c3e5cb simplify CR status updates for ceph and edgefs
The status messages on the CRs had much duplicated code
for logging an error if the status update failed.
Now any failures around the status update are logged
consistently within the updateClusterStatus method
so there is no need to return an error and duplicate
the error handling everywhere.

Signed-off-by: travisn <tnielsen@redhat.com>
2019-07-24 12:52:30 -06:00
Travis Nielsen 35e2943ec8 Merge pull request #3513 from leseb/mgr-uid-cephfs
ceph: mgr fix startup
2019-07-24 12:39:40 -06:00
Madhu Rajanna f95929da16 update csi template
ceph-csi https://github.com/ceph/ceph-csi/pull/457

made deployment template changes and also
removed node publish secrets and added not
stage secrets.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
2019-07-24 22:27:56 +05:30
Sébastien Han e52efdb8d4 ceph: mgr fix startup
Because the manager is running under the 'ceph' user the cephfs
subvolume plugin can not create directories anymore. It fails with a
permission denied because of the usage of the 'ceph' user, root is need
here, hence the special startup line to go over this.

Closes: https://github.com/ceph/ceph-csi/issues/486
Signed-off-by: Sébastien Han <seb@redhat.com>
2019-07-24 17:47:43 +02:00
Sébastien Han 72b9ec3a0c Merge pull request #3507 from travisn/minikube-1.15
Update minikube script to 1.15.1
2019-07-24 07:49:22 +02:00
travisn 24c134de38 start minikube with the default version of kubernetes
There is generally not a need to launch a specific version
of k8s, so just let minikube pick the latest version
to launch.

Signed-off-by: travisn <tnielsen@redhat.com>
2019-07-23 17:49:35 -06:00
Dmitry Yusupov e454b12af3 Merge pull request #3505 from sabbot/EdgeFS-3492
Add mgr pod liveness probe
2019-07-23 12:41:26 -07:00
Anton Skriptsov 3952874ae1 Add mgr pod liveness probe
Signed-off-by: Anton Skriptsov <sabbotagge@gmail.com>
2019-07-23 11:25:19 -07:00
Travis Nielsen 3637f6c0e8 Merge pull request #3506 from travisn/master-integration-tests
Trigger the integration tests for all branches if tags not set
2019-07-23 12:24:04 -06:00
Alexander Trost 93716bcde8 fix usage of Infof in CephClusterController (#3502)
fix usage of Infof in CephClusterController
2019-07-23 19:22:45 +02:00
travisn be8a25e07e trigger the integration tests for all branches if tags not set
The integration tests for storage providers will be skipped
for example if [test ceph] is specified. If the [test] tag
is not specified, all tests should be run. If the tag is not
specified, the Jenkinsfile will pass null, which needs to be
handled by the test code deciding to run all the tests.

Signed-off-by: travisn <tnielsen@redhat.com>
2019-07-23 10:53:22 -06:00
Sébastien Han 38240a77f0 Merge pull request #3498 from leseb/oc-privileged-pod
ceph: run pod with privileged
2019-07-23 17:57:07 +02:00
Alexander Trost 07fbea0fd9 fix usage of Infof in CephClusterController
Currently the ClusterController logs out a message like:
```
Cluster %s is not ready. Skipping orchestration.rook-ceph
```
without the "placeholders" fullfilled. This is because the code uses the
logger.Info() instead of logger.Infof() function.
This change fixes that to use Infof and therefore corrects the log
message.

Signed-off-by: Alexander Trost <galexrt@googlemail.com>
2019-07-23 16:08:16 +02:00
Michael Vollman bd58df9d3b ceph: fail when metadataDevice not used by cephvol
fail when metadataDevice is specified but ceph-volume batch does not
automatically use the metadataDevice as the block.db. (#3092)

Signed-off-by: Michael Vollman <michael.b.vollman@gmail.com>
2019-07-23 07:23:51 -04:00
Sébastien Han fdaf3bfc9c ceph: run pod with privileged
Before each pod starts, the daemon changes the ownership of
/var/log/ceph which is bindmounted on a hostPath.

On OpenShift envionments, the container must be privileged in order to
successfully change the owernship.

Closes: https://github.com/rook/rook/issues/3485
Signed-off-by: Sébastien Han <seb@redhat.com>
2019-07-23 07:43:10 +02:00
Travis Nielsen eb5fffa6ba Merge pull request #3494 from leseb/ci-run
ci: do not skip if null
2019-07-22 16:21:54 -06:00
Travis Nielsen 36479fec13 Merge pull request #3489 from leseb/switch-to-14.2.2
ceph: use latest 14.2.2 code
2019-07-22 15:15:10 -06:00
Sébastien Han ea1da23bf3 ci: do not skip if null
If no storage provider is found, let's assign env.testProvider with an
empty string so that we wil run all the tests.

Signed-off-by: Sébastien Han <seb@redhat.com>
2019-07-22 21:53:44 +02:00
Dmitry Yusupov dc85c5083c Merge pull request #3482 from dyusupov/master
make getModifiedRookImagePath be generic and use it for isgw too
2019-07-22 11:51:44 -07:00
Dmitry Yusupov a3d02fb6ea make getModifiedRookImagePath be generic and use it for isgw too
Signed-off-by: Dmitry Yusupov <dmitry.yusupov@nexenta.com>
2019-07-22 10:07:22 -07:00
Blaine Gardner 50e824ebad Ceph: Add unit tests for nfs-ganesha spec
Ganesha isn't a true "Ceph" daemon, so some of the sharable pod spec
testing functionality is teased out of the operator/ceph/test library
and a simple operator/test library is created. The ceph/test library is
updated to use the operator/test library when possible/appropriate.

Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
2019-07-22 11:06:04 -06:00
Blaine Gardner e75254f412 ceph: add missing DaemonDataPathMap unit test
Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
2019-07-22 11:06:04 -06:00
Blaine Gardner aa0d3346f2 ceph nfs: refactor to use new config pattern
Refactor the Ceph NFS (Ganesha) daemon to use the new config pattern.

Use of a custom Rook binary to run NFS has been removed in favor of
instead running the required DBUS process as a sidecar. The effect here
is that the NFS Ganesha container itself may fail and restart a few
times before the DBUS sidecar is running.

Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
2019-07-22 11:06:04 -06:00
Blaine Gardner d774fec858 Merge pull request #3470 from rohantmp/fixTopologyAware
Fix: topologyAware does not pick up failure domains.
2019-07-22 10:19:28 -06:00
Travis Nielsen e176dd1001 Merge pull request #3487 from leseb/oc-privileged
ceph: mon: init chown for openshift
2019-07-22 08:14:19 -06:00
Sébastien Han 41d2953e7e ceph: use latest 14.2.2 code
14.2.2 is out and along with it numerous fixes so let's use it.

Signed-off-by: Sébastien Han <seb@redhat.com>
2019-07-22 15:58:25 +02:00
Sébastien Han e83eb5f9e7 Merge pull request #3488 from xieyanker/master
Correct typo about skipVolumeForDirectory's code comment
2019-07-22 13:23:33 +02:00
xieyanker 7672582e0a Correct typo about skipVolumeForDirectory's code comment
Signed-off-by: xieyanker <xjsisnice@gmail.com>
2019-07-22 19:05:14 +08:00
Sébastien Han c2b85ba33c ceph: mon: init chown for openshift
OpenShift environment requires privileged containers when interacting
with the host filesystem, typically via dataDirHostPath.
The init container that changes the ownership of dataDirHostPath was
missing the privileged flag and users were getting permission denied
errors on the chown command.

Fixes: https://github.com/rook/rook/issues/3485
Signed-off-by: Sébastien Han <seb@redhat.com>
2019-07-22 11:07:29 +02:00