-
Notifications
You must be signed in to change notification settings - Fork 17
Revision of Edge Cookbook #412
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: main
Are you sure you want to change the base?
Conversation
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.
@chbrandt , I put some comment in line in the review, thanks!
| Device Code URL: "https://github.com/login/device/code" | ||
| Scopes: "read:user" | ||
| ``` | ||
|
|
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 can add that usually oidc providers have a "well_known" url where to get device and token endpoint address
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.
Maybe also in warning or info box
| - Plugin service | ||
| - interLink Kubernetes components | ||
| - Virtual node | ||
| - Pod job |
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.
Testing pod?
|
|
||
| ### Deploy Remote components | ||
| In this scenario, we need to setup 5 components: | ||
| - OIDC/OAuth proxy |
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.
OAuth2 authN proxy
|
|
||
| Yes, if you will, you can also manage all interLink processes through `systemd`. | ||
| Reach out to receive guidance on how we do it in production. You can find an | ||
| example in the interlink repo `./systemd` folder. |
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.
Is this moved somewhere else?
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.
Yes, nvm
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.
Documentation improvements for Edge Cookbook, including clearer sections and additional examples. Good contribution to the documentation quality.
|
Also here, sorry for the hiato. Let me know if this is still relevant. |
Summary
This PR contributes to the docs' edge-node cookbook by
Related issue :