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

[master] Add 'ppc64le' as a valid RPM package architecture #55150

Merged
merged 4 commits into from
Nov 13, 2019

Conversation

meaksh
Copy link
Contributor

@meaksh meaksh commented Oct 29, 2019

What does this PR do?

This PR ports #53237 to master branch and also adds unit test for it.

Tests written?

Yes

Commits signed with GPG?

Yes

@waynew
Copy link
Contributor

waynew commented Nov 12, 2019

@meaksh thanks for porting this! This is one of those places our 3-review exception exists for, as the unit test isn't terribly meaningful. If you want to remove the test here, we can get the three votes & merged.

@waynew
Copy link
Contributor

waynew commented Nov 13, 2019

I went ahead and removed the extra tests, we should be able to get this approved and merged today.

@meaksh
Copy link
Contributor Author

meaksh commented Nov 13, 2019

@waynew thanks for pushing this ahead! 👍

@garethgreenaway garethgreenaway merged commit 920c7d9 into saltstack:master Nov 13, 2019
@meaksh meaksh deleted the master-bp-53237 branch November 14, 2019 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants