-
Notifications
You must be signed in to change notification settings - Fork 121
Add deployment guide (GCP) #239
Add deployment guide (GCP) #239
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@izellevy Please see a few comments
|
||
```bash | ||
# Pull the Canopy Docker image | ||
docker pull ghcr.io/pinecone-io/canopy:{canopy-version} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be :latest
, and you can mention that we also have individual version images
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think latest is not a good practice in general for deployments. (It is handy to check the functionality though.) More context here: https://vsupalov.com/docker-latest-tag/#:~:text=You%20should%20avoid%20using%20the,apart%20from%20the%20image%20ID.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Problem
We don't have a guide for Cloud deployments.
Solution
Added a guide for GCP.
Type of Change
Test Plan
Describe specific steps for validating this change.