Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
slapos slapos
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Labels
    • Labels
  • Merge requests 122
    • Merge requests 122
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • slaposslapos
  • Merge requests
  • !26

Closed
Created Nov 03, 2015 by Kirill Smelkov@kirrOwner
  • Report abuse
Report abuse

redis: v↑ (2.8.23)

  • Overview 3
  • Commits 1
  • Changes
  • update Redis software to latest upstream in 2.8.* series (which now supports IPv6 out of the box);

  • update Redis instance template to the one from 2.8.23 and re-merge our templating changes to it (file/dir locations, port and binding, master password).

    The whole diff to pristine 2.8.23 redis conf is now this:

    diff --git a/.../redis-2.8.23/redis.conf b/slapos/recipe/redis/template/redis.conf.in
    index 870959f..2895539 100644
    --- a/.../redis-2.8.23/redis.conf
    +++ b/slapos/recipe/redis/template/redis.conf.in
    @@ -46 +46 @@ daemonize no
    -pidfile /var/run/redis.pid
    +pidfile %(pid_file)s
    @@ -50 +50 @@ pidfile /var/run/redis.pid
    -port 6379
    +port %(port)s
    @@ -69,0 +70 @@ tcp-backlog 511
    +bind %(ipv6)s
    @@ -108 +109 @@ loglevel notice
    -logfile ""
    +logfile %(log_file)s
    @@ -174 +175 @@ rdbcompression yes
    -# hit to pay (around 10%) when saving and loading RDB files, so you can disable it
    +# hit to pay (around 10%%) when saving and loading RDB files, so you can disable it
    @@ -192 +193 @@ dbfilename dump.rdb
    -dir ./
    +dir %(server_dir)s
    @@ -217 +218 @@ dir ./
    -# masterauth <master-password>
    +%(master_passwd)s

NOTE There are test failures for almost all Redis versions when machine have not small amount of CPUs:

https://github.com/antirez/redis/issues/2715#issuecomment-151608948

Because the failure is in replication test, and so far we do not use replication, and there is no feedback from upstream author to handle this (for 7 days for my detailed report, and for ~ 3 months for this issue in general), we can just disable replication test as a temporary solution. ( to handle remote patches with md5 hash easily the building recipe is changed to slapos.recipe.cmmi )

NOTE Redis updated to 2.8 version because GitLab uses this series. If/when we need more recent one we can add [redis30] in addition to [redis28].

Thanks beforehand for feedback,
Kirill

/cc @alain.takoudjou

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: redis-vup
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7