Repository navigation
feat(event-ledger): add Helm chart for self-hosted stack #762
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
Merged
Merged
Changes from 13 commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
9e96a4b
feat(event-ledger): add Helm chart for self-hosted stack
shelleyshen-0 85e060d
fix(event-ledger): upgrade image versions and fix cassandra contact p…
shelleyshen-0 7deece5
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 a98fb4e
fix(openbao): bump migrations image tag to 0.18.0
shelleyshen-0 913684b
fix(event-ledger): use fullname helper for service references in NOTE…
shelleyshen-0 de0ca1d
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 c68085f
Merge remote-tracking branch 'origin/feat/event-ledger-helm-chart' in…
shelleyshen-0 8259198
fix(event-ledger): add vault-agent-template checksum annotation and u…
shelleyshen-0 76c941a
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 60a6248
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 4afcfe5
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 377c4b3
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 9ceb7da
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 81414a0
fix(event-ledger): align openbao token wiring with POR standard
shelleyshen-0 48e19c4
fix(event-ledger): fix chart name, app version, and autoscaling repli…
shelleyshen-0 0c72d17
fix(event-ledger): address helm chart review feedback
shelleyshen-0 b1e93b9
fix(event-ledger): set least-privilege security context defaults
shelleyshen-0 476c392
docs(event-ledger): add helm chart README
shelleyshen-0 5425684
feat(event-ledger): add PDB, topologySpreadConstraints, and priorityC…
shelleyshen-0 658d586
feat(event-ledger): add digest pinning support to image helper
shelleyshen-0 bf0a0fe
chore(event-ledger): merge main into feat/event-ledger-helm-chart
shelleyshen-0 c27081a
fix(event-ledger): remove policy-bearer-token from vault agent template
shelleyshen-0 a589d77
fix(event-ledger): correct namespace and cassandra host in README
shelleyshen-0 a6e000e
fix(event-ledger): fix image helper to support all digest pinning cases
shelleyshen-0 9a0f817
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 586d869
fix(event-ledger): upgrade image versions and fix cassandra contact p…
shelleyshen-0 b8bb2a2
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 b7b78f1
fix(event-ledger): implement POR-compliant vault agent token wiring
shelleyshen-0 3117e17
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 a8e5360
Merge branch 'main' into feat/event-ledger-helm-chart
shelleyshen-0 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| # | ||
| # Licensed under the Apache License, Version 2.0 (the "License"); | ||
| # you may not use this file except in compliance with the License. | ||
| # You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
|
|
||
| apiVersion: v2 | ||
| name: helm-event-ledger | ||
| description: A Helm chart for NVCF Event Ledger deployment | ||
|
|
||
| type: application | ||
|
|
||
| version: 0.0.0 # autoversioning enabled via release pipeline | ||
|
|
||
| appVersion: "1.0.0" | ||
|
nvjmcnamee marked this conversation as resolved.
Outdated
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| {{- with secret "services/event-ledger/kv/data/cassandra/creds" }} | ||
| { | ||
| "username": {{ .Data.data.username | toJSON }}, | ||
| "password": {{ .Data.data.password | toJSON }}, | ||
| "policy-bearer-token": "self-managed" | ||
|
shelleyshen-0 marked this conversation as resolved.
Outdated
|
||
| } | ||
| {{- end }} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| Event Ledger has been deployed. | ||
|
|
||
| Service DNS (in-cluster): | ||
| {{ include "nvcf-event-ledger.fullname" . }}.{{ include "nvcf-event-ledger.namespace" . }}.svc.cluster.local:8080 | ||
|
|
||
| Health check: | ||
| kubectl -n {{ include "nvcf-event-ledger.namespace" . }} port-forward svc/{{ include "nvcf-event-ledger.fullname" . }} 8080 | ||
| curl http://localhost:8080/health | ||
|
|
||
| Metrics (Prometheus): | ||
| event-ledger.{{ include "nvcf-event-ledger.namespace" . }}.svc.cluster.local:8081 | ||
|
|
||
| The Vault Agent sidecar injects /vault/secrets/secrets.json at pod start. | ||
| If the pod is stuck in Init:CrashLoopBackOff, check the Vault Agent sidecar | ||
| logs — the OpenBao role or secret path is likely missing from the migration. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,129 @@ | ||
| {{/* | ||
| SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| Licensed under the Apache License, Version 2.0 (the "License"); | ||
| you may not use this file except in compliance with the License. | ||
| You may obtain a copy of the License at | ||
|
|
||
| https://www.apache.org/licenses/LICENSE-2.0 | ||
|
|
||
| Unless required by applicable law or agreed to in writing, software | ||
| distributed under the License is distributed on an "AS IS" BASIS, | ||
| WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| See the License for the specific language governing permissions and | ||
| limitations under the License. | ||
| */}} | ||
|
|
||
| {{/* | ||
| Expand the name of the chart. | ||
|
|
||
| Defaults to the literal "event-ledger" rather than .Chart.Name so the | ||
| runtime service identity (resource names, selectors, container name) stays | ||
| stable and matches the in-cluster DNS contract (event-ledger.nvcf.svc), | ||
| independent of the OCI chart package name. Override via | ||
| .Values.eventLedger.nameOverride. | ||
| */}} | ||
| {{- define "nvcf-event-ledger.name" -}} | ||
| {{- default "event-ledger" .Values.eventLedger.nameOverride | trunc 63 | trimSuffix "-" }} | ||
| {{- end }} | ||
|
|
||
| {{/* | ||
| Create a default fully qualified app name. | ||
| We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec). | ||
| If release name contains chart name it will be used as a full name. | ||
| */}} | ||
| {{- define "nvcf-event-ledger.fullname" -}} | ||
| {{- if .Values.eventLedger.fullnameOverride }} | ||
| {{- .Values.eventLedger.fullnameOverride | trunc 63 | trimSuffix "-" }} | ||
| {{- else }} | ||
| {{- $name := default "event-ledger" .Values.eventLedger.nameOverride }} | ||
| {{- if contains $name .Release.Name }} | ||
| {{- .Release.Name | trunc 63 | trimSuffix "-" }} | ||
| {{- else }} | ||
| {{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" }} | ||
| {{- end }} | ||
| {{- end }} | ||
| {{- end }} | ||
|
|
||
| {{/* | ||
| Allow the release namespace to be overridden via .Values.eventLedger.namespace, | ||
| falling back to .Release.Namespace. | ||
| */}} | ||
| {{- define "nvcf-event-ledger.namespace" -}} | ||
| {{- default .Release.Namespace .Values.eventLedger.namespace -}} | ||
| {{- end -}} | ||
|
|
||
| {{/* | ||
| Derive the full image reference. Fails the render if either image.registry | ||
| or image.repository is missing so the chart cannot install a default/invalid | ||
| image. | ||
| */}} | ||
| {{- define "nvcf-event-ledger.image" -}} | ||
| {{- $registry := required "A valid image registry (.Values.eventLedger.image.registry) is required!" .Values.eventLedger.image.registry -}} | ||
| {{- $repository := required "A valid image repository (.Values.eventLedger.image.repository) is required!" .Values.eventLedger.image.repository -}} | ||
| {{- $name := .Values.eventLedger.image.name | default "event-ledger" -}} | ||
| {{- $tag := .Values.eventLedger.image.tag | default .Chart.AppVersion -}} | ||
| {{- printf "%s/%s/%s:%s" $registry $repository $name $tag -}} | ||
| {{- end -}} | ||
|
|
||
| {{/* | ||
| Hashicorp Vault Agent Injector annotations. Always rendered onto the pod | ||
| so the injector mints /vault/secrets/secrets.json at pod start from the | ||
| OpenBao secret paths. The template content is rendered into a ConfigMap by | ||
| templates/configmap-vault-agent-template.yaml and mounted into the pod at | ||
| /vault/config/templates. | ||
| */}} | ||
| {{- define "nvcf-event-ledger.vaultAnnotations" -}} | ||
| {{- $role := required "A valid Vault auth role (.Values.eventLedger.vault.role) is required!" .Values.eventLedger.vault.role -}} | ||
| vault.hashicorp.com/agent-inject: "true" | ||
| vault.hashicorp.com/role: {{ $role | quote }} | ||
| vault.hashicorp.com/auth-path: {{ .Values.eventLedger.vault.jwtAuthPath | default "auth/jwt" | quote }} | ||
| {{- /* Point the agent's kubernetes auto-auth at the OpenBao-audience token, | ||
| mounted off the standard SA path. | ||
| NOTE: this literal must stay in sync with eventLedger.volumeMounts for | ||
| the `openbao-token` volume in values.yaml (mountPath + projected path). | ||
| If you change that mount, change this annotation too. */}} | ||
| vault.hashicorp.com/auth-config-token-path: "/var/run/secrets/openbao/token" | ||
| vault.hashicorp.com/agent-copy-volume-mounts: {{ include "nvcf-event-ledger.fullname" . | quote }} | ||
| vault.hashicorp.com/agent-inject-template-file-secrets.json: "/vault/config/templates/secrets.json.tmpl" | ||
| vault.hashicorp.com/secret-volume-path: "/vault/secrets" | ||
| {{- end }} | ||
|
|
||
| {{/* | ||
| Create chart name and version as used by the chart label. | ||
| */}} | ||
| {{- define "nvcf-event-ledger.chart" -}} | ||
| {{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" }} | ||
| {{- end }} | ||
|
|
||
| {{/* | ||
| Common labels | ||
| */}} | ||
| {{- define "nvcf-event-ledger.labels" -}} | ||
| helm.sh/chart: {{ include "nvcf-event-ledger.chart" . }} | ||
| {{ include "nvcf-event-ledger.selectorLabels" . }} | ||
| {{- if .Chart.AppVersion }} | ||
| app.kubernetes.io/version: {{ .Chart.AppVersion | quote }} | ||
| {{- end }} | ||
| app.kubernetes.io/managed-by: {{ .Release.Service }} | ||
| {{- end }} | ||
|
|
||
| {{/* | ||
| Selector labels | ||
| */}} | ||
| {{- define "nvcf-event-ledger.selectorLabels" -}} | ||
| app.kubernetes.io/name: {{ include "nvcf-event-ledger.name" . }} | ||
| app.kubernetes.io/instance: {{ .Release.Name }} | ||
| {{- end }} | ||
|
|
||
| {{/* | ||
| Create the name of the service account to use | ||
| */}} | ||
| {{- define "nvcf-event-ledger.serviceAccountName" -}} | ||
| {{- if .Values.eventLedger.serviceAccount.create }} | ||
| {{- default (include "nvcf-event-ledger.fullname" .) .Values.eventLedger.serviceAccount.name }} | ||
| {{- else }} | ||
| {{- default "default" .Values.eventLedger.serviceAccount.name }} | ||
| {{- end }} | ||
| {{- end }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| # | ||
| # Licensed under the Apache License, Version 2.0 (the "License"); | ||
| # you may not use this file except in compliance with the License. | ||
| # You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
|
|
||
| apiVersion: v1 | ||
| kind: ConfigMap | ||
| metadata: | ||
| name: {{ include "nvcf-event-ledger.fullname" . }}-app-config | ||
| namespace: {{ include "nvcf-event-ledger.namespace" . }} | ||
| labels: | ||
| {{- include "nvcf-event-ledger.labels" . | nindent 4 }} | ||
| data: | ||
| # Application YAML config mounted at /etc/event-ledger/config.yaml. | ||
| # The container reads it via EVENT_LEDGER_CONFIG env var (see deployment.yaml). | ||
| # Credentials (username, password, policy-bearer-token) are not here; | ||
| # they are injected by Vault Agent into /vault/secrets/secrets.json. | ||
| config.yaml: | | ||
| {{- toYaml .Values.eventLedger.config | nindent 4 }} |
29 changes: 29 additions & 0 deletions
29
deploy/helm/event-ledger/templates/configmap-vault-agent-template.yaml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| # | ||
| # Licensed under the Apache License, Version 2.0 (the "License"); | ||
| # you may not use this file except in compliance with the License. | ||
| # You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
|
|
||
| apiVersion: v1 | ||
| kind: ConfigMap | ||
| metadata: | ||
| name: {{ include "nvcf-event-ledger.fullname" . }}-vault-agent-tpl | ||
| namespace: {{ include "nvcf-event-ledger.namespace" . }} | ||
| labels: | ||
| {{- include "nvcf-event-ledger.labels" . | nindent 4 }} | ||
| data: | ||
| # Vault Agent template rendered into /vault/secrets/secrets.json on pod start. | ||
| # The service reads username/password for Cassandra and policy-bearer-token | ||
|
nvjmcnamee marked this conversation as resolved.
Outdated
|
||
| # for the api-keys-api policy evaluator from this file (run_service.go). | ||
| # Secret paths match what 22_setup_event-ledger.sh provisions in OpenBao. | ||
| secrets.json.tmpl: | | ||
| {{- .Files.Get "files/secrets.json.tmpl" | nindent 4 }} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,113 @@ | ||
| # SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| # | ||
| # Licensed under the Apache License, Version 2.0 (the "License"); | ||
| # you may not use this file except in compliance with the License. | ||
| # You may obtain a copy of the License at | ||
| # | ||
| # https://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
|
|
||
| apiVersion: apps/v1 | ||
| kind: Deployment | ||
| metadata: | ||
| name: {{ include "nvcf-event-ledger.fullname" . }} | ||
| namespace: {{ include "nvcf-event-ledger.namespace" . }} | ||
| labels: | ||
| {{- include "nvcf-event-ledger.labels" . | nindent 4 }} | ||
| spec: | ||
| replicas: {{ .Values.eventLedger.replicaCount }} | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
| selector: | ||
| matchLabels: | ||
| {{- include "nvcf-event-ledger.selectorLabels" . | nindent 6 }} | ||
| template: | ||
| metadata: | ||
| annotations: | ||
| {{- include "nvcf-event-ledger.vaultAnnotations" . | nindent 8 }} | ||
| checksum/config-app: {{ toYaml .Values.eventLedger.config | sha256sum }} | ||
|
coderabbitai[bot] marked this conversation as resolved.
|
||
| checksum/vault-agent-template: {{ .Files.Get "files/secrets.json.tmpl" | sha256sum }} | ||
| {{- with .Values.eventLedger.podAnnotations }} | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
| labels: | ||
| {{- include "nvcf-event-ledger.selectorLabels" . | nindent 8 }} | ||
| {{- with .Values.eventLedger.podLabels }} | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
| spec: | ||
| {{- with .Values.eventLedger.imagePullSecrets }} | ||
| imagePullSecrets: | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
| serviceAccountName: {{ include "nvcf-event-ledger.serviceAccountName" . }} | ||
| securityContext: | ||
| {{- toYaml .Values.eventLedger.podSecurityContext | nindent 8 }} | ||
| volumes: | ||
| - name: vault-config-templates | ||
| configMap: | ||
| name: {{ include "nvcf-event-ledger.fullname" . }}-vault-agent-tpl | ||
| items: | ||
| - key: secrets.json.tmpl | ||
| path: secrets.json.tmpl | ||
| - name: app-config | ||
| configMap: | ||
| name: {{ include "nvcf-event-ledger.fullname" . }}-app-config | ||
| items: | ||
| - key: config.yaml | ||
| path: config.yaml | ||
| {{- with .Values.eventLedger.volumes }} | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
| containers: | ||
| - name: {{ include "nvcf-event-ledger.fullname" . }} | ||
| securityContext: | ||
| {{- toYaml .Values.eventLedger.securityContext | nindent 12 }} | ||
| image: {{ include "nvcf-event-ledger.image" . | quote }} | ||
| imagePullPolicy: {{ .Values.eventLedger.image.pullPolicy }} | ||
| volumeMounts: | ||
| - name: vault-config-templates | ||
| mountPath: /vault/config/templates | ||
| readOnly: true | ||
| - name: app-config | ||
| mountPath: /etc/event-ledger | ||
| readOnly: true | ||
| {{- with .Values.eventLedger.volumeMounts }} | ||
| {{- toYaml . | nindent 12 }} | ||
| {{- end }} | ||
| env: | ||
| # Point the service at the mounted config file. | ||
| - name: EVENT_LEDGER_CONFIG | ||
| value: /etc/event-ledger/config.yaml | ||
| {{- range $key, $value := .Values.eventLedger.env }} | ||
| - name: {{ $key }} | ||
| value: {{ $value | quote }} | ||
| {{- end }} | ||
| ports: | ||
| {{- range .Values.eventLedger.ports }} | ||
| - name: {{ .name }} | ||
| containerPort: {{ .containerPort }} | ||
| protocol: {{ .protocol }} | ||
| {{- end }} | ||
| livenessProbe: | ||
| {{- toYaml .Values.eventLedger.livenessProbe | nindent 12 }} | ||
| readinessProbe: | ||
| {{- toYaml .Values.eventLedger.readinessProbe | nindent 12 }} | ||
| resources: | ||
| {{- toYaml .Values.eventLedger.resources | nindent 12 }} | ||
| {{- with .Values.eventLedger.nodeSelector }} | ||
| nodeSelector: | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
| {{- with .Values.eventLedger.affinity }} | ||
| affinity: | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
| {{- with .Values.eventLedger.tolerations }} | ||
| tolerations: | ||
| {{- toYaml . | nindent 8 }} | ||
| {{- end }} | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.