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

Add tags,device roles,tenants.. #110

Merged
merged 8 commits into from
Jul 1, 2020
Merged

Conversation

ThomasADavis
Copy link
Contributor

This adds support for:

  • setting the datacenter for vlans
  • setting the tenant for IP addresses and Devices.
  • setting device tags
  • setting the blade, chassis, and server roles.

Thomas Davis added 2 commits February 3, 2020 20:02
* sets the datacenter for vlans
* sets the tenant for IP addresses and Devices.
* setting device tags
* setting the blade, chassis, and server roles.
@Solvik
Copy link
Owner

Solvik commented Feb 17, 2020

I'm gonna write some tests since we're able to test the agent with a netbox instance, will merge this afterward!

@Solvik
Copy link
Owner

Solvik commented May 1, 2020

hi @ThomasADavis

I hope everything's ok for you and your family
I've made some modifications to your PR in order to merge it

I've fixed a typo, removed the datacenter on vlan (should be another feature in another branch) and backported your tag management to the VM features.

I've made some local tests, could you tell me if it's ok for you ? if so I'll merge this and release a new version

@Solvik Solvik mentioned this pull request Jul 1, 2020
@Solvik Solvik merged commit a60c0cd into Solvik:master Jul 1, 2020
devopstales pushed a commit to devopstales/netbox-agent2 that referenced this pull request Jun 24, 2022
* Adds support for:

* sets the tenant for IP addresses and Devices.
* setting device tags
* setting the blade, chassis, and server roles.

Co-authored-by: Thomas Davis <tdavis@nersc.gov>
Co-authored-by: Solvik <solvik@solvik.fr>
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

Successfully merging this pull request may close these issues.

2 participants