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(manager): fix awscli installation on Ubuntu24 nodes #9154

Merged
merged 1 commit into from
Nov 10, 2024

Conversation

mikliapko
Copy link
Contributor

@mikliapko mikliapko commented Nov 7, 2024

Closes scylladb/scylla-manager#4105

Changes:

  • break-system-packages allows pip to modify an externally-manager Python installation while installing package on Ubuntu24.
  • removed version specifying for the package since of compatibility issues with system-wide installation.

Testing

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

@mikliapko mikliapko self-assigned this Nov 7, 2024
@mikliapko mikliapko added the backport/none Backport is not required label Nov 7, 2024
@mikliapko mikliapko force-pushed the manager-fix-awscli-pip-install branch 2 times, most recently from a51c39b to 9898124 Compare November 7, 2024 14:15
@mikliapko mikliapko marked this pull request as ready for review November 7, 2024 15:55
@mikliapko mikliapko changed the title fix(manager): use --break-system-packages to install pkg on Ubuntu24 fix(manager): fix awscli installation on Ubuntu24 nodes Nov 7, 2024
@mikliapko mikliapko requested review from a team and grzywin November 7, 2024 15:56
mgmt_cli_test.py Outdated
apt update
apt install -y python3-pip
pip install awscli==1.18.140
pip_break_system_packages = "--break-system-packages" if node.distro.is_ubuntu24 else ""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more option is to use the environment variable for it, and you don't need a condition

Copy link
Contributor Author

@mikliapko mikliapko Nov 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree, looks way better without this condition, fixed.
Retesting: https://jenkins.scylladb.com/job/scylla-staging/job/mikita/job/manager-master/job/ubuntu22-sanity-test/38/ (PASSED)

@mikliapko mikliapko force-pushed the manager-fix-awscli-pip-install branch from 9898124 to c9f2f5a Compare November 8, 2024 09:24
Changes:
- break-system-packages allows pip to modify an externally-manager
Python installation while installing package on Ubuntu24.
- removed version specifying for the package since of compatibility
issues with system-wide installation.
@mikliapko mikliapko force-pushed the manager-fix-awscli-pip-install branch from c9f2f5a to 7ec15dc Compare November 8, 2024 09:25
@mikliapko mikliapko requested a review from fruch November 8, 2024 10:29
Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fruch fruch merged commit 93db67c into scylladb:master Nov 10, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/none Backport is not required promoted-to-master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SCT fails installing awscli python package to Scylla OS 6.1 and 6.2 nodes
3 participants