Merge pull request bootc-dev#661 from cgwalters/status-format-version

cgwalters · web-flow · commit 37e9f72f3d39 · 2024-07-02T06:42:11.000-04:00
status: Add --format-version
diff --git a/docs/src/man/bootc-container-lint.md b/docs/src/man/bootc-container-lint.md
@@ -23,4 +23,4 @@ part of a build process; it will error if any problems are detected.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-container.md b/docs/src/man/bootc-container.md
@@ -30,4 +30,4 @@ bootc-container-help(8)
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-edit.md b/docs/src/man/bootc-edit.md
@@ -36,4 +36,4 @@ Only changes to the \`spec\` section are honored.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-install-print-configuration.md b/docs/src/man/bootc-install-print-configuration.md
@@ -27,4 +27,4 @@ string-valued filesystem name suitable for passing to \`mkfs.\$type\`.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-install-to-disk.md b/docs/src/man/bootc-install-to-disk.md
@@ -144,4 +144,4 @@ firmware will be skipped
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-install-to-existing-root.md b/docs/src/man/bootc-install-to-existing-root.md
@@ -130,4 +130,4 @@ firmware will be skipped
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-install-to-filesystem.md b/docs/src/man/bootc-install-to-filesystem.md
@@ -157,4 +157,4 @@ mounting. To override this, use \`\--root-mount-spec\`.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-install.md b/docs/src/man/bootc-install.md
@@ -61,4 +61,4 @@ bootc-install-help(8)
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-rollback.md b/docs/src/man/bootc-rollback.md
@@ -34,4 +34,4 @@ rollback invocation.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-status.md b/docs/src/man/bootc-status.md
@@ -4,8 +4,8 @@ bootc-status - Display status
 
 # SYNOPSIS
 
-**bootc status** \[**\--format**\] \[**\--booted**\]
-\[**-h**\|**\--help**\]
+**bootc status** \[**\--format**\] \[**\--format-version**\]
+\[**\--booted**\] \[**-h**\|**\--help**\]
 
 # DESCRIPTION
 
@@ -30,6 +30,13 @@ The exact API format is not currently declared stable.
 >
 > -   json: Output in JSON format
 
+**\--format-version**=*FORMAT_VERSION*
+
+:   The desired format version. There is currently one supported
+    version, which is version \`0\`. Pass this option to explicitly
+    request it; it is possible that multiple versions will be supported
+    in the future
+
 **\--booted**
 
 :   Only display status for the booted deployment
@@ -40,4 +47,4 @@ The exact API format is not currently declared stable.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-switch.md b/docs/src/man/bootc-switch.md
@@ -61,4 +61,4 @@ includes a default policy which requires signatures.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-upgrade.md b/docs/src/man/bootc-upgrade.md
@@ -52,4 +52,4 @@ userspace-only restart.
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc-usr-overlay.md b/docs/src/man/bootc-usr-overlay.md
@@ -39,4 +39,4 @@ unmount\".
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/docs/src/man/bootc.md b/docs/src/man/bootc.md
@@ -72,4 +72,4 @@ bootc-help(8)
 
 # VERSION
 
-v0.1.12
+v0.1.13
diff --git a/lib/src/cli.rs b/lib/src/cli.rs
@@ -130,6 +130,13 @@ pub(crate) struct StatusOpts {
     #[clap(long)]
     pub(crate) format: Option<OutputFormat>,
 
+    /// The desired format version. There is currently one supported
+    /// version, which is version `0`. Pass this option to explicitly
+    /// request it; it is possible that multiple versions will be
+    /// supported in the future.
+    #[clap(long)]
+    pub(crate) format_version: Option<u32>,
+
     /// Only display status for the booted deployment.
     #[clap(long)]
     pub(crate) booted: bool,
@@ -837,9 +844,17 @@ fn test_parse_opts() {
         Opt::Status(StatusOpts {
             json: false,
             format: None,
+            format_version: None,
             booted: false
         })
     ));
+    assert!(matches!(
+        Opt::parse_including_static(["bootc", "status", "--format-version=0"]),
+        Opt::Status(StatusOpts {
+            format_version: Some(0),
+            ..
+        })
+    ));
 }
 
 #[test]
diff --git a/lib/src/status.rs b/lib/src/status.rs
@@ -304,6 +304,10 @@ pub(crate) fn get_status(
 /// Implementation of the `bootc status` CLI command.
 #[context("Status")]
 pub(crate) async fn status(opts: super::cli::StatusOpts) -> Result<()> {
+    match opts.format_version.unwrap_or_default() {
+        0 => {}
+        o => anyhow::bail!("Unsupported format version: {o}"),
+    };
     let host = if !Utf8Path::new("/run/ostree-booted").try_exists()? {
         Default::default()
     } else {
diff --git a/tests/booted/001-test-status.nu b/tests/booted/001-test-status.nu
@@ -5,6 +5,8 @@ tap begin "verify bootc status output formats"
 
 let st = bootc status --json | from json
 assert equal $st.apiVersion org.containers.bootc/v1alpha1
+let st = bootc status --json --format-version=0 | from json
+assert equal $st.apiVersion org.containers.bootc/v1alpha1
 let st = bootc status --format=yaml | from yaml
 assert equal $st.apiVersion org.containers.bootc/v1alpha1
 tap ok
diff --git a/tests/booted/basic.py b/tests/booted/basic.py
@@ -10,3 +10,7 @@ def test_bootc_status():
     o = subprocess.check_output(["bootc", "status", "--json"])
     st = json.loads(o)
     assert st['apiVersion'] == 'org.containers.bootc/v1alpha1'
+
+def test_bootc_status_invalid_version():
+    o = subprocess.call(["bootc", "status", "--json", "--format-version=42"])
+    assert o != 0

Original file line number	Diff line number	Diff line change
`@@ -23,4 +23,4 @@ part of a build process; it will error if any problems are detected.`
`23`	`23`
`24`	`24`	`# VERSION`
`25`	`25`
`26`		`-v0.1.12`
	`26`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
`@@ -30,4 +30,4 @@ bootc-container-help(8)`
`30`	`30`
`31`	`31`	`# VERSION`
`32`	`32`
`33`		`-v0.1.12`
	`33`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
@@ -36,4 +36,4 @@ Only changes to the \`spec\` section are honored.
`36`	`36`
`37`	`37`	`# VERSION`
`38`	`38`
`39`		`-v0.1.12`
	`39`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
@@ -27,4 +27,4 @@ string-valued filesystem name suitable for passing to \`mkfs.\$type\`.
`27`	`27`
`28`	`28`	`# VERSION`
`29`	`29`
`30`		`-v0.1.12`
	`30`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
`@@ -144,4 +144,4 @@ firmware will be skipped`
`144`	`144`
`145`	`145`	`# VERSION`
`146`	`146`
`147`		`-v0.1.12`
	`147`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
`@@ -130,4 +130,4 @@ firmware will be skipped`
`130`	`130`
`131`	`131`	`# VERSION`
`132`	`132`
`133`		`-v0.1.12`
	`133`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
@@ -157,4 +157,4 @@ mounting. To override this, use \`\--root-mount-spec\`.
`157`	`157`
`158`	`158`	`# VERSION`
`159`	`159`
`160`		`-v0.1.12`
	`160`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
`@@ -61,4 +61,4 @@ bootc-install-help(8)`
`61`	`61`
`62`	`62`	`# VERSION`
`63`	`63`
`64`		`-v0.1.12`
	`64`	`+v0.1.13`
Original file line number	Diff line number	Diff line change
`@@ -34,4 +34,4 @@ rollback invocation.`
`34`	`34`
`35`	`35`	`# VERSION`
`36`	`36`
`37`		`-v0.1.12`
	`37`	`+v0.1.13`