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

Allow for the HWADDR configuration line to optionally disabled #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

divansantana
Copy link

Unfortunately in some really unsual environements, the MAC address may not be persistent. In some environments there may even be a valid reason for the MAC address not being persistent.

In our z/vm environment a guest may move from one LPAR to another and boot up with a different MAC address. Hence it's useful to make the HWADDR line in the network config file optional. This doesn't cause an issue, since the SUBCHANNELS line exists in this case.
Ideally our z/vm environment would be configured correctly, in that the MAC address would be persistent but that is another issue altogether.

@razorsedge razorsedge self-assigned this Feb 8, 2015
@razorsedge
Copy link
Owner

Would it be possible to update the README.md, add some rspec tests, and to resubmit this PR against razorsedge:develop?

@anders-larsson
Copy link
Contributor

@divansantana @razorsedge Would it be possible to drive this pull request forward? Would love to have this functionality implemented.

Thanks.

@Phil-Friderici
Copy link

Hej @razorsedge,
I think that #69 is a more complete successor of this PR. It not only features the README update but also brings in full spec tests as addition.

Please have a look at @anders-larsson's PR.
If there should be missing anything, I am happy to contribute whatever is needed :)

Thanks & have phun
Phil

@razorsedge
Copy link
Owner

I will look at all of these PRs tomorrow at the Contributor Summit.

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

Successfully merging this pull request may close these issues.

4 participants