https://www.puppet.com/community logo
Join Slack
Powered by
# puppet
  • j
    no, just a site manifest
  • j
    we're not using an ENC
  • c
    ok… what’s in his node declaration?
  • c
    node ‘your master’ { include stuff }
  • c
    I need to stop using the word MASTER. Old habits die hard.
  • j
    we're not using node declarations like that, just a bunch of 
    if
    statements... much more flexible
  • j
    i.e. we can assign the same classes to all windows machines much easier
  • j
    okay, looks like 
    agent_specified_environment
    is a built in fact for puppet, not facter
  • c
    oh lord… a bunch of if statements….
  • c
    If your host naming is well designed, you can use REGEX to broadly apply classes against all “win” machines
  • v
    or use cert extensions
  • c
    that too
  • c
    a little mre vchepkov-y, but I’ll take it. LOLOL
  • c
    whenever possible keep it simple. as simple as is humanly possible is my standard MO
  • v
    simple as this
    Copy code
    node default {
  if $trusted['extensions']['pp_service'] {
    include $trusted['extensions']['pp_service']
  } else {
    include corp::role::server
  }
}
  • c
    doesn’t seem to have any node declarations
  • c
    I may have misunderstood there
  • j
    yea, that's pretty much the same thing we're doing, just not in the default node... i.e. ther is an if statement checking for windows os, then this
  • v
    that not a good style of organizing puppet code. I would recommend reading on role/profiles
  • c
    I can imagine a number of compilation nightmares this could engender given the right circumstances
  • j
    haven't had a single issue with it
  • c
  • c

    https://youtu.be/v9LB-NX4_KQ▾

  • c
    It’s our job to recommend “best practices” as Puppet defines it. Don’t shoot the messengers.
  • c
    It’s our job to recommend “best practices” as Puppet defines it. Don’t shoot the messengers.
  • j
    we're not using an ENC
  • v
    and not to be picky, but I never do 
    if !
    there is 
    unless
    👍🏼 1
  • c
    you don’t have to
  • c

    https://youtu.be/yZEVxL4TuKw▾

  • j
    yea, just reading the intro to this doc, it says it's not straightforward, and if manifests are working fine then it may not be applicable. We're too small to dedicate anyone to this type of project right now
1...697071...428Latest