Comments (6)
Hi @DCMattyG , thanks for your quick response.
Totally agree about using the term "metadata" instead of "tags",
I used the "tag" concept because I see that the application already uses it, and I intended to implement the functionality I need being the most minimally invasive, but any option would be valid for us.
Basically what we need is a simple description, which could be achieved by adding a simple text field.
Totally agree that displaying a tooltip on a floating window is not the most stylish and practical UI implementation ;)
We will adapt to the implementation that you consider most optimal, and will align the terraform provider to it.
Thanks!!
from ipam.
Hello @XtratusCloud, thanks so much for submitting this feature request!
First and foremost, I love this and I'm happy to incorporate this into Azure IPAM.
Couple of things:
- The term "tags" is a very Azure/Cloud specific term. Would it make more sense to call the additional field "metadata" which woulda align with other API's that already exist within Azure IPAM?
- Displaying a huge list of Key/Value pairs in a tiny column as you've shown in your mocked-up UI above doesn't feel like the best way to bubble this data up to the UI. I would imaging this can be extremely hard to ready, especially if users had a significant number of "tags" or "metadata" they were passing in.
Do you think an arbitrary dictionary of key/value pairs makes sense here? What type of data are YOU using this for, and could those same purposes be applied to other users (e.g. is this only a description, or is there something else that is important here)?
Very much appreciate the feedback and I'm quite sure we can get this folded in right away. Let's just make sure we're thinking through the proposed structure and it's intended use cases together. Thanks!
from ipam.
Thanks for the feedback @XtratusCloud!
In that case does it make sense just to add an optional description field to the API, or is there a need where a number of key/value pairs would need to be appended?
from ipam.
Hi @DCMattyG ,
We only needs to be able to add a description to the reservation, a text field would be fine.
But if you think that adding a "metadata" collection could be useful for other users, will also be good.
Thanks!!
from ipam.
I have tested the existing implementation in the "vwan-support" branch, which allows you to specify a description when creating a reservation, and it covers our needs perfectly.
@DCMattyG , this request can be closed at your convenience.
from ipam.
Hi @XtratusCloud, this is now completed and had been merged into main via PR #113. Thank you for all of your support in bringing this feature into Azure IPAM!
from ipam.
Related Issues (20)
- Restrict access to a group within a Azure AD tenant. HOT 6
- Code version release management HOT 2
- Two part deployment - Deploying part two from parameters file with privateacr set to true not working HOT 6
- Alert when low in IP addresses and make block field optional HOT 4
- Bulk import of space and blocks HOT 2
- Access Control for the IP Reservation Mamangement HOT 2
- Use Azure storage tables as backend instead of CosmosDB HOT 2
- Deployment script produces poor logs when failing on Function only deployment HOT 4
- Error in peering analysis, white screen HOT 2
- Sudden Access Denied error shown to all admins and applications trying to access IPAM API HOT 4
- Change the tag recognition to allow more than 1 tag to be used on a Vnet - allowing address ranges from two blocks to be reserved and consumed by the deployed vnet HOT 10
- Subscription page is returned blank HOT 5
- Error accessing the analysis-visualize screen HOT 2
- Document the terraform provider available to automate reservations. HOT 2
- Subscription Name in visualize and peering tooltips HOT 3
- Insufficient privileges New-MgOauth2PermissionGrant_CreateExpanded error HOT 8
- Terraform example doesn't work as expected HOT 2
- method to get an access token with only curl and jq (without az cli) for remote pipelines HOT 10
- IP reservation from Swagger API UI errors with "Authorization header missing" HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ipam.