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

Improve Digital Ocean guide #159

Merged
merged 2 commits into from
Mar 25, 2023
Merged

Conversation

adrianeriksen
Copy link
Contributor

I ran trough the Digital Ocean guide and faced some problems. I got the cluster up and running after som troubleshooting, and I want to share how I did it.

The changes introduced with this PR are:

  1. Updated selectors so the load balancer can reach the juice-balancer pod.
  2. Added a HTTP listener to enable HTTP to HTTPS redirect.
  3. Added some instructions on how to troubleshoot when the juice-balancer pod hangs on pending.

Problem 1: Provisioning the cluster

When installing multi-juicer on the cluster the juice-blancer pod is stuck on pending when I run kubectl get pods. I got the following message when I described the pod:

Warning  FailedScheduling   7m18s                default-scheduler   0/3 nodes are available: 3 Insufficient cpu. preemption: 0/3 nodes are available: 3 No preemption victims found for incoming pod..

The default node size is s-1vcpu-2g. I worked when I upgraded to s-2vcpu-2g. The downside is a more expensive cluster, approximately $66 per month.

Problem 2:

When provisioning the load balancer in step 4, I got this warning.

Warning  SyncLoadBalancerFailed  4m4s                service-controller  Error syncing load balancer: failed to ensure load balancer: load-balancer is not yet active (current status: new)

The provisioned load balancer can't reach juice-balancer. I get the following labels when I describe the juice-balancer pod.

Labels:       app.kubernetes.io/component=load-balancer
              app.kubernetes.io/instance=juice-balancer-multi-juicer
              app.kubernetes.io/managed-by=Helm
              app.kubernetes.io/name=juice-balancer
              app.kubernetes.io/part-of=multi-juicer
              app.kubernetes.io/version=5.3.0
              pod-template-hash=b756f7b9c

When I updated the selector, the load balancer provisioned as expected.

@J12934 J12934 added the documentation Improvements or additions to documentation label Mar 25, 2023
Copy link
Member

@J12934 J12934 left a comment

Choose a reason for hiding this comment

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

Ahhh yes 👍
I've recently changed the labels to be more consistent, forgot to update the guide(s).
Thank you for the fix and the feedback 🙌

@J12934 J12934 merged commit 06fcd68 into juice-shop:main Mar 25, 2023
@adrianeriksen adrianeriksen deleted the improve-do-guide branch March 25, 2023 16:40
J12934 added a commit that referenced this pull request Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants