Skip to content

Commit 615e7d6

Browse files
Graham Whaleydklyle
Graham Whaley
authored andcommitted
metrics: report: parallel: change title to 'deployment time'
The time we measure is the time to launch and delete the whole deployment, not the 'pod'. Change the graph titles to be more accurate. Signed-off-by: Graham Whaley <graham.whaley@intel.com>
1 parent 1c14965 commit 615e7d6

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

metrics/report/report_dockerfile/parallel.R

+4-4
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ boot_line_plot <- ggplot( data=data, aes(npod, boot_time, colour=testname, group
8080
geom_line( alpha=0.2) +
8181
xlab("parallel pods") +
8282
ylab("Boot time (s)") +
83-
ggtitle("Pod boot time (detail)") +
83+
ggtitle("Deployment boot time (detail)") +
8484
#ylim(0, NA) + # For big machines, better to not 0-index
8585
theme(axis.text.x=element_text(angle=90))
8686

@@ -92,14 +92,14 @@ boot_line_plot <- ggplot( data=data, aes(npod, boot_time, colour=testname, group
9292

9393
# And get a zero Y index plot.
9494
boot_line_plot_zero = boot_line_plot + ylim(0, NA) +
95-
ggtitle("Pod boot time (0 index)")
95+
ggtitle("Deployment boot time (0 index)")
9696

9797
# Show how boot time changed
9898
delete_line_plot <- ggplot( data=data, aes(npod, delete_time, colour=testname, group=dataset)) +
9999
geom_line(alpha=0.2) +
100100
xlab("parallel pods") +
101101
ylab("Delete time (s)") +
102-
ggtitle("Pod deletion time (detail)") +
102+
ggtitle("Deployment deletion time (detail)") +
103103
#ylim(0, NA) + # For big machines, better to not 0-index
104104
theme(axis.text.x=element_text(angle=90))
105105

@@ -111,7 +111,7 @@ delete_line_plot <- ggplot( data=data, aes(npod, delete_time, colour=testname, g
111111

112112
# And get a 0 indexed Y axis plot
113113
delete_line_plot_zero = delete_line_plot + ylim(0, NA) +
114-
ggtitle("Pod deletion time (0 index)")
114+
ggtitle("Deployment deletion time (0 index)")
115115

116116
# See https://www.r-bloggers.com/ggplot2-easy-way-to-mix-multiple-graphs-on-the-same-page/ for
117117
# excellent examples

0 commit comments

Comments
 (0)