Update den auth default to None in cluster factory

[carolineechen]

default value False for den auth in cluster factory caused alt_options to be set, so calling cluster("name") would sometimes result in a config mismatch and fail to reconstruct the resource correctly. set it to none in factory and correspondingly update to False in cluster init instead.

[sentry-io]

🔍 Existing Issues For Review

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

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

Function	Unhandled Issue
ondemand_cluster	AttributeError: module 'sky' has no attribute 'global_user_state' __m... Event Count: 4
ondemand_cluster	NameError: name 'sky' is not defined main in ... Event Count: 3

---

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

[carolineechen]

• Update den auth default to None in cluster factory #784 👈
• main

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

Join @carolineechen and the rest of your teammates on Graphite