Skip to content

Conversation

@doanac
Copy link
Member

@doanac doanac commented Oct 16, 2025

No description provided.

doanac added 3 commits August 21, 2025 09:43
Signed-off-by: Andy Doan <andy@foundries.io>
The current mechanism is really tied to fioconfig and doesn't make it
easy for us to add a new init function that also needs to send a config
file to the server.

It would be much easier to just let each init function make the call to
the server. However, doing this as a single request makes things a scale
slightly more resilient (one HTTP call instead of $X)

Signed-off-by: Andy Doan <andy@foundries.io>
Signed-off-by: Andy Doan <andy@foundries.io>
@doanac doanac requested a review from detsch October 16, 2025 16:38
@doanac
Copy link
Member Author

doanac commented Oct 16, 2025

@detsch - i have some changes to support remote actions queued up for fioconfig. I'd like to get them merged in because I suspect we are going to have a fair amount of conflicting code churn when we move to making this an API for fioup. This part 1 that sets things up for remote actions.

Signed-off-by: Andy Doan <andy@foundries.io>
Signed-off-by: Andy Doan <andy@foundries.io>
@doanac doanac force-pushed the initFunctionUpdates branch from f1786f4 to e6296da Compare October 16, 2025 16:42
Copy link
Member

@vkhoroz vkhoroz 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 GitHub does not show your commits as verified. Can you sign them?

@doanac doanac merged commit ab19148 into foundriesio:main Oct 16, 2025
2 checks passed
@doanac doanac deleted the initFunctionUpdates branch October 16, 2025 18:01
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.

3 participants