From b312ba0c75740694e4cb76ef99e1a27dcc327d42 Mon Sep 17 00:00:00 2001
From: Pablo Carranza <pablo@gitlab.com>
Date: Tue, 20 Jun 2017 14:06:03 +0200
Subject: [PATCH 1/6] Add themes (embedded in) to production engineers

---
 data/team.yml                         | 12 ++++++++++++
 source/includes/team/member.html.haml |  3 +++
 source/team.json.haml                 |  1 +
 3 files changed, 16 insertions(+)

diff --git a/data/team.yml b/data/team.yml
index ea90c2fc65b..64213fced8f 100644
--- a/data/team.yml
+++ b/data/team.yml
@@ -632,6 +632,9 @@
   picture: pablo.jpg
   twitter: psczg
   gitlab: pcarranza
+  projects:
+    infrastructure: maintainer
+    gitlab-cog: maintainer
   expertise:  |
               <li>On-call hero!</li>
   story:  |
@@ -2004,8 +2007,12 @@
   picture: daniele.jpg
   twitter: escociao
   gitlab: omame
+  embedded_in: Cloud Native Build
   expertise: |
+             <li>On Call Hero!</li>
              <li>Deployment Expert</li>
+             <li>Terraform Expert</li>
+             <li>Cloud Native Expert</li>
   story:  |
           Daniele fell in love with computers when he started to use a Vic 20 very early in his life and never looked back.
           In his teenage years he replaced lego with pc components and suddenly his room became a messy computer lab.
@@ -2306,6 +2313,7 @@
   reports_to: Production Lead
   picture: jtevnan.jpg
   gitlab: jtevnan
+  embedded_in: Database
   story:  |
           Jason made his way into IT via the detour of studying metallurgy. After discovering that he understood database query plans much better than the periodic table, he switched majors and never looked back. When not attempting to automate everything, he is an avid cook, tries to be outdoors as much as he can and attempts to brew the perfect beer.
 
@@ -2342,6 +2350,7 @@
   picture: ilya.png
   twitter:
   gitlab: ilyaf
+  embedded_in: Security
   story:  |
           Ilya is a self-taught UNIX System Administrator with strong interest in computer security. In those rare occasions when he's not
           configuring something, he enjoys a company of a good book.
@@ -2479,6 +2488,9 @@
   reports_to: Production Lead
   picture: victorlopez.jpg
   gitlab: victorcete
+  embedded_in: Search
+  expertise: |
+            <li>Elastic Search Expert</li>
   story:  |
           Victor is a technology enthusiast and a very passionate systems administrator.
           He enjoys all around automation, monitoring and development of tools that help everyone
diff --git a/source/includes/team/member.html.haml b/source/includes/team/member.html.haml
index f2a40fe11e7..d811caf4c18 100644
--- a/source/includes/team/member.html.haml
+++ b/source/includes/team/member.html.haml
@@ -41,6 +41,9 @@
         - if member.expertise
           %ul
             = member.expertise
+        - if member.embedded_in
+            Embedded in
+            = member.embedded_in
 
       .social
         - if member.twitter
diff --git a/source/team.json.haml b/source/team.json.haml
index e4ecef10fe6..2582333b710 100644
--- a/source/team.json.haml
+++ b/source/team.json.haml
@@ -1,4 +1,5 @@
 - team_array = []
+<li>Cloud Native
 - team = data.team.reject { |t| t.picture == 'logo-extra-whitespace.png' }
 - team.each do |member|
   - team_array << { picture: image_path("team/#{member.picture}") }
-- 
GitLab


From 2d741f49909523bb5e8957b20c098bf3262e9cbe Mon Sep 17 00:00:00 2001
From: Pablo Carranza <pablo@gitlab.com>
Date: Wed, 21 Jun 2017 13:40:52 +0200
Subject: [PATCH 2/6] Changed from embedded_in to embedded_with

---
 data/team.yml                         | 14 +++++++++-----
 source/includes/team/member.html.haml |  9 ++++++---
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/data/team.yml b/data/team.yml
index 64213fced8f..3bc95191426 100644
--- a/data/team.yml
+++ b/data/team.yml
@@ -2007,7 +2007,8 @@
   picture: daniele.jpg
   twitter: escociao
   gitlab: omame
-  embedded_in: Cloud Native Build
+  embedded_with: |
+                 <li>Cloud Native Build</li>
   expertise: |
              <li>On Call Hero!</li>
              <li>Deployment Expert</li>
@@ -2313,7 +2314,8 @@
   reports_to: Production Lead
   picture: jtevnan.jpg
   gitlab: jtevnan
-  embedded_in: Database
+  embedded_with: |
+                 <li>Database</li>
   story:  |
           Jason made his way into IT via the detour of studying metallurgy. After discovering that he understood database query plans much better than the periodic table, he switched majors and never looked back. When not attempting to automate everything, he is an avid cook, tries to be outdoors as much as he can and attempts to brew the perfect beer.
 
@@ -2350,7 +2352,8 @@
   picture: ilya.png
   twitter:
   gitlab: ilyaf
-  embedded_in: Security
+  embedded_with: |
+                 <li>Security</li>
   story:  |
           Ilya is a self-taught UNIX System Administrator with strong interest in computer security. In those rare occasions when he's not
           configuring something, he enjoys a company of a good book.
@@ -2488,9 +2491,10 @@
   reports_to: Production Lead
   picture: victorlopez.jpg
   gitlab: victorcete
-  embedded_in: Search
+  embedded_with: |
+                 <li>Search</li>
   expertise: |
-            <li>Elastic Search Expert</li>
+             <li>Elastic Search Expert</li>
   story:  |
           Victor is a technology enthusiast and a very passionate systems administrator.
           He enjoys all around automation, monitoring and development of tools that help everyone
diff --git a/source/includes/team/member.html.haml b/source/includes/team/member.html.haml
index d811caf4c18..5cdaecefe29 100644
--- a/source/includes/team/member.html.haml
+++ b/source/includes/team/member.html.haml
@@ -41,9 +41,12 @@
         - if member.expertise
           %ul
             = member.expertise
-        - if member.embedded_in
-            Embedded in
-            = member.embedded_in
+
+      .embedded_with
+        - if member.embedded_with
+          Embedded With
+          %ul
+            = member.embedded_with
 
       .social
         - if member.twitter
-- 
GitLab


From 8427c29f8590e20d5c98fcb05af35528c3a97f80 Mon Sep 17 00:00:00 2001
From: Pablo Carranza <pablo@gitlab.com>
Date: Wed, 21 Jun 2017 15:07:38 +0200
Subject: [PATCH 3/6] Add node of what embedded with means

---
 source/handbook/infrastructure/index.html.md | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/source/handbook/infrastructure/index.html.md b/source/handbook/infrastructure/index.html.md
index 7af4efda640..607c77d0cab 100644
--- a/source/handbook/infrastructure/index.html.md
+++ b/source/handbook/infrastructure/index.html.md
@@ -46,7 +46,7 @@ infrastructure team works on
      is in fact the first issue in the gitlab-ce issue tracker; for more on
      pingdom see the [monitoring page](/handbook/infrastructure/monitoring/)),
      measured per calendar month, and as recorded on
-      [pingdom](http://stats.pingdom.com/81vpf8jyr1h9/1902794/history). 
+      [pingdom](http://stats.pingdom.com/81vpf8jyr1h9/1902794/history).
 1. GitLab.com's performance.
    - Current goal: [99% of user requests < 1 second](https://performance.gitlab.net/dashboard/db/transaction-overview?panelId=2&fullscreen&orgId=1)
    - Latency here is _currently_ measured via the "Transaction Timings"
@@ -80,6 +80,22 @@ work on the topics listed above, using the principles and methods of [Site
 Reliability Engineering](https://landing.google.com/sre/book.html) a little bit
 more each day.
 
+#### Production Engineers Embedded With
+
+Additionally, specific production engineers are work embedded with different
+teams.
+
+Production engineers that are _embedded with_ will:
+
+- Participate in team calls for the team that are embedded with.
+- Keep up to date with the general Production Engineering duties: issues, fires, on-call. In fact, those will still be your priority unless the Lead explicitly says otherwise.
+- be embedded with one or more teams at a time, it depends on the level of involvement that is needed.
+- report to the Production Lead and participate in the Production team meetings as part of the duties, bringing the context from the team embedded with.
+- Help the team embedded with spending time to make their feature set or service "production ready".
+
+Since in our case these things are mostly already in production, then it means that you work on making sure that post factum the feature set or service meets the requirements written in the production readiness review questionnaire.
+
+Including improving the runbooks and documentation, alerting, monitoring, coding for resiliency, etc. Being so good that any other member of the Production Team can tend to the feature set or service as well and the "embedment" stops. At that point we name you an expert in the respective service.
 
 ## Production and Staging Access
 
-- 
GitLab


From cd43064a6a5cc319d51d7f90eb2c123b135cbe6d Mon Sep 17 00:00:00 2001
From: Pablo Carranza <pablo@gitlab.com>
Date: Wed, 21 Jun 2017 20:20:23 +0200
Subject: [PATCH 4/6] Removed bogus line

---
 source/team.json.haml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/source/team.json.haml b/source/team.json.haml
index 2582333b710..e4ecef10fe6 100644
--- a/source/team.json.haml
+++ b/source/team.json.haml
@@ -1,5 +1,4 @@
 - team_array = []
-<li>Cloud Native
 - team = data.team.reject { |t| t.picture == 'logo-extra-whitespace.png' }
 - team.each do |member|
   - team_array << { picture: image_path("team/#{member.picture}") }
-- 
GitLab


From c901ac4d35b7af0952993eb230d71bac16095b49 Mon Sep 17 00:00:00 2001
From: ernstvn <ernst@gitlab.com>
Date: Wed, 21 Jun 2017 16:03:01 -0700
Subject: [PATCH 5/6] Edits to description of what it is to be embedded and
 fixed typo

---
 data/team.yml                                |  2 +-
 source/handbook/infrastructure/index.html.md | 43 ++++++++++++--------
 2 files changed, 28 insertions(+), 17 deletions(-)

diff --git a/data/team.yml b/data/team.yml
index 3bc95191426..2a91f7f15f4 100644
--- a/data/team.yml
+++ b/data/team.yml
@@ -2494,7 +2494,7 @@
   embedded_with: |
                  <li>Search</li>
   expertise: |
-             <li>Elastic Search Expert</li>
+             <li>Elasticsearch Expert</li>
   story:  |
           Victor is a technology enthusiast and a very passionate systems administrator.
           He enjoys all around automation, monitoring and development of tools that help everyone
diff --git a/source/handbook/infrastructure/index.html.md b/source/handbook/infrastructure/index.html.md
index 607c77d0cab..504436b6bf5 100644
--- a/source/handbook/infrastructure/index.html.md
+++ b/source/handbook/infrastructure/index.html.md
@@ -80,22 +80,33 @@ work on the topics listed above, using the principles and methods of [Site
 Reliability Engineering](https://landing.google.com/sre/book.html) a little bit
 more each day.
 
-#### Production Engineers Embedded With
-
-Additionally, specific production engineers are work embedded with different
-teams.
-
-Production engineers that are _embedded with_ will:
-
-- Participate in team calls for the team that are embedded with.
-- Keep up to date with the general Production Engineering duties: issues, fires, on-call. In fact, those will still be your priority unless the Lead explicitly says otherwise.
-- be embedded with one or more teams at a time, it depends on the level of involvement that is needed.
-- report to the Production Lead and participate in the Production team meetings as part of the duties, bringing the context from the team embedded with.
-- Help the team embedded with spending time to make their feature set or service "production ready".
-
-Since in our case these things are mostly already in production, then it means that you work on making sure that post factum the feature set or service meets the requirements written in the production readiness review questionnaire.
-
-Including improving the runbooks and documentation, alerting, monitoring, coding for resiliency, etc. Being so good that any other member of the Production Team can tend to the feature set or service as well and the "embedment" stops. At that point we name you an expert in the respective service.
+#### Embedded Production Engineers
+
+Additionally, specific production engineers can be "embedded" with one or multiple
+different teams for anything from a few weeks to months.
+
+If you are an "embedded" Production Engineers, then you are expected to:
+
+- Participate in team calls for the team that you are embedded with.
+- Keep up to date with the general Production Engineering team and duties:
+   - Take care of issues, fires, and on-call. In fact, those will still be your
+priority unless the Lead explicitly says otherwise.
+   - Continue to report to the Production Lead and participate in the Production
+    team meetings.
+    - Share the context of the team you are embedded with, with the rest of the
+    Production Team.
+- Help the team that you are embedded with to make their feature set or service
+ "production ready".
+
+Since at GitLab most "feature sets or services" are mostly already in
+production, then it means that you work on making sure that the feature set or
+service _post factum_ meets the requirements for Production Readiness [TODO: add
+link to production readiness review questionnaire]. This will typically involve
+improving the runbooks and documentation, alerting, monitoring, coding for
+resiliency, etc. By the time you are done, any other member of the Production
+Team should be able to tend to the feature set or service in production as well
+as you can, and the "embedment" stops. At that point you should be listed as an
+expert in the respective service.
 
 ## Production and Staging Access
 
-- 
GitLab


From 5db72282d91819c60b936f0da60000e30f5b39d8 Mon Sep 17 00:00:00 2001
From: Pablo Carranza <pablo@gitlab.com>
Date: Thu, 22 Jun 2017 13:51:44 +0200
Subject: [PATCH 6/6] Add infrastructure as a project and some changes to the
 team

---
 data/projects.yml | 7 +++++++
 data/team.yml     | 9 ++++++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/data/projects.yml b/data/projects.yml
index 19b8225d1ea..5668b1a082c 100644
--- a/data/projects.yml
+++ b/data/projects.yml
@@ -73,6 +73,13 @@ gitlab-com-runbooks:
     Describes system components, triage procedures, and commands to use in scenarios commonly faced
     by Gitlab.com production support engineers
 
+infrastructure:
+  name: Gitlab.com - infrastructure issue tracker
+  path: gitlab-com/infrastructure
+  link: https://gitlab.com/gitlab-com/infrastructure
+  description: |
+    Used to track the infrastructure work of GitLab.com itself
+
 gitlab-cookbooks:
   name: Gitlab.com - infrastructure node provisioning by role
   path: gitlab-cookbooks
diff --git a/data/team.yml b/data/team.yml
index eb4a17ed4d4..1865f506858 100644
--- a/data/team.yml
+++ b/data/team.yml
@@ -634,7 +634,8 @@
   gitlab: pcarranza
   projects:
     infrastructure: maintainer
-    gitlab-cog: maintainer
+    gitlab-com-runbooks: maintainer
+    gitlab-cookbooks: maintainer
   expertise:  |
               <li>On-call hero!</li>
   story:  |
@@ -2012,6 +2013,8 @@
   reports_to: Production Lead
   picture: daniele.jpg
   twitter: escociao
+  projects:
+    gitlab-com-infrastructure: maintainer
   gitlab: omame
   embedded_with: |
                  <li>Cloud Native Build</li>
@@ -2320,8 +2323,12 @@
   reports_to: Production Lead
   picture: jtevnan.jpg
   gitlab: jtevnan
+  projects:
+    gitlab-cookbooks: maintainer
   embedded_with: |
                  <li>Database</li>
+  expertise: |
+             <li>On-call hero!</li>
   story:  |
           Jason made his way into IT via the detour of studying metallurgy. After discovering that he understood database query plans much better than the periodic table, he switched majors and never looked back. When not attempting to automate everything, he is an avid cook, tries to be outdoors as much as he can and attempts to brew the perfect beer.
 
-- 
GitLab