update the team list - add nvmeof component leader#1085
Conversation
|
Site built/updated successfully! https://caroav-patch-1.ceph.io |
idryomov
left a comment
idryomov
left a comment
There was a problem hiding this comment.
I think this is supposed to be an alphabetical list, so it should be placed between Dashboard and RADOS.
| </tr> | ||
| <tr> | ||
| <td><a class="a" href="#Aviv Caro">Aviv Caro</a></td> | ||
| <td>Nvme-oF</td> |
There was a problem hiding this comment.
Nit: Canonical capitalization?
| <td>Nvme-oF</td> | |
| <td>NVMe-oF</td> |
| <td>Seastore</td> | ||
| </tr> | ||
| <tr> | ||
| <td><a class="a" href="#Aviv Caro">Aviv Caro</a></td> |
There was a problem hiding this comment.
The link is going to be dead anyway, so I'd make it explicit:
| <td><a class="a" href="#Aviv Caro">Aviv Caro</a></td> | |
| <td><a class="a" href="#">Aviv Caro</a></td> |
There was a problem hiding this comment.
Looking at the other components it looks like :
<td><a class="a" href="#Name">Name</a></td>
What am I missing?
There was a problem hiding this comment.
Most of them are dead links. This doesn't matter much, but there are also many examples of href="#".
|
FYI - https://tracker.ceph.com/issues/76442 there is some work being done to move the component leads list into docs.ceph.com (along with the code/technical things) so can we close this for now - and open a PR to that list? |
4 participants
No description provided.