replace_config required about napalm HOT 6 CLOSED

Hi @t2d ,
the code says so because replace_config is not currently supported by the IOS driver :)

from napalm.

@t2d I am working on a new IOS driver that should have this (see #119 )

from napalm.

This is a documentation error in the Ansible module (napalm_install_config).

Doc string says:

    replace_config:
        description: If set to True the entire configuration on the device will be replaced during the commit. If
                     set to False, we will merge the new config with the existing one. Default: False.
        required: False

But code says:

def main():
    module = AnsibleModule(
        argument_spec=dict(
            hostname=dict(required=True),
            username=dict(required=True),
            password=dict(required=True),
            timeout=dict(required=False, default=60, type='int'),
            config_file=dict(required=True),
            dev_os=dict(required=True),
            commit_changes=dict(required=True),
            replace_config=dict(required=True),
            diff_file=dict(required=False, default=None),
            get_diffs=dict(required=False, choices=BOOLEANS, type='bool', default=True)

from napalm.

Yes, this problem will be fixed in #132

            replace_config=dict(required=False, choices=BOOLEANS, type='bool', default=False),

Once it's merged it will start behaving as described in the docstring.

from napalm.

Hey guys on a side note, I've been using the NAPALM ios integration with Ansible (before any changes from Kirk). And you may need to use get_diffs=false in the playbook. I originally added this for an older version for Arista, but needed it here so diffs aren't calculated b/c an error was being raised.

from napalm.

Closing this one as it should be fixed by now. Otherwise, feel free to reopen.

Thanks!

from napalm.