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

feat: enable helm chart #22

Merged
merged 27 commits into from
Mar 18, 2024
Merged

Conversation

evegufy
Copy link
Contributor

@evegufy evegufy commented Mar 18, 2024

Description

  • fix: remove lint issue
  • fix: solve templating issues
  • chore: change setup of cronjobs: remove hooks
  • chore: change name setup of job resources
  • chore: add line breaks
  • chore: move placeholder value into resources
  • chore: change to unique templates for db subchart
  • chore: change secret setup
  • chore: move passwords from db dependency to according section
  • chore: remove upgrade env file
  • chore: change centralidp setup
  • chore: rearrange health checks
  • chore: rearrange values file
  • chore: change ingress to trg-5.04
  • chore: fix container name and namespace
  • chore: change image tag retrieval
  • chore: change version
  • chore(db-dependency): change image tag to get latest minor updates
  • chore: set resource limits
  • chore: update readme files
  • chore: change credentialexpiry to camelcase

Why

#20
#11

Checklist

@evegufy evegufy changed the title Feature/helm charts eg 2 chore: enable helm chart Mar 18, 2024
@evegufy evegufy requested a review from Phil91 March 18, 2024 13:11
@evegufy evegufy marked this pull request as ready for review March 18, 2024 13:11
Copy link
Member

@Phil91 Phil91 left a comment

Choose a reason for hiding this comment

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

I'd recreate the migration to rename the file matching the name of the migration.

@evegufy
Copy link
Contributor Author

evegufy commented Mar 18, 2024

I'd recreate the migration to rename the file matching the name of the migration.

@Phil91 I reverted the name change of the migration

Copy link

sonarcloud bot commented Mar 18, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@evegufy evegufy requested a review from Phil91 March 18, 2024 15:09
@evegufy evegufy merged commit 9e147e3 into feature/helm-charts Mar 18, 2024
9 of 11 checks passed
@evegufy evegufy deleted the feature/helm-charts_EG-2 branch March 18, 2024 15:17
Phil91 added a commit that referenced this pull request Mar 25, 2024
* feat: add issuer component
* build: add helm chart for issuer component
* feat(notification): adjust notification endpoint
* fix(build): enable build of docker images (#21)
* feat: add callback process step
* chore: enable helm chart (#22)
* fix: remove lint issue
* fix: solve templating issues
* chore: change setup of cronjobs: remove hooks
* chore: change name setup of job resources
* chore: add line breaks
* chore: move placeholder value into resources
* chore: change to unique templates for db subchart
* chore: change secret setup
* chore: move passwords from db dependency to according section
* chore: remove upgrade env file
* chore: change centralidp setup
* chore: rearrange health checks
* chore: rearrange values file
* chore: change ingress to trg-5.04
* chore: fix container name and namespace
* chore: change image tag retrieval
* chore: change version
* chore(db-dependency): change image tag to get latest minor updates
* chore: set resource limits
* chore: update readme files
* chore: change credentialexpiry to camelcase
* chore: fix helm chart, improve workflows and docs (#23)
* chore(helm-test): fix image name and tag override at upgrade
* chore: fix owasp scan
* chore(helm-test and owasp): change set command
* chore: re-arrange values file
* chore(pre-checks): run only on changes to src/**
* docs(CONTRIBUTING.md): update to contribution details
* chore: fix db dependency secret name in cronjobs
* chore(dependencies-check): align file naming and docs
* chore: remove white space

---------

Refs: #2 #3 #4 #5 #6 #7 #8 #9 #13 #21 #22 #23
Reviewed-by: Evelyn Gurschler <[email protected]>
Reviewed-by: Norbert Truchsess <[email protected]>
Co-authored-by: Evelyn Gurschler <[email protected]>
Co-authored-by: Norbert Truchsess <[email protected]>
Phil91 added a commit that referenced this pull request Apr 8, 2024
* feat: add issuer component
* build: add helm chart for issuer component
* feat(notification): adjust notification endpoint
* fix(build): enable build of docker images (#21)
* feat: add callback process step
* chore: enable helm chart (#22)
* fix: remove lint issue
* fix: solve templating issues
* chore: change setup of cronjobs: remove hooks
* chore: change name setup of job resources
* chore: add line breaks
* chore: move placeholder value into resources
* chore: change to unique templates for db subchart
* chore: change secret setup
* chore: move passwords from db dependency to according section
* chore: remove upgrade env file
* chore: change centralidp setup
* chore: rearrange health checks
* chore: rearrange values file
* chore: change ingress to trg-5.04
* chore: fix container name and namespace
* chore: change image tag retrieval
* chore: change version
* chore(db-dependency): change image tag to get latest minor updates
* chore: set resource limits
* chore: update readme files
* chore: change credentialexpiry to camelcase
* chore: fix helm chart, improve workflows and docs (#23)
* chore(helm-test): fix image name and tag override at upgrade
* chore: fix owasp scan
* chore(helm-test and owasp): change set command
* chore: re-arrange values file
* chore(pre-checks): run only on changes to src/**
* docs(CONTRIBUTING.md): update to contribution details
* chore: fix db dependency secret name in cronjobs
* chore(dependencies-check): align file naming and docs
* chore: remove white space

---------

Refs: #2 #3 #4 #5 #6 #7 #8 #9 #13 #21 #22 #23
Reviewed-by: Evelyn Gurschler <[email protected]>
Reviewed-by: Norbert Truchsess <[email protected]>
Co-authored-by: Evelyn Gurschler <[email protected]>
Co-authored-by: Norbert Truchsess <[email protected]>
@evegufy evegufy changed the title chore: enable helm chart feat: enable helm chart Apr 15, 2024
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.

2 participants