| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |\
| |
| |
| |
| | |
Fix unrelated errors when teardowning quanrantined specs
See merge request gitlab-org/gitlab-ce!28240
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since we're skipping the example in a before hook, the @_subscribers
instance variable that is normally set in the setup phase of the test by
https://github.com/rails/rails-controller-testing/blob/21014e48be124dd8d0af250b79f682519d445015/lib/rails/controller/testing/template_assertions.rb#L21
isn't set but the teardown phase of the example still happens and tries
to iterate over the array of subcribers: https://github.com/rails/rails-controller-testing/blob/21014e48be124dd8d0af250b79f682519d445015/lib/rails/controller/testing/template_assertions.rb#L54
Using an around hook allows the @_subscribers instance variable to be
set.
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Properly handle LFS Batch API response in project import
Closes #61624
See merge request gitlab-org/gitlab-ce!28223
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Project imports were failing with `undefined method each_with_object for
String` because the import was attempting to parse the LFS Batch API and
failing due to the fact that the Content-Type wasn't a supported format
(e.g. application/vnd.git-lfs+json instead of application/json). We now
parse the body as JSON.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61624
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Simplify frontend test fixtures
See merge request gitlab-org/gitlab-ce!27531
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
for f in $(find spec/javascripts/fixtures/ -name '*.rb')
do
sed -E -e 's/(, ){0,1}(fixture_file_name|
example\.description)(, ){0,1}//' < $f > $f.tmp; mv $f.tmp $f
done
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
for f in $(find spec/javascripts/fixtures/ -name '*.rb')
do
sed 's/do |example|/do/' < $f > $f.tmp; mv $f.tmp $f
done
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
for f in $(find spec/javascripts/fixtures/ -name '*.rb')
do
grep -v store_frontend_fixture < $f > $f.tmp; mv $f.tmp $f
done
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Pull mirrors would run the `Projects::AfterImportService`, which would
force a `git gc` each time it finished. This is overkill and not
necessary now that we have refs packed more frequently
(https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/27826).
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/11556
|
| | |/
|/|
| |
| |
| |
| |
| | |
A project admin attempting to invite an already-invited user
will see a confusing Error 500 message.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61574
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Change DetectRepositoryLanguagesWorker to not receive user
Closes #60425
See merge request gitlab-org/gitlab-ce!28091
|
| | | |
| | |
| | |
| | | |
Fixes #60425
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
'master'
Resolve "Flaky spec: "Dashboard > User filters projects with search bar Sorting Sorting by Stars sorts the project list" / spec/features/dashboard/user_filters_projects_spec.rb:257"
Closes #61635
See merge request gitlab-org/gitlab-ce!28224
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| | |/ /
|/| | |
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This helper is right on the limit of the number of calls being made.
When the request store is enabled, and Sidekiq is running in inline
mode, just a couple of additional actions result in it being pushed
over the line.
The operation is entirely artificial, since in reality we'd perform
these requests in multiple separate processes, so just whitelist it
for now.
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
This will avoid timestamp comparison issues later within equality check
with attributes
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
58404 - setup max depth for graphql
Closes #58404
See merge request gitlab-org/gitlab-ce!25737
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
58404 - add change log
58404 - add spec
58404 - add more spec to test depth 2
58404 - fix spec
58404 - fix rubocop
58404 - refactor the code by Bob's advice
58404 - revert changes of all_graphql_fields_for
58404 - change text only
58404 - fix rspec according to gitlab's standard
58404 - revert previous spec
58404 - fix rubocop
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Adds header column to variables list
Closes #46806
See merge request gitlab-org/gitlab-ce!28060
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
For the variables list in CI/CD settings
this commits adds an header row with titles
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Handle errors in successful notes reply
Closes #61377
See merge request gitlab-org/gitlab-ce!28082
|
| | | | | | |
|
| | | |_|/
| |/| | |
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
Udpated user_edit_profile_spec with time preferences
Minor update form fields
|
| | | |
| | |
| | |
| | | |
Use existing form to allow users to add custom metrics via the dashboard
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Resolve "Add new Unicorn metrics"
Closes #56850
See merge request gitlab-org/gitlab-ce!27474
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This updates monitor docs to reflect the new ruby and unicorn metrics as
well as making it so we fetch process start time via the proc table
instead of via CLOCK_BOOTTIME
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
These metrics are not unicorn specific and can be used across ruby
processes
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This adds new metrics for monitoring unicorn. These metrics include
process_cpu_seconds_total, process_start_time_seconds, process_max_fds,
and unicorn_workers.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Pipelines are created by an async worker, so a rapid sequence of API
calls can trigger a state where the pipeline, whose existence is part of
determining if we wait for the pipeline to successfully complete before
merging, can trigger the MR to be immediately merged instead.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/55127
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fixes empty block in MR widget
Closes #60462
See merge request gitlab-org/gitlab-ce!27462
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This commit prevents the pipeline block to be rendered
when no pipeline information is provided
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
into 'master'
Resolve "Resolve discussion when suggestion is applied"
Closes #54405
See merge request gitlab-org/gitlab-ce!28160
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Adds color and a tooltip to describe this new behavior
- Does not resolve if discussion is already resolved
- Adds an action `resolveDiscussion` to simplify `toggleResolveNote`
- Updates docs
https://gitlab.com/gitlab-org/gitlab-ce/issues/54405
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Use a path for the related merge requests endpoint
Closes #61280
See merge request gitlab-org/gitlab-ce!28171
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Hosts using a non-standard configuration may have a different
hostname/port/scheme than what may be configured on the GitLab
server. While expose_url should generate a proper URL, there are cases
where it may not work. Since we don't need the full URL, we can use the
relative path.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61280
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Add settings for linking to external dashboard from
metrics dashboard
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Try to reduce testing viewport
Closes #58882
See merge request gitlab-org/gitlab-ce!27385
|
| | | | | | | | | |
|
