Commit Graph
11355 Commits
Author SHA1 Message Date
harshitasao fb697fb0f7 ci: github action pin dependencies
The openSSF scorecard report warns that the github actions are not pinned by hash.

This PR aims at improving this by pinning the github actions by hash.

Signed-off-by: harshitasao <harshitasao@gmail.com>
2024-08-15 19:06:30 +05:30
Travis Nielsen 9546b71097 Merge pull request #14418 from harshitasao/issue-14404
Added the scorecard github action and its badge
2024-08-14 11:09:45 -06:00
Travis Nielsen 9b5beb1ce0 Merge pull request #14566 from travisn/k8s-1.31
core: Support k8s versions 1.26 through 1.31
2024-08-13 17:54:10 -06:00
Travis Nielsen e157bb5a56 core: support k8s versions 1.26 through 1.31
With the release of K8s v1.31.0, we update the minimum
supported version to v1.26, and add v1.31 to the CI
so we can test the most recent six versions of K8s.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2024-08-13 16:54:52 -06:00
Blaine Gardner 5cf75d83ce Merge pull request #14559 from BlaineEXE/upgrade-docs
docs: update upgrade docs for v1.15
2024-08-13 11:15:02 -06:00
Blaine Gardner 6026fb1c36 docs: update upgrade docs for v1.15
Update Rook and Ceph upgrade docs for upcoming v1.15 release.
Tidy up pending release notes in the working text as well as official
doc texts.

Signed-off-by: Blaine Gardner <blaine.gardner@ibm.com>
2024-08-13 10:47:58 -06:00
Blaine Gardner fd0c687d6e Merge pull request #14557 from obnoxxx/token-perms-again
ci: fix excessive token permissions -- again
2024-08-12 12:19:29 -06:00
Michael Adam 2d292976a6 ci: fix excessive token permissions -- again
PR https://github.com/rook/rook/pull/14473

tried to improve the token-permissions score of the OpenSSF scorecard
report.

https://scorecard.dev/viewer/?uri=github.com/rook/rook

The latest scorecard run however shows that this score has  not improved.
It still shows two warbnings about missing top level permissions.

The rest looks good (just Info entries).

This change aims at  finally improving the token-permissions score

by adding top level permissions to the two warned-about workflow files.

Signed-off-by: Michael Adam <obnox@samba.org>
2024-08-12 19:24:20 +02:00
Travis Nielsen cb1ea8c7de Merge pull request #14546 from zalsader/fix-affinity-docs
docs: fix affinity docs in operator chart
2024-08-09 08:31:45 -06:00
Zuhair AlSader c146b59d6f docs: fix affinity docs in operator chart
Signed-off-by: Zuhair AlSader <zuhair@devzero.io>
2024-08-08 22:00:35 -04:00
Blaine Gardner 5f98d2ea3e Merge pull request #13807 from jklippel/feature/swift-and-keystone
rgw: implement support for authentication using keystone for s3 and swift
2024-08-08 09:55:34 -06:00
subham rai 33680d5bbc Merge pull request #14489 from subhamkrai/add-csidriveroperator-config
csi: add csi-operator operator config cr
2024-08-08 20:52:57 +05:30
ee8bcad49d rgw: add support for keystone auth + swift/s3
For the specification see:
<https://github.com/rook/rook/blob/master/design/ceph/object/swift-and-keystone-integration.md>

* extend the API object specs for swift and keystone integration

* adapt rgw to the new go-ceph version

  - The parameter lists of the API call have changes, as parameters
    ignored by the RGW Admin Ops API are no longer serialized, therefore
    the mock has to be adapted.

  - There is now validation for the user keys that are passed to the
    User get API, therefore things failed when we had empty keys in our
    User proxy object.

* expand the reconcile loop for the swift and keystone integration

* fix minor mistakes in design document

* add env var to pass extra args to minikube

  Minikube decides CPU cores and memory automatically based on the
  available resources on the machine which may be insufficient to
  run rook. This commit adds an environment variable to add arbitrary
  arguments to the minikube command, so both can be specified if
  desired.

* integration tests for swift and keystone

  The new integration of swift or s3 and keystone support by rook
  does not have any integration tests yet.

  This commit introduces integration tests for swift and keystone. The
  tests are done against a minimal keystone setup (keystone container
  image from Yaook-project (https://yaook.cloud), sqlite as database
  backend, cert-manager and trust-manager for test certificate setup).

  To prevent hardcoded credentials, passwords are generated
  by the tests. The integration tests use the openstack client
  (keystone- and swift-functionality) (https://docs.openstack.org/
  python-openstackclient/ latest/). This was a concious design decision
  to use client tooling as close as possible to the end user instead of
  using other go-libraries (such as gophercloud).

* add documentation on swift and keystone

  Currently there is no documentation on the use of Swift to access
  an object store as well as the use of OpenStack keystone for
  authentication.

  This commit adds documentation on the use of Swift and OpenStack
  keystone, as well as CRD-related documentation and an example setup.

* add integration tests for S3 via keystone

  This commit introduces integration tests for s3 and keystone. The
  tests are run against the same minimal keystone setup that the tests
  for swift and keystone use.

  The integration tests use the aws s3 client to use client tooling as
  close as possible to the end user instead of using other go-libraries.

Co-authored-by: Jan Klippel <jan.klippel@uhurutec.com>
Co-authored-by: Silvio Ankermann <silvio.ankermann@cloudandheat.com>
Signed-off-by: Sebastian Riese <sebastian.riese@cloudandheat.com>
Signed-off-by: Jan Klippel <jan.klippel@uhurutec.com>
Signed-off-by: Silvio Ankermann <silvio.ankermann@cloudandheat.com>
2024-08-08 14:26:21 +02:00
subhamkrai f40cb392c9 csi: add unit test for csi-op resources
adding unit test for csi-op resources and also
fixing small pointer issue with this commit.

Signed-off-by: subhamkrai <srai@redhat.com>
2024-08-08 15:43:16 +05:30
subhamkrai 64cce51f03 csi: add annotation to existing CSI driver
adding annotation to existing CSI driver resource so that
csi-operator can take ownership of existing CSI driver resources.

Signed-off-by: subhamkrai <srai@redhat.com>
2024-08-08 15:43:16 +05:30
subhamkrai c30b8a527b csi: add csi-operator rbac, crds files
adding the CSI-operator manifest files
inside rook repo naming deploy/examples/csi-operator-manifests.yaml.

Also, let's skip running any lint check on file `deploy/examples/csi-operator.yaml` as this is owned by CSI-operator.

Signed-off-by: subhamkrai <srai@redhat.com>
2024-08-08 15:43:12 +05:30
subhamkrai 1ad20d07e7 csi: add csi-operator operator config cr
Signed-off-by: subhamkrai <srai@redhat.com>
2024-08-08 11:21:49 +05:30
subhamkrai 667e044013 csi: add new CSI-operator config CR
adding changes to create CSI-operator config CR based on
every radosnamesapce and subvolumegroup.

Signed-off-by: subhamkrai <srai@redhat.com>
2024-08-08 11:21:49 +05:30
subhamkrai 4b0b3a55d9 csi: add code for new CSI operator CR cephcluster
adding code changes,rbac changes required for create the new
Ceph-CSI operator CR named cephCluster in api group 'csi.ceph.io'.

Signed-off-by: subhamkrai <srai@redhat.com>
2024-08-08 11:21:45 +05:30
Travis Nielsen 89f29a427a Merge pull request #14536 from rook/dependabot/go_modules/github-dependencies-49fcb8191a
build(deps): bump the github-dependencies group with 3 updates
2024-08-07 15:00:45 -06:00
Travis Nielsen ca50256e4c Merge pull request #14534 from rook/dependabot/github_actions/wagoid/commitlint-github-action-6.0.2
build(deps): bump wagoid/commitlint-github-action from 6.0.1 to 6.0.2
2024-08-07 15:00:21 -06:00
Travis Nielsen 074d2fbd6b Merge pull request #14542 from Rakshith-R/add-block-mode-ex
csi: add pvc & pod yamls for block volume mode
2024-08-07 07:26:39 -06:00
Rakshith R 53e2f8c67f csi: add pvc & pod yamls for block volume mode
This commit adds example yamls for block
volume mode.

Signed-off-by: Rakshith R <rar@redhat.com>
2024-08-07 12:50:50 +05:30
dependabot[bot] d1030cfc80 build(deps): bump the github-dependencies group with 3 updates
Bumps the github-dependencies group with 3 updates: [github.com/IBM/keyprotect-go-client](https://github.com/IBM/keyprotect-go-client), [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) and [github.com/k8snetworkplumbingwg/network-attachment-definition-client](https://github.com/k8snetworkplumbingwg/network-attachment-definition-client).

Updates `github.com/IBM/keyprotect-go-client` from 0.14.3 to 0.15.1
- [Release notes](https://github.com/IBM/keyprotect-go-client/releases)
- [Changelog](https://github.com/IBM/keyprotect-go-client/blob/master/CHANGELOG.md)
- [Commits](https://github.com/IBM/keyprotect-go-client/compare/v0.14.3...v0.15.1)

Updates `github.com/aws/aws-sdk-go` from 1.55.3 to 1.55.5
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.55.3...v1.55.5)

Updates `github.com/k8snetworkplumbingwg/network-attachment-definition-client` from 1.7.0 to 1.7.1
- [Release notes](https://github.com/k8snetworkplumbingwg/network-attachment-definition-client/releases)
- [Commits](https://github.com/k8snetworkplumbingwg/network-attachment-definition-client/compare/v1.7.0...v1.7.1)

---
updated-dependencies:
- dependency-name: github.com/IBM/keyprotect-go-client
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-dependencies
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-dependencies
- dependency-name: github.com/k8snetworkplumbingwg/network-attachment-definition-client
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-08-06 14:15:40 -06:00
Travis Nielsen 400f4edd01 Merge pull request #14455 from cobaltcore-dev/target-placement
target placement design doc
2024-08-06 11:17:11 -06:00
Blaine Gardner 394287fd99 Merge pull request #14532 from BlaineEXE/object-use-advertise-for-admin-ops
object: use advertise endpoint for admin ops
2024-08-05 15:46:04 -06:00
Blaine Gardner b4a2285aa6 object: use advertise endpoint for admin ops
RGW can only serve a single certificate. This limitation means that the
prior behavior of using the default service for admin ops when TLS is
enabled may mean it requires additional complex certificate management
to make sure the object store uses a certificate valid for Rook internal
admin ops and user connections.

This is needlessly complex for users. Instead, change Rook's behavior
and documentation to clarify that it will use the same endpoint intended
for S3 client applications. This means that users have a more
straightforward path to enabling both Rook and consuming applications.

More info: https://github.com/rook/rook/issues/14530

Signed-off-by: Blaine Gardner <blaine.gardner@ibm.com>
2024-08-05 14:32:59 -06:00
Travis Nielsen cda89fb60a Merge pull request #14537 from rook/dependabot/go_modules/golang.org/x/sync-0.8.0
build(deps): bump golang.org/x/sync from 0.7.0 to 0.8.0
2024-08-05 11:45:45 -06:00
dependabot[bot] b9e58618dc build(deps): bump golang.org/x/sync from 0.7.0 to 0.8.0
Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.7.0 to 0.8.0.
- [Commits](https://github.com/golang/sync/compare/v0.7.0...v0.8.0)

---
updated-dependencies:
- dependency-name: golang.org/x/sync
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-08-05 12:56:10 +00:00
dependabot[bot] e4106894bb build(deps): bump wagoid/commitlint-github-action from 6.0.1 to 6.0.2
Bumps [wagoid/commitlint-github-action](https://github.com/wagoid/commitlint-github-action) from 6.0.1 to 6.0.2.
- [Changelog](https://github.com/wagoid/commitlint-github-action/blob/master/CHANGELOG.md)
- [Commits](https://github.com/wagoid/commitlint-github-action/compare/v6.0.1...v6.0.2)

---
updated-dependencies:
- dependency-name: wagoid/commitlint-github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-08-05 12:05:26 +00:00
Travis Nielsen 9c98467f3c Merge pull request #14517 from iPraveenParihar/csi/explicitly-set-topology-flag
csi: explicitly set Topology feature-gate
2024-08-01 10:45:16 -06:00
Artem Torubarov e16b683257 docs: target placement storage class
Signed-off-by: Artem Torubarov <artem.torubarov@clyso.com>
2024-08-01 13:12:22 +02:00
Blaine Gardner b76631ace9 Merge pull request #14467 from BlaineEXE/object-advertise-endpoint
object: add hosting.advertiseEndpoint config
2024-07-31 16:17:24 -06:00
Travis Nielsen 1c720b115a Merge pull request #14520 from obnoxxx/fix-auto-assign-2
ci: fix syntax of auto-assign action
2024-07-31 12:54:56 -06:00
Travis Nielsen e92656c64f Merge pull request #14522 from travisn/mergify-1.15
ci: Add 1.15 branch to mergify rules
2024-07-31 12:52:54 -06:00
Travis Nielsen 519869117b ci: add 1.15 branch to mergify rules
With the creation of the 1.15 branch, add the branch
to the mergify rules. Also remove the 1.10 backport
rules since no longer needed.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2024-07-31 12:45:55 -06:00
Michael Adam 0d01341d9c ci: fix syntax of auto-assign action
Fixes: #14518

The previous attempt to fix the auto-assign action

has introduced an indentation error in the workflow yaml file.

This should finally fix it, hopefully.

Signed-off-by: Michael Adam <obnox@samba.org>
2024-07-31 20:31:09 +02:00
Travis Nielsen 0d83cd5d0b Merge pull request #14519 from obnoxxx/fix-auto-assign
ci: fix the permissions of the assign action
v1.15.0-alpha.0
2024-07-31 12:16:31 -06:00
Michael Adam c57a21a80e ci: fix the permissions of the assign action
Fixes: #14518

A recent change seems to have broken the permissions of the auto-assign
action.

This tries to fix this by making permissions more specific.

Signed-off-by: Michael Adam <obnox@samba.org>
2024-07-31 20:06:38 +02:00
Praveen M e30e5e881f csi: explicitly set Topology feature-gate
issue:
external-provisioner (v5) enabled topology feature-gate by default
and the current implementation in Rook uses a conditional block to
enable the topology feature gate. This approach now does not directly
reflect the state of the `CSI_ENABLE_TOPOLOGY`.

fix:
replacing the conditional block with a direct use of the
`CSI_ENABLE_TOPOLOGY` for flag value.

Signed-off-by: Praveen M <m.praveen@ibm.com>
2024-07-31 20:37:20 +05:30
Artem Torubarov edcedd3124 docs: pool placement design doc
Signed-off-by: Artem Torubarov <artem.torubarov@clyso.com>
2024-07-30 11:24:57 +02:00
Travis Nielsen d91b8073c5 Merge pull request #14488 from sp98/set-min-compat-client-to-reef
core: set min-compat-client to reef for upmap-read
2024-07-29 11:28:29 -06:00
sp98 ee5e710912 core: set min-compat-client to reef for upmap-read
If upmap-read balancer mode is required, then set
min-compat-client to reef

Signed-off-by: sp98 <sapillai@redhat.com>
2024-07-29 22:03:37 +05:30
subham rai 7dafd45acc Merge pull request #14509 from rook/dependabot/go_modules/github-dependencies-8a83b577c4
build(deps): bump the github-dependencies group with 3 updates
2024-07-29 21:12:03 +05:30
dependabot[bot] ed74d22f1f build(deps): bump the github-dependencies group with 3 updates
Bumps the github-dependencies group with 3 updates: [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go), [github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring](https://github.com/prometheus-operator/prometheus-operator) and [github.com/prometheus-operator/prometheus-operator/pkg/client](https://github.com/prometheus-operator/prometheus-operator).


Updates `github.com/aws/aws-sdk-go` from 1.54.20 to 1.55.3
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.54.20...v1.55.3)

Updates `github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring` from 0.75.1 to 0.75.2
- [Release notes](https://github.com/prometheus-operator/prometheus-operator/releases)
- [Changelog](https://github.com/prometheus-operator/prometheus-operator/blob/main/CHANGELOG.md)
- [Commits](https://github.com/prometheus-operator/prometheus-operator/compare/v0.75.1...v0.75.2)

Updates `github.com/prometheus-operator/prometheus-operator/pkg/client` from 0.75.1 to 0.75.2
- [Release notes](https://github.com/prometheus-operator/prometheus-operator/releases)
- [Changelog](https://github.com/prometheus-operator/prometheus-operator/blob/main/CHANGELOG.md)
- [Commits](https://github.com/prometheus-operator/prometheus-operator/compare/v0.75.1...v0.75.2)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-dependencies
- dependency-name: github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-dependencies
- dependency-name: github.com/prometheus-operator/prometheus-operator/pkg/client
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-07-29 12:38:51 +00:00
Travis Nielsen a3c487e36d Merge pull request #14500 from nicofnt/master
docs: add troubleshooting topic for containerd LimitNOFILE issue
2024-07-26 08:11:30 -06:00
nicofnt 0952e36ba1 docs: add new topic in troubleshooting guide for the LimitNOFILE issue
Create new topic in the troubleshooting guide to explain how to fix a new rook-ceph deployment that could remain in an unhealthy state or fails to config because of the LimitNOFILE value specified in the containerd.service configuration.

Signed-off-by: Nicola Fantacuzzi <nfantacuzzi@gmail.com>
2024-07-26 10:43:57 +02:00
Travis Nielsen 39cc31dbf1 Merge pull request #14499 from travisn/daily-upgrade-tests
tests: Check for correct base rook version during upgrade
2024-07-25 15:29:26 -06:00
Travis Nielsen 4cf349836f tests: check for correct base rook version during upgrade
The daily upgrade tests were checking for a specific
version of the rook image instead of the local_build tag
that is set as the rook base image since 14486.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
2024-07-25 12:44:30 -06:00
Travis Nielsen 54a2944252 Merge pull request #14382 from parth-gr/external-seamless-upgrade-doc
doc: add upgrade steps for external cluster
2024-07-25 09:50:42 -06:00