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

feat(loadgenerator): allow customizing frontend protocol #2775

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aslafy-z
Copy link
Contributor

@aslafy-z aslafy-z commented Oct 18, 2024

Background

I would like to point locust to my nginx ingress instead of the service.

Change Summary

This PR adds support for customizing the protocol locust uses while querying the online-boutique website.

Additional Notes

Backwards compatible.

Testing Procedure

  • Deploy the app with:
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: online-boutique
resources:
- https://github.com/GoogleCloudPlatform/microservices-demo//kustomize/base?ref=6af73dcf7fe1ca653d95ae7a29e12faf8cf02964
- ingress.yaml
components:
- https://github.com/GoogleCloudPlatform/microservices-demo//kustomize/components/network-policies?ref=6af73dcf7fe1ca653d95ae7a29e12faf8cf02964
- https://github.com/GoogleCloudPlatform/microservices-demo//kustomize/components/non-public-frontend?ref=6af73dcf7fe1ca653d95ae7a29e12faf8cf02964
patches:
- patch: |
  apiVersion: apps/v1
  kind: Deployment
  metadata:
    name: loadgenerator
  spec:
    template:
      spec:
        containers:
        - name: main
          image: ghcr.io/aslafy-z/microservices-demo/loadgenerator:483c6f731bb7c408ff5be379683b0ace8ea5833c
          env:
          - name: FRONTEND_PROTO
            value: https
          - name: FRONTEND_ADDR
            value: shop.example.com
  • Ensure the load testing is running correctly looking at the logs

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.

1 participant