Checkbox

Merge lp:~sylvain-pineau/checkbox/create_connection_fix_vivid into lp:checkbox

  1. create_connection_fix_vivid
  2. Merge into trunk
Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 3825
Merged at revision: 3826
Proposed branch: lp:~sylvain-pineau/checkbox/create_connection_fix_vivid
Merge into: lp:checkbox
Diff against target: 11 lines (+1/-0)
1 file modified
providers/plainbox-provider-checkbox/bin/create_connection (+1/-0)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/create_connection_fix_vivid
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+261423@code.launchpad.net

Description of the change

A fix for the create_connection script (for Vivid). Quoting the nmcli man page:

NetworkManager does not monitor changes to connection files by default. So we
need to use this command in order to tell NetworkManager to re-read the
connection profiles from disk when a change was made to them.

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approving this one.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-checkbox/bin/create_connection'
2--- providers/plainbox-provider-checkbox/bin/create_connection 2015-06-02 08:22:55 +0000
3+++ providers/plainbox-provider-checkbox/bin/create_connection 2015-06-08 16:09:05 +0000
4@@ -137,6 +137,7 @@
5 stderr=DEVNULL,
6 universal_newlines=True)
7 except CalledProcessError:
8+ check_call(['nmcli', 'con', 'reload'])
9 nmcli_con_list = check_output(['nmcli', 'con', 'show'],
10 stderr=DEVNULL,
11 universal_newlines=True)

Subscribers

People subscribed via source and target branches