Merge ~canonical-kernel-team/+git/autotest-client-tests:phlin/fix-username-ovlfs-smk into ~canonical-kernel-team/+git/autotest-client-tests:master

Proposed by Po-Hsu Lin
Status: Merged
Merged at revision: 6aa89a365ac7ed70c249d713d27caff1d2aec8aa
Proposed branch: ~canonical-kernel-team/+git/autotest-client-tests:phlin/fix-username-ovlfs-smk
Merge into: ~canonical-kernel-team/+git/autotest-client-tests:master
Diff against target: 12 lines (+1/-1)
1 file modified
ubuntu_overlayfs_smoke_test/ubuntu_overlayfs_smoke_test.py (+1/-1)
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+448053@code.launchpad.net

Commit message

BugLink: https://bugs.launchpad.net/bugs/2029103

Use login user name instead of defaulting to 'ubuntu'. As we will use
different user name on some clouds.

Description of the change

Patch tested on Azure cloud as uncommitted change and works as expected.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

+1

review: Approve
Revision history for this message
Francis Ginther (fginther) wrote :

Applied. Thanks for noticing and fixing.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/ubuntu_overlayfs_smoke_test/ubuntu_overlayfs_smoke_test.py b/ubuntu_overlayfs_smoke_test/ubuntu_overlayfs_smoke_test.py
2index 83ed71d..70c1cb1 100644
3--- a/ubuntu_overlayfs_smoke_test/ubuntu_overlayfs_smoke_test.py
4+++ b/ubuntu_overlayfs_smoke_test/ubuntu_overlayfs_smoke_test.py
5@@ -29,7 +29,7 @@ class ubuntu_overlayfs_smoke_test(test.test):
6 return
7
8 cmd = os.path.join(self.srcdir, TEST_DIR, 'tests', test_name)
9- cmd = 'sudo -iu ubuntu {}'.format(cmd)
10+ cmd = 'sudo -iu {} {}'.format(os.getlogin(), cmd)
11 self.results = utils.system_output(cmd, retain_output=True)
12 print(self.results)
13

Subscribers

People subscribed via source and target branches

to all changes: