Travis Nielsen

Results 567 comments of Travis Nielsen

Per discussion in the community meeting, sounds like the predicate is canceling the reconcile, but we should consider allowing the reconcile to continue if the generation is not current. However,...

The Squid RC is still not released. While we will plan on adding support for v19 as soon as it is released, let's not block the 1.14 release on it...

Support added with #14454. Tests/CI against v19.1.0 are looking good, so we will continue watching to confirm the tests pass with v19.2.0.

With the network encryption enabled, the v2 ports are enabled. If you change it to false, the v1 ports should be available. ``` encryption: enabled: true ```

You are referring to metrics port 9080? @Rakshith-R @Madhu-1 thoughts?

Encryption is enabled in the helm suite, but the cephfs ms_mode needs to be set in the helm chart. See the [configureRookOperatorViaHelm()](https://github.com/rook/rook/blob/f6509e10f63f67879d9954f6465e0ef62cd28d1e/tests/framework/installer/ceph_helm_installer.go#L55) method and set this under the `values["csi"]` section:...

There is an indentation issue, the `provider` element belongs directly to the `network`, not the `connections`. Try this: ```yaml network: provider: host ```

@Sunnatillo It would be great to get this in, how is testing going?

Development effort is being put more into COSI instead of the OBCs, but contributions are still welcome if you would like to work on it.

@cleanior You set `monitoring.enabled: true` in both of the charts? If only one is set, the needed RBAC won't be created. If you are already setting them, perhaps there is...