Ironic

anaconda is unable to decode config drive from base64

Bug #2076088 reported by Maryna Savchenko
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ironic
In Progress
Medium
Maryna Savchenko

Bug Description

During the anaconda deployment, Ironic encodes the config drive to base 64 and writes the encoded string to the kickstart file. When the node boots and Anaconda starts the server installation, it can not decode it.
The reason is how Python's base 64 encoding works. Its format is b'encodedstring'.
Having 'b' at the beginning makes it impossible for Anaconda to decode it at a later stage.

To reproduce you need to have data on the metadata service:
* deploy node using Anaconda:
   - set the driver
   - create image
   - create a kickstart file with {{ ks_options.config_drive }} and assign it to the server or image
* Open the console and check anaconda logs in /var/anaconda during the installation

OpenStack Infra (hudson-openstack)
Changed in ironic:
status: New → In Progress
Riccardo Pittau (rpittau)
Changed in ironic:
importance: Undecided → Medium
assignee: nobody → Maryna Savchenko (msavchen)
tags: added: bug
tags: added: triaged
tags: added: anaconda
Revision history for this message
Robert Budden (rbudden) wrote (last edit ):

Any updates on this bug? We're affected by this bug as well (in Caracal). I see a triaged tag exists. Is there a fix/patch somewhere? I'd imagine this should be a simple fix. If one removes the 'b' the base64 decode completes successfully.

Revision history for this message
Robert Budden (rbudden) wrote :

Digging a bit deeper it looks like a simple patch like this should work to strip off the encoding:

[rbudden@**********:~/src/ironic/ironic/common]$ cdi kickstart_utils.py
diff --git a/ironic/common/kickstart_utils.py b/ironic/common/kickstart_utils.py
index 4e02e2ea7..f4b985a22 100644
--- a/ironic/common/kickstart_utils.py
+++ b/ironic/common/kickstart_utils.py
@@ -130,7 +130,7 @@ def _write_config_drive_content(content, file_path):
         file_path=file_path))
     kickstart_data.append("/bin/mkdir -p $DIRPATH\n")
     kickstart_data.append("CONTENT='{content}'\n".format(
- content=content))
+ content=content.decode("utf-8")))
     kickstart_data.append("echo $CONTENT | "
                           "/usr/bin/base64 --decode > "
                           "{file_path}".format(file_path=file_path))

Revision history for this message
Robert Budden (rbudden) wrote :

I can confirm the above patch works in our environment. OpenStack Kayobe deploy w/Caracal. I'll try and get this upstreamed.

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.