airship
/
armada
Code Issues Proposed changes

Fix: Address typos and error in Armada output 

This fixes typos in the Armada output. It also fixes an error in
which the output for building dependencies was referencing a
releases's dependencies for the release name instead of the actual
release

Change-Id: Ieb30c112b838c0fb22d4af4e18ddb8c182b98fdb
This commit is contained in:
Steve Wilkerson 2018-01-19 17:22:24 -06:00
parent cacc84a1b0
commit 22771fe45c
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
armada/cli/apply.py
View File

@ -124,13 +124,13 @@ class ApplyManifest(CliAction):
def output(self, resp):
for result in resp:
if not resp[result] and not result == 'diff':
self.logger.info('Did not performed chart %s(s)', result)
self.logger.info('Did not perform chart %s(s)', result)
elif result == 'diff' and not resp[result]:
self.logger.info('No Relase changes detected')
self.logger.info('No release changes detected')
for ch in resp[result]:
if not result == 'diff':
msg = 'Chart {} was {}'.format(ch, result)
msg = 'Chart {} was {}ed'.format(ch, result)
self.logger.info(msg)
else:
self.logger.info('Chart values diff')

2
armada/handlers/chartbuilder.py
View File

@ -223,7 +223,7 @@ class ChartBuilder(object):
dependencies = []
for dep in self.chart.dependencies:
LOG.info("Building dependency chart %s for release %s.",
dep.chart.chart_name, dep.chart.release)
dep.chart.chart_name, self.chart.release)
try:
dependencies.append(ChartBuilder(dep.chart).get_helm_chart())
except Exception: