Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support logging to the system log #294

Closed
andy-maier opened this issue May 10, 2016 · 4 comments
Closed

Support logging to the system log #294

andy-maier opened this issue May 10, 2016 · 4 comments

Comments

@andy-maier
Copy link
Contributor

Add infrastructure to log to the system log.

Also, change the debug prints added in PR #290 / #293 into system log calls.

@KSchopmeyer
Copy link
Collaborator

Move to 0.10.0

@andy-maier andy-maier removed their assignment Jun 1, 2016
@andy-maier
Copy link
Contributor Author

Log these specific situations:

  • Unknown IPARAMVALUE elements for output parameters of operations (e.g. pull)
  • Namespace not set in instance path, when returning instances with path (e.g. ei)

@andy-maier
Copy link
Contributor Author

We agreed not to implement creating logs for such cases, but to issue Python warnings (e.g. see UserWarning in tupleparse)

TODO: Verify what we should do in the cases listed above.

@andy-maier
Copy link
Contributor Author

The situations mentioned above are handled as follows:

  • Unknown IPARAMVALUE elements for output parameters of operations (e.g. pull):
    Actually this is about PARAMVALUE. PR Fixes #919: Added and improved CIM-XML response checks at operation level #1322 introduces checks for that and raises a ParseError in all cases of unexpected elements.

  • Namespace not set in instance path, when returning instances with path (e.g. ei):
    In all cases where returned objects do not have a namespace by design of the CIM-XML protocol, pywbem adds the effective target namespace. In all cases where returned objects must have a namespace by design of the CIM-XML protocol, a missing namespace causes a ParseError to be raised. In all cases where returned objects have an optional namespace, the namespace will be present if returned by the server (ExecQuery is the only such case).

As a result, no further changes are needed. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants