Skip to content

[autoscaler/k8s] [minor] Kubernetes rsync verbosity fixed#11716

Merged
ericl merged 1 commit intoray-project:masterfrom
DmitriGekhtman:dmitri/quiet-kubernetes-rsync
Oct 30, 2020
Merged

[autoscaler/k8s] [minor] Kubernetes rsync verbosity fixed#11716
ericl merged 1 commit intoray-project:masterfrom
DmitriGekhtman:dmitri/quiet-kubernetes-rsync

Conversation

@DmitriGekhtman
Copy link
Copy Markdown
Contributor

@DmitriGekhtman DmitriGekhtman commented Oct 30, 2020

Why are these changes needed?

Currently, ray rsync always produces verbose output on a Kubernetes cluster. This PR changes the behavior to be more consistent with the other providers -- the output is verbose when copying files specified on the command line but quiet when syncing file mounts.

Related issue number

Resolves #11685.

Checks

  • 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 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 :(

Kubernetes syncer looks not too spammy when run on local minikube cluster.

@DmitriGekhtman DmitriGekhtman requested a review from ericl October 30, 2020 05:40
@DmitriGekhtman DmitriGekhtman added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Oct 30, 2020
@DmitriGekhtman
Copy link
Copy Markdown
Contributor Author

cc @mkoh-asapp

@ericl ericl merged commit 8816d34 into ray-project:master Oct 30, 2020
@DmitriGekhtman DmitriGekhtman deleted the dmitri/quiet-kubernetes-rsync branch November 2, 2020 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests-ok The tagger certifies test failures are unrelated and assumes personal liability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[autoscaler/k8s] Kubernetes syncer is kind of spammy

2 participants