| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
Instead of checking if a commit already exists in the upstream project
in its ProcessCommitWorker and bailing out if it does, we check the
existence of all commits in bulk in Git::BranchHooksService, so that we
can skip scheduling ProcessCommitWorker jobs for those commits
that already exist upstream entirely.
|
|
|
|
|
|
|
|
| |
It was originally returning 302 when the rate limit kicks in, because
using the the correct status code makes it easier to track rate limiting
events
Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/65974
|
|\
| |
| |
| |
| | |
Only expire branch cache once per push
See merge request gitlab-org/gitlab-ce!31653
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Whenever `PostReceive` is enqueued, `UpdateMergeRequestsWorker`
is enqueued and `MergeRequests::RefreshService` is called, it'll
check if the source branch of each MR asssociated to the push exists
or not via `MergeRequest#source_branch_exists?`. The said method will
call `Repository#branch_exists?` which is cached in `Rails.cache`.
When the cache contains outdated data and the source branch actually
exists, the `MergeRequests#RefreshService` job will close associated
MRs which is not correct.
The fix is to expire the branches cache of the project so we have
updated data during the post receive hook which will help in the
accuracy of the check if we need to close associated MRs or not.
|
|\ \
| | |
| | |
| | |
| | | |
Ensure changelog filenames have length less than 99 characters
See merge request gitlab-org/gitlab-ce!31752
|
| | |
| | |
| | |
| | | |
Signed-off-by: Balasankar "Balu" C <balasankar@gitlab.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Require `stage:` to be set with `needs:`
Closes #65568
See merge request gitlab-org/gitlab-ce!31760
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Since we are unsure what would be the behavior of `stage:`
when we work on DAG. Let's make `stage:` to be required
today with `needs:`.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fix a performance issue in Helm#can_uninstall?
Closes #65963
See merge request gitlab-org/gitlab-ce!31751
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | | |
Calling #present? was causing a DB query to happen each time around the
loop. We only wanted to check for nil as it's nil in the first loop
around so there is no need for #present?
|
|\ \ \ \
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | | |
Remove unused `BuildProcessWorker`
Closes #65539
See merge request gitlab-org/gitlab-ce!31756
|
| | | |
| | | |
| | | |
| | | |
| | | | |
We migrated all logic to `PipelineProcessWorker`
and this worker become redundant.
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
Improve cluster policy spec coverage
See merge request gitlab-org/gitlab-ce!31700
|
| | | | |
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Convert `spec/javascripts/environments/*rollback*` tests from Jasmine to Jest
Closes #61800
See merge request gitlab-org/gitlab-ce!31021
|
| | | |
| | | |
| | | |
| | | | |
This commit converts two Jasmine tests into Jest tests.
|
| | | |
| | | |
| | | |
| | | | |
Standardize punctuation and format
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Load search result counts asynchronously (CE)
See merge request gitlab-org/gitlab-ce!31663
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Creates `refresh_counts` module to dynamically fetch
and load data based on attributes of HAML elements.
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | | |
Querying all counts for the different search results in the same request
led to timeouts, so we now only calculate the count for the *current*
search results, and request the others in separate asynchronous calls.
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Properly save suggestions in project exports
Closes #65880
See merge request gitlab-org/gitlab-ce!31690
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Previously imports would fail if a merge request note included a
suggestion with an
`ActiveRecord::HasManyThroughCantAssociateThroughHasOneOrManyReflection`
exception.
This was happening because suggestions were listed as a descendant of
merge requests, but this doesn't work because suggestions are directly
associated with notes, not merge requests, and that association is lost.
Rails also disallows creating intializing a has_many association through
a different object.
We fix this by making `suggestions` a child of `notes` within a merge
request. This doesn't fix previously broken exported project exports,
but new exports will work.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65880
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Adds highlight to collapsible line in job log
Closes #65790
See merge request gitlab-org/gitlab-ce!31665
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
In the job log adds a highlight when hovering
the collapsible line
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Update GraphicsMagick from 1.3.29 to 1.3.33
See merge request gitlab-org/gitlab-ce!31692
|
| | |/ / /
| |/| | |
| | | | |
| | | | | |
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fix active metric files being wiped after the app starts
See merge request gitlab-org/gitlab-ce!31668
|
| | |/ / /
| |/| | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add author lines to project import comments
See merge request gitlab-org/gitlab-ce!31579
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Replace `username` field with `nickname` due to updates in
BitBucket Cloud API
- Add new imported spec validating addition of `authod_line`
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
into 'master'
Add BitBucketServerImport project filtering
See merge request gitlab-org/gitlab-ce!31420
|
| | | | | | | |
|
| | |/ / / /
| |/| | | | |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
'master'
Fix :wiki_can_not_be_created_total counter
See merge request gitlab-org/gitlab-ce!31673
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
'44036-someone-edited-the-issue-at-the-same-time-is-regularly-seen-despite-that-being-a-filthy-lie' into 'master'
Fix incorrectly displayed conflict warning
See merge request gitlab-org/gitlab-ce!31469
|
| | |_|_|/ /
| |/| | | |
| | | | | |
| | | | | | |
Hide warning when update request is in progress
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Make it easier to find invited group members
Closes #61948
See merge request gitlab-org/gitlab-ce!28436
|
| | | | | | | |
|
| | | | | | | |
|
| |\ \ \ \ \ \ |
|
| | | | | | | | |
|
| |\ \ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
- Add qa- selector to sort dropdown
|
| | | | | | | | | |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
We had a number of team members struggle to find invited members of a
group. Searching for the e-mail address did not work because search only
works with members with user accounts.
This commit changes two things:
1. Breaks out the invited members into a separate table.
2. Adds search capability for the invited members.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61948
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
[CE] FE fix diffs specs leaking store
See merge request gitlab-org/gitlab-ce!31698
|