Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(notebooks): remove --name from gradient notebooks start command #362

Conversation

cwetherill-ps
Copy link
Contributor

Tested against psdev, created a notebook in the UI, stopped it, ran

python3 -m gradient notebooks start \
  --clusterId <cluster ID> \
  --machineType psdev-worker
  --notebookId <notebook ID>

and saw the notebook successfully restart. Verified that --name was no
longer a valid CLI param for the command with

python3 -m gradient notebooks start --help

Tested against psdev, created a notebook in the UI, stopped it, ran
```bash
python3 -m gradient notebooks start \
  --clusterId <cluster ID> \
  --machineType psdev-worker
  --notebookId <notebook ID>
```
and saw the notebook successfully restart. Verified that `--name` was no
longer a valid CLI param for the command with
```bash
python3 -m gradient notebooks start --help
```
@cwetherill-ps cwetherill-ps merged commit c165974 into master Sep 3, 2021
@cwetherill-ps cwetherill-ps deleted the cwetherill/pla-407-remove---name-from-gradient-notebooks branch September 3, 2021 14:42
@PSBOT
Copy link

PSBOT commented Sep 3, 2021

🎉 This PR is included in version 1.7.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@PSBOT PSBOT added the released label Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants