8000 Ensure RAPIDS_ARTIFACTS_DIR is set for build metrics reports. by bdice · Pull Request #18192 · rapidsai/cudf · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Ensure RAPIDS_ARTIFACTS_DIR is set for build metrics reports. #18192

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
Merged
merged 1 commit into from
Mar 7, 2025
Merged

Ensure RAPIDS_ARTIFACTS_DIR is set for build metrics reports. #18192

merged 1 commit into from
Mar 7, 2025

Conversation

bdice
Copy link
Contributor
@bdice bdice commented Mar 7, 2025

Description

This fixes an issue where local reproductions of CI fail with the following error:

 ╭─ Finding outputs from recipe
 │ Loading variant config file: "/cudf/conda/recipes/libcudf/conda_build_config.yaml"
 │
 ╰─────────────────── (took 0 seconds)
Error:   × Failed to parse recipe

Error:
  × failed to render Jinja expression: invalid operation: Environment variable RAPIDS_ARTIFACTS_DIR not found (in <string>:1)
    ╭─[conda/recipes/libcudf/recipe.yaml:45:31]
 44 │         PARALLEL_LEVEL: ${{ env.get("PARALLEL_LEVEL") }}
 45 │         RAPIDS_ARTIFACTS_DIR: ${{ env.get("RAPIDS_ARTIFACTS_DIR") }}
    ·                               ───────────────────┬──────────────────
    ·                                                  ╰── invalid operation: Environment variable RAPIDS_ARTIFACTS_DIR not found
 46 │         SCCACHE_BUCKET: ${{ env.get("SCCACHE_BUCKET") }}
    ╰────

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@bdice bdice requested a review from a team as a code owner March 7, 2025 03:41
@bdice bdice requested a review from msarahan March 7, 2025 03:41
@davidwendt davidwendt added bug Something isn't working 3 - Ready for Review Ready for review by team non-breaking Non-breaking change labels Mar 7, 2025
@vyasr
Copy link
Contributor
vyasr commented Mar 7, 2025

/merge

@rapids-bot rapids-bot bot merged commit 4841fcc into rapidsai:branch-25.04 Mar 7, 2025
110 checks passed
@bdice bdice mentioned this pull request Mar 10, 2025
Merged
3 tasks
rapids-bot bot pushed a commit that referenced this pull request Mar 11, 2025
@bdice
Export RAPIDS_ARTIFACTS_DIR. (#18208)
c8da724 
Fix for #18192. The environment variables `RAPIDS_ARTIFACTS_DIR` is now exported, so that `rattler-build` can see it.

I verified this works for local reproductions of CI builds.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #18208
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@msarahan msarahan msarahan approved these changes

Assignees

@bdice bdice

Labels
3 - Ready for Review Ready for review by team bug Something isn't working non-breaking Non-breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bdice @vyasr @msarahan @davidwendt
0