Project

General

Profile

Bug #23933

Call timeouts everywhere

Added by John Hixson over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
No priority
Assignee:
Vaibhav Chauhan
Category:
Middleware
Target version:
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

Since the middleware call timeout option was implemented, stack traces are occurring all over the place with "Call timeout". I have implemented a sysctl module to help this (along with other settings) in src/freenas-sysctl/freenas_sysctl.c. I have also implemented an interface to this sysctl module (freenasUI/common/freenassysctl.py). We need to gradually add new sysctl nodes as we sit fit and set reasonable defaults and set these values throughout the code where middleware calls are made. This ticket is just to keep track of this for FIX branches and/or referencing in commit messages ;-)

Associated revisions

Revision 6a78981a (diff)
Added by John Hixson over 3 years ago

Set & catch call timeouts in directory service form Ticket: #23933

Revision 5488e048 (diff)
Added by John Hixson over 3 years ago

Set & catch call timeouts in directory service form Ticket: #23933 (cherry picked from commit 6a78981abce41896c1624f4438fafdba90713bfc)

History

#1 Updated by John Hixson over 3 years ago

  • Status changed from Screened to Needs Developer Review
  • Assignee changed from John Hixson to Vaibhav Chauhan
  • Target version changed from 11.1 to 11.0-RC3

#2 Updated by Vaibhav Chauhan over 3 years ago

  • Assignee changed from Vaibhav Chauhan to Marcelo Araujo

#3 Updated by Vaibhav Chauhan over 3 years ago

can you please review ?

#4 Updated by Marcelo Araujo over 3 years ago

  • Status changed from Needs Developer Review to Reviewed
  • Assignee changed from Marcelo Araujo to Vaibhav Chauhan

Looks OK.

#5 Updated by Vaibhav Chauhan over 3 years ago

  • Status changed from Reviewed to Merged

#6 Updated by Vaibhav Chauhan over 3 years ago

  • Status changed from Merged to Resolved

Also available in: Atom PDF