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

Feature/tachyon flash config blob #791

Merged
merged 6 commits into from
Feb 4, 2025

Conversation

keeramis
Copy link
Contributor

@keeramis keeramis commented Feb 3, 2025

Description

  • Update QDL binaries with support for skip-reset command
  • Support --skip_fashing_os command but flashing only the config.xml file
  • save_config will save json file with config params except registration_code
  • load_config loads the config file and adds the then generated registration_code to create the config blob
  • Retry (3 times) if passwords dont match before quitting the setup process
  • Support ~ in ssh path
  • Replaceqlril-app with particle-ril

How to Test

Related Issues / Discussions

Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA
  • Problem and solution clearly stated
  • Tests have been provided
  • Docs have been updated
  • CI is passing

@keeramis keeramis requested a review from monkbroc February 3, 2025 22:55
@keeramis keeramis force-pushed the feature/tachyon-flash-config-blob branch from 32e0630 to b62a363 Compare February 4, 2025 00:34
Copy link
Member

@monkbroc monkbroc left a comment

Choose a reason for hiding this comment

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

Looks good with some small tweaks.

async _flash({ files, skipFlashingOs, output }) {

const packagePath = files[0];

const question = {
Copy link
Member

Choose a reason for hiding this comment

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

Will this question be asked if --load_config is passed? Remember the goal of load config is to have a setup without a single question. Andrew will use this in the Tachyon CI.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will still ask the question. Let me fix it.

Copy link
Contributor Author

@keeramis keeramis Feb 4, 2025

Choose a reason for hiding this comment

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

I removed this. There is another question about product. Is this something we want to put in the config file as well?

src/cmd/flash.js Outdated
Comment on lines 217 to 222
const dir = await unzip.Open.file(zipPath);
const firehoseFile = dir.files.find(file => file.path.includes('firehose'));
if (!firehoseFile) {
throw new Error('Unable to find firehose file');
}
Copy link
Member

Choose a reason for hiding this comment

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

The logic to find the firehose file is already present in _extractFlashFilesFromZip and is more precise: open the zip, load the manifest JSON, get the field for the firehose loader.

Call _extractFlashFilesFromZip and get the loader filename from the first element of the returned files. Then load that file from the zip.

@keeramis keeramis force-pushed the feature/tachyon-flash-config-blob branch from 0d19988 to d4fdf2e Compare February 4, 2025 17:00
@keeramis keeramis force-pushed the feature/tachyon-flash-config-blob branch from d4fdf2e to 9034d4f Compare February 4, 2025 17:01
@keeramis keeramis merged commit afeef7d into master Feb 4, 2025
5 checks passed
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.

2 participants