Skip to content

Add docs for dev and ci/cd workflows#3153

Merged
nkubala merged 6 commits intoGoogleContainerTools:masterfrom
nkubala:docs
Nov 4, 2019
Merged

Add docs for dev and ci/cd workflows#3153
nkubala merged 6 commits intoGoogleContainerTools:masterfrom
nkubala:docs

Conversation

@nkubala
Copy link
Copy Markdown
Contributor

@nkubala nkubala commented Nov 1, 2019

Fixes #3074

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 1, 2019

Codecov Report

Merging #3153 into master will not change coverage.
The diff coverage is n/a.

@tejal29 tejal29 added the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment docs-controller-deployment-3153, please visit https://storage.googleapis.com/webhook-logs/logs-3153-1572648605517032097 to view logs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@balopat balopat added the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment docs-controller-deployment-3153, please visit https://storage.googleapis.com/webhook-logs/logs-3153-1572649397939723994 to view logs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@tejal29 tejal29 added the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment docs-controller-deployment-3153, please visit https://storage.googleapis.com/webhook-logs/logs-3153-1572651452840099581 to view logs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@tejal29 tejal29 added the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@container-tools-bot
Copy link
Copy Markdown

Error creating deployment docs-controller-deployment-3153, please visit https://storage.googleapis.com/webhook-logs/logs-3153-1572651987827066368 to view logs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@balopat balopat added the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@balopat balopat added docs-modifications runs the docs preview service on the given PR and removed docs-modifications runs the docs preview service on the given PR labels Nov 1, 2019

## Dev loop

When `skaffold dev` is run, Skaffold will first do a full build and deploy of all artifacts specified in the `skaffold.yaml`, identical behavior to `skaffold run`. Upon successful build and deploy, Skaffold will start watching all source file dependencies for all artifacts specified in the project. As changes are made to these source files, Skaffold will rebuild the associated artifacts, and redeploy the new changes to your cluster.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I dont think we have skaffold run documented any where. we can drop it or link to existing CLI doc on run.

By default, Skaffold uses `fsnotify` to monitor events on the local filesystem. Skaffold also supports a `polling` mode where the filesystem is checked for changes on a configurable interval, or a `manual` mode, where Skaffold waits for user input to check for file changes. These watch modes can be configured through the `--trigger` flag.


## Control API
Copy link
Copy Markdown
Contributor

@tejal29 tejal29 Nov 1, 2019

Choose a reason for hiding this comment

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

I think we should explain trigger types here and not control api.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also, I'm writing a section about the Control API :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I just wanted to have a little section about it here for discoverability, and then link in the full docs to the control API. I mentioned the trigger types above this section. do you think I should remove this?

Copy link
Copy Markdown
Contributor

@balopat balopat Nov 4, 2019

Choose a reason for hiding this comment

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

I'd make it smaller and refer to Skaffold API as an advanced topic for those who want to integrate tools with skaffold dev or have finer grained control and point them to the /docs/design/api docs - it is not essential for the CLI users. WDYT?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I agree, control API is not for CLI user and more of integration for tools.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sounds reasonable, I do think CLI users could still potentially make use of the control API, but it's not a first class feature. I cut down the blurb to just explain what it is and why you might use it, and then pointed to the API docs

@container-tools-bot
Copy link
Copy Markdown

Please visit http://35.235.117.222:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 1, 2019
@balopat balopat added the docs-modifications runs the docs preview service on the given PR label Nov 2, 2019
@container-tools-bot
Copy link
Copy Markdown

Please visit http://35.236.87.39:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 2, 2019
@balopat balopat added the priority/p0 Highest priority. We are actively looking at delivering it. label Nov 4, 2019
@balopat balopat added the docs-modifications runs the docs preview service on the given PR label Nov 4, 2019
@container-tools-bot
Copy link
Copy Markdown

Please visit http://34.94.185.45:1313 to view changes to the docs.

@container-tools-bot container-tools-bot removed the docs-modifications runs the docs preview service on the given PR label Nov 4, 2019
Co-Authored-By: Balint Pato <balopat@users.noreply.github.com>
@balopat
Copy link
Copy Markdown
Contributor

balopat commented Nov 4, 2019

:shipit: 🚢

@nkubala nkubala merged commit 22c108f into GoogleContainerTools:master Nov 4, 2019
@nkubala nkubala deleted the docs branch November 4, 2019 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes priority/p0 Highest priority. We are actively looking at delivering it. size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Complete skaffold workflows documentation

5 participants