-
Notifications
You must be signed in to change notification settings - Fork 506
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: update development.md #6258
base: main
Are you sure you want to change the base?
Conversation
Quality Gate passedIssues Measures |
cc @abhibhaw |
- 20GB+ free disk space | ||
|
||
2. [Install Devtron with CI/CD along with GitOps (Argo CD) - Full mode](https://docs.devtron.ai/install/install-devtron-with-cicd-with-gitops) |
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.
why should I install with full mode? what if I want to test non-full mode feature?
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.
probably this would be too heavy for development that it may not work on most of the pc
Not sure, but we can consider this as well
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.
why should I install with full mode? what if I want to test non-full mode feature?
Good point. I had a thought that if someone is installing full mode he may play more.
|
||
```bash | ||
kubectl -n devtroncd port-forward svc/devtron-nats 4222:4222 |
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.
nats may not be there if not full mode
yeah, but the whole documentation is assumed for full mode. we can create a separate doc for that and hyperlink here.
|
||
```bash | ||
make run |
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.
wire must be installed, need to mention abt that
``` | ||
|
||
This will start your server on `localhost:8080`. |
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.
What to do next?
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.
Do i need to mention API requests as well?
Description
Fixes #6257
Checklist:
Does this PR introduce a user-facing change?