Skip to content

Sds api: implement SDS api#3689

Closed
JimmyCYJ wants to merge 18 commits intoenvoyproxy:masterfrom
JimmyCYJ:sds_api
Closed

Sds api: implement SDS api#3689
JimmyCYJ wants to merge 18 commits intoenvoyproxy:masterfrom
JimmyCYJ:sds_api

Conversation

@JimmyCYJ
Copy link
Member

@JimmyCYJ JimmyCYJ commented Jun 21, 2018

Implement SDS api which fetches dynamic secrets.

Description:
Implement SDS api that fetches secrets from remote SDS server. Secrets are stored in Secret Manager.
Updating secrets at listener and cluster will follow.

Risk Level: Low

Fixes #1194

Signed-off-by: Jimmy Chen jimmychen.0102@gmail.com

mangchiandjjoe and others added 16 commits June 21, 2018 10:45
…urces

Signed-off-by: Jae Kim <jaebong.kim@gmail.com>
Signed-off-by: Jae Kim <jaebong.kim@gmail.com>
Signed-off-by: Jae Kim <jaebong.kim@gmail.com>
Signed-off-by: Jae Kim <jaebong.kim@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
Signed-off-by: JimmyCYJ <jimmychen.0102@gmail.com>
@mattklein123
Copy link
Member

@PiotrSikora @lizan please take first pass.

@lizan
Copy link
Member

lizan commented Jun 22, 2018

@JimmyCYJ can you create a PR from istio repo, here? I have made some changes to merge recent changes.

@JimmyCYJ
Copy link
Member Author

@lizan Sure. Will close this one and open another PR. Thanks!

@JimmyCYJ JimmyCYJ closed this Jun 22, 2018
@JimmyCYJ JimmyCYJ deleted the sds_api branch June 22, 2018 16:30
@JimmyCYJ
Copy link
Member Author

@lizan @PiotrSikora Please review #3700, thanks!

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.

5 participants