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

AWS documentation doesn't mention need to install rust/cargo #204

Open
thvasilo opened this issue Apr 12, 2024 · 8 comments
Open

AWS documentation doesn't mention need to install rust/cargo #204

thvasilo opened this issue Apr 12, 2024 · 8 comments

Comments

@thvasilo
Copy link

Hello, I was following the instructions here to install the package: https://docs.aws.amazon.com/efs/latest/ug/installing-amazon-efs-utils.html

However at some I was getting the error about cargo not being installed (I'm installing this on an AWS Ubuntu 20.04 DLAMI), should probably update the instruction to install or include cargo installation in the build-dev.sh script.

@yambottle
Copy link

yambottle commented Apr 13, 2024

Experiencing the same issue...

Suspect the cause of 2.0 release on yesterday(2024/04/12)

The repo README updated:

sudo apt-get update
sudo apt-get -y install git binutils rustc cargo pkg-config libssl-dev
git clone https://github.com/aws/efs-utils
cd efs-utils
./build-deb.sh
sudo apt-get -y install ./build/amazon-efs-utils*deb

@augustovoigt
Copy link

augustovoigt commented Apr 13, 2024

Same issue here.
We are using Alma Linux 9.3 on our servers, and the efs-utils installation was working fine until yesterday.
With the 2.0 release the installation failed because of the missing dependencies (rust, cargo and openssl-devel).
The instructions on https://docs.aws.amazon.com/efs/latest/ug/installing-amazon-efs-utils.html need to be updated.

@thvasilo
Copy link
Author

The instructions in the readme on the repo work, just need to install a couple of extra dependencies, it's just the AWS docs that need to be updated.

@jakevose
Copy link

This compilation is adding several minutes to our builds after updating the dependencies. Will be locking the version to 1.36.0 for the immediate future.

@major
Copy link

major commented Apr 15, 2024

On the Fedora side, we might be stuck on 1.36.0 for a while as well. 🤔

@RyanStan
Copy link
Member

We're updating the instructions on https://docs.aws.amazon.com/efs/latest/ug/installing-amazon-efs-utils.html. These will be out shortly. For now, the updated installation instructions are available in the readme.

@major Can you open up a new issue where we can track the problems with distributing this release on Fedora?

@jakevose We can look into ways in which we can decrease the build time. Can you open a new issue and provide some details about how your builds work? E.g. spinning up new hosts in an autoscaling group and building efs-utils via the user data script? We can help come up with a temporary solution that mitigates the longer build time problem.

@justin-octo
Copy link

This has also broken things for CentOS 7

@aiqc
Copy link

aiqc commented Aug 4, 2024

Boo. You know developers are building ubuntu images on old stable releases that don't have Rust. Why add this dependency?


Putting this here for future people debugging via google search:

When installing efs-utils via docker RUN ./build-deb.sh;:

+ cargo build --release --manifest-path /my-path/efs-utils/src/proxy/Cargo.toml
./build-deb.sh: 33: cargo: not found

Within ./build-deb.sh script:

cargo build --release --manifest-path ${BASE_DIR}/src/proxy/Cargo.toml

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

No branches or pull requests

8 participants