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

ansible.netcommon.restconf_get doesn't work with nxos #172

Closed
cidrblock opened this issue Oct 22, 2020 · 2 comments
Closed

ansible.netcommon.restconf_get doesn't work with nxos #172

cidrblock opened this issue Oct 22, 2020 · 2 comments
Labels
needs_info This issue requires further information. Please answer any outstanding questions.

Comments

@cidrblock
Copy link
Collaborator

    if output == "xml":
        accept = "application/yang-data+xml"
    if output == "json":
        accept = "application/yang.data+json"

may be an issue in netcommon/module_utils/network/restconf/restconf.py because no accept is set.

putting this here to talk more about it

@Qalthos
Copy link
Contributor

Qalthos commented Jan 13, 2021

This definitely belongs in netcommon instead of nxos.

What is output if not one of those two?

@Qalthos Qalthos added the needs_info This issue requires further information. Please answer any outstanding questions. label Jan 13, 2021
@NilashishC
Copy link
Collaborator

NilashishC commented Jul 21, 2021

Even with output set to xml, this doesn't work. The data returned by send_request() is empty. Closing this in favour of ansible-collections/ansible.netcommon#306 since the fix will go in netcommon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs_info This issue requires further information. Please answer any outstanding questions.
Projects
None yet
Development

No branches or pull requests

3 participants