Commit Graph
2020 Commits
Author SHA1 Message Date
Travis Nielsen b6acc6d414 Merge pull request #2127 from travisn/cleanup-integration-test
Make integration tests more resilient to crd deletion failure
2018-09-17 10:11:07 -06:00
travisn 1266c63135 tests: ensure previous crds are deleted before continuing with tests
Signed-off-by: travisn <tnielsen@redhat.com>
2018-09-17 08:10:14 -06:00
Travis Nielsen bcb71378d6 Merge pull request #1995 from rtnpro/support-storageclass-reclaimpolicy
Support storageclass reclaimpolicy
2018-09-14 11:02:47 -06:00
Travis Nielsen ae4e3369ca Merge pull request #2111 from rootfs/relist-cm
reset resource version if watch channel closed
2018-09-14 08:25:39 -06:00
Ratnadeep Debnath ed089777bd Added unit tests for ceph provisioner to assert if reclaimPolicy is set properly.
Signed-off-by: Ratnadeep Debnath <rtnpro@gmail.com>
2018-09-14 10:05:36 +05:30
Ratnadeep Debnath 7ae2149961 Add support for storageclass' reclaimPolicy when creating PV.
Signed-off-by: Ratnadeep Debnath <rtnpro@gmail.com>
2018-09-14 10:05:34 +05:30
Bassam Tabbara 03e5242740 Merge pull request #2090 from ganeshmaharaj/go1.11
golang 1.11 support
2018-09-13 10:53:40 -07:00
Huamin Chen 1c225185b6 adjust cluster and osd provision timeout
Signed-off-by: Huamin Chen <hchen@redhat.com>
2018-09-13 11:21:11 -04:00
Alexander Trost 6d50c739e1 Merge pull request #2126 from ayaz/master
GKE uses a non-standard FlexVolume directory which is not documented.
2018-09-13 11:53:40 +02:00
Ayaz Ahmed Khan a3087f6704 Update FlexVolume documentation to add GKE info.
Google Kubernetes Engine uses a non-standard FlexVolume directory which
is not documented. This change updates the FlexVolume documentation to
describe how to configure FlexVolume for use on Google Kubernetes
Engine.

Signed-off-by: Ayaz Ahmed Khan <ayaz@ayaz.pk>
2018-09-13 14:20:43 +05:00
Bassam Tabbara 98cefc66de use a consistent version of gofmt
Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
2018-09-12 10:53:53 -07:00
Ganesh Maharaj Mahalingam 92f91ab156 golang 1.11 support
Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
2018-09-12 10:53:53 -07:00
Alexander Trost c1da6999bf Merge pull request #2060 from galexrt/fix_981
build: Only install helm if needed and use a temp path
2018-09-12 19:22:19 +02:00
Jared Watts 662095d8f3 Merge pull request #2108 from ganeshmaharaj/pvc-create-resilience
PVC create resiliency
2018-09-12 08:44:04 -07:00
Bassam Tabbara c72c44feb0 Merge pull request #2115 from arm64b/nfs-cross-build-fix-on-arm64
Fix Rook NFS image cross-build issue on AArch64
2018-09-12 07:14:32 -07:00
Alexander Trost 7d167d9b92 Merge pull request #2116 from arm64b/bump-golang-version-to-1.11
Add golang 1.11 version support
2018-09-12 13:46:33 +02:00
Dennis Chen 1c2de4b55d Add golang 1.11 version support
Now the golang version 1.1 has been released by the official
website[1], so let's add support for this version, else we'll
get below build error when using golang 1.11:

```
build/makelib/golang.mk:103: *** unsupported go version. Please
make install one of the following supported version: '1.7|1.8|1.9|1.10'.  Stop.
```

[1] https://golang.org/dl/

Signed-off-by: Dennis Chen <dennis.chen@arm.com>
2018-09-12 17:55:02 +08:00
Dennis Chen e73fae7cd4 Fix Rook NFS image cross-build issue on AArch64
We need to pull the base image for arm64, not for amd64.
This PR add prefix `arm64v8` of `fedora:26` to pull
the image for arm64.

Signed-off-by: Dennis Chen <dennis.chen@arm.com>
2018-09-12 14:01:50 +08:00
Travis Nielsen 0cab31e250 Merge pull request #2103 from mjlodge/patch-1
Fix typo in adopters.md
2018-09-11 13:52:23 -06:00
Mathew Lodge 284687222b Fix typo in adopters.md
"On-premises" is what was meant.

Signed-off-by: mjlodge <ml@lodgephoto.com>
2018-09-11 14:48:48 -05:00
Huamin Chen 9ec37d5c00 reset resource version if watch channel closed
Signed-off-by: Huamin Chen <hchen@redhat.com>
2018-09-11 13:27:26 -04:00
Alexander Trost 5d8d2b5b5a build: Only install helm if needed and use a temp path
Resolves #981

Signed-off-by: Alexander Trost <galexrt@googlemail.com>
2018-09-11 14:21:48 +02:00
Ganesh Maharaj Mahalingam b8c7e151bc use return error code instead
Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
2018-09-08 15:23:28 -07:00
Ganesh Maharaj Mahalingam ff9230a10f PVC create resiliency
While creating a PVC, if the operator dies between successfully creating a rbd
image and associating it with a PV, that PVC will never be completed. A
restarted operator will also fail to complete the task as the requested rbd
image already exists. With the assumption that a PVC UID will always be unique,
this patch will try to use the existing volume and complete the PVC request.

Resolves: #2072

Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
2018-09-07 15:34:59 -07:00
Travis Nielsen c3b4f2d266 Merge pull request #2106 from travisn/test-retry-crd
Integration tests to retry creating the crd if already exists
2018-09-07 16:24:48 -06:00
travisn 4aa0c3b258 tests: retry creating the crd if already exists
Signed-off-by: travisn <tnielsen@redhat.com>
2018-09-07 12:49:07 -06:00
Travis Nielsen 558d51b9cc Merge pull request #2104 from rohan47/nfs_doc_update
[Documentation] Updated NFS prerequisite
2018-09-07 08:05:45 -06:00
Jared Watts d751dca2ee Merge pull request #2085 from arm64b/bump-fedora-to-multi-arch
Fix rook nfs docker image build failure on AArch64
2018-09-06 23:43:21 -07:00
Alexander Trost 9f90d72e05 Merge pull request #2035 from galexrt/update_ns_fix_some_stuff
Added missing RBAC in quickstart Cluster definition
2018-09-07 08:20:23 +02:00
rohan47 39c6bd0884 [Documentation] Updated prequisite
It is required to install nfs client packages on the nodes where the nfs
server will run.

Signed-off-by: rohan47 <rohanrgupta1996@gmail.com>
2018-09-07 06:13:25 +00:00
Travis Nielsen 7a3dde9df3 Merge pull request #2097 from ganeshmaharaj/jenkins-system-logging
Enable CI system logs in artifacts
2018-09-06 22:55:34 -06:00
Ganesh Maharaj Mahalingam 099380ba61 Enable CI system logs in artifacts and fixup log names
This is a temporary patch to help us figure out what is causing the random
issues on concurrent builds and re-using an instance.

This patch also makes the log names concise and consistent across all versions.

Signed-off-by: Ganesh Maharaj Mahalingam <ganesh.mahalingam@intel.com>
2018-09-06 20:42:24 -07:00
Travis Nielsen 94a7b4ebda Merge pull request #2096 from travisn/translate-mon-id
Refactor mon naming for use by other daemons
2018-09-06 15:53:48 -06:00
travisn 6a335ebe8a mon: refactor mon naming for use by other daemons
Signed-off-by: travisn <tnielsen@redhat.com>
2018-09-06 11:46:45 -06:00
Travis Nielsen 2770a36a0a Merge pull request #2094 from SUSE/ceph-refactor-daemon-includes
Ceph: refactor daemon includes to <dmn>daemon for grokkability
2018-09-06 11:37:34 -06:00
Blaine Gardner 4be911c5b1 Ceph: include daemons as <dmn>daemon
To help with code readability, include daemons from `pkg/daemon/ceph`
as `<dmn>daemon "github.com/rook/rook/pkg/daemon/ceph/<dmn>`. This will
make it easier to understand code lines without needing to scroll up to
check the includes, as there are mon/mds/mgr/osd/rgw packages for
daemons as well as operator and cmd.

Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
2018-09-06 09:40:00 -06:00
Alexander Trost b83595e5cb Added missing RBAC in quickstart Cluster definition
Corrected the cluster namespace in some examples to rook-ceph

Updated PodSecurityPolicy hostNetwork ports

Signed-off-by: Alexander Trost <galexrt@googlemail.com>
2018-09-06 13:27:30 +02:00
Travis Nielsen 857dc8ac95 Merge pull request #2098 from travisn/stable-upgrade-test
Retry file operations after upgrade in integration test
2018-09-05 23:51:09 -06:00
travisn 63f8f9c216 tests: retry file operations after upgrade
Signed-off-by: travisn <tnielsen@redhat.com>
2018-09-05 22:10:46 -06:00
Travis Nielsen 02accb5d62 Merge pull request #2081 from SUSE/ceph-refactor-mon-config
Ceph: Refactor daemon/mon module's config and keyring generation into cephconfig module
2018-09-05 22:03:06 -06:00
Blaine Gardner 3731f06176 Ceph: Refactor mon config/keyring to cephconfig
Create a cephconfig module in Ceph's daemon pkg source, and refactor the
config and keyring generation that exists in the mon package into the
new cephconfig package. The config/keyring generation code is used by
most all daemons and not just mon, so a new package is a more
appropriate place for this.

Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
2018-09-05 10:17:11 -06:00
Travis Nielsen 1eb11fea7b Merge pull request #2014 from travisn/design-multiple-ceph-version
Design doc for decoupling the Ceph version
2018-09-05 09:22:32 -06:00
travisn 1bf35b806b design: decoupling the Ceph version from the Rook version
Signed-off-by: travisn <tnielsen@redhat.com>
2018-09-05 08:13:56 -06:00
Alexander Trost 408ef6315a Merge pull request #2086 from arm64b/alignment-size-of-the-PV
Align the size of the PV to the actual one
2018-09-05 14:36:01 +02:00
Travis Nielsen 45102923a9 Merge pull request #2077 from liewegas/kill-osd-id
Remove unnecessary "-id" from OSD pod names
2018-09-04 17:13:20 -06:00
travisn c0af417b00 osd: delete legacy deployment with -id in the name
Signed-off-by: travisn <tnielsen@redhat.com>
2018-09-04 14:15:18 -05:00
Sage Weil 8e773612ca Remove unnecessary "-id" from OSD pod names
The -id portion of the name is unnecessary and inconsistent with the other
pod names.

Signed-off-by: Sage Weil <sage@redhat.com>
2018-09-04 14:15:18 -05:00
Travis Nielsen 089366effe Merge pull request #2018 from travisn/rename-ceph-mon
Start mons with deployments instead of replicasets and rename the mons
2018-09-04 12:20:42 -06:00
Dennis Chen 7dce5c004f Align the size of the PV to the actual one
We maybe roundup the size to the next `MB` boundary in some cases
when creating the volume, thus we need to make sure the size of
the PV generated is consistent with the actual one.

Signed-off-by: Dennis Chen <dennis.chen@arm.com>
2018-09-04 16:27:19 +08:00
Dennis Chen aab0f043d6 Fix rook nfs docker image build failure on AArch64
The original base image `fedora:25` doesn't support multi-arch,
so we get below error message with `docker pull fedora:25`:
```
25: Pulling from library/fedora
no matching manifest for unknown in the manifest list entries
```

This PR bumps the fedora base image to `:26` which is multi-arch
version, to address the build issue.

Signed-off-by: Dennis Chen <dennis.chen@arm.com>
2018-09-04 10:56:12 +08:00