Bug #2207

Dynamic tags are not detected

Added by Stéphan Gorget over 1 year ago. Updated about 1 year ago.

Status:Closed Start:04/28/2009
Priority:Normal Due date:
Assigned to:James Turnbull % Done:

100%

Category:plumbing
Target version:0.25.0
Affected version:0.25.0 Branch:
Keywords:
Votes: 0

Description

Dynamic tags based on type, class name etc are not detected with the —tags option.

This command :

puppetd --no-daemonize --server=cors003.ocre.cea.fr --test --debug --tags package

install package when it is used with 0.24.8 and not with 0.25.

the output is :

debug: //Node[default]/lynx/Package[lynx]: Not tagged with package
debug: //Node[default]/w3m/Package[w3m]: Not tagged with package

History

Updated by Stéphan Gorget over 1 year ago

  • Category set to plumbing

I’ve submitted a patch on the -dev mailing list.

Updated by Stéphan Gorget over 1 year ago

gorget wrote:

I’ve submitted a patch on the -dev mailing list.

It seems that this bug has to be solve within a refactoring of the tag functions that should be in the tagging module and not in the type class.

Updated by Stéphan Gorget about 1 year ago

  • Assigned to changed from Stéphan Gorget to Brice Figureau

Masterzen submitted a patch : http://groups.google.com/group/puppet-dev/browse_thread/thread/d2c836261ee15997

Updated by Brice Figureau about 1 year ago

  • Status changed from Unreviewed to Ready for Checkin
  • Assigned to changed from Brice Figureau to James Turnbull
  • Target version set to 0.25.0
  • % Done changed from 0 to 100

Patch reviewed and accepted.

You can find the code in the tickets/master/2207 branch of my github repository: http://github.com/masterzen/puppet/tree/tickets/master/2207

Updated by James Turnbull about 1 year ago

  • Status changed from Ready for Checkin to Closed

Pushed in commit:“1c4620544520d2b5e2967be8dd525bd584ad3886” in branch master.

Also available in: Atom PDF