Reporter: Rename node output

Rename `node` output field to `vmUnderTestActualNodeName` in order to align it with the DPDK checkup. Signed-off-by: Orel Misan <omisan@redhat.com>
kiagnose · Nov 13, 2023 · 6131001 · 6131001
1 parent 066aa9c
commit 6131001
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/pkg/internal/reporter/reporter.go b/pkg/internal/reporter/reporter.go
@@ -30,8 +30,8 @@ import (
 )
 
 const (
-	NodeKey            = "node"
-	OslatMaxLatencyKey = "oslatMaxLatencyMicroSeconds"
+	VMUnderTestActualNodeNameKey = "vmUnderTestActualNodeName"
+	OslatMaxLatencyKey           = "oslatMaxLatencyMicroSeconds"
 )
 
 type Reporter struct {
@@ -62,8 +62,8 @@ func formatResults(checkupStatus status.Status) map[string]string {
 	}
 
 	formattedResults := map[string]string{
-		NodeKey:            checkupStatus.Results.Node,
-		OslatMaxLatencyKey: fmt.Sprintf("%d", checkupStatus.Results.OslatMaxLatency.Microseconds()),
+		VMUnderTestActualNodeNameKey: checkupStatus.Results.VMUnderTestActualNodeName,
+		OslatMaxLatencyKey:           fmt.Sprintf("%d", checkupStatus.Results.OslatMaxLatency.Microseconds()),
 	}
 
 	return formattedResults

diff --git a/pkg/internal/reporter/reporter_test.go b/pkg/internal/reporter/reporter_test.go
@@ -52,7 +52,7 @@ func TestReportShouldSucceed(t *testing.T) {
 
 func TestReportShouldSuccessfullyReportResults(t *testing.T) {
 	const (
-		expectedNodeName = "rt-node"
+		expectedVMUnderTestActualNodeName = "rt-node"
 	)
 
 	const (
@@ -71,8 +71,8 @@ func TestReportShouldSuccessfullyReportResults(t *testing.T) {
 		checkupStatus.FailureReason = []string{}
 		checkupStatus.CompletionTimestamp = time.Now()
 		checkupStatus.Results = status.Results{
-			Node:            expectedNodeName,
-			OslatMaxLatency: 12 * time.Microsecond,
+			VMUnderTestActualNodeName: expectedVMUnderTestActualNodeName,
+			OslatMaxLatency:           12 * time.Microsecond,
 		}
 
 		assert.NoError(t, testReporter.Report(checkupStatus))
@@ -82,7 +82,7 @@ func TestReportShouldSuccessfullyReportResults(t *testing.T) {
 			"status.failureReason":                      "",
 			"status.startTimestamp":                     timestamp(checkupStatus.StartTimestamp),
 			"status.completionTimestamp":                timestamp(checkupStatus.CompletionTimestamp),
-			"status.result.node":                        checkupStatus.Results.Node,
+			"status.result.vmUnderTestActualNodeName":   checkupStatus.Results.VMUnderTestActualNodeName,
 			"status.result.oslatMaxLatencyMicroSeconds": fmt.Sprintf("%d", checkupStatus.Results.OslatMaxLatency.Microseconds()),
 		}
 

diff --git a/pkg/internal/status/status.go b/pkg/internal/status/status.go
@@ -26,8 +26,8 @@ import (
 )
 
 type Results struct {
-	Node            string
-	OslatMaxLatency time.Duration
+	VMUnderTestActualNodeName string
+	OslatMaxLatency           time.Duration
 }
 
 type Status struct {