This fixes OSD info being incomplete when a sidecar container is first
in the .spec.Template.Spec.Containers list.
I added a small helper function to find a container named "osd" instead
of always assuming the first one.
Signed-off-by: Jan Breitkopf <jan.breitkopf@prorocketeers.com>
(cherry picked from commit 1ff6f70311)
`updateExistingOSDs` should not wait and retry if the golang context got
cancelled. Else it would keep on retrying for 20 minutes before
exciting.
Signed-off-by: Santosh <sapillai@redhat.com>
(cherry picked from commit 33046734b3)
into a single combined integration-test-setup-cluster-resources
workflow. This ensures that the same gha runner preparation, go version,
etc. is used for both the canary and integration tests.
(cherry picked from commit c5bfe3a)
Change from the original version:
- Keep the test target k8s versions as are.
Signed-off-by: Joshua Hoblitt <josh@hoblitt.com>
Signed-off-by: Satoru Takeuchi <satoru.takeuchi@gmail.com>
With the iscsi device created now for integration tests,
the multicluster test can now use the raw device name
instead of assuming it is a partition.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit e08c183d82)
It has only been used by the flex driver which is long gone.
This removes the nodes/proxy RBAC grant from the helm chart, the example
YAMl and the security design docs.
Signed-off-by: Tobias Florek <me@ibotty.net>
(cherry picked from commit db814dcd4b)
The two-object-one-zone canary test is not waiting
long enough for the cluster to be created. The default
timeout is 120s, and it frequently hits that timeout.
When the test passes, it commonly takes between 100-120
seconds to complete, so we just need to increase the
timeout.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit 993e7b2f01)
Read affinity in Ceph v20.2.0 can cause corruption.
Disable this feature for this version of Ceph to
avoid hitting any corruption in this case. When the next
patch release is out for tentacle, this issue will be
resolved and when upgrading to it, read affinity would
automatically be re-enabled if needed.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit 76e9c04b36)
The cosi controller has moved to a new repo and added versioning. 0.2 is the last expected minor release before v1alpha2 which will need more updating.
Signed-off-by: Jade Bilkey <herself@thefumon.com>
(cherry picked from commit df0aa9c1a7)
add support for ceph.rook.io/do-not-reconcile on cephcluster crs
skip reconcile early and emit a normal reconcile-skipped event
avoid reloadmanager on updates for labeled cephclusters
add unit test for skip behavior
Signed-off-by: Oded Viner <oviner@redhat.com>
(cherry picked from commit 5be726b386)
This change mounts the rook-config-override ConfigMap in the helm chart
toolbox and merges its contents into the generated ceph.conf, matching
the behavior from commit 9570d141.
Signed-off-by: Matthew N. Heler <matthew.heler@hotmail.com>
(cherry picked from commit 690589c080)
The watch-active sidecar reconciles Services and wants to set
blockOwnerDeletion=true in the ownerReference. On clusters with
OwnerReferencesPermissionEnforcement admission controller this
requires the cephclusters/finalizers permission.
Signed-off-by: Manuel Grandeit <m.grandeit@gmail.com>
(cherry picked from commit 4f7ead07ca)
previously, if multiple mount option was passed it was
handing it wrong. For example,
```
ms_mode=prefer-secure,recover_session=clean
```
was treated as
```
ms_mode: prefer-secure,recover_sessio
```
With current fix it handles it correctly. Example,
```
ms_mode: prefer-secure,
recover_session=clean,
```
Signed-off-by: subhamkrai <srai@redhat.com>
(cherry picked from commit acf9ad8995)
The ibm keyprotect canary test was missing the csi operator
creation, so the cluster could not be reconciled.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit 8b251ff6a2)
Add a resource watch to reconcile clusters when any referenced Secrets
in cephConfigFromSecret are updated. This matches the behaviour of
reconciling clusters when cephConfig changes. Without this change, if a
Secret that is referenced by cephConfigFromSecret is updated, then the
cluster is not immediately updated, so the cluster state will not match
the configuration until something else triggers a reconcile.
Resolves#16371
Signed-off-by: Martin Rowe <martin.p.rowe@gmail.com>
(cherry picked from commit 816c67847d)
this commit adds new github action to free some disk
space. It was revert sometime back when github runner
size was increased but looks like we need to add again
as mon is complaining regarding low disk space. Integration
tests already have this adding this in canary tests.
Signed-off-by: subhamkrai <srai@redhat.com>
(cherry picked from commit c983fc39db)
Ceph Tentacle v20 comes with erasure coding optimizations
that must be enabled explicitly with a setting on the pool.
Add this setting to the docs and examples.
Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit c05768feb3)
we are now adding tentacle to the supported versions and add the
upgrade test from squid to tentacle. Also, clean some unused code.
Signed-off-by: subhamkrai <srai@redhat.com>
(cherry picked from commit 11fac627b0)
The operator supports setting options on the node exporter, such
as hostNetwork, which were previously omitted from the templated
CephCluster resource in the rook-ceph-cluster chart.
fixes: #16742
Signed-off-by: Michael Chapman <michael@aptira.com>
(cherry picked from commit 2e5dc5a16c)
The toolbox was not respecting the rook-config-override ConfigMap,
resulting in a minimal ceph.conf with only mon_host and keyring settings.
This made it difficult to run diagnostic commands that require custom
configuration parameters.
This change mounts the rook-config-override ConfigMap in all toolbox
variants and merges its contents into the generated ceph.conf. The
merge happens after writing the base config with mon endpoints, and
is re-applied whenever mon endpoints change.
The toolbox now has consistent behavior with other Rook-managed Ceph
daemons (mon, mgr, osd, mds, rgw) which already respect config overrides.
Signed-off-by: Matthew N. Heler <matthew.heler@hotmail.com>
(cherry picked from commit 9570d141de)
Jan Breitkopf
Blaine Gardner
Santosh
subham rai
Joshua Hoblitt
Travis Nielsen
Satoru Takeuchi
Tobias Florek
Michael Adam
Jade Bilkey
Oded Viner
Matthew N. Heler
Manuel Grandeit
subhamkrai
Martin Rowe
Artem Torubarov
Michael Chapman