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

Rancher Desktop Installation not detected - ide is trying to install #1030

Closed
maybeec opened this issue Jan 3, 2023 · 3 comments · Fixed by #1045
Closed

Rancher Desktop Installation not detected - ide is trying to install #1030

maybeec opened this issue Jan 3, 2023 · 3 comments · Fixed by #1045
Assignees
Labels
bug Something isn't working software software-package with 3rd party products

Comments

@maybeec
Copy link
Member

maybeec commented Jan 3, 2023

Expected behavior

As a user, I want to use docker so that I can work on my current project task. Running devonfw docker I want to get the setup for docker runtime OR start the available installation OR being asked to update, if my existing rancher desktop installation is outdated.

Actual behavior

The devonfw ide does not detect the rancher desktop installation on windows and thus staring a setup resulting in an error (in my case) during installation, that "Rancher Desktop cannot be closed. Please close it manually and retry the installation".

Steps to reproduce (bug) / Use Case of feature request (enhancement)

  1. Have Rancher Desktop installed (in my case v1.2.1)
  2. Run devon docker on windows
  3. You get a setup dialog although an installation is already available (which you might forgot :D).

image

Related/Dependent Issues

Comments/Hints:

Affected version:

  • OS: Windows 10
  • Console: Git Bash
@maybeec maybeec added bug Something isn't working software software-package with 3rd party products labels Jan 3, 2023
@maybeec
Copy link
Member Author

maybeec commented Jan 3, 2023

I have to admit, that I installed Rancher Desktop manually some time ago, which then was not detected by devonfw ide.
I uninstalled it ran the installation again by devonfw ide, which is not working even accross devonfw ide installations on my machine. Nonetheless, that means, that we might need another detection mechanism to be clear, that we don't try to reinstall rancher or even worth install rancher over docker desktop

@hohwille
Copy link
Member

hohwille commented Jan 5, 2023

Checked here:

function doIsRancherInstalled() {

Long time ago we were simply checking if docker command is available. We should re-add this to doSetup so if I already have docker installed (whatever way Rancher-Desktop, Docker-Desktop or an alternative we might not even know of) all should be fine as installing yet another docker does not seem to make sense. So lets just read-add this check again.

@Ttring
Copy link
Contributor

Ttring commented Jan 11, 2023

I could not reproduce the problem as shown in the picture. What I did was: install Rancher Desktop 1.7.0 manually and then run docker desktop.

After Rancher Desktop installation via devon, the environment variables "$PATH" is not updated for git bash. If devon docker is rerun without resetting the terminal, the script will conclude that Rancher Desktop is still not installed ($PATH is used here for detection) and hence, another Rancher Desktop installation will start again. The initial idea was to refresh environment variable.
Calling devon after devon docker does not reset the $PATH.

# initialize environment...

Ttring added a commit to Ttring/ide that referenced this issue Jan 13, 2023
Ttring added a commit to Ttring/ide that referenced this issue Jan 13, 2023
@hohwille hohwille removed the Team_IDE label Jan 17, 2023
hohwille added a commit to Amueller36/ide that referenced this issue Feb 13, 2023
…llation-not-detected---ide-is-trying-to-install
hohwille added a commit to Amueller36/ide that referenced this issue Feb 13, 2023
…llation-not-detected---ide-is-trying-to-install
hohwille added a commit to Amueller36/ide that referenced this issue Feb 28, 2023
…llation-not-detected---ide-is-trying-to-install
@hohwille hohwille linked a pull request Feb 28, 2023 that will close this issue
hohwille added a commit to Amueller36/ide that referenced this issue Mar 2, 2023
…llation-not-detected---ide-is-trying-to-install
@hohwille hohwille added this to the release:2023.02.001 milestone Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working software software-package with 3rd party products
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants