Skip to content

selkies image streaming check#45

Open
deepak7093 wants to merge 4 commits into
selkies-project:masterfrom
deepak7093:master
Open

selkies image streaming check#45
deepak7093 wants to merge 4 commits into
selkies-project:masterfrom
deepak7093:master

Conversation

@deepak7093

Copy link
Copy Markdown
Contributor
  • Added GKE image streaming pre-check for vdi images

Comment thread images/cloudbuild.yaml Outdated

apt install jq -y
export DOCKER_CLI_EXPERIMENTAL=enabled
curl -o check_docker_image.sh https://gist.githubusercontent.com/deepak7093/17a0ce3506c39bae0ecaba46d0821510/raw/a2e63872ccb9ef02b718156a2b5c7744221830c5/check_docker_image.sh

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Move this script into the repo, next to the cloudbuild.yaml file

Comment thread images/cloudbuild.yaml
./setup/scripts/check_docker_image.sh -i gcr.io/${PROJECT_ID}/${_IMAGE_PREFIX}-gst-webrtc-app:${_IMAGE_TAG} || true
waitFor:
- gst-webrtc-app-pull
- gst-webrtc-app-pull

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

extra whitespace

Comment thread images/cloudbuild.yaml
./setup/scripts/check_docker_image.sh -i gcr.io/${PROJECT_ID}/${_IMAGE_PREFIX}-watchdog:${_IMAGE_TAG} || true
waitFor:
- watchdog-pull
- watchdog-pull

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

whitespace

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