[Bug 1824990] Re: Please upgrade kylin-nm to 1.0.2

handsome_feng jianfengli at ubuntukylin.com
Tue May 21 07:14:22 UTC 2019


** Description changed:

- It is bug-fix only:
+ [Impact]
  
- - Fix the wrong connection status. (LP: #1823843)
- - Fix that unable to close after turned on the network. (LP: #1823844)
- - Implement singleton pattern. (LP: #1823845)
+  * This new version fixed 3 bugs:
  
- It was stored in the PPA:
- https://launchpad.net/~ubuntukylin-members/+archive/ubuntu/1904updates
+    - Show wrong connection status. (LP: #1823843)
+ 
+    - Unable to close the network after enable it. (LP: #1823844)
+ 
+    - It is possible to launch multiple kylin-nm if you call it from
+ terminal. (LP: #1823845)
+ 
+    And the first two problems seriously affect the daily use of users.
+ 
+ [Test Case]
+ 
+  * For “show wrong connection status",
+    1. Click on the network icon in the system tray panel.
+    2. Check whether the network status in the system tray panel is consistent with the network status in the network tool.
+    result: Even network is enabled, the network status in the kylin-nm show disabled.
+ 
+  * For "Unable to close the network after enable it"
+    1. Open network tool
+    2. Turn on the network
+    3. Turn off network connection
+    result: Failed to close network connection.
+ 
+  * For "Multiple kylin-nm exist simultaneously"
+    1. Open terminal, and type kylin-nm to launch it.
+    result: Two network icon displayed on the system tray at the same time.
+ 
+ [Regression Potential]
+ 
+  * The main solution is to modify the if condition as show below:
+ 
+ -                if(istateStr == "disconnected"){
+ +                if(istateStr == "disconnected" || istateStr == "unavailable"){
+ 
+ and added a new checkSingle() method.
+ 
+ Tt has been in Eoan archive for a mouth and all works fine.

** Patch added: "kylin-nm_1.0.0-1ubuntu0.1.debdiff"
   https://bugs.launchpad.net/ubuntu/+source/kylin-nm/+bug/1824990/+attachment/5265224/+files/kylin-nm_1.0.0-1ubuntu0.1.debdiff

** Tags removed: upgrade-software-version
** Tags added: verification-needed verification-needed-disco

** Also affects: kylin-nm (Ubuntu Disco)
   Importance: Undecided
       Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1824990

Title:
  Please upgrade kylin-nm to 1.0.2

Status in kylin-nm package in Ubuntu:
  New
Status in kylin-nm source package in Disco:
  New

Bug description:
  [Impact]

   * This new version fixed 3 bugs:

     - Show wrong connection status. (LP: #1823843)

     - Unable to close the network after enable it. (LP: #1823844)

     - It is possible to launch multiple kylin-nm if you call it from
  terminal. (LP: #1823845)

     And the first two problems seriously affect the daily use of users.

  [Test Case]

   * For “show wrong connection status",
     1. Click on the network icon in the system tray panel.
     2. Check whether the network status in the system tray panel is consistent with the network status in the network tool.
     result: Even network is enabled, the network status in the kylin-nm show disabled.

   * For "Unable to close the network after enable it"
     1. Open network tool
     2. Turn on the network
     3. Turn off network connection
     result: Failed to close network connection.

   * For "Multiple kylin-nm exist simultaneously"
     1. Open terminal, and type kylin-nm to launch it.
     result: Two network icon displayed on the system tray at the same time.

  [Regression Potential]

   * The main solution is to modify the if condition as show below:

  -                if(istateStr == "disconnected"){
  +                if(istateStr == "disconnected" || istateStr == "unavailable"){

  and added a new checkSingle() method.

  Tt has been in Eoan archive for a mouth and all works fine.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/kylin-nm/+bug/1824990/+subscriptions



More information about the Ubuntu-sponsors mailing list