Skip to content

build(ci): fix a gradle implicit dependency error #13522

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

Merged
merged 1 commit into from
May 15, 2025

Conversation

chakru-r
Copy link
Collaborator

Gradle was reporting an implicit dependency that can cause errors depending on the order of execution. This gradle error appears to be inconsistent -- but when it failed, it fails with

A problem was found with the configuration of task ':metadata-service:plugin:processTestResources' (type 'ProcessResources').
  - Gradle detected a problem with the following location: '/home/runner/work/datahub/datahub/metadata-service/plugin/src/test/resources'.
    
    Reason: Task ':metadata-service:plugin:processTestResources' uses this output of task ':metadata-service:plugin:src:test:sample-test-plugins:copyJar' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.
    
    Possible solutions:
      1. Declare task ':metadata-service:plugin:src:test:sample-test-plugins:copyJar' as an input of ':metadata-service:plugin:processTestResources'.
      2. Declare an explicit dependency on ':metadata-service:plugin:src:test:sample-test-plugins:copyJar' from ':metadata-service:plugin:processTestResources' using Task#dependsOn.
      3. Declare an explicit dependency on ':metadata-service:plugin:src:test:sample-test-plugins:copyJar' from ':metadata-service:plugin:processTestResources' using Task#mustRunAfter.
    
    For more information, please refer to https://docs.gradle.org/8.11.1/userguide/validation_problems.html#implicit_dependency in the Gradle documentation.

@github-actions github-actions bot added the devops PR or Issue related to DataHub backend & deployment label May 15, 2025
@chakru-r chakru-r force-pushed the cr-oss-build-failure branch from 779ee19 to 73be90d Compare May 15, 2025 07:23
Copy link

codecov bot commented May 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label May 15, 2025
@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels May 15, 2025
@chakru-r chakru-r merged commit 9cee32c into master May 15, 2025
59 of 60 checks passed
@chakru-r chakru-r deleted the cr-oss-build-failure branch May 15, 2025 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops PR or Issue related to DataHub backend & deployment pending-submitter-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants