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

Error if nicOrder status is None #117

Closed
Profecy opened this issue Nov 10, 2021 · 7 comments
Closed

Error if nicOrder status is None #117

Profecy opened this issue Nov 10, 2021 · 7 comments
Milestone

Comments

@Profecy
Copy link

Profecy commented Nov 10, 2021

Hi, I am doing a dry run against both my companies vcenters. However, at some point the script fails with the following error:

Traceback (most recent call last):
  File "./netbox-sync.py", line 145, in <module>
    main()
  File "./netbox-sync.py", line 121, in main
    source.apply()
  File "/opt/netbox-sync/module/sources/vmware/connection.py", line 492, in apply
    view_details.get("view_handler")(obj)
  File "/opt/netbox-sync/module/sources/vmware/connection.py", line 1480, in add_host
    if nic_order.activeNic is not None:
AttributeError: 'NoneType' object has no attribute 'activeNic'

What can I do ? Let me know if you need any more info.

@bb-Ricardo
Copy link
Owner

Hi,

which version of ESXi and vCenter are you running?

@bb-Ricardo bb-Ricardo changed the title Python Error during dry run Error if nicOrder status is None Nov 10, 2021
@bb-Ricardo
Copy link
Owner

Hey, can you try to checkout "development" branch and see if it works now?

Thant you.

@Profecy
Copy link
Author

Profecy commented Nov 11, 2021

We use vCenter 7.0.3, 18778458 and ESXi 7.0.3, 18644231

I will try the development build as well.
Thank you for the quick response!

@Profecy
Copy link
Author

Profecy commented Nov 11, 2021

I have tried the development branch and it worked perfectly.
Thank you!

@bb-Ricardo bb-Ricardo added this to the 1.2.2 milestone Nov 11, 2021
@bb-Ricardo
Copy link
Owner

great, then I will close this issue. Keep an eye out for new releases. So you can switch back to main once this fix is merged.

@bb-Ricardo
Copy link
Owner

If you like the project I would be happy about a ⭐

@Profecy
Copy link
Author

Profecy commented Nov 11, 2021

I will gladly ⭐ the project, it saves so much work!

dupondje pushed a commit to dupondje/netbox-sync that referenced this issue May 5, 2022
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

No branches or pull requests

2 participants