Feature #12259
Puppet module objects should expose their metadata
| Status: | Rejected | Start date: | 01/27/2012 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Kelsey Hightower |
% Done: | 0% |
|
| Category: | - | |||
| Target version: | - | |||
| Affected Puppet version: | Branch: | |||
| Keywords: | geordi | |||
| Votes: | 0 |
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
Updated by Daniel Pittman 4 months 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…)
Updated by Matt Robinson 3 months 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.
Updated by Daniel Pittman 2 months ago
- Target version deleted (
2.7.x)
Updated by Daniel Pittman about 1 month ago
- Assignee changed from Matt Robinson to Kelsey Hightower