summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Add a close issue slack slash commandSarah Yasonik2019-08-295-9/+105
| | | | | | Adds a slash command in slach for closing issues. See https://docs.gitlab.com/ee/integration/slash_commands.html for documentation on the wider feature set.
* Revert "Cache branch and tag names as Redis sets"Nick Thomas2019-08-292-120/+0
| | | | This reverts commit 0eff75fa2b6691b6fba31fcc2842f51debd249a9.
* Merge branch '60071-remove-gitlab-keys-usage' into 'master'Nick Thomas2019-08-294-87/+73
|\ | | | | | | | | Remove usage of gitlab-shell gitlab-keys script See merge request gitlab-org/gitlab-ce!32138
| * Auto create authorized_keys file if doesn't exist60071-remove-gitlab-keys-usagePatrick Bajao2019-08-292-0/+13
| | | | | | | | Utilize the auto repair functionality of system checks.
| * Add system check for authorized_keys file permPatrick Bajao2019-08-293-7/+54
| | | | | | | | | | This check is being removed from gitlab-shell as the file is now being managed by gitlab-rails.
| * Remove the fallback path from gitlab-cePatrick Bajao2019-08-291-80/+6
| |
* | Merge branch 'sh-support-content-for-snippets-api' into 'master'Kamil Trzciński2019-08-291-5/+9
|\ \ | |/ |/| | | | | | | | | Standardize use of `content` parameter in snippets API Closes #66673 See merge request gitlab-org/gitlab-ce!32296
| * Standardize use of `content` parameter in snippets APIsh-support-content-for-snippets-apiStan Hu2019-08-281-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was some confusion over whether `code` or `content` is the right parameter for snippets. Internally, the database stores `content`. However: 1. Project snippets use `code`. `code` gets remapped in `content` in `lib/api/project_snippets.rb`. 2. Personal snippets use `content`. To unify these APIs, allow an alias of `content` to work for project snippets. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/66673
* | Merge branch 'mk/simplify-internal-post-receive-messages' into 'master'Grzegorz Bizon2019-08-294-17/+77
|\ \ | |/ |/| | | | | | | | | Simplify internal post receive messages Closes #59808 See merge request gitlab-org/gitlab-ce!31640
| * Simplify internal post receive messagesmk/simplify-internal-post-receive-messagesMichael Kozono2019-08-284-17/+77
| | | | | | | | | | | | | | | | Instead of sending varied data to Gitaly, and making Gitaly construct various messages, build the messages first and have Gitaly print either basic messages or alert messages, in the order they come. Depends on https://gitlab.com/gitlab-org/gitaly/merge_requests/1410
* | Merge branch 'performance-bar-warnings' into 'master'Stan Hu2019-08-295-12/+90
|\ \ | |/ |/| | | | | Add warnings to performance bar response See merge request gitlab-org/gitlab-ce!31054
| * Add top-level warnings key to performance bar responseperformance-bar-warningsSean McGivern2019-08-281-0/+19
| | | | | | | | | | | | This key is useful to reduce the amount of logic needed on the frontend: if `has_warnings` is true, then the frontend knows that the request in question has warnings for some metric.
| * Return warnings for performance bar from backendSean McGivern2019-08-284-12/+71
| | | | | | | | | | | | | | | | | | | | | | For each DetailedView subclass, we add a `warnings` array to: 1. The top-level response. 2. Each individual call under the `details` key. We use the `.thresholds` hash on the DetailedView to determine what's a warning. If that hash is empty (the default), then the warnings array will always be empty.
* | Support Gitaly feature flags in workhorseJacob Vosmaer2019-08-281-2/+4
| |
* | Rename Labkit::Tracing::GRPCInterceptor to GRPC::ClientInterceptorAndrew Newdigate2019-08-281-1/+1
| | | | | | | | The original name has been deprecated
* | Update CE files for GSD projects filterVictor Zagorodny2019-08-283-11/+30
| | | | | | | | | | | | | | | | | | A new param with_security_reports was added to GET /groups/:id/projects API and the code to support this logic in GroupProjectsFinder and Project model. Also, a DB index was added to ci_job_artifacts table to speed up the search of security reports artifacts for projects
* | Changes snowplow to use cookies for sessionsJeremy Jackson2019-08-282-35/+44
| | | | | | | | | | This also restructures how and where the configuration for Snowplow lives.
* | Rename dev, org, com helperDoug Stull2019-08-281-1/+1
| |
* | Fix a nil error in Gitlab::Danger::TeammateRémy Coutable2019-08-271-2/+2
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Replace prepend_entity with prepend_if_eeWinnie Hellmann2019-08-271-0/+18
|/ | | | (cherry picked from commit 897a9d308db46b620b738b98f2b0e5630ac7d2dd)
* Merge branch ↵Lin Jen-Shin2019-08-271-2/+2
|\ | | | | | | | | | | | | | | | | '65312-use-group-labels-instead-of-legacy-team-labels-for-danger-picking-test-reviewers' into 'master' Detect the new stage labels in `Gitlab::Danger::Teammate` Closes #65312 See merge request gitlab-org/gitlab-ce!32261
| * Detect the new stage labels in Gitlab::Danger::Teammate65312-use-group-labels-instead-of-legacy-team-labels-for-danger-picking-test-reviewersRémy Coutable2019-08-271-2/+2
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Merge branch '13806-fix' into 'master'Lin Jen-Shin2019-08-271-2/+2
|\ \ | |/ |/| | | | | | | | | Add nil check for Gitlab.config.prometheus Closes #13806 See merge request gitlab-org/gitlab-ce!32201
| * Add nil check for Gitlab.config.prometheus13806-fixrpereira22019-08-261-2/+2
| |
* | Merge branch '57657-promote-label-to-group-label-via-api-endpoint' into 'master'Sean McGivern2019-08-271-0/+25
|\ \ | | | | | | | | | | | | | | | | | | Promote Label to Group Label via API endpoint Closes #57657 See merge request gitlab-org/gitlab-ce!25218
| * | Update docs targeting 12.3Robert Schilling2019-08-261-1/+1
| | |
| * | Move promote to put and add more specsRobert Schilling2019-08-261-1/+1
| | |
| * | Add docs and first specsRobert Schilling2019-08-261-3/+3
| | |
| * | API: Promote project labels to group labelsRobert Schilling2019-08-261-0/+25
| | |
* | | Merge branch '63502-encrypt-deploy-token' into 'master'Stan Hu2019-08-271-4/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Store deploy tokens encrypted" Closes #63502 See merge request gitlab-org/gitlab-ce!30679
| * | | Removed rubocop disable flags, updated changelogEtienne Baqué2019-08-261-2/+0
| | | |
| * | | Updated call to find deploy tokenEtienne Baqué2019-08-261-2/+1
| | | |
* | | | Update docs and comments about saml with allow_bypass_two_factordodocat2019-08-271-1/+6
|/ / / | | | | | | | | | allow_bypass_two_factor configration dose not work with saml provider
* | | Revert "Merge branch 'user-tracking-settings' into 'master'"Jeremy Jackson2019-08-262-44/+35
| | | | | | | | | | | | This reverts merge request !31826
* | | Remove unnecessary includeReuben Pereira2019-08-261-2/+0
| | |
* | | Changes snowplow to use cookies for sessionsJeremy Jackson2019-08-262-35/+44
| | | | | | | | | | | | | | | This also restructures how and where the configuration for Snowplow lives.
* | | Merge branch 'sh-fix-nplusone-issues' into 'master'Mayra Cabrera2019-08-261-1/+2
|\ \ \ | | | | | | | | | | | | | | | | Fix N+1 Gitaly calls in /api/v4/projects/:id/issues See merge request gitlab-org/gitlab-ce!32171
| * | | Fix N+1 Gitaly calls in /api/v4/projects/:id/issuessh-fix-nplusone-issuesStan Hu2019-08-231-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a follow-up from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/31938. In GitLab 9.0, https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9661 removed the `subscribed` flag from the API when the user requested a list of issues or merge requests since calculating this value triggers extensive Markdown processing. In GitLab 12.0 via a4fbf39e, we accidentally reintroduced this performance regression by changing `IssueBasic` to `Issue` in `entities.rb`. This showed up as a Gitaly N+1 issue since the Markdown processing would attempt to extract a commit if it detected a regex that matched a commit. We restore the prior behavior by once again removing the `subscribed` flag for the bulk list of issues and merge requests and add a test to ensure they aren't reintroduced. Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/66202
* | | | Replace echo function with a resolverBrett Walker2019-08-261-1/+3
| | | | | | | | | | | | | | | | | | | | The `GraphQL::Function` has been deprecated in favor of resolvers.
* | | | Change default visibility level for FogBugz imported projects to PrivateGeorge Koltsov2019-08-261-1/+1
| | | |
* | | | Add usage pings for merge request creatingIgor2019-08-262-1/+12
| | | | | | | | | | | | | | | | Code Review Usage Ping for Create SMAU
* | | | Danger: Map .rubocop{,_todo}.yml to ~backendPeter Leitzen2019-08-261-0/+1
| |/ / |/| |
* | | Merge branch 'db_load_balancing_service_discovery_srv' into 'master'Kamil Trzciński2019-08-261-1/+2
|\ \ \ | | | | | | | | | | | | | | | | DB Load Balancing: Support SRV lookups See merge request gitlab-org/gitlab-ce!32135
| * | | CE port: allow SRV records in DB service discoverydb_load_balancing_service_discovery_srvTiger2019-08-231-1/+2
| | | |
* | | | Add helper to exactly undo cleanup_concurrent_column_renameReuben Pereira2019-08-251-1/+40
| |_|/ |/| | | | | | | | - Also add helper to undo rename_column_concurrently.
* | | Drop existing trigger before creating new oneReuben Pereira2019-08-241-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - When renaming a column concurrently, drop any existing trigger before attempting to create a new one. When running migration specs multiple times (as it happens during local development), the down method of previous migrations are called. If any of the called methods contains a call to rename_column_concurrently, a trigger will be created and not removed. So, the next time a migration spec is run, if the same down method is executed again, it will cause an error when attempting to create the trigger (since it already exists). Dropping the trigger if it already exists will prevent this problem.
* | | Add a link to docs in project descriptionReuben Pereira2019-08-231-0/+251
| | | | | | | | | | | | Add to the service and migration both.
* | | Merge branch '66402-use-visual-review-tools-npm-package' into 'master'Mike Greiling2019-08-231-6/+0
|\ \ \ | | | | | | | | | | | | | | | | Resolve "Use Visual Review Tools NPM package" See merge request gitlab-org/gitlab-ce!32159
| * | | Move visual review toolbar to NPMshampton2019-08-231-6/+0
| | | | | | | | | | | | | | | | | | | | Remove the visual review toolbar code in favor of using the NPM package.
* | | | Read pipelines from public projects though APIMarius Bobin2019-08-231-1/+2
| | | | | | | | | | | | | | | | | | | | Allow users to read pipelines for public projects with public builds enabled without providing an access token.