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

Solved Error 400 on SERVER: Syntax error at 'inherits' at /etc/puppet/modules/dns/manifests/server/options.pp:18:29 #36

Closed
wants to merge 2 commits into from

Conversation

setola
Copy link

@setola setola commented Nov 6, 2013

Hi guys,

I've recently updated my puppet git submodules and found a syntax error on this file:
Syntax error at 'inherits' at /etc/puppet/modules/dns/manifests/server/options.pp:18:29

puppet version: 3.3.1

It comes out when using the forwarders:

  # Forwarders to google dns
  dns::server::options{ '/etc/bind/named.conf.options':
    forwarders => [ '8.8.8.8', '8.8.4.4' ]
  }

@solarkennedy
Copy link
Collaborator

Ah! I guess you can't use inheritance in a define?
I dislike inherits anyway. Looks good to me.

@solarkennedy
Copy link
Collaborator

How do you feel about going the extra mile to write a test for the forwarders part so this can be discovered in the future?

@setola
Copy link
Author

setola commented Nov 7, 2013

Tests are always good, but on this theme I'm still a noob so it will require a good amount of study time

swartel pushed a commit to swartel/puppet-dns that referenced this pull request Dec 14, 2013
@davemorrow
Copy link

I wonder if someone can help me with the use of the DNS module.
I am a relative puppet newbie so bear with me please.
I am trying to set forwarders but I constantly get an error
"Could not match ${cfg_dir}], options.pp (27)

My code is simple.

dns::server::options { '/etc/bind/named.conf.options':
forwarders => [ '8.8.8.8', '8.8.4.4' ]
}

@solarkennedy
Copy link
Collaborator

#40 solved this with tests. Closing.

@solarkennedy
Copy link
Collaborator

@davemorrow I'll help you if you open a separate github issue.

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

Successfully merging this pull request may close these issues.

3 participants