Project

General

Profile

Bug #38245

Fix bug in new UI that allowed duplicate LUNs

Added by Mark Guzman about 2 years ago. Updated about 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Waqar Ahmed
Category:
Middleware
Target version:
Severity:
Med High
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No
Tags:

Description

I managed to take down my little test cluster by not noticing that the lun number was defaulted to 0 always. This seems like something that the ui should protect a user from doing as it's a reasonable workflow to add a lun for a zfs snapshot.

Associated revisions

Revision 95b20eff (diff)
Added by Waqar Ahmed about 2 years ago

Lunid bug fix in iscsi plugin This commit fixes the bug when validation errors weren't raised for targettoextent service. Ticket: #38245

History

#1 Updated by Dru Lavigne about 2 years ago

  • Assignee changed from Release Council to Erin Clark
  • Target version changed from Backlog to 11.2-RC2

#2 Updated by Erin Clark about 2 years ago

  • Assignee changed from Erin Clark to Lola Yang

#4 Updated by Dru Lavigne about 2 years ago

  • Target version changed from 11.2-RC2 to 11.2-BETA3

#5 Updated by William Grzybowski about 2 years ago

  • Category changed from GUI (new) to Middleware
  • Assignee changed from Lola Yang to Waqar Ahmed
  • Target version changed from 11.2-BETA3 to 11.2-BETA2
  • Severity changed from New to Med High

Waqar,

We need to make sure middlewared does not accept same lun ID for target to extent. We also to allow null and use the next lun id available.

Thanks!

#6 Updated by Waqar Ahmed about 2 years ago

  • Status changed from Unscreened to In Progress

#7 Updated by Waqar Ahmed about 2 years ago

  • Status changed from In Progress to Ready for Testing

#8 Updated by Dru Lavigne about 2 years ago

  • Subject changed from iscsi ui allows you to duplicate an existing lun to Fix bug in new UI that allowed duplicate LUNs
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#9 Updated by Bonnie Follweiler about 2 years ago

22772

Test Passed in FreeNAS-11.2-MASTER-201807260859

#10 Updated by Dru Lavigne about 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF