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

Alias lab AMD server support #149

Closed

Conversation

bengland2
Copy link

@bengland2 bengland2 commented Dec 2, 2020

Do not merge yet, needs some more testing, although badfish accepts this file as syntactically correct.

This change was suggested by Chris Enright to support the new AMD servers in the Alias lab. I came across this problem because these servers were assigned to me in cloud12, and the servers are

e20-{h25-7425,h27-7425,h29-7525,h31-7525}.example.com

@QuantumPosix does this look ok?

@bengland2
Copy link
Author

intended to resolve issue 147

@QuantumPosix
Copy link
Collaborator

QuantumPosix commented Dec 2, 2020 via email

@bengland2
Copy link
Author

-t director now works on both 7525 nodes, after I did a racreset. Previously it only worked on 1 of the 2. However, the boot order displayed does not match the specified boot order (very close though). Why would this happen? Here is the log of the results.

I think this PR can be merged now.

@bengland2
Copy link
Author

bengland2 commented Dec 2, 2020

I got this error below from jetpack while it was trying to set -t director. The badfish error was "Boot order modification is not supported by this host"! So if we can't change the boot order, that's not good, unless we do not need to change boot order, and in that case we shouldn't be requesting it. So which is it? However, a minute later I tried doing exactly what jetpack was doing directly with badfish and got the message that it was already in director mode. Ughh. So I'll try putting a 2-minute delay in the script to see if it was a timing-related thing caused by --racreset.

@QuantumPosix @sadsfae @grafuls what do you think?

EDITING for sanitiation reasons

@sadsfae
Copy link
Member

sadsfae commented Dec 3, 2020

Thanks for submitting this, can you please submit it to the development branch first per:

https://github.com/redhat-performance/badfish#contributing

@sadsfae
Copy link
Member

sadsfae commented Dec 4, 2020

@bengland2 disregard, @QuantumPosix has batched up a PR for this here:

#153

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

Successfully merging this pull request may close these issues.

None yet

3 participants