Bug 1018710 - [RFE] Enhance API documentation
Summary: [RFE] Enhance API documentation
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-node
Version: 3.3
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 3.6.0
Assignee: Ryan Barry
QA Contact: bugs@ovirt.org
URL:
Whiteboard: node
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-14 09:04 UTC by Fabian Deutsch
Modified: 2016-02-10 19:40 UTC (History)
16 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Feature: Enhance API documentation Reason: Result (if any):
Clone Of:
Environment:
Last Closed: 2014-08-06 18:20:25 UTC
oVirt Team: Node


Attachments (Terms of Use)

Description Fabian Deutsch 2013-10-14 09:04:10 UTC
Description of problem:
Currently it might not always be obvious how configuration changes can be achieved using Node's API.
Documentation enhacements can improve the situation.

Comment 1 Fabian Deutsch 2013-10-14 09:41:57 UTC
Hey Yaniwang,

could you explain a bit more in depth where you would like to see what documentation?

Comment 2 Yaning Wang 2013-10-17 01:51:07 UTC
Hey Fabian

Document like the following will helpful for tester

Two top category

one is corresponding to the TUI
another is corresponding to the functions that can't be used through TUI or some other advanced/hidden feature

For the first category

- Status Inerface
  ...

- Network Interface
  ...

- Logging Interface
  
  class Logging()
     
      get_logrotate_max_log_size()
      get_rsyslog_address()

For the second category

provide the API like

- reload TUI with dev/test mode
- reset log output level
- ...

Comment 3 Itamar Heim 2014-06-29 09:03:30 UTC
is this meant as a stable public api? would it survive next-gen efforts?

Comment 4 Fabian Deutsch 2014-07-15 18:17:20 UTC
That is a good question.
To not maintain our own API I'd say we use the logic from anaconda to do the basic system configuration.
Probably work upstream on anaconda/pykickstart to support currently unsupported topics like snmp and netconsole through the ks directives.

But we could also go the other way, and use the API (which this bug is about) and wrap the anaconda parts.

In general it is also the question if Node itself should offer an public API. And if so, on what level it should be, a "Node level" (very high, like puppet) or like on the library level.

At least I am moving out this bug.

Comment 5 Fabian Deutsch 2014-08-06 18:20:25 UTC
More documentation has been added in the mean time, and even more will be added, closing this bug.


Note You need to log in before you can comment on or make changes to this bug.