-
Notifications
You must be signed in to change notification settings - Fork 14
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
Remove assumption that CoaL external NIC is e1000g0 #2
Conversation
function get_mac | ||
{ | ||
local mac | ||
|
||
mac=$(dladm show-phys -mpo address e1000g0) | ||
[[ $? -eq 0 ]] || fatal "failed to run dladm" | ||
mac=$(nictagadm list | awk '/^external/{print $2}') |
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.
Shouldn't we be using the nictagadm parseable form here, eg. -p?
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.
Yes. If I'd noticed it existed I would have. Clearly I should update this. Thanks!
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.
Something like this?
nictagadm list -p -d ' ' | awk '/^external /{print $2}'
Let me know if that looks good and I'll squash and repush. |
Old commits were nshalman@f1a88fd and nshalman@ac3ef61 |
ping! |
Apparently that code doesn't work... |
@pfmooney fixed it for me: nshalman@e7964af |
Merge pending @nshalman's testing. |
Confirmed that this worked to deploy Manta in a CoaL environment using vioif drivers (using my modified platform image). |
Reviewed by: Patrick Mooney <patrick.mooney@joyent.com>
Merged in c376881. |
While doing some follow-up testing based on https://blog.shalman.org/running-sdc-coal-on-smartos/ I realized two possible issues:
This patch addresses both of those issues.
Feedback requested.