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

do not fail on repeated enabling/disabling of sources -- v3 #93

Closed
wants to merge 4 commits into from

Conversation

satta
Copy link
Contributor

@satta satta commented Jan 19, 2019

Previous PR: #85

Makes sure to update parameters when re-enabling a source, printing a log message for each changed parameter.

See ticket 2728.

Describe changes:

  • Use descriptive variable name.
  • Restructure code flow to reduce nesting.

@satta satta requested review from jasonish and a team as code owners January 19, 2019 23:15
Copy link

@shivan1b shivan1b left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

Copy link
Member

@jasonish jasonish left a comment

Choose a reason for hiding this comment

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

Looks good. @shivan1b Can you merge this into master? We should probably get it into the next point release as it seems to be a running concern for a couple users now.

@shivan1b
Copy link

shivan1b commented Apr 3, 2019

@jasonish sure,

@shivan1b
Copy link

shivan1b commented Apr 3, 2019

Merged into master.

@shivan1b shivan1b closed this Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants