Skip to content
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

feat: connect over tcp if initiator isn't rdma capable #875

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

dsharma-dc
Copy link
Contributor

@dsharma-dc dsharma-dc commented Oct 14, 2024

  • Added a cli option to control whether fallback is required or not. default kept false. We can make this default true once tested thoroughly. have some setup limitations currently.
  • todo: adding a check(if required) to see if nvme_rdma module is available or not.

@dsharma-dc dsharma-dc force-pushed the connect_tcp_fallback branch 3 times, most recently from 53bf0a8 to 969ad17 Compare October 18, 2024 17:03
nix/overlay.nix Outdated Show resolved Hide resolved
@dsharma-dc
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Oct 22, 2024
@bors-openebs-mayastor
Copy link

try

Build failed:

@dsharma-dc
Copy link
Contributor Author

try

Build failed:

CI failed with Operation not permitted errors on tmpfs disk files. Perhaps some stale things left behind from earlier runs. Retrying once again.

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Oct 22, 2024
@dsharma-dc
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Oct 22, 2024
@bors-openebs-mayastor
Copy link

try

Build failed:

@openebs openebs deleted a comment from bors-openebs-mayastor bot Oct 22, 2024
@dsharma-dc
Copy link
Contributor Author

  • continuous-integration/jenkins/branch

ok, not seeing the infra issue anymore, but this run failed at an unrelated(volume resize) test which is flaky it seems.

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Oct 22, 2024
@bors-openebs-mayastor
Copy link

try

Build succeeded:

Signed-off-by: Diwakar Sharma <diwakar.sharma@datacore.com>
@dsharma-dc
Copy link
Contributor Author

bors merge

@bors-openebs-mayastor
Copy link

🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

@bors-openebs-mayastor
Copy link

Build succeeded:

@bors-openebs-mayastor bors-openebs-mayastor bot merged commit d881059 into develop Oct 22, 2024
4 checks passed
@bors-openebs-mayastor bors-openebs-mayastor bot deleted the connect_tcp_fallback branch October 22, 2024 17:41
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.

4 participants