Skip to content

Canary Deployment feature of Istio #116#117

Open
shubhangis91 wants to merge 4 commits intomasterfrom
feature/canary-deployment-istio
Open

Canary Deployment feature of Istio #116#117
shubhangis91 wants to merge 4 commits intomasterfrom
feature/canary-deployment-istio

Conversation

@shubhangis91
Copy link
Copy Markdown
Contributor

Implemented and tested canary deployment facility provided by Istio on the UI service

Comment on lines +17 to +29
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
name: api-gateway
spec:
hosts:
- api-gateway
gateways:
- captivate-gateway
http:
- route:
- destination:
host: api-gateway No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what is the purpose of this ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

can be removed if we don't want to define traffic rules for this service

Comment on lines +20 to +48
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: session-management
spec:
  hosts:
  - session-management
  gateways:
  - captivate-gateway
  http:
  - route:
    - destination:
        host: session-management

---

apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: user-management
spec:
  hosts:
  - user-management
  gateways:
  - captivate-gateway
  http:
  - route:
    - destination:
        host: user-management No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

May I know why we need these ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

can be removed if we don't want to define traffic rules for these services

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.

2 participants