Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: restructure deploy doc #567

Merged
merged 11 commits into from
Jun 26, 2020
Merged

docs: restructure deploy doc #567

merged 11 commits into from
Jun 26, 2020

Conversation

skoh7645
Copy link
Collaborator

@skoh7645 skoh7645 commented Apr 8, 2020

Checklist

  • commit message follows commit guidelines.
  • website builds and runs in production - for information on how to test locally go here.

Description

Restructure "deploying" doc to be clearer

Related Issues

Related to #359
Fixes: appsody/appsody#885

Note: the ### Deploying your application through Docker Hub and ### Deploying your application to a custom registry sections get fixed as part of #566

content/docs/faq.md Outdated Show resolved Hide resolved
Their entry is removed from the `project.yaml` file and the associated volumes are deleted. This validation ensures that volumes exist for current projects only and that volumes are not reused between different projects of the same name.

## 12. How do I deploy my application to a local Minikube cluster?
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this title is right. AFAIK, appsody can deploy to minikube fine - no special config should be needed. Please can you confirm this once?

Copy link
Contributor

Choose a reason for hiding this comment

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

The intent was to document how to use Docker daemon from minikube vs Docker on host.

Their entry is removed from the `project.yaml` file and the associated volumes are deleted. This validation ensures that volumes exist for current projects only and that volumes are not reused between different projects of the same name.

## 12. How do I deploy my application to a local Minikube cluster?
Copy link
Contributor

Choose a reason for hiding this comment

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

The intent was to document how to use Docker daemon from minikube vs Docker on host.

@skoh7645
Copy link
Collaborator Author

For the FAQ section, appsody init is the only command that still works fine with minikube. I'm not sure if we need to document that or if we should just say generally that appsody commands won't work here

@skoh7645 skoh7645 self-assigned this Apr 16, 2020
Copy link
Collaborator

@CameronMcWilliam CameronMcWilliam left a comment

Choose a reason for hiding this comment

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

Much better structure overall, reads much more fluently than previously did.

content/docs/installing/installing-appsody.md Outdated Show resolved Hide resolved
Copy link
Contributor

@neeraj-laad neeraj-laad left a comment

Choose a reason for hiding this comment

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

LGTM

@neeraj-laad neeraj-laad merged commit 06048ef into appsody:master Jun 26, 2020
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.

appsody extract doesn't work when using Minikube docker daemon
4 participants