Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] mvn-verify-check failed cache non-snapshot deps #12152

Closed
pxLi opened this issue Feb 17, 2025 · 1 comment
Closed

[BUG] mvn-verify-check failed cache non-snapshot deps #12152

pxLi opened this issue Feb 17, 2025 · 1 comment
Assignees
@YanxuanLiu
Labels
bug Something isn't working build Related to CI / CD or cleanly building

Comments

@pxLi
Copy link
Member

pxLi commented Feb 17, 2025 •
edited
Loading

Describe the bug

failed https://github.com/NVIDIA/spark-rapids/actions/runs/13303014299/job/37324334542?pr=12129

https://github.com/NVIDIA/spark-rapids/blob/branch-25.02/.github/workflows/mvn-verify-check/get-deps-sha1.sh#L27-L32 relies on the snapshot repo to fetch sha info, this would fail the dep updates change of #12129 which require to get info from non-snapshot repos

Please help fix in 25.02 as we will perform another release in 25.02 shortly thanks

Steps/Code to reproduce bug
Please provide a list of steps or a code sample to reproduce the issue.
Avoid posting private or sensitive data.

Expected behavior
A clear and concise description of what you expected to happen.

Environment details (please complete the following information)

  • Environment location: [Standalone, YARN, Kubernetes, Cloud(specify cloud provider)]
  • Spark configuration settings related to the issue

Additional context
Add any other context about the problem here.
@pxLi pxLi added ? - Needs Triage Need team to review and classify bug Something isn't working labels Feb 17, 2025
@pxLi pxLi changed the title [BUG] mvn-verify- [BUG] mvn-verify-check failed cache non-snapshot deps Feb 17, 2025
@pxLi pxLi added the build Related to CI / CD or cleanly building label Feb 17, 2025
@pxLi pxLi mentioned this issue Feb 17, 2025
Merged
pxLi pushed a commit that referenced this issue Feb 17, 2025
@nvauto @YanxuanLiu
Update dependency version JNI, private, hybrid to 25.02.0 [skip ci] (#…
4356842 
…12129)

Wait for the pre-merge CI job to SUCCEED

also fix #12152

---------

Signed-off-by: nvauto <70000568+nvauto@users.noreply.github.com>
Co-authored-by: YanxuanLiu <yanxuanl@nvidia.com>
@pxLi
Copy link
Member Author

pxLi commented Feb 17, 2025

fixed in #12129

@pxLi pxLi closed this as completed Feb 17, 2025
@sameerz sameerz removed the ? - Needs Triage Need team to review and classify label Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@YanxuanLiu YanxuanLiu

Labels
bug Something isn't working build Related to CI / CD or cleanly building
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@sameerz @pxLi @YanxuanLiu