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

Helm to codegen #3

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Helm to codegen #3

wants to merge 20 commits into from

Conversation

CsatariGergely
Copy link
Member

No description provided.

liamfallon and others added 19 commits February 1, 2024 17:58
…ect#101)

This PR makes minor changes to the landing page of the documentation to
make the language release neutral.
This PR fixes the "ref" and "relref" links in the documentation so that
they work in Markdown and thus Github. This will mean the links will not
work in the Hugo documentation.

We need to figure out a way to get the links working in both but they
really should work in Github.

The links are now all absolute links in order to pass linkspector and to
work in md.

The linkspector tool fails on any link with a '#' in it, that is a link
to a tag in any document. The change to the .linkspector file is to
remove the '#' character and any following text, thus allowing those
links to pass linkspector.
This PR fixes the image links in the markdown files in the Nephio
documentation. Prior to this PR, the images did not appear whent he
markdown was rendered.
This PR fixes some instructions describe in the Free5GC and OAI user
guides, removes some trail spaces, and simplifies the get logs
instructions.

Signed-off-by: Victor Morales <[email protected]>
Installation guide: Update R1 to R2 
OAI exercice: correct editor
- Adding metallb ip-address pool creation 
- Removing the extra white spaces from network diagram
Remove edge1, edge2 and regional repos in the output from `kubectl get
repositories` command

For R2, free5GC tests moved from `test-infra/e2e/tests` to
`test-infra/e2e/tests/free5gc` directory
- Correct path for yaml files
- Correct file name from 003-network.yaml to 002-network.yaml
- Correct file name from 003-secret.yaml to 002-secret.yaml
Signed-off-by: Victor Morales <[email protected]>
Based on this
[comment](https://discourse.gohugo.io/t/replace-image-reference-link/35314/8)
This will make MD cross reference links work in both Github and Hugo
rendered page.
And Images showing in both.


Note the link at the bottom 
Before:
<img width="532" alt="1"
src="https://github.com/nephio-project/docs/assets/37077655/e4925c1b-3417-4148-9600-b13f9533b3ae">
After:
<img width="532" alt="2"
src="https://github.com/nephio-project/docs/assets/37077655/161e1e5a-82ed-481f-b64c-61d1432618c4">

Note the image:
<img width="532" alt="3"
src="https://github.com/nephio-project/docs/assets/37077655/564629d1-be9f-4816-84f4-79761a4955ec">

---------

Signed-off-by: lapentafd <[email protected]>
Renamed the md file.
Deleted extra steps and added single command for install.
Linked to the manual installation of the kpt packages for the common
components.

---------

Signed-off-by: lapentafd <[email protected]>
Co-authored-by: Fiachra Corcoran <[email protected]>
Co-authored-by: Victor Morales <[email protected]>
Co-authored-by: Kushal Harish Naidu <[email protected]>
So far it is an iframe reference to doc.crds.dev. 

In the long run it might be better to use the
[swaggerui](https://www.docsy.dev/docs/adding-content/shortcodes/#swaggerui)
or the
[redoc](https://www.docsy.dev/docs/adding-content/shortcodes/#redoc)
options of Docsy. For these two later options we would need to have the
Open API specs of the API-s somewhere. I have a feeling that at the
moment we have the API-s defined in Go code and not in Open API. Is this
correct @johnbelamaric ?

Signed-off-by: Gergely Csatari <[email protected]>
This is a contributor guide describing how to use Mockery. It draws
heavily on [PR-441](nephio-project/nephio#441)
raised by @vjayaramrh

---------

Co-authored-by: Victor Morales <[email protected]>
Copy link

netlify bot commented Feb 27, 2024

Deploy Preview for nephio-docs-nokia-main ready!

Name Link
🔨 Latest commit 8426793
🔍 Latest deploy log https://app.netlify.com/sites/nephio-docs-nokia-main/deploys/65dd9bad4223600008a33a2b
😎 Deploy Preview https://deploy-preview-3--nephio-docs-nokia-main.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Gergely Csatari <[email protected]>
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.

8 participants