Skip to content
This repository has been archived by the owner on Dec 16, 2024. It is now read-only.

MPAS Docs Update #160

Merged
merged 3 commits into from
Dec 19, 2023
Merged

MPAS Docs Update #160

merged 3 commits into from
Dec 19, 2023

Conversation

phoban01
Copy link
Contributor

@phoban01 phoban01 commented Dec 7, 2023

Description

Update the MPAS documentation site.

Signed-off-by: Piaras Hoban phoban01@gmail.com

Signed-off-by: Piaras Hoban <phoban01@gmail.com>
Copy link

github-actions bot commented Dec 7, 2023

Mend Scan Summary: ❌

Repository: open-component-model/MPAS

VIOLATION DESCRIPTION NUMBER OF VIOLATIONS
HIGH/CRITICAL SECURITY VULNERABILITIES 0
MAJOR UPDATES AVAILABLE 0
LICENSE REQUIRES REVIEW 0
LICENSE RISK HIGH 7
RESTRICTIED LICENSE FOR ON-PREMISE DELIVERY 1

Detailed Logs: mend-scan-> Generate Report
Mend UI

@phoban01 phoban01 marked this pull request as ready for review December 7, 2023 15:50
Copy link
Contributor

@morri-son morri-son left a comment

Choose a reason for hiding this comment

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

just added a link to the ocm.software page in the bootstrap.md

docs/site/src/boostrap.md Outdated Show resolved Hide resolved
kubectl describe productdeployment -n mpas-podinfo-application
```

The result should look something like:
Copy link
Contributor

Choose a reason for hiding this comment

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

Truth is at this point it won't work. Two reasons

  1. The deployed helmRepository has not secret set that would permit pulling the helmcharts. So we need to add a step for the user to make the chart public until we address this shortcoming.
  2. The deployed podinfo deployment doesn't have the pullImageSecret. In order to solve this, the user should update the config.cue to add a serviceAccountName. The name should be the one generated by target.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added a note

docs/site/src/boostrap.md Outdated Show resolved Hide resolved
Signed-off-by: Piaras Hoban <phoban01@gmail.com>
@phoban01 phoban01 force-pushed the phoban01/docs-update branch from 53c85ff to 19d8cf8 Compare December 19, 2023 18:01
@phoban01 phoban01 merged commit 455913b into main Dec 19, 2023
5 checks passed
@phoban01 phoban01 deleted the phoban01/docs-update branch December 19, 2023 18:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants