Issue #6753 has been updated by Luke Kanies.

It's worth pointing out that while there's some amount of extra work here, I've 
also had consistent user feedback that they prefer to at least see enough 
information so that they understand how the code flows.  This is a reasonable 
change as long as it's clear to the user how the commands they type track to 
code -- thus, for instance, it would be a bad idea to also autocreate the 
'interface' files.
----------------------------------------
Refactor #6753: Redundant application files for interfaces is customer facing 
debt 
https://projects.puppetlabs.com/issues/6753

Author: Markus Roberts
Status: Accepted
Priority: Normal
Assignee: 
Category: interfaces
Target version: 2.6.x
Affected Puppet version: 
Keywords: 
Branch: 


The plugin form of interfaces requires a small file for each indirection noun 
of the form:

<pre>
require 'puppet/application/indirection_base'

class Puppet::Application::Catalog < Puppet::Application::IndirectionBase
end
</pre>

These share a directory with 1) base class implementations, 2) the actual 
"interface" implementation.

This has a number of unintended and confusing user-visible consequences (e.g. 
interface_base shows up in "puppet --help" as a command when it is not, etc.) 
and asking users to understand / maintain this structure imposes an unnecessary 
barrier to use / extension of the interface facility.

Therefore, in the integrated version of interfaces:

1) interface_base & indirection_base should be moved to a more appropriate 
location
2) the "boilerplate" files should be auto-discovered / created by an 
appropriate hook in core
3) the interface command should be moved into the application directory with 
agent, etc.
4) this interface-specific application directory should be eliminated





-- 
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://projects.puppetlabs.com/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.

Reply via email to