Issue #2718 has been updated by Luke Kanies.
Status changed from Unreviewed to Rejected
This is correct behaviour: If you have any in-language nodes, then all nodes
need to match an internal node specification. If you have external nodes, then
all nodes need to return something.
The two node collections are completely independent.
----------------------------------------
Bug #2718: node default { } applies to all external nodes regardless if they
match a more specific class.
http://projects.reductivelabs.com/issues/2718
Author: W. Andrew Loe III
Status: Rejected
Priority: Normal
Assigned to:
Category:
Target version:
Affected version: 0.24.6
Keywords:
Branch:
The documentation gives the impression that node default { } will only match
nodes that fail to return with a match from the external node classifier. A
change to the wiki seems to resolve the issue that node default must be defined
(http://reductivelabs.com/trac/puppet/wiki/ExternalNodes?action=diff&version=22&old_version=20)
however Andrew Schafer seems to feel that it is necessary.
Some context from #puppet
[4:13pm] andrewcshafer:
I believe regardless of what the external nodes return, you will get whatever
classes are in 'default'
[4:16pm] WALoeIII:
andrewcshafer: so node default {} is always applied regardless of if the node
matches a more specific class
[4:17pm] andrewcshafer:
I believe that is what you are seeing?
[4:17pm] WALoeIII:
yes
[4:17pm] WALoeIII:
just clarifying
[4:17pm] Volcane:
andrewcshafer: sounds like a bug needs to be filed and not a wiki update :
[4:17pm] WALoeIII:
I'm not sure if thats intended
[4:17pm] WALoeIII:
is it a bug or a documentation error is my question.
[4:17pm] andrewcshafer:
I argued that it was a bug before.
[4:17pm] Volcane:
i think its a bug
[4:18pm] WALoeIII:
ok I will file it as a bug and note in the bug that the documentation does not
match current behavior
[4:18pm] Volcane:
extnodes should be an externalised equiv of normal nodes, so default should
behave the same
[4:19pm] andrewcshafer:
WALoeIII: When I was messing with external nodes, Luke said to just make an
empty default. If I'm not mistaken, right now it won't work without one.
[4:20pm] WALoeIII:
ok its now just empty and hanging out
This may be different in different versions of Puppet.
--
You have received this notification because you have either subscribed to it,
or are involved in it.
To change your notification preferences, please click here:
http://reductivelabs.com/redmine/my/account
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Puppet Bugs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/puppet-bugs?hl=en
-~----------~----~----~----~------~----~------~--~---
