Conversation
|
/staging |
|
Your PR has been deployed to staging. Staging URL: https://static.staging.kernelci.org/ After testing, you can merge your PR. Note: This is a temporary staging URL, if anyone else will test another PR, it will override contents. |
patersonc
left a comment
patersonc
left a comment
There was a problem hiding this comment.
Hello @nuclearcat
Thank you for the detailed documentation.
I've briefly read through it all and it looks okay to me, although I obviously lack the knowledge to confirm it's functionally correct.
| ┌─────────────────────────────────────────────────────────────┐ | ||
| │ Network: private │ | ||
| │ │ | ||
| │ ┌──────────────┐ ┌──────────────┐ ┌──────────────┐ │ | ||
| │ │ kcidb-rest │ │ ingester │ │logspec-worker│ │ | ||
| │ │ (Port 443) │ │ │ │ │ │ | ||
| │ └──────┬───────┘ └──────┬───────┘ └──────┬───────┘ │ | ||
| │ │ │ │ │ | ||
| │ └───────────────────┴───────────────────┘ │ | ||
| │ │ │ | ||
| │ ┌──────▼───────┐ │ | ||
| │ │ PostgreSQL │ │ | ||
| │ │ (Port 5432)│ │ | ||
| │ └──────────────┘ │ | ||
| │ │ | ||
| │ ┌──────────────┐ ┌──────────────┐ ┌──────────────┐ │ | ||
| │ │ Backend │◄───┤ Redis │ │ Dashboard DB │ │ | ||
| │ │ (Port 8000) │ │ (Port 6379) │ │ (Port 5434) │ │ | ||
| │ └──────┬───────┘ └──────────────┘ └──────────────┘ │ | ||
| │ │ │ | ||
| └─────────┼────────────────────────────────────────────────────┘ | ||
| │ | ||
| ┌─────────▼────────────────────────────────────────────────────┐ | ||
| │ Network: public │ | ||
| │ │ | ||
| │ ┌──────────────┐ ┌──────────────┐ │ | ||
| │ │ Proxy │ │ Dashboard │ │ | ||
| │ │ (Port 80) │◄───┤ (static) │ │ | ||
| │ └──────────────┘ └──────────────┘ │ | ||
| │ │ │ | ||
| └─────────┼────────────────────────────────────────────────────┘ |
There was a problem hiding this comment.
Nit: The box alignment isn't perfect in this diagram.
|
One general comment: For the various software prerequisites, do we have any versioning requirements? |
I think mostly not, at least until now. We use containers a lot, and they maintain this details "inside". |
Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
nuclearcat
force-pushed
the
add-self-install
branch
from
0288fb5 to
0c5e1c4
Compare
patersonc
left a comment
patersonc
left a comment
There was a problem hiding this comment.
I'm happy to approve, but I'll leave it another day or so before merging to give others who may know more than me a chance to look over it.
3 participants
This documentation might help DevOps who want to setup whole KernelCI "in-house".