| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Add project path to sql query to build proper path
|
|
|
|
|
|
|
| |
API endpoints for requesting container repositories
and container repositories with their tag information
are enabled for users that want to specify the group
containing the repository rather than the specific project.
|
|\
| |
| |
| |
| | |
Only track Redis calls if Peek is enabled
See merge request gitlab-org/gitlab-ce!31438
|
| |
| |
| |
| |
| |
| |
| |
| | |
In dev environments, Sidekiq was encountering the message:
Circular dependency detected while autoloading constant Gitlab::Profiler
This saves some overhead during normal usage.
|
|\ \
| | |
| | |
| | |
| | | |
Add missing timezone when copying legacy artifacts (ci_builds)
See merge request gitlab-org/gitlab-ce!31447
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- ci_builds.artifacts_expire_at are copied to
ci_job_artifacts.expire_at with incorrect timestamps when the
database timezone is NOT utc
- ci_builds.artifacts_expire_at is `timestamp without time zone` and
ci_job_artifacts.expire_at is `timestamp with time zone` on
postgresql
- Tests fail locally for `rspec
./spec/lib/gitlab/import_export/import_export_spec.rb` without this
change
|
|\ \
| | |
| | |
| | |
| | | |
Add outbound setting for system hooks
See merge request gitlab-org/gitlab-ce!31177
|
| | |
| | |
| | |
| | |
| | | |
`allow_local_requests_for_hooks_and_services` was renamed to
`allow_local_requests_for_web_hooks_and_services`.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Simplify SystemHookUrlValidator to inherit from PublicUrlValidator
Refactor specs to move out shared examples to be used in both
system hooks and public url validators.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This MR adds new application setting to network section
`allow_local_requests_from_system_hooks`. Prior to this change
system hooks were allowed to do local network requests by default
and we are adding an ability for admins to control it.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Increase clair scanner from v11 to v12
See merge request gitlab-org/gitlab-ce!30809
|
| | | | |
|
| | |/
| |/| |
|
|/ / |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Also creates specs
Only allow Helm to be uninstalled if it's the only app
- Remove Tiller leftovers after reser command
- Fixes specs and offenses
Adds changelog file
Fix reset_command specs
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Make `needs:` to require a strong reference
Closes #65512
See merge request gitlab-org/gitlab-ce!31419
|
| |/
| |
| |
| |
| |
| |
| |
| |
| | |
This changes `needs:` from weak reference
to have a strong reference.
This means that job will not be created
unless all needs are present as part of
a pipeline.
|
|\ \
| |/
|/|
| |
| |
| |
| | |
'master'
Resolve "Breakage in displaying SVG in the same repository"
See merge request gitlab-org/gitlab-ce!31352
|
| |
| |
| |
| | |
Changes generated URL to raw instead of blob
|
| |
| |
| |
| |
| |
| |
| | |
Currently we write out empty CSS classes (`class=""`) every time we
create a new tag. This adds 9 unnecessary bytes per span element. In a
recent trace, I have counted 11950 span elements. So we transported 105
unnecessary kilobytes!
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Previously, both InfluxSampler and RubySampler were relying on the
`GC::Profiler.total_time` data which is the sum over the list
of captured GC events. Also, both samplers asynchronously called
`GC::Profiler.clear` which led to incorrect metric data because
each sampler has the wrong assumption it is the only object who calls
`GC::Profiler.clear` and thus could rely on the gathered results between
such calls.
We should ensure that `GC::Profiler.total_time` is called only in one
place making it possible to rely on accumulated data between such wipes.
Also, we need to track the amount of profiler reports we lost.
|
|/
|
|
| |
Changes * to + to be more precise with acceptable branch names
|
|\
| |
| |
| |
| |
| |
| | |
Use NotesFinder to fetch notes on API and Controllers
Closes #52123
See merge request gitlab-org/gitlab-ce!31300
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Remove project from NotesFinder constructor
Add project parameter to specs
Also look for methods in private scope
Fix specs to match new NotesFinder constructor
|
|\ \
| | |
| | |
| | |
| | | |
Add exclusive lease to mergeability check process
See merge request gitlab-org/gitlab-ce!31082
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Concurrent calls to UserMergeToRef RPC updating a single ref
can lead to an opaque fail that is being rescued at Gitaly.
So this commit adds an exclusive lease to the mergeability
check process with the key as the current MR ID.
|
| | |
| | |
| | |
| | |
| | |
| | | |
This implements the support for `needs:` keyword
as part of GitLab CI. That makes some of the jobs
to be run out of order.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Add additional paths to clean from backtrace
See merge request gitlab-org/gitlab-ce!31363
|
| | | |
| | | |
| | | |
| | | |
| | | | |
`Gitlab::Profiler` can also be used to profile Sidekiq jobs.
Add some call traces that can be omitted from the backtrace.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fix SystemStackError when Peek bar is active with Rugged calls
Closes #65404
See merge request gitlab-org/gitlab-ce!31357
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Peek attempts to serialize results with `to_json`, which calls
`ActiveSupport::JSON`. If an object is passed to `to_json` that contains
instance variables, `ActiveSupport` will attempt to recursively traverse
all variables.
The problem is that we can get into an infinite loop if the instance
references to an instance that references to something else that points
back to the same instance.
To avoid this mess, we just call `to_s` on the object. It appears only
`Gitlab::Git::Repository` and `::Repository` are the culprits here.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65404
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Add Feature.remove
Closes #65363
See merge request gitlab-org/gitlab-ce!31315
|
| | |/
| |/| |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Add project scope to live trace feature flag
See merge request gitlab-org/gitlab-ce!31325
|
| |/ /
| | |
| | |
| | | |
Add project scope
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix sidekiq memory killer warning message
Closes gitlab-com/gl-infra/infrastructure#7371
See merge request gitlab-org/gitlab-ce!31264
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
undefined in prior stages
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- After uninstalling the knative helm chart it's necessary to also
remove some leftover resources to allow the cluster to be clean
and knative to be reinstalleable.
- Adds knative uninstall disclaimer
- Uninstall ksvc before uninstalling knative
Make list of Knative and Ingres resources explicit
- To avoid deleting unwanted resources we are listing exact
which resources will be deleted rather than simply deleting any
resource that contains istio or knative words.
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Fix Danger finding a test reviewer
Closes #65317
See merge request gitlab-org/gitlab-ce!31292
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Since now the role name can be:
"<a href=\"/job-families/engineering/test-automation-engineer/\">Senior Test Automation Engineer</a>, Create:Source Code"
We need to cope with </a> in the middle.
|
| |_|/
|/| |
| | |
| | | |
Creates a new item in help dropdown to show configured support page link
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
Fix exception handling in Gitaly autodetection
Closes #65328
See merge request gitlab-org/gitlab-ce!31285
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
In SELinux, the file cannot be written, and `Errno::EACCES`, not
`Errno::ACCESS` is thrown.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65328
|
|/ /
| |
| |
| |
| |
| | |
This rake task had been broken for a while. This fixes the breakages,
adds a test to help avoid future breakages, and adds a few ergonomic
improvements to the task itself.
|
|/
|
|
|
|
|
|
|
| |
1. The output isn't great. It can be hard to find hotspots and, even
when you do find them, to find why those are hotspots.
2. It uses some jQuery-specific frontend code which we can remove now
that we don't have this any more.
3. It's only possible to profile the initial request, not any subsequent
AJAX requests.
|
|\
| |
| |
| |
| | |
Have deterministic DNS for specs
See merge request gitlab-org/gitlab-ce!31086
|
| |
| |
| |
| | |
Rather than relying on NoMethodError deep inside faraday
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
'63547-add-system-notes-for-when-a-zoom-call-was-added-removed-from-an-issue' into 'master'
Resolve "Add system notes for when a zoom call was added/removed from an issue"
Closes #63547
See merge request gitlab-org/gitlab-ce!30857
|