Skip to content

Add helm chart for NGTS-capable agent#789

Merged
SgtCoDFish merged 1 commit intomasterfrom
ngts-helm-chart
Apr 9, 2026
Merged

Add helm chart for NGTS-capable agent#789
SgtCoDFish merged 1 commit intomasterfrom
ngts-helm-chart

Conversation

@SgtCoDFish
Copy link
Copy Markdown
Contributor

@SgtCoDFish SgtCoDFish commented Apr 9, 2026

This has been tested to the point of being able to upload data to the backend through NGTS. I haven't been able to fully test that the uploaded data appears in the UI, but the upload itself is successful.

This doesn't add a release mechanism for the new chart, so adding this is still a no-op in terms of customer-facing functionality.

Example config:

config:
  tsgID: "1111111111"
  serverURL: https://1111111111.data.plane.example.com
  clusterName: test-cluster
  clusterDescription: "test cluster for agent"

volumes:
- name: custom-ca-volume
  configMap:
    name: custom-ca

volumeMounts:
- name: custom-ca-volume
  mountPath: /etc/ssl/certs

Example credentials:

apiVersion: v1
data:
  clientID: MmNkNzdlNzQtMmU3Ni0xMWYxLWIzN2QtYjIzZTk5MmMzYjAz
  privatekey.pem: <redacted>
kind: Secret
metadata:
  creationTimestamp: "2026-04-09T13:48:54Z"
  name: discovery-agent-credentials
  namespace: paloalto
  resourceVersion: "766"
  uid: f7aaca15-7bd4-4bb6-90fb-7d9d75501de5
type: Opaque

@SgtCoDFish SgtCoDFish force-pushed the ngts-helm-chart branch 4 times, most recently from 1c00c23 to fd88300 Compare April 9, 2026 14:20
This has been tested to the point of being able to upload data to the
backend through NGTS. I haven't been able to fully test that the
uploaded data appears in the UI, but the upload itself is successful.

Signed-off-by: Ashley Davis <ashley.davis@cyberark.com>
@SgtCoDFish
Copy link
Copy Markdown
Contributor Author

I had a huddle with Georgi and walked him through the changes here. I explained that most of this is additive, so there's no impact from merging now. We can tweak afterwards - but we're merging this now.

@SgtCoDFish SgtCoDFish merged commit 16ed06d into master Apr 9, 2026
7 of 8 checks passed
@SgtCoDFish SgtCoDFish deleted the ngts-helm-chart branch April 9, 2026 14:55
@maelvls
Copy link
Copy Markdown
Member

maelvls commented Apr 11, 2026

I'll also start using the namespace paloalto in Firefly.

Comment on lines +164 to +165
# Push data every 12 hours unless changed.
period: "0h1m0s"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How come this says 1 minute but the comment says 12 hours? Also, 12 hours seems way to large of a value?

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.

3 participants