Skip to content

Implement dry run mode#10

Open
Domisys wants to merge 34 commits into
odolbeau:masterfrom
Domisys:implementDryRunMode
Open

Implement dry run mode#10
Domisys wants to merge 34 commits into
odolbeau:masterfrom
Domisys:implementDryRunMode

Conversation

@Domisys
Copy link
Copy Markdown
Contributor

@Domisys Domisys commented Jan 27, 2015

Dry run mode compare existing configuration and new one and output computation result with different verbosity modes.

Aymeric PLANCHE added 30 commits January 20, 2015 11:54
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Prefer the opposite to avoid useless imbrication:

if (!$currentParameters instanceof Response) {
    return;
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

of course, will be modified

@odolbeau
Copy link
Copy Markdown
Owner

I just tried this option on my laptop. :)
Works well, except for bindings. Seems that they are always considered as updated, event when they already exist:

Binding : sms:sms................................. [UPDATE]

Does Rabbit return the correct response code?

@Domisys
Copy link
Copy Markdown
Contributor Author

Domisys commented Feb 3, 2015

I don't have the same issue.
It happens for every bindings you have or a specific one ?
Could you send me an anonymised version of your yaml file ?

@Domisys Domisys closed this Feb 3, 2015
@Domisys Domisys reopened this Feb 3, 2015
@odolbeau
Copy link
Copy Markdown
Owner

odolbeau commented Feb 5, 2015

Forget it.
Just tried again and it works perfectly. :D

@odolbeau
Copy link
Copy Markdown
Owner

odolbeau commented Jun 1, 2015

Can you rebase your PR please? :)

@jeremyFreeAgent
Copy link
Copy Markdown

Any news about that PR? ping @Domisys

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants