Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Fix missing redis-commaner password support #6131

Merged
merged 46 commits into from
Aug 12, 2024

Conversation

mrnicegyu11
Copy link
Member

What do these changes do?

When the password authentication for redis was added, I forgot to add the password to the redis-commander. This PR adds it.

Related issue/s

How to test

Dev-ops checklist

mrnicegyu11 and others added 30 commits April 22, 2024 11:43
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Merged on behalf of @bisgaard-itis . Needed for staging release
@mrnicegyu11 mrnicegyu11 added the a:redis redis service label Aug 2, 2024
@mrnicegyu11 mrnicegyu11 added this to the Tom Bombadil milestone Aug 2, 2024
@mrnicegyu11 mrnicegyu11 self-assigned this Aug 2, 2024
@mrnicegyu11 mrnicegyu11 marked this pull request as ready for review August 2, 2024 11:19
Copy link

codecov bot commented Aug 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.9%. Comparing base (cafbf96) to head (b3162b7).
Report is 419 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #6131      +/-   ##
=========================================
+ Coverage    84.5%   87.9%    +3.4%     
=========================================
  Files          10    1401    +1391     
  Lines         214   58790   +58576     
  Branches       25    1480    +1455     
=========================================
+ Hits          181   51733   +51552     
- Misses         23    6746    +6723     
- Partials       10     311     +301     
Flag Coverage Δ
integrationtests 64.8% <ø> (?)
unittests 85.8% <ø> (+1.3%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1357 files with indirect coverage changes

Copy link
Contributor

@GitHK GitHK left a comment

Choose a reason for hiding this comment

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

thanks

Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

Why is it once REDIS_PASSWORD and in the other TEST_REDIS_PASSWORD??
Also minor but your commits are huge for just 2 files

@mrnicegyu11
Copy link
Member Author

Why is it once REDIS_PASSWORD and in the other TEST_REDIS_PASSWORD?

The conftest.py of this test already had the common prefix TEST_ for the variables ;) So I am sticking with what was already there

Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

Thx.

TIP: Make sure you have a clean commit history in your PR. Note that you have 45 commits in the history for only two trivial file changes. :-)

Copy link

@pcrespov pcrespov requested a review from sanderegg August 12, 2024 09:19
@mrnicegyu11 mrnicegyu11 merged commit 684a11c into ITISFoundation:master Aug 12, 2024
56 checks passed
@mrnicegyu11 mrnicegyu11 deleted the 2024/07/redisPassword branch August 12, 2024 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:redis redis service bug buggy, it does not work as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants