-
Notifications
You must be signed in to change notification settings - Fork 41
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
feat: support use of pxeurls in machine provisioning #143
Conversation
Welcome @rsmitty! |
35849c3
to
65caefb
Compare
A few things.
|
This PR will add a new field to packetmachine that allows for the passing of a pxe url string. If present, we check that the OS is also set to "custom_ipxe" so that we can fail early. Signed-off-by: Spencer Smith <robertspencersmith@gmail.com>
@deitch I ran I'll defer to what you all want on using the OS field vs. having a distinct PXEUrl field. But as a data point, they're two unique fields in packngo, as well as in the UI: |
I was afraid of that. If they’re two distinct fields in the core api and packngo, then let’s leave it as is. |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gianarb, rsmitty The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR will add a new field to packetmachine that allows for the
passing of a pxe url string. If present, we check that the OS is also
set to "custom_ipxe" so that we can fail early.
Will close #131
Related issue #118
Signed-off-by: Spencer Smith robertspencersmith@gmail.com