This came out of testing the no-puppetmaster patch. If you use tokens (so no need to supply an accurate IP to Foreman) and don't use Puppet, Foreman will never get the real IP of the system (as facts will never be uploaded).

Since the Host's IP can be got from the request (either request.ip or an embedded param) we could add a setting to allow Foreman to directly update the Host IP once the machine is built (or maybe tie it to the token setting?).