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

Support changes of IP address on reboot of VM #61

Open
Erik142 opened this issue Apr 8, 2023 · 5 comments · May be fixed by #62
Open

Support changes of IP address on reboot of VM #61

Erik142 opened this issue Apr 8, 2023 · 5 comments · May be fixed by #62

Comments

@Erik142
Copy link

Erik142 commented Apr 8, 2023

When using packer in combination with this plugin, to create images of certain operating systems such as Debian, I am having the following issue:

I am using the Debian 11.6.0 netinst iso to install Debian via network mirrors. This means that the VM retrieves an IP address very early during the installation process, which this packer plugin notices. However, when the VM restarts after a successful installation of the OS, the IP address for the VM tends to change, since there is a change in the DHCP client ID. I am inclined to believe that Debian isn't the only OS that would have this issue.

Therefore, I think it would be beneficial for this plugin to be able to react to changes of the IP address for the VM. If the HTTP server is used to detect the IP address for the VM, it would be up to the user to perform an HTTP request to the HTTP server when the IP address has changed. Otherwise, the plugin should continuously check whether the IP address has changed by means of the xe-guest-utilities package on the VM.

@Erik142 Erik142 linked a pull request Apr 10, 2023 that will close this issue
@ddelnano
Copy link
Owner

@Erik142 thanks for the contribution! I'm traveling through April 23rd, but I'm hoping to have time to review your PR over the next week.

@ddelnano
Copy link
Owner

ddelnano commented Jul 6, 2023

@Erik142 apologies for the delay on this. I'm hoping to get to it in the next week.

@hoerup
Copy link

hoerup commented Nov 8, 2023

@ddelnano any progress ?

@Erik142
Copy link
Author

Erik142 commented Jan 25, 2024

@ddelnano just giving a friendly nudge 😄 Is there anything that I can provide in order to ease the review of #62 ? I recently force pushed my commit in the PR so that it builds again on the latest main branch. Is there anything that is missing from my side? Thanks! 👍🏻

@mire3212
Copy link

mire3212 commented Mar 9, 2024

I'm also experiencing this issue on Debian 12.5 netinst ISO

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 a pull request may close this issue.

4 participants