dashboard
dashboard copied to clipboard
add `data directory` configuration component to advanced tab in rke2 create/edit cluster interface
Summary
Fixes #10824
Occurred changes and/or fixed issues
- add
data directory configuration
component to advanced tab in rke2 create/edit cluster interface - add unit test for
data directory configuration
component
Technical notes summary
- full manual test is dependent on https://github.com/rancher/rancher/issues/45038
Areas or cases that should be tested
- Go to RKE2 cluster creation interface and make sure the
data configuration
area is present on theAdvanced
tab of Cluster Configuration - Make sure data persists and all works fine
- Make sure that cluster editing, the
data configuration
is disabled (not allowed to change the values)
Areas which could experience regressions
Screenshot/Video
NOTE: this video still doesn't cover the edit scenario properly, since we still don't have the changes needed on the backend to persist the data, but it proves the payload sent is correct
https://github.com/rancher/dashboard/assets/97888974/2f09bc61-8c02-4927-8ebf-98257261fc6d
Checklist
- [x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
- [x] The PR has a Milestone
- [x] The PR template has been filled out
- [x] The PR has been self reviewed
- [x] The PR has a reviewer assigned
- [x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
- [x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes
I think even though you are not allowed to change the values later, you should still be able to see the exact values you entered when you created the cluster. For example, I added separate paths for each of the three inputs when I created the cluster but this is what I see in edit mode:
I think even though you are not allowed to change the values later, you should still be able to see the exact values you entered when you created the cluster. For example, I added separate paths for each of the three inputs when I created the cluster but this is what I see in edit mode:
That's because the backend part is still not done. The data doesn't persist yet. It's normal, for now 😬
But they payload is sent correctly. Check the RFC doc linked to the SURE issue where they show the agreed data struct.
I think even though you are not allowed to change the values later, you should still be able to see the exact values you entered when you created the cluster. For example, I added separate paths for each of the three inputs when I created the cluster but this is what I see in edit mode:
That's because the backend part is still not done. The data doesn't persist yet. It's normal, for now 😬
But they payload is sent correctly. Check the RFC doc linked to the SURE issue where they show the agreed data struct.
now that makes sense! 😅
I removed the optional chaining before (I agree it should not be needed) but this caused failures in at least another unit test... I just re-added them to move this forward :P
I removed the optional chaining before (I agree it should not be needed) but this caused failures in at least another unit test... I just re-added them to move this forward :P
It seems you can fix all those failing tests by adding:
dataDirectories: {
systemAgent: '',
provisioning: '',
k8sDistro: '',
}
to: https://github.com/rancher/dashboard/blob/b42c20ed5640d91b20a1f100e79c99a2522c4a50/shell/edit/provisioning.cattle.io.cluster/tests/utils/cluster.ts#L62