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

Fix batch install on FreeBSD using pkgng #48730

Merged
merged 4 commits into from
Aug 2, 2018

Conversation

fzipi
Copy link
Contributor

@fzipi fzipi commented Jul 24, 2018

For fully automated batch installing in FreeBSD you need to pass BATCH=true.

What does this PR do?

Add a new option to pkg install, batch=True.

What issues does this PR fix or reference?

#27056, #29907, #29909

Previous Behavior

When installing postfix in FreeBSD, it keeps asking for confirmation to change mailer.conf. When using the postfix formula, it does not work because installation is not automated (you can break things in production, etc.).

New Behavior

Installation is succesful. You may require to configure additional steps, but at least the package is there.

Tests written?

No

Commits signed with GPG?

Yes

For fully automated batch installing in FreeBSD you need to pass BATCH=true.
@ghost ghost self-requested a review July 24, 2018 08:56
@rallytime
Copy link
Contributor

Welcome @fzipi! Thank you for submitting this. This looks good to me, but I'm not super familiar with FreeBSD.

@abednarik You've been involved with FreeBSD support - can you review this?

@abednarik
Copy link
Contributor

abednarik commented Jul 29, 2018 via email

@abednarik
Copy link
Contributor

Hi @rallytime

This looks good to me.

Thanks.

@rallytime rallytime added the bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch label Jul 31, 2018
@rallytime rallytime merged commit 1520bd4 into saltstack:develop Aug 2, 2018
@rallytime rallytime added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels Aug 2, 2018
rallytime pushed a commit that referenced this pull request Aug 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants