Skip to content

greengrass component: adjust minimal architecture to match Go binary#564

Open
Kriechi wants to merge 1 commit intoaws:mainlinefrom
Kriechi:gg-component-architecture
Open

greengrass component: adjust minimal architecture to match Go binary#564
Kriechi wants to merge 1 commit intoaws:mainlinefrom
Kriechi:gg-component-architecture

Conversation

@Kriechi
Copy link

@Kriechi Kriechi commented Apr 18, 2024

SSM-Agent is already compiled and tested for armv6 (see makefile GOARM), so changing the GG component recipe brings it down to match the minimal architecture version. GG Nucleus requires at least armv7l, so the regex for the architecture can be shortened accordingly. This matches the recommended best practice of other AWS-provided Greengrass components.

See Platform.architecture recipe key:
https://docs.aws.amazon.com/greengrass/v2/developerguide/component-recipe-reference.html#recipe-format

See "Supported platforms" of Greengrass Nucleus:
https://docs.aws.amazon.com/greengrass/v2/developerguide/setting-up.html#installation-requirements

Issue #, if available:
#539
#445

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

SSM-Agent is already compiled and tested for armv6 (see makefile GOARM), so changing the GG component recipe brings it down to match the minimal architecture version. GG Nucleus requires at least armv7l, so the regex for the architecture can be shortened accordingly. This matches the recommended best practice of other AWS-provided Greengrass components.

See `Platform.architecture` recipe key:
https://docs.aws.amazon.com/greengrass/v2/developerguide/component-recipe-reference.html#recipe-format

See "Supported platforms" of Greengrass Nucleus:
https://docs.aws.amazon.com/greengrass/v2/developerguide/setting-up.html#installation-requirements
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.

1 participant