search

Isilon / isilon_sdk

Official repository for isilon_sdk. All language bindings are available for download under the 'Releases' tab.
MIT License
50 stars 21 forks source link

Logging and technical debt reduction #42

Closed Atomicology closed 6 years ago

Atomicology commented 6 years ago

This PR accomplishes 4 things to reduce technical debt:

  1. Replaces print statements with configurable logging levels
  2. Corrects invalid conversion of object names from plural to singular
  3. Excludes all non-JSON (e.g. plaintext and HTML) PAPI handlers
  4. Resolves errors that were excluding some PAPI handlers from the SDK

In some cases, an additional 20+ PAPI handlers have been added to the Swagger config. The output below gives examples of how the number of successfully processed and excluded handlers changes as a result of this PR.

OneFS 7.2 previous output:

End points successfully processed: 209, failed to process: 4, excluded: 17.

After fixes:

04:10:13 INFO - End points successfully processed: 210, failed to process: 0, excluded: 20.

OneFS 8.0.0 previous output:

End points successfully processed: 356, failed to process: 26, excluded: 4.

After fixes:

04:10:58 INFO - End points successfully processed: 377, failed to process: 0, excluded: 11.

OneFS 8.0.1 previous output:

End points successfully processed: 391, failed to process: 27, excluded: 4.

After fixes:

04:13:47 INFO - End points successfully processed: 411, failed to process: 0, excluded: 11.

OneFS 8.1.0 previous output:

End points successfully processed: 398, failed to process: 27, excluded: 4.

After fixes:

04:16:31 INFO - End points successfully processed: 418, failed to process: 0, excluded: 11.