Skip to content

Conversation

@ricardoamaro
Copy link

…troubleshooting steps

Background

Fixes

Change Summary

Additional Notes

Testing Procedure

Related PRs or Issues

@ricardoamaro ricardoamaro requested review from a team and yoshi-approver as code owners December 18, 2025 22:56
@google-cla
Copy link

google-cla bot commented Dec 18, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Member

@bourgeoisor bourgeoisor left a comment

Choose a reason for hiding this comment

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

Left a few comments

Comment on lines +11 to +22
**Install skaffold:**
```sh
# Linux
curl -Lo skaffold https://storage.googleapis.com/skaffold/releases/latest/skaffold-linux-amd64
chmod +x skaffold
sudo mv skaffold /usr/local/bin/

# macOS
brew install skaffold

# Or download from: https://skaffold.dev/docs/install/
```
Copy link
Member

Choose a reason for hiding this comment

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

I would rather not have these instructions here, since we'd be duplicating a source of truth that already exists (which is linked in the doc: https://skaffold.dev/docs/install/)

Comment on lines +143 to +146
**If you encounter build errors** (see Troubleshooting section below), use pre-built images instead:
```shell
kubectl apply -f ./release/kubernetes-manifests.yaml
```
Copy link
Member

Choose a reason for hiding this comment

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

I would remove this text. This doc is for developing on Bank of Anthos, so pulling pre-built images defeats the purpose of this guide)

@bourgeoisor bourgeoisor changed the title update development guide with skaffold installation instructions and … Update development guide Dec 31, 2025
@bourgeoisor
Copy link
Member

@ricardoamaro you will also have to sign the CLA. See: #3185 (comment)

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.

2 participants