RDKEMW-15105: Let's get connected" screen displayed during Migration testing#288
RDKEMW-15105: Let's get connected" screen displayed during Migration testing#288gururaajar wants to merge 25 commits intosupport/1.12.0from
Conversation
…testing Reason for Change: Deleting all the connections only in case of BOOT_MIGRATION so that networkmanager connecting to the eth0 connection can be avoided and we can create new "Wired Connection 1" to connect. Test Procedure: Migration Scenario Priority: P1 Risks: Medium Signed-off-by: Gururaaja ESR<Gururaja_ErodeSriranganRamlingham@comcast.com>
…testing Reason for Change: Deleting all the connections only in case of BOOT_MIGRATION so that networkmanager connecting to the eth0 connection can be avoided and we can create new "Wired Connection 1" to connect. Test Procedure: Migration Scenario Priority: P1 Risks: Medium Signed-off-by: Gururaaja ESR<Gururaja_ErodeSriranganRamlingham@comcast.com>
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
There was a problem hiding this comment.
Pull request overview
This PR aims to prevent the “Let’s get connected” screen during migration testing by deleting existing NetworkManager connections only on migration boot, so a fresh “Wired connection 1” can be created/activated without NetworkManager auto-connecting to an existing eth0 profile.
Changes:
- Add an async delete callback for NetworkManager connections.
- On interface enable, read
/tmp/bootTypeand (if migration boot) delete all NM connections before activating the target interface’s known connection.
💡 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.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
💡 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.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
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.
You can also share your feedback on Copilot code review. Take the survey.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 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.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 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.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 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.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
💡 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.
…kmanager into topic/RDKEMW-15105_8.3
…kmanager into topic/RDKEMW-15105_8.3
Reason for Change: Deleting all the connections only in case of BOOT_MIGRATION so that networkmanager connecting to the eth0 connection can be avoided and we can create new "Wired Connection 1" to connect.
Test Procedure: Migration Scenario
Priority: P1
Risks: Medium
Signed-off-by: Gururaaja ESRGururaja_ErodeSriranganRamlingham@comcast.com