The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Feature #12259

Puppet module objects should expose their metadata

Added by Matt Robinson about 2 years ago. Updated about 2 years ago.

Status:RejectedStart date:01/27/2012
Priority:NormalDue date:
Assignee:Kelsey Hightower% Done:

0%

Category:-
Target version:-
Affected Puppet version: Branch:
Keywords:geordi

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

Currently some of the metadata for puppet module objects is converted to methods, but not all. All the metadata should be exposed through a metadata method for easy access. This is necessary for a few other pieces of functionality, so I want to get this done and merged sooner rather than later so that different branches don’t implement this differently.

History

#1 Updated by Daniel Pittman about 2 years ago

Matt Robinson wrote:

Currently some of the metadata for puppet module objects is converted to methods, but not all. All the metadata should be exposed through a metadata method for easy access.

Wouldn’t it be better to have individual methods for accessing the parts of the metadata – that way you avoid coupling other code on the data format, which is fairly invisible, rather than on the methods themselves. The latter is much easier to deal with over time.

(Ideally, of course, the object wouldn’t just be a bundle of data, but also the behaviour around the data, so that third parties didn’t need to peek into the data at all…)

#2 Updated by Matt Robinson about 2 years ago

  • Status changed from Accepted to Rejected

I was able to just expose the few extra pieces of metadata that I needed (forge name and dependencies), but we may want a way of accessing arbitrary metadata in the future. Until that need arises, I’m happy to reject this ticket.

#3 Updated by Daniel Pittman about 2 years ago

  • Target version deleted (2.7.x)

#4 Updated by Daniel Pittman about 2 years ago

  • Assignee changed from Matt Robinson to Kelsey Hightower

Also available in: Atom PDF