Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scarthgap tegra jetpack5 v3 #402

Closed

Conversation

apbr
Copy link
Contributor

@apbr apbr commented Aug 21, 2024

Starting with Mender Client 4.0, the binary is split into mender-auth and mender-update, so we should use mender-update.
Also includes a first step towards compatibility with JP 6.

apbr added 2 commits August 21, 2024 22:48
/opt/nvidia/esp/ does not exist anymore with JP6.

Signed-off-by: Jonathan Schnitzler <jonathan.schnitzler@faro.com>
Signed-off-by: Jonathan Schnitzler <jonathan.schnitzler@faro.com>
@apbr apbr force-pushed the scarthgap-tegra-jetpack5-v3 branch from a74d47c to 11fe8ff Compare August 21, 2024 20:48
Copy link
Contributor

@zach-welch-aquabyte zach-welch-aquabyte left a comment

Choose a reason for hiding this comment

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

I have been testing with the 3.x client and only recently tried to switch to 4.x, as the fixes required for it to build for us only recently landed upstream. My initial testing with 4.x on this branch was... unsuccessful. I ended up rolling back to 3.x for the time being, in part because I don't want to change workflows that use the old CLI syntax. I will be investigating further, but I need to use 3.x in the meantime.

Naively, I might propose a simple mender wrapper script be installed by the 4.x recipe in meta-mender-core. This would call through to the new 4.x commands, allowing shell-driven workflows written for 3.x to continue working as expected with either version. This might need to be optional, enabled only via PACKAGECONFIG[mender-3-compat] or similar. In writing this, I imagine there are others that have their own automation built on the mender CLI, so I wonder if there might be a good reason that Mender hasn't added this already.

If that change isn't feasible for the core, then the switch-rootfs script might instead be updated to detect which command is available and do the right thing accordingly. Compatibility achieved.

@apbr
Copy link
Contributor Author

apbr commented Aug 22, 2024

@zach-welch-aquabyte Oh, I assumed the Mender client pre 4.0 could be called by mender or mender-update. Is that not the case?

@TheYoctoJester
Copy link
Contributor

@apbr back from AFKness now, sorry for the delay. Is this still relevant? My understanding is that it "just" fixes the EFI mount paths. As I can't judge, can @zach-welch-aquabyte or @dwalkes give me a yay or nay please?

@apbr
Copy link
Contributor Author

apbr commented Sep 10, 2024

My understanding is that it "just" fixes the EFI mount paths.

That is a requirement for mender updates to work on JP6. (The old EFI mount path is not available anymore on JP6)
Though as the other part of this PR (calling mender-update instead of mender) is not general enough, this PR probably shouldn't be merged.

@apbr apbr closed this Sep 10, 2024
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