Skip to content

Fixes #39212 - Add still_applicable column to errata report#10945

Open
pavanshekar wants to merge 1 commit into
theforeman:developfrom
pavanshekar:issue-39212
Open

Fixes #39212 - Add still_applicable column to errata report#10945
pavanshekar wants to merge 1 commit into
theforeman:developfrom
pavanshekar:issue-39212

Conversation

@pavanshekar

@pavanshekar pavanshekar commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Adds a still_applicable column to the Host - Applied Errata report template. This column indicates whether an applied erratum is currently still showing as applicable to the host, helping admins identify potential issues like package downgrades or manual changes. Removes "pending" status from the Applied Errata report template because the new architecture only tracks completed errata applications.

Related Issues

Testing Steps

  1. Apply the Katello PR changes
  2. Apply errata to a host
  3. Generate "Host - Applied Errata" report
  4. Verify "pending" is not available in the Status filter dropdown
  5. Verify still_applicable column shows false for successfully applied errata
  6. Downgrade a package and verify still_applicable shows true

@ianballou ianballou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally +1 once Katello/katello#11690 is green.

@pavanshekar

Copy link
Copy Markdown
Contributor Author

Rebased with the develop branch

Comment thread app/views/unattended/report_templates/host_-_applied_errata.erb
Comment thread app/views/unattended/report_templates/host_-_applied_errata.erb
@jeremylenz jeremylenz dismissed their stale review June 11, 2026 20:46

changes addressed

Comment thread app/views/unattended/report_templates/host_-_applied_errata.erb Outdated
@pavanshekar pavanshekar force-pushed the issue-39212 branch 2 times, most recently from 557180a to 407b5d9 Compare June 12, 2026 13:42

@jeremylenz jeremylenz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @pavanshekar!

Approving with one optional comment.

version: 3.12.0
-%>
<%- report_headers 'date', 'hostname', 'erratum_id', 'erratum_type', 'issued', 'severity', 'status' -%>
<%- report_headers 'date', 'hostname', 'erratum_id', 'erratum_type', 'issued', 'severity', 'status', 'still_applicable' -%>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<%- report_headers 'date', 'hostname', 'erratum_id', 'erratum_type', 'issued', 'severity', 'status', 'still_applicable' -%>
<%- report_headers 'Date', 'Hostname', 'Erratum ID', 'Erratum Type', 'Issued', 'Severity', 'Status', 'Still Applicable' -%>

@pavanshekar

Copy link
Copy Markdown
Contributor Author

Rebased with develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants