Skip to content

add Loadgen#90

Draft
stevemcghee wants to merge 3 commits intomainfrom
loadgen
Draft

add Loadgen#90
stevemcghee wants to merge 3 commits intomainfrom
loadgen

Conversation

@stevemcghee
Copy link
Collaborator

to fix #89

also this is a good chance to test out how to add a service from an existing repo to fit into this structure.

@stevemcghee stevemcghee self-assigned this Jun 21, 2024
@stevemcghee
Copy link
Collaborator Author

looking for your suggestions when you have time @MKand -- is this the right track?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The k8s folder needs to be structured to work with skaffold and kustomize as the deployments are different for workload clusters (where the archetype based workload is deployed) and other clusters (where just the virutal service and namespace) definitions are applied. If you look at the structure of any other k8s folder such as frontend/app-repo/k8s, it'll become apparent

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure when the build.yaml is moved from frontend to loadgen. Did you mean to copy-paste?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure when the deploy-app.yaml is moved from frontend to loadgen. Did you mean to copy-paste?

Copy link
Collaborator

Choose a reason for hiding this comment

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

should this be copy-paste instead of move?

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.

re-add loadgenerator service

2 participants