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.

Sign up for GitHub

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

Jump to bottom

fix(nvmf): fixing duplicate QID error #1616

Merged
merged 1 commit into from
Apr 3, 2024
Merged

fix(nvmf): fixing duplicate QID error #1616

merged 1 commit into from
Apr 3, 2024

Conversation

dsavitskiy
Copy link
Contributor

  • Updating to the SPDK with the fix
  • A test for many parallel rebuilds added

@dsavitskiy
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Mar 29, 2024
Try #1616:
e339863
dsharma-dc
dsharma-dc approved these changes Mar 29, 2024
View reviewed changes
io-engine/tests/nexus_rebuild_parallel.rs Outdated Show resolved Hide resolved
@bors-openebs-mayastor
Copy link

try

Build succeeded:

@dsavitskiy
Copy link
Contributor Author

bors merge

bors-openebs-mayastor bot pushed a commit that referenced this pull request Apr 2, 2024
@dsavitskiy
Merge #1616
8680636 
1616: fix(nvmf): fixing duplicate QID error r=dsavitskiy a=dsavitskiy

* Updating to the SPDK with the fix
* A test for many parallel rebuilds added

Co-authored-by: Dmitry Savitskiy <dmitry.savitskiy@datacore.com>
@tiagolobocastro
Copy link
Contributor

@dsavitskiy you haven't merged the spdk PR?

@tiagolobocastro
Copy link
Contributor

Make sure the commit hash matches...

@bors-openebs-mayastor
Copy link

Timed out.

@dsavitskiy
fix(nvmf): fixing duplicate QID error
1b00f56 
* Updating to the SPDK with the fix
* A test for many parallel rebuilds added

Signed-off-by: Dmitry Savitskiy <dmitry.savitskiy@datacore.com>
@dsavitskiy
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

👎 Rejected by PR status

@dsavitskiy
Copy link
Contributor Author

bors merge

bors-openebs-mayastor bot pushed a commit that referenced this pull request Apr 3, 2024
@dsavitskiy
Merge #1616
7713ac7 
1616: fix(nvmf): fixing duplicate QID error r=dsavitskiy a=dsavitskiy

* Updating to the SPDK with the fix
* A test for many parallel rebuilds added

Co-authored-by: Dmitry Savitskiy <dmitry.savitskiy@datacore.com>
@bors-openebs-mayastor
Copy link

Timed out.

@dsavitskiy
Copy link
Contributor Author

bors merge

bors-openebs-mayastor bot pushed a commit that referenced this pull request Apr 3, 2024
@dsavitskiy
Merge #1616
7d4419b 
1616: fix(nvmf): fixing duplicate QID error r=dsavitskiy a=dsavitskiy

* Updating to the SPDK with the fix
* A test for many parallel rebuilds added

Co-authored-by: Dmitry Savitskiy <dmitry.savitskiy@datacore.com>
@tiagolobocastro
Copy link
Contributor

bors merge

@bors-openebs-mayastor
Copy link

Already running a review

@tiagolobocastro
Copy link
Contributor

bors cancel

@bors-openebs-mayastor
Copy link

Canceled.

@tiagolobocastro
Copy link
Contributor

bors merge

@bors-openebs-mayastor
Copy link

Build succeeded:

@bors-openebs-mayastor bors-openebs-mayastor bot merged commit f221158 into openebs:develop Apr 3, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hrudaya21 hrudaya21 hrudaya21 approved these changes

@dsharma-dc dsharma-dc dsharma-dc approved these changes

@tiagolobocastro tiagolobocastro Awaiting requested review from tiagolobocastro

@chriswldenyer chriswldenyer Awaiting requested review from chriswldenyer

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dsavitskiy @tiagolobocastro @hrudaya21 @dsharma-dc