LTE-2882: Parodus process was not up#21
Conversation
There was a problem hiding this comment.
Pull request overview
This PR addresses cases where Parodus fails to start when NVRAM-derived configuration is missing/corrupted by adding a partner-default JSON fallback for critical WebPA URLs.
Changes:
- Add partner-JSON lookup helpers to fetch partner-specific WebPA/TOKEN/DNS URL parameters.
- Invoke the partner-JSON fallback during startup when a PartnerID is available and URL values are empty.
- Broaden the “default webpaUrl” assignment to also trigger when
webpaUrlis an empty string.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR aims to prevent parodus startup failures when NVRAM-backed configuration is corrupted by adding a fallback to partner defaults JSON for critical WebPA URLs.
Changes:
- Add partner-defaults JSON lookup for WebPA “crucial URLs” (server/token/DNS TXT) when unset.
- Update startup logic to treat empty
webpaUrlas unset and populate it fromWEBPA_SERVER_URL. - Introduce helper functions and TR-181 parameter name constants for partner-specific URL retrieval.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
0021ea0 to
2d5d75b
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
ce04bfd to
17db562
Compare
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
17db562 to
24ce9bc
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
24ce9bc to
194c7ec
Compare
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
194c7ec to
1b9aae9
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
1b9aae9 to
95ec0dc
Compare
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
95ec0dc to
7c257b8
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
7c257b8 to
64c592b
Compare
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
64c592b to
3d54e68
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
guruchandru
force-pushed
the
feature/LTE-2882
branch
from
3d54e68 to
990da66
Compare
LTE-2882: Parodus process was not running in XLE
Reason for change: To avoid parodus not up when nvram is corrupted.
Test Procedure: Remove the following files and restart parodus it should run
1. /nvram/bootstrap.json
2. /opt/secure/bootstrap.json
3. /nvram/partners_defaults.json
Risks: Low
Priority: P1