Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Add limited internal storage and postgresql #3813

Merged
merged 28 commits into from
Nov 21, 2019
Merged

Conversation

hzy46
Copy link
Contributor

@hzy46 hzy46 commented Nov 8, 2019

@hzy46 hzy46 requested review from yqwang-ms, mydmdm, 39Delta, debuggy, sunqinzheng, ydye and abuccts and removed request for 39Delta November 8, 2019 08:36
@fanyangCS
Copy link
Contributor

Can you elaborate the fault tolerance design in the document?

@abuccts abuccts mentioned this pull request Nov 11, 2019
42 tasks
@fanyangCS
Copy link
Contributor

do you consider the service dependency? i.e., I assume postgresql can only be deployed after the internal storage has been set up successfully. do you have some script to check the dependency?

@hzy46 hzy46 requested a review from fanyangCS November 13, 2019 03:30
@hzy46
Copy link
Contributor Author

hzy46 commented Nov 13, 2019

do you consider the service dependency? i.e., I assume postgresql can only be deployed after the internal storage has been set up successfully. do you have some script to check the dependency?

Sure. Please refer to the prerequisite section at https://github.com/microsoft/pai/blob/zhiyuhe/loop_storage_and_db/src/postgresql/deploy/service.yaml#L23 . If internal storage is not ready, the postgresql won't be deployed.

@hzy46 hzy46 requested a review from qfyin November 20, 2019 03:23
@hzy46 hzy46 merged commit 6ca7505 into master Nov 21, 2019
@hzy46 hzy46 deleted the zhiyuhe/loop_storage_and_db branch December 25, 2019 07:17
@hzy46 hzy46 mentioned this pull request Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants