Skip to content

chore: move quote service to port 8081 - #464

Open
mayankpande88 wants to merge 1 commit into
mainfrom
demo/quote-port-change
Open

chore: move quote service to port 8081#464
mayankpande88 wants to merge 1 commit into
mainfrom
demo/quote-port-change

Conversation

@mayankpande88

Copy link
Copy Markdown
Contributor

Aligning quote's listen port with the new internal port convention.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the values.yaml file to set the QUOTE_PORT environment variable to 8081 for the quote component. The reviewer correctly identified that changing this port without updating the corresponding Kubernetes Service and Pod port configurations will cause a port mismatch and lead to connection failures. A suggestion was provided to update the ports configuration accordingly.

Comment thread deploy/kubernetes/sample-app/values.yaml
@nudgebee nudgebee deleted a comment from nudgebee-dev Bot Aug 7, 2026
@nudgebee-dev

nudgebee-dev Bot commented Aug 7, 2026

Copy link
Copy Markdown

Nudgebee Change Gate

Review

  • quote: Changing the 'QUOTE_PORT' environment variable to '8081' matches a previous failure pattern. In incident 2026-08-06, modifying 'QUOTE_PORT' to '8081' caused connectivity failures because the Kubernetes Service 'targetPort' remained at '8080', leading to HTTP 500 errors in the dependent 'shipping' service. Please verify that the Service 'targetPort' is also updated to match.

Based on Nudgebee rightsizing recommendations and incident investigations from the last 30 days.

View the gate agent investigation

@github-actions

Copy link
Copy Markdown

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants