starlingx/openstack-armada-app
Code Issues Proposed changes

Fix for stx-openstack upload failure

Browse Source 
During the review of the change [1], a block of source code has
been completely indented. This completely changed the structure of
the source code previously validated at Patchset 8 [2], causing the
stx-openstack upload failure.

The fix was to undo the wrong indentation, taking the code back to the
correct indentation tested at Patchset 8 [2].

[1] https://review.opendev.org/c/starlingx/openstack-armada-app/+/938407
[2] https://review.opendev.org/c/starlingx/openstack-armada-app/+/938407/8

Test Plan:

PASS: build-pkg openstack -c -l openstack
PASS: build openstack tarball
PASS: upload and apply openstack on a system

Closes-Bug: #2093370

Change-Id: I3c405b473515011a0ffb4568873708e72909ce3a
Signed-off-by: kgoncalv <kayo.goncalvesdacosta@windriver.com>
This commit is contained in:
kgoncalv 2025-01-09 12:41:11 -03:00 committed by Kayo Lourenço Gonçalves da Costa
parent 42cbe2261a
commit 70784aa0da
1 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
python3-k8sapp-openstack/k8sapp_openstack/k8sapp_openstack/helm/nova.py
View File

@ -681,24 +681,24 @@ class NovaHelm(openstack.OpenstackBaseHelm):
}
}
}
# add first host to config_map
if not config_map:
config_map.append({
'conf': host_nova['conf'],
'name': [hostname]
})
else:
# check if an identical configuration already exists in the config_map
for config in config_map:
if config['conf'] == host_nova['conf']:
config['name'].append(hostname)
break
else:
# add new config to config_map
# add first host to config_map
if not config_map:
config_map.append({
'conf': host_nova['conf'],
'name': [hostname]
})
else:
# check if an identical configuration already exists in the config_map
for config in config_map:
if config['conf'] == host_nova['conf']:
config['name'].append(hostname)
break
else:
# add new config to config_map
config_map.append({
'conf': host_nova['conf'],
'name': [hostname]
})
host_list.extend(config_map)
return host_list