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

Minimal install #69

Merged
merged 23 commits into from
Nov 2, 2023
Merged

Minimal install #69

merged 23 commits into from
Nov 2, 2023

Conversation

liamfallon
Copy link
Member

No description provided.

This commit creates the Nephio developer-guide subdirectory and starts
the developer guide with instructions on how to do a minimal local
install. This minimal install is enough to get the Nephio operators
running in vscode towards a local Kind cluster with the minimal Nephio
dependencies installed there.

Signed-off-by: liamfallon <liam.fallon@est.tech>
This commit creates the Nephio developer-guide subdirectory and starts
the developer guide with instructions on how to do a minimal local
install. This minimal install is enough to get the Nephio operators
running in vscode towards a local Kind cluster with the minimal Nephio
dependencies installed there.

Signed-off-by: liamfallon <liam.fallon@est.tech>
@nephio-prow nephio-prow bot requested review from s3wong and tliron October 26, 2023 11:50
@liamfallon
Copy link
Member Author

/assign @electrocucaracha @efiacor @lapentad

Copy link
Contributor

@lapentad lapentad left a comment

Choose a reason for hiding this comment

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

/approve

Nephio Developer Guide
======================

The developer guide is for anyone wanting to either write code for the Nephio project. THis document is written as an extension of the [Kubernetes Developer Guides](https://github.com/kubernetes/community/tree/master/contributors/devel#the-process-of-developing-and-contributing-code-to-the-kubernetes-project) and therefore covers topics that are specific to Nephio development.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The developer guide is for anyone wanting to either write code for the Nephio project. THis document is written as an extension of the [Kubernetes Developer Guides](https://github.com/kubernetes/community/tree/master/contributors/devel#the-process-of-developing-and-contributing-code-to-the-kubernetes-project) and therefore covers topics that are specific to Nephio development.
The developer guide is for anyone wanting to either write code for the Nephio project. This document is written as an extension of the [Kubernetes Developer Guides](https://github.com/kubernetes/community/tree/master/contributors/devel#the-process-of-developing-and-contributing-code-to-the-kubernetes-project) and therefore covers topics that are specific to Nephio development.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done


The developer guide is for anyone wanting to either write code for the Nephio project. THis document is written as an extension of the [Kubernetes Developer Guides](https://github.com/kubernetes/community/tree/master/contributors/devel#the-process-of-developing-and-contributing-code-to-the-kubernetes-project) and therefore covers topics that are specific to Nephio development.

* How to set up a [Minimal Development Environment]minimal-environment.md) defines common terminology used in the Nephio project.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* How to set up a [Minimal Development Environment]minimal-environment.md) defines common terminology used in the Nephio project.
* How to set up a [Minimal Development Environment](minimal-environment.md) defines common terminology used in the Nephio project.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

Minimal Environment install for development
===========================================

The following environment install works on a Macbook Pro M1 to get the nephio-operators running in vscode talking to a local kind cluster running in Docker. Note that depending on what aprt of Nephio you are working on, you may wish to install less or more of the components below. It should work on other environments with appropriate tweaking.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The following environment install works on a Macbook Pro M1 to get the nephio-operators running in vscode talking to a local kind cluster running in Docker. Note that depending on what aprt of Nephio you are working on, you may wish to install less or more of the components below. It should work on other environments with appropriate tweaking.
The following environment install works on a Macbook Pro M1 to get the nephio-operators running in vscode talking to a local kind cluster running in Docker. Note that depending on what part of Nephio you are working on, you may wish to install less or more of the components below. It should work on other environments with appropriate tweaking.

Copy link
Member Author

Choose a reason for hiding this comment

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

done


## Installation steps:

1. Install Docker using the appropriate method for you system.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
1. Install Docker using the appropriate method for you system.
1. [Install Docker](https://docs.docker.com/engine/install/) using the appropriate method for you system.

Copy link
Member Author

Choose a reason for hiding this comment

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

done


1. Install Docker using the appropriate method for you system.

2. Install kind using the appropriate method for you system (homebrew on mac)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
2. Install kind using the appropriate method for you system (homebrew on mac)
2. [Install kind](https://kind.sigs.k8s.io/docs/user/quick-start/#installation) using the appropriate method for you system (homebrew on mac)

Copy link
Member Author

Choose a reason for hiding this comment

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

done

Comment on lines 43 to 45
```sh
brew install kind
```
Copy link
Member

Choose a reason for hiding this comment

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

Providing a pointer to the kind installation, the following instruction can be omitted

Suggested change
```sh
brew install kind
```

Copy link
Member Author

Choose a reason for hiding this comment

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

Done


# Connect to Gitea on your browser

COnnecting to Gitea allows you to see the actions that Nephio takes on Gitea.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
COnnecting to Gitea allows you to see the actions that Nephio takes on Gitea.
Connecting to Gitea allows you to see the actions that Nephio takes on Gitea.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@liamfallon
Copy link
Member Author

@electrocucaracha thanks for the review and sorry for all the typos.

@electrocucaracha
Copy link
Member

/lgtm

@nephio-prow nephio-prow bot added the lgtm label Oct 27, 2023
1. [Install Docker](https://docs.docker.com/engine/install/) using the appropriate method for you system.

2. [Install kind](https://kind.sigs.k8s.io/docs/user/quick-start/#installation) using the appropriate method for you system (homebrew on mac)

Copy link
Contributor

Choose a reason for hiding this comment

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

I think a step is needed here to install kpt - it's not included in the referenced Kubernetes local development guide so should be here, I think.

Copy link
Contributor

Choose a reason for hiding this comment

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

e.g.

3. [Install kpt cli](https://kpt.dev/installation/kpt-cli) using the appropriate method for your system (e.g. homebrew on mac)

Copy link
Member Author

Choose a reason for hiding this comment

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

I added this step, thanks @tomkivlin for the review.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @liamfallon

Co-authored-by: Tom Kivlin <52716470+tomkivlin@users.noreply.github.com>
@nephio-prow nephio-prow bot removed the lgtm label Oct 31, 2023
liamfallon and others added 4 commits October 31, 2023 14:47
Co-authored-by: Tom Kivlin <52716470+tomkivlin@users.noreply.github.com>
Co-authored-by: Tom Kivlin <52716470+tomkivlin@users.noreply.github.com>
@gvbalaji
Copy link
Contributor

gvbalaji commented Nov 1, 2023

/approve

@johnbelamaric
Copy link
Member

/approve
/lgtm

Nice!

@gvbalaji
Copy link
Contributor

gvbalaji commented Nov 1, 2023

@liamfallon thanks a lot. This requires a rebase before merging.

@nephio-prow nephio-prow bot removed the lgtm label Nov 2, 2023
Copy link
Contributor

@efiacor efiacor left a comment

Choose a reason for hiding this comment

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

A few minor comments.

developer-guide/README.md Outdated Show resolved Hide resolved
Minimal Environment install for development
===========================================

The following environment install works on a Macbook Pro M1 or via SSH to Ubuntu 22.04 to get the nephio-operators running in vscode talking to a local kind cluster running in Docker. Note that depending on what part of Nephio you are working on, you may wish to install less or more of the components below. It should work on other environments with appropriate tweaking.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need a dev machine spec table?

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe later, I don't think we need it right now.

developer-guide/minimal-environment.md Show resolved Hide resolved
liamfallon and others added 11 commits November 2, 2023 11:17
This commit creates the Nephio developer-guide subdirectory and starts
the developer guide with instructions on how to do a minimal local
install. This minimal install is enough to get the Nephio operators
running in vscode towards a local Kind cluster with the minimal Nephio
dependencies installed there.

Signed-off-by: liamfallon <liam.fallon@est.tech>
Co-authored-by: Tom Kivlin <52716470+tomkivlin@users.noreply.github.com>
Co-authored-by: Tom Kivlin <52716470+tomkivlin@users.noreply.github.com>
Co-authored-by: Tom Kivlin <52716470+tomkivlin@users.noreply.github.com>
@efiacor
Copy link
Contributor

efiacor commented Nov 2, 2023

/approve

Copy link
Contributor

nephio-prow bot commented Nov 2, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: efiacor, gvbalaji, johnbelamaric, lapentad

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

The pull request process is described 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

@liamfallon
Copy link
Member Author

Can I get a LGTM on this please?

@electrocucaracha
Copy link
Member

/lgtm

@nephio-prow nephio-prow bot added the lgtm label Nov 2, 2023
@nephio-prow nephio-prow bot merged commit db1fb80 into nephio-project:main Nov 2, 2023
3 checks passed
@liamfallon liamfallon deleted the minimal-install branch January 30, 2024 14:57
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.

7 participants