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

netbox module fails: The netbox execution module cannot be loaded #51795

Closed
misch42 opened this issue Feb 25, 2019 · 4 comments
Closed

netbox module fails: The netbox execution module cannot be loaded #51795

misch42 opened this issue Feb 25, 2019 · 4 comments
Labels
Bug broken, incorrect, or confusing behavior Confirmed Salt engineer has confirmed bug/feature - often including a MCVE fixed-pls-verify fix is linked, bug author to confirm fix P4 Priority 4 severity-low 4th level, cosemtic problems, work around exists
Milestone

Comments

@misch42
Copy link
Contributor

misch42 commented Feb 25, 2019

Description of Issue/Question

When I use the netbox module i.e netbox.openconfig_interfaces I get the error
The netbox execution module cannot be loaded:
pynetbox library is not installed.

It seems that the import of netbox does not work properly. When I comment the line about pynetbox.lib everything works. Please check if pynetbox.lib exists at all.

try:
import pynetbox
# from pynetbox.lib import RequestError
HAS_PYNETBOX = True
except ImportError:
HAS_PYNETBOX = False

Setup

(Please provide relevant configs and/or SLS files (Be sure to remove sensitive info).)

Steps to Reproduce Issue

(Include debug logs if possible and relevant.)

Versions Report

(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)

@dwoz
Copy link
Contributor

dwoz commented Feb 25, 2019

@misch42 Thanks for reporting this.

@Ichabond @zachmoody Can either of you look into this please?

@dwoz dwoz added Bug broken, incorrect, or confusing behavior P4 Priority 4 severity-low 4th level, cosemtic problems, work around exists labels Feb 25, 2019
@dwoz dwoz added this to the Approved milestone Feb 25, 2019
dwoz pushed a commit that referenced this issue Jan 6, 2020
dwoz added a commit that referenced this issue Jan 6, 2020
@stale
Copy link

stale bot commented Jan 9, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Jan 9, 2020
@waynew waynew added Confirmed Salt engineer has confirmed bug/feature - often including a MCVE fixed-pls-verify fix is linked, bug author to confirm fix labels Jan 13, 2020
@stale
Copy link

stale bot commented Jan 13, 2020

Thank you for updating this issue. It is no longer marked as stale.

1 similar comment
@stale
Copy link

stale bot commented Jan 13, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Confirmed Salt engineer has confirmed bug/feature - often including a MCVE fixed-pls-verify fix is linked, bug author to confirm fix P4 Priority 4 severity-low 4th level, cosemtic problems, work around exists
Projects
None yet
Development

No branches or pull requests

4 participants