summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add instance-wide domain info43231-document-the-instance-level-auto-devops-domain-optionMatija Čupić2018-02-141-1/+3
|
* Merge branch 'fl-mr-widget-4' into 'master'Tim Zallmann2018-02-147-60/+86
|\ | | | | | | | | Moves mr widget components into vue files See merge request gitlab-org/gitlab-ce!17110
| * Moves mr widget components into vue filesfl-mr-widget-4Filipa Lacerda2018-02-137-60/+86
| | | | | | | | | | Adds i18n Improves test cases
* | Merge branch 'update-namespaces-api-docs' into 'master'Achilleas Pipinellis2018-02-141-5/+5
|\ \ | | | | | | | | | | | | Make parent_id field integer/null value instead of string See merge request gitlab-org/gitlab-ce!17085
| * | Make parent_id field integer/null value instead of stringupdate-namespaces-api-docsTomasz Maczukin2018-02-131-5/+5
| | |
* | | Merge branch 'qa/rs-suppress-git-output' into 'master'Rémy Coutable2018-02-142-3/+17
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Suppress git output when running QA as a non-default user Closes gitlab-qa#157 See merge request gitlab-org/gitlab-ce!17006
| * | | Suppress git output when running QA as a non-default userRobert Speicher2018-02-141-2/+12
| | | |
| * | | Add Runtime::User.default_nameRobert Speicher2018-02-141-1/+5
|/ / /
* | | Merge branch '41722-track-gcp-billing-enabled-project-changes' into 'master'Kamil Trzciński2018-02-144-23/+109
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Track GCP Billing enabled project changes" Closes #41722 See merge request gitlab-org/gitlab-ce!16962
| * | | Count all billing_state transitions with labels41722-track-gcp-billing-enabled-project-changesMatija Čupić2018-02-122-19/+16
| | | |
| * | | Fix GCP cluster feature specMatija Čupić2018-02-121-2/+2
| | | |
| * | | Log billing state changes in CheckGcpProjectBillingWorkerMatija Čupić2018-02-121-0/+13
| | | |
| * | | Refactor CheckGcpProjectBillingWorkerMatija Čupić2018-02-123-41/+34
| | | |
| * | | Use Prometheus counter instead of redisMatija Čupić2018-02-122-14/+9
| | | |
| * | | Add missing newline in CheckGcpProjectBillingWorker specMatija Čupić2018-02-121-0/+1
| | | |
| * | | Add specs for billing_enabled change counterMatija Čupić2018-02-121-0/+66
| | | |
| * | | Implement billing_enabled change counterMatija Čupić2018-02-121-0/+21
| | | |
* | | | Merge branch ↵Kamil Trzciński2018-02-143-2/+16
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '39885-ensure-users-cannot-create-environments-with-leading-slashes' into 'master' Resolve "Environment with starting slash in name causes error" Closes #39885 See merge request gitlab-org/gitlab-ce!17071
| * | | | Update issue-39885.ymlKamil Trzciński2018-02-141-2/+2
| | | | |
| * | | | Ensure users can't create environments with leading or trailing slashes ↵39885-ensure-users-cannot-create-environments-with-leading-slashesDylan Griffith2018-02-133-2/+16
| | | | | | | | | | | | | | | | | | | | (Fixes #39885)
* | | | | Merge branch ↵Rémy Coutable2018-02-143-120/+58
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '43240-external-tracker-issues-don-t-have-the-correct-closing-text-appended-to-mrs' into 'master' Resolve "External tracker issues don't have the correct closing text appended to MRs" Closes #1 et #43240 See merge request gitlab-org/gitlab-ce!17109
| * | | | | Fix closing issues text added to MRs for external issue trackersSean McGivern2018-02-143-120/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before, this would: 1. Not use the correct reference for non-JIRA external trackers. 2. Append 'Closes ' if an external tracker was enabled, but no issue matched the branch name.
* | | | | | Merge branch 'process-update' into 'master'Marin Jankovski2018-02-141-5/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update Process document with changes to the exception request process. See merge request gitlab-org/gitlab-ce!17118
| * | | | | | Update Process document with changes to the exception request process.Marin Jankovski2018-02-141-5/+8
| | | | | | |
* | | | | | | Merge branch 'feature-flags-for-user-apps' into 'master'Achilleas Pipinellis2018-02-141-1/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | clarify what kind of feature flags we support See merge request gitlab-org/gitlab-ce!17113
| * | | | | | | clarify what kind of feature flags we supportJob van der Voort2018-02-141-1/+6
| | | | | | | |
* | | | | | | | Merge branch 'ce-port-ee-f531c07299' into 'master'Sean McGivern2018-02-141-8/+7
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix a too strict array-matching expectation See merge request gitlab-org/gitlab-ce!17093
| * | | | | | | | Fix a too strict array-matching expectationRémy Coutable2018-02-141-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | | | | Merge branch 'jramsay-i18n-proofreader-process-docs' into 'master'Douwe Maan2018-02-142-31/+54
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Add clearer proofreader process to docs See merge request gitlab-org/gitlab-ce!17103
| * | | | | | | | Fix punctuation and address feedbackjramsay-i18n-proofreader-process-docsJames Ramsay2018-02-132-6/+8
| | | | | | | | |
| * | | | | | | | Add clearer proofreader process to docsJames Ramsay2018-02-132-27/+48
| | | | | | | | |
* | | | | | | | | Merge branch 'mk-fix-pg-undefined-table-ci-errors' into 'master'Rémy Coutable2018-02-146-32/+15
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Build Is Red: Admin > Users > Impersonation Tokens token creation allows creation of a token" Closes gitlab-ee#4914 See merge request gitlab-org/gitlab-ce!17102
| * | | | | | | | | Avoid dropping tables in testMichael Kozono2018-02-146-32/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | And use :migration tag to use deletion strategy, and to avoid caching tables, and to lock into a particular schema. Attempting to fix intermittent spec errors `PG::UndefinedTable: ERROR: relation "public.untracked_files_for_uploads" does not exist`.
* | | | | | | | | | Merge branch '42800-change-usage-of-avatar_icon' into 'master'Yorick Peterse2018-02-1435-59/+127
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change all occurrences of ApplicationHelper#avatar_icon to use a User object where possible Closes #42800 See merge request gitlab-org/gitlab-ce!16976
| * | | | | | | | | | Extract repeated logic into #avatar_icon_for.42800-change-usage-of-avatar_iconAndreas Brandl2018-02-134-16/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This essentially allows to pass both user and email, so that we can either prefer the user to retrieve the avatar or (if user is not present) fall back to the email lookup.
| * | | | | | | | | | Add changelog.Andreas Brandl2018-02-131-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #42800.
| * | | | | | | | | | Always eagerly load a note's author.Andreas Brandl2018-02-131-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Always eagerly load an issue's author.Andreas Brandl2018-02-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather radical but avoids n+1 queries in the rather common case we want to include information about the author.
| * | | | | | | | | | Extract method to improve readability.Andreas Brandl2018-02-131-7/+11
| | | | | | | | | | |
| * | | | | | | | | | Remove generic #avatar_icon helper.Andreas Brandl2018-02-132-36/+0
| | | | | | | | | | |
| * | | | | | | | | | Explicit use of avatar_icon_* calls depending on situation.Andreas Brandl2018-02-134-13/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to drop the generic #avatar_icon helper that supports both an email and a user object being passed in. Instead, we want to explicitly use the #avatar_icon_for_user and #avatar_icon_for_email helpers depending on what we have at hand. This allows us to avoid unnecessary database queries (e.g. call User.find_by_any_email if we already have the user). In situations like here, this makes it less convenient to use.
| * | | | | | | | | | Use avatar_icon_for_email for commits.Andreas Brandl2018-02-132-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to keep the email lookup as we want to fall back to displaying the gravatar result for a given email if the user does not exist.
| * | | | | | | | | | Retrieve project creator's avatar by creator, not creator#email.Andreas Brandl2018-02-131-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Retrieve namespace owner's avatar by owner, not owner#email.Andreas Brandl2018-02-131-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Retrieve issue author's avatar by user, not user#email.Andreas Brandl2018-02-131-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Use more specific #avatar_icon_for_email.Andreas Brandl2018-02-133-3/+3
| | | | | | | | | | |
| * | | | | | | | | | Use more specific #avatar_icon_for_user.Andreas Brandl2018-02-1321-23/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Whenever we already deal with a User object, let's use the more specific method avatar_icon_for_user.
| * | | | | | | | | | Refactor and split ApplicationHelper#avatar_icon.Andreas Brandl2018-02-132-7/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we don't use the original `ApplicationHelper#avatar_icon` anymore, we can just remove it (and its specs). Closes #42800.
* | | | | | | | | | | Merge branch '38135-remove-ignore-column-from-events' into 'master'Yorick Peterse2018-02-141-4/+0
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove ignore_column :data from Event Closes #38135 See merge request gitlab-org/gitlab-ce!17094
| * | | | | | | | | | | Remove ignore_column from Events#data.38135-remove-ignore-column-from-eventsAndreas Brandl2018-02-131-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #38135.