-
Notifications
You must be signed in to change notification settings - Fork 0
[3.0.1] Improve PUA startup time #32
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
base: main
Are you sure you want to change the base?
Conversation
Should be merged after #22 to allow the logs to be viewable |
Hi @cjnolan , there is an error in Lint manifest CI/CD. Do we need to update any other files?
|
We can bump to 1.0.1-dev |
why are we reducing the ticker interval? cc @daniele-moro and @raphaelvrosa this has an implication for scale and simulation of the agents. Can you elaborate why is it necessary? |
The changes of tickerInterval has been reverted. PR description updated. Theoretically, a shorter tickerInterval allows the PUA to receive a response from MM more quickly, enabling it to send the READY status heartbeat faster. However, after testing, we found it only saves 10 seconds, so we decided to revert the changes. |
Description
This PR improves the PUA startup time by:
There is a timeout issue observed in this operation, which is delaying the overall PUA bring-up time. The purpose of this operation is to modify the INBM configuration file to remove certain software checks during system update. We decide to remove this operation from the PUA and let the OM to handle it.
Fixes # (issue)
Any Newly Introduced Dependencies
Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: