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

Chart modifications for official release #1

Merged
merged 10 commits into from
Oct 22, 2024
Merged

Chart modifications for official release #1

merged 10 commits into from
Oct 22, 2024

Conversation

tomhasura
Copy link
Collaborator

@tomhasura tomhasura commented Oct 17, 2024

  • OTEL variable changes within values.yaml. Setting OTEL to be enabled by default. Ensure that certain variables are required when observability.enabled is set to true
  • Modified examples within READMEs to account for OTEL being enabled by default
  • connector prefixes renamed to connectorEnvVars
  • Modified version in all Chart.yaml to vYYYY-MM-DD
  • Modified appVersion in all Chart.yaml from 1.16.0 to 3.0.0
  • Re-arranged documented parameters within READMEs
  • Sanitize code via lint
  • Add NOTES.txt under all Helm charts
  • Create CONNECTORS file which lists out the connector types supported via Helm charts. File will be referenced via Public Docs for easily getting the connector type value

@tomhasura tomhasura changed the title Modify Connector ENV names Chart modifications for release Oct 17, 2024
@tomhasura tomhasura changed the title Chart modifications for release Chart modifications for v2024-10-18 release Oct 18, 2024
nithindv and others added 2 commits October 18, 2024 19:33
add validation for dataplane.key and id
remove trailing spaces
add custom helm release notes for v3-engine
other fixes
@tomhasura tomhasura requested a review from shahidhk October 18, 2024 15:07
@tomhasura tomhasura changed the title Chart modifications for v2024-10-18 release Chart modifications for official release Oct 22, 2024
@tomhasura tomhasura merged commit c6541fb into main Oct 22, 2024
1 check passed
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