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

hotfix for malfunctioning example #47

Merged
merged 2 commits into from Jul 1, 2021
Merged

hotfix for malfunctioning example #47

merged 2 commits into from Jul 1, 2021

Conversation

jamsman94
Copy link
Contributor

@jamsman94 jamsman94 commented Jun 30, 2021

Signed-off-by: Min Min minmin@koderover.com

What this PR does / Why we need it:

This is a hotfix for a bugged example which will cause the ingress to lost its backend service.

Problem Summary:
The ingress rule service name does not match the real service name, causing a visiting problem for this example project.

What is changed and how it works?

What's Changed:
changed the service name to match the ingress rules.

How it Works:

More information


This change is Reviewable

Signed-off-by: Min Min <minmin@koderover.com>
@jamsman94 jamsman94 self-assigned this Jun 30, 2021
@jamsman94 jamsman94 added the bug Something isn't working label Jun 30, 2021
labels:
app: nginx3
app: nginx
tier: backend
version: "{{.nginxVersion4}}"
Copy link
Contributor

Choose a reason for hiding this comment

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

x

Signed-off-by: Min Min <minmin@koderover.com>
@27149chen 27149chen merged commit 8296843 into koderover:main Jul 1, 2021
@jamsman94 jamsman94 deleted the hotfix/examples branch September 26, 2021 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants