-
Notifications
You must be signed in to change notification settings - Fork 68
Update AKO online installation docs #2042
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
base: master
Are you sure you want to change the base?
Conversation
|
No JIRA Ids found for the PR. Jira id is mandatory to update fix version in jira. Please update respective Jira id in PR title or commit message if the PR is intented for default branches of repo. For mandatory exemptions, comment trigger phrase 'skip jira-id-check' in PR. For manual trigger, comment trigger phrase 'run jira-id-check' in PR. |
1 similar comment
|
No JIRA Ids found for the PR. Jira id is mandatory to update fix version in jira. Please update respective Jira id in PR title or commit message if the PR is intented for default branches of repo. For mandatory exemptions, comment trigger phrase 'skip jira-id-check' in PR. For manual trigger, comment trigger phrase 'run jira-id-check' in PR. |
|
skip jira-id-check |
docs/openshift/openshift_helm.md
Outdated
| helm pull oci://projects.packages.broadcom.com/ako/helm-charts/ako --version 2.1.1 --untar | ||
| helm show chart oci://projects.packages.broadcom.com/ako/helm-charts/ako --version 2.1.1 | ||
| Pulled: projects.packages.broadcom.com/ako/helm-charts/ako:2.1.1 | ||
| Digest: sha256:4471ebdb08c87cd48715148120f9ed8a56527274dfe63f593e87ecf604d23c9c |
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.
we should redact the sha value
docs/install/helm.md
Outdated
| ``` | ||
| helm pull oci://projects.packages.broadcom.com/ako/helm-charts/ako --version 2.2.1 --untar | ||
| ``` | ||
| Step 3: Use the values.yaml from this chart to edit values related to the Avi Load Balancerconfiguration. To get the values.yaml for a release, run the following command: |
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.
nit: add space between Balancer and configuration
| **Note**: Starting from AKO-2.2.1, the AKO Helm chart has a dependency chart `ako-crd-operator`. The installation can be enabled/disabled by setting `ako-crd-operator.enabled` in the AKO values.yaml. | ||
| **Note**: Starting from AKO-2.1.1, the AKO Helm chart has a dependency chart `ako-crd-operator`. The installation can be enabled/disabled by setting `ako-crd-operator.enabled` in the AKO values.yaml. | ||
|
|
||
| Step 5: Install AKO |
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.
There are no new line when viewed between L44 and L45.
97e368e to
a451034
Compare
|
No JIRA Ids found for the PR. Jira id is mandatory to update fix version in jira. Please update respective Jira id in PR title or commit message if the PR is intented for default branches of repo. For mandatory exemptions, comment trigger phrase 'skip jira-id-check' in PR. For manual trigger, comment trigger phrase 'run jira-id-check' in PR. |
|
ako_ha also need wording changes |
a451034 to
397d561
Compare
|
No JIRA Ids found for the PR. Jira id is mandatory to update fix version in jira. Please update respective Jira id in PR title or commit message if the PR is intented for default branches of repo. For mandatory exemptions, comment trigger phrase 'skip jira-id-check' in PR. For manual trigger, comment trigger phrase 'run jira-id-check' in PR. |
ashishnayak96
left a comment
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.
LGTM
|
skip jira-id-check |
Modified doc as installation for 2.1.1 is changed.