Skip to content

Make the network adapter UI in the web interface more flexible#653

Merged
akuker merged 1 commit intodevelopfrom
rdmark-flexible-network
Feb 8, 2022
Merged

Make the network adapter UI in the web interface more flexible#653
akuker merged 1 commit intodevelopfrom
rdmark-flexible-network

Conversation

@rdmark
Copy link
Copy Markdown
Member

@rdmark rdmark commented Feb 7, 2022

  • Since rascsi now supports attaching SCBR and SCDP to LUNs > 0, add this functionality to the web interface
  • Also, add the Host Bridge (SCBR) as a selectable type, while retaining DaynaPORT (SCDP) as the default choice.

@rdmark rdmark requested a review from bzeiss February 7, 2022 06:24
@bzeiss
Copy link
Copy Markdown
Collaborator

bzeiss commented Feb 7, 2022

@rdmark I'm messing around with this (without Mac for testing). The type selection works for me. I'm not sure how to mount a bridge or daynaport network adapter to a LUN different to 0 though. It always complains "LUNs for device ID 5 are not consecutive". It's probably my lack of understanding how this is actually used though.

What I have figured out is a way to make the web ui crash:

  1. Attach Daynaport to ID 6 (or any free id)
  2. Attach Host Bridge to ID 5 (or any free id)

==> Internal Server Error

@rdmark
Copy link
Copy Markdown
Member Author

rdmark commented Feb 8, 2022

@bzeiss Thanks for testing!

The way LUNs are implemented in RaSCSI is that they have to be concurrent, meaning you have to attach LUN 0 to a SCSI ID first, then LUN 1 on top of that, etc. So the way you can test this is to attach any non-networking type device first at LUN 0, then SCDP or SCBR after that as LUN 1.

As for the crash -- this looks like an issue in rascsi backend. See #652 for which Uwe just fixed and I merged. Can you please test again when you get the chance?

@akuker akuker merged commit 5c0d6fc into develop Feb 8, 2022
@akuker akuker deleted the rdmark-flexible-network branch February 8, 2022 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants