Commit Graph

258 Commits

Author SHA1 Message Date
Marcel Amirault fd5749b2c9 Docs: Merge EE doc/gitlab-geo to CE 2019-05-05 16:27:37 +00:00
Marcel Amirault b05936cbfa Docs: Merge Various EE /doc dirs to CE 2019-05-05 15:59:22 +00:00
Marcel Amirault 8b710c8ba4 Docs: Merge EE doc/analytics to CE 2019-05-01 07:13:12 +00:00
Rémy Coutable 84103e2d2f
Disable HTTP for the nginx-ingress Review App service
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-04-30 09:49:10 +02:00
Rémy Coutable 9649c2ef47
Organize better Review Apps and QA jobs
Also, play manual jobs once dependency jobs are done instead of polling
for the dependent jobs to be finished.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-04-24 19:25:37 +02:00
Rémy Coutable d0efb430ce
Make the Review App cleanup more efficient
Ensure that the cleanup starts right away.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-04-19 10:50:43 +02:00
Rémy Coutable 955136b56d
Don't fail the job if the Review App doesn't deploy properly
It seems the deploy function causes the job to fail if it doesn't
succeed. That wasn't the intent as we want to curl the Review App after
the deploy finished (even if it failed) because sometimes the Review App
is just a bit long to be ready.

This change wraps the Review App deployment with "set +e"/"set -e" to
ensure that the job doesn't fail right away if the deploy fails.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-04-17 10:49:02 +02:00
Rémy Coutable 64960ae762
Conform scripts/review_apps/review-apps.sh to shellcheck
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-04-09 15:57:04 +02:00
Rémy Coutable 6dadd965dc
Poll Review App after it's deployed to ensure it's accessible
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-04-09 15:56:53 +02:00
Lin Jen-Shin 700e8d1917 Merge branch 'enable-rubocop-application-record' into 'master'
Document ApplicationRecord / pluck_primary_key

Closes #59690

See merge request gitlab-org/gitlab-ce!26764
2019-04-01 15:55:46 +00:00
Nick Thomas 98a5976b78
Document ApplicationRecord / pluck_primary_key
We also enable the rubocop that makes it mandatory
2019-03-29 20:54:06 +00:00
Rémy Coutable 42102cd13a
Add more logging in the Review Apps deployment steps
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-03-28 11:56:38 +01:00
Tim Zallmann 5b05d2b987 First iteration of GitLab own Stylelints
DRYed the code

Changed new rules to warnings

Manual prettified and added stylelint rules
2019-03-26 08:13:21 +00:00
Rémy Coutable dc2a482ba1
Fix the .review-deploy-base job so that env variables are passed
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-03-19 08:31:20 +01:00
Balasankar "Balu" C 313022c79a
Use gitlab-ce as default project name in asset image building
Signed-off-by: Balasankar "Balu" C <balasankar@gitlab.com>
2019-03-15 19:15:54 +05:30
Balasankar "Balu" C de2106998a
Set asset image name properly for all possible CE/EE project names
Signed-off-by: Balasankar "Balu" C <balasankar@gitlab.com>
2019-03-15 17:00:57 +05:30
Nick Thomas 9f05e97aad
Run rubocop -a 2019-03-13 13:42:43 +00:00
Balasankar "Balu" C 24426f37d7
Tag asset docker image with GitLab version also
Signed-off-by: Balasankar "Balu" C <balasankar@gitlab.com>
2019-03-12 16:41:50 +05:30
Stan Hu f11030173b Batch insert CI rspec_profiling data
Instead of inserting a row after each example to an external database,
we save the CI profiling reports into the `rspec_profiling` directory
and insert the data in the update-tests-metadata CI stage.  This should
make each spec run faster and also reduce the number of PostgreSQL
connections needed by concurrent CI builds.

`scripts/insert-rspec-profiling-data` also inserts one file at a time
via the PostgreSQL COPY command for faster inserts. The one side effect
is that the `created_at` and `updated_at` timestamps aren't available
since they aren't generated in the CSV.

Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/10154
2019-03-12 09:51:37 +00:00
Mike Greiling c46b791911
Remove node 6 trailing comma support
This removes trailing commas from node scripts which previously
omitted them for compatiblity with node 6
2019-03-11 14:16:41 -05:00
Stan Hu fb6a4e21d4 Bring back Rugged implementation of find_commit
This brings back some of the changes in
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/20339.

For users using Gitaly on top of NFS, accessing the Git data directly
via Rugged is more performant than Gitaly. This merge request introduces
the feature flag `rugged_find_commit` to activate Rugged paths.

There are also Rake tasks `gitlab:features:enable_rugged` and
`gitlab:features:disable_rugged` to enable/disable these feature
flags altogether.

Part of four Rugged changes identified in
https://gitlab.com/gitlab-org/gitlab-ce/issues/57317.
2019-03-01 08:45:51 -08:00
James Fargher 6c4ca56b83 Move ChatOps docs from EE to core
ChatOps used to be in the Ultimate tier.
2019-02-21 03:34:42 +00:00
John T Skarbek a992c908b0 Adds the commit sha to the pushed assets
* This will upload the sha of the docker image containing assets to
  assist with building specific sha builds in the future.
* Addresses: https://gitlab.com/gitlab-org/release/framework/issues/51
2019-02-19 13:51:36 +00:00
John T Skarbek 00cb8de1ec
Revert "Adds the commit sha to the pushed assets"
This reverts commit 481938de84.
2019-02-18 08:57:57 -05:00
John T Skarbek 481938de84
Adds the commit sha to the pushed assets
* This will upload the sha of the docker image containing assets to
  assist with building specific sha builds in the future.
* Addresses: gitlab-org/release/framework#51
2019-02-18 08:55:49 -05:00
Elan Ruusamäe 83b1c44a77 add changelogs/README.md
this is to quickstart contributors who don't work daily on the codebase
2019-02-18 04:57:57 +00:00
Tim Zallmann 0b53566111 Added Stylelint Setup
Added stylelint to static-analysis
Updated yarn dependencies
CSS Fixes and rule adoptions of stylelint
Added stylelint-scss
Deduplicated yarn.lock to clear dependencies
2019-02-14 14:40:50 +01:00
Felipe Artur 65e9d33ec5 Fix security harness script
"[[" Syntax is not supported on debian based systems.
2019-02-08 15:28:44 -02:00
Rémy Coutable 6b39f55d3f
Add cache to the 'compile-assets' and 'gitlab:assets:compile' jobs
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-02-05 14:04:54 +01:00
Rémy Coutable 18dae64c49
Only use 2 replicas for nginx-ingress.controller
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-02-01 14:36:24 +01:00
Rémy Coutable e86f0cc212
Don't install Prometheus in Review Apps
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-02-01 14:33:27 +01:00
Rémy Coutable 3440aa7fa9
Fine-tune Review Apps pod resource requests
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-02-01 14:33:27 +01:00
Rémy Coutable 79eec04799
Pass $CI_COMMIT_TAG as GITLAB_TAG to the CNG triggered pipelines
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-01-30 09:42:48 +01:00
Lin Jen-Shin 64a738c0a1 Introduce echoinfo for information 2019-01-24 22:00:52 +08:00
Rémy Coutable f8a10191f0
Use $CI_COMMIT_REF_NAME for tags in trigger-build
Also, pass GITLAB_VERSION and GITLAB_ASSETS_TAG to CNG triggered
pipelines.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-01-24 08:43:11 +01:00
Rémy Coutable 67dfd8786b
Use $CI_COMMIT_REF_SLUG in review-apps.sh
Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-01-24 08:42:45 +01:00
Rémy Coutable a4f7324170
[Review Apps] Don't exit when dependable job fails
The function that retrieves the dependable job is pretty basic and
retrieves the first job found with the matching name, but this job can
be failed but then successfully retried. In that case, we would exit the
depending job even though the dependable job actually succeeded (the
second time). Let's simplify things, be optimistic and continue with the
depending job even if the dependable job fails.

That reverts to the original behavior.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2019-01-17 18:56:41 +01:00
Rémy Coutable 2064565abc Merge branch '55323-review-apps-use-2-replicas-for-the-tiller-deploy-deployment' into 'master'
[Review Apps] Use 2 replicas for the `tiller-deploy` Deployment

Closes #55323

See merge request gitlab-org/gitlab-ce!23806
2019-01-10 12:36:06 +00:00
Rémy Coutable 25d98e9d81 Merge branch 'exit-error-if-job-waited-fails' into 'master'
Exit `wait_for_job_to_be_done` as error if the job fails

See merge request gitlab-org/gitlab-ce!23997
2019-01-07 10:36:53 +00:00
Sean McGivern 1854f142e2 Stop using deprecated argument to `gem`
`--no-rdoc` and `--no-ri` have been deprecated since RubyGems 2.0:
https://github.com/rubygems/rubygems/blob/v2.0.0/History.txt#L43

They were finally removed in a recent release:
https://github.com/rubygems/rubygems/pull/2354
2018-12-31 09:30:23 +00:00
Mark Lapierre 38a10e3870 Exit as error if job fails
If a script is waiting for a job to be done and that job fails,
exit with an error status so that the script doesn't continue
with a prerequisite in an invalid state.
2018-12-21 18:30:52 +00:00
Rémy Coutable ebf73a19ce
Use 2 replicas for the Deployment
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-12-19 11:23:43 +01:00
Rémy Coutable a76ce8b46b
Specify the 'sync' policy for the external-dns deployments
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-12-18 17:49:53 +01:00
Mark Lapierre 0f751f6f17 Run gitlab:assets:compile on qa jobs
Optionally compiling assets when needed for qa jobs proved
unworkable so instead we fall back on compiling them for
all qa jobs.
2018-12-17 15:30:52 -05:00
Rémy Coutable 35c69ea218 Merge branch 'rs-prepare-build-parity' into 'master'
CE-EE parity for prepare_build script

See merge request gitlab-org/gitlab-ce!23819
2018-12-17 10:02:15 +00:00
Jasper Maes 44fef4fe4c Remove rails 4 support in CI, Gemfiles, bin/ and config/ 2018-12-14 19:36:22 +01:00
Robert Speicher a183aa3c47
CE-EE parity for prepare_build script
This removes EE-only directives in favor of file existence checks.
2018-12-13 14:32:42 -06:00
Robert Speicher 48ae572847
Remove CREATE_DB_USER variable in prepare_build
The last usage of this variable was removed in
0ab6469187.
2018-12-13 12:20:40 -06:00
Rémy Coutable 73dce6c870
[QA] Better retrieve job ID for a retried job
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-12-06 10:06:15 +01:00
Yorick Peterse 008c99bf02
Removed the merge-train script
We're moving the automatic merging logic to a separate repository, so
there's no need in keeping this file around.

[ci skip]
2018-12-05 13:42:32 +01:00