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

Allow XVA builder to use templates already on XenCenter #67

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dongyuzheng
Copy link

Set source_template in config rather than source_path

Use case:
Build from Windows XVA, which can be very big. This can dramatically decrease build time.

@kirannhegde
Copy link

Hello @dongyuzheng Is there a way i could get this fix of yours? This is a very useful fix and i did like this to be merged into the master branch of packer-builder-xenserver. Do you have any other branch from where i could consume your fixes?

Please let me know.

Regards,
Kiran Hegde

@dongyuzheng
Copy link
Author

Hi @kirannhegde,

Here's the most up-to-date fork containing all of my changes + more:

https://github.com/ctlam/packer-builder-xenserver

You'll have to build it yourself.

Let me know if you run into any issues.

Thanks,
Gary

@kirannhegde
Copy link

Hello @dongyuzheng Thanks. Much appreciated. Does the above repo contain all the changes from the official packer-builder-xenserver repository?

Regards,
Kiran Hegde

@dongyuzheng
Copy link
Author

dongyuzheng commented Oct 20, 2018

@kirannhegde: Not that I know of. Feel free to fork it and pull in the changes yourself.

I don't understand why the authors never look at my PRs. Cough cough @rdobson

@kirannhegde
Copy link

Hello @dongyuzheng Great work, i would say on your part. I have now merged all your changes into a forked repository of the official repository. I will let you know how it goes.

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