Bug 2051945

Please share if there is plan to include a fix for this bug in the next z stream of Satellite 6.10.

Hi Shimon,
when I was playing with the bug, I recall that I prevented new instances of malformed hosts statuses (with empty type) by this fix:

diff a/lib/inventory_sync/async/inventory_full_sync.rb b/lib/inventory_sync/async/inventory_full_sync.rb
--- a/lib/inventory_sync/async/inventory_full_sync.rb
+++ b/lib/inventory_sync/async/inventory_full_sync.rb

      def add_missing_hosts_statuses(hosts_ids)
        InventorySync::InventoryStatus.create(
          hosts_ids.map do |host_id|
            {
              host_id: host_id,
              status: InventorySync::InventoryStatus::DISCONNECT,
              reported_at: DateTime.current,
+             type: InventorySync::InventoryStatus.name,
            }
          end
        )
      end


Does it make sense to have that as PR / potential fix?

Upon review of our valid but aging backlog the Satellite Team has concluded that this Bugzilla does not meet the criteria for a resolution in the near term, and are planning to close in a month. This message may be a repeat of a previous update and the bug is again being considered to be closed. If you have any concerns about this, please contact your Red Hat Account team.  Thank you.

The empty host_status type does not necessarily mean the type must be InventorySync::InventoryStatus . I mean, it is quite possible there were bugs in 6.10 or 6.11 where InventoryStatus type was not set, but now I was testing 6.14 and a few potential ways of getting to the https://bugzilla.redhat.com/show_bug.cgi?id=2051945#c10 - and even *without* explicitly setting the type, the status type is properly populated. Also a few other attempts to make the InventoryStatus string purged away failed.

So I suppose that:
- there are some bugs in 6.10 and 6.11 wrt InventoryStatus that are fixed since 6.12-ish time
- there might be some other bug that, say, after manifest refresh (and something) in non-SCA mode, Katello::SubscriptionStatus string is replaced by empty type  -and that bug remains in 6.12+ versions. But we dont have a reproducer at all

Since 6.11 and older is unsupported, I recommend to:
- upgrade to any supported Satellite version first
- if problem persists after the upgrade, then collect:
  1) sosreport
  2) output of psql command
       echo "SELECT h.id,h.name, hs.* FROM hosts as h, host_status as hs WHERE hs.host_id = h.id AND hs.type IS null ORDER BY h.id " | su - postgres -c "psql foreman"
  3) from that data, locate newest reported_at timestamp, and provide all logs since that time (i.e. "tar cvzf bz2051945_logs.tar.gz $(find /var/log -newermt 2024-02-11)" for reported_at = 2024-02-11).

I will further check the last support case from Sat6.13, if I get similar data from it.

Thank you for your interest in Red Hat Satellite. We have evaluated this request, and while we recognize that it is a valid request, we do not expect this to be implemented in the product in the foreseeable future. This is due to other priorities for the product, and not a reflection on the request itself. We are therefore closing this out as WONTFIX. If you have any concerns about this feel free to contact your Red Hat Account Team. Thank you.

The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days