Merge pull request #24 from WhatsARanjit/release_polish

Release polish

Author: WhatsARanjit

CHANGELOG.md

@@ -1,3 +1,16 @@
+## 2017-05-12  - Release 0.4.2
+
+### Summary
+
+- Added a `node_manager` face for classifier API
+- Added ability to manage node_groups using SSL or token authentication
+- Added ability to manage node_groups from a remote client
+- Set `https` provider as default
+
+#### Bugfixes
+
+- `classes` attribute Hash is now deep-sorted to maintain idempotency
+
 ## 2017-03-31 - Release 0.4.1
 
 ### Summary

HTTPS.md

@@ -21,9 +21,7 @@ Create and manage PE node groups as resources.
 
 * \*nix operating system
 * Puppet >= 3.7.1
-* [puppetclassify](https://github.com/puppetlabs/puppet-classify) gem
-* [puppetlabs/pe_gem module](https://forge.puppetlabs.com/puppetlabs/pe_gem)
-* NOTE: new `https` provider which doesn't need gem dependency at [HTTPS.md](HTTPS.md)
+* New `https` provider which doesn't need `puppetclassify` gem
 
 ## Classes
 
@@ -119,34 +117,14 @@ node_group { 'PE MCollective':
 
   The UID for the data group. Can be specified by group name or UID.
 
-  Default: `default`
+  Default: `All Nodes`
 
 * `rules`
 
   An array of classification rules.
 
   Default (empty array): `[]`
 
-### Puppet_environment
-
-Enumerate all puppet environments:
-
-* `puppet resource puppet_environment`
-
-Example output for `puppet resource puppet_environment production`
-
-```puppet
-puppet_environment { 'production':
-  ensure => 'present',
-}
-```
-
-#### Puppet_environment parameters
-
-* `name`
-
-  (namevar) Name of the Puppet environment on disk, i.e. the directory name in `$environmentpath`.
-
 ## Functions
 
 ### node_groups()
@@ -157,10 +135,10 @@ Retrieve all or one node_group and its data.
 
 ```puppet
 {
-  "default"=>{
+  "All Nodes"=>{
     "environment_trumps"=>false,
     "parent"=>"00000000-0000-4000-8000-000000000000",
-    "name"=>"default",
+    "name"=>"All Nodes",
     "rule"=>["and", ["~", "name", ".*"]],
     "variables"=>{}, "id"=>"00000000-0000-4000-8000-000000000000",
     "environment"=>"production",
@@ -180,14 +158,14 @@ Retrieve all or one node_group and its data.
 }
 ```
 
-`node_groups('default')` will return:
+`node_groups('All Nodes')` will return:
 
 ```puppet
 {
-  "default"=>{
+  "All Nodes"=>{
     "environment_trumps"=>false,
     "parent"=>"00000000-0000-4000-8000-000000000000",
-    "name"=>"default",
+    "name"=>"All Nodes",
     "rule"=>["and", ["~", "name", ".*"]],
     "variables"=>{}, "id"=>"00000000-0000-4000-8000-000000000000",
     "environment"=>"production",
@@ -228,8 +206,7 @@ See 'puppet man node_manager' or 'man puppet-node_manager' for full help.
 
 ## Things to do
 
-* Remove `puppetclassify` dependency
-* Get feedback on `https` provider, new [HTTPS.md](HTTPS.md)
+* Remove `puppetclassify` classes
 
 ## Maintainers
 

README.md

@@ -2,6 +2,7 @@
 require 'puppet_x/node_manager/common'
 
 Puppet::Type.type(:node_group).provide(:https) do
+  defaultfor :feature => :posix
 
   def initialize(value={})
     super(value)

docs/face.md

@@ -125,7 +125,6 @@ def get_classified(name, expl = false, facts = {}, trusted = {})
     url_array << 'explanation' if expl
     data = facts.merge(trusted)
     res  = do_https(url_array.join('/'), 'POST', data)
-    require 'pry'; binding.pry
     if res.code.to_i != 200
       Puppet.debug("Response code: #{res.code}")
       Puppet.debug("Response message: #{res.body}")

lib/puppet/provider/node_group/https.rb

@@ -1,4 +1,13 @@
-class node_manager{
+class node_manager (
+  $warn = true,
+) {
 
   require node_manager::puppetclassify::install
+
+  if $warn {
+    notify { 'This class is deprecated':
+      message  => 'HTTPS provider no longer needs puppetclassify gem.',
+      loglevel => 'warning',
+    }
+  }
 }

lib/puppet/util/nc_https.rb

@@ -1,15 +1,20 @@
 {
   "name": "WhatsARanjit-node_manager",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "author": "WhatsARanjit",
   "summary": "Create and manage PE Console node groups as resources.",
   "license": "Apache-2.0",
   "source": "https://github.com/WhatsARanjit/prosvcs-node_manager",
   "project_page": "https://github.com/WhatsARanjit/prosvcs-node_manager",
   "issues_url": "https://github.com/WhatsARanjit/prosvcs-node_manager/issues",
-  "dependencies": [ 
-    {"name":"puppetlabs/pe_gem","version_requirement":"0.1.2"}
+  "tags": [
+    "PE",
+    "puppet enterprise",
+    "node group",
+    "node_group",
+    "classification"
   ],
+  "dependencies": [],
   "operatingsystem_support": [
     {
       "operatingsystem": "RedHat",
@@ -35,9 +40,18 @@
     {
       "operatingsystem": "Ubuntu",
       "operatingsystemrelease": [
+        "10.04",
         "12.04",
-        "14.04"
+        "14.04",
+        "16.06"
       ]
     }
-  ]
+  ],
+  "requirements": [
+    {
+      "name": "puppet",
+      "version_requirement": ">= 3.7.1 < 5.0.0"
+    }
+  ],
+  "description": "Node_manager module"
 }