Skip to content

Conversation

@Shubham9t9
Copy link
Contributor

Description

Fixed all broken images in user-access.md file.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have tested it for all user roles

| `servicePort` | service port for the container |
| `envoyPort` | envoy port for the container. |
| `idleTimeout` | the duration of time that a connection is idle before the connection is terminated. |
| `name` | name of the container. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Name of the port not container

| `idleTimeout` | the duration of time that a connection is idle before the connection is terminated. |
| `name` | name of the container. |
| `port` | port for the container. |
| `servicePort` | service port for the container. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Port of the corresponding kubernetes service

| `port` | port for the container. |
| `servicePort` | service port for the container. |
| `supportStreaming` | Used for high performance protocols like grpc where timeout needs to be disabled. |
| `useHTTP2` | container can accept HTTP2 requests. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Envoy container can accept HTTP2 requests.


### Init Containers
```yaml
initContainers: []
Copy link
Collaborator

Choose a reason for hiding this comment

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

Include Example, it would be almost same as Containers section

```yaml
server:
deployment:
deployment:0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why the 0 ?

TargetCPUUtilizationPercentage: 90
TargetMemoryUtilizationPercentage: 80
```

Copy link
Collaborator

Choose a reason for hiding this comment

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

Line 540 to 547 is not an example of custom metrics.

@prakarsh-dt prakarsh-dt merged commit 840f038 into devtron-labs:main Aug 27, 2021
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.

2 participants