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
feat: Add node pool parameter #802
feat: Add node pool parameter #802
Conversation
… This will perform any modifications needed to replace types with runtime data, etc. An example is select.nodepool. This value should set the options to the currently available node types.
…logic to replace select.nodepool values with runtime values.
db/yaml/maskrcnn/20201221195937.yaml
Outdated
ports: | ||
- containerPort: 6006 | ||
name: tensorboard | ||
sidecars: |
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.
This indentation seems incorrect - it should line up with nodeSelector
db/yaml/maskrcnn/20201221195937.yaml
Outdated
@@ -11,13 +11,13 @@ arguments: | |||
value: annotation-dump/sample_dataset |
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.
Bit confused about this file, are you deleting an old migration file here?
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.
@rushtehrani This migration adds the select.nodepool
parameter
db/yaml/tfod/README.md
Outdated
@@ -0,0 +1 @@ | |||
https://github.com/onepanelio/templates/blob/master/workflows/maskrcnn-training/template.yaml |
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.
don't think we need this README here
…match onepanel templates repo
…sources for migrations. Also updated migration yamls to the latest based on templates repo.
@Vafilor can you address the conflict? |
… This will perform any modifications needed to replace types with runtime data, etc. An example is select.nodepool. This value should set the options to the currently available node types.
…logic to replace select.nodepool values with runtime values.
…match onepanel templates repo
…sources for migrations. Also updated migration yamls to the latest based on templates repo.
@rushtehrani Sure thing - should be good. |
What this PR does:
Which issue(s) this PR fixes:
Fixes onepanelio/core#
Special notes for your reviewer:
Checklist
Please check if applies
Required