Restructure cluster.run to be more clear.

[rohinb2]

don't really need the self.on_this_cluster() section anymore.

to me it looks like -- default_env will always be set to something, and so we'll use self.call for all the on_this_cluster stuff, which will branch in there on its own.

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: runhouse/resources/hardware/cluster.py

Function	Unhandled Issue
run	ValueError: Error checking server: Internal Server Error. Is the server running? ... Event Count: 5

---

_{Did you find this useful? React with a 👍 or 👎}

[rohinb2]

• Add tests for cluster.run run within the cluster itself. #944
• Remove port_forward arg in cluster.run. #934
• Disallow passing node when on_this_cluster. #946
• Restructure cluster.run to be more clear. #937 👈
• No longer save the "run" of anything run via ssh. #936
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rohinb2 and the rest of your teammates on Graphite

[rohinb2]

Merge activity

• Jun 30, 5:33 PM EDT: @rohinb2 started a stack merge that includes this pull request via Graphite.
• Jun 30, 5:36 PM EDT: Graphite rebased this pull request as part of a merge.
• Jun 30, 5:38 PM EDT: @rohinb2 merged this pull request with Graphite.