Bug 2222890 - update event is not created after updating CV name
Summary: update event is not created after updating CV name
Keywords:
Status: VERIFIED
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Content Views
Version: 6.14.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: 6.14.0
Assignee: Samir Jha
QA Contact: Sam Bible
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-14 11:09 UTC by Peter Ondrejka
Modified: 2023-08-10 15:50 UTC (History)
5 users (show)

Fixed In Version: rubygem-katello-4.9.0.10-1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 36628 0 High New update event is not created after updating CV name 2023-08-01 17:45:25 UTC
Red Hat Issue Tracker SAT-18991 0 None None None 2023-07-17 12:24:55 UTC

Description Peter Ondrejka 2023-07-14 11:09:48 UTC
Description of problem:
When existing content view is updated, corresponding audit entry is not created

Version-Release number of selected component (if applicable):
6.14 sn 7

How reproducible:
always

Steps to Reproduce:
1. create a CV
2. update its name 
3. under Monitor > Audits search for 'type=katello/content_view and action=update'

Actual results:
no result

Expected results:
an update event should show up

Additional info:
Doesn't occur in 6.13
Found by QE automation: 
pytest tests/foreman/ui/test_audit.py -k  test_positive_update_event

The cv create event appears as expected

Comment 1 Adam Ruzicka 2023-07-17 12:34:02 UTC
Most likely caused by https://projects.theforeman.org/issues/36268 

@sajha Hi, is there a reason why cv name/label/description are excluded from being audited?

Comment 2 Samir Jha 2023-07-17 13:04:36 UTC
In 6.14, we use these update records to track changes that need publish. Name/label/description changes on a CV do not need a publish and hence are not audited. We can add it back and exclude those if needed.

Comment 3 Samir Jha 2023-08-01 17:45:24 UTC
Created redmine issue https://projects.theforeman.org/issues/36628 from this bug

Comment 4 Bryan Kearney 2023-08-02 20:03:20 UTC
Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/36628 has been resolved.

Comment 6 Sam Bible 2023-08-10 15:50:03 UTC
Verified on - 6.14 - Snap 11

Steps to Reproduce:
1. Create a CV
2. Update it's name and description

Actual results:
Update event for both shows up.

Expected results:
Update even for both shows up.


Note You need to log in before you can comment on or make changes to this bug.