Merge ~erik-m-bray+ubuntu/cloud-init:doc/format/part-handler-fixes into cloud-init:master

Proposed by E. Madison Bray
Status: Merged
Merged at revision: 145410f81c144a46cf5ce0324ff4454fa9f54ad0
Proposed branch: ~erik-m-bray+ubuntu/cloud-init:doc/format/part-handler-fixes
Merge into: cloud-init:master
Diff against target: 31 lines (+7/-6)
1 file modified
doc/rtd/topics/format.rst (+7/-6)
Reviewer Review Type Date Requested Status
Scott Moser Approve
Review via email: mp+313607@code.launchpad.net

Description of the change

Fix typos and clarify some aspects of the part-handler

The existing documentation referred to a handle_type method when it
really should be handle_part. It also referred to 'methods' when it
really should say 'functions' to be clear (while it's true the built-in
handlers are classes with methods of these names, in this context we
mean module-level functions).

Also clarified that a part-handler should come before the parts that
it handles, and can override built-in handlers.

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/doc/rtd/topics/format.rst b/doc/rtd/topics/format.rst
2index ed87d3e..436eb00 100644
3--- a/doc/rtd/topics/format.rst
4+++ b/doc/rtd/topics/format.rst
5@@ -127,11 +127,11 @@ Begins with: ``#cloud-boothook`` or ``Content-Type: text/cloud-boothook`` when u
6 Part Handler
7 ============
8
9-This is a ``part-handler``. It will be written to a file in ``/var/lib/cloud/data`` based on its filename (which is generated).
10-This must be python code that contains a ``list_types`` method and a ``handle_type`` method.
11-Once the section is read the ``list_types`` method will be called. It must return a list of mime-types that this part-handler handles.
12+This is a ``part-handler``: It contains custom code for either supporting new mime-types in multi-part user data, or overriding the existing handlers for supported mime-types. It will be written to a file in ``/var/lib/cloud/data`` based on its filename (which is generated).
13+This must be python code that contains a ``list_types`` function and a ``handle_part`` function.
14+Once the section is read the ``list_types`` method will be called. It must return a list of mime-types that this part-handler handles. Because mime parts are processed in order, a ``part-handler`` part must precede any parts with mime-types it is expected to handle in the same user data.
15
16-The ``handle_type`` method must be like:
17+The ``handle_part`` function must be defined like:
18
19 .. code-block:: python
20
21@@ -141,8 +141,9 @@ The ``handle_type`` method must be like:
22 # filename = the filename of the part (or a generated filename if none is present in mime data)
23 # payload = the parts' content
24
25-Cloud-init will then call the ``handle_type`` method once at begin, once per part received, and once at end.
26-The ``begin`` and ``end`` calls are to allow the part handler to do initialization or teardown.
27+Cloud-init will then call the ``handle_part`` function once before it handles any parts, once per part received, and once after all parts have been handled.
28+The ``'__begin__'`` and ``'__end__'`` sentinels allow the part handler to do initialization or teardown before or after
29+receiving any parts.
30
31 Begins with: ``#part-handler`` or ``Content-Type: text/part-handler`` when using a MIME archive.
32

Subscribers

People subscribed via source and target branches