Project

General

Profile

Bug #36698

Catch up with API null value changes in legacy UI for iSCSI

Added by William Grzybowski about 1 year ago. Updated 5 months ago.

Status:
Done
Priority:
No priority
Assignee:
Waqar Ahmed
Category:
Middleware
Target version:
Severity:
Medium
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

Description

Changes were made to the API to not allow default null values everywhere.

Legacy UI needs to be update to not send said null values.

Risk
Low risk task since its mostly fixing things currently broken

Acceptance Criteria
Once this is done one should be able to create all kinds of iSCSI entries (specially entries that allow no values to be specified, e.g. no target alias, no auth group, etc)

Associated revisions

Revision 75c05344 (diff)
Added by Waqar Ahmed about 1 year ago

Null values fix for iSCSI plugin

A range of errors was introduced when null was disallowed in middlewared. This commit fixes that and other minor improvements to the code base
Ticket: #36698

Revision 100dc54c (diff)
Added by Waqar Ahmed about 1 year ago

Null values fix for iSCSI plugin

A range of errors was introduced when null was disallowed in middlewared. This commit fixes that and other minor improvements to the code base
Ticket: #36698

History

#1 Updated by William Grzybowski about 1 year ago

  • Status changed from Unscreened to Not Started
  • Target version changed from Backlog to 11.3

#2 Updated by William Grzybowski about 1 year ago

  • Assignee changed from William Grzybowski to Waqar Ahmed

#3 Updated by Waqar Ahmed about 1 year ago

  • Status changed from Not Started to In Progress

#4 Updated by Bug Clerk about 1 year ago

  • Status changed from In Progress to Ready for Testing

#5 Updated by Dru Lavigne 9 months ago

  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#6 Updated by Dru Lavigne 7 months ago

  • Target version changed from 11.3 to 11.3-BETA1

#10 Avatar?id=55038&size=24x24 Updated by Zackary Welch 6 months ago

  • File Screenshot-2019-03-01-16-29-59.png added

Tested minimal options and I saw 'None' choices as well as required fields. However, when I create a portal in the legacy UI, the sharing page is broken. Clicking 'Sharing->Block->Target Global Configuration' does nothing, but going to another area (such as VMs) and going back works. Is this a bug?

#11 Updated by Bonnie Follweiler 6 months ago

  • Status changed from Ready for Testing to Blocked
  • Reason for Blocked set to Waiting for feedback

#12 Updated by Waqar Ahmed 6 months ago

  • Status changed from Blocked to Ready for Testing
  • Reason for Blocked deleted (Waiting for feedback)

#14 Avatar?id=55038&size=24x24 Updated by Zackary Welch 6 months ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

Valid none options include 'Discovery Auth Method', 'Discovery Auth Group', and 'Comment' in Portal, 'Comment' in Initiator, 'Peer User' and 'Peer Secret' in Authorized Access, 'Target Alias', 'Initiator Group ID', 'Auth Method', and 'Authentication Group number' in Targets, 'Available Space Threshold' and 'Comment' in Extents, and 'LUN ID' in Associated Targets.

#15 Avatar?id=55038&size=24x24 Updated by Zackary Welch 6 months ago

  • File deleted (Screenshot-2019-03-01-16-29-59.png)

#16 Updated by Dru Lavigne 6 months ago

  • Status changed from Passed Testing to Done

#17 Updated by Dru Lavigne 5 months ago

  • Target version changed from 11.3-BETA1 to 11.3-ALPHA1

Also available in: Atom PDF