Skip to content

Tsung as k8s pod#53

Open
alekodu wants to merge 13 commits into
EricssonResearch:mainfrom
alekodu:dev
Open

Tsung as k8s pod#53
alekodu wants to merge 13 commits into
EricssonResearch:mainfrom
alekodu:dev

Conversation

@alekodu

@alekodu alekodu commented May 9, 2022

Copy link
Copy Markdown
Member

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

Closes #5

@alekodu
alekodu requested a review from salehsedghpour May 9, 2022 14:37
Comment thread client/tsung.yaml
@@ -0,0 +1,33 @@
apiVersion: apps/v1

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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