voxpupuli / puppet-augeasproviders_sysctl

Augeas-based sysctl type and provider for Puppet
Apache License 2.0
10 stars 36 forks source link

discrepancy between title and name #37

Closed igalic closed 5 years ago

igalic commented 5 years ago

This code doesn't work

sysctl { "4 - Refresh Route config":
  name    => 'net.ipv4.route.flush',
  value   => 1,
  persist => false,
}

"and throws the error: Could not evaluate: net.ipv4.route.flush is not a valid sysctl key"

This code works:

sysctl { "'net.ipv4.route.flush",
  value   => 1,
  persist => false,
}

The reason might stem from net.ipv4.route.flush not showing up in sysctl --all (self.instances)

I realize this is a very special case, that might be better served with an exec, especially since we want a refresh-only here! But i'm submitting it non-the-less as someone might stumble of a similar issue.

raphink commented 5 years ago

Hey @igalic! I'm guessing there's a typo in the working code, where there's an extra ' in the title, correct?

igalic commented 5 years ago

maaaaaaaaaaybe 🙇‍♀