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

Use correct rpm URL for aarch64 #196

Merged
merged 3 commits into from Jul 10, 2023
Merged

Use correct rpm URL for aarch64 #196

merged 3 commits into from Jul 10, 2023

Conversation

jeffgus
Copy link
Contributor

@jeffgus jeffgus commented Jun 15, 2023

Pull Request Checklist

Is this in reference to an existing issue?

No

General

  • Remove any versioning you did yourself if applicable

  • Update Changelog following the conventions laid out on Keep A Changelog with all new changes under ## [Unreleased] and using a ### Added, Fixed, Changed, or Breaking Change sub-header.

  • Update README with any necessary changes

  • RuboCop passes

  • Foodcritic passes

  • Existing tests pass

Purpose

Running the current recipe on aarch64 would fail when installing the rpm. This change updates the URL so when the client arch is aarch64, the rpm URL for aarch64 is used.

Known Compatibility Issues

@jeffgus jeffgus requested a review from a team as a code owner June 15, 2023 20:53
@jeffgus
Copy link
Contributor Author

jeffgus commented Jun 20, 2023

Accidentally clicked request review after the review was completed. Sorry about that.

@sumo-drosiek
Copy link
Contributor

@jeffgus Could you add changelog entry?

@jeffgus
Copy link
Contributor Author

jeffgus commented Jul 7, 2023

@jeffgus Could you add changelog entry?

I added a changelog entry in commit b0a1202.

CHANGELOG.md Outdated Show resolved Hide resolved
@sumo-drosiek sumo-drosiek merged commit 71b86cf into SumoLogic:main Jul 10, 2023
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.

None yet

2 participants