Issue #6753 has been updated by Nigel Kersten.

Affected Puppet version deleted (2.7.0rc1)

Adding Randall as a watcher for API/UX visibility.
----------------------------------------
Refactor #6753: Redundant application files for interfaces is customer facing 
debt 
https://projects.puppetlabs.com/issues/6753

Author: Markus Roberts
Status: Accepted
Priority: High
Assignee: 
Category: Faces
Target version: Telly
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