Skip to content

[Data] Increase size of test_telemetry#52184

Merged
bveeramani merged 2 commits intomasterfrom
test-telemetry
Apr 9, 2025
Merged

[Data] Increase size of test_telemetry#52184
bveeramani merged 2 commits intomasterfrom
test-telemetry

Conversation

@bveeramani
Copy link
Copy Markdown
Member

Why are these changes needed?

To avoid timeouts.

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
@bveeramani bveeramani requested a review from a team as a code owner April 9, 2025 17:56
@bveeramani bveeramani enabled auto-merge (squash) April 9, 2025 19:12
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Apr 9, 2025
@github-actions github-actions bot disabled auto-merge April 9, 2025 19:13
@bveeramani bveeramani enabled auto-merge (squash) April 9, 2025 19:13
@bveeramani bveeramani disabled auto-merge April 9, 2025 23:02
@bveeramani bveeramani merged commit c563076 into master Apr 9, 2025
6 checks passed
@bveeramani bveeramani deleted the test-telemetry branch April 9, 2025 23:02
han-steve pushed a commit to han-steve/ray that referenced this pull request Apr 11, 2025
<!-- Thank you for your contribution! Please review
https://github.com/ray-project/ray/blob/master/CONTRIBUTING.rst before
opening a pull request. -->

<!-- Please add a reviewer to the assignee section when you create a PR.
If you don't have the access to it, we will shortly find a reviewer and
assign them to your PR. -->

## Why are these changes needed?

<!-- Please give a short summary of the change and the problem this
solves. -->

To avoid timeouts.

## Related issue number

<!-- For example: "Closes ray-project#1234" -->

## Checks

- [ ] I've signed off every commit(by using the -s flag, i.e., `git
commit -s`) in this PR.
- [ ] I've run `scripts/format.sh` to lint the changes in this PR.
- [ ] I've included any doc changes needed for
https://docs.ray.io/en/master/.
- [ ] I've added any new APIs to the API Reference. For example, if I
added a
method in Tune, I've added it in `doc/source/tune/api/` under the
           corresponding `.rst` file.
- [ ] I've made sure the tests are passing. Note that there might be a
few flaky tests, see the recent failures at https://flakey-tests.ray.io/
- Testing Strategy
   - [ ] Unit tests
   - [ ] Release tests
   - [ ] This PR is not tested :(

Signed-off-by: Balaji Veeramani <bveeramani@berkeley.edu>
Signed-off-by: Steve Han <stevehan2001@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-backlog go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants