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

feat(ci): Add a matrix style test for different versions of k8s & helm #418

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

jonstacks
Copy link
Collaborator

What

While trying to reproduce #416, I made a GitHub Actions matrix job that allows us to install to do an e2e style deployment of our helm charts. We aren't providing an actual auth token or API key currently, so it will just fail to connect, but we can use this to validate that chart versions install properly along with the CRDs. If we do find a case that can re-create #416, this will also help serve as a regression test in the future.

How

Created a GitHub action that allows for workflow_dispatch, so we can test any of our helm chart versions. I chose the latest 4 versions of helm as well as k8s to test against using a kind cluster

Breaking Changes

No, this is an internal tool for testing.

@jonstacks jonstacks added meta/project Issues/PRs related to maintaining the project area/ci Issues/PRs relating to CI labels Aug 28, 2024
@jonstacks jonstacks self-assigned this Aug 28, 2024
@jonstacks jonstacks requested a review from a team as a code owner August 28, 2024 22:57
@github-actions github-actions bot removed the meta/project Issues/PRs related to maintaining the project label Aug 28, 2024
@jonstacks
Copy link
Collaborator Author

Here is one of the test runs for reference https://github.com/jonstacks/ngrok-ingress-controller/actions/runs/10605121421

Copy link
Contributor

@nikolay-ngrok nikolay-ngrok left a comment

Choose a reason for hiding this comment

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

👍

@jonstacks jonstacks merged commit 49b93b9 into ngrok:main Aug 29, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues/PRs relating to CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants