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

Improve UX of installation/setup scripts #118

Open
3 tasks
avinal opened this issue Jun 18, 2022 · 6 comments
Open
3 tasks

Improve UX of installation/setup scripts #118

avinal opened this issue Jun 18, 2022 · 6 comments
Labels

Comments

@avinal
Copy link
Member

avinal commented Jun 18, 2022

Currently, there are many tradeoffs and missing scenarios in the installation and setup scripts. This issue will track the overall progress of improving the user experience. Please update the list as needed.

  • Clean the containers in case of failure
  • Stop running kcp processes in case of failure
  • Add parameter/environment variable to enable compilation of kcp binaries when not found
@fgiloux
Copy link
Contributor

fgiloux commented Jun 20, 2022

Note: This is for local development environment.

@adambkaplan
Copy link
Contributor

Some other notes from my "local development" experiences:

  1. kcp in a pod requests a full CPU. This often means that kcp can't be scheduled given that we install a lot of other components that are resource-heavy.
  2. Our script simply stops when kcp fails to deploy, with no error message.

@adambkaplan
Copy link
Contributor

More notes:

  • After overcoming ckcp install issues, I struggle with the "what's next" step. Our current README docs aren't exactly clear on this.
  • The setup scripts default to laying out credentials for KCP and the compute cluster in a temporary directory. Don't developers need these to do any real work? Should we change the location of this data to be somewhere that is more local to the script?

Roming22 pushed a commit that referenced this issue Dec 19, 2022
Svpi61 vault init
Roming22 pushed a commit that referenced this issue Dec 19, 2022
Svpi61 vault init
@Roming22
Copy link
Contributor

@avinal A lot changed since you opened this issue, and many improvement were made. I'll mark this issue as stale.
Please put a comment if you still have pain points that need to be looked after.

@Roming22 Roming22 added the stale label Feb 22, 2023
@avinal
Copy link
Member Author

avinal commented Feb 22, 2023

Sure @Roming22 I don't think this issue is relevant now. We should close it.

@adambkaplan
Copy link
Contributor

/close

Per @avinal's comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants