Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Alert utility example #446

Merged

Conversation

russellnelsonbriggs
Copy link
Contributor

Description

Provides an example module (alert_util.py) that provides a CLI to summarize Alerts using the SDK

Issues Resolved

[List any issues this PR will resolve. e.g., Fixes #1]

Check List

  • [NA - example only ] New functionality includes testing.
    • [NA - example only ] All tests pass for Python 2.7+ & 3.4+($ tox).
  • [NA ] New functionality has been documented in the README if applicable.
    • [NA ] New functionality has been thoroughly documented in the examples (please include helpful comments).
    • [NA ] New endpoints supported are updated in the endpoints-support.md file.
  • [ NA] Changes are documented in the CHANGELOG.

russellnelsonbriggs and others added 2 commits August 21, 2019 07:52
adding example module for counting\exporting alerts
@russellnelsonbriggs
Copy link
Contributor Author

Hi folks, my build failed in Travis with the following that I am not sure what to do about...
Can anyone help me with this? thank you in advance!

$ pyenv global system 3.6
pyenv: version `3.6' not installed
The command "pyenv global system 3.6" failed and exited with 1 during .

@russellnelsonbriggs
Copy link
Contributor Author

found this. https://travis-ci.community/t/error-pyenv-version-3-6-not-installed-since-a-week/1400

any chance someone with access to the build env could check into it? thank you!

@sijeesh
Copy link
Collaborator

sijeesh commented Aug 22, 2019

@russellnelsonbriggs, Thank you for your contribution.
Let me try to fix this build issue.

@sijeesh
Copy link
Collaborator

sijeesh commented Aug 22, 2019

@russellnelsonbriggs, This issue has been fixed in PR #448 .
Now you can update your branch with the latest changes and push it.

@russellnelsonbriggs
Copy link
Contributor Author

Thank you sir!

added better documentation, general command cleanup
fix for tox error
 examples/alert_util.py:106:20: E999 SyntaxError: invalid syntax
Copy link
Collaborator

@sijeesh sijeesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sijeesh sijeesh merged commit 678d53b into HewlettPackard:release/v4.8.0 Aug 28, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backup - Error in settings.py
3 participants