diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2019-12-05 12:07:27 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2019-12-05 12:07:27 +0300 |
commit | 8f764d21b0011056e1492d92afe3bd40b847b9f7 (patch) | |
tree | c1b5bc3eed8ac4d4b458466a9e4a548d94f389d9 | |
parent | f7ead77657cd7dc539319624301c22bf91b986e4 (diff) |
Add latest changes from gitlab-org/gitlab@master
-rw-r--r-- | changelogs/unreleased/20709-add-runner-info-in-build-event.yaml | 5 | ||||
-rw-r--r-- | db/migrate/20191119220425_rename_software_license_policies_approval_status_to_classification.rb | 17 | ||||
-rw-r--r-- | db/post_migrate/20191119221041_cleanup_software_license_policies_classification_rename.rb | 17 | ||||
-rw-r--r-- | db/schema.rb | 2 | ||||
-rw-r--r-- | doc/user/project/integrations/webhooks.md | 9 | ||||
-rw-r--r-- | lib/gitlab/data_builder/build.rb | 14 | ||||
-rw-r--r-- | locale/gitlab.pot | 24 | ||||
-rw-r--r-- | spec/lib/gitlab/data_builder/build_spec.rb | 7 |
8 files changed, 90 insertions, 5 deletions
diff --git a/changelogs/unreleased/20709-add-runner-info-in-build-event.yaml b/changelogs/unreleased/20709-add-runner-info-in-build-event.yaml new file mode 100644 index 00000000000..4a9d79d7299 --- /dev/null +++ b/changelogs/unreleased/20709-add-runner-info-in-build-event.yaml @@ -0,0 +1,5 @@ +--- +title: Add runner information in build web hook event +merge_request: 20709 +author: Gaetan Semet +type: added diff --git a/db/migrate/20191119220425_rename_software_license_policies_approval_status_to_classification.rb b/db/migrate/20191119220425_rename_software_license_policies_approval_status_to_classification.rb new file mode 100644 index 00000000000..98c14f42558 --- /dev/null +++ b/db/migrate/20191119220425_rename_software_license_policies_approval_status_to_classification.rb @@ -0,0 +1,17 @@ +# frozen_string_literal: true + +class RenameSoftwareLicensePoliciesApprovalStatusToClassification < ActiveRecord::Migration[5.2] + include Gitlab::Database::MigrationHelpers + + DOWNTIME = false + + disable_ddl_transaction! + + def up + rename_column_concurrently :software_license_policies, :approval_status, :classification + end + + def down + undo_rename_column_concurrently :software_license_policies, :approval_status, :classification + end +end diff --git a/db/post_migrate/20191119221041_cleanup_software_license_policies_classification_rename.rb b/db/post_migrate/20191119221041_cleanup_software_license_policies_classification_rename.rb new file mode 100644 index 00000000000..eb605dce5aa --- /dev/null +++ b/db/post_migrate/20191119221041_cleanup_software_license_policies_classification_rename.rb @@ -0,0 +1,17 @@ +# frozen_string_literal: true + +class CleanupSoftwareLicensePoliciesClassificationRename < ActiveRecord::Migration[5.2] + include Gitlab::Database::MigrationHelpers + + DOWNTIME = false + + disable_ddl_transaction! + + def up + cleanup_concurrent_column_rename :software_license_policies, :approval_status, :classification + end + + def down + undo_cleanup_concurrent_column_rename :software_license_policies, :approval_status, :classification + end +end diff --git a/db/schema.rb b/db/schema.rb index 3bdf41e609f..c33223f8b59 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -3673,7 +3673,7 @@ ActiveRecord::Schema.define(version: 2019_12_02_031812) do create_table "software_license_policies", id: :serial, force: :cascade do |t| t.integer "project_id", null: false t.integer "software_license_id", null: false - t.integer "approval_status", default: 0, null: false + t.integer "classification", default: 0, null: false t.index ["project_id", "software_license_id"], name: "index_software_license_policies_unique_per_project", unique: true t.index ["software_license_id"], name: "index_software_license_policies_on_software_license_id" end diff --git a/doc/user/project/integrations/webhooks.md b/doc/user/project/integrations/webhooks.md index f027fb69157..2f2ab65c5dc 100644 --- a/doc/user/project/integrations/webhooks.md +++ b/doc/user/project/integrations/webhooks.md @@ -1223,6 +1223,7 @@ X-Gitlab-Event: Job Hook "build_duration": null, "build_allow_failure": false, "build_failure_reason": "script_failure", + "pipeline_id": 2366, "project_id": 380, "project_name": "gitlab-org/gitlab-test", "user": { @@ -1248,10 +1249,18 @@ X-Gitlab-Event: Job Hook "git_ssh_url": "git@192.168.64.1:gitlab-org/gitlab-test.git", "git_http_url": "http://192.168.64.1:3005/gitlab-org/gitlab-test.git", "visibility_level": 20 + }, + "runner": { + "active": true, + "is_shared": false, + "id": 380987, + "description": "shared-runners-manager-6.gitlab.com" } } ``` +Note that `commit.id` is the id of the pipeline, not the id of the commit. + ## Image URL rewriting From GitLab 11.2, simple image references are rewritten to use an absolute URL diff --git a/lib/gitlab/data_builder/build.rb b/lib/gitlab/data_builder/build.rb index 3407380127e..f7b7db50b2f 100644 --- a/lib/gitlab/data_builder/build.rb +++ b/lib/gitlab/data_builder/build.rb @@ -31,6 +31,8 @@ module Gitlab build_duration: build.duration, build_allow_failure: build.allow_failure, build_failure_reason: build.failure_reason, + pipeline_id: commit.id, + runner: build_runner(build.runner), # TODO: do we still need it? project_id: project.id, @@ -43,6 +45,7 @@ module Gitlab }, commit: { + # note: commit.id is actually the pipeline id id: commit.id, sha: commit.sha, message: commit.git_commit_message, @@ -75,6 +78,17 @@ module Gitlab author = commit.try(:author) author ? Gitlab::Routing.url_helpers.user_url(author) : "mailto:#{pipeline.git_author_email}" end + + def build_runner(runner) + return unless runner + + { + id: runner.id, + description: runner.description, + active: runner.active?, + is_shared: runner.instance_type? + } + end end end end diff --git a/locale/gitlab.pot b/locale/gitlab.pot index 5f8c553bc80..41f0902147d 100644 --- a/locale/gitlab.pot +++ b/locale/gitlab.pot @@ -1119,6 +1119,9 @@ msgstr "" msgid "Adds an issue to an epic." msgstr "" +msgid "Adjust your filters/search criteria above." +msgstr "" + msgid "Admin Area" msgstr "" @@ -5904,9 +5907,6 @@ msgstr "" msgid "Designs" msgstr "" -msgid "Designs coming soon." -msgstr "" - msgid "Destroy" msgstr "" @@ -9292,6 +9292,9 @@ msgstr "" msgid "If using GitHub, you’ll see pipeline statuses on GitHub for your commits and pull requests. %{more_info_link}" msgstr "" +msgid "If you believe this may be an error, please refer to the %{linkStart}Geo Troubleshooting%{linkEnd} documentation for more information." +msgstr "" + msgid "If you believe this page to be an error, check out the links below for more information." msgstr "" @@ -10130,12 +10133,21 @@ msgstr "" msgid "Last reply by" msgstr "" +msgid "Last repository check run" +msgstr "" + msgid "Last seen" msgstr "" +msgid "Last successful sync" +msgstr "" + msgid "Last successful update" msgstr "" +msgid "Last time verified" +msgstr "" + msgid "Last update" msgstr "" @@ -11500,6 +11512,9 @@ msgstr "" msgid "No %{providerTitle} repositories found" msgstr "" +msgid "No Design Repositories match this filter" +msgstr "" + msgid "No Epic" msgstr "" @@ -11692,6 +11707,9 @@ msgstr "" msgid "None" msgstr "" +msgid "Not Implemented" +msgstr "" + msgid "Not all data has been processed yet, the accuracy of the chart for the selected timeframe is limited." msgstr "" diff --git a/spec/lib/gitlab/data_builder/build_spec.rb b/spec/lib/gitlab/data_builder/build_spec.rb index b170ef788d9..fdb855de786 100644 --- a/spec/lib/gitlab/data_builder/build_spec.rb +++ b/spec/lib/gitlab/data_builder/build_spec.rb @@ -3,7 +3,8 @@ require 'spec_helper' describe Gitlab::DataBuilder::Build do - let(:build) { create(:ci_build) } + let(:runner) { create(:ci_runner, :instance) } + let(:build) { create(:ci_build, :running, runner: runner) } describe '.build' do let(:data) do @@ -20,6 +21,10 @@ describe Gitlab::DataBuilder::Build do it { expect(data[:build_failure_reason]).to eq(build.failure_reason) } it { expect(data[:project_id]).to eq(build.project.id) } it { expect(data[:project_name]).to eq(build.project.full_name) } + it { expect(data[:pipeline_id]).to eq(build.pipeline.id) } + it { expect(data[:commit][:id]).to eq(build.pipeline.id) } + it { expect(data[:runner][:id]).to eq(build.runner.id) } + it { expect(data[:runner][:description]).to eq(build.runner.description) } context 'commit author_url' do context 'when no commit present' do |