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

fix(crater): install awscli from snap #607

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

MarcoIeni
Copy link
Member

@MarcoIeni MarcoIeni commented Oct 8, 2024

Ansible in master fails on crater (which is now on ubuntu 24).
This is because awscli is not installed anymore in apt.
With this PR we install aws from snap instead of apt.

How to deploy this:

  • uninstall awscli with ssh
  • run ansible apply

A newer version of the aws cli will be pulled so maybe there are some breaking changes that we need to fix after. (this should work)

@MarcoIeni
Copy link
Member Author

Finally I found the queue empty this morning.

image

I ran

sudo apt-get remove awscli

but ubuntu couldn't locate the awscli package.

So I manually deleted the aws binary with sudo rm /usr/local/bin/aws

Now I'm running ansible apply.

@MarcoIeni
Copy link
Member Author

Ansible finished:

crater.infra.rust-lang.org : ok=128 changed=5 unreachable=0 failed=0 skipped=90 rescued=0 ignored=1

So I'm merging this

@MarcoIeni MarcoIeni force-pushed the fix-crater-install-awscli-from-snap branch from 95c0ec2 to 1d28cda Compare November 13, 2024 09:50
@MarcoIeni MarcoIeni force-pushed the fix-crater-install-awscli-from-snap branch from 1d28cda to 4c242be Compare November 13, 2024 09:51
@MarcoIeni MarcoIeni enabled auto-merge November 13, 2024 09:51
@MarcoIeni MarcoIeni merged commit 5e26b96 into master Nov 13, 2024
3 checks passed
@MarcoIeni MarcoIeni deleted the fix-crater-install-awscli-from-snap branch November 13, 2024 09:52
@MarcoIeni
Copy link
Member Author

MarcoIeni commented Nov 13, 2024

image Crater still works 🥳

rust-lang/rust#131362 (comment)

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.

2 participants