> @@ -171,8 +172,8 @@ String buildBootstrapConfiguration(BootstrapConfig
> bootstrapConfig) {
> }
>
> @Override
> - public Statement createBootstrapScriptForGroup(String group) {
> - return groupToBootScript.apply(group);
> + public Statement createBootstrapScriptForGroup(String group, @Nullable
> String nodeName) {
> + return groupToBootScript.apply(group, null);
2 methods added to the implementing class, BaseChefService, as well as the
interface class, ChefService.
> On Dec 4, 2014, at 9:46 AM, Ignasi Barrera <[email protected]> wrote:
>
> In apis/chef/src/main/java/org/jclouds/chef/internal/BaseChefService.java:
>
> > @@ -171,8 +172,8 @@ String buildBootstrapConfiguration(BootstrapConfig
> > bootstrapConfig) {
> > }
> >
> > @Override
> > - public Statement createBootstrapScriptForGroup(String group) {
> > - return groupToBootScript.apply(group);
> > + public Statement createBootstrapScriptForGroup(String group, @Nullable
> > String nodeName) {
> > + return groupToBootScript.apply(group, null);
> No. I think I didn't explain properly. There must be two methods. The old and
> the new one so we don't break existing code.
>
> The previous comment pretended to say this, remarking the fact that ot was
> that way in some previous revision of this PR.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jclouds/jclouds/pull/622/files#r21321723>.
>
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/622/files#r21322472
