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

Adding versioning support to R2 #132

Closed
wants to merge 16 commits into from

Conversation

CsatariGergely
Copy link
Contributor

No description provided.

electrocucaracha and others added 15 commits February 7, 2024 09:39
Signed-off-by: Victor Morales <v.morales@samsung.com>
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 <francesco.lapenta@est.tech>
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 <francesco.lapenta@est.tech>
Co-authored-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
Co-authored-by: Victor Morales <chipahuac@hotmail.com>
Co-authored-by: Kushal Harish Naidu <159911459+kushnaidu@users.noreply.github.com>
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 <gergely.csatari@nokia.com>
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 <chipahuac@hotmail.com>
- Focus more on that this is only the source of the documentation
- Replace GitHub link with docs.nephio.org ones
- Add Netlify build status

---------

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com>
Co-authored-by: Victor Morales <chipahuac@hotmail.com>
The current pr contains the user-guide as well as developer-guide for
helm-to-operator-codegen-sdk which is a part of nephio sdk
(https://github.com/nephio-project/nephio-sdk)

---------

Co-authored-by: Gergely Csatari <gergely.csatari@nokia.com>
Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
As discussed in the architecture meeting on Friday, and again in the SIG
Network Architecture meeting today, here is the PR for the architecture
documentation in C4.

---------

Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>
Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com>
Co-authored-by: Alexis de Talhouët <adetalhouet89@gmail.com>
According to playbook in test-infra for provisioning, enabling or
disabling KinD cluster shall be achieved by
`kind.enabled` not `kind.enable`
([reference](https://github.com/nephio-project/test-infra/blob/2e8de787693dcfb07f0d2e4088d3ed3ccdac1250/e2e/provision/playbooks/roles/bootstrap/README.md?plain=1#L27)).

However, the document had `kind.enable` which sets up a KinD cluster
that overwrites the current kubeconfig which I had. This seems not to be
an expected behavior.
This PR is the result of the discussion on Nephio TSC channel regarding
the docs licensing. It was confirmed that CC-BY-4.0 license had to be
used and two things had to be changed:

1. Primary LICENSE file
2. SPDX identifier in code files

Quoting verbatim:
---Jenn Bonner (on 21st March 2024)---
Thread regarding the Docs repo incorrect license.
I've heard back from LF Legal. They've done a thorough review, and
recommend the following steps:

1. replace the Apache-2.0 license information in the Documentation repo
2. change all the SPDX identifiers for documentation int hat repo to
CC-BY-4.0 from Apache-2.0
3. Additionally, please have EasyCLA changed so that the documentation
repo is not included in EasyCLA. [DONE]
---

---------

Signed-off-by: Rahul Jadhav <nyrahul@gmail.com>
After some reviews in the modelling (WG4) meetings somewhat of a
consensus has been reached on a view of the Nephio models, and this is
the attempt to describe diagrammatically at a hight level what is
currently there.

---------

Co-authored-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
@nephio-prow nephio-prow bot requested a review from henderiw June 3, 2024 13:51
Copy link
Contributor

nephio-prow bot commented Jun 3, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign efiacor for approval by writing /assign @efiacor in a comment. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nephio-prow nephio-prow bot requested a review from s3wong June 3, 2024 13:51
Copy link

netlify bot commented Jun 3, 2024

Deploy Preview for nephio-r2 ready!

Name Link
🔨 Latest commit a6d278c
🔍 Latest deploy log https://app.netlify.com/sites/nephio-r2/deploys/666149ddea827e0008449849
😎 Deploy Preview https://deploy-preview-132--nephio-r2.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.

our [Learning Nephio with R1](https://wiki.nephio.org/display/HOME/Learning+with+Nephio+R1) series, which includes a
[demo video](https://youtu.be/mFl71sy2Pdc) and short articles about different aspects of Nephio R1.
{{% blocks/feature icon="fas fa-rocket" title="R2" %}}
Welcome to the R2 release of Nephio. Please use the documentation links below to learn more about Nephio R2.
{{% /blocks/feature %}}

{{% blocks/feature icon="fab fa-wikipedia-w" title="Wiki" %}}
Copy link
Contributor

Choose a reason for hiding this comment

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

@CsatariGergely It might be a bit out of the scope of this MR, but I suppose we can change the description of the below link from "Mass amount of developer information is available in the [wiki]" to "To know about different Sigs in Nephio you can check out this [wiki]".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is outside the scope as with this change I remove all references to the wiki :)

our [Learning Nephio with R1](https://wiki.nephio.org/display/HOME/Learning+with+Nephio+R1) series, which includes a
[demo video](https://youtu.be/mFl71sy2Pdc) and short articles about different aspects of Nephio R1.
{{% blocks/feature icon="fas fa-rocket" title="R2" %}}
Welcome to the R2 release of Nephio. Please use the documentation links below to learn more about Nephio R2.
Copy link
Contributor

Choose a reason for hiding this comment

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

@CsatariGergely Isn't using R2 and release at the same time repetitive?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. In my head we have a release what is called R2, but I'm happy to change this if it is not clear like this.

How about Welcome to "R2 of Nephio" or "Welcome to release 2 (R2) of Nephio"?

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for the late reply, "How about Welcome to Release 2 of Nephio" If not then we can use the first one "Welcome to R2 of Nephio"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 9116e71 in #133

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Copy link
Contributor

nephio-prow bot commented Jun 6, 2024

Keywords which can automatically close issues and at(@) or hashtag(#) mentions are not allowed in commit messages.

The list of commits with invalid commit messages:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@CsatariGergely
Copy link
Contributor Author

Darn, I was mistakenly merging main in to this R2 pr.

@CsatariGergely
Copy link
Contributor Author

I've created #133 with the same content plus fix for #132 (review) except the main branch merge.

nephio-prow bot pushed a commit that referenced this pull request Jun 17, 2024
Redoing #132 now without
merging the main branch to the R2 one.

Partly fixes nephio-project/nephio#738

---------

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.