Skip to content

Conversation

quge009
Copy link
Collaborator

@quge009 quge009 commented Sep 18, 2025

fix bug: missing CLUSTER_ID, which is used to pass the endpoint value into Copilot

@quge009 quge009 requested review from yukirora and Copilot September 18, 2025 01:41
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a missing environment variable bug by adding the CLUSTER_ID environment variable to the Copilot deployment configuration. The CLUSTER_ID is used to pass endpoint values into Copilot functionality.

  • Adds CLUSTER_ID environment variable to copilot-chat deployment template

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@quge009
Copy link
Collaborator Author

quge009 commented Sep 18, 2025

Hi @yukirora, would you please help revie the correctness?
I assume Copilot can read the endpoint value from the CLUSTER_ID env, please double check the correcness of configuration of CLUSTER_ID, thank you

@yukirora
Copy link

Hi @yukirora, would you please help revie the correctness? I assume Copilot can read the endpoint value from the CLUSTER_ID env, please double check the correcness of configuration of CLUSTER_ID, thank you
{{ cluster_cfg["cluster"]["common"]["cluster-id"] }}

@quge009 quge009 merged commit 6dc0954 into release/1.3 Sep 18, 2025
1 check passed
@quge009 quge009 deleted the lequ/copilot-fix-bug-missing-endpoint branch September 18, 2025 05:10
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