Skip to content

Fixup redis_backend_use_ssl docs#5387

Merged
auvipy merged 1 commit into
celery:masterfrom
huangsam:docs_redis_backend_use_ssl
Mar 17, 2019
Merged

Fixup redis_backend_use_ssl docs#5387
auvipy merged 1 commit into
celery:masterfrom
huangsam:docs_redis_backend_use_ssl

Conversation

@huangsam

Copy link
Copy Markdown
Contributor

Note: Before submitting this pull request, please review our contributing
guidelines
.

Description

Aimed to resolve #4812.

Here is the content that I wrote to help future developers with setting this configuration value for the Redis backend (not the Redis broker):

Screen Shot 2019-03-13 at 8 59 38 AM

Comment thread docs/userguide/configuration.rst
Comment thread docs/userguide/configuration.rst

@georgepsarakis georgepsarakis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Thank you.

Comment thread docs/userguide/configuration.rst
@codecov

codecov Bot commented Mar 13, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5387 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5387   +/-   ##
=======================================
  Coverage   83.39%   83.39%           
=======================================
  Files         144      144           
  Lines       16468    16468           
  Branches     2051     2051           
=======================================
  Hits        13733    13733           
  Misses       2526     2526           
  Partials      209      209

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95fd16f...ca5bbef. Read the comment docs.

@auvipy
auvipy merged commit 1423fab into celery:master Mar 17, 2019
@huangsam
huangsam deleted the docs_redis_backend_use_ssl branch March 17, 2019 15:59
@huangsam

Copy link
Copy Markdown
Contributor Author

Looks like this got merged before #5371 got resolved. @thedrow @georgepsarakis FYI

@thedrow

thedrow commented Mar 17, 2019

Copy link
Copy Markdown
Contributor

This is fine. The documentation you provided is still valid.

jeyrce pushed a commit to jeyrce/celery that referenced this pull request Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update documentation for Redis Broker SSL

4 participants