Skip to content

PD-84 - Update getting started guide: refine CVM creation instructions and add online CVM image #86

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

Merged
merged 3 commits into from
Jul 4, 2025

Conversation

SammyOina
Copy link
Contributor

@SammyOina SammyOina commented Jun 9, 2025

What type of PR is this?

What does this do?

Which issue(s) does this PR fix/relate to?

Resolves #84

Have you included tests for your changes?

Did you document any new/modified features?

Notes

@SammyOina SammyOina requested a review from Copilot June 9, 2025 11:43
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refines the CVM creation docs by updating the curl example, removing certificate steps, and adding a note and image for when the CVM comes online. It also tweaks the computation introduction and fixes a typo in the results table.

  • Updated CVM creation instructions and JSON payload fields
  • Removed certificate issuance/download steps, added “online CVM” note and image
  • Corrected typo in computation microservice section and fixed table entry
Comments suppressed due to low confidence (2)

docs/getting-started.md:147

  • [nitpick] The backend field is left empty; consider clarifying that a valid backend provider ID must be supplied here or show how to list available backends.
  "backend": "",

docs/getting-started.md:148

  • [nitpick] It may help to note that agent_log_level defaults to info and is optional, or remove it from this example if it's not required for initial setup.
  "agent_log_level": "info"

@drasko drasko merged commit 6efa5ec into ultravioletrs:main Jul 4, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove references to create backend instead of cvm
3 participants