benWize commented on a change in pull request #14915:
URL: https://github.com/apache/beam/pull/14915#discussion_r644253241
##########
File path: .github/workflows/java_tests.yml
##########
@@ -162,6 +160,15 @@ jobs:
service_account_key: ${{ secrets.GCP_SA_KEY }}
project_id: ${{ secrets.GCP_PROJECT_ID }}
export_default_credentials: true
+ - name: Set Java Version
+ uses: actions/setup-java@v1
+ with:
+ java-version: 8
+ - name: Remove default github maven configuration
+ # This step is a workaround to avoid a decryption issue of Beam's
+ # gradle-command-action plugin and github's provided maven
+ # settings.xml file
+ run: rm ~/.m2/settings.xml
Review comment:
Thanks Brian!
I think the problem is that
[setup-java](https://github.com/actions/setup-java) plugin creates by default a
`~/.m2/settings.xml `file that cannot be decrypted by `gradle-command-action`
plugin because there isn't a `~/.m2/settings-security.xml`. In this context I
think is not necessary the `~/.m2/settings.xml` file so we can keep it like
this. A possible fix in `gradle-command-action` could be to detect if there is
not a `~/.m2/settings-security.xml `file and in that case, do not try to
decrypt the password. I saw similar issues
[here](https://github.com/mark-vieira/gradle-maven-settings-plugin/issues/15)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
