-
Notifications
You must be signed in to change notification settings - Fork 690
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
Test minimal Envoy functionality before release #2223
Comments
xref #2048 |
Probably need to spin off a separate issue for generating the deployment YAML to run envoy with extensions disabled. That might depend on doing something with kustomize, since we want to be able to deploy Contour in a standard way but just slam in some new flags to the envoy container. xref #2088 |
Add the bare minimum CI machinery to fetch a Kubernetes toolchain and verify that Contour can be installed in the example deployment configuration. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Add the bare minimum CI machinery to fetch a Kubernetes toolchain and verify that Contour can be installed in the example deployment configuration. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Add the bare minimum CI machinery to fetch a Kubernetes toolchain and verify that Contour can be installed in the example deployment configuration. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Completed manual testing for Contour release v1.2.0 |
Add the bare minimum CI machinery to fetch a Kubernetes toolchain and verify that Contour can be installed in the example deployment configuration. This updates #2223. Signed-off-by: James Peach <jpeach@vmware.com>
@jpeach what's the status on the test infra? Can we merge that and start incorporating even if it's not 100% complete. We also need to find a way to track changes as new features are added. For example, the ingress class work that was just added should be tracked somewhere to be added to the functional tests. |
The test driver works and can run declarative tests. I've got a handful of tests written for Contour.
Yes, we should start that. I need to publish the repo and add goreleaser so that there is an artifact that we can run. I have a separate repo with the test cases, which I think would end up being merged into Contour repository.
We could add a label, or file an issue for each test as we think of it? |
I think that creating issues with a label is the way to go |
Add an initial set of inte3gration test documents, along with helper scripts to install a kind cluster and to run the tests. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Add an initial set of integration test documents, along with helper scripts to install a kind cluster and to run the tests. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Add an initial set of integration test documents, along with helper scripts to install a kind cluster and to run the tests. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Add an initial set of integration test documents, along with helper scripts to install a kind cluster and to run the tests. This updates projectcontour#2223. Signed-off-by: James Peach <jpeach@vmware.com>
Add an initial set of integration test documents, along with helper scripts to install a kind cluster and to run the tests. This updates #2223. Signed-off-by: James Peach <jpeach@vmware.com>
What's the action item on this issue? Should we consider this closed now that we have some integration tests in place? |
Yes, I think that we should close this out now that we have integration tests in place. More specific items should generate new issues. |
This issue covers work to test the minimum Envoy functionality in #2048 with a release candidate before the final release is cut.
This step should be performed between creating the release branch and cutting the final release in the release runbook https://projectcontour.io/resources/release-process/.
The text was updated successfully, but these errors were encountered: