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

fix: allow special server_version such as TDE #772

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

Conversation

marcofoyer
Copy link

@marcofoyer marcofoyer commented Dec 3, 2024

show server_version;

server_version

15.8_TDE_1.1.8
(1 row)

Thank you for your interest in contributing to Checkmk!
Consider looking into Readme regarding process details.

General information

We are running Cybertec Postgresql version and the server_string doesn't match for our monitoring.

Bug reports

Please include:

  • RedHat 8.10
  • psql (15.8_TDE_1.1.8)

Proposed changes

Sometimes it is hard for us to assess the quality of a fix.
While it may work for you, it is our job to ensure that it works for everybody.
These are some ways to help us:

  • What is the expected behavior?
  • What is the observed behavior?
  • If it's not obvious from the above: In what way does your patch change the current behavior?
  • Consider writing a unit test that would have failed without your fix.
  • Is this a new problem? What made you submit this PR (new firmware, new device, changed device behavior)?

# show server_version;
 server_version
----------------
 15.8_TDE_1.1.8
(1 row)
Copy link

github-actions bot commented Dec 3, 2024


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA or my organization already has a signed CLA.


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

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.

1 participant