Skip to content

fiotest: Update the way we set the test "target"#79

Merged
doanac merged 1 commit into
foundriesio:mainfrom
doanac:remote-action-target-fix
Feb 23, 2026
Merged

fiotest: Update the way we set the test "target"#79
doanac merged 1 commit into
foundriesio:mainfrom
doanac:remote-action-target-fix

Conversation

@doanac
Copy link
Copy Markdown
Member

@doanac doanac commented Feb 23, 2026

When we run remote-actions we must set a "target" to use the fiotest backend APIs. However, by setting x-ats-target, the device gateway may update the device's actual target name while fiotest is running leading to confusion.

The device gateway was changed to allow a target name to be set in the test body to work around this.

When we run remote-actions we must set a "target" to use the fiotest
backend APIs. However, by setting x-ats-target, the device gateway may
update the device's actual target name while fiotest is running leading
to confusion.

The device gateway was changed to allow a target name to be set in the
test body to work around this.

Signed-off-by: Andy Doan <andy@foundries.io>
@doanac doanac requested review from detsch and mike-sul February 23, 2026 15:14
@mike-sul
Copy link
Copy Markdown
Contributor

@doanac Will we need to bump this fioconfig version in fioup?

@doanac
Copy link
Copy Markdown
Member Author

doanac commented Feb 23, 2026

@doanac Will we need to bump this fioconfig version in fioup?

yes. i'll be testing this in staging today to sort that out.

Copy link
Copy Markdown

@mike-scott mike-scott left a comment

Choose a reason for hiding this comment

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

LGTM

@doanac doanac merged commit 6384248 into foundriesio:main Feb 23, 2026
2 checks passed
@doanac doanac deleted the remote-action-target-fix branch February 23, 2026 19:30
@doanac
Copy link
Copy Markdown
Member Author

doanac commented Feb 23, 2026

tested in staging.

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.

4 participants