-
Notifications
You must be signed in to change notification settings - Fork 19
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
Integration with krkn-hub #923
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see krknhub_exceptions.py used anywhere?
Otherwise, I'm OK with this.
Yes, currently I didnt use it but its a place holder for the future |
Could you add a comment to it to that effect, then? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
krkn_exceptions.py needs a comment that the file is currently unused but is added as a placeholder for the future.
955f036
to
4d6531c
Compare
You right, added a Base class for all KrknHub error classes |
4d6531c
to
4b8ee41
Compare
4b8ee41
to
5163af0
Compare
5163af0
to
28633af
Compare
28633af
to
7478b22
Compare
Type of change
Note: Fill x in []
Description
krkn-hub integration setup:
'krknhub_workload' e.g. node-scenarios
For more scenarios, see the documentation
'krknhub_environment_variable' e.g.
export CLOUD_TYPE=
export BMC_USER=
export BMC_PASSWORD=
export BMC_ADDR=
export NODE_NAME=
export ENABLE_ES=
export ES_SERVER=
export ES_PORT=
export ES_METRICS_INDEX=
export ES_COLLECT_METRICS=
Save krkn-hub pod logs
For security reasons, all pull requests need to be approved first before running any automated CI