-
Notifications
You must be signed in to change notification settings - Fork 91
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
Feature/nbd #488
Feature/nbd #488
Conversation
Signed-off-by: ben sims <ben.sims@citrix.com>
Signed-off-by: Mark Syms <mark.syms@citrix.com>
Signed-off-by: ben sims <ben.sims@citrix.com>
Signed-off-by: ben sims <ben.sims@citrix.com>
job 2828149 jusr passed |
Storage CBT (suite run 125572) succeeded I had to resubmit the suite CBT-BVT-LVM under job id 2829037 which also passed. Storage CBT Functional (suite run 125573) succeeded apart from cbtnbdaccess which i The initial failures were due to unrelated merge errors in xenrt. |
As we discussed yesterday, this entire area is a bit yucky and crying out for some tidying and refactoring. Let's create an improvement ticket to do that. |
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.
Baring the pre-existing yuckiness of this area of code this looks good to go further.
Ive create CP-33286, to track the code quality issue. |
This has past the Storage BST (125469), the storagecache suite is part of the BST, but im running independently job id 2828149