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

Tsung as k8s pod #53

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

Tsung as k8s pod #53

wants to merge 13 commits into from

Conversation

alekodu
Copy link
Member

@alekodu alekodu commented May 9, 2022

This PR generates dockerfile and k8s deployment yaml files for Tsung.

Closes #5

@@ -0,0 +1,33 @@
apiVersion: apps/v1
Copy link
Collaborator

Choose a reason for hiding this comment

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

How are we going to pass tsung config file to the running container? Are we going to pass it when we are building the container or when we are deploying container?

Copy link
Member Author

Choose a reason for hiding this comment

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

It should be passed when deploying the container as a mounted volume

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, but using this yaml file, we are not passing anything when deploying.

Copy link
Member Author

Choose a reason for hiding this comment

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

The user doesn't pass the file as a parameter while deploying Tsung. Instead, the path to the conf.xml file should be updated under volumes:hostPath:path.

Copy link
Collaborator

Choose a reason for hiding this comment

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

If the end user wants to pass different traffic scenario:
1- first discover where the load generator is going to be deployed (specified by k8s scheduler)
2- Then ssh to the node
3- create/update the tsung file under the path!

It makes the whole process of experiments complicated, we could easily pass it as configmap or environment variable.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, as discussed this will be modified accordingly with either configmap or env variables.

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.

Support interactions of frontend/backend microservices -> client
2 participants