Skip to content

[BUGFIX] State 'pkg.installed' was not found fix for openEuler and Issabel PBX#58973

Merged
Ch3LL merged 5 commits intosaltstack:masterfrom
ioncebotari:master
Mar 30, 2021
Merged

[BUGFIX] State 'pkg.installed' was not found fix for openEuler and Issabel PBX#58973
Ch3LL merged 5 commits intosaltstack:masterfrom
ioncebotari:master

Conversation

@ioncebotari
Copy link
Copy Markdown
Contributor

What does this PR do?

This PR adds support for package installation using Salt on openEuler and IssabelPBX.

What issues does this PR fix or reference?

Fixes: #57490

Previous Behavior

Installation would fail on both openEuler and Issabel PBX with the following error:
image

New Behavior

Installation successful on both distributions. Tested manually using Virtualbox VMs

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@ioncebotari ioncebotari requested a review from a team as a code owner November 18, 2020 12:23
@ioncebotari ioncebotari requested review from Ch3LL and removed request for a team November 18, 2020 12:23
@welcome
Copy link
Copy Markdown

welcome Bot commented Nov 18, 2020

Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey.
Please be sure to review our Code of Conduct. Also, check out some of our community resources including:

There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar.
If you have additional questions, email us at core@saltstack.com. We’re glad you’ve joined our community and look forward to doing awesome things with you!

Copy link
Copy Markdown
Contributor

@Ch3LL Ch3LL left a comment

Choose a reason for hiding this comment

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

can you add a changelog entry for this change?

@ioncebotari
Copy link
Copy Markdown
Contributor Author

@Ch3LL I've added the changelog. This is my first experience doing it, please review if done correctly. Thank you

@Ch3LL
Copy link
Copy Markdown
Contributor

Ch3LL commented Mar 29, 2021

@ioncebotari apologies I let this one go a little too long. I resolved a merge conflict and the tests are running and then i'll get this merged.

@Ch3LL Ch3LL added the Silicon v3004.0 Release code name label Mar 29, 2021
@Ch3LL Ch3LL merged commit a30ac55 into saltstack:master Mar 30, 2021
@welcome
Copy link
Copy Markdown

welcome Bot commented Mar 30, 2021

Congratulations on your first PR being merged! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Silicon v3004.0 Release code name

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] State 'pkg.installed' was not found

4 participants