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.

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

test branch behavior without new telemetry setup #341

Draft
wants to merge 6 commits into
base: branch-0.42
Choose a base branch
from

Conversation

msarahan
Copy link
Contributor

@msarahan msarahan commented Dec 6, 2024

This is meant to test the behavior of the new telemetry stuff in repos like UCXX that do not yet have it configured. The hope is that this new stuff will not show distracting errors.

Copy link

copy-pr-bot bot commented Dec 6, 2024

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@msarahan
Copy link
Contributor Author

msarahan commented Dec 6, 2024

/ok to test

1 similar comment
@msarahan
Copy link
Contributor Author

msarahan commented Dec 6, 2024

/ok to test

@msarahan msarahan closed this Dec 6, 2024
@msarahan msarahan reopened this Dec 6, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 6, 2024

/ok to test

1 similar comment
@msarahan
Copy link
Contributor Author

msarahan commented Dec 6, 2024

/ok to test

@msarahan msarahan closed this Dec 6, 2024
@msarahan msarahan reopened this Dec 6, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 6, 2024

/ok to test

2 similar comments
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

cycling to refresh shared-workflows branch

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

cycling to refresh shared-workflows branch

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

1 similar comment
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

@msarahan msarahan closed this Dec 9, 2024
@msarahan msarahan reopened this Dec 9, 2024
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

1 similar comment
@msarahan
Copy link
Contributor Author

msarahan commented Dec 9, 2024

/ok to test

raydouglass and others added 6 commits January 23, 2025 15:02
Forward-merge branch-0.42 into branch-0.43
This migrates amd64 CI jobs (PRs and nightlies) to use L4 GPUs from the NVKS cluster.

xref: rapidsai/build-infra#184

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

Approvers:
  - Gil Forsyth (https://github.com/gforsyth)

URL: rapidsai#358
Contributes to rapidsai/build-planning#146

Closes rapidsai#262

Proposes:

* setting `[tool.scikit-build].ninja.make-fallback = false`, so `scikit-build-core` will not silently fallback to using GNU Make if `ninja` is not available
* adding `devcontainers` CI jobs here

## Notes for reviewers

### Why add `devcontainers` CI jobs in this PR?

The `devcontainers` are useful for interactive development on this project (including for outside contributors).

`ucxx` building successfully in them is also important for other all-of-RAPIDS builds, like those from https://github.com/rapidsai/devcontainers and other builds based on that repo. Having CI jobs run in PRs (as most of RAPIDs does) allows us to catch issues right here in `ucxx` when they happen, instead of only finding out when `ucxxx` is built as part of a larger devcontainers-based build.

### How did you set up the `.devcontainers` folder?

Copied https://github.com/rapidsai/cugraph/tree/branch-25.04/.devcontainer and changed all the `cugraph` references to `ucxx`.

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

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

URL: rapidsai#363
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants