Skip to content

smpmgr: upload: add --bypass-inspection option for non-MCUboot images#93

Merged
JPHutchins merged 1 commit intointercreate:mainfrom
axelnxp:feature/image_inspection_bypass
Apr 7, 2026
Merged

smpmgr: upload: add --bypass-inspection option for non-MCUboot images#93
JPHutchins merged 1 commit intointercreate:mainfrom
axelnxp:feature/image_inspection_bypass

Conversation

@axelnxp
Copy link
Copy Markdown
Contributor

@axelnxp axelnxp commented Apr 7, 2026

As done for the upgrade command, allow to upload non-MCUboot images.

It was discussed in #92 the following with @JPHutchins :

I'd like to merge this and then I will follow up to make the following changes to the API before releasing:

* remove `--bypass-inspect` option

* replace it with an optional `--image-type=<image type>` field

* default `--image-type` to `"mcuboot"`

* allow `--image-type=unknown` to do exactly what `--bypass-inspect` does now

The goal is to support multiple image types in the long run rather than ignoring everything that is not mcuboot.

LMK if that sounds OK.

As this hasn't be implemented yet, I propose to add a similar bypass mechanism to the upload command until the final rework.

@axelnxp axelnxp force-pushed the feature/image_inspection_bypass branch from 2eecfb8 to 3830312 Compare April 7, 2026 09:21
As done for the upgrade command, allow to upload non-MCUboot images.

Signed-off-by: Axel Le Bourhis <axel.lebourhis@nxp.com>
@axelnxp axelnxp force-pushed the feature/image_inspection_bypass branch from 3830312 to 7758417 Compare April 7, 2026 09:29
Copy link
Copy Markdown
Collaborator

@JPHutchins JPHutchins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@JPHutchins JPHutchins merged commit 7354775 into intercreate:main Apr 7, 2026
15 checks passed
@axelnxp axelnxp deleted the feature/image_inspection_bypass branch April 8, 2026 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants