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

Add state check to check_device_booted #137

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Add state check to check_device_booted #137

merged 1 commit into from
Oct 26, 2023

Conversation

kiya956
Copy link
Contributor

@kiya956 kiya956 commented Oct 21, 2023

impact:
snappy-device-agents/src/testflinger_device_connectors/devices/oemrecovery/oemrecovery.py

description:
currently check_device_booted not really check device current state
and we found it could end up login into install mode state or run mode but before device ready.
For making sure everything is ready before we start testing procedure, we add state checking.

test:
test on x8high-pdk, x8high-jace9000, x8med-pdk, x6med-pdk pass

Description

Resolved issues

Documentation

Tests

impact:
    snappy-device-agents/src/testflinger_device_connectors/devices/oemrecovery/oemrecovery.py

description:
    currently check_device_booted not really check device current state
    and we found it could end up login into install mode state or run mode but before device ready.
    For making sure everything is ready before we start testing procedure, we add state checking.

test:
    test on x8high-pdk, x8high-jace9000, x8med-pdk, x6med-pdk pass

Signed-off-by: ChunAn Wu <an.wu@canonical.com>
Copy link
Collaborator

@plars plars left a comment

Choose a reason for hiding this comment

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

Excellent, it looks like you moved it over from the old repo very nicely. Just one question - do you know if this was tested on any other existing oemrecover devices? or just the new ones that you are trying to use it with, and if so, which ones?

@kiya956
Copy link
Contributor Author

kiya956 commented Oct 26, 2023

Excellent, it looks like you moved it over from the old repo very nicely. Just one question - do you know if this was tested on any other existing oemrecover devices? or just the new ones that you are trying to use it with, and if so, which ones?

I only tested on x8high-pdk, x8high-jace9000 (Post GM, currently in cert Lab), x8med-pdk, X6med-pdk(still in pr-GM). But I didn't test on other devices in cert Lab.

Copy link
Collaborator

@plars plars left a comment

Choose a reason for hiding this comment

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

I found some other oemrecovery devices to test it on. It worked fine on the one with an ubuntu core image. I could only find one other device with a classic image but I think it was misconfigured and shouldn't have been oemrecovery. It didn't work at all, but it doesn't seem to be related to your change here. The other jobs actually using this are skipping provisioning, so I don't think we actually have any other classic oemrecovery images.
So with your testing and the extra that I did, +1

@plars plars merged commit 0dd05bf into canonical:main Oct 26, 2023
2 checks passed
@kiya956
Copy link
Contributor Author

kiya956 commented Oct 27, 2023

Paul, Thank you so much.

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