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

CI: Run TLS authentication tests #1037

Merged
merged 1 commit into from
Sep 14, 2023
Merged

Conversation

julienduchesne
Copy link
Member

The provider supports the case where Grafana is proxied and has TLS cert auth in front of it
This currently not tested in CI and that makes PRs like #1013 harder to test

This adds a pipeline that checks that the feature works correctly

@github-actions
Copy link

In order to lower resource usage and have a faster runtime, PRs will not run Cloud tests automatically. To do so, a Grafana Labs employee must promote the Drone build.

For maintainers, it's better to run only the Cloud tests you need, rather than all of them. You can do so by setting the following parameter when promoting:

TESTARGS='-run=<testname>'

@julienduchesne julienduchesne force-pushed the julienduchesne/tls-tests-ci branch 6 times, most recently from 8be5e26 to ddea160 Compare September 14, 2023 13:55
The provider supports the case where Grafana is proxied and has TLS cert auth in front of it
This currently not tested in CI and that makes PRs like #1013 harder to test

This adds a pipeline that checks that the feature works correctly
@julienduchesne julienduchesne marked this pull request as ready for review September 14, 2023 14:09
@julienduchesne julienduchesne requested a review from a team as a code owner September 14, 2023 14:09
@julienduchesne julienduchesne merged commit 0c9a8ea into master Sep 14, 2023
3 checks passed
@julienduchesne julienduchesne deleted the julienduchesne/tls-tests-ci branch September 14, 2023 14:56
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.

None yet

2 participants