Skip to content

recipes-sota: Add example recipe for custom C++ aklite client#437

Merged
ricardosalveti merged 1 commit intomasterfrom
custom-sota-client-example
Oct 6, 2021
Merged

recipes-sota: Add example recipe for custom C++ aklite client#437
ricardosalveti merged 1 commit intomasterfrom
custom-sota-client-example

Conversation

@doanac
Copy link
Copy Markdown
Member

@doanac doanac commented Oct 5, 2021

This recipe can be used as a starting point for people wanting to create
their own custom sota clients. It can be enabled by:

setting SOTA_CLIENT="custom-sota-client"

Signed-off-by: Andy Doan [email protected]

@doanac
Copy link
Copy Markdown
Member Author

doanac commented Oct 5, 2021

@rsalveti @mike-sul - this is related to foundriesio/docs#280

We have an aklite revision bump in review right now: #429 I think I'll hold off on bumping that SRCREV until we get both of these merged. Make sense?

DEPENDS = "jsoncpp boost aktualizr"

BRANCH = "master"
SRCREV = "05c4180840ef84c21b0632be4a542580d720491f"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wonder if something like this SRCREV = aktualizr_SRCREV :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

^^ this would be cool. I wasn't sure if there was a convention for that.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not unless we control the aktualizr revision outside the aktualizr recipe.

I think it is fine for us to manually track this for now, I was surprised to see that the custom sota client reference is part of the aktualizr-lite source tree, was expecting that to be a different tree on its own (but not an issue, what was done here is correct).


[Install]
WantedBy=multi-user.target

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nit, drop this extra empty line.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done in a fixup commit that we can squash

Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM, minor nits only.

@doanac
Copy link
Copy Markdown
Member Author

doanac commented Oct 6, 2021

i see you already gave the thumbs up, so I'll squash and git-push -f to my branch now

@doanac doanac force-pushed the custom-sota-client-example branch 2 times, most recently from 591999e to 5ed69b5 Compare October 6, 2021 14:45
This recipe can be used as a starting point for people wanting to create
their own custom sota clients. It can be enabled by:

setting SOTA_CLIENT="custom-sota-client"

Signed-off-by: Andy Doan <[email protected]>
@doanac doanac force-pushed the custom-sota-client-example branch from 5ed69b5 to 409e655 Compare October 6, 2021 14:55
@ricardosalveti ricardosalveti merged commit bce2c7b into master Oct 6, 2021
@doanac doanac deleted the custom-sota-client-example branch October 7, 2021 13:47
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