Skip to content
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

Syncing latest changes from release-1.13 for rook #543

Merged
merged 94 commits into from
Dec 7, 2023

Conversation

df-build-team
Copy link

PR containing the latest commits from release-1.13 branch

parth-gr and others added 30 commits November 16, 2023 19:13
Add this reporting with the cephcluster reconcile,
Similar way we reported other telemetry's

Closes: rook#12344

Signed-off-by: parth-gr <[email protected]>
core: report node metrics using ceph telemetry
Upgrade minimum kubernetes support to 1.23.17 from 1.22.17.
Also, upgrade kubernetes version latest minor release.

Signed-off-by: subhamkrai <[email protected]>
correct the csi ceph.conf path
in the documentation.

Signed-off-by: Madhu Rajanna <[email protected]>
ci: upgrade min k8s support to 1.23.17
Bumps the github-dependencies group with 1 update: [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go).

- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](aws/aws-sdk-go@v1.47.9...v1.48.0)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
including the namespace tag in the monitoring manifests to streamline
the automated process of modifying them through scripting

Signed-off-by: Redouane Kachach <[email protected]>
…space

manifest: adding the namespace tag to the monitoring manifests
…dependencies-df6d6e1067

build(deps): bump the github-dependencies group with 1 update
Bumps the k8s-dependencies group with 4 updates: [k8s.io/api](https://github.com/kubernetes/api), [k8s.io/apiextensions-apiserver](https://github.com/kubernetes/apiextensions-apiserver), [k8s.io/cli-runtime](https://github.com/kubernetes/cli-runtime) and [k8s.io/cloud-provider](https://github.com/kubernetes/cloud-provider).

Updates `k8s.io/api` from 0.28.3 to 0.28.4
- [Commits](kubernetes/api@v0.28.3...v0.28.4)

Updates `k8s.io/apiextensions-apiserver` from 0.28.3 to 0.28.4
- [Release notes](https://github.com/kubernetes/apiextensions-apiserver/releases)
- [Commits](kubernetes/apiextensions-apiserver@v0.28.3...v0.28.4)

Updates `k8s.io/cli-runtime` from 0.28.3 to 0.28.4
- [Commits](kubernetes/cli-runtime@v0.28.3...v0.28.4)

Updates `k8s.io/cloud-provider` from 0.28.3 to 0.28.4
- [Commits](kubernetes/cloud-provider@v0.28.3...v0.28.4)

---
updated-dependencies:
- dependency-name: k8s.io/api
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-dependencies
- dependency-name: k8s.io/apiextensions-apiserver
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-dependencies
- dependency-name: k8s.io/cli-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-dependencies
- dependency-name: k8s.io/cloud-provider
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: k8s-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
…endencies-3e07744c0b

build(deps): bump the k8s-dependencies group with 4 updates
Due to multi-site Ceph object resources (realms, zonegroups, zones)
reconciliation CLI commands being proxied through CephCluster's Mgr pods
in Multus-based scenarios, return codes were being additionally wrapped
by K8S' command streams and weren't adequately handled in-code, causing
reconciliation of CephObject(Realm|ZoneGroup|Zone) resources to fail, as
an exit code of 0, instead of the expected syscall.ENOENT (2) was returned.

Fixes rook#12833

Signed-off-by: zer0def <[email protected]>
rgw: correctly handle mgr-proxied rgw cli commands in multus scenarios
csi-node-driver-registrar: v2.9.1
csi-resizer: v1.9.2
csi-provisioner: v3.6.2
csi-attacher: v4.4.2
csi-snapshotter: v6.3.2
Signed-off-by: Praveen M <[email protected]>
…ons in debug

During certain maintenance tasks the admin will own running
operations on the ceph mgr, rgw, mds and rbd-mirror daemons
and the operator should not interfere with those operations.

Co-authored-by: gauravsitlani <[email protected]>
Signed-off-by: subhamkrai <[email protected]>
build: use /usr/bin/env to look up script interpreters
Add tmate debugging to daily/nightly jobs to help debugging.

Signed-off-by: Blaine Gardner <[email protected]>
Similar to the ceph crash collector daemon that generates a keyring with more
restrictive privileges, the exporter should also generate and use a more limited keyring.

Signed-off-by: avanthakkar <[email protected]>
…ific-keyring

exporter: run exporter with specific keyring
…rsions

csi: update csi sidecars' image version
core: operator to skip reconcile of mgr, rgw, mds and rbd-mirror
this change updates the serviceMonitor interval of the the rook-ceph-exporter
with the value from the MonitoringSpec.

closes: rook#13159

Signed-off-by: Redouane Kachach <[email protected]>
this commit adds csi-addon sidecar to
cephfsplugin provisioner deployment

Signed-off-by: Riya Singhal <[email protected]>
mgr: get servicemonitor exporter's interval from MonitoringSpec
csi: add csi-addons sidecar to cephfs deployment
mergify bot and others added 21 commits December 2, 2023 11:04
operator: allow setting ceph config options via ceph cluster crd (backport rook#13246)
Signed-off-by: Anthony D'Atri <[email protected]>
(cherry picked from commit 59d0240)
12976 : Improve Documentation/Storage-Configuration/Ceph-CSI/ceph-csi… (backport rook#12977)
This addresses rook#12660. As discussed in the issue when zone failure
happens and if the operator running in that zone, then it takes the
default toleration of 300 seconds. We want to override this default
toleration seconds to 5 seconds so that operator gets redeployed
immediately

Signed-off-by: Mahesh Shetty <[email protected]>
(cherry picked from commit 5185371)
Add toleration of 5 seconds to rook-ceph operator deployment to override default toleration seconds of 300 seconds (backport rook#12983)
The K8s v1.25.17 release does not exist, which was mistakenly
set in the CI for the master and release tests. Now we revert
it back to v1.25.16.

Signed-off-by: travisn <[email protected]>
(cherry picked from commit 5fc7237)
Signed-off-by: Cyril Jouve <[email protected]>
(cherry picked from commit d65c5b6)
helm: fix namespace for objstore ingress (backport rook#13312)
instead of use metadata name as the backend resource name
we need to add a new name in the spec which can be the
actual backend name

Closes: rook#13220

Signed-off-by: parth-gr <[email protected]>
(cherry picked from commit 5bf3847)
Add the missing explanation of pgHealthCheckTimeout and pgHealthyRegex
in the CephCluster' CRD doc. In addition, remove a verbose sentence
from the explanation of `osdMaintenanceTimeout`.

Signed-off-by: Satoru Takeuchi <[email protected]>
(cherry picked from commit ab1f1d9)
In some cases, it's desired to use `must` than `should`.

Signed-off-by: Satoru Takeuchi <[email protected]>
(cherry picked from commit 2059a04)
doc: add the description of some fields and update the table of supported configurations for OSD creation (backport rook#13325)
Loop variables cannot be reliably uses since they will
change with each iteration. Update these loop variable
uses to be safe by indexing the slice rather than
using the loop variable directly.

Also suppress the linter issues for passwords used
in tests.

Signed-off-by: travisn <[email protected]>
(cherry picked from commit 6f8e424)
namespace: add name spec in cephBlockPoolRadosNamespace CRD (backport rook#13270)
core: Fix golang linter issues with variables in loops and update linter version (backport rook#13324)
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.48.4 to 1.48.11
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](aws/aws-sdk-go@v1.48.4...v1.48.11)

Updates `github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring` from 0.69.1 to 0.70.0
- [Release notes](https://github.com/prometheus-operator/prometheus-operator/releases)
- [Changelog](https://github.com/prometheus-operator/prometheus-operator/blob/main/CHANGELOG.md)
- [Commits](prometheus-operator/prometheus-operator@v0.69.1...v0.70.0)

Updates `github.com/prometheus-operator/prometheus-operator/pkg/client` from 0.69.1 to 0.70.0
- [Release notes](https://github.com/prometheus-operator/prometheus-operator/releases)
- [Changelog](https://github.com/prometheus-operator/prometheus-operator/blob/main/CHANGELOG.md)
- [Commits](prometheus-operator/prometheus-operator@v0.69.1...v0.70.0)

---
updated-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/prometheus-operator/prometheus-operator/pkg/apis/monitoring
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-dependencies
- dependency-name: github.com/prometheus-operator/prometheus-operator/pkg/client
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
(cherry picked from commit cbf8ac5)
Run make go.mod.update to update all the go modules instead
of just the ones updated by the dependabot.

Signed-off-by: travisn <[email protected]>
(cherry picked from commit 6b9f897)
build(deps): bump the github-dependencies group with 3 updates (backport rook#13316)
This commit updates default cephcsi driver version
to v3.10.0 and filesystem reconciler now creates
csi subvolumegroup by default.

Signed-off-by: Rakshith R <[email protected]>
(cherry picked from commit a3220d8)
csi: update default cephcsi version to 3.10.0 (backport rook#13327)
@df-build-team df-build-team requested a review from a team December 7, 2023 08:08
The OWNERS file is needed to work with the bot used
by the downstream integration for approvals and lgtm.

Signed-off-by: travisn <[email protected]>
@travisn
Copy link

travisn commented Dec 7, 2023

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 7, 2023
Copy link

openshift-ci bot commented Dec 7, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: df-build-team, travisn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 7, 2023
@travisn travisn merged commit f3778a5 into release-4.15 Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.