| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
(cherry picked from commit b87526e8c47b7a3938ab12842bbac99a17fcb750)
|
| |
|
|
| |
(cherry picked from commit a9454734d851ecc6d9e4a86cd667a33a492fdef9)
|
| |
|
|
| |
(cherry picked from commit 897a9d308db46b620b738b98f2b0e5630ac7d2dd)
|
| | |
|
| |\
| |
| |
| |
| | |
DB Load Balancing: Support SRV lookups
See merge request gitlab-org/gitlab-ce!32135
|
| | | |
|
| | |
| |
| |
| | |
- Also add helper to undo rename_column_concurrently.
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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 to the service and migration both.
|
| |\ \
| | |
| | |
| | |
| | | |
Resolve "Use Visual Review Tools NPM package"
See merge request gitlab-org/gitlab-ce!32159
|
| | | |
| | |
| | |
| | |
| | | |
Remove the visual review toolbar code
in favor of using the NPM package.
|
| | | |
| | |
| | |
| | |
| | | |
Allow users to read pipelines for public projects
with public builds enabled without providing an access token.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
- Introducting StageEvents to define the available events
- Define the event pairing rules, since some events are not compatible
- Express default Cycle Analytics stages with the event structure
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Adding gitaly feature flag for go implementation of get all lfs pointers
See merge request gitlab-org/gitlab-ce!31696
|
| | |/ / |
|
| | | |
| | |
| | |
| | | |
Add specs for new parameter and updated documentation as well.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Handle when server info doesn't have the storage in question
See merge request gitlab-org/gitlab-ce!32023
|
| | |/ / |
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
Eliminate Gitaly N+1 queries with notes API
See merge request gitlab-org/gitlab-ce!32089
|
| | |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Similar to https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/31834,
we see that in https://gitlab.com/gitlab-org/gitlab-ce/issues/65957
there can be hundreds, even thousands, of Gitaly requests in the
`/api/:version/projects/:id/merge_requests/:noteable_id/notes` endpoint.
Previously, the API to retrieve notes generated hundreds of Gitaly calls
to determine whether a system note should be shown to the user. It did
this by:
1. Rendering the Markdown
2. Extracting cross-references from the Markdown
3. Issuing a Gitaly `FindCommit` RPC for every reference to validate
that the commit exists.
The last step is unnecessary because we don't need to display a commit
if the user doesn't have access to the project in the first place.
`RendersNotes#prepare_notes_for_rendering` is already used in
`MergeRequestsController`, which is why we don't see N+1 Gitaly calls
there. We use it here to optimize the note redaction process.
|
| |\ \
| | |
| | |
| | |
| | | |
CE: Update sort options for issues list
See merge request gitlab-org/gitlab-ce!31849
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Increase sort options for issues list from updated_at and create_at,
to include more options close to what is required in actual issue list
UI.
This helps us to use REST API for issues list with sorting capabilities
https://gitlab.com/gitlab-org/gitlab-ce/issues/57402
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
Add frozen_string_literal to lib part 2
See merge request gitlab-org/gitlab-ce!32094
|
| | | | |
|
| | | | |
|
| | |/
|/| |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Root namespaces have storage statistics.
This commit allows namespace owners to get those stats via GraphQL
queries like the following one
{
namespace(fullPath: "a_namespace_path") {
rootStorageStatistics {
storageSize
repositorySize
lfsObjectsSize
buildArtifactsSize
packagesSize
wikiSize
}
}
}
|
| | | |
|
| |\ \
| |/
|/|
| |
| | |
feat: smime signed notification emails
See merge request gitlab-org/gitlab-ce!30644
|
| | |
| |
| |
| |
| |
| | |
- Add mail interceptor the signs outgoing email with SMIME
- Add lib and helpers to work with SMIME data
- New configuration params for setting up SMIME key and cert files
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
'ee-2502-refactor-ee-app-assets-javascripts-approvals-components-approvers_select-vue-to-remove-approverusers' into 'master'
Add a new method to `Api.js`: `projectUsers`
See merge request gitlab-org/gitlab-ce!31801
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This functionality is available in the /autocomplete users pseudo-API.
We're attempting to replace that with the canonical API, so it needs
support for this parameter too.
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Put cancelled job in DeadSet
See merge request gitlab-org/gitlab-ce!32070
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
This replicates Sidekiq behavior
of pushing dead job into DeadSet.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Upgrade to gitaly 1.60.0 and uncomment get_commit_signatures feature flag
Closes #65991
See merge request gitlab-org/gitlab-ce!31981
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Ensure the evaluation of right-hand side expression always
results in the returning of an object or an empty String
|
| |\ \ \ \ \
| | |/ / /
| |/| | |
| | | | |
| | | | | |
Allow to interrupt running sidekiq jobs
See merge request gitlab-org/gitlab-ce!31818
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Transform `CancelledError` into `JobRetry::Skip`
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
- Retry connection when it fails
- Properly shutdown daemon
- Stop monitor if the Exception is raised
- Properly guard exception handling
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
If we process message that is not designated to us
previously we would fire a separate Thread for that.
We don't need to do it. We can cheaply check if thread
is available, if it is, we can perform expensive operation
then.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This makes:
- very shallow `Middleware::Monitor` to only request tracking
of sidekiq jobs,
- `SidekiqStatus::Monitor` to be responsible to maintain persistent
connection to receive messages,
- `SidekiqStatus::Monitor` to always use structured logging
and instance variables
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This adds a middleware to track all threads
for running jobs.
This makes sidekiq to watch for redis-delivered notifications.
This makes be able to send notification to interrupt
running sidekiq jobs.
This does not take into account any native code,
as `Thread.raise` generates exception once the control gets
back to Ruby.
The separate measure should be taken to interrupt gRPC, shellouts,
or anything else that escapes Ruby.
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Adding, destroying and toggling emoji previously lacked services and
instead were performed through methods called on Awardable models.
This led to inconsistencies where relevant todos would be marked as done
only when emoji were awarded through our controllers, but not through
the API. Todos could also be marked as done when an emoji was being
removed.
Behaviour changes
- Awarding emoji through the API will now mark a relevant Todo as done
- Toggling an emoji off (destroying it) through our controllers will no
longer mark a relevant Todo as done
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/63372
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Added Gitlab::Ci::Config::Entry::Rules and Gitlab::Ci::Config::Entry::Rules:Rule
to handle lists of Rule objects to be evalauted for job inclusion
- Added `if:` and `changes:` as available Rules::Rule::Clause classes
- Added Rules handling logic to Seed::Build#included? with extra specs
- Use DisallowedKeysValidator to mutually exclude rules: from only:/except: on job config
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Current `auth.log` uses `fullpath` and `ip`, while `api_json.log` uses
`remote_ip` and `path` for the same fields. Let's standardize these
namings to make it easier for people working with the data.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/66167
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Count notes for commits and merge requests
See merge request gitlab-org/gitlab-ce!31912
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This extends our existing `Gitlab::UsageDataCounters::NoteCounter` to
also count notes on commits and merge requests
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Fix for https://gitlab.com/gitlab-org/gitlab-ce/issues/66319.
|
