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(jcloud): add storage section #5831

Merged
merged 4 commits into from
Apr 28, 2023
Merged

docs(jcloud): add storage section #5831

merged 4 commits into from
Apr 28, 2023

Conversation

npitsillos
Copy link
Contributor

Goals:

  • Fixes naming for components in jcloud

  • Adds missing storage section

  • Fixes description of target in autoscaling

  • check and update documentation. See guide and ask the team.

@github-actions github-actions bot added size/S area/docs This issue/PR affects the docs labels Apr 25, 2023
@JoanFM JoanFM requested a review from alexcg1 April 25, 2023 15:17
Comment on lines 231 to 234
| min | 1 | int | Minimum number of replicas (`0` means serverless) |
| max | 2 | int, up to 5 | Maximum number of replicas |
| metric | concurrency | `concurrency` / `rps` | Metric for scaling |
| target | 100 | int | Target value of `metric` after which replicas autoscale |
Copy link
Member

Choose a reason for hiding this comment

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

some stuff should likely be wrapped in backticks here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could you be more specific here @alexcg1 which parts should be wrapped?

docs/concepts/jcloud/configuration.md Outdated Show resolved Hide resolved
@JoanFM JoanFM requested a review from alexcg1 April 26, 2023 17:15
docs/concepts/jcloud/configuration.md Outdated Show resolved Hide resolved
@JoanFM JoanFM requested a review from alexcg1 April 27, 2023 09:02
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
@github-actions
Copy link

📝 Docs are deployed on https://docs-jcloud-storage--jina-docs.netlify.app 🎉

@JoanFM JoanFM merged commit ecd8605 into master Apr 28, 2023
@JoanFM JoanFM deleted the docs-jcloud-storage branch April 28, 2023 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs This issue/PR affects the docs size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants