From 6c8ef673f90420e159571895c8fc78e98571b3d3 Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Wed, 17 Apr 2024 09:39:16 -0400 Subject: [PATCH] container: Make ManifestDiff serializable But don't serialize the layers or manifests as that's large; just the numeric diff which is a lot smaller amount of data. Fetching manifests and layers should be done via a dedicated API. --- lib/src/container/mod.rs | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/lib/src/container/mod.rs b/lib/src/container/mod.rs index f34f085c3..ee9b94c56 100644 --- a/lib/src/container/mod.rs +++ b/lib/src/container/mod.rs @@ -28,6 +28,7 @@ use anyhow::anyhow; use containers_image_proxy::oci_spec; use ostree::glib; +use serde::Serialize; use std::borrow::Cow; use std::collections::HashMap; @@ -295,27 +296,31 @@ impl std::fmt::Display for OstreeImageReference { } /// Represents the difference in layer/blob content between two OCI image manifests. -#[derive(Debug)] +#[derive(Debug, Serialize)] pub struct ManifestDiff<'a> { /// The source container image manifest. + #[serde(skip)] pub from: &'a oci_spec::image::ImageManifest, /// The target container image manifest. + #[serde(skip)] pub to: &'a oci_spec::image::ImageManifest, /// Layers which are present in the old image but not the new image. + #[serde(skip)] pub removed: Vec<&'a oci_spec::image::Descriptor>, /// Layers which are present in the new image but not the old image. + #[serde(skip)] pub added: Vec<&'a oci_spec::image::Descriptor>, - /// Total number of packages + /// Total number of layers pub total: u64, - /// Size of total number of packages. + /// Size of total number of layers. pub total_size: u64, - /// Number of packages removed + /// Number of layers removed pub n_removed: u64, - /// Size of the number of packages removed + /// Size of the number of layers removed pub removed_size: u64, /// Number of packages added pub n_added: u64, - /// Size of the number of packages added + /// Size of the number of layers added pub added_size: u64, }