Merge branch 'qa-remove-unneeded-only-extend' into 'master'

Fixes two bugs with the `package-and-qa-manual:master` and `build-qa-image` jobs

See merge request gitlab-org/gitlab-ce!32584

.gitlab/ci/qa.gitlab-ci.yml

@@ -10,34 +10,31 @@
     - install_gitlab_gem
     - ./scripts/trigger-build omnibus
   only:
+    refs:
       - branches@gitlab-org/gitlab-ce
       - branches@gitlab-org/gitlab-ee
-  except:
-    - master
 
 package-and-qa-manual:
-  extends:
-    - .package-and-qa-base
-    - .except-docs-qa
+  extends: .package-and-qa-base
   except:
     refs:
       - master
-  needs: ["build-qa-image", "gitlab:assets:compile pull-cache"]
   when: manual
+  needs: ["build-qa-image", "gitlab:assets:compile pull-cache"]
 
 package-and-qa-manual:master:
-  extends:
-    - .package-and-qa-base
-  needs: ["build-qa-image", "gitlab:assets:compile"]
+  extends: .package-and-qa-base
   only:
     refs:
       - master
   when: manual
+  needs: ["build-qa-image", "gitlab:assets:compile"]
 
 package-and-qa:
   extends: .package-and-qa-base
-  needs: ["build-qa-image", "gitlab:assets:compile pull-cache"]
-  allow_failure: true
   only:
+    refs:
       - /(^qa[\/-].*|.*-qa$)/@gitlab-org/gitlab-ce
       - /(^qa[\/-].*|.*-qa$)/@gitlab-org/gitlab-ee
+  needs: ["build-qa-image", "gitlab:assets:compile pull-cache"]
+  allow_failure: true

.gitlab/ci/review.gitlab-ci.yml

@@ -40,7 +40,7 @@
 build-qa-image:
   extends:
     - .review-docker
-    - .except-docs-qa
+    - .except-docs
   only:
     refs:
       - branches@gitlab-org/gitlab-ce