Skip to content

Commit 7338e8a

Browse files
committed
pmrep: complete and fix csv header unit info change
Recognize the option in the config file. Make pcp2xxx tools to ignore it. Add zsh completion. Sanitize man page description. Clean up implementation. Fix --no-inst-info for CSV while at it. It was inconsistent whether -U should work as --csv-unit-info, QA suggested so but the implementation didn't work like that. Avoid doing so to avoid confusion with one option (-U) having the opposite meaning in different cases. Correct and update QA.
1 parent 8d9ae8f commit 7338e8a

22 files changed

Lines changed: 188 additions & 243 deletions

File tree

qa/1136

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -68,22 +68,22 @@ pmrep -z -a $tmp.archive.f -p -I -r -u $metrics
6868
echo "=== pmrep csv output"
6969
pmrep $log -o csv -p -I -r -u $metrics
7070

71-
echo "=== pmrep csv output (units via -U)"
72-
pmrep $log -o csv -p -I -r -u -U $metrics
73-
7471
echo "=== pmrep csv output - filter"
7572
pmrep $log -o csv -p -I -r -u -i '.*sdb.*' -i '.*sdd.*' $metrics
7673

77-
echo "=== pmrep csv output (no units)"
78-
pmrep $log -o csv -p -I -r -u $metrics
74+
echo "=== pmrep csv output without unit info"
75+
pmrep $log -o csv -p -I -r -u $metrics
76+
77+
echo "=== pmrep csv output with unit info"
78+
pmrep $log -o csv -p -I -r -u --csv-unit-info $metrics
7979

8080
echo "=== pmrep stdout output"
8181
pmrep $log -o stdout -p -I -r -u $metrics
8282

8383
echo "=== pmrep stdout output - filter"
8484
pmrep $log -o stdout -p -I -r -u -i '.*sdb.*' -i '.*sdd.*' $metrics
8585

86-
echo "=== pmrep stdout output (no units)"
86+
echo "=== pmrep stdout output without unit info"
8787
pmrep $log -o stdout -p -I -r -u -U $metrics
8888

8989
echo "=== pmrep colxrow output"

qa/1136.out

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -71,28 +71,6 @@ Time,"disk.dev.read[sda]","disk.dev.read[sdb]","disk.dev.read[sdc]","disk.dev.re
7171
1970-01-01 00:00:17,,17,,,,,,,,,,
7272
1970-01-01 00:00:18,,18,,,,,18,,,,,
7373
1970-01-01 00:00:19,,19,,,,,19,,,,,
74-
=== pmrep csv output (units via -U)
75-
Time,"disk.dev.read[sda]","disk.dev.read[sdb]","disk.dev.read[sdc]","disk.dev.read[sdd]","disk.dev.read[sde]","disk.dev.write[sda]","disk.dev.write[sdb]","disk.dev.write[sdc]","disk.dev.write[sdd]","disk.dev.write[sde]","mem.util.free","mem.util.used"
76-
1970-01-01 00:00:00,0,0,,,,0,0,,,,,
77-
1970-01-01 00:00:01,1,1,,,,1,1,,,,,
78-
1970-01-01 00:00:02,2,2,2,2,,2,2,2,2,,,2
79-
1970-01-01 00:00:03,3,3,3,3,,3,3,3,3,,,3
80-
1970-01-01 00:00:04,,4,,4,,,4,,4,,,4
81-
1970-01-01 00:00:05,,5,,5,,,5,,5,,,5
82-
1970-01-01 00:00:06,,6,,6,6,,6,,6,,,
83-
1970-01-01 00:00:07,,7,,7,7,,7,,7,,,
84-
1970-01-01 00:00:08,8,8,8,8,8,8,8,8,8,8,,8
85-
1970-01-01 00:00:09,9,9,9,9,9,9,9,9,9,9,,9
86-
1970-01-01 00:00:10,10,10,10,10,10,10,10,10,10,10,,10
87-
1970-01-01 00:00:11,11,11,11,11,11,11,11,11,11,11,,11
88-
1970-01-01 00:00:12,12,12,12,12,12,12,12,12,12,12,,12
89-
1970-01-01 00:00:13,13,,13,,,13,,13,,,,13
90-
1970-01-01 00:00:14,14,,14,,,14,,14,,,,14
91-
1970-01-01 00:00:15,,,,,,,,,,,15,
92-
1970-01-01 00:00:16,,,,,,,,,,,16,
93-
1970-01-01 00:00:17,,17,,,,,,,,,,
94-
1970-01-01 00:00:18,,18,,,,,18,,,,,
95-
1970-01-01 00:00:19,,19,,,,,19,,,,,
9674
=== pmrep csv output - filter
9775
Time,"disk.dev.read[sdb]","disk.dev.read[sdd]","disk.dev.write[sdb]","disk.dev.write[sdd]","mem.util.free","mem.util.used"
9876
1970-01-01 00:00:00,0,,0,,,
@@ -115,7 +93,7 @@ Time,"disk.dev.read[sdb]","disk.dev.read[sdd]","disk.dev.write[sdb]","disk.dev.w
11593
1970-01-01 00:00:17,17,,,,,
11694
1970-01-01 00:00:18,18,,18,,,
11795
1970-01-01 00:00:19,19,,19,,,
118-
=== pmrep csv output (no units)
96+
=== pmrep csv output without unit info
11997
Time,"disk.dev.read[sda]","disk.dev.read[sdb]","disk.dev.read[sdc]","disk.dev.read[sdd]","disk.dev.read[sde]","disk.dev.write[sda]","disk.dev.write[sdb]","disk.dev.write[sdc]","disk.dev.write[sdd]","disk.dev.write[sde]","mem.util.free","mem.util.used"
12098
1970-01-01 00:00:00,0,0,,,,0,0,,,,,
12199
1970-01-01 00:00:01,1,1,,,,1,1,,,,,
@@ -137,6 +115,28 @@ Time,"disk.dev.read[sda]","disk.dev.read[sdb]","disk.dev.read[sdc]","disk.dev.re
137115
1970-01-01 00:00:17,,17,,,,,,,,,,
138116
1970-01-01 00:00:18,,18,,,,,18,,,,,
139117
1970-01-01 00:00:19,,19,,,,,19,,,,,
118+
=== pmrep csv output with unit info
119+
Time,"disk.dev.read[sda](count)","disk.dev.read[sdb](count)","disk.dev.read[sdc](count)","disk.dev.read[sdd](count)","disk.dev.read[sde](count)","disk.dev.write[sda](count)","disk.dev.write[sdb](count)","disk.dev.write[sdc](count)","disk.dev.write[sdd](count)","disk.dev.write[sde](count)","mem.util.free(Kbyte)","mem.util.used(Kbyte)"
120+
1970-01-01 00:00:00,0,0,,,,0,0,,,,,
121+
1970-01-01 00:00:01,1,1,,,,1,1,,,,,
122+
1970-01-01 00:00:02,2,2,2,2,,2,2,2,2,,,2
123+
1970-01-01 00:00:03,3,3,3,3,,3,3,3,3,,,3
124+
1970-01-01 00:00:04,,4,,4,,,4,,4,,,4
125+
1970-01-01 00:00:05,,5,,5,,,5,,5,,,5
126+
1970-01-01 00:00:06,,6,,6,6,,6,,6,,,
127+
1970-01-01 00:00:07,,7,,7,7,,7,,7,,,
128+
1970-01-01 00:00:08,8,8,8,8,8,8,8,8,8,8,,8
129+
1970-01-01 00:00:09,9,9,9,9,9,9,9,9,9,9,,9
130+
1970-01-01 00:00:10,10,10,10,10,10,10,10,10,10,10,,10
131+
1970-01-01 00:00:11,11,11,11,11,11,11,11,11,11,11,,11
132+
1970-01-01 00:00:12,12,12,12,12,12,12,12,12,12,12,,12
133+
1970-01-01 00:00:13,13,,13,,,13,,13,,,,13
134+
1970-01-01 00:00:14,14,,14,,,14,,14,,,,14
135+
1970-01-01 00:00:15,,,,,,,,,,,15,
136+
1970-01-01 00:00:16,,,,,,,,,,,16,
137+
1970-01-01 00:00:17,,17,,,,,,,,,,
138+
1970-01-01 00:00:18,,18,,,,,18,,,,,
139+
1970-01-01 00:00:19,,19,,,,,19,,,,,
140140
=== pmrep stdout output
141141
d.d.read d.d.read d.d.read d.d.read d.d.read d.d.write d.d.write d.d.write d.d.write d.d.write m.u.free m.u.used
142142
sda sdb sdc sdd sde sda sdb sdc sdd sde
@@ -185,7 +185,7 @@ Time,"disk.dev.read[sda]","disk.dev.read[sdb]","disk.dev.read[sdc]","disk.dev.re
185185
00:00:17 17 N/A N/A N/A N/A N/A
186186
00:00:18 18 N/A 18 N/A N/A N/A
187187
00:00:19 19 N/A 19 N/A N/A N/A
188-
=== pmrep stdout output (no units)
188+
=== pmrep stdout output without unit info
189189
d.d.read d.d.read d.d.read d.d.read d.d.read d.d.write d.d.write d.d.write d.d.write d.d.write m.u.free m.u.used
190190
sda sdb sdc sdd sde sda sdb sdc sdd sde
191191
00:00:00 0 0 N/A N/A N/A 0 0 N/A N/A N/A N/A N/A

qa/1169

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -77,23 +77,23 @@ pmrep $log -p -u -X test -1 -S @00:00:02 -s 2 -J 3 -e 'a = 100 * rate(disk.dev.r
7777
echo "== csv test"
7878
pmrep $log -p -u -o csv -1 disk.dev.read disk.dev.write mem.util.used | _archive_filter
7979

80-
echo "== csv test with units in header (-U)"
81-
pmrep $log -p -u -o csv -1 -U disk.dev.read disk.dev.write mem.util.used | _archive_filter
82-
8380
echo "== csv test with extended header"
8481
pmrep $log -p -u -o csv -1 -x -C disk.dev.read disk.dev.write mem.util.used | _archive_filter
8582

8683
echo "== csv test with repeating header"
8784
pmrep $log -p -u -o csv -1 -E 3 disk.dev.read disk.dev.write mem.util.used | _archive_filter
8885

89-
echo "== csv test with repeating header with units (-U)"
90-
pmrep $log -p -u -o csv -1 -E 3 -U disk.dev.read disk.dev.write mem.util.used | _archive_filter
86+
echo "== csv test with repeating header with units"
87+
pmrep $log -p -u -o csv -1 -E 3 --csv-unit-info disk.dev.read disk.dev.write mem.util.used | _archive_filter
88+
89+
echo "== csv test with units in header"
90+
pmrep $log -p -u -o csv -1 --csv-unit-info disk.dev.read disk.dev.write mem.util.used | _archive_filter
9191

9292
echo "== csv test with extended/repeating header"
93-
pmrep $log -p -u -o csv -1 -x -E 3 disk.dev.read disk.dev.write mem.util.used | _archive_filter
93+
pmrep $log -p -u -o csv -1 -x -E 3 disk.dev.read disk.dev.write mem.util.used | _archive_filter
9494

95-
echo "== csv test with extended/repeating header with units (-U)"
96-
pmrep $log -p -u -o csv -1 -x -E 3 disk.dev.read disk.dev.write mem.util.used -U | _archive_filter
95+
echo "== csv test with extended/repeating header with units"
96+
pmrep $log -p -u -o csv -1 -x -E 3 --csv-unit-info disk.dev.read disk.dev.write mem.util.used | _archive_filter
9797

9898
echo "== fixed header test"
9999
pmrep $log -p -u -X test -7 disk.dev.read disk.dev.write mem.util.used | _archive_filter

0 commit comments

Comments
 (0)