ghe0/brain-mock-clients-mocks
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

updater #1

Merged
ghe0 merged 13 commits from updater into master 2024-12-25 23:00:16 +00:00
Conversation 1 Commits 13 Files Changed 4 +4 -5
3 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 29e5de8424 - Show all commits

4
cli-mock/brain.proto
View File

@ -54,7 +54,6 @@ message UpdateVMReq {
message UpdateVMResp {
string uuid = 1;
string timestamp = 2;
string error = 3;
}
@ -72,6 +71,7 @@ message VMContract {
string kernel_sha = 11;
string dtrfs_sha = 12;
string created_at = 13;
string updated_at = 14;
}
message ListVMContractsReq {
@ -127,4 +127,4 @@ service BrainCliService {
rpc ListNodes (NodeFilters) returns (stream NodeListResp);
rpc DeleteVM (DeleteVMReq) returns (Empty);
rpc UpdateVM (UpdateVMReq) returns (UpdateVMResp);
}
}

4
daemon-mock/brain.proto
View File

@ -54,7 +54,6 @@ message UpdateVMReq {
message UpdateVMResp {
string uuid = 1;
string timestamp = 2;
string error = 3;
}
@ -72,6 +71,7 @@ message VMContract {
string kernel_sha = 11;
string dtrfs_sha = 12;
string created_at = 13;
string updated_at = 14;
}
message ListVMContractsReq {
@ -127,4 +127,4 @@ service BrainCliService {
rpc ListNodes (NodeFilters) returns (stream NodeListResp);
rpc DeleteVM (DeleteVMReq) returns (Empty);
rpc UpdateVM (UpdateVMReq) returns (UpdateVMResp);
}
}

1
daemon-mock/src/main.rs
View File

@ -148,7 +148,6 @@ async fn handle_update_vm_requests(
while let Some(update_vm) = req.recv().await {
let confirmation = UpdateVmResp {
uuid: update_vm.uuid,
timestamp: "brain's job".to_string(),
error: "No errors yet".to_string(),
};
ghe0 marked this conversation as resolved Outdated
ghe0 commented 2024-12-25 20:44:04 +00:00
Outdated
Review
Copy Link

The timestamp must be set from the brain, for security reasons.
Also, this does not require chrono.

Check this:

src/data.rs Line 191 in 7905c1c165
created_at: format!("{:?}", std::time::SystemTime::now()),

The timestamp must be set from the brain, for security reasons. Also, this does not require chrono. Check this: https://gitea.detee.cloud/ghe0/brain-mock/src/commit/7905c1c165ca6911ad8f3d9f83ff160b2d9dc9bb/src/data.rs#L191
info!("Sending UpdateVmResp: {confirmation:?}");