-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Add IPSW plugin #12873
Comments
The integration has been submitted in the past, and is listed here: https://developer.hashicorp.com/packer/integrations?components=data-source&filter=ipsw But the docs link is broken, likely due to the new way that the docs are published. |
https://github.com/torarnv/packer-plugin-ipsw/actions/runs/8204818234/job/22440273928 Shows I assume that's because the plugin hasn't been added to the docs yet? |
I believe you helped me with this in the past @lbajolet-hashicorp See #12541 |
Hi @torarnv, It seems we have forgotten to include you in our round for integrations, apologies about that! Good to see you've updated the docs structure, with the current docs I believe we'll be able to add them to the internal integrations (or update what exists at least), I'll see to this right now. Thanks for signalling this to us! |
Thanks @lbajolet-hashicorp <3 |
Hi @torarnv, You can try to run the integration action once again, on our side we did the updates to the documentation, this should work now |
Thanks @lbajolet-hashicorp, the integration now worked. Do I need to do anything else? I see https://developer.hashicorp.com/packer/plugins/datasources/ipsw is still 404, but perhaps it takes a while to go live? Does the integration automatically publish the new docs, or does it trigger a PR or something similar on your side? |
Hi again! I can confirm your plugin is available, not on plugins anymore though (https://developer.hashicorp.com/packer/integrations/torarnv/ipsw), I'm surprised the link doesn't work anymore, there should be some redirect in place to avoid this problem, that's a discussion I need to have internally. |
Thanks @lbajolet-hashicorp , no rush! 😄 Have a good weekend! |
I'm looking at perhaps renaming this plugin to
|
Hi @torarnv, No objection to you renaming the project, I'd think Github does redirect so there's a good chance users may be able to download it this way, that being said, I'm not completely certain the installation/loading logic will work if the source has one name, and the binary another (i.e. in this case the plugin source would be If you plan on using In any case, the fact that the main docs and the datasources aren't visible most likely point to a problem with our configuration, so I'll take a look this way, and let you know if there's something to change in your repo, this fix will likely be applicable to both plugins. |
Hi again @torarnv, I just opened torarnv/packer-plugin-ipsw#1, this removes an invalid flag from the metadata. This invalid flag prevented the ingestion of the payload sent by the publication earlier, hence why we're seeing an empty documentation for the plugin at the moment. Once merged, you'll need to run the ingestion action once more (no need to bump the release number, it can still be 0.0.12), this will re-trigger the ingestion, and provided there are no other errors, the docs should get updated. I've also opened a PR internally to enable the redirection from the old URIs, this way links like the one you shared above should work again. |
Ah, awesome, thank you so much @lbajolet-hashicorp ! |
Looks like the docs are now live at https://developer.hashicorp.com/packer/integrations/torarnv/ipsw Thanks again @lbajolet-hashicorp ❤️ As for the invalid flag, I initially followed this guide: Which has the same problematic line in https://github.com/hashicorp/packer-plugin-scaffolding/blob/main/.web-docs/metadata.hcl#L11, so perhaps that needs to be updated? |
Very true, the scaffolding HCL file is indeed wrong, thanks for the notice! I'll fix that immediately. |
Oh also FYI the old URIs should work too in addition to the new one, I fixed the redirections for your plugin, so now the old links will redirect you to the new docs |
Awesome!! I think we can close this one then, thanks for all your help, it makes working with Packer a joy <3 |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
The IPSW data-source plugin can be used with HashiCorp Packer to fetch information about Apple device firmware, for example for building macOS images with Tart.
https://github.com/torarnv/packer-plugin-ipsw
Integration Tier
Checklist
metadata.hcl
file in plugin repository.docs
directory..web-docs
directory ready for publishing to the integrations portal.The text was updated successfully, but these errors were encountered: