delta/gitlab/gitlab-ce.git - gitlab.com: gitlab-org/gitlab-ce.git

	Commit message (Collapse)	Author	Age	Files	Lines
*	Add feature spec for search with spacefe-fix-merge-url-params-with-plus	minghuan lei	2019-08-19	1	-5/+25
\|
*	Fix mergeUrlParams handling of `+`	Paul Slaughter	2019-08-19	1	-0/+6
\| \| \| \| \| \|	What was the issue? If a param value had `+`, it would be encoded as a literal `+` instead of a space.
*	Clean Sidekiq metrics from multiproc dir on start	Aleksei Lipniagov	2019-08-19	1	-0/+51
\| \| \| \| \| \| \|	After moving the multiproc dir cleanup into `config.ru`:`warmup`, we stopped cleaning Sidekiq metrics dir which is not correct. This MR intended to fix that. More details: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/31668
*	Add warning when LE certificate can't be obtained64630-add-warning-to-pages-domains-that-obtaining-deploying-ssl-certificates-through-let-s-encrypt-can-take-some-time	Vladimir Shushlin	2019-08-19	2	-4/+70
\| \| \| \|	* Refactor some tests as well
*	Move cycle analytics stages templates to vue	Ezekiel Kigbo	2019-08-19	1	-0/+177
\| \| \| \| \| \| \| \| \| \|	The existing stage list items are rendered in haml, migrating them to vuejs for future work. Fix alignment of median value Test for stage_nav_item.vue
*	Fix pipelines not always being created after a pushsh-fix-pipelines-not-being-created	Stan Hu	2019-08-17	1	-0/+16
\| \| \| \| \| \| \| \| \|	https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/31741 introduced a regression where not all the right parameters would be passed into `Ci::CreatePipelineService`. We fix this by breaking out the pipeline parameters and reusing a method from `Gitlab::DataBuilder::Push`. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/66196
*	UI for disabling group/project email notification	Brett Walker	2019-08-17	8	-0/+137
\| \| \| \| \| \|	- Adds UI to configure in group and project settings - Removes notification configuration for users when disabled at group or project level
*	Merge branch 'legacy-attachments-migrate-fix' into 'master'	Michael Kozono	2019-08-16	3	-4/+360
\|\ \| \| \| \| \| \| \| \|	Migrate legacy uploads rake tasks See merge request gitlab-org/gitlab-ce!29409
\| *	Add rake tasks for migrating leacy uploadslegacy-attachments-migrate-fix	Jarka Košanová	2019-08-16	3	-4/+360
\| \| \| \| \| \| \| \| \| \|	- move uploads created by AttachmentUploader - handle also files created for legacy_diff_notes
* \|	Merge branch '66023-starrers-count-do-not-match-after-searching' into 'master'	Stan Hu	2019-08-16	1	-40/+112
\|\ \ \| \| \| \| \| \| \| \| \| \| \| \|	Fix starrers counts after searching See merge request gitlab-org/gitlab-ce!31823
\| * \|	Fix starrers counters after searching	Douglas Barbosa Alexandre	2019-08-15	1	-40/+112
\| \| \|
* \| \|	Merge branch 'dm-process-commit-worker-n+1' into 'master'	Stan Hu	2019-08-16	2	-50/+60
\|\ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Look up upstream commits once before queuing ProcessCommitWorkers Closes #65464 See merge request gitlab-org/gitlab-ce!31871
\| * \| \|	Look up upstream commits once before queuing ProcessCommitWorkers	Douwe Maan	2019-08-16	2	-50/+60
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	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.
* \| \| \|	Expire project caches once per push instead of once per ref	Stan Hu	2019-08-16	5	-12/+144
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously `ProjectCacheWorker` would be scheduled once per ref, which would generate unnecessary I/O and load on Sidekiq, especially if many tags or branches were pushed at once. `ProjectCacheWorker` would expire three items: 1. Repository size: This only needs to be updated once per push. 2. Commit count: This only needs to be updated if the default branch is updated. 3. Project method caches: This only needs to be updated if the default branch changes, but only if certain files change (e.g. README, CHANGELOG, etc.). Because the third item requires looking at the actual changes in the commit deltas, we schedule one `ProjectCacheWorker` to handle the first two cases, and schedule a separate `ProjectCacheWorker` for the third case if it is needed. As a result, this brings down the number of `ProjectCacheWorker` jobs from N to 2. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/52046
* \| \| \|	Remove Security Dashboard feature flag	rossfuhrman	2019-08-16	1	-5/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This removes the group_overview_security_dashboard feature flag
* \| \| \|	Add clipboard button to metric chart dropdown	Tristan Read	2019-08-16	2	-2/+86
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Adds a clipboard button to the metrics dashboard, that allows copying a link to an individual chart.
* \| \| \|	Merge branch 'mc/feature/pipeline-tracking-config-ce' into 'master'	Kamil Trzciński	2019-08-16	5	-7/+40
\|\ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add `needs:` CI config option CE Closes gitlab-ee#12334 See merge request gitlab-org/gitlab-ce!31346
\| * \| \| \|	Port changes from EEmc/feature/pipeline-tracking-config-ce	Matija Čupić	2019-08-04	5	-7/+40
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Ports changes from https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/12343
* \| \| \| \|	Merge branch 'fix/commits-api-empty-refname' into 'master'	Nick Thomas	2019-08-16	1	-0/+6
\|\ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	fix handling of empty ref_name parameter string in commits api Closes #64745 See merge request gitlab-org/gitlab-ce!31687
\| * \| \| \| \|	remove encoding comment created by emacs ruby-mode	Armin Hohenegger	2019-08-13	1	-1/+0
\| \| \| \| \| \|
\| * \| \| \| \|	add a test for empty ref_name to api commits spec	Armin Hohenegger	2019-08-13	1	-0/+7
\| \| \| \| \| \|
* \| \| \| \| \|	Merge branch '56130-deployment-date' into 'master'	Grzegorz Bizon	2019-08-16	2	-0/+6
\|\ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Expose finished_at for the Deployment entity See merge request gitlab-org/gitlab-ce!31808
\| * \| \| \| \| \|	Add finished_at to the internal API Deployment entity56130-deployment-date	Krasimir Angelov	2019-08-15	2	-0/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/56130.
* \| \| \| \| \| \|	Merge branch ↵	Grzegorz Bizon	2019-08-16	5	-29/+256
\|\ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	'47003-user-onboarding-replace-current-email-confirmation-flow-with-a-soft-email-confirmation-flow' into 'master' Soft email confirmation flow Closes #47003 See merge request gitlab-org/gitlab-ce!31245
\| * \| \| \| \| \| \|	Fix invite spec	Alex Buijs	2019-08-08	1	-14/+58
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The invitation flow changed a bit, because now there is no more need to sign in after confirming. When signing up with a different email address, the user will see the invitation page straight after signing up when redirected, without needing to sign in.
\| * \| \| \| \| \| \|	Add some feature specs	Alex Buijs	2019-08-08	1	-0/+35
\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \|	Don't redirect to the Almost there page	Alex Buijs	2019-08-08	2	-15/+65
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Don't redirect to the Almost there page after registration and after resending confirmation instructions
\| * \| \| \| \| \| \|	Add ConfirmEmailWarning concern	Alex Buijs	2019-08-08	1	-0/+98
\| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \|	Merge branch 'dm-fix-quarantined-spec' into 'master'	Nick Thomas	2019-08-16	1	-5/+2
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix quarantined spec by waiting for requests Closes #65329 See merge request gitlab-org/gitlab-ce!31862
\| * \| \| \| \| \| \| \|	Fix quarantined spec by waiting for requestsdm-fix-quarantined-spec	Douwe Maan	2019-08-15	1	-5/+2
\| \| \|_\|_\|_\|/ / / \| \|/\| \| \| \| \| \|
* \| \| \| \| \| \| \|	Rewrite changed_file_icon_spec in Jest	Paul Slaughter	2019-08-16	2	-63/+123
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Uses vue-test-utils - More complete coverage with parameterized tests
* \| \| \| \| \| \| \|	Add feature specs for multiple group issue boards	Felipe Artur	2019-08-16	2	-0/+159
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add feature specs for multiple issue boards for groups
* \| \| \| \| \| \| \|	Merge branch 'fe-add-unbinds-to-discussion-keyboard-navigator' into 'master'	Kushal Pandya	2019-08-16	1	-0/+27
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add key unbinds to DiscussionKeyboardNavigator See merge request gitlab-org/gitlab-ce!31857
\| * \| \| \| \| \| \| \|	Add key unbinds to DiscussionKeyboardNavigatorfe-add-unbinds-to-discussion-keyboard-navigator	Paul Slaughter	2019-08-15	1	-0/+27
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Also adds comment to explain why this works on the diff-tab when it's only used in `notes-app`. https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/30144#note_204066538
* \| \| \| \| \| \| \| \|	Merge branch '24705-multi-selection-for-delete-on-registry-page' into 'master'	Thong Kuah	2019-08-15	4	-33/+196
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Resolve "Multi selection for delete on registry page" Closes #24705 See merge request gitlab-org/gitlab-ce!30837
\| * \| \| \| \| \| \| \| \|	Adjustments to event removal and modal description	Nick Kipling	2019-07-30	1	-0/+1
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Reworked how deletion works with multi vs single	Nick Kipling	2019-07-30	1	-4/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Single deletion no longer requires a prop Modal description is now generated on demand Added dedicated functions for deleting Updated tests to match new function naming Updated css class name to be more specific
\| * \| \| \| \| \| \| \| \|	Applying feedback changes	Nick Kipling	2019-07-30	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Updated table registry to remove singleItemToBeDeleted Renamed usages of idx to index Tidied and simplified css styling Added clarification comment to test regex Updated pot file
\| * \| \| \| \| \| \| \| \|	Updated select all to be more explicit	Nick Kipling	2019-07-30	1	-11/+8
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Updating with suggestions as per review	Nick Kipling	2019-07-30	1	-30/+53
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Updating table_registry tests	Nick Kipling	2019-07-30	1	-6/+12
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Support bulk registry tag delete	Giorgenes Gelatti	2019-07-30	1	-0/+33
\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \|	Implement multi select deletion for container registry	Nick Kipling	2019-07-30	3	-17/+119
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Added checkboxes to each image row Added delete selected images button Changed row delete button to appear on row hover Changed confirmation modal message Changed delete logic to support multi Added tests for multi select Updated pot file Updated rspec test for new functionality
* \| \| \| \| \| \| \| \| \|	Support query parameters in metrics embeds	Sarah Yasonik	2019-08-15	3	-10/+61
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	https://gitlab.com/gitlab-org/gitlab-ce/issues/62971 Adds support for embedding specific charts from the metrics dashboard. Expected parameters are dashboard, title, group, and y_label.
* \| \| \| \| \| \| \| \| \|	Squash project templates on update	Hordur Freyr Yngvason	2019-08-15	2	-0/+21
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	As per https://gitlab.com/gitlab-org/gitlab-ce/issues/46043, project templates should be squashed before updating, so that repositories created from these templates don't include the full history of the backing repository.
* \| \| \| \| \| \| \| \| \|	Merge branch 'sh-fix-discussions-spec' into 'master'	Mayra Cabrera	2019-08-15	1	-3/+5
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix failing N+1 spec in spec/requests/api/discussions_spec.rb See merge request gitlab-org/gitlab-ce!31865
\| * \| \| \| \| \| \| \| \| \|	Fix failing N+1 spec in spec/requests/api/discussions_spec.rb	Stan Hu	2019-08-15	1	-3/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This test was failing in EE because the API helper `get()` creates a personal access token each time it's run. We can avoid that by pre-creating a personal access token and passing it each time.
* \| \| \| \| \| \| \| \| \| \|	Migrations for Cycle Analytics backend	Adam Hegyi	2019-08-15	3	-0/+11
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This change lays the foundation for customizable cycle analytics stages. The main reason for the change is to extract the event definitions to separate objects (start_event, end_event) so that it could be easily customized later on.
* \| \| \| \| \| \| \| \| \| \|	Only read rebase status from the model	Nick Thomas	2019-08-15	2	-40/+1
\|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Prior to 12.1, rebase status was looked up directly from Gitaly. In https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14417 , a DB column was added to track the status instead. However, we couldn't stop looking at the gitaly status immediately, since some rebases may been running across the upgrade. Now that we're in 12.3, it is safe to remove the direct-to-gitaly lookup. This also happens to fix a 500 error that is seen when viewing an MR for a fork where the source project has been removed. We still look at the Gitaly status in the service, just in case Gitaly and Sidekiq get out of sync - I assume this is possible, and it's a relatively cheap check. Since we atomically check and set `merge_requests.rebase_jid`, we should never enqueue two `RebaseWorker` jobs in parallel.
* \| \| \| \| \| \| \| \| \|	Removes db/fixtures from database files	Mayra Cabrera	2019-08-15	1	-4/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This will avoid Danger to suggest a database review for files inside db/fixtures