-
Notifications
You must be signed in to change notification settings - Fork 40
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
CSI-Addons + Rook doesn't work anymore due to authentication being enabled by default #768
Comments
@SkalaNetworks you might be using the csi-addons unreleased version, In Rook we support only know released version, This is already fixed in Rook master branch. |
@Madhu-1 Do you know if there's a release planned soon? It's kinda hard running a production IPv6 cluster right now, I'd like to switch to non-canary images to be on the safe side. Same goes for the TLS bug which is problematic when running those versions. |
TLS problem is fixed in Rook. @nixpanic @Rakshith-R we can plan for a patch release this week or early next week? |
fyi: https://github.com/csi-addons/kubernetes-csi-addons/releases/tag/v0.12.0 has been released earlier this week. |
The documentation doesn't mention the authentication being enabled by default.
This commit dd9be53 enabled it a month ago, whereas it was disabled before.
The Rook serviceaccount is not equipped with sufficient RBAC for TolenReviews, which makes the documentations outdated and the setup not working out of the box.
Example message on the CSIAddonNode when it is trying to connect to the ceph-csi sidecars deployed by Rook:
I don't know what the rationale is for authentication, it probably is a good thing, but it needs to be documented. The change will break downstream CSIs that use the sidecar, possibly breaking the addons on upgrades of CSI-Addons.
The text was updated successfully, but these errors were encountered: