Add LastRebootTime to Device status

Exposes the last reboot timestamp of a device in its status.

Author: felix-kaestner

api/core/v1alpha1/device_types.go

@@ -143,6 +143,10 @@ type DeviceStatus struct {
 	// +optional
 	FirmwareVersion string `json:"firmwareVersion,omitempty"`
 
+	// LastRebootTime is the timestamp of the last reboot of the Device, if known.
+	// +optional
+	LastRebootTime metav1.Time `json:"lastRebootTime,omitempty"`
+
 	// Provisioning is the list of provisioning attempts for the Device.
 	// +listType=map
 	// +listMapKey=startTime
@@ -275,6 +279,7 @@ const (
 // +kubebuilder:printcolumn:name="Model",type=string,JSONPath=".status.model",priority=1
 // +kubebuilder:printcolumn:name="SerialNumber",type=string,JSONPath=".status.serialNumber",priority=1
 // +kubebuilder:printcolumn:name="FirmwareVersion",type=string,JSONPath=".status.firmwareVersion",priority=1
+// +kubebuilder:printcolumn:name="RebootTime",type="date",JSONPath=".status.lastRebootTime",priority=1
 // +kubebuilder:printcolumn:name="Ports",type=string,JSONPath=".status.portSummary",priority=1
 // +kubebuilder:printcolumn:name="Phase",type="string",JSONPath=".status.phase"
 // +kubebuilder:printcolumn:name="Ready",type=string,JSONPath=`.status.conditions[?(@.type=="Ready")].status`

api/core/v1alpha1/zz_generated.deepcopy.go

@@ -39,6 +39,10 @@ spec:
       name: FirmwareVersion
       priority: 1
       type: string
+    - jsonPath: .status.lastRebootTime
+      name: RebootTime
+      priority: 1
+      type: date
     - jsonPath: .status.portSummary
       name: Ports
       priority: 1
@@ -368,6 +372,11 @@ spec:
                 description: FirmwareVersion is the firmware version running on the
                   Device.
                 type: string
+              lastRebootTime:
+                description: LastRebootTime is the timestamp of the last reboot of
+                  the Device, if known.
+                format: date-time
+                type: string
               manufacturer:
                 description: Manufacturer is the manufacturer of the Device.
                 type: string

charts/network-operator/templates/crd/devices.networking.metal.ironcore.dev.yaml

@@ -36,6 +36,10 @@ spec:
       name: FirmwareVersion
       priority: 1
       type: string
+    - jsonPath: .status.lastRebootTime
+      name: RebootTime
+      priority: 1
+      type: date
     - jsonPath: .status.portSummary
       name: Ports
       priority: 1
@@ -365,6 +369,11 @@ spec:
                 description: FirmwareVersion is the firmware version running on the
                   Device.
                 type: string
+              lastRebootTime:
+                description: LastRebootTime is the timestamp of the last reboot of
+                  the Device, if known.
+                format: date-time
+                type: string
               manufacturer:
                 description: Manufacturer is the manufacturer of the Device.
                 type: string

config/crd/bases/networking.metal.ironcore.dev_devices.yaml

@@ -1002,6 +1002,7 @@ _Appears in:_
 | `model` _string_ | Model is the model identifier of the Device. |  | Optional: \{\} <br /> |
 | `serialNumber` _string_ | SerialNumber is the serial number of the Device. |  | Optional: \{\} <br /> |
 | `firmwareVersion` _string_ | FirmwareVersion is the firmware version running on the Device. |  | Optional: \{\} <br /> |
+| `lastRebootTime` _[Time](https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.35/#time-v1-meta)_ | LastRebootTime is the timestamp of the last reboot of the Device, if known. |  | Optional: \{\} <br /> |
 | `provisioning` _[ProvisioningInfo](#provisioninginfo) array_ | Provisioning is the list of provisioning attempts for the Device. |  | Optional: \{\} <br /> |
 | `ports` _[DevicePort](#deviceport) array_ | Ports is the list of ports on the Device. |  | Optional: \{\} <br /> |
 | `portSummary` _string_ | PortSummary shows a summary of the port configured, grouped by type, e.g. "1/4 (10g), 3/64 (100g)". |  | Optional: \{\} <br /> |

docs/api-reference/index.md

@@ -310,6 +310,7 @@ func (r *DeviceReconciler) reconcile(ctx context.Context, device *v1alpha1.Devic
 	device.Status.Model = info.Model
 	device.Status.SerialNumber = info.SerialNumber
 	device.Status.FirmwareVersion = info.FirmwareVersion
+	device.Status.LastRebootTime = metav1.NewTime(info.LastRebootTime)
 
 	conditions.Set(device, metav1.Condition{
 		Type:    v1alpha1.ReadyCondition,

internal/controller/core/device_controller.go

@@ -121,6 +121,7 @@ var _ = Describe("Device Controller", func() {
 				g.Expect(resource.Status.Model).To(Equal("Model"))
 				g.Expect(resource.Status.SerialNumber).To(Equal("123456789"))
 				g.Expect(resource.Status.FirmwareVersion).To(Equal("1.0.0"))
+				g.Expect(resource.Status.LastRebootTime.Time).To(BeTemporally("==", lastRebootTime))
 
 				g.Expect(resource.Status.Ports).To(HaveLen(8))
 				g.Expect(resource.Status.Ports[0].Name).To(Equal("eth1/1"))

internal/controller/core/device_controller_test.go

@@ -46,6 +46,8 @@ var (
 	k8sManager   ctrl.Manager
 	testProvider = NewProvider()
 	testLocker   *resourcelock.ResourceLocker
+
+	lastRebootTime = time.Date(2025, 1, 1, 0, 0, 0, 0, time.UTC)
 )
 
 func TestControllers(t *testing.T) {
@@ -479,6 +481,7 @@ func (p *Provider) GetDeviceInfo(context.Context) (*provider.DeviceInfo, error)
 		Model:           "Model",
 		SerialNumber:    "123456789",
 		FirmwareVersion: "1.0.0",
+		LastRebootTime:  lastRebootTime,
 	}, nil
 }
 

internal/controller/core/suite_test.go

@@ -182,7 +182,8 @@ func (p *Provider) GetDeviceInfo(ctx context.Context) (*provider.DeviceInfo, err
 	m := new(Model)
 	s := new(SerialNumber)
 	fw := new(FirmwareVersion)
-	if err := p.client.GetState(ctx, m, s, fw); err != nil {
+	bt := new(BootTime)
+	if err := p.client.GetState(ctx, m, s, fw, bt); err != nil {
 		return nil, err
 	}
 
@@ -191,6 +192,7 @@ func (p *Provider) GetDeviceInfo(ctx context.Context) (*provider.DeviceInfo, err
 		Model:           string(*m),
 		SerialNumber:    string(*s),
 		FirmwareVersion: string(*fw),
+		LastRebootTime:  bt.Time,
 	}, nil
 }
 

internal/provider/cisco/nxos/provider.go

@@ -5,7 +5,10 @@ package nxos
 
 import (
 	"context"
+	"encoding/json"
 	"fmt"
+	"strconv"
+	"time"
 
 	"google.golang.org/grpc"
 
@@ -66,6 +69,38 @@ func (*BootPOAP) XPath() string {
 	return "/System/boot-items/poap"
 }
 
+type BootTime UnixTime
+
+func (*BootTime) XPath() string {
+	return "System/procsys-items/bootTime"
+}
+
+func (t *BootTime) UnmarshalJSON(b []byte) error {
+	return (*UnixTime)(t).UnmarshalJSON(b)
+}
+
+// UnixTime is a wrapper around time.Time that marshals/unmarshals to/from a Unix timestamp in seconds.
+type UnixTime struct {
+	time.Time `json:"-"`
+}
+
+// UnmarshalJSON implements json.Unmarshaler.
+func (t *UnixTime) UnmarshalJSON(b []byte) error {
+	var unix int64
+	if err := json.Unmarshal(b, &unix); err != nil {
+		var str string
+		if err := json.Unmarshal(b, &str); err != nil {
+			return fmt.Errorf("failed to unmarshal UnixTime: %w", err)
+		}
+		unix, err = strconv.ParseInt(str, 10, 64)
+		if err != nil {
+			return fmt.Errorf("failed to parse UnixTime string: %w", err)
+		}
+	}
+	t.Time = time.Unix(unix, 0)
+	return nil
+}
+
 func Reboot(ctx context.Context, conn *grpc.ClientConn) error {
 	request := system.RebootRequest{
 		Method: system.RebootMethod_COLD,