Menu

#839 saflogger command fails with TRYAGAIN in streamOpen

4.3.3
fixed
None
defect
log
tools
4.3
major
2014-04-23
2014-04-08
Hans Feldt
No

The saflogger command lack TRYAGAIN loops which needs to be added.

1 Attachments

Related

Tickets: #442
Tickets: #839
Wiki: ChangeLog-4.3.3
Wiki: ChangeLog-4.4.1

Discussion

  • elunlen

    elunlen - 2014-04-08

    I suppose this means the saflogger tool.

     
  • Hans Feldt

    Hans Feldt - 2014-04-08
    • summary: saflog command fails with TRYAGAIN in streamOpen --> saflogger command fails with TRYAGAIN in streamOpen
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1 +1 @@
    -The saflog command lack TRYAGAIN loops which needs to be added.
    +The saflogger command lack TRYAGAIN loops which needs to be added.
    
     
  • Mathi Naickan

    Mathi Naickan - 2014-04-08

    Are we talking about adding a timeout value as a command line option?

     
  • Mathi Naickan

    Mathi Naickan - 2014-04-08
    • status: unassigned --> assigned
    • assigned_to: Mathi Naickan
    • Part: - --> tools
     
  • elunlen

    elunlen - 2014-04-08

    See also ticket [#442]

     

    Related

    Tickets: #442

  • Mathi Naickan

    Mathi Naickan - 2014-04-10

    The scope of this ticket is to support try again for the APIs used in saflogger. Currently, there is no good use-case/value-add yet seen for supporting a timeout option for saflogger.

     
  • Mathi Naickan

    Mathi Naickan - 2014-04-10
    • status: assigned --> accepted
    • Milestone: 4.4.1 --> 4.3.3
     
  • Hans Feldt

    Hans Feldt - 2014-04-10

    probably related to the problems described in #841
    the queue for all app streams is one(1) message by default.

     
  • Mathi Naickan

    Mathi Naickan - 2014-04-10

    Yeah, i had a discussion with Lennart, the appstreamlimit of 1 is being corrected as a part of [#793]

     

    Related

    Tickets: #793

  • Mathi Naickan

    Mathi Naickan - 2014-04-11
     
  • Mathi Naickan

    Mathi Naickan - 2014-04-11

    Attached is a patch, testing this now.

     
  • elunlen

    elunlen - 2014-04-11

    [#442] is another related ticked that maybe can be fixed together with this one.

     

    Related

    Tickets: #442

  • Mathi Naickan

    Mathi Naickan - 2014-04-14
    • status: accepted --> review
     
  • Mathi Naickan

    Mathi Naickan - 2014-04-23
    • status: review --> fixed
     

Log in to post a comment.