CARVIEW |
Navigation Menu
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Feature/ashraf/lib cicd #13894
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/ashraf/lib cicd #13894
Conversation
unit-tests/run-unit-tests.py
Outdated
print( ' --device <> Run only on the specified devices; ignore any test that does not match (implies --live)' ) | ||
print( ' --no-reset Do not try to reset any devices, with or without a hub' ) | ||
print( ' --hub-reset If a hub is available, reset the hub itself' ) | ||
print( ' --custom-fw If custom fw provided flash it if its different that the current fw installed' ) |
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.
Maybe --custom-fw-d400
?
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.
agreed
image_file = find_image_or_exit(product_name) if not custom_fw_path else custom_fw_path | ||
cmd = [fw_updater_exe, '-r', '-f', image_file] | ||
if custom_fw_path: | ||
# unsiged fw |
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.
unsigned
unit-tests/test-fw-update.py
Outdated
if recovered or 'nightly' not in test.context: | ||
if recovered or 'nightly' not in test.context or custom_fw_path: | ||
# In nightly, we always update; otherwise we try to save time, so do not do anything! | ||
# If custom fw was provided always update for now. |
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.
At the libci param description you wrote
'If custom fw provided flash it if its different that the current fw installed'
Why not implementing it?
acb1aaa
to
7a630e2
Compare
7a630e2
to
e28eebb
Compare
no_reset = False | ||
hub_reset = False | ||
skip_disconnected = False | ||
custom_fw_path='' |
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.
d400
unit-tests/test-fw-update.py
Outdated
filename = os.path.basename(file_path) | ||
match = re.search(r'Image(\d+)_(\d+)_(\d+)_(\d+)', filename) | ||
if match: | ||
return rsutils.version(".".join(match.groups()[:3])) # Skip the last digit |
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.
But the last digit is not always 0, right?
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.
what happens when the last digit is not 0?
will the fw version include the last digit ? cause i saw that currently it doesn't and it shows only 5.7.6 not 5.7.6.0
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.
updated to skip last digits only if its zero
6c160ad
to
8255e40
Compare
8255e40
to
55f0e76
Compare
custom fw install