Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

Commit

Permalink
CR fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
viovanov authored and Vlad Iovanov committed Apr 11, 2020
1 parent 687f782 commit 6b87b1c
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions pkg/bosh/manifest/cmd_instance_group_resolver.go
Original file line number Diff line number Diff line change
Expand Up @@ -325,11 +325,9 @@ func (igr *InstanceGroupResolver) renderBPM() error {
return errors.Wrapf(err, "Rendering BPM failed for instance group %s", igr.instanceGroup.Name)
}

err = igr.renderJobBPM(job, jobSpecFile, template)
if err != nil {
if err := igr.renderJobBPM(job, jobSpecFile, template); err != nil {
return errors.Wrapf(err, "Rendering BPM failed for instance group %s", igr.instanceGroup.Name)
}

}

return nil
Expand Down Expand Up @@ -385,7 +383,6 @@ func (igr *InstanceGroupResolver) renderJobBPM(currentJob *Job, jobSpecFile stri
jobIndexBPM := make([]bpm.Config, len(jobInstances))
for i, jobInstance := range jobInstances {
var renderedBPM = bpm.Config{}
var err error

properties := currentJob.Properties.ToMap()

Expand Down Expand Up @@ -433,6 +430,7 @@ func (igr *InstanceGroupResolver) renderJobBPM(currentJob *Job, jobSpecFile stri

// Merge processes if they also exist in Quarks
if currentJob.Properties.Quarks.BPM != nil && len(currentJob.Properties.Quarks.BPM.Processes) > 0 {
var err error
renderedBPM.Processes, err = renderedBPM.MergeProcesses(currentJob.Properties.Quarks.BPM.Processes)
if err != nil {
return errors.Wrapf(err, "failed to merge bpm information from quarks for job '%s'", currentJob.Name)
Expand Down

0 comments on commit 6b87b1c

Please sign in to comment.