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

Update support email for nvml integration #2353

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mirzanurkic-dd
Copy link

What does this PR do?

Updates the nvml check support and sales email address in manifest.json

Motivation

Hi @cep21,

I'm a support engineer at Datadog, and recently had a customer reach out because the nvml integration stopped working with the most recent Datadog Agent. As you're listed in the CODEOWNERS file, I wanted to update the manifest.json to match and make it clearer for users of this integration to contact you for similar issues. This matches other integrations in the repo - no update to the README.md.

If you do not agree to this, please let me know. I'll then work to make sure this integration is marked as orphaned instead, to prevent anyone from reaching out to you in future.

Internal ticket reference (not publicly viewable): https://datadog.zendesk.com/agent/tickets/1645334

The error in question, for reference:

2024-04-12 05:47:11 UTC | CORE | DEBUG | (pkg/collector/python/loader.go:157 in Load) | Unable to load python module - datadog_checks.nvml: unable to import module 'datadog_checks.nvml': Traceback (most recent call last):
  File "/opt/datadog-agent/embedded/lib/python3.11/site-packages/datadog_checks/nvml/__init__.py", line 5, in <module>
    from .nvml import NvmlCheck
  File "/opt/datadog-agent/embedded/lib/python3.11/site-packages/datadog_checks/nvml/nvml.py", line 16, in <module>
    from .api_pb2 import ListPodResourcesRequest
  File "/opt/datadog-agent/embedded/lib/python3.11/site-packages/datadog_checks/nvml/api_pb2.py", line 25, in <module>
    _LISTPODRESOURCESREQUEST = _descriptor.Descriptor(
                               ^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/datadog-agent/embedded/lib/python3.11/site-packages/google/protobuf/descriptor.py", line 296, in __new__
    _message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

This was resolved with:
sudo -u dd-agent -H /opt/datadog-agent/embedded/bin/pip3 install grpcio pynvml==11.5.0 protobuf==3.20.3

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

Additional Notes

Anything else we should know when reviewing?

@maxgio92
Copy link

maxgio92 commented Nov 26, 2024

Hi @mirzanurkic-dd, I think my PR #2535 is introducing the support you were looking for.

Also, is this PR like #2535 not merged because of missing approval from approver with write access? cc @cswatt

@hkrutzer
Copy link

On #2535 (review) @cep21 writes

The PR is fine, but I'm unable to maintain this module anymore. Can I be removed from the maintainers?

so it should probably be marked orphaned.

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

Successfully merging this pull request may close these issues.

4 participants