Ubuntu
walinuxagent package

[SRU] Add "readme" to Azure's ephemeral (resource) disk

Bug #1382257 reported by Stephen A. Zarkos
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
walinuxagent (Ubuntu)
Fix Released
Undecided
Unassigned
Precise
Fix Released
Undecided
Unassigned
Trusty
Fix Released
Undecided
Unassigned
Utopic
Fix Released
Undecided
Unassigned
Vivid
Fix Released
Undecided
Unassigned

Bug Description

[JUSTIFICATION] Ephemeral disks on Azure are per-boot ephemeral are unsafe for storing data. Microsoft has asked that a warning file be placed on the disks. This would normally be handled by the agent, however, since Ubuntu uses cloud-init for provisioning of the ephemeral disk, the warning file is not placed.

[FIX] Added an upstart job that places the warning file on the ephemeral device when the instance is provisioned using Cloud-init.

[TEST]
1. Install new version of walinuxagent
2. Reboot
3. Confirm that DATALOSS_WARNING_README.txt exists
4. Run "chattr -i /mnt/DATALOSS_WARNING_README.txt"
5. Reboot
6. Confirm that DATALOSS_WARNING_README.txt is re-written

[ORIGINAL REPORT]
When the Azure Linux agent handles the resource disk it adds a file on the newly formatted partition called "DATALOSS_WARNING_README.txt". The purpose of this file is to warn customers that the disk is temporary and may be wiped if the VM is rebooted or moved to another host.

Since cloud-init handles the resource disk for Ubuntu VMs in Azure, can we have cloud-init add this file as well when it formats the resource/ephemeral disk?

In the Linux agent we have to following code to define this file:

---------------------------------
README_FILENAME="DATALOSS_WARNING_README.txt"
README_FILECONTENT="""\
WARNING: THIS IS A TEMPORARY DISK.
Any data stored on this drive is SUBJECT TO LOSS and THERE IS NO WAY TO RECOVER IT.
Please do not use this disk for storing any personal or application data.
For additional details to please refer to the MSDN documentation at : http://msdn.microsoft.com/en-us/library/windowsazure/jj672979.aspx
"""
---------------------------------

See original description
Ben Howard (darkmuggle-deactivatedaccount)
Changed in cloud-init (Ubuntu):
assignee: nobody → Ben Howard (utlemming)
importance: Undecided → High
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Uploaded a new version of walinuxagent that includes an upstart job that will provide the warning.

Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

This will need to be SRU'd back to prior releases. I debated making this a build recipe piece, however, I think that the logic here should be up-datable.

After it appears in Vivid, I'll start the SRU process for this.

summary: - Add "readme" to Azure's ephemeral (resource) disk
+ [SRU] Add "readme" to Azure's ephemeral (resource) disk
description: updated
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Uploaded new version for 12.04, 14.04 and 14.10; pending acceptance to -proposed and the usual SRU baking period.

I spent a good afternoon testing this to make sure that it follows the desired behavior. I tested it both on new images built from PPA and upgraded machines.

Revision history for this message
Chris J Arges (arges) wrote :

I see this is fixed in vivid, so marking it so.

Changed in cloud-init (Ubuntu Vivid):
status: New → Fix Released
Revision history for this message
Chris J Arges (arges) wrote :

Ben, the upload for trusty seems to be the utopic version, can you ensure that the right version was uploaded for trusty that contains just the SRU? Thanks,

Revision history for this message
Chris Halse Rogers (raof) wrote : Please test proposed package

Hello Stephen, or anyone else affected,

Accepted walinuxagent into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/walinuxagent/2.0.8-0ubuntu1~12.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote : Re: [Bug 1382257] Re: [SRU] Add "readme" to Azure's ephemeral (resource) disk

Chris, the upload for Trusty is correct, at least according to guidance
I have received from the foundations team in the past. The guidance is
that when the only difference between what is in a prior release and the
current development release is the same sans the topic of the SRU, to
backport the entire package. Since 15.04 has walinux_2.0.8-0ubunut3, and
the only difference between ubuntu1 and ubuntu3 is the warning script, I
simply backported the 15.04 package to 14.04 and 14.10. If the guidance
has changed, I'll be more than happy to re-upload.

Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello Stephen, or anyone else affected,

Accepted walinuxagent into utopic-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/walinuxagent/2.0.8-0ubuntu3~14.10.0 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Revision history for this message
Chris J Arges (arges) wrote :

Hello Stephen, or anyone else affected,

Accepted walinuxagent into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/walinuxagent/2.0.8-0ubuntu3~14.04.0 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

no longer affects: cloud-init (Ubuntu)
no longer affects: cloud-init (Ubuntu Precise)
no longer affects: cloud-init (Ubuntu Trusty)
no longer affects: cloud-init (Ubuntu Utopic)
no longer affects: cloud-init (Ubuntu Vivid)
Changed in walinuxagent (Ubuntu Vivid):
status: New → Fix Released
assignee: nobody → Ben Howard (utlemming)
Changed in walinuxagent (Ubuntu Utopic):
status: New → Fix Committed
Changed in walinuxagent (Ubuntu Precise):
status: New → Fix Committed
Changed in walinuxagent (Ubuntu Trusty):
status: New → Fix Committed
Revision history for this message
Stephen A. Zarkos (stevez) wrote :

We tested this package using the [TEST] steps above and can confirm that the readme is created as expected. As an additional step I also resized the VM to ensure that the temporary disk would need to be reformatted, and saw in this case that the readme was also created as expected. Well done!

Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Built test images from -proposed for 12.04, 14.04 and 14.10 as well as upgraded existing instances. I can confirm that the test case has passed in addition to the testing by Microsoft in comment #10. Marking as "verification done".

tags: added: verification-done
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package walinuxagent - 2.0.8-0ubuntu1~12.04.1

---------------
walinuxagent (2.0.8-0ubuntu1~12.04.1) precise; urgency=medium

  * Backport of ephemeral disk warning from 15.04 (LP: #1382257).
 -- Ben Howard <email address hidden> Fri, 07 Nov 2014 13:57:47 -0700

Changed in walinuxagent (Ubuntu Precise):
status: Fix Committed → Fix Released
Revision history for this message
Chris J Arges (arges) wrote : Update Released

The verification of the Stable Release Update for walinuxagent has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package walinuxagent - 2.0.8-0ubuntu3~14.04.0

---------------
walinuxagent (2.0.8-0ubuntu3~14.04.0) trusty; urgency=medium

  * Backport to 14.04 (LP: #1382257)
 -- Ben Howard <email address hidden> Fri, 07 Nov 2014 10:36:23 -0700

Changed in walinuxagent (Ubuntu Trusty):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package walinuxagent - 2.0.8-0ubuntu3~14.10.0

---------------
walinuxagent (2.0.8-0ubuntu3~14.10.0) utopic; urgency=medium

  * Backport to 14.10 (LP: #1382257)
 -- Ben Howard <email address hidden> Fri, 07 Nov 2014 10:36:37 -0700

Changed in walinuxagent (Ubuntu Utopic):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.