Skip to content

Enable TLS between MariaDB and TrustyAI Service#190

Merged
rnetser merged 4 commits intoopendatahub-io:mainfrom
sheltoncyril:feature/mariadb-tls-support
Mar 24, 2025
Merged

Enable TLS between MariaDB and TrustyAI Service#190
rnetser merged 4 commits intoopendatahub-io:mainfrom
sheltoncyril:feature/mariadb-tls-support

Conversation

@sheltoncyril
Copy link
Copy Markdown
Contributor

Description

Enable and enforce TLS for MariaDB connections.
Add fixture to copy mariadb-ca certificate to trustyai-ca-cert secret.

How Has This Been Tested?

Ran all the tests within TestFairnessMetricsWithDBStorage and verified on the console if the desired state is reached.
Test was run multiple times to check for conflict errors which may indicate dangling resources and none were found, the clean-up works perfectly.

Merge criteria:

  • The commits are squashed in a cohesive manner and have meaningful messages.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

@github-actions
Copy link
Copy Markdown

The following are automatically added/executed:

Available user actions:

  • To mark a PR as WIP, add /wip in a comment. To remove it from the PR comment /wip cancel to the PR.
  • To block merging of a PR, add /hold in a comment. To un-block merging of PR comment /hold cancel.
  • To mark a PR as approved, add /lgtm in a comment. To remove, add /lgtm cancel.
    lgtm label removed on each new commit push.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
Supported labels

{'/lgtm', '/verified', '/wip', '/hold'}

Enable and enforce TLS for MariaDB connections

Add fixture to copy mariadb-ca certificate to
trustyai-ca-cert secret.
@sheltoncyril sheltoncyril force-pushed the feature/mariadb-tls-support branch from 7812a13 to cc21be1 Compare March 20, 2025 11:54
@sheltoncyril sheltoncyril marked this pull request as ready for review March 20, 2025 12:05
@sheltoncyril sheltoncyril requested a review from a team as a code owner March 20, 2025 12:05
@sheltoncyril
Copy link
Copy Markdown
Contributor Author

/verified

@github-actions github-actions bot added the Verified Verified pr in Jenkins label Mar 24, 2025
@rnetser rnetser merged commit 3d17823 into opendatahub-io:main Mar 24, 2025
5 checks passed
@sheltoncyril sheltoncyril deleted the feature/mariadb-tls-support branch June 12, 2025 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs Verified Verified pr in Jenkins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants