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

[receiver/vcenter] Refresh User Sessions #13733

Merged
merged 4 commits into from
Aug 31, 2022

Conversation

schmikei
Copy link
Contributor

@schmikei schmikei commented Aug 30, 2022

Description: Attempt to refresh user sessions when the monitoring user gets logged out.

Submitting in draft to run a couple long runs to validate.

Link to tracking Issue: Potentially solves #13447

Testing: Long form testing and unit testing using the simulator

Documentation: no new documentation

@schmikei schmikei marked this pull request as ready for review August 31, 2022 13:17
@schmikei schmikei requested a review from a team August 31, 2022 13:17
@schmikei schmikei requested a review from djaglowski as a code owner August 31, 2022 13:17
@schmikei
Copy link
Contributor Author

@djaglowski

I believe this is a good change to make, I've ran a couple long runs (5+ hours) with this config and appears to be revalidating the sessions.

receivers:
  vcenter:
    endpoint: <redacted>
    username: "<redacted>"
    password: "<redacted>"
    collection_interval: 1m
    tls:
      insecure: true
      insecure_skip_verify: true

Do you mind taking a look at this?

@djaglowski djaglowski merged commit e2da789 into open-telemetry:main Aug 31, 2022
@schmikei schmikei deleted the vcenter-refresh-sessions branch August 31, 2022 17:32
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