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

Cannot manage 2008 R2 localised (french) #769

Closed
jothoma1 opened this issue Jul 26, 2017 · 5 comments
Closed

Cannot manage 2008 R2 localised (french) #769

jothoma1 opened this issue Jul 26, 2017 · 5 comments
Assignees

Comments

@jothoma1
Copy link

Description of problem

  • What did you do?
    Using puppet module to manage my sensu clients
  • What happened?
    On french 2008 R2, sensu-puppet failed. I have some errors :

Could not find group Administrators
(for this one, on a french 2008 R2, it should be Administrateurs

/Stage[main]/Sensu::Package/File[C:/opt/sensu/plugins]: Could not evaluate: Puppet::Util::Log requires a message

  • What did you expect to happen?
    Installing sensu clients and managing them
  • How can someone reproduce the problem?
    Try to deploy sensu on a french 2008 R2 with puppet

Command used and debugging output

  • What does your puppet config look like (including any hiera config)
  • Is this a masterless or master based puppet setup?

Platform and version information

  • Your OS: Windows 2008 R2 French
  • Your Ruby version: 2.4
  • Your version of Puppet: 3.8.7
  • Your version of Sensu: 1.0
  • Your version of this module:

Anything else to add that you think will be helpful?

@ghoneycutt
Copy link
Collaborator

Thank you for this great bug report! I do not have access to Windows 2008 R2 in French. Could you please try changing this line

https://github.com/sensu/sensu-puppet/blob/master/manifests/init.pp#L700

from

$group = 'Administrators'

to

$group = 'Administrateurs'

and reporting back here to see if that fixed it. If so, we can easily solve this for you by making owner/group parameters of the sensu class.

@jothoma1
Copy link
Author

jothoma1 commented Jul 27, 2017

Hi,
Yes it's working when i change to $group = 'Administrateurs'
I have a lot of sensu clients, some are 2012 R2 in english, some are 2008 R2 in french...
Now the "french" ones work but not the "english" ones
Thanks in advance !

@alvagante
Copy link
Collaborator

@jothoma1 is there a fact that can reveal what's the language used on the system?

@ghoneycutt
Copy link
Collaborator

@alvagante instead of making an exception for the french language, let's just expose user and group as parameters with the defaults that we currently use in the code. Then the end users can change the group as they please.

alvagante added a commit to alvagante/sensu-puppet that referenced this issue Sep 7, 2017
ghoneycutt added a commit that referenced this issue Sep 7, 2017
Added sensu_user and sensu_group params to sensu class #769
@ghoneycutt
Copy link
Collaborator

Released in v2.36.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants