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

feature/add manifests for k8s installation #143

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

yrosaguiar
Copy link

@yrosaguiar yrosaguiar commented Jul 28, 2023

Description of changes:

  • Adding manifests for k8s installation support
  • Adding deployment, service, namespace
  • Adding directory structure for k8s and kustomize
  • Adding README for k8s setup

Kubernetes installation

Setup graph-explorer on kubernetes for tests using kustomize

  • Resources that will be created:

  • Namespace graph-explorer

  • Deployment (1 pod replica)

  • Service

kubectl apply -k k8s/manifests/overlays/default

Using port-forward to access the graph-explorer

kubectl port-forward svc/graph-explorer 8080:443

Removing all resources:

kubectl delete -k k8s/manifests/overlays/default

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rochaalexandre
Copy link

I have being using it inside the project I'm working and this changed helped a lot making easier to visualize the graph records!

Thank @yrosaguiar

@kmcginnes kmcginnes self-requested a review April 30, 2024 21:15
@kmcginnes
Copy link
Collaborator

@yrosaguiar Thank you for this great contribution.

We have decided to defer inclusion of this change. We will likely reconsider at a later date.

Anyone who has a need to use Kubernetes with Graph Explorer can follow your wonderful example in this PR.

@kmcginnes kmcginnes removed their request for review May 8, 2024 14:27
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.

3 participants