Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Allow `rake cache:clear` clearing pipeline status cache44582-clear-pipeline-status-cache | Lin Jen-Shin | 2018-04-10 | 1 | -1/+1 |
| | | | | | | | | | * Use the correct key prefix * Clear old cache keys TODO: At some point we could remove clearing old cache keys. | ||||
* | Use StrongMemoize and enable/disable cops properly | Lin Jen-Shin | 2017-11-18 | 1 | -3/+4 |
| | |||||
* | Add cop to make sure we don't use ivar in a module | Lin Jen-Shin | 2017-09-18 | 1 | -0/+1 |
| | |||||
* | Whitelist or fix additional `Gitlab/PublicSend` cop violationsrs-more-public-send-whitelists | Robert Speicher | 2017-08-14 | 1 | -1/+1 |
| | | | | | An upcoming update to rubocop-gitlab-security added additional violations. | ||||
* | Rename the methods to make it fit with current name | Lin Jen-Shin | 2017-07-18 | 1 | -9/+9 |
| | |||||
* | Follow feedback on the merge request | Lin Jen-Shin | 2017-07-18 | 1 | -14/+10 |
| | |||||
* | Update the comments for the new functionality | Lin Jen-Shin | 2017-07-18 | 1 | -3/+19 |
| | |||||
* | Use RequestStoreWrap for Commit#author | Lin Jen-Shin | 2017-07-18 | 1 | -12/+34 |
| | | | | | | | We also try to use instance variable to cache the result if RequestStore is not available, so we could keep the same logic, using the same cache key. Also introduce a way to specify method specific cache key | ||||
* | Add RequestStoreWrap to cache via RequestStore | Lin Jen-Shin | 2017-07-17 | 1 | -0/+60 |
| | | | | | | | | | | | | | | | | | I don't like the idea of `RequestStore` at all, because it's just a global state which shouldn't be used at all. But we have a number of places calling `ProtectedBranch.protected?` and `ProtectedTag.protected?` in a loop for the same user, project, and ref whenever we're checking against if the jobs for a given pipeline is accessible for a given user. This means we're effectively making N queries for the same thing over and over. To properly fix this, we need to change how we check the permission, and that could be a huge work. To solve this quickly, adding a cache layer for the given request would be quite simple to do. We're already doing this in Commit#author, and this is extending that idea and make it generalized. | ||||
* | Support multiple Redis instances based on queue type | Paul Charlton | 2017-07-11 | 1 | -5/+5 |
| | |||||
* | Enable Style/DotPosition Rubocop :cop: | Grzegorz Bizon | 2017-06-21 | 1 | -2/+2 |
| | |||||
* | Load a project's CI status in batch from redis | Bob Van Landuyt | 2017-04-26 | 1 | -9/+44 |
| | |||||
* | Take the ref of a pipeline into account when caching status | Bob Van Landuyt | 2017-04-07 | 1 | -9/+18 |
| | |||||
* | Rename `ProjectBuildStatus` -> `ProjectPipelineStatus` | Bob Van Landuyt | 2017-04-07 | 1 | -1/+1 |
| | |||||
* | Wrap updating of cache after pipeline transition in class method | Bob Van Landuyt | 2017-04-07 | 1 | -0/+4 |
| | |||||
* | Rename Ci::PipelineStatus -> Ci::ProjectBuildStatus | Bob Van Landuyt | 2017-04-07 | 1 | -0/+90 |