Remove port_forward arg in cluster.run.

[rohinb2]

Not sure why we need this. Maybe I should just deprecate it.. but very very much doubt anyone is using it.

[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:42 PM EDT: Graphite rebased this pull request as part of a merge.
• Jun 30, 5:43 PM EDT: @rohinb2 merged this pull request with Graphite.