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

Set mode "raw" to existing non-VLAN interfaces on Debian #94

Merged
merged 2 commits into from
Feb 18, 2015
Merged

Set mode "raw" to existing non-VLAN interfaces on Debian #94

merged 2 commits into from
Feb 18, 2015

Conversation

vholer
Copy link
Contributor

@vholer vholer commented Feb 16, 2015

When reading interfaces configuration on Debian, "raw" mode isn't set for non-VLAN interfaces. Puppet then fixes the mode from undefined to raw again and again and again with every run. E.g.:

Notice: /Stage[main]/Network_config[ib0]/mode: mode changed '' to 'raw'

Following simple patch fixes this behaviour on Debian, including tests.

@jhoblitt
Copy link
Member

This looks good. 👍

@ffrank
Copy link
Contributor

ffrank commented Feb 16, 2015

Can you elaborate on exactly what is going on?

I can find no mention of the mode of an iface in interfaces(5) on sid or wheezy, nor any mention of what raw signifies.

@jhoblitt
Copy link
Member

@ffrank :raw means the interface being configured is not a vlan-raw-device. It could equally have been :plain, :normal, :standard, or :generic but I went with :raw.

@ffrank
Copy link
Contributor

ffrank commented Feb 16, 2015

Thanks @jhoblitt.

👍 from me as well then.

@jhoblitt
Copy link
Member

@ffrank It turns out that even for something as "standardized" as networking, it's difficult to find terms that convey the same meaning across multiple platforms.

@vholer
Copy link
Contributor Author

vholer commented Feb 18, 2015

If you are OK with that, is it possible to merge previous commits so that the resource stops changing mode to raw again and again and again please? Thank you.

adrienthebo added a commit that referenced this pull request Feb 18, 2015
Set mode "raw" to existing non-VLAN interfaces on Debian
@adrienthebo adrienthebo merged commit 250f523 into voxpupuli:master Feb 18, 2015
@adrienthebo
Copy link
Member

Thanks for the contribution!

@vholer
Copy link
Contributor Author

vholer commented Feb 20, 2015

Thank you too, looks good to me. Thanks!

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.

4 participants