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

Updated ingress definitions to be compatiable with K8S 1.22 #6

Merged
merged 17 commits into from
Apr 7, 2022

Conversation

keyvaann
Copy link
Collaborator

@keyvaann keyvaann commented Nov 27, 2021

Description of the change

Ingress definitions are now compatible with version 1.22 and unified having same format across all charts.

Benefits

Allowing RADAR-Base installation on Kubernetes 1.22 and above.

Possible drawbacks

Not compatible with old versions of Kubernetes anymore.

Applicable issues

  • fixes #

Additional information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [<name_of_the_chart>])

@keyvaann keyvaann requested a review from blootsvoets November 27, 2021 05:44
@github-actions
Copy link

github-actions bot commented Nov 27, 2021

Great PR! Please pay attention to the following items before merging:

Files matching charts/*/values.yaml:

  • Is the PR adding a new container? Please reviewer, add it to the models (internal process)
  • Is the PR adding a new parameter? Please, ensure it’s documented in the README.md

This is an automatically generated QA checklist based on modified files

@keyvaann keyvaann requested a review from nivemaham November 27, 2021 05:44
@keyvaann
Copy link
Collaborator Author

This PR is still not ready pending for testing #5

blootsvoets and others added 4 commits December 21, 2021 10:15
…pdate-ingress

* 'main' of github.com:RADAR-base/radar-helm-charts: (54 commits)
  Fix docs
  remove trailing spaces
  changes based PR review
  bump version
  lint fixes
  update versions and docs
  remove unknown property from radar-integration
  Make the kafka properties configurable
  Bump version
  Update README
  Keep previous var configuration
  Keep previous var configuration
  Use environemnt key for vars in values.yaml
  Add support for SSL protocol in radar-jdbc-connector
  Bump api version
  Bump radar-jdbc-connector
  Bump appserver tag and version
  Use dedicated db for radar-appserver
  Fix readme
  Enable CORS in radar-gateway
  ...
…rts into update-ingress

* 'update-ingress' of github.com:RADAR-base/radar-helm-charts:
@keyvaann
Copy link
Collaborator Author

keyvaann commented Apr 7, 2022

Merging without review.

@keyvaann keyvaann merged commit d3d00a5 into main Apr 7, 2022
@blootsvoets blootsvoets deleted the update-ingress branch July 18, 2022 09:15
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