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

[SVE] [TC-ACL-2.10] Step 10 fails after DUT is rebooted #29164

Closed
jrhees-cae opened this issue Sep 11, 2023 · 4 comments
Closed

[SVE] [TC-ACL-2.10] Step 10 fails after DUT is rebooted #29164

jrhees-cae opened this issue Sep 11, 2023 · 4 comments
Labels

Comments

@jrhees-cae
Copy link
Contributor

Reproduction steps

TC-ACL-2.10 fails step 10 after DUT is rebooted in step 9. It appears that this is because TH1 is still attempting to use the the previous CASE session and therefore fails.

If I remove the PICS_SDK_CI_ONLY from the following step in the YAML (the step immediately after the reboot, but before step 10), I can get the test to run successfully:

    - label: "TH1 Connects to the device again"
      PICS: PICS_SDK_CI_ONLY
      cluster: "DelayCommands"
      command: "WaitForCommissionee"
      arguments:
          values:
              - name: "nodeId"
                value: nodeId

I am testing with a Thread DUT.

Bug prevalence

Consistent

GitHub hash of the SDK that was being used

5ccb774

Platform

raspi

Platform Version(s)

No response

Type

Test Improvement

Testing

YAML tested

(Optional) If manually tested please explain why this is only manually tested

No response

Anything else?

No response

@bzbarsky-apple
Copy link
Contributor

Yeah, I am not sure why that PICS_SDK_CI_ONLY is there. @manjunath-grl please take a look?

Separately, it looks like a lot of that YAML is PICS_SDK_CI_ONLY including some things that probably shouldn't be.

@manjunath-grl
Copy link
Contributor

manjunath-grl commented Sep 12, 2023

@bzbarsky-apple Yes, for the above step PICS_SDK_CI_ONLY should not exist, we will fix it in the upcoming PR.
And for the rest of the steps using PICS PICS_SDK_CI_ONLY added because of the issue 768, we made the prompt steps for all TH2 steps and for the sample app we made it as automated steps with PICS_SDK_CI_ONLY.

@manjunath-grl
Copy link
Contributor

@jrhees-cae fixed in PR 29179 please verify it and close the ticket.

@jrhees-cae
Copy link
Contributor Author

Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants