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.

Sign up for GitHub

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

Jump to bottom

integrate istio #129

Closed
wants to merge 4 commits into from
Closed

integrate istio #129

wants to merge 4 commits into from

Conversation

mirr254
Copy link
Contributor

@mirr254 mirr254 commented Jun 25, 2020

What does this PR

  • Adds Istio service mesh to Gluu server cloud native deployment.

Description of work done.

  • Rename all the services' manifests to conform with istio service naming convention.
  • Creates Istio gateway to give access Gluu server to the outside world.
  • Create VirtualServices for service objects that instructs the gateway where to send the traffic of certain requests.

Questions

  • Currently having trouble with routing rules in VirtualServices. Everything else works fine but getting a blank screen with 503 error. Which means there is a route rule in that isn't redirecting traffic to the requested pod.

@moabu moabu self-requested a review June 25, 2020 12:23
@moabu moabu self-assigned this Jun 25, 2020
@moabu moabu added the enhancement New feature or request label Jun 25, 2020
@moabu moabu linked an issue Jun 25, 2020 that may be closed by this pull request
Istio integration #3
Closed
@moabu
Copy link
Member

moabu commented Aug 5, 2020

@mirr254 this PR is not functional and outdated. Please review this PR and fix. Look at your routes and also there is additional unneeded files committed.

@moabu moabu closed this Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@moabu moabu Awaiting requested review from moabu

Assignees

@moabu moabu

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Istio integration
2 participants
@mirr254 @moabu