-
Notifications
You must be signed in to change notification settings - Fork 33
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
Reworking of App-Mesh adapter #41
Conversation
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
thanks for the review @DelusionalOptimist
will get the other operation working soon as well @sayantan1413 can we get that workflows up before merging this PR |
Wow! That screenshot is exciting! |
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
when i run the smi conformance operation i get this error
|
@Shreyas220 you need a load-balancer to expose SMI conformance tool. There should be a guide somewhere to do so based on your k8s cluster. |
@DelusionalOptimist @Utkarsh-pro looks like we are able to do all the operation but the the app-mes-controller image is |
Some good work going on by @Shreyas220. : ) Action items now:
|
Signed-off-by: Shreyas220 <shreyas.ny@gmail.com>
Description
Made some changes according to the design spec for adapter
This PR fixes #40
Notes for Reviewers
Signed commits