-
Notifications
You must be signed in to change notification settings - Fork 1k
vlan and bond support in networkd #6324
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
Conversation
82c09ae to
b1ac8af
Compare
|
@TheRealFalcon @holmanb Can you PTAL at this PR? I'm not sure if the failure is legit. If not, please take this fix if things look good. Thanks. |
89d66a9 to
7576d8f
Compare
TheRealFalcon
left a comment
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.
@sshedi , I left some comments inline, but overall things look good!
Thank you so much for the review and inputs. Tried addressing all your suggestions. PTAL when you get time. |
28317db to
2fc6c0e
Compare
Signed-off-by: Shreenidhi Shedi <[email protected]>
Signed-off-by: Shreenidhi Shedi <[email protected]>
TheRealFalcon
left a comment
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.
LGTM! Thanks!
Networkd renderer used in Photon OS doesn't support vlan and bond settings specified through network config yaml. This is an effort towards supporting the same in networkd renderer. Signed-off-by: Shreenidhi Shedi <[email protected]>
Proposed Commit Message
feat(networkd): support vlan and bond rendering
Additional Context
Test Steps
Merge type