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

Added NODE_SIDE_TYPE flag to helm chart #8293

Merged
merged 11 commits into from
Dec 5, 2023

Conversation

PeterChung241
Copy link
Member

No description provided.

@PeterChung241 PeterChung241 changed the title [WIP] Added NODE_SIDE_TYPE flag to helm chart Added NODE_SIDE_TYPE flag to helm chart Nov 30, 2023
@madhavajay
Copy link
Collaborator

@PeterChung241 Include in helm values.yaml template. Check helm.py, then test with helm install --arg

@shubham3121
Copy link
Member

Example: helm install <node_name> <repo> --set nodeSideType=low/high

@rasswanth-s
Copy link
Collaborator

rasswanth-s commented Dec 5, 2023

Command: Low Side 🔅

helm install syft-gateway ./syft --set node.settings.nodeType="gateway" --set node.settings.nodeName="un-petlab" --set node.settings.nodeSideType="low"

Screenshot 2023-12-05 at 8 16 13 AM

@rasswanth-s
Copy link
Collaborator

Command: High Side 🔆

helm install syft-domain ./syft --set node.settings.nodeType="domain" --set node.settings.nodeName="canada" --set node.settings.nodeSideType="high"

Screenshot 2023-12-05 at 8 28 26 AM

Copy link
Collaborator

@rasswanth-s rasswanth-s left a comment

Choose a reason for hiding this comment

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

@shubham3121 shubham3121 enabled auto-merge December 5, 2023 03:14
@shubham3121 shubham3121 merged commit 0def89a into OpenMined:dev Dec 5, 2023
27 checks passed
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.

4 participants