Skip to content

Conversation

@0xJMart
Copy link
Member

@0xJMart 0xJMart commented May 14, 2025

Enhanced Documentation and Cluster Creation Feedback

Changes

Documentation Updates

  1. Fixed directory paths in development workflow documentation

    • Updated all references from ./cert-manager to ./charts/cert-manager to match actual project structure
    • Corrected Helm command examples to use proper paths
    • Fixed task name from helm-deploy to helm-install
  2. Added cluster creation examples

    • Added examples showing how to create clusters with custom names
    • Demonstrated both standard and embedded cluster creation with custom names
  3. Added REPLICATED_API_TOKEN prerequisite

    • Added clear instructions for obtaining the token from Vendor Portal
    • Included example of how to export the token
    • Added link to the specific location in Vendor Portal

Task Improvements

  1. Enhanced cluster-create task feedback
    • Added detailed cluster information display when cluster already exists:
      • Cluster ID
      • Status
      • Distribution type
      • Creation date
      • Expiration date
    • Improved status check using jq for better output formatting

Related Issues

  • Fixes unclear feedback when cluster already exists
  • Improves documentation accuracy for directory structure
  • Addresses missing prerequisite for REPLICATED_API_TOKEN

0xJMart added 2 commits May 13, 2025 14:04
…oken setup and correct chart paths for Helm commands
…clusters and output detailed information if found. This improves user feedback during the cluster setup process.
Copy link
Member

@chris-sanders chris-sanders left a comment

Choose a reason for hiding this comment

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

lgtm

@0xJMart 0xJMart marked this pull request as ready for review May 15, 2025 14:41
@0xJMart 0xJMart merged commit 4d24cb6 into main May 15, 2025
1 check passed
@0xJMart 0xJMart deleted the updated-README branch May 15, 2025 18:02
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.

4 participants