Add verify provider lifecycle to CLI#2207
Conversation
✅ Deploy Preview for transformerlab canceled.
|
Paragon SummaryThis pull request review analyzed 5 files and found no issues. The review examined code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools. Paragon did not detect any problems in the current diff. Proceed with merge after your normal checks. Adds a Key changes:
Confidence score: 5/5
5 files reviewed, 0 comments Tip: |
![polarityinc[bot]](https://avatars.githubusercontent.com/in/1544258?s=60&v=4){kind=small}
| print(json.dumps({"job_id": job_id, "status": "launched"})) | ||
| else: | ||
| console.print(f"[success]✓[/success] Storage probe launched as job [bold]{job_id}[/bold].") | ||
| console.print(f" Check with: lab provider verify-lifecycle {provider_id} (or inspect job {job_id}).") |
There was a problem hiding this comment.
Is this true? I thought this would launch a new job? Could just make this say check the job?
There was a problem hiding this comment.
You're right, re-running verify-lifecycle launches a fresh probe job rather than checking this one. Fixed the message to point at the job directly: Inspect it with: lab job info {job_id}.
dadmobile
left a comment
dadmobile
left a comment
There was a problem hiding this comment.
Changes look good. There are merge conflicts but I'm going to approve so you can merge after you fix them.
2 participants
No description provided.