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

Prometheus telemetry K8S compatibility + functional test enhancements #566

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

tsebastiani
Copy link
Collaborator

@tsebastiani tsebastiani commented Feb 5, 2024

This PR introduces the new fully K8S compatible version of the telemetry collection feature. I also introduces an enhanced version of the functional test Action that will allow to run also a functional tests suite on the merge event. Has been introduced to run more complex functional test (like the telemetry_test) that needs to access Action secrets to be performed.

@tsebastiani tsebastiani force-pushed the k8s_telemetry_prometheus branch 9 times, most recently from 84c0979 to 120e419 Compare February 9, 2024 14:52
@tsebastiani tsebastiani changed the title Prometheus telemetry compatibility + functional tests Prometheus telemetry compatibility + functional test enhancements Feb 9, 2024
Signed-off-by: Tullio Sebastiani <[email protected]>
added comment on the node selector input.yaml

Signed-off-by: Tullio Sebastiani <[email protected]>
@tsebastiani tsebastiani changed the title Prometheus telemetry compatibility + functional test enhancements Prometheus telemetry K8S compatibility + functional test enhancements Feb 9, 2024
Copy link
Collaborator

@chaitanyaenr chaitanyaenr left a comment

Choose a reason for hiding this comment

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

LGTM

@tsebastiani tsebastiani merged commit 2e38b8b into krkn-chaos:main Feb 9, 2024
4 checks passed
@tsebastiani tsebastiani deleted the k8s_telemetry_prometheus branch February 9, 2024 15:38
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