search

vmware-tanzu / cartographer

Cartographer is a Supply Chain Choreographer.
https://cartographer.sh
Apache License 2.0
447 stars 64 forks source link

Unable to run make deploy-local #85

Closed emmjohnson closed 3 years ago

emmjohnson commented 3 years ago 
make deploy-local

ytt -f ./config -f local-dev/local-registry.yaml --ignore-unknown-comments | kbld --images-annotation -f - | kapp deploy --yes -a cartographer -f -
Target cluster 'https://127.0.0.1:62814' (nodes: cartographer-control-plane)
ytt: Error: Overlaying (in following order: local-registry.yaml):
  Document on line local-registry.yaml:18:
    Expected number of matched nodes to be 1, but was 0

If I remove line 18 from local-dev/local-registry.yaml and run the ytt command - 

ytt -f ./config -f local-dev/local-registry.yaml --ignore-unknown-comments | kbld --images-annotation -f - | kapp deploy --yes -a cartographer -f -                                                                                             emily.johnson@emjohnson-a01
Target cluster 'https://127.0.0.1:62814' (nodes: cartographer-control-plane)
kbld: Error: 
- Resolving image 'ko://github.com/vmware-tanzu/cartographer/cmd/cartographer': Get "https://ko/v2/": dial tcp: lookup ko on 10.16.188.201:53: no such host
- Resolving image 'controller-image': GET https://index.docker.io/v2/library/controller-image/manifests/latest: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:library/controller-image Type:repository]]

This is as far as I got debugging.

cirocosta commented 3 years ago

oh yeah without having a registry hardcoded anymore, we don't need to have an overlay file for forcing the push to a local registry as we can se that with KO_DOCKER_REPO

this was my bad - in #1 I modified manager/deployment.yaml to not include kbld's config anymore (as there's no need for it anymore) but forgot about the local-dev scripts

i'll assign myself to fix it