diff options
author | Mek Stittri <mstittri@gitlab.com> | 2019-02-13 20:52:40 +0000 |
---|---|---|
committer | Mek Stittri <mstittri@gitlab.com> | 2019-02-13 20:52:40 +0000 |
commit | 66b20a66f4b25a69ff3f59da855ac2cb38c8bca9 (patch) | |
tree | aebb64e75c513607b8f203104f6e8f8398e006e8 | |
parent | 17b1033850e53c1323e55cdc86e13b3d40f66897 (diff) | |
parent | ef33aee2604b9e162ca84575387324717cd5163c (diff) | |
download | gitlab-ce-66b20a66f4b25a69ff3f59da855ac2cb38c8bca9.tar.gz |
Merge branch 'improve-review-apps-docs' into 'master'
[Docs] Improve the Review Apps documentation
See merge request gitlab-org/gitlab-ce!25161
-rw-r--r-- | doc/development/testing_guide/review_apps.md | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/doc/development/testing_guide/review_apps.md b/doc/development/testing_guide/review_apps.md index 3af97717775..85b47f88cc4 100644 --- a/doc/development/testing_guide/review_apps.md +++ b/doc/development/testing_guide/review_apps.md @@ -143,17 +143,21 @@ thousands of unused Docker images.** **How big are the Kubernetes clusters (`review-apps-ce` and `review-apps-ee`)?** > The clusters are currently set up with a single pool of preemptible nodes, - with a minimum of 1 node and a maximum of 100 nodes. + with a minimum of 1 node and a maximum of 50 nodes. **What are the machine running on the cluster?** - > We're currently using `n1-standard-4` (4 vCPUs, 15 GB memory) machines. + > We're currently using `n1-standard-16` (16 vCPUs, 60 GB memory) machines. **How do we secure this from abuse? Apps are open to the world so we need to find a way to limit it to only us.** > This isn't enabled for forks. +## Other resources + +* [Review Apps integration for CE/EE (presentation)](https://docs.google.com/presentation/d/1QPLr6FO4LduROU8pQIPkX1yfGvD13GEJIBOenqoKxR8/edit?usp=sharing) + [charts-1068]: https://gitlab.com/charts/gitlab/issues/1068 [gitlab-pipeline]: https://gitlab.com/gitlab-org/gitlab-ce/pipelines/44362587 [gitlab:assets:compile]: https://gitlab.com/gitlab-org/gitlab-ce/-/jobs/149511610 |