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.

Sign up for GitHub

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

Jump to bottom

Add Ability to Provision Postgres and Fides Workers #23

Merged
merged 9 commits into from
Feb 7, 2023
Merged

Add Ability to Provision Postgres and Fides Workers #23

merged 9 commits into from
Feb 7, 2023

Conversation

RobertKeyser
Copy link
Contributor

@RobertKeyser RobertKeyser commented Feb 6, 2023 •
edited
Loading

Description of Changes

This PR adds the ability to optionally provision a Postgres instance and to configure how many Fides workers are running for a given deployment.

  • Add Postgres Bitnami chart as an optional dependency
  • Add Fides workers
  • Upgrade Redis dependency
  • Increment chart version

Pre-merge Checklist

  • All CI Pipelines Succeeded
  • Documentation Updated
  • Increment Applicable Chart Versions
  • Relevant Follow-Up Issues Created

@RobertKeyser RobertKeyser self-assigned this Feb 6, 2023
@github-actions github-actions bot added the helm/fides The Fides Helm Chart label Feb 6, 2023
@RobertKeyser RobertKeyser mentioned this pull request Feb 6, 2023
Closed
4 tasks
@RobertKeyser
Copy link
Contributor Author

While adding the workers, I ran into several issues that are now documented as Fides issues:

@RobertKeyser RobertKeyser marked this pull request as ready for review February 7, 2023 18:15
@RobertKeyser RobertKeyser requested a review from a team as a code owner February 7, 2023 18:15
daveqnet
daveqnet approved these changes Feb 7, 2023
View reviewed changes
Copy link
Contributor

@daveqnet daveqnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved!

@RobertKeyser RobertKeyser merged commit e222059 into main Feb 7, 2023
@RobertKeyser RobertKeyser deleted the rk-postgres branch February 7, 2023 18:47
@RobertKeyser RobertKeyser mentioned this pull request Feb 7, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daveqnet daveqnet daveqnet approved these changes

Assignees

@RobertKeyser RobertKeyser

Labels
chore helm/fides The Fides Helm Chart
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RobertKeyser @daveqnet