Merge ~smoser/cloud-init:bug/disable_vmware_customization-fix into cloud-init:master

Proposed by Scott Moser
Status: Merged
Merged at revision: 05afe04edbe4c28f2170194d226821c1e755ee2d
Proposed branch: ~smoser/cloud-init:bug/disable_vmware_customization-fix
Merge into: cloud-init:master
Diff against target: 25 lines (+5/-4)
1 file modified
tools/ds-identify (+5/-4)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Needs Fixing
Sankar Tanguturi (community) Approve
cloud-init Commiters Pending
Review via email: mp+318493@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Sankar Tanguturi (sankaraditya) wrote :

Awesome. Thanks Scott.

I have only one minor nit. In the commit message, can you also add a comment that /etc/cloud/cloud.cfg is also being used to check for the flag besides /etc/cloud/cloud.cfg.d/*

Thanks

review: Approve
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:cd32185dd161dd8c8e194e34d1bca1fc892490a5
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/318493/+edit-commit-message

https://jenkins.ubuntu.com/server/job/cloud-init-ci/67/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-arm64/67
    SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-ppc64el/67
    SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-s390x/67
    SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=vm-amd64/67
    SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=vm-i386/67

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/67/rebuild

review: Needs Fixing (continuous-integration)

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tools/ds-identify b/tools/ds-identify
2index 34bf064..e618963 100755
3--- a/tools/ds-identify
4+++ b/tools/ds-identify
5@@ -574,16 +574,17 @@ ovf_vmware_guest_customization() {
6 done
7 [ -n "$found" ] || return 1
8
9- # disable_vmware_customization defaults to False.
10- # any value then other than false means disabled.
11+ # vmware customization is disabled by default
12+ # (disable_vmware_customization=true). If it is set to false, then
13+ # user has requested customization.
14 local key="disable_vmware_customization"
15 local match="" bp="${PATH_CLOUD_CONFD}/cloud.cfg"
16- match="$bp.d/*[Oo][Vv][Ff]*.cfg"
17+ match="$bp $bp.d/*[Oo][Vv][Ff]*.cfg"
18 if check_config "$key" "$match"; then
19 debug 2 "${_RET_fname} set $key to $_RET"
20 case "$_RET" in
21 0|false|False) return 0;;
22- *) return;;
23+ *) return 1;;
24 esac
25 fi
26

Subscribers

People subscribed via source and target branches