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

make sure the title shows server name #27

Merged
merged 2 commits into from
Aug 5, 2023
Merged

Conversation

samos123
Copy link
Contributor

@samos123 samos123 commented Aug 5, 2023

No description provided.

@samos123
Copy link
Contributor Author

samos123 commented Aug 5, 2023

Testing with:

apiVersion: substratus.ai/v1
kind: Server
metadata:
  name: falcon-7b-instruct
spec:
  image: substratusai/model-server-basaran:pr-27
  model:
    name: falcon-7b-instruct
  resources:
    gpu:
      type: nvidia-l4
      count: 1

@samos123
Copy link
Contributor Author

samos123 commented Aug 5, 2023

Verified that it now succesfully shows the pod hostname instead of always showing /content/saved-model and you had no idea what server you were talking to.

@samos123
Copy link
Contributor Author

samos123 commented Aug 5, 2023

Going to merge because of small change

@samos123 samos123 merged commit e69e901 into main Aug 5, 2023
6 checks 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.

1 participant