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

multiple proxmox clusters - support setting site_id on each cluster config #156

Open
ITJamie opened this issue Nov 9, 2023 · 2 comments
Assignees
Labels
accepted This issue has been accepted for implementation feature Introduction of new functionality to the application
Milestone

Comments

@ITJamie
Copy link

ITJamie commented Nov 9, 2023

the current example config sets site_id under the netbox section.
however multiple clusters likely exist across multiple sites.

i would suggest that an optional key should exist under the proxmox config blocks allowing overriding of the netbox site id

eg:


PLUGINS_CONFIG = {
    'netbox_proxbox': {
        'proxmox': [
            {
                'domain': 'proxbox.example.com',    # May also be IP address
                'http_port': 8006,
                'user': 'root@pam',   # always required
                'password': 'Strong@P4ssword', # only required, if you don't want to use token based authentication
                'token': {
                    'name': 'tokenID',	# Only type the token name and not the 'user@pam:tokenID' format
                    'value': '039az154-23b2-4be0-8d20-b66abc8c4686'
                },
                'ssl': False
            },
            # The following json is optional and applies only for multi-cluster use
            {
                'domain': 'proxbox2.example.com',    # May also be IP address
                'http_port': 8006,
                'user': 'root@pam',   # always required
                'password': 'Strong@P4ssword', # only required, if you don't want to use token based authentication
                'token': {
                    'name': 'tokenID',	# Only type the token name and not the 'user@pam:tokenID' format
                    'value': '039az154-23b2-4be0-8d20-b66abc8c4686'
                },
                'ssl': False,
                'netbox_site_id': 11, # site_id override
            }
        ],
        'netbox': {
            'domain': 'localhost',     # Ensure localhost is added to ALLOWED_HOSTS
            'http_port': 8001,     # Gunicorn port.
            'token': '0dd7cddfaee3b38bbffbd2937d44c4a03f9c9d38',
            'ssl': False,	# There is no support to SSL on Netbox yet, so let it always False.
            'settings': {
                'virtualmachine_role_id' : 0,
                'node_role_id' : 0,
                'site_id': 0
            }
        }
    }
}
@emersonfelipesp emersonfelipesp self-assigned this Nov 15, 2023
@emersonfelipesp emersonfelipesp added feature Introduction of new functionality to the application accepted This issue has been accepted for implementation labels Nov 15, 2023
@emersonfelipesp emersonfelipesp added this to the v0.0.6 milestone Nov 16, 2023
@eliyahuadam
Copy link

Hello

Any update regrading the feature ?

Thanks..

@snk26
Copy link

snk26 commented Jan 12, 2024

Hi,
We are really looking forward to this feature. If the movement is on task?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted This issue has been accepted for implementation feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

4 participants