-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
external: add support for rados namespace for rbd ec pools #13769
Conversation
currently addded the support for rados namesapce for rbd ec pools upstream closes: rook#13633 Signed-off-by: parth-gr <[email protected]>
@bauerjs1 any feedback? |
@parth-gr the "missing rados namespace" error is gone! However I am still experiencing another error:
I added some debugging output to that location in the script and it turns out that this request just gets an HTTP 403 response. Not sure if that is somehow related to #13856 (comment) Another thing that looks suspicious to me are some of the client caps, e.g.:
(from the dry-run output) Since |
Can you paste the exact command ou used to run the script and the exact output, And I believe the original issue is fixed and we are good with the pr |
Command: python3 create-external-cluster-resources.py
--format bash
--namespace storage
--v2-port-enable
--restricted-auth-permission true
--run-as-user=client.staging
--k8s-cluster-name staging
--rgw-endpoint rgw.example.org:443
--rgw-skip-tls true
--rbd-data-pool-name ceph-hdd-block
--rbd-metadata-ec-pool-name ceph-hdd-block-metadata
--rados-namespace staging
--cephfs-filesystem-name ceph-hdd-file
--cephfs-data-pool-name ceph-hdd-file-erasure-coded
--subvolume-group ceph-hdd-file-csi-staging Output:
Of course Additional output when using the
|
@bauerjs1 checking again |
@travisn I believe we are good with this fix. |
external: add support for rados namespace for rbd ec pools (backport #13769)
currently addded the support for rados namesapce
for rbd ec pools upstream
closes: #13633
Checklist: