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

Manual sync of upstream to downstream master #770

Merged
merged 32 commits into from
Nov 7, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
448f42a
test: make debug-ci label work again
satoru-takeuchi Oct 30, 2024
04741fb
build: make generating make targets more systematic
obnoxxx Oct 28, 2024
c017412
Merge pull request #14922 from obnoxxx/make-generate-systematic
BlaineEXE Oct 30, 2024
4e8d4de
Merge pull request #14934 from cybozu-go/make-debug-ci-label-work-again
BlaineEXE Oct 30, 2024
2ff429e
csi: remove version check for k8s and cephcsi
travisn Oct 30, 2024
f60f444
test: remove obsolete object user test flag
travisn Oct 30, 2024
47ca3fd
doc: avoid always enabling tmate session
satoru-takeuchi Oct 30, 2024
d725b31
Merge pull request #14942 from travisn/no-csi-version-check
travisn Oct 31, 2024
0ae1916
Merge pull request #14944 from cybozu-go/test-avoidalways--running-tm…
travisn Oct 31, 2024
7fbe4e7
osd: allow scheduling on unschedulable nodes
travisn Oct 30, 2024
f489f99
core: set resources on the cmd reporter jobs
travisn Oct 30, 2024
62d9933
doc: clarify dataDirHostPath must be unique for each cluster
satoru-takeuchi Oct 30, 2024
98638de
Merge pull request #14936 from cybozu-go/doc-take-multicluster-into-a…
travisn Oct 31, 2024
18c9261
Merge pull request #14943 from travisn/obsolete-object-test
travisn Nov 1, 2024
99d60c5
mds: wait for mds standby upgrade for same fs
travisn Nov 1, 2024
bebe5bd
Merge pull request #14949 from travisn/schedule-all-nodes
travisn Nov 1, 2024
13495ae
Merge pull request #14941 from travisn/detect-version-resources
travisn Nov 4, 2024
0b666c0
Merge pull request #14952 from travisn/fs-standby-update
travisn Nov 4, 2024
b229240
ci: default to the latest stable squid instead of devel
travisn Nov 4, 2024
fb02f50
Merge pull request #14962 from travisn/ci-latest-squid
travisn Nov 4, 2024
9785ef5
rbdmirror: enable periodic monitoring for rados namespace
parth-gr Oct 23, 2024
8ad2bf8
rgw: keep default-placement in zone config
arttor Nov 1, 2024
97a1362
core: add capabilities to securityContext to fix CIS 5.2.8
prazumovsky Oct 9, 2024
8dea772
pool: fix CephBlockPool enableRBDStats not clearing rbd_stats_pools
OdedViner Oct 23, 2024
7692cae
Merge pull request #14896 from parth-gr/rbd-mirror-rados-monitor
travisn Nov 5, 2024
1ed6251
Merge pull request #14951 from cobaltcore-dev/keep-default-placement
travisn Nov 5, 2024
8f6da3d
Merge pull request #14897 from OdedViner/delete_pool_item
BlaineEXE Nov 5, 2024
c5df2fe
Merge pull request #14824 from prazumovsky/fix-5-2-8
travisn Nov 5, 2024
501e0ea
csi: add new RBAC required for csiaddons
Madhu-1 Nov 6, 2024
70c1f27
Merge pull request #14970 from Madhu-1/add-rbac-csiaddons-sidecar
travisn Nov 6, 2024
22cc630
Merge remote-tracking branch 'us/master' into sync_us-ds
Nikhil-Ladha Nov 7, 2024
b6113ae
csv: add generated changes
Nikhil-Ladha Nov 7, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 21 additions & 0 deletions .github/workflows/canary-integration-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -342,6 +343,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -419,6 +421,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -471,6 +474,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -520,6 +524,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -578,6 +583,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -630,6 +636,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -687,6 +694,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -763,6 +771,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -815,6 +824,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -870,6 +880,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -938,6 +949,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -992,6 +1004,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1056,6 +1069,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1146,6 +1160,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1216,6 +1231,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1273,6 +1289,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1532,6 +1549,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1627,6 +1645,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: run encryption KMS IBM Key Protect
uses: ./.github/workflows/encryption-pvc-kms-ibm-kp
Expand Down Expand Up @@ -1657,6 +1676,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down Expand Up @@ -1738,6 +1758,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/canary-test-config
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/integration-test-helm-suite.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup latest cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup latest cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/integration-test-mgr-suite.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/integration-test-object-suite.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup latest cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/integration-test-smoke-suite.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/integration-test-upgrade-suite.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down Expand Up @@ -83,6 +84,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: setup cluster resources
uses: ./.github/workflows/integration-test-config-latest-k8s
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/multus.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ jobs:
uses: ./.github/workflows/tmate_debug
with:
use-tmate: ${{ secrets.USE_TMATE }}
debug-ci: ${{ contains(github.event.pull_request.labels.*.name, 'debug-ci') }}

- name: Setup multus
run: ./tests/scripts/multus/setup-multus.sh
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/tmate_debug/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,15 @@ inputs:
use-tmate:
description: "boolean for enabling TMATE"
required: true
debug-ci:
description: "boolean for debug-ci label in PR"
requnred: false
runs:
using: "composite"
steps:
- name: consider debugging
shell: bash --noprofile --norc -eo pipefail -x {0}
if: runner.debug || contains(github.event.pull_request.labels.*.name, 'debug-ci')
if: runner.debug || inputs.debug-ci == 'true'
run: |
# Enable tmate only in the Rook fork, where the USE_TMATE secret is set in the repo, or if the action is re-run
if [ "$GITHUB_REPOSITORY_OWNER" = "rook" ] || [ -n "${{ inputs.use-tmate }}" ] || [ "$GITHUB_RUN_ATTEMPT" -gt 1 ]; then
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,9 @@ If any setting is unspecified, a suitable default will be used automatically.
- `interval`: frequency of the snapshots. The interval can be specified in days, hours, or minutes using d, h, m suffix respectively.
- `startTime`: optional, determines at what time the snapshot process starts, specified using the ISO 8601 time format.

!!! note
If mirroring is enabled, whether to monitor the status and the interval of status updates is based on the `statusCheck` spec values of the parent CephBlockPool CR.

## Creating a Storage Class

Once the RADOS namespace is created, an RBD-based StorageClass can be created to
Expand Down
5 changes: 4 additions & 1 deletion Documentation/CRDs/Cluster/ceph-cluster-crd.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ Settings can be specified at the global level to apply to the cluster as a whole
Using the general `v19` tag is not recommended in production because it may lead to inconsistent versions of the image running across different nodes in the cluster.
* `allowUnsupported`: If `true`, allow an unsupported major version of the Ceph release. Currently Reef and Squid are supported. Future versions such as Tentacle (v20) would require this to be set to `true`. Should be set to `false` in production.
* `imagePullPolicy`: The image pull policy for the ceph daemon pods. Possible values are `Always`, `IfNotPresent`, and `Never`. The default is `IfNotPresent`.
* `dataDirHostPath`: The path on the host ([hostPath](https://kubernetes.io/docs/concepts/storage/volumes/#hostpath)) where config and data should be stored for each of the services. If the directory does not exist, it will be created. Because this directory persists on the host, it will remain after pods are deleted. Following paths and any of their subpaths **must not be used**: `/etc/ceph`, `/rook` or `/var/log/ceph`.
* `dataDirHostPath`: The path on the host ([hostPath](https://kubernetes.io/docs/concepts/storage/volumes/#hostpath)) where config and data should be stored for each of the services. If there are multiple clusters, the directory must be unique for each cluster. If the directory does not exist, it will be created. Because this directory persists on the host, it will remain after pods are deleted. Following paths and any of their subpaths **must not be used**: `/etc/ceph`, `/rook` or `/var/log/ceph`.
* **WARNING**: For test scenarios, if you delete a cluster and start a new cluster on the same hosts, the path used by `dataDirHostPath` must be deleted. Otherwise, stale keys and other config will remain from the previous cluster and the new mons will fail to start.
If this value is empty, each pod will get an ephemeral directory to store their config files that is tied to the lifetime of the pod running on that node. More details can be found in the Kubernetes [empty dir docs](https://kubernetes.io/docs/concepts/storage/volumes/#emptydir).
* `skipUpgradeChecks`: if set to true Rook won't perform any upgrade checks on Ceph daemons during an upgrade. Use this at **YOUR OWN RISK**, only if you know what you're doing. To understand Rook's upgrade process of Ceph, read the [upgrade doc](../../Upgrade/rook-upgrade.md#ceph-version-upgrades).
Expand Down Expand Up @@ -94,6 +94,8 @@ For more details on the mons and when to choose a number other than `3`, see the
* For non-PVCs: `placement.all` and `placement.osd`
* For PVCs: `placement.all` and inside the storageClassDeviceSets from the `placement` or `preparePlacement`
* `flappingRestartIntervalHours`: Defines the time for which an OSD pod will sleep before restarting, if it stopped due to flapping. Flapping occurs where OSDs are marked `down` by Ceph more than 5 times in 600 seconds. The OSDs will stay down when flapping since they likely have a bad disk or other issue that needs investigation. If the issue with the OSD is fixed manually, the OSD pod can be manually restarted. The sleep is disabled if this interval is set to 0.
* `scheduleAlways`: Whether to always schedule OSD pods on nodes declared explicitly in the "nodes" section, even if they are
temporarily not schedulable. If set to true, consider adding placement tolerations for unschedulable nodes.
* `fullRatio`: The ratio at which Ceph should block IO if the OSDs are too full. The default is 0.95.
* `backfillFullRatio`: The ratio at which Ceph should stop backfilling data if the OSDs are too full. The default is 0.90.
* `nearFullRatio`: The ratio at which Ceph should raise a health warning if the cluster is almost full. The default is 0.85.
Expand Down Expand Up @@ -485,6 +487,7 @@ You can set resource requests/limits for Rook components through the [Resource R
It scrapes for Ceph daemon core dumps and sends them to the Ceph manager crash module so that core dumps are centralized and can be easily listed/accessed.
You can read more about the [Ceph Crash module](https://docs.ceph.com/docs/master/mgr/crash/).
* `logcollector`: Set resource requests/limits for the log collector. When enabled, this container runs as side-car to each Ceph daemons.
* `cmd-reporter`: Set resource requests/limits for the jobs that detect the ceph version and collect network info.
* `cleanup`: Set resource requests/limits for cleanup job, responsible for wiping cluster's data after uninstall
* `exporter`: Set resource requests/limits for Ceph exporter.

Expand Down
Loading
Loading