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

[C8-6741] added env GitHub application installation url #54

Merged
merged 2 commits into from
Oct 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,9 @@ All notable changes to this Helm chart will be documented in this file.
### Fixed

- Corrected the `matchLabels` selector in the Backend PodDisruptionBudget (PDB) to ensure proper matching of backend pods during disruptions.

## [0.0.26] - 2024-10-04

### Added

- Added GITHUB_APP_INSTALL_URL variable
2 changes: 1 addition & 1 deletion helm-charts/c8/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@ name: c8
description: A Helm chart for the C8 app
type: application

version: 0.0.25
version: 0.0.26
appVersion: "1.0.0"
1 change: 1 addition & 0 deletions helm-charts/c8/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -248,6 +248,7 @@ The table below lists the key application variables that can be configured durin
| variables.USE_SMTP_STRATEGY | string | `"true"` | Flag to use SMTP for emails |
| variables.VERSION | string | `"v1.0.0"` | Application version |
| variables.GITHUB_CUSTOM_RATE_LIMIT | string | `""` | Github custom rate limit |
| variables.GITHUB_APP_INSTALL_URL | string | `""` | GitHub application installation url |
| variables.THROTTLE_ENABLED | string | `"true"` | Enable or disable throttling |
| variables.THROTTLE_TTL | string | `"60000"` | Throttling ttl |
| variables.THROTTLE_LIMIT | string | `"10"` | Throttling requests per ttl |
Expand Down
2 changes: 2 additions & 0 deletions helm-charts/c8/templates/backend-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,8 @@ spec:
value: {{ .Values.variables.THROTTLE_TTL | quote }}
- name: THROTTLE_LIMIT
value: {{ .Values.variables.THROTTLE_LIMIT | quote }}
- name: GITHUB_APP_INSTALL_URL
value: {{ .Values.variables.GITHUB_APP_INSTALL_URL | quote }}
envFrom:
- configMapRef:
name: {{ include "c8.fullname" . }}-common-params
Expand Down
2 changes: 2 additions & 0 deletions helm-charts/c8/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -647,6 +647,8 @@ variables:
SWAGGER_ENABLED: "false"
# -- Enable or disable analytics
DISABLE_ANALYTICS: "true"
# -- GitHub application installation url
GITHUB_APP_INSTALL_URL: ""

##### SSA #########
# -- Enable or disable swagger documentation
Expand Down
Binary file added store/c8-0.0.26.tgz
Binary file not shown.
Loading