← Back to team overview

cloud-init-dev team mailing list archive

[Merge] ~tribaal/cloud-init:dont-overwrite-etc-hosts-in-example into cloud-init:master

 

Chris Glass has proposed merging ~tribaal/cloud-init:dont-overwrite-etc-hosts-in-example into cloud-init:master.

Commit message:
Make the "command on first boot" example append to /etc/hosts instead of overwriting.

Requested reviews:
  cloud init development team (cloud-init-dev)

For more details, see:
https://code.launchpad.net/~tribaal/cloud-init/+git/cloud-init/+merge/310043

This branch simply fixes the "Run commands on first boot" example to append to /etc/hosts instead of overwriting it.
It's only an example, so hopefully people don't run it as-is, but it is probably good to have an accurate example.

This also served me as a learning experience to "kick the tires" for the contribution process, should the need to contribute further arise. Should my addition be wrong or misguided please feel free to close this merge request without further notice.
-- 
Your team cloud init development team is requested to review the proposed merge of ~tribaal/cloud-init:dont-overwrite-etc-hosts-in-example into cloud-init:master.
diff --git a/doc/examples/cloud-config-boot-cmds.txt b/doc/examples/cloud-config-boot-cmds.txt
index 3e59755..5c05bef 100644
--- a/doc/examples/cloud-config-boot-cmds.txt
+++ b/doc/examples/cloud-config-boot-cmds.txt
@@ -11,5 +11,5 @@
 # - the INSTANCE_ID variable will be set to the current instance id.
 # - you can use 'cloud-init-boot-per' command to help only run once
 bootcmd:
- - echo 192.168.1.130 us.archive.ubuntu.com > /etc/hosts
+ - echo 192.168.1.130 us.archive.ubuntu.com >> /etc/hosts
  - [ cloud-init-per, once, mymkfs, mkfs, /dev/vdb ]