summaryrefslogtreecommitdiff
path: root/lib/gitlab/ci
Commit message (Collapse)AuthorAgeFilesLines
* Improve naming convention in ci configuration modulefix/improve-naming-convention-in-ci-configGrzegorz Bizon2016-11-1429-104/+102
|
* Merge branch '22191-delete-dynamic-envs-mr' into 'master' Rémy Coutable2016-10-191-2/+16
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Delete dynamic environments - Adds "close environment" action to a merge request - Adds tabs to environments list - Adds close button to each environment row in environments list - Replaces Destroy button with Close button inside an environment - Adds close button to builds list inside an environment #### Configuration In order to enable stopping environments a valid `.gitlab-ci.yml` syntax has to be used: ``` review: environment: name: review/$app on_stop: stop_review stop_review: script: echo Delete My App when: manual environment: name: review/$app action: stop ``` This MR requires that `stop_review` has to have: `when`, `environment:name` and `environment:action` defined. The next MR after this one will verify that and enforce that these settings are configured. It will also implicitly configure these settings, making it possible to define it like this: ``` review: environment: name: review/$app on_stop: stop_review stop_review: script: echo Delete My App ``` Closes #22191 See merge request !6669
| * Add specs to test on_stop and action on environmentsKamil Trzcinski2016-10-171-0/+4
| |
| * Improve after code reviewKamil Trzcinski2016-10-171-2/+2
| |
| * Add `action` and `on_stop` to `environment` in .gitlab-ci.ymlKamil Trzcinski2016-10-171-3/+11
| |
| * Initial support for closing environmentsKamil Trzcinski2016-10-061-1/+3
| |
* | Merge remote-tracking branch 'upstream/master' into pipeline-emailsLin Jen-Shin2016-09-212-22/+87
|\ \ | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (206 commits) Implement fourth round of comments from @DouweM. Fix `CreateDeploymentService` spec. Reload issues in spec to ensure label<->issue mapping properly loaded Fix build. Remove unnecessary #{} in cycle analytics template. Update cycle analytics icon and fix color of the dismiss button. Use triple dashes for the empty value in cycle analytics. Fix typo on cycle analytics copy. Add page title and fix sub menu width in Cycle Analytics page. Update Cycle Analytics Read more link URL. Display the cycle analytics navbar based on the `:read_cycle_analytics` ability. Improve indentation in `Gitlab::Database::Median` Add a spec for merge request metric caching while refreshing a merge request from a forked project. Use the `IssuableBaseService` lifecycle hooks to cache `MergeRequestsClosingIssues` Implement a second round of review comments from @DouweM. Add docs on Cycle Analytics Test if issue authors can access private projects Update .pkgr.yml with Ubuntu 16.04 dependencies fix issues mr counter Move JSON generation (cycle analytics) into a controller method. ...
| * Fix scope of the CI config key nodes in jobs entryGrzegorz Bizon2016-09-191-13/+13
| |
| * Restore validation of CI job environment nameGrzegorz Bizon2016-09-191-0/+9
| |
| * Fix CI job environment configuration attributesGrzegorz Bizon2016-09-192-2/+3
| |
| * Fix CI job environment configuration entry classGrzegorz Bizon2016-09-191-8/+11
| |
| * Cleanup changesKamil Trzcinski2016-09-191-5/+2
| |
| * Update support for dynamic environmentsKamil Trzcinski2016-09-192-15/+11
| |
| * Add validation of URL and validation of nameKamil Trzcinski2016-09-191-0/+5
| |
| * Add new CI configuration entry for the environmentGrzegorz Bizon2016-09-192-9/+30
| |
| * Add support for dynamic environmentsKamil Trzcinski2016-09-191-0/+33
| | | | | | | | Environments that can have a URL with predefined CI variables.
* | Make various trace methods take last_lines argument:Lin Jen-Shin2016-09-211-0/+49
|/ | | | | So that we could read last few lines rather than read the entire file which could be huge.
* Merge branch 'smart-pipeline-duration' into 'master' Rémy Coutable2016-09-081-0/+141
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Smartly calculate real running time and pending time ## What does this MR do? Try to smartly calculate the running time and pending time for pipelines, instead of just use wall clock time from start to end. The algorithm is based on: > Suppose we have A, B, and C jobs: > * A: from 1 to 3 > * B: from 2 to 4 > * C: from 6 to 7 > The processing time should be accumulated from 1 to 4, and 6 to 7, totally 4, excluding retires, and calculate on `%w[success failed running canceled]` jobs (if a job is not finished yet, assume it's `Time.now`) ## Are there points in the code the reviewer needs to double check? I would actually like to test `Gitlab::Ci::PipelineDuration#process_segments`, but it's a private method right now and it's not very convenient to test it. Is there a way to test it without changing the original code too much? Note that I would like to avoid saving merged segments because it's not used and should be garbage collected. ## Screenshots: ![Screen_Shot_2016-09-05_at_6.45.32_PM](/uploads/a82bfaf316661091e383b743a2f11334/Screen_Shot_2016-09-05_at_6.45.32_PM.png) ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - Tests - [x] Added for this feature/bug ## What are the relevant issue numbers? Closes #18260, #19804 See merge request !6084
| * Struct.new could take a block for defining methods, feedback:smart-pipeline-durationLin Jen-Shin2016-09-081-2/+1
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_15091858
| * Split try_merge_period into overlap? and merge:Lin Jen-Shin2016-09-081-7/+9
| | | | | | | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_15083507
| * Be more specific since it's not needed to be generic now,Lin Jen-Shin2016-09-061-3/+5
| | | | | | | | | | feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14995016
| * Sort by database, feedback:Lin Jen-Shin2016-09-061-2/+4
| | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14991226 and https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14994233
| * Just use module because there's nothing to save, feedback:Lin Jen-Shin2016-09-061-14/+10
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14992064
| * Use sum, feedback:Lin Jen-Shin2016-09-061-3/+1
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14991392
| * Forget about pending duration for now, need more discussionLin Jen-Shin2016-09-051-39/+10
| |
| * Just sum all the queuing time, indication for needing more runnersLin Jen-Shin2016-09-021-6/+7
| |
| * Actually we still need to use total - running to get:Lin Jen-Shin2016-09-021-3/+4
| | | | | | | | | | | | real pending time, because that's actually by definition, (the time that it's not running!) or we could end up with awfully complicated algorithm :(
| * Merge remote-tracking branch 'upstream/master' into smart-pipeline-durationLin Jen-Shin2016-09-022-3/+2
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * upstream/master: (289 commits) Fix a typo Change minimum Unicorns required to two Update memory requirements Change the inline code to codeblocks for the new features doc guideline Update CHANGELOG with 8.11.4 entries. removed null return - renamed 'placeTop' to 'placeProfileAvatarsToTop' Change widths of content in MR pipeline tab Add curve to generic commit status pipeline Rubocop syntax 2.3 Some minor updates for upgrade guides for 8.12. Remove inconsistent font weight for sidebar's labels Replace play icon font with svg Project tools visibility level Added todo filter tests Fixed project filtering Review changes, simplified dropdown init Removed select2 from todos feature spec Removed inline JS and improved dropdown labels Added type and action dropdowns, need to finalize by removing all inline and polishing off the selected dropdown states Completed project filter dropdown, still need to move it from inline to ProjectSelect.js (or different) ...
| * | Calculate real queueing time to exclude gaps from:Lin Jen-Shin2016-09-021-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | retries and probably also manual actions! Feedback: * https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14735478 * https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14737804
| * | Use guard clause, feedback:Lin Jen-Shin2016-08-311-11/+9
| | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6084#note_14754681
| * | Introduction to PipelineDurationLin Jen-Shin2016-08-311-0/+98
| | |
| * | Fix renamingLin Jen-Shin2016-08-301-1/+1
| | |
| * | Avoid shadowing method name. Just use existing methodLin Jen-Shin2016-08-301-2/+2
| | |
| * | Rename to periods since it's easier to understandLin Jen-Shin2016-08-301-22/+22
| | |
| * | Use algorithm from Kamil:Lin Jen-Shin2016-08-301-28/+9
| | | | | | | | | | | | | | | Excluding sorting, this is O(n) which should be much faster and much simpler and easier to understand.
| * | build might not start yetLin Jen-Shin2016-08-301-1/+1
| | |
| * | no builds no pendingLin Jen-Shin2016-08-291-0/+2
| | |
| * | Smartly calculate real running time and pending timeLin Jen-Shin2016-08-291-0/+90
| | |
* | | Merge branch 'master' into refactor/ci-config-add-logical-validationrefactor/ci-config-add-logical-validationGrzegorz Bizon2016-09-032-3/+2
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (414 commits) Remove suggested colors hover underline Fix markdown anchor icon interaction Fix expiration date picker after update Refactored code to rely less on IDs that could change Move CHANGELOG entry for !5858 from 8.11 to 8.12 Hides merge request section in edit project when disabled Fix a typo Change minimum Unicorns required to two Update memory requirements Added `.term-bold` declaration. Change the inline code to codeblocks for the new features doc guideline Fix GitLab import button Rename behaviour to behavior in bug issue template for consistency Convert datetime coffeescript spec to ES6 Align add button on repository view Update CHANGELOG with 8.11.4 entries. removed null return - renamed 'placeTop' to 'placeProfileAvatarsToTop' Refactor Ci::Build#raw_trace Move CHANGELOG entry to a proper version Change widths of content in MR pipeline tab ... Conflicts: lib/gitlab/ci/config/node/jobs.rb
| * | Rename CI config hidden job entry to hidden entryGrzegorz Bizon2016-08-292-2/+2
| | |
| * | Do not enforce using a hash with hidden ci keyGrzegorz Bizon2016-08-291-1/+0
| |/
* | Add method that returns commands for ci job entryGrzegorz Bizon2016-08-291-1/+6
| |
* | Make it possible to inherit global ci config nodesGrzegorz Bizon2016-08-291-0/+15
| |
* | Add [] method for accessing ci entry dependenciesGrzegorz Bizon2016-08-251-0/+4
| |
* | Remove temporary stub method from ci config nodesGrzegorz Bizon2016-08-253-8/+2
| |
* | Expose compose method in the ci config entry nodesGrzegorz Bizon2016-08-256-44/+47
| |
* | Rename unspecified and undefined CI config nodesGrzegorz Bizon2016-08-254-42/+46
| |
* | Inject dependencies into each CI config entry nodeGrzegorz Bizon2016-08-255-8/+15
|/
* Memoize CI config node validator to prevent leaksfix/improve-ci-node-validatable-to-prevent-memory-leakGrzegorz Bizon2016-08-091-6/+4
|
* Merge branch 'rubocop/enable-access-modifiers-cops' into 'master' Robert Speicher2016-07-291-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable Rubocop cops that check access modifiers ## What does this MR do? This MR enables Rubocop cops that detect methods that should be restricted but are the part of public API because of access modifiers used improperly. This also fixes existing offenses. ## Why was this MR needed? Some method in our codebase are public instead of being private because it is sometimes difficult to get it right without static analysis. ## What are the relevant issue numbers? See #17478 Closes #17372 See merge request !5014
View Lorry Controller Status