Skip to content

Comments

Add flatpak module and state#67937

Closed
EllisGoodman wants to merge 3 commits intosaltstack:masterfrom
EllisGoodman:flatpak
Closed

Add flatpak module and state#67937
EllisGoodman wants to merge 3 commits intosaltstack:masterfrom
EllisGoodman:flatpak

Conversation

@EllisGoodman
Copy link

@EllisGoodman EllisGoodman commented Apr 2, 2025

What does this PR do?

Adds a new execution module and a new state module for managing flatpak packages and flatpak remotes.

Note that there was a previous PR for flatpak support: #51993. It was never merged into master, though. There was also #56907 but it had no outcome either. The code in this PR is based on #51993 with substantial changes and modernization.

What issues does this PR fix or reference?

None

Merge requirements satisfied?

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

Commits signed with GPG?

Yes

@welcome
Copy link

welcome bot commented Apr 2, 2025

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 saltproject.pdl@broadcom.com. We’re glad you’ve joined our community and look forward to doing awesome things with you!

@EllisGoodman EllisGoodman marked this pull request as ready for review April 2, 2025 11:53
@EllisGoodman EllisGoodman requested a review from a team as a code owner April 2, 2025 11:53
@twangboy
Copy link
Contributor

twangboy commented Apr 2, 2025

@EllisGoodman Starting in Salt 3008 and forward Salt has removed non-core Execution and State modules. Non-core modules are to be supported by the community. There is a GitHub org set up for this purpose: https://github.com/salt-extensions. This code would be better served as a salt-extension.

There is currently discussion and work being done on the salt-extensions Discord channel (https://discordapp.com/channels/1200072194781368340/1208165123240370197) to document and build the infrastructure for community-supported salt extensions. There is also a Salt-Extensions Working Group that takes place on the 1st and 3rd Thursday of every month to coordinate salt-extension efforts.

Please consider creating a salt-extension for this code and join the community in supporting the development and testing of salt-extensions.

@twangboy twangboy closed this Apr 2, 2025
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.

2 participants