Skip to content
This repository has been archived by the owner on Dec 1, 2024. It is now read-only.

Update test plans to use secured connectivity with the operator #14

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

Conversation

gabriel-farache
Copy link

Update test plans to use secured connectivity with the operator + rename edgedeployment in edgeworkload

payload.json Outdated Show resolved Hide resolved
payload.json Outdated Show resolved Hide resolved
scripts/jmeter.log Outdated Show resolved Hide resolved
scripts/generate_certs.sh Outdated Show resolved Hide resolved
test_plans/flotta_test_plan.jmx Outdated Show resolved Hide resolved
scripts/run_test_plan.sh Outdated Show resolved Hide resolved
name: flotta-svc
namespace: flotta
spec:
type: NodePort
Copy link
Collaborator

Choose a reason for hiding this comment

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

installing MetalLB + defining address pool will provide an IP address from the local subnet of the node using service type LoadBalancer.
Then that IP can be accessed by port-forwarding.
see

Copy link
Author

Choose a reason for hiding this comment

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

I'll take a look to those links and update the README, in the meantime, I'll remove this yaml file

Copy link
Author

Choose a reason for hiding this comment

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

Now that I understand better MetalLB, I fail to see why this is better than creating a NodePort. I mean, in this context of testing, I do not think that it is better to install MetalLB, create an AddressPool and a LoadBalancer servcice and then run port-forward, it's a lot of steps to have the same output IMO.
In fact, a simple port-forward added in the run script will be better as operator should be accessible on 127.0.0.1 or domain name project-flotta.io ==> by using port-forwarding, we can use 127.0.0.1 as address so we won't have issue with the certificates CN/alternateNames checks

scripts/run_test_plan.sh Outdated Show resolved Hide resolved
scripts/run_test_plan.sh Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants