Skip to content

Conversation

@ctao456
Copy link
Contributor

@ctao456 ctao456 commented Sep 14, 2024

Tested all command scripts on Gaudi instance, and verified that all services are working, including both VLLM and TGI services.
Docker container printouts and yaml configs are up-to-date.

Signed-off-by: Chun Tao <chun.tao@intel.com>
Copy link
Collaborator

@tomlenth tomlenth left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@dbkinder dbkinder left a comment

Choose a reason for hiding this comment

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

Fix guardrail misspelling and some heading issues.

Signed-off-by: Chun Tao <chun.tao@intel.com>
Copy link
Contributor

@dbkinder dbkinder left a comment

Choose a reason for hiding this comment

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

Removing the indent lost the code-block formatting. Put the four-space indent back.

Copy link
Contributor

@dbkinder dbkinder left a comment

Choose a reason for hiding this comment

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

One more heading fix

Signed-off-by: Chun Tao <chun.tao@intel.com>
Copy link
Contributor

@dbkinder dbkinder left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@dbkinder dbkinder merged commit 99592d1 into opea-project:main Sep 16, 2024
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