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.

Sign up for GitHub

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

Jump to bottom

PR: add vm_role_relation #56

Closed
david-sieg opened this issue Mar 30, 2021 · 2 comments
Closed

PR: add vm_role_relation #56

david-sieg opened this issue Mar 30, 2021 · 2 comments
Labels
awaiting reply Awaiting reply from issue owner
Milestone
1.1.0

Comments

@david-sieg
Copy link

Hi Ricardo,

it would be nice to have an extra option to map vms to specific roles.
Same like #vm_tenant_relation = grafana.* = Infrastructure.

Should also be:

#Map vms to specified roles
#vm_role_relation = grafana.* = Grafana_role

Best regards,
David
bb-Ricardo added a commit that referenced this issue Apr 26, 2021
@bb-Ricardo
adds host_role_relation and vm_role_relation config options
3835313 
This commit deprecates following config options
* netbox_host_device_role
* netbox_vm_device_role

refs: #56
@bb-Ricardo
Copy link
Owner

Added this FR to the current "development" branch. Can you check it out and test it please?

@bb-Ricardo bb-Ricardo added the awaiting reply Awaiting reply from issue owner label Apr 26, 2021
@david-sieg
Copy link
Author

Hi Ricardo,

vm_role_relation runs fine, thx!

David

@bb-Ricardo bb-Ricardo added this to the 1.1.0 milestone Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
awaiting reply Awaiting reply from issue owner
Projects
None yet
Milestone
1.1.0
Development

No branches or pull requests
2 participants
@bb-Ricardo @david-sieg