jumpstarter-driver-flashers: Fix additional review comments

Author: mangelajo

packages/jumpstarter-driver-flashers/jumpstarter_driver_flashers/client.py

@@ -82,7 +82,7 @@ def flash(
         skip_exporter_http = False
         image_url = ""
         operator_scheme = None
-        if path.startswith(("http://")) and not force_exporter_http:
+        if path.startswith("http://") and not force_exporter_http:
             # busybox can handle the http from a remote directly, unless target is isolated
             image_url = path
             skip_exporter_http = True
@@ -346,7 +346,7 @@ def _upload_artifact(self, storage, path: PathBuf, operator: Operator):
         if storage.exists(filename):
             # TODO: check hash for existing files
             self.logger.info(f"Artifact {filename} already exists in storage, skipping")
-        storage.write_from_path(path, operator=operator)
+        storage.write_from_path(filename, path, operator=operator)
 
     @contextmanager
     def _services_up(self):

packages/jumpstarter-driver-flashers/oci_bundles/build_bundle.sh

@@ -2,6 +2,8 @@
 
 # oras login quay.io -u mangelajo
 
+set -e
+
 FLASHER_OCI_CONTAINER="${1:-quay.io/jumpstarter-dev/jumpstarter-flasher-test:latest}"
 BUNDLE_FILES=${2:-"./test/"}
 

packages/jumpstarter-driver-flashers/pyproject.toml

@@ -1,7 +1,7 @@
 [project]
 name = "jumpstarter-driver-flashers"
 dynamic = ["version", "urls"]
-description = "Add your description here"
+description = "Jumpstarter software defined flasher driver"
 readme = "README.md"
 license = { text = "Apache-2.0" }
 authors = [