Discovery Service Timestamps

[JannikSt]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces timestamp support for Discovery nodes, enabling tracking of when nodes are updated and created. Key changes include sorting nodes by timestamp in the store layer, adding new timestamp fields (last_updated and created_at) in the shared DiscoveryNode model, and updating tests to verify the timestamp behavior.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
discovery/src/store/node_store.rs	Added sorting of nodes based on timestamp fields
shared/src/models/node.rs	Introduced last_updated and created_at fields in the DiscoveryNode model
shared/Cargo.toml	Added the chrono dependency with serde support
discovery/src/api/routes/node.rs	Updated tests to verify the proper handling of timestamp fields
orchestrator/src/discovery/monitor.rs	Updated test instantiation of nodes with explicit timestamp values
shared/src/utils/google_cloud.rs	Modified test to skip execution when required environment variables are missing

Comments suppressed due to low confidence (1)

shared/src/utils/google_cloud.rs:134

• The environment variable name 'S3_CREDENTIALS' appears inconsistent with the context of Google Cloud usage; consider renaming it to something like 'GOOGLE_CLOUD_CREDENTIALS' for clarity.

let credentials_base64 = match std::env::var("S3_CREDENTIALS") {