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
mgr/dashboard: osd form preselect db/wal device filters #47574
Conversation
...ard/frontend/src/app/ceph/cluster/inventory/inventory-devices/inventory-devices.component.ts
Show resolved
Hide resolved
|
jenkins test api |
|
jenkins test dashboard |
|
jenkins test make check |
|
make check failure seems related to the PR |
|
jenkins test api |
|
jenkins test dashboard |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
If the hostname is selected for the primary devices, then we can preselect the hostname filter for the db/wal devices because osds will be deployed only on the hostname of the primary device. If preselected it'll be clear that only this devices will be used to deploy. Addition to this, usually ssd devices are used for db/wal devices. So I am preselecting these too in the filters. Fixes: https://tracker.ceph.com/issues/57118 Signed-off-by: Nizamudeen A <nia@redhat.com>
1cd640d
to
5f8eff3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice UX addition!
|
cephadm e2e failure tracked here: https://tracker.ceph.com/issues/57511 |
If the hostname is selected for the primary devices, then we can
preselect the hostname filter for the db/wal devices because osds will
be deployed only on the hostname of the primary device. If preselected
it'll be clear that only this devices will be used to deploy.
Addition to this, usually ssd devices are used for db/wal devices. So I
am preselecting these too in the filters.
scrnli_12_08_2022_21-00-16.mp4
Fixes: https://tracker.ceph.com/issues/57118
Signed-off-by: Nizamudeen A nia@redhat.com
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume toxjenkins test windows