summaryrefslogtreecommitdiff
path: root/app/controllers/projects/artifacts_controller.rb
Commit message (Collapse)AuthorAgeFilesLines
* Extract ref_name and path in before_action, feedback:Lin Jen-Shin2016-08-261-5/+10
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14469768
* Just use instance variable instead, feedback:Lin Jen-Shin2016-08-251-7/+2
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14400736
* Path could also have slashes! Feedback:Lin Jen-Shin2016-08-241-3/+11
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14347729
* Not sure why I missed this renamingLin Jen-Shin2016-08-181-1/+1
|
* Use path rather than URL because it should work for http 302:Lin Jen-Shin2016-08-181-2/+2
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14035941
* Use switch case in a helper, feedback:Lin Jen-Shin2016-08-171-7/+3
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13988401
* Rename to latest_succeeded, feedback:Lin Jen-Shin2016-08-161-1/+1
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13908017
* Use URL helper, feedback:Lin Jen-Shin2016-08-161-3/+7
| | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13880889
* Rename latest_success* to latest_successful:Lin Jen-Shin2016-07-191-1/+1
| | | | | Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13164642
* Merge branch 'master' into artifacts-from-ref-and-build-nameLin Jen-Shin2016-07-181-3/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (335 commits) Disable transaction when adding index for Ci::Pipeline added changelog allow empty repos on import/export Modify test for Build tabs Add tests to project builds for pending tab Remove unused .js-running-count class Add test for new pending tab and update tests for running tab Add Pending Tab to Admin Builds added changelog limit project expor retry to only 3 use method in validates statement Fix spec to set import_url before attempting to create import_data Allow a project import URL to be blank to prevent false positives preventing settings from being saved Refactor gitlab_ci_yaml_processor variables tests Fix CI yaml example Align cancel and retry buttons Remove deploy to production button Remove irrelevant comments Fix gitlab_ci_yaml_processor_spec.rb Fix AddWhenAndYamlVariablesToCiBuilds migration ...
| * Just give regular 404, feedback:Lin Jen-Shin2016-07-111-1/+1
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5094#note_12984211
| * Use Gitlab-Workhorse-Send-Data to send entry:Lin Jen-Shin2016-07-051-2/+1
| | | | | | | | | | | | | | Closes #19224, Closes #19128 Also requires this MR to work: https://gitlab.com/gitlab-org/gitlab-workhorse/merge_requests/53
* | Use find_by, feedback:Lin Jen-Shin2016-07-181-1/+1
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13125486
* | Introduce Project#latest_success_builds_for:Lin Jen-Shin2016-07-151-2/+2
| | | | | | | | | | So it's more accessible for views to access the names of jobs. Only filter Build#name from where we really need to download it.
* | Update routes based on feedback from:Lin Jen-Shin2016-07-141-1/+1
| | | | | | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13058785 And note that job/build_name could contain `/`
* | Rename to ref_name so it's aligning with APILin Jen-Shin2016-07-131-2/+2
| |
* | Could be faster when params[:path] is missingLin Jen-Shin2016-07-121-2/+2
| |
* | Add all the tests and fix stuffs along the way:Lin Jen-Shin2016-07-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | It turns out they are different: builds.success.latest.first and builds.latest.success.first If we put success first, that latest would also filter via success, and which is what we want here.
* | Remove redundant returnLin Jen-Shin2016-07-111-1/+1
| |
* | Using plain if/else is much easier to understandLin Jen-Shin2016-07-111-4/+4
| |
* | Give latest succeeded one, don't give pending/running onesLin Jen-Shin2016-07-111-1/+3
| |
* | Introduce Projects::ArtifactsController#search:Lin Jen-Shin2016-07-111-2/+22
|/ | | | | | | | | | | | So we redirect from ref and build_name to the particular build, namely: * /u/r/artifacts/ref/build_name/* -> /u/r/builds/:build_id/artifacts/* For: * download * browse * file
* Test controllers if they allow to keep artifactsKamil Trzcinski2016-06-131-0/+1
|
* Validate existence of artifacts in ArtifactsController, render 404 if not foundKamil Trzcinski2016-06-131-6/+5
|
* Move keep to ArtifactsControllerKamil Trzcinski2016-06-131-0/+5
|
* Remove 'unscoped' from project builds selectionTomasz Maczukin2016-06-041-1/+1
|
* Make the CI permission model simplerKamil Trzcinski2016-02-021-11/+1
| | | | | | | | | | | | | This MR simplifies CI permission model: - read_build: allows to read a list of builds, artifacts and trace - update_build: allows to cancel and retry builds - create_build: allows to create builds from gitlab-ci.yml (not yet implemented) - admin_build: allows to manage triggers, runners and variables - read_commit_status: allows to read a list of commit statuses (including the overall of builds) - create_commit_status: allows to create a new commit status using API Remove all extra methods to manage permission. Made all controllers to use explicitly the new permissions.
* Fix nonexistent method in artifacts controllerGrzegorz Bizon2016-01-151-1/+1
|
* Improve readability of artifacts browser `Entry` related codeGrzegorz Bizon2016-01-141-5/+5
|
* Improvements, readability for artifacts browserGrzegorz Bizon2016-01-141-7/+7
|
* Check if file exists in metadata in download actionGrzegorz Bizon2016-01-141-3/+8
|
* Render only valid paths in artifacts metadataGrzegorz Bizon2016-01-141-1/+4
| | | | | In this version we will support only relative paths in artifacts metadata. Support for absolute paths will be introduced later.
* Simplify implementation of build artifacts browser (refactoring)Grzegorz Bizon2016-01-141-1/+7
|
* Add method that checks if path exists in `StringPath`Grzegorz Bizon2016-01-141-1/+1
|
* Render 404 when artifacts path is invalidGrzegorz Bizon2016-01-141-0/+1
|
* Extract artifacts metadata implementation to separate classGrzegorz Bizon2016-01-141-4/+1
|
* Parse artifacts metadata stored in JSON formatGrzegorz Bizon2016-01-141-2/+2
|
* Use metadata stored in artifacats metadata fileGrzegorz Bizon2016-01-141-2/+2
|
* Add artifacts metadata uploader filedGrzegorz Bizon2016-01-141-1/+1
| | | | | Artifacts metadata field will be used to store a filename of gzipped file containing metadata definition for given artifacts archive.
* Update specs for CI Build, add `artifacts?` methodGrzegorz Bizon2016-01-141-0/+2
| | | | `artifacts?` method checks if artifacts archive is available.
* Remove artifacts metadata column from databaseGrzegorz Bizon2016-01-141-4/+1
|
* Add artifacts browserGrzegorz Bizon2016-01-141-4/+2
| | | | | This implementation makes it possible to browse artifacts, it depends on artifacts metadata.
* Use `Gitlab::StringPath` in CI build artifacts controllerGrzegorz Bizon2016-01-141-1/+8
|
* Move artifacts controller level upGrzegorz Bizon2016-01-141-0/+40
This reverts nesting artifacts controller in builds module.
View Lorry Controller Status