Commit Graph
6658 Commits
Author SHA1 Message Date
Sébastien Han 89f4c4a81a ci: always run golangci-lint
This action wasn't not running on release branch. Now enabling it.

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-12 17:37:59 +02:00
Sébastien Han cefa7de4ed Merge pull request #7591 from leseb/mergify-fix-auto-merge-rule-master
bot: fix mergify auto-merge rule
2021-04-12 17:24:34 +02:00
Sébastien Han 63b2b0da3d bot: fix mergify auto-merge rule
We need to pass the complete name of the action.

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-12 17:08:48 +02:00
Travis Nielsen 06c25746fe Merge pull request #7463 from umangachapagain/label-monitoring
ceph: add labels to monitoring spec
2021-04-12 08:56:41 -06:00
Sébastien Han ad60c41d8b Merge pull request #7579 from leseb/cancel-canary-mons
ceph: cancel canary retry if necessary
2021-04-12 16:19:49 +02:00
Sébastien Han 050d61c49e Merge pull request #7580 from alrs/ceph-object-imports
ceph: fix multiple imports
2021-04-12 15:54:44 +02:00
Sébastien Han fc1e824a8d ceph: cancel canary retry if necessary
If the cluster is deleted and the mon canaries cannot be scheduled for
any reasons, let's not wait and return.

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-12 15:46:58 +02:00
Travis Nielsen 6c863db516 Merge pull request #7585 from leseb/auth-minikube
ci: use authenticated requests for minikube action on suite
2021-04-12 07:45:49 -06:00
Sébastien Han 15901277d5 ci: use authenticated requests for minikube action on suite
We have seen the CI failing to run the minikube action, reaching the API
limit. By using authenticated requests, we can overcome this limitation.
Follow-up of https://github.com/rook/rook/pull/7503/commits/1504fb869b5ed5b8f7df362bc9df3b45dd8ac9ea

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-12 14:26:38 +02:00
Sébastien Han c8b6aab718 Merge pull request #7581 from leseb/v16-monitoring-rules
ceph: add monitoring rules for pacific
2021-04-12 14:12:43 +02:00
Sébastien Han 31e22ad44f Merge pull request #7582 from subhamkrai/object-openshift
ceph: sslCertificateRef can't be null
2021-04-12 14:07:07 +02:00
subhamkrai d1cb929c41 ceph: sslCertificateRef can't be null
comment sslCertificateRef as it can't be
null.

Signed-off-by: subhamkrai <srai@redhat.com>
2021-04-12 16:17:04 +05:30
Sébastien Han 74c71880f1 ceph: add monitoring rules for pacific
New release, new files!

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-12 12:37:25 +02:00
Lars Lehtonen f1b341ba23 ceph: fix multiple imports
This fixes libraries that were being imported multiple times in
pkg/operator/ceph/object and its subpackages.

Signed-off-by: Lars Lehtonen <lars.lehtonen@gmail.com>
2021-04-12 02:57:13 -07:00
Umanga Chapagain 68c2db5d3a ceph: add monitoring key to cephcluster.spec.labels
CephCluster.Spec.Labels takes a key "monitoring". This is used
to inject labels into all the monitoring resources created by
the Rook operator.

Signed-off-by: Umanga Chapagain <chapagainumanga@gmail.com>
2021-04-12 12:02:27 +05:30
Travis Nielsen 8b96114e80 Merge pull request #7567 from alrs/ceph-file-imports
ceph: `pkg/operator/ceph/file`: multiple imports
2021-04-09 12:51:46 -06:00
Lars Lehtonen 017590c920 ceph: fix multiple imports
This fixes libraries that were being imported multiple times in
pkg/operator/ceph/file and its subpackages.

Signed-off-by: Lars Lehtonen <lars.lehtonen@gmail.com>
2021-04-09 09:52:25 -07:00
Blaine Gardner 194bd9ab95 Merge pull request #7564 from BlaineEXE/auto-merge-backport-1.6
bot: auto merge backports to release-1.6
2021-04-09 10:19:49 -06:00
Blaine Gardner ef9a889994 bot: auto merge backports to release-1.6
Previously forgot to add mergify rule to auto merge backports to
release-1.6. Backport PRs would still be created but not merged.

Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
2021-04-09 09:48:11 -06:00
Travis Nielsen c17a3447a9 Merge pull request #7566 from travisn/release-branch-tests
ceph: Integration tests use master tag even in release branch
2021-04-09 07:36:45 -06:00
Travis Nielsen 9546a2ef94 ceph: integration tests use master tag even in release branch
The integration tests pick up the manifests directly from the examples
folder, including with the release tag. In the release branch the Jenkins
build still uses the master build when building locally before tagging
it with the release tag. So the tests need to use the master tag.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit 456e2f3a79)
2021-04-08 23:43:16 -06:00
Satoru Takeuchi 0373171617 Merge pull request #7544 from BlaineEXE/osd-purge-docs
ceph: update osd purge docs
2021-04-09 06:06:51 +09:00
Travis Nielsen 1b77b9e4d0 Merge pull request #7556 from travisn/revert-default-sa
ceph: Revert rook-ceph-default service account so the daemons will use the built-in default SA again
2021-04-08 12:54:09 -06:00
Travis Nielsen 721acd1a8a Revert "ceph: added rook-ceph-default service account"
This reverts commit 737fb099fe.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2021-04-08 11:46:12 -06:00
Travis Nielsen 0bf766bc3f Revert "ceph: set default service account in scc"
This reverts commit c0019996a1.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2021-04-08 11:46:12 -06:00
Travis Nielsen 930827210b Revert "ceph: correctly tag default service account for csv"
This reverts commit 855236596e.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2021-04-08 11:46:07 -06:00
Blaine Gardner ac91fa77cc ceph: update osd-purge docs to stop operator
Upate osd purge docs to inform users they need to stop the operator
before purging OSDs when using node-based OSDs.

Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
2021-04-08 11:22:42 -06:00
Travis Nielsen 69bac95f83 Merge pull request #7540 from leseb/mon-maintenance
ceph: allow monitor maintenance mode
2021-04-08 09:06:27 -06:00
Sébastien Han e8c304d9b3 ceph: allow monitor maintenance mode
If the health spec of the CephCluster CRD has a timeout set to 0 like
so:

```
  healthCheck:
    daemonHealth:
      mon:
        disabled: false
        interval: 45s
        timeout: 0
```

And the mon goes out of quorum then Rook will not fail over the mon.
This is interesting when doing maintenance on a monitor and we don't
want to create a new one.

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-08 16:27:53 +02:00
Sébastien Han 0e86b8ddcf Merge pull request #7553 from subhamkrai/action-minikube-version
ci: update minikube version
2021-04-08 11:35:16 +02:00
subhamkrai 81624b255f ci: update minikube version
this commit update the github action
minikube version as minikube setup on
latest k8s version are failing.

Signed-off-by: subhamkrai <srai@redhat.com>
2021-04-08 12:32:04 +05:30
Blaine Gardner 20bc478844 Merge pull request #7547 from BlaineEXE/add-status-subresources
ceph: add status subresource to CRDS
2021-04-07 16:00:17 -06:00
Blaine Gardner 6478137aae Merge pull request #7548 from BlaineEXE/backport-release-1.6-label
bot: auto backport on  backport-release-1.6 label
2021-04-07 15:58:12 -06:00
Blaine Gardner b8aeef9f77 ceph: add status subresource to CRDS
Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
2021-04-07 15:36:41 -06:00
Blaine Gardner 706bb0118e bot: auto backport on backport-release-1.6 label
Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
2021-04-07 15:13:31 -06:00
Travis Nielsen e471fd4e5f Merge pull request #7545 from BlaineEXE/allow-empty-compression-mode
ceph: allow empty compressionMode in CRD pools
v1.6.0-alpha.0
2021-04-07 14:15:29 -06:00
Blaine Gardner 3f447c8e08 ceph: allow empty compressionMode in CRD pools
Allow empty ("") compressionMode in CRDs' 'pool' spec.

Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
2021-04-07 13:46:44 -06:00
Blaine Gardner 36e98aa44d Revert "ceph: remove auth in osd-purge job"
This reverts commit 80d2cde5c7.

Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
2021-04-07 13:12:19 -06:00
Sébastien Han fc8accb3ae Merge pull request #7536 from BlaineEXE/purge-osd-auth
ceph: remove auth in osd-purge job
2021-04-07 18:39:07 +02:00
Travis Nielsen 3a5f2181a8 Merge pull request #7529 from subhamkrai/gh-ci-master
ci: Enable master and release branch integration tests on push
2021-04-07 10:38:18 -06:00
Travis Nielsen 52a9dfc26d Merge pull request #7510 from travisn/support-pacific
ceph: Enable Ceph Pacific as a supported version
2021-04-07 10:10:35 -06:00
Travis Nielsen 665856b90a ceph: enable pacific as a supported ceph version
With the Ceph Pacific release coming this week we add support
in Rook for Pacific with the Rook v1.6 release coming soon.
The integration tests will now run across nautilus, octopus,
and pacific to cover all supported Ceph versions. The default
examples still specify Octopus until there is more bake time
for Pacific.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2021-04-07 06:59:04 -06:00
Travis Nielsen f620df9fd8 ceph: remove duplicate function to validate rgw in the ci
The CI had a duplicate function for validating the rgw daemon
was running

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2021-04-07 06:59:04 -06:00
Sébastien Han 62d45239d4 ceph: use minutes instead of seconds
So we don't have to do the math :)

Signed-off-by: Sébastien Han <seb@redhat.com>
2021-04-07 12:31:21 +02:00
Sébastien Han f1a784e2c4 Merge pull request #7532 from travisn/ssl-cert-empty
ceph: SSL cert property cannot be empty
2021-04-07 09:51:41 +02:00
Sébastien Han eeccf13d81 Merge pull request #7539 from travisn/default-sa-delimiters
ceph: Tag default service account for CSV generation
2021-04-07 09:19:52 +02:00
subhamkrai 20ab5befb0 ci: enable mgr suite for PR only
this commit enable mgr suite to run only
for PRs.

Signed-off-by: subhamkrai <srai@redhat.com>
2021-04-07 11:34:41 +05:30
subhamkrai 24ec105b58 ci: enable it test on push
this commit enable IT test to run on master push.

Signed-off-by: subhamkrai <srai@redhat.com>
2021-04-07 11:32:42 +05:30
Travis Nielsen 855236596e ceph: correctly tag default service account for csv
The csv was not generating the rook-ceph-default service account
because the comment delimiting it had a typo.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2021-04-06 21:31:20 -06:00
Travis Nielsen c5b94e6032 Merge pull request #7537 from rohan47/force_delete_stuck_rgw_pods
ceph: added rgw app label to rookAppLabels for deleting stuck rook pods
2021-04-06 21:03:29 -06:00