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.

Sign up for GitHub

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

Jump to bottom

fix(install): Fix Debian post installation script when awscli2 is already installed or the installation failed previously #6276

Merged
merged 1 commit into from
Aug 20, 2024
Merged

fix(install): Fix Debian post installation script when awscli2 is already installed or the installation failed previously #6276

merged 1 commit into from
Aug 20, 2024

Conversation

ashleykleynhans
Copy link
Contributor

More fixes to the post installation script for the Debian package.

  • Only installs the specified version of awscliv2 if it is not already installed.
  • Cleans up the ./aws before unzipping awscliv2.zip if the installation failed previously and the ./aws directory already exists.
  • Use curl -s for downloads for cleaner output.

@ashleykleynhans
fix(install): Fix Debian post installation script when awscli2 is alr…
1ed3e9f 
…eady installed or the installation failed previously
@ashleykleynhans ashleykleynhans mentioned this pull request Aug 20, 2024
Closed
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for a merge label Aug 20, 2024
@mergify mergify bot added the auto merged Merged automatically by a bot label Aug 20, 2024
@mergify mergify bot merged commit 35a30d3 into spinnaker:master Aug 20, 2024
23 checks passed
@dbyron-sf
Copy link
Contributor

@Mergifyio backport release-1.35.x

@mergify Mergify
Copy link
Contributor

mergify bot commented Aug 20, 2024 •
edited
Loading

backport release-1.35.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Aug 20, 2024
@ashleykleynhans @mergify
fix(install): Fix Debian post installation script when awscli2 is alr…
3604807 
…eady installed or the installation failed previously (#6276)

(cherry picked from commit 35a30d3)
@mergify mergify bot mentioned this pull request Aug 20, 2024
Merged
mergify bot added a commit that referenced this pull request Aug 20, 2024
@mergify @ashleykleynhans
fix(install): Fix Debian post installation script when awscli2 is alr…
f4dbb7c 
…eady installed or the installation failed previously (#6276) (#6277)

(cherry picked from commit 35a30d3)

Co-authored-by: Ashley Kleynhans <ashleykleynhans@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbyron-sf dbyron-sf dbyron-sf approved these changes

Assignees
No one assigned
Labels
auto merged Merged automatically by a bot ready to merge Approved and ready for a merge target-release/1.36
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ashleykleynhans @dbyron-sf @spinnakerbot