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

Added Interface VLAN Support, and support for interface without IP addr #57

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

Conversation

f3rr
Copy link

@f3rr f3rr commented Feb 17, 2015

Hey,
I just added support for "VLAN=yes" for interface file.
There was also a bug, i wasnt able to setup an interface without IP address. Thats also fixed now.

@razorsedge
Copy link
Owner

Hey,
Two issues: 1) Can you split this out into two separate PRs? One of your improvements may already be addressed in issue #48. 2) Resubmit this PR against razorsedge:develop.

@Peter-Git-Hub
Copy link

I need to setup a trunked interface and the fact this functionality has not made it into the main branch means I can't proceed. Can someone please work on this issue.

@razorsedge
Copy link
Owner

@Peter-Git-Hub Does the following not work for you?

Normal interface - VLAN - no IP:

class { 'network::global':
  vlan => 'yes',
}

network::if { 'eth0.330':
  ensure => 'up',
}

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.

3 participants