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.

Sign up for GitHub

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

Jump to bottom

Redfish support #75

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Redfish support #75

wants to merge 18 commits into from

Conversation

abays
Copy link
Member

@abays abays commented Nov 14, 2019

Net effect is that it currently is not used unless you manually edit terraform.tfvars and add enable_redfish = true. In other words, nothing should change in the existing IPMI-based experience.

abays added 18 commits November 7, 2019 14:48
@abays
Install python3 and redfishtool during prep
0b21be7
@abays
Redfish working
240f41d
@abays
Install python3 and redfishtool during prep
05e76ef
@abays
Redfish working
7592275
@abays
Fix merge conflict
0189337
@abays
Add missing tf redfish config declaration
e52d620
@abays
Ignore redfishtool return code for workers
71187d5
@abays
Ignore redfishtool return code for workers
9d68469
@abays
Install redfishtool outside of python3 install conditional
2797cb0
@abays
Change order of redfish commands
7eb9d3f
@abays
Install python3 and redfishtool during prep
a6eca7c
@abays
Redfish working
235e4f6
@abays
Add missing tf redfish config declaration
736622a
@abays
Ignore redfishtool return code for workers
3d98792
@abays
Ignore redfishtool return code for workers
21ac8b4
@abays
Install redfishtool outside of python3 install conditional
dfb812e
@abays
Change order of redfish commands
5f97398
@abays
Merge branch 'redfish' of ssh://github.com/redhat-nfvpe/kni-upi-lab i…
d37632c 
…nto redfish
@abays abays requested review from yrobla and atyronesmith November 14, 2019 18:13
abays
abays commented Nov 14, 2019
View reviewed changes
prep_bm_host.sh
) || exit 1
fi

pip3.6 install redfishtool
Copy link
Member Author

Choose a reason for hiding this comment

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

Might need sudo here too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yrobla yrobla Awaiting requested review from yrobla

@atyronesmith atyronesmith Awaiting requested review from atyronesmith

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@abays