- 12 Jan, 2016 5 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
- 09 Jan, 2016 4 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
- 08 Jan, 2016 4 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
Inline diff is going to be generated client side now. #3945
-
Rubén Dávila authored
Now we apply syntax highlighting to the whole old and new files. This basically help us to highlight adequately multiline content.
-
Rubén Dávila authored
-
- 07 Jan, 2016 3 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
- 31 Dec, 2015 5 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
-
- 30 Dec, 2015 3 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Rubén Dávila authored
* It is more performant given now we process all the diff file instead of processing line by line. * Multiline comments are highlighted correctly.
-
- 28 Dec, 2015 16 commits
-
-
Rubén Dávila authored
-
Rubén Dávila authored
-
Yorick Peterse authored
Handle missing settings table for metrics See merge request !2232
-
Achilleas Pipinellis authored
Fix broken link in permissions page See merge request !2233
-
Yorick Peterse authored
This ensures we can still boot, even when the "application_settings" table doesn't exist.
-
Achilleas Pipinellis authored
-
Yorick Peterse authored
Move InfluxDB settings to ApplicationSetting This moves the settings from the YAML files to the database. cc @sytses See merge request !2228
-
Yorick Peterse authored
Fixed syntax in gitlab.yml.example See merge request !2230
-
Yorick Peterse authored
-
Yorick Peterse authored
-
Dmitriy Zaporozhets authored
Restart settings are moved too. See merge request !2226
-
Sytse Sijbrandij authored
-
Sytse Sijbrandij authored
add issue weight to contributing cc @dzaporozhets See merge request !2223
-
Dmitriy Zaporozhets authored
Storing of application metrics in InfluxDB This adds support for tracking metrics in InfluxDB, which in turn can be visualized using Grafana. For more information see #2936. See merge request !2042
-
Job van der Voort authored
-
Dmitriy Zaporozhets authored
Environment variables in the app Fixes #3717 and #3519 ## Why environment variables? We need environmental variables, they are an expected way to configure apps https://medium.com/@kelseyhightower/12-fractured-apps-1080c73d481c#.ntrdiyu4c This causes many tools and to tutorials to make it easy to set environmental variables and harder to supply a configuration file. So even though we agree they are not ideal https://support.cloud.engineyard.com/hc/en-us/articles/205407508-Environment-Variables-and-Why-You-Shouldn-t-Use-Them the market has spoken. ## Why for GitLab the application and not for the Omnibus packages? Environmental variables are also needed by people that do not run our Omnibus packages, for example natively bundled apps (Debian apt-get) and idiomatic Docker packages (Mesos, Kubernetes, etc.). Of course it should work great with Omnibus packages too so any advise is welcome in that regard. There is an MR https://gitlab.com/gitlab-org/omnibus-gitlab/merge_requests/575/diffs to be able to set any variable in gitlab.rb via environmental variables. I think both that and this MR should be merged to solve the configuration problem for both Omnibus and non-Omnibus installations. When both are merged the documentation should be crosslinked. ## Why uppercase? Need to be all cap according to Google Shell guideline: "Constants and Environment Variable Names => All caps, separated with underscores, declared at the top of the file." https://google.github.io/styleguide/shell.xml#Constants_and_Environment_Variable_Names Or as explained on http://stackoverflow.com/a/673940/613240 Keeping to this convention, you can rest assured that you don't need to know every environment variable used by UNIX tools or shells in order to avoid overwriting them. If it's your variable, lowercase it. If you export it, uppercase it. /cc @JobV @DouweM @marin @jacobvosmaer @ayufan @pravi See merge request !2215
-