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

Installer Fails When Removing Docker-Compose #1458

Open
max-wyze opened this issue Feb 11, 2025 · 4 comments
Open

Installer Fails When Removing Docker-Compose #1458

max-wyze opened this issue Feb 11, 2025 · 4 comments
Assignees
Labels
bug Something isn't working Installation Installation issues

Comments

@max-wyze
Copy link

max-wyze commented Feb 11, 2025

Bug Report: Installer Fails When Removing Docker-Compose

Describe the Bug

When installing the new version of EMBA, the installer detects an existing docker-compose installation and recommends that it be removed. However, if docker-compose is removed before running the installer, the installation fails with errors.

Error when docker-compose is removed:

Activating Python Environment - ./external/emba_venv

WARNING: Old docker-compose installation found
It is recommend to remove the current installation and restart the EMBA installation afterwards!

After removing docker-compose and running the installer, the installation fails. To work around the issue, I reinstalled docker-compose and then ran the installer successfully.

Installer output after reinstalling docker-compose:

cd /opt
git clone https://github.com/e-m-b-a/emba.git
cd emba
sudo ./installer.sh -d
linux-modules-extra-6.8.0-1018-aws
Description: Linux kernel extra modules for version 6.8.0 on 64 bit x86 SMP
Download-Size: 70.8 MB
linux-modules-extra-6.8.0-1018-aws won't be updated.
[*] Testing for unattended update settings

Automatic updates are enabled - this could result in unexpected behavior during installation!
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
0 upgraded, 0 newly installed, 0 to remove and 5 not upgraded.

I01_default_apps
=================================================================

I13_disasm
=================================================================

[+] I05_emba_docker_image_dl
=================================================================

embeddedanalyzer/emba docker image
Description: EMBA docker images used for firmware analysis.
Download-Size : 5610 MB

docker.io and the EMBA docker image (if not already on the system) will be downloaded and installed!
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
0 upgraded, 0 newly installed, 0 to remove and 5 not upgraded.
730
EMBA docker image will be downloaded.
CONTAINER VARIABLE SET TO embeddedanalyzer/emba:1.5.1b
1.5.1b: Pulling from embeddedanalyzer/emba
Digest: sha256:30e52933857ec4d13e98a066e23bbe91a46ce5450368429cdd846ea68a114a9e
Status: Image is up to date for embeddedanalyzer/emba:1.5.1b
**docker.io/embeddedanalyzer/emba:1.5.1b
./installer/I05_emba_docker_image_dl.sh: line 57: up: command not found
------------------------------------------------------------------------------------------------
Error detected - status code 127
Command:  echo -e "Command:  ${ORANGE}${BASH_COMMAND}${NC}"
Location:  ./installer/I05_emba_docker_image_dl.sh, line 57
Stack Trace:
    [1] I05_emba_docker_image_dl(): ./installer/I05_emba_docker_image_dl.sh, line 57 -> I05_emba_docker_image_dl
    [2] main(): ./installer.sh, line 369 -> main -d

To Reproduce
1. Run the EMBA installer (either in dev-mode or default mode).
2. Remove the existing docker-compose installation when prompted.
3. Execute the installer again:

apt remove docker-compose
sudo ./installer.sh -d

Observe the failure.

Note: Reinstalling docker-compose before running the installer allows the installation to complete successfully.

Expected Behavior

The installer should either:
• Handle the removal of docker-compose gracefully without failing, or
• Provide a clear, actionable error message if docker-compose must remain installed during the installation process.

Desktop
• OS:#19~22.04.1-Ubuntu SMP Wed Oct 9 16:48:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
• EMBA version: current master branch
• Installation method: Default installation using the up-to-date docker image

Priority Issue

Are you already a Sponsor? (Y/N)
No

Copy link

Thank you for contributing an issue!

Welcome to the EMBA firmware analysis community!

We are glad you are here and appreciate your contribution. Please keep in mind our contributing guidelines here and here.
Also, please check existing open issues and consider to open a discussion in the dedicated discussion area.
Additionally, we have collected a lot of details around EMBA, the installation and the usage of EMBA in our Wiki.

If you like EMBA you have the chance to support us by becoming a Sponsor or buying some beer here.

To show your love for EMBA with nice shirts or other merch you can check our Spreadshop.

This is an automatic message. Allow for time for the EMBA community to be able to read the issue and comment on it.

@m-1-k-3 m-1-k-3 added bug Something isn't working Installation Installation issues labels Feb 11, 2025
@m-1-k-3
Copy link
Member

m-1-k-3 commented Feb 11, 2025

looks as there is no docker installation performed.

@max-wyze
Copy link
Author

the docker-compose is recommended to be removed but not installed by the installer.

@m-1-k-3
Copy link
Member

m-1-k-3 commented Feb 11, 2025

looks as you uninstalled docker-compose but docker by itself is available and so the installer was not able to handle this situation here

if ! command -v docker > /dev/null || ! command -v docker compose > /dev/null ; then

We will improve this check

@m-1-k-3 m-1-k-3 self-assigned this Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Installation Installation issues
Projects
None yet
Development

No branches or pull requests

2 participants