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

fb_networkmanager: new cookbook to manage NetworkManager #127

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Commits on Dec 9, 2020

  1. fb_networkmanager: new cookbook to manage NetworkManager

    This cookbook aims to manage NetworkManager in the most controlled
    way possible. It might-could-should be sent over to CPE cookbooks,
    I can redirect over there if preferred.
    
    While, much to my dismay, some people do run NM on servers, and this
    will work for that, it was built to handle this client machine case.
    
    NetworkManager treats its config files as databases and can change them,
    so they can't be handled simply as templates - more finesse is needed.
    
    However, write-once doesn't provide the level of management that
    FB-API-style cookbooks expect. This marries the two. All desired
    settings are managed 100%, and NetworkManager is allowed to make
    additive settings (such as "seen-bssids" or "password" [sic]) so that it
    can continue to function for users.
    
    It is immune to the format changes that NM can make - spacing and
    ordering changes will not cause the file to be updated any more than
    ignorable settings. When updating the file, care is taken to stick
    close to NM's style.
    
    I considered making blacklist/whitelists of fields that NM could
    modify, but in practice since we merge in the user's config, the user's
    config is effectively a blacklist, and whitelist would likely be far
    too fragile.
    
    It has full docs, including on it's Migration Feature which will be
    useful for anyone who had their own Chef to do this but wants to
    migrate.
    
    It also comes with pretty good test coverage.
    
    Signed-off-by: Phil Dibowitz <[email protected]>
    jaymzh committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    7f397ab View commit details
    Browse the repository at this point in the history
  2. code review

    jaymzh committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    15cf94f View commit details
    Browse the repository at this point in the history
  3. review bits

    jaymzh committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    1f84c31 View commit details
    Browse the repository at this point in the history
  4. add defaults feature

    jaymzh committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    c332057 View commit details
    Browse the repository at this point in the history
  5. docs for defaults

    jaymzh committed Dec 9, 2020
    Configuration menu
    Copy the full SHA
    b644633 View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2020

  1. fix doc typos

    jaymzh committed Dec 11, 2020
    Configuration menu
    Copy the full SHA
    aaf6487 View commit details
    Browse the repository at this point in the history