Fix #602.

[csm]

Fixes #602.

• grails-app/controllers/com/netflix/asgard/ImageController.groovy (ImageDeleteCommand): remove configService. Call into grailsApplication.config directly, it appears that configService doesn't get autowired in this command.

[cloudbees-pull-request-builder]

asgard-pull-requests #495 FAILURE
Looks like there's a problem with this pull request

[csm]

I don't have any idea what failed, there aren't any details.

Note that this is against the 1.5.1 tag, so that could be it.

[cloudbees-pull-request-builder]

asgard-pull-requests #496 FAILURE
Looks like there's a problem with this pull request

[csm]

OK, now I have no clue what your builtbot is complaining about. The test results show zero failures, and zero codenarc issues. What's the deal?

[claymccoy]

Looks like a compilation error:

Compilation error: startup failed:
/scratch/jenkins/workspace/asgard-pull-requests/grails-app/controllers/com/netflix/asgard/ImageController.groovy:
433: unexpected token: = @ line 433, column 17.
=
command.grailsApplication.config.promote?.targetServerRootUrls ?: []
^

On Tue, May 12, 2015 at 1:35 PM, Casey Marshall notifications@github.com
wrote:

OK, now I have no clue what your builtbot is complaining about. The
test results show zero failures, and zero codenarc issues. What's the deal?

—
Reply to this email directly or view it on GitHub
#670 (comment).

[cloudbees-pull-request-builder]

asgard-pull-requests #497 SUCCESS
This pull request looks good

[csm]

Indeed, that wasn't quite the way to split a long line in Groovy.