integrations-core icon indicating copy to clipboard operation
integrations-core copied to clipboard

Add support for Kerberos

Open alopezz opened this issue 3 years ago • 2 comments

What does this PR do?

Enables Kerberos support for the Vertica integration via the options exposed by the official Vertica Python client.

Motivation

AI-2576

Additional Notes

The kerberos package requires compilation in all systems except for MacOS, which requires some kerberos-related packages to be available in the system for installation. That's why we're instructing the end-user that needs it to install it manually.

Review checklist (to be filled by reviewers)

  • [ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • [ ] PR title must be written as a CHANGELOG entry (see why)
  • [ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • [ ] PR must have changelog/ and integration/ labels attached

alopezz avatar Jul 29 '22 14:07 alopezz

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

github-actions[bot] avatar Jul 29 '22 14:07 github-actions[bot]

Codecov Report

Merging #12641 (749f5ec) into master (0b2db55) will increase coverage by 0.04%. The diff coverage is 100.00%.

:exclamation: Current head 749f5ec differs from pull request most recent head 0547563. Consider uploading reports for the commit 0547563 to get more accurate results

Flag Coverage Δ
ddev ?
postgres ?
vertica 98.50% <100.00%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

codecov[bot] avatar Jul 29 '22 15:07 codecov[bot]

closing for now, feel free to re-open if needed

FlorentClarret avatar Nov 08 '23 10:11 FlorentClarret