katello 3.15.0.1 → 3.15.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: 92aa6e8e0a5a914d6c6351cd79438c043c6949bd2a5e045bf200377e9b4ff804
-  data.tar.gz: b3ce81e811303eb3d96b0db1128579a0c65666c23e072bdb769ce85144870ca4
+SHA1:
+  metadata.gz: 1491d165d0f7f2ba0be7ed2d06f792caad8a4a8d
+  data.tar.gz: 8d7fac36c88ffd9369b8006634c6458e0413b1e6
 SHA512:
-  metadata.gz: d214ba3f5a060b6c83045488b0e83d03ca7e89990ccf5e5bf1f0469b7aa3812e2499fce157ed4a10bffe7d0e20843a85b9d744d3f9c4f0ebdf4a4c61aedcf12a
-  data.tar.gz: 296802f07b77527a7efcc1181d4b2c05aeaaf55551f0db865f0c5c1e3b45ad9f049189436bb2c1d35769ca7ab5470499d2bad343eb798278120e86328b1fb567
+  metadata.gz: 3560ef1c772a5e08504f7aacba65568dc8721ee570c067b4c3e0026f1f308bc3e44812b20226a8b058d577f58bfee12e27f7a95c18ba65097a1388fc16284935
+  data.tar.gz: 00f6596ceb566699ca560236991bf71a7e2f48d5e6b9c037ea0078b31d517ac8c4531f76350d87e0170bfdd32f9282436ed82e030acfd0a762c157318d3fb469

data/app/controllers/katello/api/v2/docker_tags_controller.rb

@@ -25,6 +25,23 @@ module Katello
       end
     end
 
+    api :GET, "/docker_tags/:id/repositories", N_("List of repositories for a docker meta tag")
+    def repositories
+      tag = DockerMetaTag.find(params[:id])
+
+      if tag.repositories.size > 1 #pulp3
+        repos = tag.repositories.non_archived
+      else
+        repos = []
+        tag.related_tags.each do |related|
+          repos << related.repositories.non_archived
+        end
+        repos.flatten!
+      end
+
+      respond_with_template_collection('index', 'repositories', collection: full_result_response(repos))
+    end
+
     private
 
     def find_repositories

data/app/lib/actions/katello/capsule_content/sync_capsule.rb

@@ -31,6 +31,10 @@ module Actions
           end
         end
 
+        def resource_locks
+          :link
+        end
+
         def rescue_strategy
           Dynflow::Action::Rescue::Skip
         end

data/app/lib/actions/katello/host/upload_profiles.rb

@@ -33,16 +33,51 @@ module Actions
           Dynflow::Action::Rescue::Skip
         end
 
+        def self.upload_modules_to_pulp(available_streams, host)
+          query_name_streams = available_streams.map do |profile|
+            ::Katello::ModuleStream.where(profile.slice(:name, :stream))
+          end
+
+          updated_profiles = []
+          unless query_name_streams.empty?
+            query_name_streams = query_name_streams.inject(&:or)
+
+            bound_library_instances = host.content_facet.bound_repositories.map(&:library_instance_or_self)
+            query = ::Katello::ModuleStream.in_repositories(bound_library_instances).
+                                            select(:name, :stream, :version, :context, :arch).
+                                              merge(query_name_streams)
+
+            updated_profiles = query.map do |module_stream|
+              module_stream.slice(:name, :stream, :version, :context, :arch)
+            end
+          end
+
+          # We also need to pass module streams that are not found in the ModuleStream table
+          # but are present on the content host
+          unassociated_profiles = available_streams.select do |profile|
+            updated_profiles.none? { |p| p[:name] == profile[:name] && p[:stream] == profile[:stream] }
+          end
+
+          module_stream_profile = updated_profiles + unassociated_profiles
+
+          unless module_stream_profile.empty?
+            begin
+              ::Katello::Pulp::Consumer.new(host.content_facet.uuid).
+                upload_module_stream_profile(module_stream_profile)
+            rescue RestClient::ResourceNotFound
+              Rails.logger.warn("Host with ID %s was not known to Pulp, continuing" % host.id)
+            end
+          end
+        end
+
         def import_module_streams(payload, host)
           enabled_payload = payload.map do |profile|
-            profile.slice("name", "stream", "version", "context", "arch") if profile["status"] == "enabled"
+            profile.slice("name", "stream", "version", "context", "arch").with_indifferent_access if profile["status"] == "enabled"
           end
           enabled_payload.compact!
 
-          ::Katello::Pulp::Consumer.new(host.content_facet.uuid).upload_module_stream_profile(enabled_payload)
+          UploadProfiles.upload_modules_to_pulp(enabled_payload, host)
           host.import_module_streams(payload)
-        rescue RestClient::ResourceNotFound
-          Rails.logger.warn("Host with ID %s was not known to Pulp, continuing" % input[:host_id])
         end
 
         def import_deb_package_profile(host, profile)
@@ -69,6 +104,7 @@ module Actions
             payload = profiles.dig("deb_package_profile", "deb_packages") || []
             import_deb_package_profile(host, payload)
           else
+            module_streams = []
             profiles.each do |profile|
               payload = profile["profile"]
               case profile["content_type"]
@@ -79,9 +115,13 @@ module Actions
               when "enabled_repos"
                 host.import_enabled_repositories(payload)
               else
-                import_module_streams(payload, host)
+                module_streams << payload
               end
             end
+
+            module_streams.each do |module_stream_payload|
+              import_module_streams(module_stream_payload, host)
+            end
           end
         end
       end

data/app/lib/actions/katello/orphan_cleanup/remove_orphans.rb

@@ -13,6 +13,19 @@ module Actions
                 Actions::Pulp3::Orchestration::OrphanCleanup::RemoveOrphans,
                 proxy)
             end
+            plan_self
+          end
+        end
+
+        def run
+          models = []
+          ::Katello::RepositoryTypeManager.repository_types.each_value do |repo_type|
+            indexable_types = repo_type.content_types_to_index
+            models += indexable_types&.map(&:model_class)
+            models.select! { |model| model.many_repository_associations }
+          end
+          models.each do |model|
+            model.joins("left join katello_#{model.repository_association} on #{model.table_name}.id = katello_#{model.repository_association}.#{model.unit_id_field}").where("katello_#{model.repository_association}.#{model.unit_id_field} IS NULL").destroy_all
           end
         end
       end

data/app/lib/actions/katello/pulp_selector.rb

@@ -3,6 +3,7 @@ module Actions
     module PulpSelector
       def plan_pulp_action(backend_actions, repository, smart_proxy, *args)
         fail "nil smart_proxy passed to PulpSelector" if smart_proxy.nil?
+        smart_proxy.fix_pulp3_capabilities(repository.content_type)
         planned = plan_optional_pulp_action(backend_actions, repository, smart_proxy, *args)
         fail "Could not locate an action for type #{smart_proxy.backend_service_type(repository)}" unless planned
         planned

data/app/lib/katello/errors.rb

@@ -45,6 +45,12 @@ module Katello
       end
     end
 
+    class PulpcoreMissingCapabilities < StandardError
+      def message
+        _("A smart proxy seems to have been refreshed without pulpcore being running.  You may want to ")
+      end
+    end
+
     class ConnectionRefusedException < StandardError; end
 
     class MaxHostsReachedException < StandardError; end

data/app/models/katello/concerns/host_managed_extensions.rb

@@ -106,7 +106,7 @@ module Katello
 
       def import_package_profile(simple_packages)
         found = import_package_profile_in_bulk(simple_packages)
-        sync_package_associations(found.map(&:id))
+        sync_package_associations(found.map(&:id).uniq)
       end
 
       def import_package_profile_in_bulk(simple_packages)
@@ -127,9 +127,14 @@ module Katello
                                           :arch => simple_package.arch)
         end
         InstalledPackage.import(installed_packages, validate: false, on_duplicate_key_ignore: true)
+        #re-lookup all imported to pickup any duplicates/conflicts
+        imported = InstalledPackage.where(:nvrea => installed_packages.map(&:nvrea)).select(:id).to_a
 
-        found << installed_packages
-        found.flatten
+        if imported.count != installed_packages.count
+          Rails.logger.warn("Mismatch found in installed package insertion, expected #{installed_packages.count} but only could find #{imported.count}.  This is most likley a bug.")
+        end
+
+        (found + imported).flatten
       end
 
       def import_enabled_repositories(repos)

data/app/models/katello/concerns/http_proxy_extensions.rb

@@ -8,6 +8,7 @@ module Katello
           :inverse_of => :http_proxy, :dependent => :nullify
         after_update :update_default_proxy_setting
         after_commit :update_repository_proxy_details
+        before_destroy :remove_references_to_proxy
 
         def self.default_global_content_proxy
           if Setting[:content_default_http_proxy]
@@ -16,6 +17,34 @@ module Katello
         end
       end
 
+      def repositories_with_proxy(proxy_id)
+        root_repos = RootRepository.with_selected_proxy(proxy_id)
+
+        if self == HttpProxy.default_global_content_proxy
+          root_repos += RootRepository.with_global_proxy
+        end
+
+        root_repos
+      end
+
+      def remove_references_to_proxy
+        root_repos = repositories_with_proxy(nil).uniq.sort
+
+        setting = Setting.find_by(name: 'content_default_http_proxy')
+        if setting&.value && setting.value == self.name
+          setting.update_attribute(:value, '')
+        end
+
+        unless root_repos.empty?
+          ForemanTasks.async_task(
+            ::Actions::BulkAction,
+            ::Actions::Katello::Repository::Update,
+            root_repos,
+            http_proxy_policy: Katello::RootRepository::GLOBAL_DEFAULT_HTTP_PROXY,
+            http_proxy_id: nil)
+        end
+      end
+
       def update_default_proxy_setting
         changes = self.previous_changes
         if changes.key?(:name)
@@ -32,13 +61,7 @@ module Katello
         changes = self.previous_changes
         if changes.key?(:url) || changes.key?(:username) || changes.key?(:password)
 
-          root_repos = RootRepository.with_selected_proxy(id)
-
-          if self == HttpProxy.default_global_content_proxy
-            root_repos += RootRepository.with_global_proxy
-          end
-
-          repos = root_repos.uniq.collect(&:library_instance)
+          repos = repositories_with_proxy(id).uniq.collect(&:library_instance)
 
           unless repos.empty?
             ForemanTasks.async_task(

data/app/models/katello/concerns/smart_proxy_extensions.rb

@@ -124,6 +124,8 @@ module Katello
           config.ssl_client_key = ::Cert::Certs.ssl_client_key
           config.debugging = true
           config.logger = ::Foreman::Logging.logger('katello/pulp_rest')
+          config.username = self.setting(PULP3_FEATURE, 'username')
+          config.password = self.setting(PULP3_FEATURE, 'password')
         end
       end
 
@@ -152,6 +154,19 @@ module Katello
         end
       end
 
+      def missing_pulp3_capabilities?
+        pulp3_enabled? && self.capabilities(PULP3_FEATURE).empty?
+      end
+
+      def fix_pulp3_capabilities(type)
+        if missing_pulp3_capabilities? && !pulp2_preferred_for_type?(type)
+          self.refresh
+          if self.capabilities(::SmartProxy::PULP3_FEATURE).empty?
+            fail Katello::Errors::PulpcoreMissingCapabilities
+          end
+        end
+      end
+
       def pulp3_repository_type_support?(repository_type, check_pulp2_preferred = true)
         repository_type_obj = repository_type.is_a?(String) ? Katello::RepositoryTypeManager.repository_types[repository_type] : repository_type
         fail "Cannot find repository type #{repository_type}, is it enabled?" unless repository_type_obj
@@ -182,7 +197,7 @@ module Katello
         pulp3_uri!.host
       end
 
-      def pulp3_url(path = '/pulp/api/v3/')
+      def pulp3_url(path = '/pulp/api/v3')
         pulp_url = self.setting(PULP3_FEATURE, 'pulp_url')
         path.blank? ? pulp_url : "#{pulp_url.sub(%r|/$|, '')}/#{path.sub(%r|^/|, '')}"
       end

data/app/models/katello/docker_meta_tag.rb

@@ -107,7 +107,7 @@ module Katello
       if grouped
         grouped_fields = "#{table_name}.name, katello_repositories.root_id"
         ids = distinct.select("ON (#{grouped_fields}) #{table_name}.id").joins(:repositories)
-        where(:id => ids).where("#{self.table_name}.schema1_id in (#{sql}) or #{self.table_name}.schema2_id in (#{sql})")
+        self.where(:id => ids)
       else
         self.where("#{self.table_name}.schema1_id in (#{sql}) or #{self.table_name}.schema2_id in (#{sql})")
       end

data/app/models/katello/installed_package.rb

@@ -4,8 +4,8 @@ module Katello
       allow :nvra, :nvrea, :name
     end
 
-    has_many :hosts, :through => :host_installed_packages, :class_name => "::Host"
     has_many :host_installed_packages, :class_name => "Katello::HostInstalledPackage", :dependent => :destroy, :inverse_of => :installed_package
+    has_many :hosts, :through => :host_installed_packages, :class_name => "::Host"
 
     scoped_search :on => :name, :complete_value => true
     scoped_search :on => :nvrea

data/app/models/katello/repository.rb

@@ -422,8 +422,12 @@ module Katello
       path
     end
 
+    def library_instance_or_self
+      self.library_instance || self
+    end
+
     def generate_repo_path(content_path = nil)
-      _org, _content, content_path = (self.library_instance || self).relative_path.split("/", 3) if content_path.blank?
+      _org, _content, content_path = library_instance_or_self.relative_path.split("/", 3) if content_path.blank?
       content_path = content_path.sub(%r|^/|, '')
       if self.environment
         cve = ContentViewEnvironment.where(:environment_id => self.environment,
@@ -496,7 +500,7 @@ module Katello
         end
       end
       clone = Repository.new(:environment => to_env,
-                     :library_instance => self.library_instance || self,
+                     :library_instance => library_instance_or_self,
                      :root => self.root,
                      :content_view_version => to_version,
                      :saved_checksum_type => checksum_type)

data/app/models/katello/root_repository.rb

@@ -97,7 +97,7 @@ module Katello
     scope :with_no_proxy, -> { where(:http_proxy_policy => RootRepository::NO_DEFAULT_HTTP_PROXY) }
     scope :with_selected_proxy, ->(http_proxy_id) {
       where(:http_proxy_policy => RootRepository::USE_SELECTED_HTTP_PROXY).
-      where("http_proxy_id = ?", http_proxy_id)
+      where(:http_proxy_id => http_proxy_id)
     }
     delegate :redhat?, :provider, :organization, to: :product
 

data/app/models/setting/content.rb

@@ -2,7 +2,7 @@ class Setting::Content < Setting
   #rubocop:disable Metrics/MethodLength
   #rubocop:disable Metrics/AbcSize
 
-  validate :content_default_http_proxy, if: proc { |s| s.name == 'content_default_http_proxy' }
+  validate :content_default_http_proxy, if: proc { |s| s.name == 'content_default_http_proxy' && HttpProxy.table_exists? }
 
   after_save :add_organizations_and_locations_if_global_http_proxy
 

data/app/services/katello/pulp/simple_package.rb

@@ -13,8 +13,8 @@ module Katello
       end
 
       def nvrea
-        if epoch == "0"
-          @nvra
+        if epoch.nil? || epoch.to_s == "0"
+          nvra
         else
           "#{@name}-#{@epoch}:#{@version}-#{@release}.#{@arch}"
         end

data/app/views/foreman/job_templates/install_errata_-_katello_ansible_default.erb

@@ -15,4 +15,4 @@ kind: job_template
 %>
 
 <% advisories = input(:errata).split(',').map { |e| "--advisory=#{e}" }.join(' ') %>
-<%= render_template('Run Command - Ansible Default', :command => "yum update-minimal #{advisories}") %>
+<%= render_template('Run Command - Ansible Default', :command => "yum -y update-minimal #{advisories}") %>

data/app/views/foreman/job_templates/install_group_-_katello_ansible_default.erb

@@ -13,4 +13,4 @@ provider_type: Ansible
 kind: job_template
 %>
 
-<%= render_template('Run Command - Ansible Default', :command => "yum group install #{input('package')}") %>
+<%= render_template('Run Command - Ansible Default', :command => "yum -y group install #{input('package')}") %>

data/app/views/foreman/job_templates/remove_group_-_katello_ansible_default.erb

@@ -13,4 +13,4 @@ provider_type: Ansible
 kind: job_template
 %>
 
-<%= render_template('Run Command - Ansible Default', :command => "yum group remove #{input('package')}") %>
+<%= render_template('Run Command - Ansible Default', :command => "yum -y group remove #{input('package')}") %>

data/app/views/foreman/job_templates/update_group_-_katello_ansible_default.erb

@@ -13,4 +13,4 @@ provider_type: Ansible
 kind: job_template
 %>
 
-<%= render_template('Run Command - Ansible Default', :command => "yum group update #{input('package')}") %>
+<%= render_template('Run Command - Ansible Default', :command => "yum -y group update #{input('package')}") %>

data/app/views/foreman/job_templates/update_package_-_katello_ansible_default.erb

@@ -13,4 +13,4 @@ provider_type: Ansible
 kind: job_template
 %>
 
-<%= render_template('Run Command - Ansible Default', :command => "yum update #{input('package')}") %>
+<%= render_template('Run Command - Ansible Default', :command => "yum -y update #{input('package')}") %>

data/app/views/katello/api/v2/docker_tags/_base.json.rabl

@@ -18,7 +18,7 @@ child :docker_manifest => :manifest do
   attributes :schema_version, :digest, :manifest_type
 end
 
-child :repository => :repository do
+child :repositories => :repositories do
   attributes :id, :name, :full_path
 end
 

data/app/views/katello/api/v2/repositories/base.json.rabl

@@ -57,3 +57,11 @@ end
 child :content_view => :content_view do |_repo|
   attribute :id, :name
 end
+
+child :content_view_version do
+  attributes :id, :name, :content_view_id
+end
+
+child :environment do
+  attributes :id, :name
+end

data/config/routes/api/v2.rb

@@ -163,6 +163,9 @@ Katello::Engine.routes.draw do
             get :auto_complete_name
             get :compare
           end
+          member do
+            get :repositories
+          end
         end
 
         api_resources :environments, :only => [:index, :show, :create, :update, :destroy] do

data/db/migrate/20200429153103_installed_package_bad_nvrea.rb

@@ -0,0 +1,14 @@
+class InstalledPackageBadNvrea < ActiveRecord::Migration[5.2]
+  def up
+    Katello::InstalledPackage.where(:epoch => "0").find_each do |pkg|
+      simple = Katello::Pulp::SimplePackage.new(pkg.attributes)
+      if pkg.nvrea != simple.nvrea
+        pkg.update_column(:nvrea, simple.nvrea)
+      end
+    end
+  end
+
+  def down
+    #noop
+  end
+end

data/db/migrate/20200501155054_installed_package_unique_nvrea.rb

@@ -0,0 +1,64 @@
+class InstalledPackageUniqueNvrea < ActiveRecord::Migration[5.2]
+  def fix_missing_attributes
+    #bug in dynflow may have resulted in old code running and not properly populating fields
+    # This block is basically copied from 20200129172534_add_epoch_version_release_arch_to_katello_installed_packages.rb
+    epoch_non_0 = ::Katello::Rpm.where.not(epoch: [0, nil]).pluck(:nvra, :epoch).to_h
+    installed_packages = []
+    ::Katello::InstalledPackage.where(:nvrea => nil).each do |pkg|
+      epoch = epoch_non_0[pkg.nvra] || "0"
+
+      attributes_hash = ::Katello::Util::Package.parse_nvrea(pkg.nvra)
+      attributes_hash[:epoch] = epoch
+      attributes_hash[:nvra] = pkg.nvra
+      if epoch == "0"
+        attributes_hash[:nvrea] = pkg.nvra
+      else
+        attributes_hash[:nvrea] = "#{pkg.name}-#{epoch}:#{attributes_hash[:version]}-"\
+                                  "#{attributes_hash[:release]}.#{attributes_hash[:arch]}"
+      end
+
+      installed_packages << ::Katello::InstalledPackage.new(attributes_hash)
+    end
+    ::Katello::InstalledPackage.import(installed_packages, validate: false, batch_size: 50_000,
+                                       on_duplicate_key_update: {conflict_target: [:nvra],
+                                                                 columns: [:nvrea, :epoch, :version, :release, :arch]})
+  end
+
+  def consolidate_duplicate_nvreas
+    host_installed_packages = []
+    deletable_installed_package_ids = []
+    Katello::InstalledPackage.having('count(nvrea) > 1').group(:nvrea).pluck(:nvrea).each do |nvrea|
+      found = Katello::InstalledPackage.includes(:host_installed_packages).where(:nvrea => nvrea).to_a
+      to_keep = found.pop
+      found.each do |duplicate|
+        duplicate.host_ids.each do |host_id|
+          host_installed_packages << {:installed_package_id => to_keep.id, :host_id => host_id}
+        end
+        deletable_installed_package_ids << duplicate.id
+      end
+    end
+    if host_installed_packages.any?
+      Katello::HostInstalledPackage.import(host_installed_packages, validate: false, on_duplicate_key_ignore: true)
+    end
+    if deletable_installed_package_ids.any?
+      Katello::HostInstalledPackage.where(installed_package_id: deletable_installed_package_ids).delete_all
+      Katello::InstalledPackage.where(id: deletable_installed_package_ids).delete_all
+    end
+  end
+
+  def up
+    fix_missing_attributes
+    #now there should be no NULL nvreas
+    change_column :katello_installed_packages, :nvrea, :string, :null => false
+
+    consolidate_duplicate_nvreas
+    add_index "katello_installed_packages", [:nvrea], :unique => true
+    remove_index "katello_installed_packages", [:nvra]
+  end
+
+  def down
+    remove_index "katello_installed_packages", [:nvrea]
+    add_index "katello_installed_packages", [:nvra], :unique => true
+    change_column :katello_installed_packages, :nvrea, :string, :null => true
+  end
+end

data/db/seeds.d/111-upgrade_tasks.rb

@@ -12,6 +12,7 @@ UpgradeTask.define_tasks(:katello) do
     {:name => 'katello:upgrades:3.12:remove_pulp2_notifier'},
     {:name => 'katello:upgrades:3.13:republish_deb_metadata'},
     {:name => 'katello:upgrades:3.15:set_sub_facet_dmi_uuid'},
-    {:name => 'katello:upgrades:3.15:reindex_rpm_modular'}
+    {:name => 'katello:upgrades:3.15:reindex_rpm_modular'},
+    {:name => 'katello:upgrades:3.16:update_applicable_el8_hosts'}
   ]
 end

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/docker-tags/details/docker-tag-environments.controller.js

@@ -20,36 +20,24 @@ angular.module('Bastion.docker-tags').controller('DockerTagEnvironmentsControlle
             'sort_order': 'ASC',
             'paged': false
         };
+
         var nutupane = new Nutupane(DockerTag, params, null, {disableAutoLoad: true});
 
         var renderTable = function () {
-            var ids = _.map($scope.tag.related_tags, 'id');
-            var newParams = {
-                'organization_id': CurrentOrganization,
-                'search': $location.search().search || "",
-                'sort_by': 'name',
-                'sort_order': 'ASC',
-                'paged': false,
-                'ids[]': ids
-            };
-            $scope.table = nutupane.table;
-            nutupane.setParams(newParams);
-            $scope.panel.loading = false;
-            if (!_.isEmpty(ids)) {
-                nutupane.refresh();
-            }
+            params.action = 'repositories';
+            params.id = $scope.tag.id;
+
+            nutupane.setParams(params);
+            nutupane.refresh();
         };
 
+        $scope.table = nutupane.table;
+
         $scope.controllerName = 'katello_docker_tags';
 
-        if ($scope.tag) {
+        $scope.tag.$promise.then(function() {
             $scope.panel.loading = false;
-        }
-
-        if ($scope.tag && $scope.tag.related_tags) {
             renderTable();
-        } else {
-            $scope.tag.$promise.then(renderTable);
-        }
+        });
     }
 ]);

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/docker-tags/details/views/docker-tag-environments.html

@@ -27,19 +27,19 @@
 
       <tbody>
         <tr bst-table-row
-            ng-repeat="tag in table.rows | filter:dockerTagFilter">
+            ng-repeat="repo in table.rows | filter:dockerTagFilter">
           <td bst-table-cell>
-            <a ui-sref="environment.details({environmentId: tag.environment.id})">
-              {{ tag.environment.name }}
+            <a ui-sref="environment.details({environmentId: repo.kt_environment.id})">
+              {{ repo.kt_environment.name }}
             </a>
           </td>
           <td bst-table-cell>
-            <a ui-sref="content-view.version.docker({contentViewId: tag.content_view_version.content_view_id, versionId: tag.content_view_version.id})">
-              {{ tag.content_view_version.name }}
+            <a ui-sref="content-view.version.docker({contentViewId: repo.content_view_version.content_view_id, versionId: repo.content_view_version.id})">
+              {{ repo.content_view_version.name }}
             </a>
           </td>
           <td bst-table-cell>
-            {{ tag.repository.full_path }}:{{ tag.name }}
+            {{ repo.full_path }}:{{ tag.name }}
           </td>
         </tr>
       </tbody>

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/docker-tags/docker-tags.controller.js

@@ -44,5 +44,13 @@ angular.module('Bastion.docker-tags').controller('DockerTagsController',
             }
             return versions.join(", ");
         };
+
+        $scope.getRepositoryNames = function (tag) {
+            return tag.repositories.map(function(repo) {
+                return repo.name;
+            }).filter(function(item, index, names) {
+                return names.indexOf(item) >= index;
+            }).sort().join(', ');
+        };
     }]
 );

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/docker-tags/docker-tags.factory.js

@@ -10,7 +10,7 @@
 angular.module('Bastion.docker-tags').factory('DockerTag',
     ['BastionResource', function (BastionResource) {
 
-        return BastionResource('katello/api/v2/docker_tags/:id/',
+        return BastionResource('katello/api/v2/docker_tags/:id/:action/',
             {id: '@id'},
             {
                 autocomplete: {method: 'GET', isArray: true, params: {id: 'auto_complete_search'}},

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/docker-tags/views/docker-tags.html

@@ -39,7 +39,7 @@
             {{ tag.product.name }}
           </td>
           <td bst-table-cell>
-            {{ tag.repository.name }}
+            {{ getRepositoryNames(tag) }}
           </td>
         </tr>
       </tbody>

data/lib/katello/permission_creator.rb

@@ -258,7 +258,12 @@ module Katello
                            'katello/api/v2/package_groups' => [:index, :show, :auto_complete_search, :compare],
                            'katello/api/v2/docker_manifests' => [:index, :show, :auto_complete_search, :compare],
                            'katello/api/v2/docker_manifest_lists' => [:index, :show, :auto_complete_search, :compare],
-                           'katello/api/v2/docker_tags' => [:index, :show, :auto_complete_search, :auto_complete_name, :compare],
+                           'katello/api/v2/docker_tags' => [:index,
+                                                            :show,
+                                                            :auto_complete_search,
+                                                            :auto_complete_name,
+                                                            :compare,
+                                                            :repositories],
                            'katello/api/v2/file_units' => [:index, :show, :auto_complete_search, :compare],
                            'katello/api/v2/ostree_branches' => [:index, :show, :auto_complete_search, :compare],
                            'katello/api/v2/errata' => [:index, :show, :auto_complete_search, :compare],

data/lib/katello/tasks/common.rake

@@ -0,0 +1,7 @@
+namespace :katello do
+  task :disable_dynflow do
+    #Ensures that we run as a dynflow 'client'
+    ::Rails.application.dynflow.initialize!
+    ForemanTasks.dynflow.config.remote = true
+  end
+end

data/lib/katello/tasks/delete_orphaned_content.rake

@@ -1,7 +1,9 @@
+load "#{Katello::Engine.root}/lib/katello/tasks/common.rake"
+
 namespace :katello do
   desc "Remove orphaned and unneeded content/repos from a smart proxy.\
         Run with SMART_PROXY_ID=1 to run for a single smart proxy."
-  task :delete_orphaned_content => ["environment"] do
+  task :delete_orphaned_content => ["environment", "disable_dynflow"] do
     User.current = User.anonymous_admin
     smart_proxy_id = ENV['SMART_PROXY_ID']
     if smart_proxy_id

data/lib/katello/tasks/pulp3_migration.rake

@@ -1,3 +1,5 @@
+load "#{Katello::Engine.root}/lib/katello/tasks/common.rake"
+
 namespace :katello do
   desc "Runs a Pulp 2 to 3 Content Migration for supported types.  May be run multiple times.  Use wait=false to immediately return with a task url."
   task :pulp3_migration => ["environment", "disable_dynflow", "check_ping"] do

data/lib/katello/tasks/repository.rake

@@ -1,8 +1,6 @@
-namespace :katello do
-  task :disable_dynflow do
-    ForemanTasks.dynflow.config.remote = true
-  end
+load "#{Katello::Engine.root}/lib/katello/tasks/common.rake"
 
+namespace :katello do
   def commit?
     ENV['COMMIT'] == 'true' || ENV['FOREMAN_UPGRADE'] == '1'
   end

data/lib/katello/tasks/upgrades/3.16/update_applicable_el8_hosts.rake

@@ -0,0 +1,29 @@
+namespace :katello do
+  namespace :upgrades do
+    namespace '3.16' do
+      desc <<-DESCRIPTION
+      Update the applicability calculations for Rhel8 hosts.
+      This migration is to be run to address -> https://bugzilla.redhat.com/show_bug.cgi?id=1814095
+      DESCRIPTION
+      task :update_applicable_el8_hosts, [:input_file] => ["environment"] do
+        User.current = User.anonymous_api_admin
+
+        # Find me only those hosts that follow ALL the conditions below
+        # 1) Have a module stream enabled.
+        # 2) Bound to Non Library repositories. (i.e must belong to a CV thats not the default)
+        # 3) Bound repositories must have module streams in them
+        hosts = Host.joins(:content_facet => :content_facet_repositories).
+                 where("#{Host.table_name}.id" => ::Katello::HostAvailableModuleStream.enabled.select(:host_id)).
+                 where("#{Katello::ContentFacetRepository.table_name}.repository_id" =>
+                        ::Katello::Repository.joins(:repository_module_streams).
+                                          in_non_default_view.
+                                          non_archived)
+        hosts.each do |host|
+          available_streams = ::Katello::HostAvailableModuleStream.joins(:available_module_stream).
+                                                enabled.where(:host_id => host).select(:name, :stream)
+          ::Actions::Katello::Host::UploadProfiles.upload_modules_to_pulp(available_streams, host)
+        end
+      end
+    end
+  end
+end

data/lib/katello/version.rb

@@ -1,3 +1,3 @@
 module Katello
-  VERSION = "3.15.0.1".freeze
+  VERSION = "3.15.3.1".freeze
 end

data/webpack/components/Content/ContentTable.js

@@ -34,6 +34,8 @@ const ContentTable = ({ content, tableSchema, onPaginationChange }) => {
 ContentTable.propTypes = {
   content: PropTypes.shape({
     loading: PropTypes.bool,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
     pagination: PropTypes.shape({}),
     itemCount: PropTypes.number,

data/webpack/components/Content/Details/ContentDetails.js

@@ -59,9 +59,12 @@ ContentDetails.propTypes = {
   contentDetails: PropTypes.shape({
     loading: PropTypes.bool,
     name: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     profiles: PropTypes.array,
     repositories: PropTypes.array,
     artifacts: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
     stream: PropTypes.string,
   }).isRequired,
   schema: PropTypes.arrayOf(PropTypes.shape({})).isRequired,

data/webpack/scenes/AnsibleCollections/Details/AnsibleCollectionDetails.js

@@ -83,8 +83,11 @@ AnsibleCollectionDetails.propTypes = {
     name: PropTypes.string,
     namespace: PropTypes.string,
     version: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     repositories: PropTypes.array,
     tags: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
   }).isRequired,
 };
 

data/webpack/scenes/ModuleStreams/Details/ModuleStreamDetails.js

@@ -81,9 +81,12 @@ ModuleStreamDetails.propTypes = {
   moduleStreamDetails: PropTypes.shape({
     loading: PropTypes.bool,
     name: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     profiles: PropTypes.array,
     repositories: PropTypes.array,
     artifacts: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
     stream: PropTypes.string,
   }).isRequired,
 };

data/webpack/scenes/RedHatRepositories/RedHatRepositoriesPage.js

@@ -111,6 +111,8 @@ RedHatRepositoriesPage.propTypes = {
     recommended: PropTypes.bool,
     loading: PropTypes.bool,
     search: PropTypes.shape({}),
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     missingPermissions: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/RedHatRepositories/components/EnabledRepository/EnabledRepository.js

@@ -94,6 +94,8 @@ EnabledRepository.propTypes = {
   search: PropTypes.shape({
     query: PropTypes.string,
     searchList: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     filters: PropTypes.array,
   }),
   pagination: PropTypes.shape({

data/webpack/scenes/RedHatRepositories/components/RepositorySetRepository/RepositorySetRepository.js

@@ -141,6 +141,8 @@ RepositorySetRepository.propTypes = {
   enabledSearch: PropTypes.shape({
     query: PropTypes.string,
     searchList: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     filters: PropTypes.array,
   }),
   enabledPagination: PropTypes.shape({

data/webpack/scenes/Subscriptions/Details/SubscriptionDetailAssociations.js

@@ -45,6 +45,8 @@ SubscriptionDetailAssociations.propTypes = {
   subscriptionDetails: PropTypes.shape({
     id: PropTypes.number,
     host_count: PropTypes.number,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     activation_keys: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/Subscriptions/Details/SubscriptionDetailProductContent.js

@@ -50,6 +50,8 @@ const SubscriptionDetailProductContent = ({ productContent }) => {
 
 SubscriptionDetailProductContent.propTypes = {
   productContent: PropTypes.shape({
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/Subscriptions/Details/SubscriptionDetailProducts.js

@@ -18,6 +18,8 @@ const SubscriptionDetailProducts = ({ subscriptionDetails }) => (
 
 SubscriptionDetailProducts.propTypes = {
   subscriptionDetails: PropTypes.shape({
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     provided_products: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/Subscriptions/Manifest/ManageManifestModal.js

@@ -352,6 +352,8 @@ ManageManifestModal.propTypes = {
   simpleContentAccess: PropTypes.bool,
   manifestHistory: PropTypes.shape({
     loading: PropTypes.bool,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
   }).isRequired,
   setModalClosed: PropTypes.func.isRequired,

data/webpack/scenes/Subscriptions/SubscriptionsPage.js

@@ -264,10 +264,13 @@ SubscriptionsPage.propTypes = {
   loadTableColumns: PropTypes.func.isRequired,
   simpleContentAccess: PropTypes.bool,
   subscriptions: PropTypes.shape({
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     disconnected: PropTypes.bool,
     tableColumns: PropTypes.array,
     selectedTableColumns: PropTypes.array,
     missingPermissions: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
   }).isRequired,
   activePermissions: PropTypes.shape({
     can_delete_manifest: PropTypes.bool,

data/webpack/scenes/Subscriptions/UpstreamSubscriptions/UpstreamSubscriptionsPage.js

@@ -262,6 +262,8 @@ UpstreamSubscriptionsPage.propTypes = {
   upstreamSubscriptions: PropTypes.shape({
     loading: PropTypes.bool,
     itemCount: PropTypes.number,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
     pagination: PropTypes.shape({}),
     task: PropTypes.shape({

data/webpack/scenes/Subscriptions/components/SubscriptionsTable/SubscriptionsTable.js

@@ -276,6 +276,8 @@ SubscriptionsTable.propTypes = {
   subscriptions: PropTypes.shape({
     loading: PropTypes.bool,
     availableQuantities: PropTypes.shape({}),
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
   }).isRequired,
   subscriptionDeleteModalOpen: PropTypes.bool.isRequired,

data/webpack/scenes/Subscriptions/components/SubscriptionsTable/components/Dialogs/UpdateDialog.js

@@ -47,7 +47,7 @@ const UpdateDialog = ({
 UpdateDialog.propTypes = {
   show: PropTypes.bool.isRequired,
   updateQuantity: PropTypes.func.isRequired,
-  updatedQuantity: PropTypes.shape(PropTypes.Object).isRequired,
+  updatedQuantity: PropTypes.shape({}).isRequired,
   showUpdateConfirm: PropTypes.func.isRequired,
   enableEditing: PropTypes.func.isRequired,
 };

data/webpack/scenes/Subscriptions/components/SubscriptionsTable/components/Dialogs/index.js

@@ -17,10 +17,10 @@ const Dialogs = ({
 );
 
 Dialogs.propTypes = {
-  updateDialog: PropTypes.shape(UpdateDialog.propTypes).isRequired,
-  unsavedChangesDialog: PropTypes.shape(UnsavedChangesDialog.propTypes).isRequired,
-  inputsErrorsDialog: PropTypes.shape(InputsErrorsDialog.propTypes).isRequired,
-  deleteDialog: PropTypes.shape(DeleteDialog.propTypes).isRequired,
+  updateDialog: PropTypes.shape({}).isRequired,
+  unsavedChangesDialog: PropTypes.shape({}).isRequired,
+  inputsErrorsDialog: PropTypes.shape({}).isRequired,
+  deleteDialog: PropTypes.shape({}).isRequired,
 };
 
 export default Dialogs;

data/webpack/scenes/Subscriptions/components/SubscriptionsTable/components/Table.js

@@ -95,6 +95,8 @@ Table.propTypes = {
     searchIsActive: PropTypes.bool,
     itemCount: PropTypes.number,
     pagination: PropTypes.shape({}),
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
   }).isRequired,
   loadSubscriptions: PropTypes.func.isRequired,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: katello
 version: !ruby/object:Gem::Version
-  version: 3.15.0.1
+  version: 3.15.3.1
 platform: ruby
 authors:
 - N/A
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-21 00:00:00.000000000 Z
+date: 2020-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -84,16 +84,28 @@ dependencies:
   name: foreman-tasks
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
+    - - "!="
+      - !ruby/object:Gem::Version
+        version: 1.1.1
+    - - "!="
       - !ruby/object:Gem::Version
-        version: 0.14.1
+        version: 1.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
+    - - "!="
+      - !ruby/object:Gem::Version
+        version: 1.1.1
+    - - "!="
       - !ruby/object:Gem::Version
-        version: 0.14.1
+        version: 1.1.2
 - !ruby/object:Gem::Dependency
   name: dynflow
   requirement: !ruby/object:Gem::Requirement
@@ -1894,6 +1906,8 @@ files:
 - db/migrate/20200121213430_katello_repository_rpms_id_big_int.rb
 - db/migrate/20200129172534_add_epoch_version_release_arch_to_katello_installed_packages.rb
 - db/migrate/20200402130013_add_repsoitory_docker_meta_tag_f_key.rb
+- db/migrate/20200429153103_installed_package_bad_nvrea.rb
+- db/migrate/20200501155054_installed_package_unique_nvrea.rb
 - db/seeds.d/101-locations.rb
 - db/seeds.d/102-organizations.rb
 - db/seeds.d/104-proxy.rb
@@ -4123,6 +4137,7 @@ files:
 - lib/katello/tasks/clean_backend_objects.rake
 - lib/katello/tasks/clean_old_file_repos.rake
 - lib/katello/tasks/clean_published_repo_directories.rake
+- lib/katello/tasks/common.rake
 - lib/katello/tasks/delete_orphaned_content.rake
 - lib/katello/tasks/import_applicability.rake
 - lib/katello/tasks/import_subscriptions.rake
@@ -4149,6 +4164,7 @@ files:
 - lib/katello/tasks/upgrades/3.13/republish_deb_metadata.rake
 - lib/katello/tasks/upgrades/3.15/reindex_rpm_modular.rake
 - lib/katello/tasks/upgrades/3.15/set_sub_facet_dmi_uuid.rake
+- lib/katello/tasks/upgrades/3.16/update_applicable_el8_hosts.rake
 - lib/katello/tasks/upgrades/3.8/clear_checksum_type.rake
 - lib/katello/tasks/virt_who_report.rake
 - lib/katello/url_constrained_cookie_store.rb
@@ -4649,7 +4665,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubyforge_project: 
+rubygems_version: 2.5.2.1
 signing_key: 
 specification_version: 4
 summary: Content and Subscription Management plugin for Foreman