Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-11606

use llapi_error() for errors in liblustreapi files

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.13.0
    • Lustre 2.12.0
    • 9223372036854775807

    Description

      Change calls to printf(...) and fprintf(stderr, ...) to use llapi_printf() and llapi_error() so that it is possible to override these calls when liblustreapi is linked into some other application.

      The usage of fprintf(fd, ...) in llapi_json_write_list() should not be changed.

      Attachments

        Activity

          [LU-11606] use llapi_error() for errors in liblustreapi files
          pjones Peter Jones added a comment -

          Landed for 2.13

          pjones Peter Jones added a comment - Landed for 2.13

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33877/
          Subject: LU-11606 utils: Call llapi_error() for errors in liblustreapi files
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 4983fa82bb2d26fe631bf32815a92733bf910007

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33877/ Subject: LU-11606 utils: Call llapi_error() for errors in liblustreapi files Project: fs/lustre-release Branch: master Current Patch Set: Commit: 4983fa82bb2d26fe631bf32815a92733bf910007

          Arshad Hussain (arshad.super@gmail.com) uploaded a new patch: https://review.whamcloud.com/33877
          Subject: LU-11606 utils: Call llapi_error() for errors in liblustreapi files
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 9f59037192d8dcf1e55fd365208ef8f9327710bb

          gerrit Gerrit Updater added a comment - Arshad Hussain (arshad.super@gmail.com) uploaded a new patch: https://review.whamcloud.com/33877 Subject: LU-11606 utils: Call llapi_error() for errors in liblustreapi files Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 9f59037192d8dcf1e55fd365208ef8f9327710bb

          People

            arshad512 Arshad Hussain
            adilger Andreas Dilger
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: