Bug 2095448

Summary: Logconv.pl CSV file contains mismatched header and data columns
Product: Red Hat Enterprise Linux 9 Reporter: Eric Negaard <eric.negaard>
Component: 389-ds-baseAssignee: LDAP Maintainers <idm-ds-dev-bugs>
Status: ASSIGNED --- QA Contact: LDAP QA Team <idm-ds-qe-bugs>
Severity: medium Docs Contact:
Priority: medium    
Version: 9.3CC: bsmejkal, idm-ds-dev-bugs, mreynolds, vashirov
Target Milestone: betaKeywords: Triaged
Target Release: 9.4   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: redhat-ds-11-8080020221130182235.022a399e Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Eric Negaard 2022-06-09 17:58:15 UTC
Description of problem:

When logconv.pl is run in a mode that generates a CSV file, for example by supplying -M accessreport.csv on the command line, the first (header) row of the generated file contains 19 columns but the following data rows contain 20 columns each.

The missing header column is the next-to-last column (corresponding to the "notesF" metric), just before "ElapsedTime", and should be "Invalid filter" or something like that.
 
Version-Release number of selected component (if applicable):

Access Log Analyzer v8.2

How reproducible:

Always

Steps to Reproduce:
1. Run logconv.pl -M accessreport.csv /var/log/dirsrv/slapd-*/access
2. Run awk -F, 'NR==1{print NF}NR==2{print NF}NR>2{exit}' accessreport.csv
3.

Actual results:

19
20

Expected results:

20
20

Additional info:

Comment 4 Viktor Ashirov 2023-02-16 09:20:16 UTC
Hi Mark,

I see that notesF column is now removed instead of generating report for invalid filters. Is this expected?

Thanks.

Comment 5 mreynolds 2023-02-16 13:15:23 UTC
(In reply to Viktor Ashirov from comment #4)
> Hi Mark,
> 
> I see that notesF column is now removed instead of generating report for
> invalid filters. Is this expected?

I think at the time I thought it wasn't that useful?  Not sure why I just didn't just add the column.  I'll add it back...