foreman_puppet 1.0.0 → 2.0.0.alpha.2

data/db/migrate_foreman/20090722141107_create_environments.rb

@@ -1,11 +1,11 @@
-class CreateEnvironments < ActiveRecord::Migration[5.0]
+class CreateEnvironments < ActiveRecord::Migration[4.2]
   def up
     create_table :environments do |t|
       t.string :name, null: false, limit: 255
       t.timestamps null: true
     end
     create_table :environments_puppetclasses do |t|
-      t.references :puppetclass, foreign_key: true
+      t.references :puppetclass, null: false
       t.references :environment, foreign_key: true
     end
   end

data/db/migrate_foreman/20090802062223_create_puppetclasses.rb

@@ -0,0 +1,28 @@
+class CreatePuppetclasses < ActiveRecord::Migration[4.2]
+  def up
+    create_table :puppetclasses do |t|
+      t.string :name, limit: 255
+      t.string :nameindicator, limit: 255
+      t.integer :operatingsystem_id
+
+      t.timestamps null: true
+    end
+    add_foreign_key(:environments_puppetclasses, :puppetclasses)
+
+    create_table :hosts_puppetclasses, id: false do |t|
+      t.references :puppetclass, null: false
+      t.references :host, null: false
+    end
+
+    create_table :operatingsystems_puppetclasses, id: false do |t|
+      t.references :puppetclass, null: false
+      t.references :operatingsystem, null: false
+    end
+  end
+
+  def down
+    drop_table :puppetclasses
+    drop_table :hosts_puppetclasses
+    drop_table :operatingsystems_puppetclasses
+  end
+end

data/db/migrate_foreman/20090905150132_create_hostgroups_puppetclasses.rb

@@ -0,0 +1,12 @@
+class CreateHostgroupsPuppetclasses < ActiveRecord::Migration[6.0]
+  def up
+    create_table :hostgroups_puppetclasses, id: false, if_not_exists: true do |t|
+      t.references :hostgroup, foreign_key: true, null: false
+      t.references :puppetclass, foreign_key: true, null: false
+    end
+  end
+
+  def down
+    drop_table :hostgroups_puppetclasses
+  end
+end

data/db/migrate_foreman/20110412103238_remove_unused_fields_from_puppet_classes.rb

@@ -0,0 +1,9 @@
+class RemoveUnusedFieldsFromPuppetClasses < ActiveRecord::Migration[4.2]
+  def up
+    remove_column :puppetclasses, :nameindicator
+    remove_column :puppetclasses, :operatingsystem_id
+  end
+
+  def down
+  end
+end

data/db/migrate_foreman/20110712070522_create_host_class.rb

@@ -0,0 +1,11 @@
+class CreateHostClass < ActiveRecord::Migration[4.2]
+  def up
+    rename_table :hosts_puppetclasses, :host_classes
+    add_column :host_classes, :id, :primary_key
+  end
+
+  def down
+    remove_column :host_classes, :id
+    rename_table :host_classes, :hosts_puppetclasses
+  end
+end

data/db/migrate_foreman/20120824142048_add_some_indexes.rb

@@ -0,0 +1,14 @@
+class AddSomeIndexes < ActiveRecord::Migration[4.2]
+  def change
+    # puppetclasses
+    add_index :puppetclasses, :name
+
+    # turn off Foreign Key checks
+    execute 'SET CONSTRAINTS ALL DEFERRED;' if ActiveRecord::Base.connection.adapter_name == 'PostgreSQL'
+
+    add_foreign_key 'host_classes', 'hosts', name: 'host_classes_host_id_fk'
+    add_foreign_key 'host_classes', 'puppetclasses', name: 'host_classes_puppetclass_id_fk'
+    add_foreign_key 'operatingsystems_puppetclasses', 'operatingsystems', name: 'operatingsystems_puppetclasses_operatingsystem_id_fk'
+    add_foreign_key 'operatingsystems_puppetclasses', 'puppetclasses', name: 'operatingsystems_puppetclasses_puppetclass_id_fk'
+  end
+end

data/db/migrate_foreman/20121018152459_create_hostgroup_classes.rb

@@ -0,0 +1,11 @@
+class CreateHostgroupClasses < ActiveRecord::Migration[4.2]
+  def up
+    rename_table :hostgroups_puppetclasses, :hostgroup_classes
+    add_column :hostgroup_classes, :id, :primary_key
+  end
+
+  def down
+    remove_column :hostgroup_classes, :id
+    rename_table :hostgroup_classes, :hostgroups_puppetclasses
+  end
+end

data/db/migrate_foreman/20161205142618_delete_orphaned_smart_class_parameters.rb

@@ -2,7 +2,7 @@ class DeleteOrphanedSmartClassParameters < ActiveRecord::Migration[4.2]
   def up
     condition = 'NOT EXISTS (SELECT 1 FROM environment_classes WHERE environment_classes.puppetclass_lookup_key_id = lookup_keys.id)'
     LookupValue.joins(:lookup_key).where(condition).where("lookup_keys.type = 'PuppetclassLookupKey'").delete_all
-    PuppetclassLookupKey.where(condition).delete_all
+    LookupKey.where(condition).where(type: 'PuppetclassLookupKey').delete_all
   end
 
   def down

data/db/migrate_foreman/20180816134832_cast_lookup_key_values.rb

@@ -1,35 +1,5 @@
 class CastLookupKeyValues < ActiveRecord::Migration[5.1]
   def up
-    # Different LookupKey types handle casting a bit differently
-    PuppetclassLookupKey.unscoped.preload(:lookup_values).where(override: true).where.not(key_type: 'string').find_each do |key|
-      cast_key_and_values(key)
-    end
-  end
-
-  private
-
-  def cast_key_and_values(key)
-    fix_value(key, :default_value)
-    key.lookup_values.each do |lv|
-      fix_value(lv, :value)
-    end
-  end
-
-  def safemode_box
-    @safemode_box ||= Safemode::Box.new
-  end
-
-  def fix_value(obj, attribute)
-    return if obj.omit && !obj.try(:merge_default)
-    value = obj.send(attribute)
-    return unless value.is_a? String
-    return if value.contains_erb?
-    fixed = safemode_box.eval(value)
-    obj.update_column(attribute, fixed)
-  rescue StandardError => e
-    say "Failed to cast #{attribute} for #{obj.inspect}:"
-    say "Value: #{value}", subitem: true
-    say "Error: #{e.message}", subitem: true
-    say 'Perhaps it is invalid? Casting skipped, manual action may be needed.', subitem: true
+    # Do nothing, if this is upgrade from too old version, we are not supporting it.
   end
 end

data/db/migrate_foreman/20180831115634_add_uniqueness_to_puppetclass_name.rb

@@ -0,0 +1,24 @@
+class AddUniquenessToPuppetclassName < ActiveRecord::Migration[5.1]
+  class FakePuppetclass < ApplicationRecord
+    self.table_name = 'puppetclasses'
+  end
+
+  def up
+    names = FakePuppetclass.group(:name).count.select { |_key, value| value > 1 }.keys
+    unless names.empty?
+      names.each do |name|
+        classes = FakePuppetclass.where name: name
+        say "#{classes.count} Puppet classes with duplicate name detected: #{name}"
+      end
+      raise 'Please make sure there are no duplicate Puppet classes before continuing.'
+    end
+
+    remove_index :puppetclasses, :name
+    add_index :puppetclasses, :name, unique: true
+  end
+
+  def down
+    remove_index :puppetclasses, :name
+    add_index :puppetclasses, :name
+  end
+end

data/db/migrate_foreman/20181023112532_add_environment_puppetclass_id.rb

@@ -2,6 +2,6 @@
 class AddEnvironmentPuppetclassId < ActiveRecord::Migration[5.2]
   def change
     add_index :environment_classes, %i[environment_id puppetclass_id]
-    remove_index :environment_classes, :environment_id
+    remove_index(:environment_classes, :environment_id) if index_exists?(:environment_classes, :environment_id)
   end
 end

data/lib/foreman_puppet/engine.rb

@@ -1,38 +1,9 @@
 module ForemanPuppet
   class Engine < ::Rails::Engine
-    config.before_configuration do
-      unless ForemanPuppet.extracted_from_core?
-        require 'graphql'
-
-        module BaseObjectClassMethodPath
-          def field(*args, **kwargs, &block)
-            return if args.first == :environment && args.second.to_s == 'Types::Environment'
-            return if args.first == :environments && args.second.node_type.to_s == 'Types::Environment'
-            return if args.first == :puppetclass && args.second.to_s == 'Types::Puppetclass'
-            return if args.first == :puppetclasses && args.second.node_type.to_s == 'Types::Puppetclass'
-
-            super
-          end
-        end
-
-        module RelayClassicMutationClassMethodPath
-          # rubocop:disable Metrics/ParameterLists
-          def argument(name, type, *rest, loads: nil, **kwargs, &block)
-            # rubocop:enable Metrics/ParameterLists
-            return if [::Types::Environment, ::Types::Puppetclass].include?(loads)
-
-            super
-          end
-        end
-
-        GraphQL::Types::Relay::BaseObject.extend(BaseObjectClassMethodPath)
-        GraphQL::Schema::RelayClassicMutation.extend(RelayClassicMutationClassMethodPath)
-      end
-    end
-
     engine_name 'foreman_puppet'
     isolate_namespace ForemanPuppet
 
+    config.paths['db/migrate'] << 'db/migrate_foreman' if Gem::Dependency.new('', ">= #{ForemanPuppet::FOREMAN_DROP_MIGRATIONS_VERSION}").match?('', SETTINGS[:version].notag)
     config.paths['config/routes.rb'].unshift('config/api_routes.rb')
 
     initializer 'foreman_puppet.register_plugin', before: :finisher_hook do |_app|
@@ -79,6 +50,8 @@ module ForemanPuppet
       ::TemplateCombination.include ForemanPuppet::Extensions::TemplateCombination
       ::ProvisioningTemplate.include ForemanPuppet::Extensions::ProvisioningTemplate
 
+      ::HostCounter.prepend ForemanPuppet::Extensions::HostCounter
+
       ::Api::V2::BaseController.include ForemanPuppet::Extensions::ApiBaseController
       ::Api::V2::HostsController.include ForemanPuppet::Extensions::ApiV2HostsController
       ::Api::V2::HostgroupsController.include ForemanPuppet::Extensions::ApiHostgroupsController
@@ -93,15 +66,12 @@ module ForemanPuppet
 
       ::SmartProxiesHelper::TABBED_FEATURES << 'Puppet'
 
-      unless ForemanPuppet.extracted_from_core?
-        ::HostInfo.local_entries.delete('HostInfoProviders::PuppetInfo'.safe_constantize)
-        ::HostInfo.local_entries.delete('HostInfoProviders::ConfigGroupsInfo'.safe_constantize)
-        ::ProxyStatus.status_registry.delete('ProxyStatus::Puppet'.safe_constantize)
-        Foreman.input_types_registry.input_types.delete('puppet_parameter')
-      end
       Foreman.input_types_registry.register(ForemanPuppet::InputType::PuppetParameterInput)
       ::ProxyStatus.status_registry.add(ForemanPuppet::ProxyStatus::Puppet)
 
+      # Extend smart_proxies API functionality
+      ::Api::V2::SmartProxiesController.include ForemanPuppet::Extensions::ApiSmartProxiesController
+
       # GraphQL
       ::Types::Host.include(ForemanPuppet::Types::HostExtensions)
       ::Types::Hostgroup.include(ForemanPuppet::Types::HostgroupExtensions)

data/lib/foreman_puppet/register.rb

@@ -1,34 +1,15 @@
 Foreman::Plugin.register :foreman_puppet do
-  requires_foreman '>= 2.0.0'
+  requires_foreman '>= 3.0.0'
   # Add Global JS file for extending foreman-core components and routes
   register_global_js_file 'fills'
 
   apipie_documented_controllers(["#{ForemanPuppet::Engine.root}/app/controllers/foreman_puppet/api/v2/*.rb"])
 
-  unless ForemanPuppet.extracted_from_core?
-    # Remove core permissions
-    cfgs = %i[view_config_groups create_config_groups edit_config_groups destroy_config_groups]
-    plks = %i[view_external_parameters create_external_parameters edit_external_parameters
-              destroy_external_parameters]
-    pcls = %i[view_puppetclasses create_puppetclasses edit_puppetclasses destroy_puppetclasses import_puppetclasses]
-    (cfgs | plks | pcls).each do |perm_name|
-      p = Foreman::AccessControl.permission(perm_name)
-      Foreman::AccessControl.remove_permission(p)
-    end
-
-    delete_menu_item(:top_menu, :puppetclasses)
-    delete_menu_item(:top_menu, :config_groups)
-    delete_menu_item(:top_menu, :puppetclass_lookup_keys)
-    delete_menu_item(:top_menu, :environments)
-  end
-
   # TODO: maybe this would not be necessary if we rething the form
   %i[create_hostgroups edit_hostgroups].each do |perm|
     p = Foreman::AccessControl.permission(perm)
-    if ForemanPuppet.extracted_from_core?
-      p.actions << 'hostgroups/environment_selected'
-      p.actions << 'hostgroups/puppetclass_parameters'
-    end
+    p.actions << 'hostgroups/environment_selected'
+    p.actions << 'hostgroups/puppetclass_parameters'
     p.actions << 'foreman_puppet/puppetclasses/parameters'
   end
   p = Foreman::AccessControl.permission(:edit_hostgroups)
@@ -37,15 +18,13 @@ Foreman::Plugin.register :foreman_puppet do
   end
   %i[create_hosts edit_hosts].each do |perm|
     p = Foreman::AccessControl.permission(perm)
-    if ForemanPuppet.extracted_from_core?
-      p.actions << 'hosts/hostgroup_or_environment_selected'
-      p.actions << 'hosts/puppetclass_parameters'
-      if perm == :edit_hosts
-        p.actions << 'hosts/select_multiple_environment'
-        p.actions << 'hosts/update_multiple_environment'
-        p.actions << 'hosts/select_multiple_puppet_proxy'
-        p.actions << 'hosts/update_multiple_puppet_proxy'
-      end
+    p.actions << 'hosts/hostgroup_or_environment_selected'
+    p.actions << 'hosts/puppetclass_parameters'
+    if perm == :edit_hosts
+      p.actions << 'hosts/select_multiple_environment'
+      p.actions << 'hosts/update_multiple_environment'
+      p.actions << 'hosts/select_multiple_puppet_proxy'
+      p.actions << 'hosts/update_multiple_puppet_proxy'
     end
     p.actions << 'foreman_puppet/puppetclasses/parameters'
   end
@@ -168,13 +147,13 @@ Foreman::Plugin.register :foreman_puppet do
   register_facet ForemanPuppet::HostPuppetFacet, :puppet do
     configure_host do
       # extend_model ForemanPuppet::Extensions::Host
-      api_view list: 'foreman_puppet/api/v2/host_puppet_facets/base',
+      api_view list: 'foreman_puppet/api/v2/host_puppet_facets/host_list',
         single: 'foreman_puppet/api/v2/host_puppet_facets/host_single'
       template_compatibility_properties :environment, :environment_id, :environment_name
       set_dependent_action :destroy
     end
     configure_hostgroup(ForemanPuppet::HostgroupPuppetFacet) do
-      api_view list: 'foreman_puppet/api/v2/hostgroup_puppet_facets/base',
+      api_view list: 'foreman_puppet/api/v2/hostgroup_puppet_facets/hostgroup_list',
         single: 'foreman_puppet/api/v2/hostgroup_puppet_facets/hostgroup_single'
       template_compatibility_properties :environment, :environment_id, :environment_name
       set_dependent_action :destroy
@@ -185,14 +164,6 @@ Foreman::Plugin.register :foreman_puppet do
     base_scope.preload(puppet: :environment)
   end
 
-  unless ForemanPuppet.extracted_from_core?
-    Rails.application.config.after_initialize do
-      list = Pagelets::Manager.instance.instance_variable_get(:@pagelets)['hosts/_form'][:main_tabs]
-      core_pagelet = list.detect { |pagelet| pagelet.opts[:id] == :puppet_klasses }
-      list.delete(core_pagelet)
-    end
-  end
-
   register_graphql_query_field :environment, 'ForemanPuppet::Types::Environment', :record_field
   register_graphql_query_field :environments, 'ForemanPuppet::Types::Environment', :collection_field
   register_graphql_query_field :puppetclass, 'ForemanPuppet::Types::Puppetclass', :record_field
@@ -212,12 +183,10 @@ Foreman::Plugin.register :foreman_puppet do
         priority: 100,
         onlyif: (host_onlyif if resource_type == :host)
 
-      if ForemanPuppet.extracted_from_core?
-        context.add_pagelet :main_tab_fields,
-          partial: 'hosts/foreman_puppet/form_main_tab_fields',
-          resource_type: resource_type,
-          priority: 100
-      end
+      context.add_pagelet :main_tab_fields,
+        partial: 'hosts/foreman_puppet/form_main_tab_fields',
+        resource_type: resource_type,
+        priority: 100
     end
   end
 end

data/lib/foreman_puppet/version.rb

@@ -1,3 +1,3 @@
 module ForemanPuppet
-  VERSION = '1.0.0'.freeze
+  VERSION = '2.0.0.alpha.2'.freeze
 end

data/lib/foreman_puppet.rb

@@ -1,10 +1,5 @@
 module ForemanPuppet
-  FOREMAN_EXTRACTION_VERSION = '3.0'.freeze
-
-  def self.extracted_from_core?
-    ENV['PUPPET_EXTRACTED'].to_s == '1' ||
-      Gem::Dependency.new('', ">= #{FOREMAN_EXTRACTION_VERSION}").match?('', SETTINGS[:version].notag)
-  end
+  FOREMAN_DROP_MIGRATIONS_VERSION = '3.1'.freeze
 end
 
 require 'foreman_puppet/engine'

data/locale/en/foreman_puppet.po

@@ -4,7 +4,7 @@
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: foreman_puppet 0.0.2\n"
+"Project-Id-Version: foreman_puppet 1.0.0.pre\n"
 "Report-Msgid-Bugs-To: \n"
 "PO-Revision-Date: 2014-08-20 08:54+0100\n"
 "Last-Translator: Foreman Team <foreman-dev@googlegroups.com>\n"
@@ -66,10 +66,13 @@ msgstr ""
 msgid "Add a Puppet class to host group"
 msgstr ""
 
+msgid "Add:"
+msgstr ""
+
 msgid "All environments - (not filtered)"
 msgstr ""
 
-msgid "Allow assigning Puppet environmets and classes to the Foreman Hosts."
+msgid "Allow assigning Puppet environments and classes to the Foreman Hosts."
 msgstr ""
 
 msgid "Available Classes"
@@ -84,12 +87,33 @@ msgstr ""
 msgid "Can't find a valid Proxy with a Puppet feature"
 msgstr ""
 
+msgid "Cancel"
+msgstr ""
+
 msgid "Change Environment"
 msgstr ""
 
 msgid "Change Puppet Master"
 msgstr ""
 
+msgid "Changed environments"
+msgstr ""
+
+msgid "Check/Uncheck all"
+msgstr ""
+
+msgid "Check/Uncheck all %s changes"
+msgstr ""
+
+msgid "Check/Uncheck new"
+msgstr ""
+
+msgid "Check/Uncheck obsolete"
+msgstr ""
+
+msgid "Check/Uncheck updated"
+msgstr ""
+
 msgid "Classes"
 msgstr ""
 
@@ -273,9 +297,6 @@ msgstr ""
 msgid "If true, will raise an error if there is no default value and no matcher provide a value"
 msgstr ""
 
-msgid "If you are planning to use Foreman as an external node classifier you should provide information about one or more environments. This information is commonly imported from a pre-existing Puppet configuration by the use of the %{link_start}Puppet classes and environment importer%{link_end}."
-msgstr ""
-
 msgid "Ignored classes in the environments: %s"
 msgstr ""
 
@@ -288,6 +309,9 @@ msgstr ""
 msgid "Ignored environments: %s"
 msgstr ""
 
+msgid "Ignored:"
+msgstr ""
+
 msgid "Import"
 msgstr ""
 
@@ -324,9 +348,6 @@ msgstr ""
 msgid "Invalid proxy selected!"
 msgstr ""
 
-msgid "Learn more about this in the documentation."
-msgstr ""
-
 msgid "List all Puppet class IDs for host"
 msgstr ""
 
@@ -399,6 +420,9 @@ msgstr ""
 msgid "Name"
 msgstr ""
 
+msgid "New"
+msgstr ""
+
 msgid "No changes to your environments detected"
 msgstr ""
 
@@ -435,6 +459,9 @@ msgstr ""
 msgid "Number of classes"
 msgstr ""
 
+msgid "Obsolete"
+msgstr ""
+
 msgid "Omit"
 msgstr ""
 
@@ -447,6 +474,9 @@ msgstr ""
 msgid "Operating system default"
 msgstr ""
 
+msgid "Operation"
+msgstr ""
+
 msgid "Optional comma-delimited stringcontaining either 'new', 'updated', or 'obsolete'that is used to limit the imported Puppet classes"
 msgstr ""
 
@@ -504,9 +534,15 @@ msgstr ""
 msgid "Puppet ENC"
 msgstr ""
 
+msgid "Puppet Environment"
+msgstr ""
+
 msgid "Puppet Environments"
 msgstr ""
 
+msgid "Puppet Modules"
+msgstr ""
+
 msgid "Puppet YAML"
 msgstr ""
 
@@ -543,12 +579,18 @@ msgstr ""
 msgid "Remove this override"
 msgstr ""
 
+msgid "Remove:"
+msgstr ""
+
 msgid "Required parameter without value.<br/><b>Please override!</b><br/>"
 msgstr ""
 
 msgid "Select environment"
 msgstr ""
 
+msgid "Select the changes you want to apply to Foreman"
+msgstr ""
+
 msgid "Set parameters to defaults"
 msgstr ""
 
@@ -629,6 +671,9 @@ msgstr ""
 msgid "To update the class signature, go to the Puppet Classes page and select \"Import\"."
 msgstr ""
 
+msgid "Toggle"
+msgstr ""
+
 msgid "Total"
 msgstr ""
 
@@ -641,6 +686,9 @@ msgstr ""
 msgid "Unable to generate output, Check log files"
 msgstr ""
 
+msgid "Update"
+msgstr ""
+
 msgid "Update a Puppet class"
 msgstr ""
 
@@ -656,6 +704,12 @@ msgstr ""
 msgid "Update an override value for a specific smart class parameter"
 msgstr ""
 
+msgid "Update:"
+msgstr ""
+
+msgid "Updated"
+msgstr ""
+
 msgid "Updated hosts: changed environment"
 msgstr ""