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.

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

Partner Experience: update platform server prerequisites (#143) #170

Merged
merged 4 commits into from
Aug 14, 2023

Conversation

urvisavla
Copy link
Contributor

@urvisavla urvisavla commented Jul 5, 2023

What

Update platform server prerequisites.

Hardware requirements largely remain the same as the current specifications of real-time private instance, except for updated disk space to accommodate on-disk captive-core mode and reduced history retention. These specifications reflect minimums due to a lack of benchmarks for API instances.

Why

#143

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

Something went wrong with PR preview build please check

@stellar-jenkins
Copy link

5 similar comments
@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@urvisavla urvisavla force-pushed the 143/horizon-prerequisites branch from 602e66b to 5e1f0d5 Compare July 10, 2023 23:05
@stellar-jenkins
Copy link

@stellar stellar deleted a comment from gre3n3yes34 Jul 11, 2023

In this scenario, the hardware specifications are more demanding than what is necessary for the day-to-day operation of real-time ingestion, but catch-up only needs to occur once.
**_PostgreSQL database server compute instance:_**
Copy link
Contributor

Choose a reason for hiding this comment

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

this is currently a placeholder, we will state min compute requirements for postgres here correct? This seems like good place to also stipulate the postgres server version requirements, such as from team discussion. Just for consideration, maybe we relocate the low level postgres server configs mentioned in Configuring, inline here instead, to keep all postgres/vendor specifics about the db here in one place, then rest of docs just refer to postgres more generically as the database.

CC'ing @tamirms for postgres context

Copy link
Contributor

Choose a reason for hiding this comment

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

We run postgres v 12.13 in prod

@stellar-jenkins
Copy link

@urvisavla urvisavla marked this pull request as ready for review August 9, 2023 21:53
| **Database** | **CPU**: 16 cores (min: 8)<br/>**RAM**: 64 GB (min: 32GB)<br/>**Storage**: 2 TB<br/>**IOPS**: 20K (min: 15K) | **CPU**: 16 (12)<br/>**RAM**: 128 GB (64)<br/>**Storage**: 4 TB<br/>**IOPS**: 20K (15K) | **CPU**: 64 (32)<br/>**RAM**: 512 GB (256)<br/>**Storage**: 10 TB<br/>**IOPS**: 20k (15k) |
| **Storage**<br/>(all same) | | **SSD** (NVMe, Direct Attached Storage preferred) | |
| **AWS**<br/>(reference) | **Captive Core**: `m5.2xlarge`<br/>**Database**: `r5.2xlarge` | **Captive Core**: `m5.4xlarge`<br/>**DB**: `r5.4xlarge` | **Captive Core**: `c5.2xlarge` (x2)<br/>**DB**: `r5.16xlarge` (ro)<br/>`r5.8xlarge` (rw) |
To achieve high availability, redundancy, and high throughput, explore the [scaling](./scaling.mdx) strategy. It provides detailed prerequisites and guidelines to determine the appropriate [number of Horizon instances](./configuring.mdx#multiple-instance-deployment) to deploy.
Copy link
Contributor

Choose a reason for hiding this comment

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

hmm, do we need to update scaling.mdx, it hasn't been touched as of the rest of partner experience tickets, maybe it falls under the purview of prerequisites?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I'll take a look at the scaling doc.

Copy link
Contributor

@sreuland sreuland left a comment

Choose a reason for hiding this comment

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

looks good, left a couple suggestions for review.

Update docs/run-platform-server/prerequisites.mdx

Co-authored-by: shawn <[email protected]>

Update docs/run-platform-server/prerequisites.mdx

Co-authored-by: shawn <[email protected]>

Update docs/run-platform-server/prerequisites.mdx

Co-authored-by: shawn <[email protected]>

Update docs/run-platform-server/prerequisites.mdx

Co-authored-by: shawn <[email protected]>

Update docs/run-platform-server/prerequisites.mdx

Co-authored-by: shawn <[email protected]>

Update docs/run-platform-server/prerequisites.mdx

Co-authored-by: shawn <[email protected]>

Addressing review comments
@urvisavla urvisavla force-pushed the 143/horizon-prerequisites branch from d3eb316 to f8f7f73 Compare August 10, 2023 21:57
@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@urvisavla urvisavla merged commit c665fd1 into partner_experience Aug 14, 2023
@urvisavla urvisavla deleted the 143/horizon-prerequisites branch August 14, 2023 16:36
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.

4 participants