CLUSTER_UUID is in cluster celery config, but tftpd needs it
Bug #1086239 reported by
Jeroen T. Vermeulen
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
High
|
Jeroen T. Vermeulen | ||
| 1.2 |
Fix Released
|
High
|
Jeroen T. Vermeulen | ||
Bug Description
The tftps now needs CLUSTER_UUID in order to pass it to the pxeconfig API call. But we don't want to import the celery config module (with its side effect of reading configuration) into the tftp process.
So the CLUSTER_UUID setting would more properly fit into maas_cluster.conf.
Related branches
lp:~jtv/maas/pkg-bug-1086239
- Julian Edwards (community): Approve
-
Diff: 112 lines (+63/-10)3 files modifieddebian/changelog (+1/-1)
debian/maas-cluster-controller.maas-pserv.upstart (+16/-2)
debian/maas-cluster-controller.postinst (+46/-7)
lp:~jtv/maas/bug-1086239
- Gavin Panella (community): Approve
-
Diff: 393 lines (+111/-68)14 files modifiedetc/democeleryconfig_cluster.py (+0/-4)
services/cluster-worker/run (+5/-0)
services/pserv/run (+5/-0)
src/provisioningserver/auth.py (+0/-8)
src/provisioningserver/boot_images.py (+5/-5)
src/provisioningserver/cluster_config.py (+44/-0)
src/provisioningserver/dhcp/leases.py (+2/-2)
src/provisioningserver/dhcp/tests/test_leases.py (+0/-9)
src/provisioningserver/start_cluster_controller.py (+1/-8)
src/provisioningserver/tags.py (+2/-6)
src/provisioningserver/tests/test_auth.py (+0/-12)
src/provisioningserver/tests/test_boot_images.py (+0/-10)
src/provisioningserver/tests/test_cluster_config.py (+46/-0)
src/provisioningserver/tftp.py (+1/-4)
lp:~jtv/maas/q-pkg-bug-1086239
- Scott Moser (community): Needs Fixing
- Raphaël Badin (community): Approve
-
Diff: 125 lines (+69/-10)3 files modifieddebian/changelog (+7/-1)
debian/maas-cluster-controller.maas-pserv.upstart (+16/-2)
debian/maas-cluster-controller.postinst (+46/-7)
lp:~jtv/maas/1.2-bug-1086239
- Jeroen T. Vermeulen (community): Approve
-
Diff: 393 lines (+111/-68)14 files modifiedetc/democeleryconfig_cluster.py (+0/-4)
services/cluster-worker/run (+5/-0)
services/pserv/run (+5/-0)
src/provisioningserver/auth.py (+0/-8)
src/provisioningserver/boot_images.py (+5/-5)
src/provisioningserver/cluster_config.py (+44/-0)
src/provisioningserver/dhcp/leases.py (+2/-2)
src/provisioningserver/dhcp/tests/test_leases.py (+0/-9)
src/provisioningserver/start_cluster_controller.py (+1/-8)
src/provisioningserver/tags.py (+2/-6)
src/provisioningserver/tests/test_auth.py (+0/-12)
src/provisioningserver/tests/test_boot_images.py (+0/-10)
src/provisioningserver/tests/test_cluster_config.py (+46/-0)
src/provisioningserver/tftp.py (+1/-4)
lp:~jtv/maas/p-sru-pkg-bug-1086239
- Raphaël Badin (community): Approve
-
Diff: 125 lines (+69/-10)3 files modifieddebian/changelog (+7/-1)
debian/maas-cluster-controller.maas-pserv.upstart (+16/-2)
debian/maas-cluster-controller.postinst (+46/-7)
| Changed in maas: | |
| assignee: | nobody → Jeroen T. Vermeulen (jtv) |
| status: | Triaged → In Progress |
| Changed in maas: | |
| status: | In Progress → Fix Committed |
| Changed in maas: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
