Merge "Permit Ironic to notify IPA it can support MD5"

ironic/api/controllers/v1/ramdisk.py

@@ -59,6 +59,7 @@ def config(token):
         # explicit True statement for newer agents to lock the setting
         # and behavior into place.
         'agent_token_required': True,
+        'agent_md5_checksum_enable': CONF.agent.allow_md5_checksum,
     }
 
 

ironic/conf/agent.py

@@ -163,6 +163,12 @@ opts = [
                help=_('Path to the TLS CA that is used to start the bare '
                       'metal API. In some boot methods this file can be '
                       'passed to the ramdisk.')),
+    cfg.BoolOpt('allow_md5_checksum',
+                default=True,
+                help=_('When enabled, the agent will be notified it is '
+                       'permitted to consider MD5 checksums. This option '
+                       'is expected to change to a default of False in a '
+                       '2024 release of Ironic.')),
 ]
 
 

ironic/tests/unit/api/controllers/v1/test_ramdisk.py

@@ -80,6 +80,7 @@ class TestLookup(test_api_base.BaseApiTest):
             'heartbeat_timeout': CONF.api.ramdisk_heartbeat_timeout,
             'agent_token': mock.ANY,
             'agent_token_required': True,
+            'agent_md5_checksum_enable': CONF.agent.allow_md5_checksum,
         }
         self.assertEqual(expected_config, data['config'])
         self.assertIsNotNone(data['config']['agent_token'])

releasenotes/notes/permit-ironic-to-tell-agent-about-md5-support-5c94da38c2f5dd10.yaml

@@ -0,0 +1,6 @@
+---
+features:
+  - |
+    Adds a ``[agent]allow_md5_checksum`` configuration
+    option which can be used to tell ``ironic-python-agent`` versions
+    newer than version ``9.4.0`` if MD5 is a permitted algorithm.