| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Utilize `json_fields` to expose fields via
`Service#as_json(only: json_fields)`.
|
|
|
|
|
|
| |
Because we don't have any destroy callbacks (or other logic
triggered on event destroy), there is no reason for deleting events
inefficiently one by one, instead we can use :delete_all.
|
|
|
|
| |
Just replace RSA.new with PKey.read
|
|\
| |
| |
| |
| |
| |
| | |
Enable Knative installation on group and instance level clusters
Closes #62667 and #62666
See merge request gitlab-org/gitlab-ce!32128
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Show Knative install button on group/instance cluster pages
- Allow Knative to be installed on group/instance clusters
- Add feature specs for installing applications on group/instance
clusters
- Add changelog entry
- Update docs to reflect that Knative can now be installed on
group-level and instance-level clusters
|
|\ \
| |/
|/|
| |
| | |
Port CreateGithubPullRequestEvents migration from EE
See merge request gitlab-org/gitlab-ce!31802
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Detect if pipeline runs for a GitHub pull request
When using a mirror for CI/CD only we register a pull_request
webhook. When a pull_request webhook is received, if the
source branch SHA matches the actual head of the branch in the
repository we create immediately a new pipeline for the
external pull request. Otherwise we store the
pull request info for when the push webhook is received.
When using "only/except: external_pull_requests" we can detect
if the pipeline has a open pull request on GitHub and create or
not the job based on that.
|
|\ \
| | |
| | |
| | |
| | | |
Group level JupyterHub
See merge request gitlab-org/gitlab-ce!32512
|
| | |
| | |
| | |
| | | |
Removes limitations on cluster types that can install JupyterHub
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Since it is not possible to dynamically detect if a job is automatically
cancellable or not, a this new attribute is necessary. Moreover, it let
the maintainer of the repo to adjust the behaviour of the auto cancellation
feature to match exactly what he needs.
|
| |/
|/|
| |
| |
| |
| |
| | |
These are the structural changes for supporting the EE feature of moving
"code_owner_approval_required" state from existing on a project to being
on the protected branches individually, allowing for CODEOWNER
validation on push events.
|
|\ \
| |/
|/|
| |
| | |
Adjust routable metric
See merge request gitlab-org/gitlab-ce!32591
|
| |
| |
| |
| |
| |
| | |
* Define the counter once.
* Make metric name follow Prometheus naming conventions.
* Add label for call method.
|
|/
|
|
|
|
|
|
|
|
|
| |
- Add new service that transfers milestones
from a group to a project
- Include new service in Projects transfer service
- Include FromUnion module in Milestone model
to use in transfer service
- Add specs for new milestones service
- Add specs for transferring milestones in
project transfer service
|
|
|
|
| |
Improve diff lines count efficiency
|
|
|
|
|
| |
Creates new event when an epic is created, closed, reopened or
commented.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes a high frequency N+1 issue:
`RoutableActions#find_routable!` is used across many controllers to
retrieve e.g. the Project or Namespace by path. The `#find_routable!`
method calls `#ensure_canonical_path` which in turn retrieves
`#full_path` from the given Routable.
This in turn triggers a lookup on `routes`, leading to a high frequency
of these queries:
```sql
SELECT "routes".* FROM "routes" WHERE "routes"."source_id" = $1 AND
"routes"."source_type" = $2 LIMIT $3
```
This is unnecessary as we already join `routes` in
`Routable#find_by_full_path` anyways.
|
|
|
|
|
|
| |
Modified schema via migrations.
Added one-to-one relationship between the two models.
Added changelog file
|
|\
| |
| |
| |
| | |
Expose `name` and `merge_request_event_type` in serialized json and predefined variables
See merge request gitlab-org/gitlab-ce!32323
|
| |
| |
| |
| |
| | |
This commit adds pipeline.type key to PipelineEntity.
This key will be used in MR widget in the next iteration.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Add system hooks for project/group membership updates
Closes gitlab-ee#12252
See merge request gitlab-org/gitlab-ce!32371
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When updating group and project members, new system hooks
`user_update_for_group` and `user_update_for_team` will be executed.
Fixes https://gitlab.com/gitlab-org/gitlab-ee/issues/12252
|
|\ \ \
| |_|/
|/| |
| | |
| | | |
Backport retries options for cancel_running
See merge request gitlab-org/gitlab-ce!32525
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
It performs an expensive operation, but isn't required in
most of tests
|
| | |
| | |
| | |
| | | |
This commit removes one RuboCop offense from its todo file.
|
|/ /
| |
| |
| |
| |
| |
| | |
Before this commit we redefined properties for supported events on each
new instance of ChatNotificationService during "runtime".
This commit defines the properties only once during "compile time".
|
| |
| |
| |
| | |
Wrong format on MS Teams integration push events with multi line commit messages
|
|\ \
| | |
| | |
| | |
| | | |
Avoid calling freeze on already frozen strings in app/models
See merge request gitlab-org/gitlab-ce!32499
|
| | | |
|
|\ \ \
| |/ /
|/| |
| | |
| | | |
Improve chatops help output
See merge request gitlab-org/gitlab-ce!32208
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This improves the output produced when running an unknown command,
running the "help" command, and when trying to run a command you are not
allowed to run. The new help output includes links to the project of the
chatops integration, and a link to the chatops documentation.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Remove dependency on IgnorableColumn concern
Closes #66746
See merge request gitlab-org/gitlab-ce!32427
|
| | | |
| | | |
| | | |
| | | | |
This is to accomodate prepended modules.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This change limits the number of emails
for new access requests notifications to
10 most recently active owners/maintainers
|
| | | |
| | | |
| | | |
| | | | |
This does not support upgrading from earlier versions
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Perform two-step Routable lookup by path
See merge request gitlab-org/gitlab-ce!32272
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We should see the ratio drop down when enabling the Feature.
Recommendation by @andrewn
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In order to lookup a Project or Namespace by path, we prefer an exact
match (case-sensitive) but in absence of that, we'd also take a
case-insensitive match.
The case-insensitive matching with preference for the exact match is a
bit more involved in SQL as the exact lookup. Yet, the majority of cases
will be an exact match. The thinking here is that we can optimize the
lookup by performing an exact match first and only if there is no
result, we perform the case-insensitive lookup.
Data for GitLab.com:
* We have about 15M records in routes table
* About 2,500 routes exist where there's more than one record
with the same `lower(path)`
It is possible for a user to craft requests that would always trigger
the 2-step search (e.g. we have a route for `/foo/bar`, the request is
always for `/FOO/bar`). In this case, the change at hand is not
beneficial as it would run an additional query.
However, based on the data, it is highly likely that the vast majority
of requests can be satisfied with an exact match only.
The context for this change is
https://gitlab.com/gitlab-org/gitlab-ce/issues/64590#note_208156463.
|
|/ / / / |
|
|\ \ \ \ |
|
| |\ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Use image proxy to mitigate stealing ip addresses
Closes #2812
See merge request gitlab/gitlabhq!2926
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
User images and videos will get proxied through
the Camo server in order to keep malicious
sites from collecting the IP address of users.
|
| |\ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
into 'master'
Filter out old system notes for epics in notes api endpoint response
See merge request gitlab/gitlabhq!3224
|
| | | | | | | |
|
| | |/ / / / |
|
| |\ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
into 'master'
Fix HTML injection for label description
See merge request gitlab/gitlabhq!3250
|