Skip to content

feat: use laptop hostname in instance names and display instance info #1

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 1 commit into from
Jun 13, 2025

Conversation

yossiovadia
Copy link
Contributor

Summary

Improves instance naming and deployment feedback by using the operator's laptop hostname instead of a generic "GPU-Instance" prefix and displaying comprehensive instance information upon deployment completion.

Changes Made

  • Instance Naming: Replace GPU-Instance with ansible_hostname in instance naming to make instances more identifiable by operator
  • Deployment Summary: Add comprehensive instance information display at deployment completion including:
    • Instance ID and Name
    • Instance Type and IPs (public/private)
    • Ready-to-use SSH command
    • Details file location

Benefits

  • Better instance identification and management when multiple operators are using the provisioner
  • Immediate access to connection information without needing to check separate files
  • Improved user experience with clear deployment completion feedback

Testing

  • ✅ Verified instance naming works with laptop hostname
  • ✅ Confirmed deployment summary displays all relevant information
  • ✅ SSH command generation works correctly

@rootfs rootfs merged commit 4af4f34 into redhat-et:main Jun 13, 2025
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.

2 participants