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.

Sign up for GitHub

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

Jump to bottom

feat: K8s Manifest Operator #266

Open
zccz14 opened this issue Nov 22, 2023 · 1 comment · Fixed by #273
Open

feat: K8s Manifest Operator #266

zccz14 opened this issue Nov 22, 2023 · 1 comment · Fixed by #273
Assignees
@Thrimbda
Labels
enhancement New feature or request

Comments

@zccz14
Copy link
Contributor

zccz14 commented Nov 22, 2023 •
edited
Loading

We need a k8s operator to simplify deployment.

  • define a CRD named manifest.yuan.ntnl.io.
  • deploy a controller in the k8s cluster, and watch manifest resources

Once a manifest resource is created:

  1. download the npm package
  2. load the extension bundle js
  3. make k8s resources
  4. add owner-reference property and refer to the manifest resource.
  5. create the resources in the k8s cluster.

Others:

  • Support config NPM_REGISTRY for other npm registry.
  • For security reasons, only accept trusted packages. (avoid remote code execution)
@zccz14 zccz14 converted this from a draft issue Nov 22, 2023
@zccz14 zccz14 moved this from Todo to In Progress in Yuan Public WorkItem Nov 22, 2023
@zccz14 zccz14 added the enhancement New feature or request label Nov 22, 2023
@zccz14 zccz14 linked a pull request Nov 24, 2023 that will close this issue
feat: add k8s manifest operator #273
Merged
@github-project-automation github-project-automation bot moved this from In Progress to Done in Yuan Public WorkItem Nov 24, 2023
@zccz14 zccz14 reopened this Nov 24, 2023
@zccz14
Copy link
Contributor Author

zccz14 commented Nov 24, 2023

I reopened this issue because #273 misses some features:

  • Support config NPM_REGISTRY for other npm registries.
  • For security reasons, only accept trusted packages. (avoid remote code execution)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Thrimbda Thrimbda

Labels
enhancement New feature or request
Projects
Yuan Public WorkItem
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add k8s manifest operator Thrimbda/Yuan
2 participants
@zccz14 @Thrimbda