-
Notifications
You must be signed in to change notification settings - Fork 560
add self hosting docs for azure #1954
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
Conversation
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Free review on us!CodeRabbit is offering free reviews until Wed May 28 2025 to showcase some of the refinements we've made. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
PR Summary
Added comprehensive documentation for self-hosting Digger on Azure, including Helm chart installation, GitHub App setup, and Azure OIDC authentication configuration.
- File paths in code examples need to be corrected (e.g.,
path/to/terraform/dir
should be more specific) - Default credentials in Helm values (
abc123
,salkfjadslkfj
) should be replaced with secure placeholder values - Inconsistent file naming between
mydigger.yml
anddigger.yml
in Helm upgrade command - Screenshot file paths are incorrect (e.g.,
/docs/images/Screenshot2025-05-26at19.00.55.png
doesn't exist) - Azure storage account name in Terraform example should use a placeholder to avoid conflicts
💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!
1 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile
|
||
httpBasicAuthUsername: "admin" | ||
httpBasicAuthPassword: "abc123" | ||
bearerAuthToken: "salkfjadslkfj" # You should generate |
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.
logic: Default password 'abc123' and bearer token 'salkfjadslkfj' should not be included in docs as they may be copied directly. Use placeholders like '<your-secure-password>' instead
bearerAuthToken: "salkfjadslkfj" # You should generate | |
bearerAuthToken: "<your-secure-token>" # Generate a secure random token |
Next is to update the helmchart: | ||
|
||
``` | ||
$ helm upgrade mydigger digger/digger-backend -f digger.yml |
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.
logic: File reference inconsistency: Command uses 'digger.yml' but earlier example uses 'mydigger.yml'
$ helm upgrade mydigger digger/digger-backend -f digger.yml | |
$ helm upgrade mydigger digger/digger-backend -f mydigger.yml |
|
||
## Configure Github workflow: | ||
|
||
create this github workflow in your repository under `.github/workflow/digger_workflow.yml` |
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.
syntax: Incorrect path: '.github/workflow/' should be '.github/workflows/'
create this github workflow in your repository under `.github/workflow/digger_workflow.yml` | |
create this github workflow in your repository under `.github/workflows/digger_workflow.yml` |
No description provided.