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

yum::config fails with comma separated values #21

Closed
matonb opened this issue Dec 7, 2016 · 3 comments
Closed

yum::config fails with comma separated values #21

matonb opened this issue Dec 7, 2016 · 3 comments

Comments

@matonb
Copy link

matonb commented Dec 7, 2016

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 4.8.0
  • Ruby: ruby 2.1.9p490 (2016-03-30 revision 54437) [x86_64-linux]
  • Distribution: OEL Linux 7.3
  • Module version: 0.9.15

How to reproduce (e.g Puppet code you use)

Give a yum::config resources a comma separated value e.g.

yum::config{'exclude':
  ensure => 'kernel, kernel-uek',
}

What are you seeing

Error: /Stage[main]/Profile_base::Linux::Yum/Yum::Config[exclude]/Augeas[yum.conf_main_exclude]: Could not evaluate: Saving failed, see debug

What behaviour did you expect instead

exclude=kernel, kernel-uek

in /etc/yum.conf

Output log

Any additional information you'd like to impart

https://ask.puppet.com/question/17826/using-commas-in-augeas-set-commands/

Maybe accept an array of values

yum::config{'exclude':
  ensure => [ 'kernel', 'kernel-uek' ],
}

So that augeas vaues can be correctly defined ?

@bastelfreak
Copy link
Member

Hi @matonb. thanks for the bugreport. Are you able to provide a patch for this?

@matonb
Copy link
Author

matonb commented Jan 14, 2017

Hi @bastelfreak

Sure.
After poking around with augtool it turns out to be much simpler than anticipated.
See PR #29

@bastelfreak
Copy link
Member

closing this since we pushed the fix

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

No branches or pull requests

2 participants