Commit Graph

214 Commits

Author SHA1 Message Date
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
Yorick Peterse ee6fb2b1ee
Automatically merge CE master into EE master
This sets up GitLab CI to automatically push CE master changes into EE
master, or revert them if the changes cause merge conflicts. The CI
configuration contains a single job to do this: `merge:master`. This job
is executed for every push to master, and periodically using a CI
schedule.

The periodic job is necessary because incremental jobs may not be able
to revert commits if newly added commits depend on these commits. By
re-running the job periodically (including all changes since a large
enough time frame), we can ensure that such commits are also reverted
(if they still conflict at that time).

The job runs in its own "merge" stage, _after_ the build and prepare
stages, but _before_ running the tests. This ensures that randomly
failing tests won't prevent code from being merged into EE. Running the
stage after the "prepare" stage reduces the chances of the job reverting
CE changes just because it ran before a corresponding EE MR was merged
into EE master.
2018-12-04 14:40:04 +01:00
Rémy Coutable 93420f50b1
Pass GITLAB_REF_SLUG to CNG triggered pipelines
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-29 10:49:30 +01:00
Sean McGivern e8db29d086 Merge branch '49570-add-review_qa-job' into 'master'
Add an automatic job to run QA smoke tests against Review Apps

Closes #49570

See merge request gitlab-org/gitlab-ce!22853
2018-11-28 10:39:56 +00:00
Rémy Coutable 17756db81d
Override CI_COMMIT_REF_SLUG for QA branches
The `gitlab:assets:compile` job isn't run for the QA branches, thus
there's no Docker image correspinding these branches in the registry.

By overriding `CI_COMMIT_REF_SLUG` to `master` for QA branches, the
`fetch-assets` job in the `omnibus-gitlab` pipeline will pull the
`master` assets Docker image.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-26 15:08:01 +01:00
Rémy Coutable f4b8ecbf9f
Add a manual job to run QA against a Review App
1. Renames review to review-deploy.
2. Renames stop_review to review-stop.
3. Adds a build-qa-image job to the prepare stage and save the QA
  Docker image as artifact (example:
  https://gitlab.com/gitlab-org/gitlab-ce/-/jobs/120967019).
4. Adds a manual review-qa job to the test stage to run QA tests
  against the Review App (example:
  https://gitlab.com/gitlab-org/gitlab-ce/-/jobs/120967238).
5. Make the review-deploy job plays the review-qa job as soon as the
  Review App is deployed (example:
  https://gitlab.com/gitlab-org/gitlab-ce/-/jobs/120988775).
6. Updates the Review Apps documentation accordingly.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-26 14:49:03 +01:00
Kushal Pandya 7002752d47 Merge branch 'fix-typos-occured' into 'master'
Fix typos occured to occurred

See merge request gitlab-org/gitlab-ce!23105
2018-11-22 14:02:52 +00:00
Rémy Coutable b57c1fa71c
Pass GITLAB_REF_SLUG to the Omnibus triggers
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-22 09:32:36 +01:00
Hiroyuki Sato 692f87215d Merge remote-tracking branch 'origin/master' into fix-typos-occured
Conflicts:
- app/assets/javascripts/ide/stores/modules/pipelines/actions.js
- spec/javascripts/ide/stores/modules/pipelines/actions_spec.js
2018-11-21 11:58:52 +09:00
Ian Baum 553eaf01f7 Add BUILD_ASSETS_IMAGE CI variable
* Defaults to FALSE
* Can be used to disable building/pushing the assets image in the
gitlab:compile:assets job
2018-11-15 09:45:04 -06:00
Hiroyuki Sato b689b20f20 Fix typos occured to occurred 2018-11-16 00:34:24 +09:00
Jan Provaznik 62bd2aca04 Switch rails 4 to 5 in gitlab-ci
CI jobs will be triggered both with rails 4 and 5
to make sure we keep backward compatibility if it turns out
we have to switch back to rails 4.

Rails 4 jobs are not allowed to fail for now, these jobs will be
removed in a follow-up MR next cycle.
2018-11-14 12:38:30 +01:00
Rémy Coutable 4383c7947e
Ensure we abort the cleanup if release is empty
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-13 18:01:46 +01:00
Balasankar "Balu" C 2b7ad9da09
Disable usage pings in review apps 2018-11-11 17:15:03 +05:30
Rémy Coutable f7f42cf345
Ensure we create the secrets at the right time
In `deploy`, if the previous deployment failed, we delete/cleanup all
the objects related to the release, including secrets. The problem is
that if we create the root password before that, it will be then
recreated during the deploy with a random value!

By creatigng the secret just before actually deplying a new release, we
ensure that it won't be overriden.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-09 12:30:51 +01:00
Rémy Coutable 6a189c1fcd
Allow to override the gitlab-qa ref for the downstream trigger
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-11-06 16:17:01 +01:00
Ian Baum ce4aae2379
Use CI_COMMIT_REF_SLUG when building assets image
* Using CI_COMMIT_REF_NAME fails when the branch or tag contains special
characters
2018-11-01 14:26:51 -05:00
Achilleas Pipinellis 7f47e13218 Merge branch 'gt-update-typos-in-comments-and-specs' into 'master'
Fix typos in comments and specs

See merge request gitlab-org/gitlab-ce!22683
2018-11-01 10:34:23 +00:00
George Tsiolis 733ae94921 Fix typos in comments and specs 2018-11-01 08:59:20 +02:00
Filipa Lacerda 59d4a6d81c Merge branch 'prettify-all-remaining-things' into 'master'
Prettify remaining files with differences in CE and EE

See merge request gitlab-org/gitlab-ce!22702
2018-10-31 22:42:22 +00:00
Mike Greiling 31f26a9382
Add prettier to static-analysis job 2018-10-31 14:34:42 -05:00
Ian Baum 70ee4e1b3e Build a docker container storing only the frontent assets
* Run as part of gitlab:assets:compile job

* Will be used by omnibus-gitlab and the CNG images to avoid
compiling multiple times

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/22235
2018-10-31 11:21:14 -05:00
Ian Baum 41994c2adf Add omitted indentation fix 2018-10-23 10:05:57 -05:00
Ian Baum 58690a63fa Add review app to gitlab-ce
* Uses the same supporting code as used in EE
* Includes automated cleanup
* Install external-dns helm chart to review apps cluster if it isn't
already
* Adds variables REVIEW_APPS_AWS_SECRET_KEY and
REVIEW_APPS_AWS_ACCESS_key
* review-apps-ce uses a different cipher
2018-10-23 10:05:57 -05:00
Mike Greiling a32511692f Fix/update prettier script 2018-10-22 17:47:03 +00:00
Rémy Coutable 25d8c8d1f0
Improve automated Review Apps cleanup
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-10-22 09:57:50 +02:00
Rémy Coutable 4d7c6a61d7
Improve the variables passed in trigger-build
Make the variable names consistent by following the
`TOP_UPSTREAM_SOURCE_*` pattern.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-10-10 12:10:04 +02:00
Rémy Coutable 02b456dbe8
Pass 'TOP_UPSTREAM_SOURCE_SHA' in trigger-build
This is needed because `GITLAB_VERSION` has a special meaning in
`omnibus-gitlab` triggers: this is the GitLab version to build.

The problem is that `omnibus-gitlab` also has triggers to run QA for an
`omnibus-gitlab` commit, and if we use `GITLAB_VERSION` in that case,
the comment would be posted on the GitLab CE/EE commit (stored in
`GITLAB_VERSION`), whci hwouldn't make any sense.
Thus we need `TOP_UPSTREAM_SOURCE_SHA` to represent the commit on
which we want to leave a comment.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-10-10 12:10:04 +02:00
Rémy Coutable 1922ca0e67
Improve downstream pipeine trigger class
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-10-08 14:23:58 +02:00
Alejandro Rodríguez a99bf447a2 Remove Gitlab::Git::Repository#rugged and Gollum code
Cleanup code, and refactor tests that still use Rugged. After this, there should
be no Rugged code that access the instance's repositories on non-test
environments. There is still some rugged code for other tasks like the
repository import task, but since it doesn't access any repository storage path
it can stay.
2018-10-02 16:34:28 -03:00
Rémy Coutable 159b650d74
Cleanup review apps more rapidly
- Stop review app's environment after 2 days
- Delete review app's environment after 3 days
- Delete Helm release after 4 days

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-10-01 14:08:40 +02:00
Rémy Coutable 3299680cdd
[CE] Port review apps file to CE
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-09-25 16:41:27 +02:00
Rémy Coutable 45d8c213fd
Refactor scripts/trigger-build and post a commit note with the downstream pipeline URL for omnibus triggers
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-09-24 18:24:13 +02:00
Achilleas Pipinellis 8cb39fa8da Merge branch 'docs-port-ee-community-contributions' into 'master'
Docs port ee community contributions

See merge request gitlab-org/gitlab-ce!21311
2018-09-03 10:18:42 +00:00
Jasper Maes 8f51727d95 Backport schema_changed.sh from EE which prints the diff if the schema is different 2018-08-30 08:29:08 +02:00
John Eismeier 2c06a13854
Update scripts/lint-doc.sh
Originally submitted at
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/3056.

Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-08-29 17:35:41 +02:00
gfyoung 8a3d565c8e Don't use arguments keyword in gettext script
Causing SAST build to fail unnecessarily.

Follow-up to !19378.
2018-08-19 01:05:50 -07:00
🌴🇿🇦 Lukas Eipert (OOO until Summit) 8f9442fcdf Frontend: Proper gettext extraction with gettext-extractor 2018-08-16 19:03:57 +00:00
Ahmad Hassan 8ea9c81593 Use rugged to validate ref name 2018-07-19 12:23:46 +02:00
Winnie Hellmann d42cd343ca Removed return statements from prettier script 2018-07-09 09:53:23 +00:00
Achilleas Pipinellis 72fa064000
Simplify downstream branch name 2018-07-05 11:02:38 +02:00
Mike Greiling 70bf08073d Resolve "100% CPU for webpack-dev-server running in GDK" 2018-06-25 19:30:37 +00:00
Achilleas Pipinellis 82012d4608
Always run review app for docs-only branches 2018-06-25 15:21:15 +02:00
Rémy Coutable 67f310e41f
Ignore GitLab API hiccups in scripts/trigger-build
Signed-off-by: Rémy Coutable <remy@rymai.me>
2018-06-20 17:17:29 +02:00
Tim Zallmann e04dfff858
Fixed the searching on the whole disk :-P 2018-06-15 12:43:04 +02:00