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

more accurate handle for .zip kapitan-dependency #912

Merged
merged 5 commits into from
Jan 13, 2023

Conversation

metacoma
Copy link
Contributor

@metacoma metacoma commented Dec 9, 2022

I have the following dependency in kapitan:

...
  kapitan:
    dependencies:
      - type: http
        output_path: ./freeplane/freeplane.zip
        source: https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download
        unpack: true
...

without this patch, the kapitan compile --fetch leads to

$ kapitan compile --fetch 
Dependency https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download: fetching now                        
Dependency https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download: successfully fetched                
Dependency https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download: Content-Type application/octet-stream is not supported for unpack. Ignoring save 
...

with patch:

$ kapitan compile --fetch
Dependency https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download: fetching now
Dependency https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download: successfully fetched
Dependency https://sourceforge.net/projects/freeplane/files/freeplane%20stable/freeplane_bin-1.10.6.zip/download: extracted to freeplane/freeplane.zip
...

@ademariag
Copy link
Contributor

Thank you @metacoma ! We will review it.

Feel free to join out #kapitan on kubernetes.slack if you want to chat. get invited

@ademariag ademariag added the enhancement enhancement to an existing feature label Dec 9, 2022
@metacoma metacoma requested a review from ademariag December 9, 2022 15:00
metacoma added a commit to metacoma/mindwm that referenced this pull request Dec 9, 2022
@metacoma metacoma requested review from ramaro and removed request for ademariag December 10, 2022 12:55
@metacoma
Copy link
Contributor Author

just friendly ping

metacoma added a commit to metacoma/mindwm that referenced this pull request Dec 21, 2022
@metacoma metacoma requested review from ademariag and removed request for ramaro December 21, 2022 06:20
@ademariag
Copy link
Contributor

Hello @metacoma, we'll be merging it soon, thank you for your help with this

@ramaro ramaro enabled auto-merge (squash) January 13, 2023 09:45
@ramaro ramaro disabled auto-merge January 13, 2023 09:48
@ramaro ramaro enabled auto-merge (squash) January 13, 2023 09:48
@ramaro ramaro disabled auto-merge January 13, 2023 09:48
@ramaro ramaro merged commit 161d59c into kapicorp:master Jan 13, 2023
metacoma added a commit to metacoma/mindwm that referenced this pull request Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement enhancement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants