Skip to content

feat(charts): add k6 synthetic probe#1289

Open
MahirAbbas wants to merge 1 commit into
mainfrom
ma/k6-probe
Open

feat(charts): add k6 synthetic probe#1289
MahirAbbas wants to merge 1 commit into
mainfrom
ma/k6-probe

Conversation

@MahirAbbas
Copy link
Copy Markdown
Contributor

No description provided.

@MahirAbbas MahirAbbas self-assigned this May 8, 2026
@MahirAbbas MahirAbbas changed the title Ma/k6 probe feat(charts): add k6 synthetic probe May 8, 2026
@MahirAbbas MahirAbbas requested review from TBThomas56 and davehadley and removed request for davehadley May 8, 2026 14:11
@MahirAbbas MahirAbbas marked this pull request as ready for review May 8, 2026 14:11
Comment thread charts/k6-operator/templates/k6-probe-configmap.yaml Outdated
Comment thread charts/k6-operator/templates/k6-probe-configmap.yaml Outdated
Comment thread charts/k6-operator/templates/k6-probe-configmap.yaml Outdated
Comment thread charts/k6-operator/templates/probe-cron.yaml Outdated
Comment thread charts/k6-operator/templates/k6-probe-configmap.yaml
Comment thread charts/k6-operator/templates/k6-probe-configmap.yaml
Comment thread charts/k6-operator/tests/common.ts
Comment thread charts/k6-operator/tests/ping-graph.ts Outdated
limits:
cpu: 1000m
memory: 1Gi
sessionspace:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

these values appear to be unused?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

as in this is made redundant due to sessionspace.yaml in templates?

cpu: 2000m
memory: 2Gi

sessionspace:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

see comment above about unused values

@@ -0,0 +1,16 @@
{{- if eq .Values.cluster "pollux" }}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think that this requires a version bump in the secrets chart (although I notice that CI didn't catch that)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

CI failed when I gave a version bump? I reverted the bump. not sure if I did it in the right place

@MahirAbbas MahirAbbas force-pushed the ma/k6-probe branch 4 times, most recently from f00b051 to d0085d4 Compare May 11, 2026 10:53
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