Skip to content

Modify plainbox config to fit the new suspend_cycles_with_reboot jobs, and set the SANP_TASK_TIMEOUT#58

Merged
seankingyang merged 2 commits intocanonical:mainfrom
seankingyang:Modify_plainbox_config
Jan 16, 2024
Merged

Modify plainbox config to fit the new suspend_cycles_with_reboot jobs, and set the SANP_TASK_TIMEOUT#58
seankingyang merged 2 commits intocanonical:mainfrom
seankingyang:Modify_plainbox_config

Conversation

@seankingyang
Copy link
Copy Markdown
Contributor

@seankingyang seankingyang commented Jan 5, 2024

Modify the plainbox.conf for these two purpose:

  • This PR: Retire pm_test from PC suspend 30 cycles and separate IoT suspend cycles (New) checkbox#601 will need the new checkbox config variables
    • STRESS_S3_ITERATIONS : suspend times in each reboot cycles
    • STRESS_SUSPEND_REBOOT_ITERATIONS : total reboot cycles

      Note: Total suspend number = STRESS_S3_ITERATIONS x STRESS_SUSPEND_REBOOT_ITERATION

    • STRESS_S3_SLEEP_DELAY: sleep dealy
    • STRESS_S3_WAIT_DELAY: device check delay
    • STRESS_SUSPEND_SLEEP_THRESHOLD : suspend time threshold
    • STRESS_SUSPEND_RESUME_THRESHOLD: resume time threshold
  • Set the SNAPD_TASK_TIMEOUT to extend the timeout, and make the SANPD test plan in Auto test plan not easy to be timeout by the snap store network not always stable

@seankingyang seankingyang changed the title Modify plainbox config Modify plainbox config to fit the new suspend_cycles_with_reboot jobs, and set the SANP_TASK_TIMEOUT Jan 5, 2024
Copy link
Copy Markdown
Collaborator

@hanhsuan hanhsuan left a comment

Choose a reason for hiding this comment

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

Due to the tests of s3 and s2idle depending on the same variables. It would be great to have comments to let user knows what these variables would impact.

Copy link
Copy Markdown
Contributor

@rickwu666666 rickwu666666 left a comment

Choose a reason for hiding this comment

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

LGTM

@seankingyang seankingyang force-pushed the Modify_plainbox_config branch from 6a71546 to 58d2eb6 Compare January 16, 2024 06:01
@seankingyang seankingyang requested a review from hanhsuan January 16, 2024 06:01
@seankingyang seankingyang force-pushed the Modify_plainbox_config branch from 58d2eb6 to 91edef8 Compare January 16, 2024 06:13
Copy link
Copy Markdown
Collaborator

@hanhsuan hanhsuan left a comment

Choose a reason for hiding this comment

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

LGTM

@seankingyang seankingyang merged commit 73625a8 into canonical:main Jan 16, 2024
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.

3 participants