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

feat(ubuntu): Install brew package manager on x86_64 #1853

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cdalvaro
Copy link

@cdalvaro cdalvaro commented Sep 25, 2024

Add Homebrew package manager to Ubuntu (x86_64) machines.

This package is needed to allow brew testability on Linux machines after having extended the mac_brew_pkg.py module to allow Linux in saltstack/salt#66609

twangboy
twangboy previously approved these changes Sep 30, 2024
@cdalvaro
Copy link
Author

cdalvaro commented Oct 1, 2024

I'm sorry, @twangboy. I didn't escape a $ sign properly. I think now is right.

twangboy
twangboy previously approved these changes Oct 1, 2024
@cdalvaro
Copy link
Author

cdalvaro commented Oct 2, 2024

It seems installation has failed:

[14:11:49] ==> amazon-ebs.image: Provisioning with shell script: /tmp/packer-shell1663122472
           ==> amazon-ebs.image: + [[ x86_64 == x86_64 ]]
           ==> amazon-ebs.image: /tmp/script_4580.sh: 2: [[: not found

I'll try a different approach.

@cdalvaro cdalvaro force-pushed the feature/ubuntu/add_brew_package_manager branch from d335d6e to 386b5f3 Compare October 2, 2024 09:31
twangboy
twangboy previously approved these changes Oct 2, 2024
@cdalvaro
Copy link
Author

cdalvaro commented Oct 3, 2024

I'm not sure if there's a more appropriate user than ssh_username for installing brew.

@twangboy
Copy link
Contributor

twangboy commented Oct 8, 2024

@cdalvaro I wanted to give you a heads up on this. We're being asked to make a major migration of our CICD off of AWS by the end of the month. This is probably going to affect our testing for awhile till we get it all worked out. So, we're going to hold off on this until things settle down.

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