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

minor fixes and update images #165

Merged
merged 4 commits into from
Apr 2, 2024
Merged

minor fixes and update images #165

merged 4 commits into from
Apr 2, 2024

Conversation

rcarrata
Copy link

minor fixes and update images of RHOAI 2.8

@rcarrata
Copy link
Author

@erwangranger @guimou this PR solves some of the issues in #163

Can we double check that the Deploy of the Model in lab4 is using the Multi-Model serving platform?

@erwangranger
Copy link
Collaborator

Currently, it does. We can look at using Single-Model serving instead in the example, but it feels less urgent than other changes. Multi-Model is fine for now IMO, but we can discuss.

@rcarrata
Copy link
Author

rcarrata commented Apr 1, 2024

due the updates of the Multi-Model serving platform "only" applies to the RHOAI 2.8 version, should be wait until we moved the lab to this version and only fix the minor typos, not updating the images for RHOAI 2.8 in the lab? what do you think @erwangranger @guimou ?

@guimou
Copy link
Collaborator

guimou commented Apr 2, 2024

Merging as the Lab now runs on RHOAI 2.8

@guimou guimou merged commit 4267ff5 into dev Apr 2, 2024
1 check failed
@guimou guimou deleted the bugfix/minor-typos branch April 2, 2024 19:28
ritzshah referenced this pull request in redhat-gpte-devopsautomation/rad-model-deployment Aug 14, 2024
…minor-typos

minor fixes and update images
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.

3 participants