Merge ~dbungert/curtin:log-source-handler into curtin:master

Proposed by Dan Bungert
Status: Merged
Approved by: Dan Bungert
Approved revision: ca2c26453c4d5286b2a8b54d93c8e391c4c3e8c7
Merge reported by: Server Team CI bot
Merged at revision: not available
Proposed branch: ~dbungert/curtin:log-source-handler
Merge into: curtin:master
Diff against target: 20 lines (+2/-0)
1 file modified
curtin/commands/extract.py (+2/-0)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
Michael Hudson-Doyle Approve
Review via email: mp+451309@code.launchpad.net

Commit message

extract: log source information

If you try hard enough and follow mount commands backwards you can
surely get this same information, but when analyzing what the extract is
actually doing it's nice to just be able to see what the image stack is.

To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) :
review: Approve
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/curtin/commands/extract.py b/curtin/commands/extract.py
2index fc7749c..206790e 100644
3--- a/curtin/commands/extract.py
4+++ b/curtin/commands/extract.py
5@@ -114,6 +114,7 @@ class LayeredSourceHandler(AbstractSourceHandler):
6
7 def setup(self):
8 self._tmpdir = tempfile.mkdtemp()
9+ LOG.debug(f"Setting up Layered Source for stack {self.image_stack}")
10 try:
11 self._download()
12 # Check that all images exists on disk and are not empty
13@@ -158,6 +159,7 @@ class TrivialSourceHandler(AbstractSourceHandler):
14 self.path = path
15
16 def setup(self):
17+ LOG.debug(f"Setting up Trivial Source for stack {self.path}")
18 return self.path
19
20 def cleanup(self):

Subscribers

People subscribed via source and target branches