Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Enable security context in chart #6

Merged
merged 6 commits into from
Jul 26, 2024
Merged

Conversation

gabfelp
Copy link
Contributor

@gabfelp gabfelp commented Jul 24, 2024

Description of the change

These changes aim for enabling security context.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests
  • All tests related to the changed code pass in development

Code review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
  • Jira issue referenced in commit message and/or PR title

Testing

We tested using a PG data repo and a Snowflake data repo with this helm sidecar.

@gabfelp gabfelp self-assigned this Jul 24, 2024
@gabfelp gabfelp marked this pull request as ready for review July 25, 2024 21:12
@antoniomrfranco antoniomrfranco merged commit 889cfe5 into main Jul 26, 2024
@antoniomrfranco antoniomrfranco deleted the support-security-context branch July 26, 2024 19:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants