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

Mac Packaging / Notarization #8878

Merged
merged 11 commits into from
Mar 5, 2021
Merged

Mac Packaging / Notarization #8878

merged 11 commits into from
Mar 5, 2021

Conversation

ivorybilled
Copy link
Contributor

@ivorybilled ivorybilled commented Feb 18, 2021

This update will run a script on 'release' deployments that will sign and notarize our Mac OS packages. Notarization requires the .app format and the binary will be distributed within the Resources of the .app, and the .app will be distributed within a DMG image. Note that those who need direct access to the Telegraf binary can still go into the .app 'folder' and retrieve it as normal.

What this PR accomplishes in a practical manner is that users can now double-click/open the Telegraf app directly after downloading it, and mac OS will not block them from opening it. Users can also run the Telegraf.app through the command line, as follows:

open -a Telegraf.app

you can also pass any Telegraf arguments as follows:

open -a Telegraf.app --args (arguments)

resolves #8580

Copy link
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

🤝 ✅ CLA has been signed. Thank you!

@ivorybilled ivorybilled changed the title Mac Packaging / Notarization (in progress, draft) Mac Packaging / Notarization Feb 19, 2021
@ivorybilled ivorybilled marked this pull request as ready for review February 19, 2021 16:47
@ivorybilled
Copy link
Contributor Author

ivorybilled commented Feb 19, 2021

EDIT: .tar.gz package will remain included and the signed/notarized package will be another option that comes with it.

@Hipska
Copy link
Contributor

Hipska commented Feb 24, 2021

Not sure, but I think it might still be needed for homebrew: https://formulae.brew.sh/formula/telegraf

I guess brew can also unpack from the dmg..

@ivorybilled ivorybilled merged commit b6f043c into master Mar 5, 2021
@ivorybilled ivorybilled deleted the macNotarization branch March 5, 2021 15:14
ssoroka pushed a commit that referenced this pull request Mar 10, 2021
* initial commit

* Updated config.yml

* Update mac-signing.sh

* Updated config.yml

* Updated config.yml

* Keep the .tar.gz artifact along with the signed and notarized DMG

* change to test temporarily

* for testing

* Updated config.yml

* Update config.yml

* Update config.yml

(cherry picked from commit b6f043c)
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
* initial commit

* Updated config.yml

* Update mac-signing.sh

* Updated config.yml

* Updated config.yml

* Keep the .tar.gz artifact along with the signed and notarized DMG

* change to test temporarily

* for testing

* Updated config.yml

* Update config.yml

* Update config.yml

(cherry picked from commit b6f043c)
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.

Telegraf for macOS should be signed
3 participants