-
Notifications
You must be signed in to change notification settings - Fork 392
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
[vsphere]: Update README with limitations in Virtual Machine metrics #8331
Conversation
🌐 Coverage report
|
packages/vsphere/docs/README.md
Outdated
@@ -13,6 +13,8 @@ To access the metrices, the url https://host:port(8989)/sdk needs to be passed t | |||
|
|||
The virtual machine consists of a set of specification and configuration files and is backed by the physical resources of a host. Every virtual machine has virtual devices that provide the same functionality as physical hardware but are more portable, secure and easier to manage. | |||
|
|||
Note: vSphere Integration currently supports network names of VM's conecected only to vSS (Virtual Standard Switch) and not vDS (virtual Dustributed Switches). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: vSphere Integration currently supports network names of VM's conecected only to vSS (Virtual Standard Switch) and not vDS (virtual Dustributed Switches). | |
> Note: vSphere Integration currently supports network names of VMs connected only to vSS (Virtual Standard Switch) and not vDS (virtual Distributed Switches). |
Notes are usually in blockquote. Also spelling mistakes ^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not an explicit mention that blockquote is required. Would prefer having it as continuation of the documentation of virtualmachine metricset.
Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
- version: 1.9.1 | ||
changes: | ||
- description: Update the README with limitations in Virtual Machine metrics. | ||
type: bugfix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it make sense to treat it an enhancement? Or is it related to a bugfix, that's why we are calling it a bugfix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its an addition of a missing documentation, so bugfix should be good
Co-authored-by: subham sarkar <sarkar.subhams2@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small comment about the update being an enhancement, rest looks good to me.
Package vsphere - 1.9.1 containing this change is available at https://epr.elastic.co/search?package=vsphere |
Checklist
changelog.yml
file.Related issues