-
Notifications
You must be signed in to change notification settings - Fork 143
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
os configure: Locate the boot partition w/o using the device-type.json's partition field #2901
base: master
Are you sure you want to change the base?
Conversation
bd042dc
to
b64723f
Compare
b64723f
to
245a966
Compare
245a966
to
08d2d3c
Compare
Change-type: patch
08d2d3c
to
3ed7317
Compare
…n's partition field Update balena-device-init from 8.0.0 to 8.1.0 Change-type: minor
3ed7317
to
c36c530
Compare
…ce-type.json contents Change-type: minor
c36c530
to
6281bb1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice tests! Not approving only due to package version on balena-device-init.
const command: string[] = [ | ||
`os configure ${tmpMatchingDtJsonPartitionPath}`, | ||
'--device-type jetson-nano', | ||
'--version 6.0.13', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove the version option now that getOsVersionFromImage()
reads from the image. It also would provide another implicit test.
const command: string[] = [ | ||
`os configure ${tmpNonMatchingDtJsonPartitionPath}`, | ||
'--device-type jetson-nano', | ||
'--version 6.1.25', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove this version option now like for the preceding test.
On top of #2899, so let's get that in first for better debugging in case of an issue.
Update balena-device-init from 8.0.0 to 8.1.0
Depends-on: balena-io-modules/balena-device-init#47
Change-type: patch