-
Notifications
You must be signed in to change notification settings - Fork 30
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
added istio and osm service mesh #41
Conversation
Signed-off-by: kumarabd <abishekkumar92@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kumarabd Are you ok with reducing to smi
?
@@ -21,7 +21,12 @@ var ( | |||
PortSvcB: "9091", | |||
PortSvcC: "9091", | |||
} | |||
istioConfig = &test_gen.Linkerd{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kumarabd do we still support Linkerd? Or was our support incomplete earlier?
@leecalcote yes should work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving. If we switch to just smi
- that's great. If not, we can get it another time.
docker push smi-conformance:${GITHUB_SHA::6} | ||
cd ../smi-conformance && docker build --no-cache -t ${{ secrets.IMAGE_NAME }}:smi . | ||
docker tag ${{ secrets.IMAGE_NAME }}:smi ${{ secrets.IMAGE_NAME }}:smi-${GITHUB_SHA::6} | ||
docker push ${{ secrets.IMAGE_NAME }}:smi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kumarabd given the update to the tag name of smi
, we may have to update adapter logic accordingly.
Signed-off-by: kumarabd abishekkumar92@gmail.com
Description
This PR fixes #40
Notes for Reviewers
Signed commits