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

README: puppet refuses to parse class attributes as strings #247

Merged
merged 2 commits into from
Feb 27, 2025

Conversation

m8t
Copy link

@m8t m8t commented Aug 30, 2024

Pull Request (PR) description

Fixes this kind of error:

Syntax error at 'global_config' (file: /etc/puppetlabs/code/environments/rsyslog/site/profile/manifests/rsyslog.pp, line: 5, column: 5)

     1  # Class: profile::rsyslog
     2  # By default provides client configuration, can optionally provide server configuration
     3  class profile::rsyslog {
     4    class { 'rsyslog::config':
     5      'global_config' => {}
     [...]

Also in this commit a minor typofix in REFERENCE.

@TheMeier
Copy link

TheMeier commented Aug 30, 2024

The typo needs to be corrected here https://github.com/voxpupuli/puppet-rsyslog/blob/master/manifests/config.pp#L63 and the then the REFERENCE.md must be recreated using the rake task https://voxpupuli.org/docs/how_to_run_tests/#referencemd-update

Mike Massonnet and others added 2 commits February 26, 2025 23:48
Fixes this kind of error:

```
Syntax error at 'global_config' (file: /etc/puppetlabs/code/environments/rsyslog/site/profile/manifests/rsyslog.pp, line: 5, column: 5)

     1  # Class: profile::rsyslog
     2  # By default provides client configuration, can optionally provide server configuration
     3  class profile::rsyslog {
     4    class { 'rsyslog::config':
     5      'global_config' => {}
     [...]
```
@kenyon kenyon added bug Something isn't working docs Improvements or additions to documentation labels Feb 27, 2025
@kenyon kenyon merged commit 681144c into voxpupuli:master Feb 27, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants