Skip to content
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.

Sign up for GitHub

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

Jump to bottom

sztp: use postgresql #108

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

sztp: use postgresql #108

wants to merge 1 commit into from

Conversation

glimchb
Copy link
Member

@glimchb glimchb commented Jul 13, 2022

Fixes #104

Signed-off-by: Boris Glimcher [email protected]

@glimchb glimchb marked this pull request as draft July 13, 2022 01:44
@glimchb glimchb added the bug Something isn't working label Jul 13, 2022
mestery
mestery reviewed Jul 13, 2022
View reviewed changes
Copy link
Contributor

@mestery mestery left a comment

Choose a reason for hiding this comment

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

I'm adding a DB in #105 so should we combine these two so we have only a single DB container?

@glimchb
sztp: use postgresql
0e5463f 
Fixes opiproject#104

Signed-off-by: Boris Glimcher <[email protected]>
@glimchb
Copy link
Member Author

glimchb commented Jul 13, 2022

I'm adding a DB in #105 so should we combine these two so we have only a single DB container?

no, because this provisioning DB is outside xPUs while your API gw is probably inside xPU if I understand it correctly

@glimchb glimchb mentioned this pull request Oct 6, 2022
Open
@glimchb glimchb force-pushed the main branch 2 times, most recently from 3c34007 to 0e5463f Compare February 11, 2023 18:16
@glimchb glimchb force-pushed the main branch 2 times, most recently from d467ce5 to 0e5463f Compare May 14, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mestery mestery mestery left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
bug Something isn't working provisioning
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add postgresql to sztp
2 participants
@glimchb @mestery