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

21 modular install #36

Merged
merged 20 commits into from
Apr 25, 2024
Merged

21 modular install #36

merged 20 commits into from
Apr 25, 2024

Conversation

JoaquinRivesGambin
Copy link
Contributor

@JoaquinRivesGambin JoaquinRivesGambin commented Apr 18, 2024

This solves the issue the "Recommended number of CPUs required to run the pipeline too high". Now, the users can choose different installation options:

Please choose the deployment option:
[1] Kubeflow (all components)
[2] Kubeflow (without monitoring)
[3] Standalone KFP
[4] Standalone KFP (without monitoring)
[5] Standalone KFP and Kserve
[6] Standalone KFP and Kserve (without monitoring)
Enter the number of your choice [1-6] (default is [1]):

Other improvements:

  • changed cluster name from old kind-kind-ep to kind-mlops-platform
  • added uninstall.sh script to remove the cluster and the local docker repository if present
  • improved the message error of the infinite loop:
  Retrying to apply resources.
  Be patient, this might take a while... (Errors are expected until all resources are available!)
  
  Help:
    If the errors persists, please check the pods status with: kubectl get pods --all-namespaces
    All pods should be either in Running state, or ContainerCreating if they are still starting up.
    Check specific pod errors with: kubectl describe pod -n [NAMESPACE] [POD_NAME]
  echo "  For further help, see the Troubleshooting section in setup.md"
  echo

Copy link
Collaborator

@dmuiruri dmuiruri left a comment

Choose a reason for hiding this comment

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

Typo on "University" of Helsinki
If a link to the research group is desired: (https://www.helsinki.fi/en/researchgroups/empirical-software-engineering)

@JoaquinRivesGambin
Copy link
Contributor Author

Typo on "University" of Helsinki If a link to the research group is desired: (https://www.helsinki.fi/en/researchgroups/empirical-software-engineering)

Good catch, I will fix and add the link

@K123AsJ0k1
Copy link
Collaborator

K123AsJ0k1 commented Apr 18, 2024

Setup.sh options 3-6 tested on a laptop. I wonder if setup.md should list the requested amount of CPU and RAM for these options or atleast mention that these can be checked with:

kubectl describe nodes

K123AsJ0k1
K123AsJ0k1 previously approved these changes Apr 18, 2024
Copy link
Collaborator

@K123AsJ0k1 K123AsJ0k1 left a comment

Choose a reason for hiding this comment

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

Setup.sh (options 3-6 tested) and uninstall.sh work as expected, while file additions and notebooks look good, so approved.

@JoaquinRivesGambin
Copy link
Contributor Author

Setup.sh options 3-6 tested on a laptop. I wonder if setup.md should list the requested amount of CPU and RAM for these options or atleast mention that these can be checked with:

kubectl describe nodes

Sure, I will add the recommended CPU and memory also on the setup README

@dmuiruri
Copy link
Collaborator

Tested all the modular installation configurations [1]- [6] on the CSC environment, and they were successful

@dmuiruri dmuiruri merged commit a7356c8 into main Apr 25, 2024
@JoaquinRivesGambin JoaquinRivesGambin deleted the 21-modular-install branch May 29, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Recommended number of CPUs required to run the pipeline too high
3 participants