Add site and site_slug labels to device services

[jkrcma]

Describe your changes

Hello,

when developing an internal Service Discovery framework I found out that Netbox devices do not necessarily have the Site label present. When digging I found out that the condition applies only to devices without Cluster set. I've made this PR to prioritize locally configured Site of the device object, instead of Rack or Cluster. Given how loose the constraints in Netbox database are, I might have missed something. Let me know please!

Issue ticket number and link

None.

Checklist before requesting a review

• I have performed a self-review of my code
• If it is a core feature, I have added thorough tests.

[FlxPeters]

@jkrcma tests seem to be broken. Can you please check and fix this?
I agree with the change at all, so if the build is green we can move forward.

[jkrcma]

Looks like a consequence of the change I did in b40f30e. Despite using Docker I wasn't able to make tests running on my local laptop without that commit. Thanks for the CI output, I will dig in it.

[jkrcma]

Alright, figured out. I did a mistake in the first step of delegating the venv to poetry itself. I apologize, haven't worked with it much lately 🙂

[FlxPeters]

Looks good to me regarding the code. Can you please reformat the commits to comply with the semantic commit convention (https://github.com/FlxPeters/netbox-plugin-prometheus-sd?tab=readme-ov-file#conventional-commits)?

[jkrcma]

Done, let me know if you have any suggestion for the scope of the feat one. I was thinking of api or services, but I chose to not specify it at all.