Skip to content

Conversation

@NguyenHoangSon96
Copy link
Contributor

  • This PR only syncs and fixes the "vsphere.conf" and "vsphere.md". So this additional PR will fix and sync contents for the rest of the ".conf" and ".md" files.
  • Looks like the instructions from "src/writeData/utils/README.md" are obsolete, so I modified the scripts in "updateTelegrafPlugins.mjs" and regenerated the "/telegraf/etc/telegraf.conf" file to make it work.
    updateTelegrafPlugins.txt

** More than 300+ files have been generated or edited, so I don't really know how to check, to be honest 😅, so if you guys think this PR is too much and unnecessary at the moment, please close it.

Checklist

Authors and Reviewer(s), please verify the following:

  • A PR description, regardless of the triviality of this change, that communicates the value of this PR
  • Well-formatted conventional commit messages that provide context into the change
  • Documentation updated or issue created (provide link to issue/PR)
  • Signed CLA (if not already signed)
  • Feature flagged, if applicable

@NguyenHoangSon96 NguyenHoangSon96 self-assigned this Nov 5, 2025
@NguyenHoangSon96
Copy link
Contributor Author

@wdoconnell @philjb
I have to move this #7089 here because #7089 is from my fork repo, so I can't
"update the PR to merge into master (not influxdata:master)" like @wdoconnell suggested.

@NguyenHoangSon96 NguyenHoangSon96 marked this pull request as ready for review November 5, 2025 04:10
@NguyenHoangSon96 NguyenHoangSon96 requested review from a team as code owners November 5, 2025 04:10
@philjb
Copy link
Contributor

philjb commented Nov 5, 2025

@wdoconnell -- my comment here was some questions for you:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants