-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy #5836
Comments
@ezimanyi may be able to shed some light on the artifact behavior here. |
Here is the stack trace:
|
@ezimanyi I got past the error above. The config artifacts were returning a HashMap (not sure if that's a groovy behavior vs java). Anyhow, I converted the HashMap to Artifact object (from kork project) and orca is not failing anymore. P.S. It's working now with my fix, I successfully deployed. I'll submit my code change later. |
@guido9j : Thanks for the update, and figuring this out! It does indeed look like the change in #spinnaker/orca#3008 did not correctly account for whether the Please do submit your fix for this, and feel free to tag me on the PR! |
groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact.
…rencing github config artifact for GAE deploy - spinnaker/spinnaker#5836 (#3739) * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact.' fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 * Change to objectMapper * Removed left over Slf4j lines
…rencing github config artifact for GAE deploy - spinnaker/spinnaker#5836 (#3739) * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact.' fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 * Change to objectMapper * Removed left over Slf4j lines (cherry picked from commit ebb04d2)
…rencing github config artifact for GAE deploy - spinnaker/spinnaker#5836 (#3739) (#3757) * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact.' fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 * Change to objectMapper * Removed left over Slf4j lines (cherry picked from commit ebb04d2) Co-authored-by: guido9j <james_guido@homedepot.com>
* fix(provider/appengine): Temporarily revert fix This reverts commit 711c21a. After adding tests to demonstrate the bug and fix, we'll re-apply. * test(appengine): Add tests demonstrating AppEngine artifact bug This commit adds tests to demonstrate the regression in the AppEngine artifact handling. The next commit will fix the bug and update the tests. * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 (#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact. Co-Authored-By: Eric Zimanyi <ezimanyi@google.com> Co-authored-by: guido9j <james_guido@homedepot.com>
* fix(provider/appengine): Temporarily revert fix This reverts commit 711c21a. After adding tests to demonstrate the bug and fix, we'll re-apply. * test(appengine): Add tests demonstrating AppEngine artifact bug This commit adds tests to demonstrate the regression in the AppEngine artifact handling. The next commit will fix the bug and update the tests. * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 (#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact. Co-Authored-By: Eric Zimanyi <ezimanyi@google.com> Co-authored-by: guido9j <james_guido@homedepot.com> (cherry picked from commit 4a26d90)
* fix(provider/appengine): Temporarily revert fix This reverts commit 711c21a. After adding tests to demonstrate the bug and fix, we'll re-apply. * test(appengine): Add tests demonstrating AppEngine artifact bug This commit adds tests to demonstrate the regression in the AppEngine artifact handling. The next commit will fix the bug and update the tests. * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 (#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact. Co-Authored-By: Eric Zimanyi <ezimanyi@google.com> Co-authored-by: guido9j <james_guido@homedepot.com> (cherry picked from commit 4a26d90)
… (#3774) * fix(provider/appengine): Temporarily revert fix This reverts commit 711c21a. After adding tests to demonstrate the bug and fix, we'll re-apply. * test(appengine): Add tests demonstrating AppEngine artifact bug This commit adds tests to demonstrate the regression in the AppEngine artifact handling. The next commit will fix the bug and update the tests. * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 (#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact. Co-Authored-By: Eric Zimanyi <ezimanyi@google.com> Co-authored-by: guido9j <james_guido@homedepot.com> (cherry picked from commit 4a26d90) Co-authored-by: Eric Zimanyi <ezimanyi@google.com>
…3772) (#3775) * test(appengine): Add tests demonstrating AppEngine artifact bug (#3772) * fix(provider/appengine): Temporarily revert fix This reverts commit 711c21a. After adding tests to demonstrate the bug and fix, we'll re-apply. * test(appengine): Add tests demonstrating AppEngine artifact bug This commit adds tests to demonstrate the regression in the AppEngine artifact handling. The next commit will fix the bug and update the tests. * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 (#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact. Co-Authored-By: Eric Zimanyi <ezimanyi@google.com> Co-authored-by: guido9j <james_guido@homedepot.com> (cherry picked from commit 4a26d90) * test(appengine): Fix incorrect class names and imports A refactor between 1.19 and 1.20 changed a lot of class names and import paths, so the backport didn't apply cleanly. This commit updates the (non-compiling) backport so that it refers to the classes as they exist on the 1.19 branch. Co-authored-by: Eric Zimanyi <ezimanyi@google.com>
@ezimanyi : Now that this issue been resolved. Am I supposed to close this issue? (actually, I don't think I can close it) Never mind! I see you are working on test cases. Thanks |
@guido9j : I ended up merging the test cases, so I think this can be closed. Thanks for following up! |
…naker#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact.
…naker#3772) * fix(provider/appengine): Temporarily revert fix This reverts commit 711c21a. After adding tests to demonstrate the bug and fix, we'll re-apply. * test(appengine): Add tests demonstrating AppEngine artifact bug This commit adds tests to demonstrate the regression in the AppEngine artifact handling. The next commit will fix the bug and update the tests. * fix(provider/appengine): Fix for issue spinnaker/spinnaker#5836 (spinnaker#3755) groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy. The config artifacts were of instance of HashMap not an Artifact object. Added logic to translate HashMap to Artifact. Co-Authored-By: Eric Zimanyi <ezimanyi@google.com> Co-authored-by: guido9j <james_guido@homedepot.com>
Issue Summary:
groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy
Cloud Provider(s):
App Engine
Environment:
GCP
Feature Area:
Orca
Description:
groovy.lang.MissingMethodException when referencing github config artifact for GAE deploy
Steps to Reproduce:
Create a pipeline with the following JSON
Attempt to run the pipeline. The following error is shown in Deck.
Why is the artifact being bound to a HashMap?
The text was updated successfully, but these errors were encountered: