Open johnbelamaric opened 10 months ago
For the Porch e2e tests, it does some of them now but not the ones that require a cluster. We need:
make ci-test-porch
needs to create a Kind cluster, deploy the code (make deploy
), then set E2E=1
and do the make test
. Right now, it doesn't set E2E
so those tests are skipped.See also https://github.com/nephio-project/porch/pull/4#issuecomment-1861976730 and the subsequent comments.
@electrocucaracha can you help with the above?
Once this is done, we will want to update the docs to all use porchctl
instead of kpt alpha
. We will still use kpt
for some things (like function execution, etc.).
Once this is done, we will want to update the docs to all use
porchctl
instead ofkpt alpha
. We will still usekpt
for some things (like function execution, etc.).
Done
For the Porch e2e tests, it does some of them now but not the ones that require a cluster. We need:
* Image build and push working for PRs for the nephio-project/porch repo * The `make ci-test-porch` needs to create a Kind cluster, deploy the code (`make deploy`), then set `E2E=1` and do the `make test`. Right now, it doesn't set `E2E` so those tests are skipped.
See also nephio-project/porch#4 (comment) and the subsequent comments.
PR raised here - https://github.com/nephio-project/porch/pull/21
Triaged Triage Comment: Complete in R3
Tracking issue for the migration discussed here: https://github.com/nephio-project/nephio/discussions/450
Migration
porchctl
CLI tool (instead ofkpt alpha
)Clean Up